Re: [PATCH 2/4] acer-wmi: check the existence of internal 3G device when set capability

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

 



On Sun, May 22, 2011 at 2:02 AM, Joey Lee <jlee@xxxxxxxxxx> wrote:
> æ æï2011-05-22 æ 00:43 +0100ïCarlos Corbacho æåï
>> On Sunday 22 May 2011 00:33:52 Lee, Chun-Yi wrote:
>> > That will be better to check the existence of internal 3G device when
>> > we set threeg capability and generate killswitch for threeg. It can
>> > avoid userland access 3G rfkill but the machine doesn't have internal
>> > 3G device.
>> >
>> > Reference: bko#32862
>> > Â Â https://bugzilla.kernel.org/show_bug.cgi?id=32862
>> >
>> > Tested on Acer Aspire 8930G, Acer Travelmate 8572
>> >
>> > Tested-by: Hector Martin <hector@xxxxxxxxxxxxxx>
>>
>> Acked-by: Carlos Corbacho <carlos@xxxxxxxxxxxxxxxxxxx>
>>
>> If you can now autodetect 3G and already have the relevant rfkill device, then
>> you should also rip out the remaining sysfs support for it as it's no longer
>> necessary.
>>
>> -Carlos
>
> Got it!
> I will send another patch to remove threeg sysfs file.

I'm not sure you can drop sysfs files like that, as it will break the ABI.

Could you document acer-wmi in
Documentation/ABI/testing/sysfs-platform-acer-wmi.txt ?
This allows to check when a sysfs file was added, who use it, etc...

Thanks,

-- 
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