Re: [PATCH v2 1/2] platform/x86: asus-wmi: export function for evaluating WMI methods

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

 



On Mon, Jun 11, 2018 at 3:05 PM, Jiri Kosina <jikos@xxxxxxxxxx> wrote:
> On Thu, 31 May 2018, Andy Shevchenko wrote:
>
>> On Tue, May 22, 2018 at 11:55 PM, Daniel Drake <drake@xxxxxxxxxxxx> wrote:
>> > Export asus_wmi_evaluate_method() and related headers for use by other
>> > drivers.
>> >
>> > hid-asus is going to use this to avoid advertising that it has a keyboard
>> > backlight when the keyboard backlight is controlled via WMI.
>> >
>>
>> I'm fine with this as long as Darren and Benjamin agree on the approach.
>>
>> Acked-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>
>>
>> One nitpick below.
>
> So I'll take both patches (this one with your ack) through hid.git, ok?

As far as you are okay with new dependency to the HID driver
(Personally I don't think it's a good idea, which I commented and
proposed a better approach, i.e. #ifdef in header around exported
function and properly prepared Kconfig dependency line in HID section)


-- 
With Best Regards,
Andy Shevchenko



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

  Powered by Linux