Patchworkβ acpi: thermal: [TRIVIAL] Add EOL to the trip_point_N_type strings

login
register
about
Submitter Amit Kucheria
Date 2009-10-20 06:11:14
Message ID <37786d4b0910192311t7a9aad87i7adc62cf86bc29e2@mail.gmail.com>
Download mbox | patch
Permalink /patch/54892/
State New
Headers show

Comments

Amit Kucheria - 2009-10-20 06:11:14
Rui,

This patch should perhaps have been cc'ed to you and the linux-acpi
list as well.

Regards,
Amit

---------- Forwarded message ----------
From:  <amit.kucheria@canonical.com>
Date: Fri, Oct 16, 2009 at 12:46 PM
Subject: [PATCH] acpi: thermal: [TRIVIAL] Add EOL to the
trip_point_N_type strings
To: List Linux Kernel <linux-kernel@vger.kernel.org>
Cc: len.brown@intel.com, Amit Kucheria <amit.kucheria@canonical.com>


From: Amit Kucheria <amit.kucheria@canonical.com>

Make the trip_point_N_type sysfs files return a string ending in EOL for
consistency with other sysfs files.

Signed-off-by: Amit Kucheria <amit.kucheria@canonical.com>
---
 drivers/thermal/thermal_sys.c |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)


--
1.6.3.3
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Zhang Rui - 2009-10-20 06:16:52
Acked-by: Zhang Rui <rui.zhang@intel.com>

thanks,
rui

On Tue, 2009-10-20 at 14:11 +0800, Amit Kucheria wrote:
> Rui,
> 
> This patch should perhaps have been cc'ed to you and the linux-acpi
> list as well.
> 
> Regards,
> Amit
> 
> ---------- Forwarded message ----------
> From:  <amit.kucheria@canonical.com>
> Date: Fri, Oct 16, 2009 at 12:46 PM
> Subject: [PATCH] acpi: thermal: [TRIVIAL] Add EOL to the
> trip_point_N_type strings
> To: List Linux Kernel <linux-kernel@vger.kernel.org>
> Cc: len.brown@intel.com, Amit Kucheria <amit.kucheria@canonical.com>
> 
> 
> From: Amit Kucheria <amit.kucheria@canonical.com>
> 
> Make the trip_point_N_type sysfs files return a string ending in EOL for
> consistency with other sysfs files.
> 
> Signed-off-by: Amit Kucheria <amit.kucheria@canonical.com>
> ---
>  drivers/thermal/thermal_sys.c |   10 +++++-----
>  1 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/thermal/thermal_sys.c b/drivers/thermal/thermal_sys.c
> index 4e83c29..6f8d8f9 100644
> --- a/drivers/thermal/thermal_sys.c
> +++ b/drivers/thermal/thermal_sys.c
> @@ -180,15 +180,15 @@ trip_point_type_show(struct device *dev, struct
> device_attribute *attr,
> 
>        switch (type) {
>        case THERMAL_TRIP_CRITICAL:
> -               return sprintf(buf, "critical");
> +               return sprintf(buf, "critical\n");
>        case THERMAL_TRIP_HOT:
> -               return sprintf(buf, "hot");
> +               return sprintf(buf, "hot\n");
>        case THERMAL_TRIP_PASSIVE:
> -               return sprintf(buf, "passive");
> +               return sprintf(buf, "passive\n");
>        case THERMAL_TRIP_ACTIVE:
> -               return sprintf(buf, "active");
> +               return sprintf(buf, "active\n");
>        default:
> -               return sprintf(buf, "unknown");
> +               return sprintf(buf, "unknown\n");
>        }
>  }
> 
> --
> 1.6.3.3

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

Patch

diff --git a/drivers/thermal/thermal_sys.c b/drivers/thermal/thermal_sys.c
index 4e83c29..6f8d8f9 100644
--- a/drivers/thermal/thermal_sys.c
+++ b/drivers/thermal/thermal_sys.c
@@ -180,15 +180,15 @@  trip_point_type_show(struct device *dev, struct
device_attribute *attr,

       switch (type) {
       case THERMAL_TRIP_CRITICAL:
-               return sprintf(buf, "critical");
+               return sprintf(buf, "critical\n");
       case THERMAL_TRIP_HOT:
-               return sprintf(buf, "hot");
+               return sprintf(buf, "hot\n");
       case THERMAL_TRIP_PASSIVE:
-               return sprintf(buf, "passive");
+               return sprintf(buf, "passive\n");
       case THERMAL_TRIP_ACTIVE:
-               return sprintf(buf, "active");
+               return sprintf(buf, "active\n");
       default:
-               return sprintf(buf, "unknown");
+               return sprintf(buf, "unknown\n");
       }
 }