RE: [PATCH][ACPI][BUTTON] remove procfs-interface

[Zhang, Rui]

Well, the ACPI sysfs conversion is not finished yet
and some user space tools still use the ACPI procfs.
The schedule of removing ACPI procfs I/F will be changed to Jan 08.
I'm about to send a patch to update it.

I'm not sure if the button sysfs I/F is already finished.
We'd better make a double check. :)

Thanks,
Rui

-----Original Message-----
From: linux-acpi-owner@xxxxxxxxxxxxxxx [mailto:linux-acpi-owner@xxxxxxxxxxxxxxx] On Behalf Of Henne
Sent: 2007年7月12日 17:41
To: Arjan van de Ven
Cc: Brown, Len; linux-acpi@xxxxxxxxxxxxxxx; akpm@xxxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
Subject: Re: [PATCH][ACPI][BUTTON] remove procfs-interface

Arjan van de Ven wrote:

> On Thu, 2007-07-12 at 10:58 +0200, Henne wrote:
>   
>> Removes the proc_fs interface from the ACPI button driver.
>> Signed-off-by: Henrik Kretzschmar <henne@xxxxxxxxxxxxxxxx>
>>
>> ---
>>     
>
> you forgot to say why.....
>
>
>   
First, because refering to /Documentation/feature-removal-schedule.txt it should be done:

>What:   /proc/acpi/button
>When:   August 2007
>Why:    /proc/acpi/button has been replaced by events to the input layer
>        since 2.6.20.
>Who:    Len Brown <len.brown@xxxxxxxxx>

Second, refering to /Documentation/feature-removal-schedule.txt
 
>What:   ACPI procfs interface
>When:   July 2007
>Why:    After ACPI sysfs conversion, ACPI attributes will be duplicated
>        in sysfs and the ACPI procfs interface should be removed.
>Who:    Zhang Rui <rui.zhang@xxxxxxxxx>

The button driver uses acpi_root_dir which won't be available if acpi is removed this month.
That would lead to errors, I think.



-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/