Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20211008222327.9324-1-angiolucci@gmail.com> <CAKdAkRTdCSC_6UqWkcT3p697+d0kwEqMSSR=M9vw-NPAjdZ4SQ@mail.gmail.com>
 <CALawNGC3HTV86+LAkMNf_nApqnUoAQ7UPEXOj9WXake5DhHkpQ@mail.gmail.com> <YYWoRW9NRC58zQ9w@google.com>
In-Reply-To: <YYWoRW9NRC58zQ9w@google.com>
From:   =?UTF-8?Q?Vin=C3=ADcius_Reis?= <itsme.vreis@gmail.com>
Date:   Sat, 6 Nov 2021 20:48:29 -0300
Message-ID: <CALawNGBookRkFSGdGJrW1hboJ8kpsPj_rwAkNXQ3mU32oDv-sA@mail.gmail.com>
Subject: Re: [PATCH] HID: hid-asus.c: Maps key 0x35 (display off) to KEY_SCREENLOCK
To:     Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc:     Jiri Kosina <jikos@kernel.org>,
        "linux-input@vger.kernel.org" <linux-input@vger.kernel.org>,
        lkml <linux-kernel@vger.kernel.org>, angiolucci@gmail.com
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Em sex., 5 de nov. de 2021 =C3=A0s 18:55, Dmitry Torokhov
<dmitry.torokhov@gmail.com> escreveu:
>
> On Fri, Nov 05, 2021 at 12:16:45PM -0300, Vin=C3=ADcius Reis wrote:
> > Em qui., 28 de out. de 2021 =C3=A0s 13:07, Dmitry Torokhov
> > <dmitry.torokhov@gmail.com> escreveu:
> > >
> > > Hi Vin=C3=ADcius,
> > >
> > > On Fri, Oct 8, 2021 at 3:24 PM Vin=C3=ADcius Angiolucci Reis
> > > <itsme.vreis@gmail.com> wrote:
> > > >
> > > > On Windows systems, ASUS laptops uses the "turn display off" key
> > > > (usually fn+f6) to turn both display and keyboard backlit off. On L=
inux
> > > > systems, this key has no effect at all since most desktop enviromen=
ts
> > > > don't deal with KEY_DISPLAY_OFF. By mapping it to KEY_SCREENLOCK
> > > > instead, would enable desktop environments to handle this key as a
> > > > screen lock intent from the user, out of the box.
> > >
> > > But is it the intent of the user? The fact that current desktop
> > > environments do not handle this key is not a reason to change kernel
> > > behavior.
> > >
> > > If screen lock is simply your preference then you can change the
> > > mapping via udev on your system without the need to patch the kernel.
> > >
> > > Thanks.
> > >
> > > --
> > > Dmitry
> >
> > (Sorry, I'm sending this response again because it seems the first
> > attempt didn't make it to the mailing lists for some reason.)
> >
> > Hi Dmitry,
> >
> > I've picked up this particular key event because it would trigger a clo=
se
> > behavior to the one seen on Windows. It is far from being my preference=
,
> > which would be an exact mimic of the Windows behavior (and can be
> > achieved by xset dpms force on/off from user space).
> >
> > The idea of this patch is to improve the user experience, by giving tha=
t key a
> > behavior as close as possible to what users might experience on Windows=
.
> > Because most Linux desktop environments also turn the display off after
> > locking the screen, this would be a good approximation to what happens =
on
> > Windows. I think it is more about giving (consistent) behavior to it
> > than changing it.
> >
> > I agree it is possible to control that via udev or something else. But
> > most regular
> > users won't be able to do that, ending up with an unused key, though.
> > The hardcore user that would prefer a very different key behavior
> > could keep using
> > udev to do that.
> >
> > So IMHO I think that setting this up on the kernel would be a good
> > approach in this case.
>
> I am sorry, I disagree. I understand that patching the kernel might be
> easier than implementing new shortcut/behavior in various desktop
> environments, but that does not make it the right approach.
>
> Thanks.
>
> --
> Dmitry

Hi Dmitry,

Thanks for explaining this a little bit more, I've got the point: The curre=
nt
key code triggered by the kernel is correct, the real issue is the lack of
support for that particular code by current desktop environments.

I'm new to contributing to the Linux kernel; the proper way to rollback
this would be by me just sending a revert patch to the respective
maintainer?

Thank you,
Vin=C3=ADcius.