]> www.infradead.org Git - linux.git/commitdiff
ARM: dts: Fix undocumented LM75 compatible nodes
authorRob Herring <robh@kernel.org>
Fri, 16 Aug 2024 16:47:14 +0000 (10:47 -0600)
committerArnd Bergmann <arnd@arndb.de>
Thu, 5 Sep 2024 14:37:43 +0000 (14:37 +0000)
"lm75" without any vendor is undocumented. It works with the Linux
kernel since the I2C subsystem will do matches of the compatible string
without a vendor prefix to the i2c_device_id and/or driver name.

Mostly replace "lm75" with "national,lm75" as that's the original part
vendor and the compatible which matches what "lm75" matched with. In a
couple of cases the node name or compatible gives a clue to the actual
part and vendor and a more specific compatible can be used. In these
cases, it does change the variant the kernel picks.

"nct75" is an OnSemi part which is compatible with TI TMP75C based on
a comparison of the OnSemi NCT75 datasheet and configuration the Linux
driver uses. Adding an OnSemi compatible would be an ABI change.

"nxp,lm75" is most likely an NXP part. Alexander Stein says the i.MX53
boards are a NXP LM75A as well. NXP makes a LM75A and LM75B. Both are
11-bit resolution and 100ms sample time. The "national,lm75a" is
9-bit, so "national,lm75b" is the closest match for both NXP variants.

While we're here, fix the node names to use the generic name
"temperature-sensor".

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Kevin Hilman <khilman@baylibre.com> # am335x-nano.dts
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.com> # imx53-mba53.dts, imx53-tqma53.dtsi
Link: https://lore.kernel.org/r/20240816164717.1585629-1-robh@kernel.org
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-greatlakes.dts
arch/arm/boot/dts/intel/socfpga/socfpga_cyclone5_vining_fpga.dts
arch/arm/boot/dts/marvell/armada-385-clearfog-gtr.dtsi
arch/arm/boot/dts/nuvoton/nuvoton-npcm730-kudo.dts
arch/arm/boot/dts/nuvoton/nuvoton-npcm750-evb.dts
arch/arm/boot/dts/nxp/imx/imx53-mba53.dts
arch/arm/boot/dts/nxp/imx/imx53-tqma53.dtsi
arch/arm/boot/dts/nxp/lpc/lpc4357-ea4357-devkit.dts
arch/arm/boot/dts/nxp/lpc/lpc4357-myd-lpc4357.dts
arch/arm/boot/dts/ti/omap/am335x-nano.dts
arch/arm/boot/dts/xilinx/zynq-zturn-common.dtsi

index 998598c15fd084ab569948ba274c1a9518255fe6..49914a4a179fa2e03df25e38338576bf547543bf 100644 (file)
 &i2c12 {
        status = "okay";
        temperature-sensor@4f {
-               compatible = "lm75";
+               compatible = "national,lm75";
                reg = <0x4f>;
        };
 };
index 65f390bf897534c89d558bda0bd2307ffca5bab1..84f39dec3c42508c6dd57cc09fa4d176194f8e79 100644 (file)
                #gpio-cells = <2>;
        };
 
-       temp: lm75@48 {
-               compatible = "lm75";
+       temp: temperature-sensor@48 {
+               compatible = "national,lm75";
                reg = <0x48>;
        };
 
index f3a3cb6ac31148360d84b2231a43c7ca5bb015de..8208c6a9627a8cee2ed584d21b87c53888742307 100644 (file)
        status = "okay";
 
        /* U26 temperature sensor placed near SoC */
-       temp1: nct75@4c {
-               compatible = "lm75";
+       temp1: temperature-sensor@4c {
+               compatible = "ti,tmp75c";
                reg = <0x4c>;
        };
 
        /* U27 temperature sensor placed near RTC battery */
-       temp2: nct75@4d {
-               compatible = "lm75";
+       temp2: temperature-sensor@4d {
+               compatible = "ti,tmp75c";
                reg = <0x4d>;
        };
 
index 1f07ba38291087879f09b2855c439a7123dafcb6..886a87dfcd0d831b5a1c241bc20cb4ff9a9afa8b 100644 (file)
                        reg = <4>;
 
                        // INLET1_T
-                       lm75@5c {
-                               compatible = "ti,lm75";
+                       temperature-sensor@5c {
+                               compatible = "national,lm75";
                                reg = <0x5c>;
                        };
                };
                        reg = <5>;
 
                        // OUTLET1_T
-                       lm75@5c {
-                               compatible = "ti,lm75";
+                       temperature-sensor@5c {
+                               compatible = "national,lm75";
                                reg = <0x5c>;
                        };
                };
                        reg = <6>;
 
                        // OUTLET2_T
-                       lm75@5c {
-                               compatible = "ti,lm75";
+                       temperature-sensor@5c {
+                               compatible = "national,lm75";
                                reg = <0x5c>;
                        };
                };
                        reg = <7>;
 
                        // OUTLET3_T
-                       lm75@5c {
-                               compatible = "ti,lm75";
+                       temperature-sensor@5c {
+                               compatible = "national,lm75";
                                reg = <0x5c>;
                        };
                };
                        reg = <3>;
 
                        // M2_ZONE_T
-                       lm75@28 {
-                               compatible = "ti,lm75";
+                       temperature-sensor@28 {
+                               compatible = "national,lm75";
                                reg = <0x28>;
                        };
                };
                        reg = <4>;
 
                        // BATT_ZONE_T
-                       lm75@29 {
-                               compatible = "ti,lm75";
+                       temperature-sensor@29 {
+                               compatible = "national,lm75";
                                reg = <0x29>;
                        };
                };
                        reg = <5>;
 
                        // NBM1_ZONE_T
-                       lm75@28 {
-                               compatible = "ti,lm75";
+                       temperature-sensor@28 {
+                               compatible = "national,lm75";
                                reg = <0x28>;
                        };
                };
                        reg = <6>;
 
                        // NBM2_ZONE_T
-                       lm75@29 {
-                               compatible = "ti,lm75";
+                       temperature-sensor@29 {
+                               compatible = "national,lm75";
                                reg = <0x29>;
                        };
                };
index f53d45fa1de8788bae45f557e2899d528b3b1144..bcdcb30c7bf6e51766640f0e9580f656e3f5db00 100644 (file)
        clock-frequency = <100000>;
        status = "okay";
        lm75@48 {
-               compatible = "lm75";
+               compatible = "national,lm75";
                reg = <0x48>;
                status = "okay";
        };
 &i2c1 {
        clock-frequency = <100000>;
        status = "okay";
-       lm75@48 {
-               compatible = "lm75";
+       temperature-sensor@48 {
+               compatible = "national,lm75";
                reg = <0x48>;
                status = "okay";
        };
index 2117de872703bfdf730f1eacedc653dcaaf8650c..0d336cbdb4513e8603f78899798ae78fc50709b7 100644 (file)
                gpio-controller;
        };
 
-       sensor2: lm75@49 {
-               compatible = "lm75";
+       sensor2: temperature-sensor@49 {
+               compatible = "national,lm75b";
                reg = <0x49>;
        };
 };
index b2d7271d1d24cc4e3aebf206569725411e040913..c34ee84bd71675569753ac16c6358143b1b70e6e 100644 (file)
                interrupts = <6 4>; /* PATA_DATA6, active high */
        };
 
-       sensor1: lm75@48 {
-               compatible = "lm75";
+       sensor1: temperature-sensor@48 {
+               compatible = "national,lm75b";
                reg = <0x48>;
        };
 
index 224f80a4a31d800ec4896539d63eaa28ea42bf32..4aefbc01dfc0fe7abbf87a8cdd0d8da179aae3d5 100644 (file)
                reg = <0x1d>;
        };
 
-       lm75@48 {
-               compatible = "nxp,lm75";
+       temperature-sensor@48 {
+               compatible = "national,lm75b";
                reg = <0x48>;
        };
 
index 1f84654df50c38c717a531bb7c5cb41bde98fb92..846afb8ccbf1de08cccaf445106beeeddee93454 100644 (file)
        clock-frequency = <400000>;
 
        sensor@49 {
-               compatible = "lm75";
+               compatible = "national,lm75";
                reg = <0x49>;
        };
 
index 26b5510cb3d1661efd0584d149ea2e59aeb97111..56929059f5af093ee86442cc5c06ae51b332c9e6 100644 (file)
        };
 
        temperature-sensor@48 {
-               compatible = "lm75";
+               compatible = "national,lm75";
                reg = <0x48>;
        };
 
index dfb1fbafe3aa4e78927c5da7a83878e079fd4b7d..33b02e05ce827309ffdca2643b67cae44d8c74c4 100644 (file)
@@ -97,9 +97,9 @@
        status = "okay";
        clock-frequency = <400000>;
 
-       stlm75@49 {
+       temperature-sensor@49 {
                status = "okay";
-               compatible = "lm75";
+               compatible = "st,stlm75";
                reg = <0x49>;
        };