* [PATCH] acer-wmi: schedule threeg and interface sysfs for feature removal
@ 2011-05-30 0:21 Lee, Chun-Yi
2011-05-30 7:47 ` Thomas Renninger
0 siblings, 1 reply; 3+ messages in thread
From: Lee, Chun-Yi @ 2011-05-30 0:21 UTC (permalink / raw)
To: mjg
Cc: platform-driver-x86, Lee, Chun-Yi, Carlos Corbacho,
Dmitry Torokhov, Corentin Chary, Thomas Renninger
we can now autodetect internal 3G device and already have the threeg
rfkill device. So, we plan to remove threeg sysfs support for it's no
longer necessary.
We also plan to remove interface sysfs file that exposed which ACPI-WMI
interface that was used by acer-wmi driver. It will replaced by information
log when acer-wmi initial.
We keep it around for userspace compatibility reasons, schedule removal
in 2012.
Cc: Carlos Corbacho <carlos@strangeworlds.co.uk>
Cc: Matthew Garrett <mjg@redhat.com>
Cc: Dmitry Torokhov <dtor@mail.ru>
Cc: Corentin Chary <corentincj@iksaif.net>
Cc: Thomas Renninger <trenn@suse.de>
Signed-off-by: Lee, Chun-Yi <jlee@novell.com>
---
Documentation/feature-removal-schedule.txt | 13 +++++++++++++
drivers/platform/x86/acer-wmi.c | 3 +++
2 files changed, 16 insertions(+), 0 deletions(-)
diff --git a/Documentation/feature-removal-schedule.txt b/Documentation/feature-removal-schedule.txt
index 492e81d..8427ad0 100644
--- a/Documentation/feature-removal-schedule.txt
+++ b/Documentation/feature-removal-schedule.txt
@@ -580,3 +580,16 @@ Why: These legacy callbacks should no longer be used as i2c-core offers
Who: Jean Delvare <khali@linux-fr.org>
----------------------------
+
+What: threeg and interface sysfs files in /sys/devices/platform/acer-wmi
+When: 2012
+Why: In 2.6.40, we can now autodetect internal 3G device and already have
+ the threeg rfkill device. So, we plan to remove threeg sysfs support
+ for it's no longer necessary.
+
+ We also plan to remove interface sysfs file that exposed which ACPI-WMI
+ interface that was used by acer-wmi driver. It will replaced by
+ information log when acer-wmi initial.
+Who: Lee, Chun-Yi <jlee@novell.com>
+
+----------------------------
diff --git a/drivers/platform/x86/acer-wmi.c b/drivers/platform/x86/acer-wmi.c
index 1495539..7e7e668 100644
--- a/drivers/platform/x86/acer-wmi.c
+++ b/drivers/platform/x86/acer-wmi.c
@@ -1396,6 +1396,8 @@ static ssize_t show_bool_threeg(struct device *dev,
{
u32 result; \
acpi_status status;
+
+ pr_info("This threeg sysfs will be removed in 2012.");
if (wmi_has_guid(WMID_GUID3))
status = wmid3_get_device_status(&result,
ACER_WMID3_GDS_THREEG);
@@ -1421,6 +1423,7 @@ static DEVICE_ATTR(threeg, S_IRUGO | S_IWUSR, show_bool_threeg,
static ssize_t show_interface(struct device *dev, struct device_attribute *attr,
char *buf)
{
+ pr_info("This interface sysfs will be removed in 2012.");
switch (interface->type) {
case ACER_AMW0:
return sprintf(buf, "AMW0\n");
--
1.6.0.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] acer-wmi: schedule threeg and interface sysfs for feature removal
2011-05-30 0:21 [PATCH] acer-wmi: schedule threeg and interface sysfs for feature removal Lee, Chun-Yi
@ 2011-05-30 7:47 ` Thomas Renninger
2011-05-30 9:05 ` Joey Lee
0 siblings, 1 reply; 3+ messages in thread
From: Thomas Renninger @ 2011-05-30 7:47 UTC (permalink / raw)
To: Lee, Chun-Yi
Cc: mjg, platform-driver-x86, Lee, Chun-Yi, Carlos Corbacho,
Dmitry Torokhov, Corentin Chary
On Monday, May 30, 2011 02:21:40 AM Lee, Chun-Yi wrote:
> we can now autodetect internal 3G device and already have the threeg
> rfkill device. So, we plan to remove threeg sysfs support for it's no
> longer necessary.
>
> We also plan to remove interface sysfs file that exposed which ACPI-WMI
> interface that was used by acer-wmi driver. It will replaced by information
> log when acer-wmi initial.
>
> We keep it around for userspace compatibility reasons, schedule removal
> in 2012.
...
> diff --git a/drivers/platform/x86/acer-wmi.c b/drivers/platform/x86/acer-wmi.c
> index 1495539..7e7e668 100644
> --- a/drivers/platform/x86/acer-wmi.c
> +++ b/drivers/platform/x86/acer-wmi.c
> @@ -1396,6 +1396,8 @@ static ssize_t show_bool_threeg(struct device *dev,
> {
> u32 result; \
> acpi_status status;
> +
> + pr_info("This threeg sysfs will be removed in 2012.");
You could also add:
" - used by: %s\n", current->comm);
then it's easier to identify and fix the app still making use of it.
> if (wmi_has_guid(WMID_GUID3))
> status = wmid3_get_device_status(&result,
> ACER_WMID3_GDS_THREEG);
> @@ -1421,6 +1423,7 @@ static DEVICE_ATTR(threeg, S_IRUGO | S_IWUSR, show_bool_threeg,
> static ssize_t show_interface(struct device *dev, struct device_attribute *attr,
> char *buf)
> {
> + pr_info("This interface sysfs will be removed in 2012.");
Same.
> switch (interface->type) {
> case ACER_AMW0:
> return sprintf(buf, "AMW0\n");
Thomas
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] acer-wmi: schedule threeg and interface sysfs for feature removal
2011-05-30 7:47 ` Thomas Renninger
@ 2011-05-30 9:05 ` Joey Lee
0 siblings, 0 replies; 3+ messages in thread
From: Joey Lee @ 2011-05-30 9:05 UTC (permalink / raw)
To: trenn; +Cc: corentincj, dtor, Joey Lee, mjg, carlos, platform-driver-x86
Hi Thomas,
於 一，2011-05-30 於 09:47 +0200，Thomas Renninger 提到：
> On Monday, May 30, 2011 02:21:40 AM Lee, Chun-Yi wrote:
> > we can now autodetect internal 3G device and already have the threeg
> > rfkill device. So, we plan to remove threeg sysfs support for it's no
> > longer necessary.
> >
> > We also plan to remove interface sysfs file that exposed which ACPI-WMI
> > interface that was used by acer-wmi driver. It will replaced by information
> > log when acer-wmi initial.
> >
> > We keep it around for userspace compatibility reasons, schedule removal
> > in 2012.
>
> ...
>
> > diff --git a/drivers/platform/x86/acer-wmi.c b/drivers/platform/x86/acer-wmi.c
> > index 1495539..7e7e668 100644
> > --- a/drivers/platform/x86/acer-wmi.c
> > +++ b/drivers/platform/x86/acer-wmi.c
> > @@ -1396,6 +1396,8 @@ static ssize_t show_bool_threeg(struct device *dev,
> > {
> > u32 result; \
> > acpi_status status;
> > +
> > + pr_info("This threeg sysfs will be removed in 2012.");
> You could also add:
> " - used by: %s\n", current->comm);
> then it's easier to identify and fix the app still making use of it.
>
>
> > if (wmi_has_guid(WMID_GUID3))
> > status = wmid3_get_device_status(&result,
> > ACER_WMID3_GDS_THREEG);
> > @@ -1421,6 +1423,7 @@ static DEVICE_ATTR(threeg, S_IRUGO | S_IWUSR, show_bool_threeg,
> > static ssize_t show_interface(struct device *dev, struct device_attribute *attr,
> > char *buf)
> > {
> > + pr_info("This interface sysfs will be removed in 2012.");
> Same.
> > switch (interface->type) {
> > case ACER_AMW0:
> > return sprintf(buf, "AMW0\n");
>
> Thomas
Thank's for your review and suggestion, I modified patch like the
following:
From 0dc691b3f99794a6b91728413919720454a9be44 Mon Sep 17 00:00:00 2001
From: Lee, Chun-Yi <jlee@novell.com>
Date: Mon, 30 May 2011 16:58:27 +0800
Subject: [PATCH v2] acer-wmi: schedule threeg and interface sysfs for feature removal
we can now autodetect internal 3G device and already have the threeg
rfkill device. So, we plan to remove threeg sysfs support for it's no
longer necessary.
We also plan to remove interface sysfs file that exposed which ACPI-WMI
interface that was used by acer-wmi driver. It will replaced by information
log when acer-wmi initial.
We keep it around for userspace compatibility reasons, schedule removal
in 2012.
Cc: Carlos Corbacho <carlos@strangeworlds.co.uk>
Cc: Matthew Garrett <mjg@redhat.com>
Cc: Dmitry Torokhov <dtor@mail.ru>
Cc: Corentin Chary <corentincj@iksaif.net>
Cc: Thomas Renninger <trenn@suse.de>
Signed-off-by: Lee, Chun-Yi <jlee@novell.com>
---
Documentation/feature-removal-schedule.txt | 13 +++++++++++++
drivers/platform/x86/acer-wmi.c | 11 +++++++++--
2 files changed, 22 insertions(+), 2 deletions(-)
diff --git a/Documentation/feature-removal-schedule.txt b/Documentation/feature-removal-schedule.txt
index ff31b1c..c8c18d7 100644
--- a/Documentation/feature-removal-schedule.txt
+++ b/Documentation/feature-removal-schedule.txt
@@ -564,3 +564,16 @@ Why: Superseded by the UVCIOC_CTRL_QUERY ioctl.
Who: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
----------------------------
+
+What: threeg and interface sysfs files in /sys/devices/platform/acer-wmi
+When: 2012
+Why: In 2.6.40, we can now autodetect internal 3G device and already have
+ the threeg rfkill device. So, we plan to remove threeg sysfs support
+ for it's no longer necessary.
+
+ We also plan to remove interface sysfs file that exposed which ACPI-WMI
+ interface that was used by acer-wmi driver. It will replaced by
+ information log when acer-wmi initial.
+Who: Lee, Chun-Yi <jlee@novell.com>
+
+----------------------------
diff --git a/drivers/platform/x86/acer-wmi.c b/drivers/platform/x86/acer-wmi.c
index 005417b..9408cd5 100644
--- a/drivers/platform/x86/acer-wmi.c
+++ b/drivers/platform/x86/acer-wmi.c
@@ -1400,6 +1400,9 @@ static ssize_t show_bool_threeg(struct device *dev,
{
u32 result; \
acpi_status status;
+
+ pr_info("This threeg sysfs will be removed in 2012"
+ " - used by: %s\n", current->comm);
if (wmi_has_guid(WMID_GUID3))
status = wmid3_get_device_status(&result,
ACER_WMID3_GDS_THREEG);
@@ -1415,8 +1418,10 @@ static ssize_t set_bool_threeg(struct device *dev,
{
u32 tmp = simple_strtoul(buf, NULL, 10);
acpi_status status = set_u32(tmp, ACER_CAP_THREEG);
- if (ACPI_FAILURE(status))
- return -EINVAL;
+ pr_info("This threeg sysfs will be removed in 2012"
+ " - used by: %s\n", current->comm);
+ if (ACPI_FAILURE(status))
+ return -EINVAL;
return count;
}
static DEVICE_ATTR(threeg, S_IRUGO | S_IWUSR, show_bool_threeg,
@@ -1425,6 +1430,8 @@ static DEVICE_ATTR(threeg, S_IRUGO | S_IWUSR, show_bool_threeg,
static ssize_t show_interface(struct device *dev, struct device_attribute *attr,
char *buf)
{
+ pr_info("This interface sysfs will be removed in 2012"
+ " - used by: %s\n", current->comm);
switch (interface->type) {
case ACER_AMW0:
return sprintf(buf, "AMW0\n");
--
1.6.0.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-05-30 9:05 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-05-30 0:21 [PATCH] acer-wmi: schedule threeg and interface sysfs for feature removal Lee, Chun-Yi
2011-05-30 7:47 ` Thomas Renninger
2011-05-30 9:05 ` Joey Lee
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.