* [PATCH][RESEND] acpi: fan.c: printk replacement
@ 2014-08-28 11:33 Sudip Mukherjee
0 siblings, 0 replies; 3+ messages in thread
From: Sudip Mukherjee @ 2014-08-28 11:33 UTC (permalink / raw)
To: Zhang Rui, Rafael J. Wysocki, Len Brown
Cc: Sudip Mukherjee, linux-acpi, linux-kernel
printk replaced with corresponding dev_err and dev_info
fixed one broken user-visible string
multiine comment edited for correct commenting style
asm/uaccess.h replaced with linux/uaccess.h
PREFIX removed
Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
---
modified with reference to the discussion at https://lkml.org/lkml/2014/8/22/176
drivers/acpi/fan.c | 18 ++++++++----------
1 file changed, 8 insertions(+), 10 deletions(-)
diff --git a/drivers/acpi/fan.c b/drivers/acpi/fan.c
index 8acf53e..5328b10 100644
--- a/drivers/acpi/fan.c
+++ b/drivers/acpi/fan.c
@@ -27,12 +27,10 @@
#include <linux/module.h>
#include <linux/init.h>
#include <linux/types.h>
-#include <asm/uaccess.h>
+#include <linux/uaccess.h>
#include <linux/thermal.h>
#include <linux/acpi.h>
-#define PREFIX "ACPI: "
-
#define ACPI_FAN_CLASS "fan"
#define ACPI_FAN_FILE_STATE "state"
@@ -127,8 +125,9 @@ static const struct thermal_cooling_device_ops fan_cooling_ops = {
};
/* --------------------------------------------------------------------------
- Driver Interface
- -------------------------------------------------------------------------- */
+ * Driver Interface
+ * --------------------------------------------------------------------------
+*/
static int acpi_fan_add(struct acpi_device *device)
{
@@ -143,7 +142,7 @@ static int acpi_fan_add(struct acpi_device *device)
result = acpi_bus_update_power(device->handle, NULL);
if (result) {
- printk(KERN_ERR PREFIX "Setting initial power state\n");
+ dev_err(&device->dev, "Setting initial power state\n");
goto end;
}
@@ -168,10 +167,9 @@ static int acpi_fan_add(struct acpi_device *device)
&device->dev.kobj,
"device");
if (result)
- dev_err(&device->dev, "Failed to create sysfs link "
- "'device'\n");
+ dev_err(&device->dev, "Failed to create sysfs link 'device'\n");
- printk(KERN_INFO PREFIX "%s [%s] (%s)\n",
+ dev_info(&device->dev, "ACPI: %s [%s] (%s)\n",
acpi_device_name(device), acpi_device_bid(device),
!device->power.state ? "on" : "off");
@@ -217,7 +215,7 @@ static int acpi_fan_resume(struct device *dev)
result = acpi_bus_update_power(to_acpi_device(dev)->handle, NULL);
if (result)
- printk(KERN_ERR PREFIX "Error updating fan power state\n");
+ dev_err(dev, "Error updating fan power state\n");
return result;
}
--
1.8.1.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH][RESEND] acpi: fan.c: printk replacement
@ 2014-08-28 12:53 edubezval
2014-08-28 13:37 ` Sudip Mukherjee
0 siblings, 1 reply; 3+ messages in thread
From: edubezval @ 2014-08-28 12:53 UTC (permalink / raw)
To: Sudip Mukherjee
Cc: Zhang Rui, Rafael J. Wysocki, Len Brown,
linux-acpi@vger.kernel.org, LKML
Hello Sudip,
On Thu, Aug 28, 2014 at 7:33 AM, Sudip Mukherjee
<sudipm.mukherjee@gmail.com> wrote:
> printk replaced with corresponding dev_err and dev_info
> fixed one broken user-visible string
> multiine comment edited for correct commenting style
> asm/uaccess.h replaced with linux/uaccess.h
> PREFIX removed
>
Just a minor tip. When sending a new version of your patches do not
use the RESEND prefix in you subject line. RESEND means you did not
make changes on your patches and you are just resending it because,
for instance, people did not answer it. When you modify your changes
after, say updating it due to review comments, it is a good practice
to use 'vX' prefix, so those who are following the patch will
understand that this patch has something new from previous version. In
this case, I suppose the current patch should be '[PATCH v2]' instead
of [PATCH][RESEND].
BR,
> Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
> ---
>
> modified with reference to the discussion at https://lkml.org/lkml/2014/8/22/176
>
> drivers/acpi/fan.c | 18 ++++++++----------
> 1 file changed, 8 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/acpi/fan.c b/drivers/acpi/fan.c
> index 8acf53e..5328b10 100644
> --- a/drivers/acpi/fan.c
> +++ b/drivers/acpi/fan.c
> @@ -27,12 +27,10 @@
> #include <linux/module.h>
> #include <linux/init.h>
> #include <linux/types.h>
> -#include <asm/uaccess.h>
> +#include <linux/uaccess.h>
> #include <linux/thermal.h>
> #include <linux/acpi.h>
>
> -#define PREFIX "ACPI: "
> -
> #define ACPI_FAN_CLASS "fan"
> #define ACPI_FAN_FILE_STATE "state"
>
> @@ -127,8 +125,9 @@ static const struct thermal_cooling_device_ops fan_cooling_ops = {
> };
>
> /* --------------------------------------------------------------------------
> - Driver Interface
> - -------------------------------------------------------------------------- */
> + * Driver Interface
> + * --------------------------------------------------------------------------
> +*/
>
> static int acpi_fan_add(struct acpi_device *device)
> {
> @@ -143,7 +142,7 @@ static int acpi_fan_add(struct acpi_device *device)
>
> result = acpi_bus_update_power(device->handle, NULL);
> if (result) {
> - printk(KERN_ERR PREFIX "Setting initial power state\n");
> + dev_err(&device->dev, "Setting initial power state\n");
> goto end;
> }
>
> @@ -168,10 +167,9 @@ static int acpi_fan_add(struct acpi_device *device)
> &device->dev.kobj,
> "device");
> if (result)
> - dev_err(&device->dev, "Failed to create sysfs link "
> - "'device'\n");
> + dev_err(&device->dev, "Failed to create sysfs link 'device'\n");
>
> - printk(KERN_INFO PREFIX "%s [%s] (%s)\n",
> + dev_info(&device->dev, "ACPI: %s [%s] (%s)\n",
> acpi_device_name(device), acpi_device_bid(device),
> !device->power.state ? "on" : "off");
>
> @@ -217,7 +215,7 @@ static int acpi_fan_resume(struct device *dev)
>
> result = acpi_bus_update_power(to_acpi_device(dev)->handle, NULL);
> if (result)
> - printk(KERN_ERR PREFIX "Error updating fan power state\n");
> + dev_err(dev, "Error updating fan power state\n");
>
> return result;
> }
> --
> 1.8.1.2
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
--
Eduardo Bezerra Valentin
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH][RESEND] acpi: fan.c: printk replacement
2014-08-28 12:53 [PATCH][RESEND] acpi: fan.c: printk replacement edubezval
@ 2014-08-28 13:37 ` Sudip Mukherjee
0 siblings, 0 replies; 3+ messages in thread
From: Sudip Mukherjee @ 2014-08-28 13:37 UTC (permalink / raw)
To: edubezval@gmail.com
Cc: Zhang Rui, Rafael J. Wysocki, Len Brown,
linux-acpi@vger.kernel.org, LKML
On Thu, Aug 28, 2014 at 08:53:45AM -0400, edubezval@gmail.com wrote:
> Hello Sudip,
>
> On Thu, Aug 28, 2014 at 7:33 AM, Sudip Mukherjee
> <sudipm.mukherjee@gmail.com> wrote:
> > printk replaced with corresponding dev_err and dev_info
> > fixed one broken user-visible string
> > multiine comment edited for correct commenting style
> > asm/uaccess.h replaced with linux/uaccess.h
> > PREFIX removed
> >
>
> Just a minor tip. When sending a new version of your patches do not
> use the RESEND prefix in you subject line. RESEND means you did not
> make changes on your patches and you are just resending it because,
> for instance, people did not answer it. When you modify your changes
> after, say updating it due to review comments, it is a good practice
> to use 'vX' prefix, so those who are following the patch will
> understand that this patch has something new from previous version. In
> this case, I suppose the current patch should be '[PATCH v2]' instead
> of [PATCH][RESEND].
>
> BR,
Hi,
Thanks for the information. I will say it a major tip for me now. And infact now I understand why i didn't get a reply for another patch i posted around 8 - 10 days back.
thanks again
sudip
>
> > Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
> > ---
> >
> > modified with reference to the discussion at https://lkml.org/lkml/2014/8/22/176
> >
> > drivers/acpi/fan.c | 18 ++++++++----------
> > 1 file changed, 8 insertions(+), 10 deletions(-)
> >
> > diff --git a/drivers/acpi/fan.c b/drivers/acpi/fan.c
> > index 8acf53e..5328b10 100644
> > --- a/drivers/acpi/fan.c
> > +++ b/drivers/acpi/fan.c
> > @@ -27,12 +27,10 @@
> > #include <linux/module.h>
> > #include <linux/init.h>
> > #include <linux/types.h>
> > -#include <asm/uaccess.h>
> > +#include <linux/uaccess.h>
> > #include <linux/thermal.h>
> > #include <linux/acpi.h>
> >
> > -#define PREFIX "ACPI: "
> > -
> > #define ACPI_FAN_CLASS "fan"
> > #define ACPI_FAN_FILE_STATE "state"
> >
> > @@ -127,8 +125,9 @@ static const struct thermal_cooling_device_ops fan_cooling_ops = {
> > };
> >
> > /* --------------------------------------------------------------------------
> > - Driver Interface
> > - -------------------------------------------------------------------------- */
> > + * Driver Interface
> > + * --------------------------------------------------------------------------
> > +*/
> >
> > static int acpi_fan_add(struct acpi_device *device)
> > {
> > @@ -143,7 +142,7 @@ static int acpi_fan_add(struct acpi_device *device)
> >
> > result = acpi_bus_update_power(device->handle, NULL);
> > if (result) {
> > - printk(KERN_ERR PREFIX "Setting initial power state\n");
> > + dev_err(&device->dev, "Setting initial power state\n");
> > goto end;
> > }
> >
> > @@ -168,10 +167,9 @@ static int acpi_fan_add(struct acpi_device *device)
> > &device->dev.kobj,
> > "device");
> > if (result)
> > - dev_err(&device->dev, "Failed to create sysfs link "
> > - "'device'\n");
> > + dev_err(&device->dev, "Failed to create sysfs link 'device'\n");
> >
> > - printk(KERN_INFO PREFIX "%s [%s] (%s)\n",
> > + dev_info(&device->dev, "ACPI: %s [%s] (%s)\n",
> > acpi_device_name(device), acpi_device_bid(device),
> > !device->power.state ? "on" : "off");
> >
> > @@ -217,7 +215,7 @@ static int acpi_fan_resume(struct device *dev)
> >
> > result = acpi_bus_update_power(to_acpi_device(dev)->handle, NULL);
> > if (result)
> > - printk(KERN_ERR PREFIX "Error updating fan power state\n");
> > + dev_err(dev, "Error updating fan power state\n");
> >
> > return result;
> > }
> > --
> > 1.8.1.2
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> > Please read the FAQ at http://www.tux.org/lkml/
>
>
>
> --
> Eduardo Bezerra Valentin
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-08-28 13:37 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-08-28 12:53 [PATCH][RESEND] acpi: fan.c: printk replacement edubezval
2014-08-28 13:37 ` Sudip Mukherjee
-- strict thread matches above, loose matches on Subject: below --
2014-08-28 11:33 Sudip Mukherjee
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox