Re: [PATCH 0/1] Input: soc_button_array - Work around DSDTs which modify the irqflags

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

 



Hi Hans,

On Sun, Sep 06, 2020 at 02:20:15PM +0200, Hans de Goede wrote:
> Hi Dmitry,
> 
> This patch is a bit of a kludge, but the problem it fixes has been
> encountered on 2 different models now, so it seems that we really
> need a workaround for this.
> 
> This patch applies on top of these 2 patches:
> 
> "Input: soc_button_array - Add active_low setting to soc_button_info"
> "Input: soc_button_array - Add support for INT33D3 tablet-mode switch devices"
> 
> Which I have posted multiple times upstream already (they are from May!),
> but these have not been getting any attention.

Sorry about that, I merged them just now.
> 
> The soc_button_array code really is x86 specific glue code to translate
> various incarnations of gpio-keys in ACPI tables to gpio_keys_platform_data.
> As such I wonder if it would not be better to move this driver to
> drivers/platform/x86?
> 
> I seem to be doing most if not all of the recent work on soc_button_array,
> and soon I will be a co-maintainer of drivers/platform/x86. So having it
> there and adding me in MAINTAINERS as maintaining it seems to be best?
> 
> If you want I can do a patch moving soc_button_array to drivers/platform/x86
> and then add the other 3 patches on top and then we can merge all of this
> through drivers/platform/x86?

Sorry, misread this first time through, so already merged the 3 patches,
but I to not mind at all moving the driver to platform tree. If you send
me such a patch I will apply it.

Thanks.

-- 
Dmitry



[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux