<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On 10 July 2013 16:20, Dmitry Torokhov <span dir="ltr"><<a href="mailto:dmitry.torokhov@gmail.com" target="_blank">dmitry.torokhov@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wednesday, July 10, 2013 10:50:26 PM Grant Likely wrote:<br>
> On Wed, Jul 10, 2013 at 5:52 PM, Dmitry Torokhov<br>
><br>
> <<a href="mailto:dmitry.torokhov@gmail.com">dmitry.torokhov@gmail.com</a>> wrote:<br>
> > On Wed, Jul 10, 2013 at 04:14:57PM +0100, Grant Likely wrote:<br>
> >> On Fri, 28 Jun 2013 07:19:06 -0600, Mathieu Poirier<br>
<<a href="mailto:mathieu.poirier@linaro.org">mathieu.poirier@linaro.org</a>> wrote:<br>
> >> > On 13-06-28 12:09 AM, Dmitry Torokhov wrote:<br>
> >> > >>>> I do not agree.  We want the binding to be generic and not tied<br>
> >> > >>>> specifically to the keyreset functionality.  As such<br>
> >> > >>>> 'input-keyset' or<br>
> >> > >>>> 'input-keychord' are more appropriate.<br>
> >> > >>><br>
> >> > >>> The binding is defined specifically for sysrq and specifically to<br>
> >> > >>> perform reset action.<br>
> >> > >><br>
> >> > >> Yes for now but as the examples in the binding show, it is easy to<br>
> >> > >> envision how other drivers could use it.<br>
> >> > ><br>
> >> > > I think you over-complicate things here. Unlike matrix-keypad<br>
> >> > > binding,<br>
> >> > > where you have a common parsing code, here we have an individual<br>
> >> > > driver.<br>
> >> > > I really do not see anyone else using such sequences or chords as<br>
> >> > > such<br>
> >> > > processing should be done in userspace. Sysrq is quite an exception.<br>
> >> ><br>
> >> > To be honest I don't have a very strong opinion on the binding.  I made<br>
> >> > it as generic as possible on the guidance of the DT people.  Let's see<br>
> >> > what they think of it.<br>
> >><br>
> >> Hi Mathieu,<br>
> >><br>
> >> As per our conversation just now at Connect, the binding should probably<br>
> >> look like this:<br>
> >><br>
> >> Sysrq keyset binding:<br>
> >><br>
> >> The /chosen node can contain a linux,input-keyset-sysrq child node to<br>
> >> define a set of keys that will generate a sysrq when pressed together.<br>
> ><br>
> > Hmm, we would have only one such node, /sysrq, or /linux,sysrq,<br>
> > whatever. The sysrq setting is system-wide and applicable to all<br>
> > devices. Given that it is used only on mobile, where there not that<br>
> > many input devices (a few keys and touchscreen) I do not believe we<br>
> > should consider adding per-device settings.<br>
><br>
> It's in /chosen, that isn't per-device.<br>
><br>
> >> Required properties:<br>
> >> keyset: array of keycodes<br>
> ><br>
> > Please, let's call it 'key-reset-seq', because it is exactly the reset<br>
> > sequence. There won't be any additional sequences or chords as those<br>
> > should be handled in userspace, sysrq is a special case here.<br>
><br>
> This is absolutely a linux-specific binding. It encodes the Linux<br>
> keycodes, and generates a linux meaning. I'm usually all about<br>
> carrying the OS-independent banner when defining DT bindings, but in<br>
> this case the linux prefix and sysrq reference is completely<br>
> appropriate.<br>
<br>
OK, I have no idea what "/chosen" actually means. What I am trying to say<br>
that there should be either "sysrq" or "linux,sysrq" node and that is what<br>
sysrq driver will be looking for.<br></blockquote><div><br></div><div><br></div><div>Chosen pertains to system wide parameters that aren't related to hw specific devices.  Correct, the driver will look exactly for "linux,sysrq-reset-seq" in the "chosen" node and nowhere else.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
The entire node is Linux-specific and therefore there is no point in<br>
marking only one of the properties (the key sequence) Linux-specific while<br>
leaving other ones generic.<br>
<br>
Thanks.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Dmitry<br>
</font></span></blockquote></div><br></div></div>