Re: [PATCH] platform/x86: barco-p50-gpio: use KEY_VENDOR for button insted of KEY_RESTART

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On 10/22/21 17:40, Peter Korsgaard wrote:
>>>>>> "Peter" == Peter Korsgaard <peter@xxxxxxxxxxxxx> writes:
> 
> Ups, s/insted/instead/ in the subject. Do you want me to send a v2 for that?

No need for a v2, I'll fix this up when applying.

Regards,

Hans


> 
>  > From: Peter Korsgaard <peter.korsgaard@xxxxxxxxx>
>  > It turns out that systemd-logind by default listens for KEY_RESTART input
>  > events and reboots the machine, which isn't great - So use KEY_VENDOR for
>  > the vendor specific identify button instead to not conflict.
> 
>  > Signed-off-by: Peter Korsgaard <peter.korsgaard@xxxxxxxxx>
>  > ---
>  >  drivers/platform/x86/barco-p50-gpio.c | 2 +-
>  >  1 file changed, 1 insertion(+), 1 deletion(-)
> 
>  > diff --git a/drivers/platform/x86/barco-p50-gpio.c b/drivers/platform/x86/barco-p50-gpio.c
>  > index ca0b2564c407..f5c72e33f9ae 100644
>  > --- a/drivers/platform/x86/barco-p50-gpio.c
>  > +++ b/drivers/platform/x86/barco-p50-gpio.c
>  > @@ -101,7 +101,7 @@ static struct gpio_led_platform_data leds_pdata = {
>  >  /* GPIO keyboard */
>  >  static struct gpio_keys_button buttons[] = {
>  >  	{
>  > -		.code = KEY_RESTART,
>  > +		.code = KEY_VENDOR,
>  >  		.gpio = P50_GPIO_LINE_BTN,
>  >  		.active_low = 1,
>  >  		.type = EV_KEY,
>  > -- 
>  > 2.20.1
> 
> 




[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux