Re: [PATCH 1/4] wmi: add a pair of interfaces to set/get wmi driver data

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

 



On Fri, Apr 9, 2010 at 9:11 PM, Carlos Corbacho
<carlos@xxxxxxxxxxxxxxxxxxx> wrote:
> On Friday 09 April 2010 18:49:02 Dmitry Torokhov wrote:
>> On Fri, Apr 09, 2010 at 11:03:20PM +0800, Yong Wang wrote:
>> > Currently there is no easy way to associate driver data kind of thing
>> > with wmi drivers thus global variables have to be used to pass that
>> > information between functions. Add a pair of functions to support
>> > that.
>
> NAK
>
>> It looks like once you compete the conversion and have platform device
>> you can attach to it instead of needing driver data in WMI.
>>
>
> I'm with Dmitry on this one. Please avoid adding this kind of stuff to WMI -
> it's just intended to be a thin wrapper around ACPI calls, not a full driver
> class with its' own mechanisms for data storage, etc.
>

You can probably just use
platform_get_drvdata(_dev) - platform_set_drvdata(_dev,data)
wihtout a lot of change in the following patchs.
-- 
Corentin Chary
http://xf.iksaif.net
--
To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

  Powered by Linux