diff mbox series

hwmon: (ntc_thermistor): Add Samsung 1404-001221 NTC

Message ID 20211215174241.1496169-1-linus.walleij@linaro.org (mailing list archive)
State Accepted
Headers show
Series hwmon: (ntc_thermistor): Add Samsung 1404-001221 NTC | expand

Commit Message

Linus Walleij Dec. 15, 2021, 5:42 p.m. UTC
This adds the Samsung 1404-001221 NTC thermistor to the
NTC thermistor driver. As far as I can tell it is electrically
compatible with the Murata 47K NTC thermistor.

This thermistor is mounted in a variety of Samsung products.

Cc: Peter Rosin <peda@axentia.se>
Cc: Chris Lesiak <chris.lesiak@licor.com>
Cc: devicetree@vger.kernel.org
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
The DT binding oneline is included, I see no point in splitting
that into a separate patch.
---
 Documentation/devicetree/bindings/hwmon/ntc-thermistor.yaml | 1 +
 drivers/hwmon/ntc_thermistor.c                              | 4 ++++
 2 files changed, 5 insertions(+)

Comments

Rob Herring Dec. 16, 2021, 8:29 p.m. UTC | #1
On Wed, 15 Dec 2021 18:42:41 +0100, Linus Walleij wrote:
> This adds the Samsung 1404-001221 NTC thermistor to the
> NTC thermistor driver. As far as I can tell it is electrically
> compatible with the Murata 47K NTC thermistor.
> 
> This thermistor is mounted in a variety of Samsung products.
> 
> Cc: Peter Rosin <peda@axentia.se>
> Cc: Chris Lesiak <chris.lesiak@licor.com>
> Cc: devicetree@vger.kernel.org
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> The DT binding oneline is included, I see no point in splitting
> that into a separate patch.
> ---
>  Documentation/devicetree/bindings/hwmon/ntc-thermistor.yaml | 1 +
>  drivers/hwmon/ntc_thermistor.c                              | 4 ++++
>  2 files changed, 5 insertions(+)
> 

Acked-by: Rob Herring <robh@kernel.org>
Guenter Roeck Dec. 17, 2021, 3:43 p.m. UTC | #2
On Wed, Dec 15, 2021 at 06:42:41PM +0100, Linus Walleij wrote:
> This adds the Samsung 1404-001221 NTC thermistor to the
> NTC thermistor driver. As far as I can tell it is electrically
> compatible with the Murata 47K NTC thermistor.
> 
> This thermistor is mounted in a variety of Samsung products.
> 
> Cc: Peter Rosin <peda@axentia.se>
> Cc: Chris Lesiak <chris.lesiak@licor.com>
> Cc: devicetree@vger.kernel.org
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> Acked-by: Rob Herring <robh@kernel.org>

Applied.

Thanks,
Guenter

> ---
> The DT binding oneline is included, I see no point in splitting
> that into a separate patch.
> ---
>  Documentation/devicetree/bindings/hwmon/ntc-thermistor.yaml | 1 +
>  drivers/hwmon/ntc_thermistor.c                              | 4 ++++
>  2 files changed, 5 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/hwmon/ntc-thermistor.yaml b/Documentation/devicetree/bindings/hwmon/ntc-thermistor.yaml
> index 9e77cee07dbc..3d3b139a91a2 100644
> --- a/Documentation/devicetree/bindings/hwmon/ntc-thermistor.yaml
> +++ b/Documentation/devicetree/bindings/hwmon/ntc-thermistor.yaml
> @@ -76,6 +76,7 @@ properties:
>        - const: murata,ncp15wl333
>        - const: murata,ncp03wf104
>        - const: murata,ncp15xh103
> +      - const: samsung,1404-001221
>        # Deprecated "ntp," compatible strings
>        - const: ntc,ncp15wb473
>          deprecated: true
> diff --git a/drivers/hwmon/ntc_thermistor.c b/drivers/hwmon/ntc_thermistor.c
> index 00356c28e8c8..414204f5704c 100644
> --- a/drivers/hwmon/ntc_thermistor.c
> +++ b/drivers/hwmon/ntc_thermistor.c
> @@ -45,6 +45,7 @@ enum {
>  	NTC_NCP15XH103,
>  	NTC_NCP18WB473,
>  	NTC_NCP21WB473,
> +	NTC_SSG1404001221,
>  	NTC_LAST,
>  };
>  
> @@ -58,6 +59,7 @@ static const struct platform_device_id ntc_thermistor_id[] = {
>  	[NTC_NCP15XH103]      = { "ncp15xh103",      TYPE_NCPXXXH103 },
>  	[NTC_NCP18WB473]      = { "ncp18wb473",      TYPE_NCPXXWB473 },
>  	[NTC_NCP21WB473]      = { "ncp21wb473",      TYPE_NCPXXWB473 },
> +	[NTC_SSG1404001221]   = { "ssg1404-001221",  TYPE_NCPXXWB473 },
>  	[NTC_LAST]            = { },
>  };
>  
> @@ -671,6 +673,8 @@ static const struct of_device_id ntc_match[] = {
>  		.data = &ntc_thermistor_id[NTC_NCP18WB473] },
>  	{ .compatible = "murata,ncp21wb473",
>  		.data = &ntc_thermistor_id[NTC_NCP21WB473] },
> +	{ .compatible = "samsung,1404-001221",
> +		.data = &ntc_thermistor_id[NTC_SSG1404001221] },
>  
>  	/* Usage of vendor name "ntc" is deprecated */
>  	{ .compatible = "ntc,ncp03wb473",
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/hwmon/ntc-thermistor.yaml b/Documentation/devicetree/bindings/hwmon/ntc-thermistor.yaml
index 9e77cee07dbc..3d3b139a91a2 100644
--- a/Documentation/devicetree/bindings/hwmon/ntc-thermistor.yaml
+++ b/Documentation/devicetree/bindings/hwmon/ntc-thermistor.yaml
@@ -76,6 +76,7 @@  properties:
       - const: murata,ncp15wl333
       - const: murata,ncp03wf104
       - const: murata,ncp15xh103
+      - const: samsung,1404-001221
       # Deprecated "ntp," compatible strings
       - const: ntc,ncp15wb473
         deprecated: true
diff --git a/drivers/hwmon/ntc_thermistor.c b/drivers/hwmon/ntc_thermistor.c
index 00356c28e8c8..414204f5704c 100644
--- a/drivers/hwmon/ntc_thermistor.c
+++ b/drivers/hwmon/ntc_thermistor.c
@@ -45,6 +45,7 @@  enum {
 	NTC_NCP15XH103,
 	NTC_NCP18WB473,
 	NTC_NCP21WB473,
+	NTC_SSG1404001221,
 	NTC_LAST,
 };
 
@@ -58,6 +59,7 @@  static const struct platform_device_id ntc_thermistor_id[] = {
 	[NTC_NCP15XH103]      = { "ncp15xh103",      TYPE_NCPXXXH103 },
 	[NTC_NCP18WB473]      = { "ncp18wb473",      TYPE_NCPXXWB473 },
 	[NTC_NCP21WB473]      = { "ncp21wb473",      TYPE_NCPXXWB473 },
+	[NTC_SSG1404001221]   = { "ssg1404-001221",  TYPE_NCPXXWB473 },
 	[NTC_LAST]            = { },
 };
 
@@ -671,6 +673,8 @@  static const struct of_device_id ntc_match[] = {
 		.data = &ntc_thermistor_id[NTC_NCP18WB473] },
 	{ .compatible = "murata,ncp21wb473",
 		.data = &ntc_thermistor_id[NTC_NCP21WB473] },
+	{ .compatible = "samsung,1404-001221",
+		.data = &ntc_thermistor_id[NTC_SSG1404001221] },
 
 	/* Usage of vendor name "ntc" is deprecated */
 	{ .compatible = "ntc,ncp03wb473",