Re: [PATCH 2/4 v2] hwmon: ntc_thermistor: Use the manufacturer name properly

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

 




Hello Naveen,

On Wed, Jun 25, 2014 at 8:29 AM, Naveen Krishna Chatradhi
<ch.naveen@xxxxxxxxxxx> wrote:
> Murata Manufacturing Co., Ltd is the vendor for
> NTC (Negative Temperature coefficient) based Thermistors.
> But, the driver extensively uses "NTC" as the vendor name.
>
> This patch corrects the vendor name also updates the
> compatibility strings according to the vendor-prefix.txt
>
> Note: Drivers continue to support the previous compatible strings
> but further addition of these compatible strings in device tree
> is deprecated.
>
> Signed-off-by: Naveen Krishna Chatradhi <ch.naveen@xxxxxxxxxxx>
> Cc: Guenter Roeck <linux@xxxxxxxxxxxx>
> ---
> Changes since v1:
> 1. Kept the old compatible string marked as <DEPRECATED> in code and
>    in the Documentation.
>
>  .../devicetree/bindings/arm/samsung/exynos-adc.txt |    2 +-
>  .../devicetree/bindings/hwmon/ntc_thermistor.txt   |   20 ++++++++++++++------
>  Documentation/hwmon/ntc_thermistor                 |    8 ++++----
>  drivers/hwmon/Kconfig                              |    5 +++--
>  drivers/hwmon/ntc_thermistor.c                     |   14 +++++++++++++-
>  5 files changed, 35 insertions(+), 14 deletions(-)
>

Looks good to me.

Reviewed-by: Javier Martinez Canillas <javier.martinez@xxxxxxxxxxxxxxx>

> diff --git a/Documentation/devicetree/bindings/arm/samsung/exynos-adc.txt b/Documentation/devicetree/bindings/arm/samsung/exynos-adc.txt
> index 5d49f2b..832fe8c 100644
> --- a/Documentation/devicetree/bindings/arm/samsung/exynos-adc.txt
> +++ b/Documentation/devicetree/bindings/arm/samsung/exynos-adc.txt
> @@ -48,7 +48,7 @@ adc@12D10000 {
>
>         /* NTC thermistor is a hwmon device */
>         ncp15wb473@0 {
> -               compatible = "ntc,ncp15wb473";
> +               compatible = "murata,ncp15wb473";
>                 pullup-uv = <1800000>;
>                 pullup-ohm = <47000>;
>                 pulldown-ohm = <0>;
> diff --git a/Documentation/devicetree/bindings/hwmon/ntc_thermistor.txt b/Documentation/devicetree/bindings/hwmon/ntc_thermistor.txt
> index c6f6667..b117b2e 100644
> --- a/Documentation/devicetree/bindings/hwmon/ntc_thermistor.txt
> +++ b/Documentation/devicetree/bindings/hwmon/ntc_thermistor.txt
> @@ -3,11 +3,19 @@ NTC Thermistor hwmon sensors
>
>  Requires node properties:
>  - "compatible" value : one of
> -       "ntc,ncp15wb473"
> -       "ntc,ncp18wb473"
> -       "ntc,ncp21wb473"
> -       "ntc,ncp03wb473"
> -       "ntc,ncp15wl333"
> +       "murata,ncp15wb473"
> +       "murata,ncp18wb473"
> +       "murata,ncp21wb473"
> +       "murata,ncp03wb473"
> +       "murata,ncp15wl333"
> +
> +/* Usage of vendor name "ntc" is deprecated */
> +<DEPRECATED>   "ntc,ncp15wb473"
> +<DEPRECATED>   "ntc,ncp18wb473"
> +<DEPRECATED>   "ntc,ncp21wb473"
> +<DEPRECATED>   "ntc,ncp03wb473"
> +<DEPRECATED>   "ntc,ncp15wl333"
> +
>  - "pullup-uv"  Pull up voltage in micro volts
>  - "pullup-ohm" Pull up resistor value in ohms
>  - "pulldown-ohm" Pull down resistor value in ohms
> @@ -21,7 +29,7 @@ Read more about iio bindings at
>
>  Example:
>         ncp15wb473@0 {
> -               compatible = "ntc,ncp15wb473";
> +               compatible = "murata,ncp15wb473";
>                 pullup-uv = <1800000>;
>                 pullup-ohm = <47000>;
>                 pulldown-ohm = <0>;
> diff --git a/Documentation/hwmon/ntc_thermistor b/Documentation/hwmon/ntc_thermistor
> index 3bfda94..057b770 100644
> --- a/Documentation/hwmon/ntc_thermistor
> +++ b/Documentation/hwmon/ntc_thermistor
> @@ -1,7 +1,7 @@
>  Kernel driver ntc_thermistor
>  =================
>
> -Supported thermistors:
> +Supported thermistors from Murata:
>  * Murata NTC Thermistors NCP15WB473, NCP18WB473, NCP21WB473, NCP03WB473, NCP15WL333
>    Prefixes: 'ncp15wb473', 'ncp18wb473', 'ncp21wb473', 'ncp03wb473', 'ncp15wl333'
>    Datasheet: Publicly available at Murata
> @@ -15,9 +15,9 @@ Authors:
>  Description
>  -----------
>
> -The NTC thermistor is a simple thermistor that requires users to provide the
> -resistance and lookup the corresponding compensation table to get the
> -temperature input.
> +The NTC (Negative Temperature Coefficient) thermistor is a simple thermistor
> +that requires users to provide the resistance and lookup the corresponding
> +compensation table to get the temperature input.
>
>  The NTC driver provides lookup tables with a linear approximation function
>  and four circuit models with an option not to use any of the four models.
> diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig
> index 08531a1..154851b 100644
> --- a/drivers/hwmon/Kconfig
> +++ b/drivers/hwmon/Kconfig
> @@ -1052,7 +1052,7 @@ config SENSORS_PC87427
>           will be called pc87427.
>
>  config SENSORS_NTC_THERMISTOR
> -       tristate "NTC thermistor support"
> +       tristate "NTC thermistor support from Murata"
>         depends on !OF || IIO=n || IIO
>         help
>           This driver supports NTC thermistors sensor reading and its
> @@ -1060,7 +1060,8 @@ config SENSORS_NTC_THERMISTOR
>           send notifications about the temperature.
>
>           Currently, this driver supports
> -         NCP15WB473, NCP18WB473, NCP21WB473, NCP03WB473, and NCP15WL333.
> +         NCP15WB473, NCP18WB473, NCP21WB473, NCP03WB473, and NCP15WL333
> +         from Murata.
>
>           This driver can also be built as a module.  If so, the module
>           will be called ntc-thermistor.
> diff --git a/drivers/hwmon/ntc_thermistor.c b/drivers/hwmon/ntc_thermistor.c
> index e76feb86..bdfbe91 100644
> --- a/drivers/hwmon/ntc_thermistor.c
> +++ b/drivers/hwmon/ntc_thermistor.c
> @@ -163,6 +163,18 @@ static int ntc_adc_iio_read(struct ntc_thermistor_platform_data *pdata)
>  }
>
>  static const struct of_device_id ntc_match[] = {
> +       { .compatible = "murata,ncp15wb473",
> +               .data = &ntc_thermistor_id[0] },
> +       { .compatible = "murata,ncp18wb473",
> +               .data = &ntc_thermistor_id[1] },
> +       { .compatible = "murata,ncp21wb473",
> +               .data = &ntc_thermistor_id[2] },
> +       { .compatible = "murata,ncp03wb473",
> +               .data = &ntc_thermistor_id[3] },
> +       { .compatible = "murata,ncp15wl333",
> +               .data = &ntc_thermistor_id[4] },
> +
> +       /* Usage of vendor name "ntc" is deprecated */
>         { .compatible = "ntc,ncp15wb473",
>                 .data = &ntc_thermistor_id[0] },
>         { .compatible = "ntc,ncp18wb473",
> @@ -534,7 +546,7 @@ static struct platform_driver ntc_thermistor_driver = {
>
>  module_platform_driver(ntc_thermistor_driver);
>
> -MODULE_DESCRIPTION("NTC Thermistor Driver");
> +MODULE_DESCRIPTION("NTC Thermistor Driver from Murata");
>  MODULE_AUTHOR("MyungJoo Ham <myungjoo.ham@xxxxxxxxxxx>");
>  MODULE_LICENSE("GPL");
>  MODULE_ALIAS("platform:ntc-thermistor");
> --
> 1.7.9.5
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" 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 devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux