| Message ID | 20221214095917.964695-1-alexander.stein@ew.tq-group.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [1/7] arm64: dts: tqma8mpql: Fix temperature sensor compatible | expand |
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 > > >
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>; };
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/imx8mp-tqma8mpql-mba8mpxl.dts | 5 ++--- arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql.dtsi | 5 ++--- 2 files changed, 4 insertions(+), 6 deletions(-)