* [PATCH 2/7] arm64: dts: mba8mx: Fix temperature sensor compatible
2022-12-14 9:59 [PATCH 1/7] arm64: dts: tqma8mpql: Fix temperature sensor compatible Alexander Stein
@ 2022-12-14 9:59 ` Alexander Stein
2022-12-14 9:59 ` [PATCH 3/7] arm64: dts: imx8mq-tqma8mq: " Alexander Stein
` (5 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: Alexander Stein @ 2022-12-14 9:59 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam
Cc: Alexander Stein, linux, devicetree, linux-arm-kernel
Use the correct compatible 'nxp,se97b' as it is an SE97BTP chip.
While at it, fix the node name according to device tree spec
recommendations.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
arch/arm64/boot/dts/freescale/mba8mx.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/mba8mx.dtsi b/arch/arm64/boot/dts/freescale/mba8mx.dtsi
index 446a8ec3eb6fc..94d83f20233b7 100644
--- a/arch/arm64/boot/dts/freescale/mba8mx.dtsi
+++ b/arch/arm64/boot/dts/freescale/mba8mx.dtsi
@@ -260,8 +260,8 @@ tlv320aic3x04: audio-codec@18 {
ldoin-supply = <®_vcc_3v3>;
};
- sensor1: sensor@1f {
- compatible = "nxp,se97", "jedec,jc-42.4-temp";
+ sensor1: temperator-sensor@1f {
+ compatible = "nxp,se97b", "jedec,jc-42.4-temp";
reg = <0x1f>;
};
--
2.34.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH 3/7] arm64: dts: imx8mq-tqma8mq: Fix temperature sensor compatible
2022-12-14 9:59 [PATCH 1/7] arm64: dts: tqma8mpql: Fix temperature sensor compatible Alexander Stein
2022-12-14 9:59 ` [PATCH 2/7] arm64: dts: mba8mx: " Alexander Stein
@ 2022-12-14 9:59 ` Alexander Stein
2022-12-14 9:59 ` [PATCH 4/7] arm64: dts: imx8mn-tqma8mqnl: " Alexander Stein
` (4 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: Alexander Stein @ 2022-12-14 9:59 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam
Cc: Alexander Stein, linux, devicetree, linux-arm-kernel
Use the correct compatible 'nxp,se97b' as it is an SE97BTP chip.
While at it, fix the node name according to device tree spec
recommendations. The EEPROM is a separate node anyway.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
arch/arm64/boot/dts/freescale/imx8mq-tqma8mq.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mq-tqma8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq-tqma8mq.dtsi
index 9a95e30fb42d9..5ca6b22525469 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq-tqma8mq.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq-tqma8mq.dtsi
@@ -194,8 +194,8 @@ vgen6_reg: vgen6 {
};
};
- sensor0: temperature-sensor-eeprom@1b {
- compatible = "nxp,se97", "jedec,jc-42.4-temp";
+ sensor0: temperature-sensor@1b {
+ compatible = "nxp,se97b", "jedec,jc-42.4-temp";
reg = <0x1b>;
};
--
2.34.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH 4/7] arm64: dts: imx8mn-tqma8mqnl: Fix temperature sensor compatible
2022-12-14 9:59 [PATCH 1/7] arm64: dts: tqma8mpql: Fix temperature sensor compatible Alexander Stein
2022-12-14 9:59 ` [PATCH 2/7] arm64: dts: mba8mx: " Alexander Stein
2022-12-14 9:59 ` [PATCH 3/7] arm64: dts: imx8mq-tqma8mq: " Alexander Stein
@ 2022-12-14 9:59 ` Alexander Stein
2022-12-14 9:59 ` [PATCH 5/7] arm64: dts: imx8mm-tqma8mqml: " Alexander Stein
` (3 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: Alexander Stein @ 2022-12-14 9:59 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam
Cc: Alexander Stein, linux, devicetree, linux-arm-kernel
Use the correct compatible 'nxp,se97b' as it is an SE97BTP chip.
While at it, fix the node name according to device tree spec
recommendations. The EEPROM is a separate node anyway.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
arch/arm64/boot/dts/freescale/imx8mn-tqma8mqnl.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mn-tqma8mqnl.dtsi b/arch/arm64/boot/dts/freescale/imx8mn-tqma8mqnl.dtsi
index 48b965386e879..391ca5516e4c5 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn-tqma8mqnl.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mn-tqma8mqnl.dtsi
@@ -77,8 +77,8 @@ &i2c1 {
sda-gpios = <&gpio5 15 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
status = "okay";
- sensor0: temperature-sensor-eeprom@1b {
- compatible = "nxp,se97", "jedec,jc-42.4-temp";
+ sensor0: temperature-sensor@1b {
+ compatible = "nxp,se97b", "jedec,jc-42.4-temp";
reg = <0x1b>;
};
--
2.34.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH 5/7] arm64: dts: imx8mm-tqma8mqml: Fix temperature sensor compatible
2022-12-14 9:59 [PATCH 1/7] arm64: dts: tqma8mpql: Fix temperature sensor compatible Alexander Stein
` (2 preceding siblings ...)
2022-12-14 9:59 ` [PATCH 4/7] arm64: dts: imx8mn-tqma8mqnl: " Alexander Stein
@ 2022-12-14 9:59 ` Alexander Stein
2022-12-14 9:59 ` [PATCH 6/7] ARM: dts: mba6ulx: " Alexander Stein
` (2 subsequent siblings)
6 siblings, 0 replies; 9+ messages in thread
From: Alexander Stein @ 2022-12-14 9:59 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam
Cc: Alexander Stein, linux, devicetree, linux-arm-kernel
Use the correct compatible 'nxp,se97b' as it is an SE97BTP chip.
While at it, fix the node name according to device tree spec
recommendations. The EEPROM is a separate node anyway.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
arch/arm64/boot/dts/freescale/imx8mm-tqma8mqml.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-tqma8mqml.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-tqma8mqml.dtsi
index 88d89cd46c74a..b4466a26d838a 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-tqma8mqml.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm-tqma8mqml.dtsi
@@ -87,8 +87,8 @@ &i2c1 {
sda-gpios = <&gpio5 15 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
status = "okay";
- sensor0: temperature-sensor-eeprom@1b {
- compatible = "nxp,se97", "jedec,jc-42.4-temp";
+ sensor0: temperature-sensor@1b {
+ compatible = "nxp,se97b", "jedec,jc-42.4-temp";
reg = <0x1b>;
};
--
2.34.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH 6/7] ARM: dts: mba6ulx: Fix temperature sensor compatible
2022-12-14 9:59 [PATCH 1/7] arm64: dts: tqma8mpql: Fix temperature sensor compatible Alexander Stein
` (3 preceding siblings ...)
2022-12-14 9:59 ` [PATCH 5/7] arm64: dts: imx8mm-tqma8mqml: " Alexander Stein
@ 2022-12-14 9:59 ` Alexander Stein
2022-12-14 9:59 ` [PATCH 7/7] ARM: dts: imx6ul-tqma6ul: " Alexander Stein
2022-12-15 9:27 ` [PATCH 1/7] arm64: dts: tqma8mpql: " Marco Felsch
6 siblings, 0 replies; 9+ messages in thread
From: Alexander Stein @ 2022-12-14 9:59 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam
Cc: Alexander Stein, linux, devicetree, linux-arm-kernel
Use the correct compatible 'nxp,se97b' as it is an SE97BTP chip.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
arch/arm/boot/dts/mba6ulx.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/mba6ulx.dtsi b/arch/arm/boot/dts/mba6ulx.dtsi
index b8e576ab8860d..d08f525acfe56 100644
--- a/arch/arm/boot/dts/mba6ulx.dtsi
+++ b/arch/arm/boot/dts/mba6ulx.dtsi
@@ -229,7 +229,7 @@ tlv320aic32x4: audio-codec@18 {
};
jc42: temperature-sensor@19 {
- compatible = "nxp,se97", "jedec,jc-42.4-temp";
+ compatible = "nxp,se97b", "jedec,jc-42.4-temp";
reg = <0x19>;
};
--
2.34.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH 7/7] ARM: dts: imx6ul-tqma6ul: Fix temperature sensor compatible
2022-12-14 9:59 [PATCH 1/7] arm64: dts: tqma8mpql: Fix temperature sensor compatible Alexander Stein
` (4 preceding siblings ...)
2022-12-14 9:59 ` [PATCH 6/7] ARM: dts: mba6ulx: " Alexander Stein
@ 2022-12-14 9:59 ` Alexander Stein
2022-12-15 9:27 ` [PATCH 1/7] arm64: dts: tqma8mpql: " Marco Felsch
6 siblings, 0 replies; 9+ messages in thread
From: Alexander Stein @ 2022-12-14 9:59 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam
Cc: Alexander Stein, linux, devicetree, linux-arm-kernel
Use the correct compatible 'nxp,se97b' as it is an SE97BTP chip.
While at it, fix the node name according to device tree spec
recommendations. The EEPROM is a separate node anyway.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
---
arch/arm/boot/dts/imx6ul-tqma6ul-common.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/imx6ul-tqma6ul-common.dtsi b/arch/arm/boot/dts/imx6ul-tqma6ul-common.dtsi
index eca94ed6451b0..57e647fc32374 100644
--- a/arch/arm/boot/dts/imx6ul-tqma6ul-common.dtsi
+++ b/arch/arm/boot/dts/imx6ul-tqma6ul-common.dtsi
@@ -116,8 +116,8 @@ reg_vldo4: vldo4 {
};
};
- jc42_1a: eeprom-temperature-sensor@1a {
- compatible = "nxp,se97", "jedec,jc-42.4-temp";
+ jc42_1a: eeprom-temperature@1a {
+ compatible = "nxp,se97b", "jedec,jc-42.4-temp";
reg = <0x1a>;
};
--
2.34.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH 1/7] arm64: dts: tqma8mpql: Fix temperature sensor compatible
2022-12-14 9:59 [PATCH 1/7] arm64: dts: tqma8mpql: Fix temperature sensor compatible Alexander Stein
` (5 preceding siblings ...)
2022-12-14 9:59 ` [PATCH 7/7] ARM: dts: imx6ul-tqma6ul: " Alexander Stein
@ 2022-12-15 9:27 ` Marco Felsch
6 siblings, 0 replies; 9+ messages in thread
From: Marco Felsch @ 2022-12-15 9:27 UTC (permalink / raw)
To: Alexander Stein
Cc: Rob Herring, Krzysztof Kozlowski, Shawn Guo, Sascha Hauer,
Pengutronix Kernel Team, Fabio Estevam, devicetree,
linux-arm-kernel, linux
Hi Alexander,
On 22-12-14, Alexander Stein wrote:
> Use the correct compatible 'nxp,se97b' as it is an SE97BTP chip.
> While at it, fix the node name according to device tree spec
> recommendations. The EEPROM is a separate node anyway.
Just FTR: sometimes other components like bootloaders use the node names
as well, therefore changing node names should be done with caution. For
this series it is okay I think.
>
> Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
The whole series lgtm, feel free to add my.
Reviewed-by: Marco Felsch <m.felsch@pengutronix.de>
> ---
> arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl.dts | 5 ++---
> arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql.dtsi | 5 ++---
> 2 files changed, 4 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl.dts b/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl.dts
> index a3340814099aa..f7f657b007a18 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl.dts
> @@ -516,9 +516,8 @@ tlv320aic3x04: audio-codec@18 {
> ldoin-supply = <®_vcc_3v3>;
> };
>
> - /* NXP SE97BTP with temperature sensor + eeprom */
> - se97_1c: temperature-sensor-eeprom@1c {
> - compatible = "nxp,se97", "jedec,jc-42.4-temp";
> + se97_1c: temperature-sensor@1c {
> + compatible = "nxp,se97b", "jedec,jc-42.4-temp";
> reg = <0x1c>;
> };
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql.dtsi b/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql.dtsi
> index 7bd680a926ce7..ebc29a950ba9a 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql.dtsi
> @@ -63,9 +63,8 @@ &i2c1 {
> sda-gpios = <&gpio5 15 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
> status = "okay";
>
> - /* NXP SE97BTP with temperature sensor + eeprom */
> - se97: temperature-sensor-eeprom@1b {
> - compatible = "nxp,se97", "jedec,jc-42.4-temp";
> + se97: temperature-sensor@1b {
> + compatible = "nxp,se97b", "jedec,jc-42.4-temp";
> reg = <0x1b>;
> };
>
> --
> 2.34.1
>
>
>
^ permalink raw reply [flat|nested] 9+ messages in thread