From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 2FADEC52D71 for ; Fri, 9 Aug 2024 07:52:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Type: Content-Transfer-Encoding:MIME-Version:References:In-Reply-To:Message-ID:Date :Subject:Cc:To:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=qEKb2tBDFYtPQQYes4QRP4tJM5p3ULThCqzPcDFNfNc=; b=lezm60M3oCmaN5dQGKOuwbnkJk ztlkyX4b8XSpe9cmdjf8QPl3xYOA6mZ4stOR+hfTe93nqTVmgXp7z5XzZJE8mTdCV7US0s0cSBU1M G2VMt0EFUPNVWFahpHTb4qsU9nh8E4H1wOoYNBxl9JtTY1BPkHfHjAo+OQxp8btd2JSS2DOzfYkKp zAK+sQnlWwkBeZ+n9HaIxUIbL3wMfwdP2Js9QAV2Mri0CFw5r14M2gDzA5M3qmDm3uXy5H2wD4jcr DxTIAyl1Xwgh6MyDm9+x8gUjkrcdaKElU74VmNxQhimDDUIn1ieaBCA3duS4DXFchNeQo5jrPocTu 9DUIkLiQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1scKPh-0000000Ac01-1nis; Fri, 09 Aug 2024 07:51:53 +0000 Received: from mx1.tq-group.com ([93.104.207.81]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1scKP7-0000000AbuR-3KQd for linux-arm-kernel@lists.infradead.org; Fri, 09 Aug 2024 07:51:19 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1723189877; x=1754725877; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=qEKb2tBDFYtPQQYes4QRP4tJM5p3ULThCqzPcDFNfNc=; b=FYFr8wsxEQm3Wv8lx8hwvA40dezna8GHoVGCtLLqW1OB4N2gR/SpH269 N3O20z1xkjCeR0HsJ39xu+sSkKf/Z43qOeEDF2XinSaDL8IZbxU8aFBZN s2t9Gf8m7dzCx6SbvCjEQVTHrEHdbvxJSGt7HyVLKqQAsHH7skeL6rTNK s1AOJ60qTMKSfjlcic3lBwJJHXL20lqd33o8PSAuFBXcmkDg+qRUGwOss ZkkragJk4LcKpZGs92s8KIp6B+OSNiI/IYSVNMsCEhRcFKwtHIqCOe8MV xiRt3qeF3ZnetNmjAmqXV1o6Ad41HXfbC8W3uYrYtgAGyD4iGabNA4y// g==; X-CSE-ConnectionGUID: JwcAz+0XR/aH896pBitJDw== X-CSE-MsgGUID: kVdMZKR1RUmNWIVSMJj3Ag== X-IronPort-AV: E=Sophos;i="6.09,275,1716242400"; d="scan'208";a="38329660" Received: from vmailcow01.tq-net.de ([10.150.86.48]) by mx1.tq-group.com with ESMTP; 09 Aug 2024 09:51:11 +0200 X-CheckPoint: {66B5CA6F-0-F0206917-F54EFEED} X-MAIL-CPID: 27227B5B5898C08BF0A2BD19511608BD_4 X-Control-Analysis: str=0001.0A782F1A.66B5CA6F.0103,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0 Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id E6E8C163A4B; Fri, 9 Aug 2024 09:51:01 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ew.tq-group.com; s=dkim; t=1723189866; h=from:subject:date:message-id:to:cc:mime-version:content-type: content-transfer-encoding:in-reply-to:references; bh=qEKb2tBDFYtPQQYes4QRP4tJM5p3ULThCqzPcDFNfNc=; b=AA13aPQoadQSDdUv+s7cijsOmK61bm3M1gbgjqsMZ1SJ/kkwz2qelWR3Ov56morwxcIApY F3Q3T8cakL6YGCgH2h3benXKQ95OuY7vyPe//NM/2IDJK+B/4BJDZdjaHC1j8V/o33C6Yx e5LNf5Ol3x7oh+oGRPf0zO+jVnUSo8XooBtvWpaWHjz8qkWHq2s6UQJLrtFgF9Zo8bVY48 NPiXjXUklM5xqEhpQQ8xesdLf0RsYPK8wpNZPuG83hODi6SF6ayNMdSjoopRyCd93YJOU9 NFUoodsMHZbTFHBp5XMbTQJ7hHGOJqshzycYN8qA3+fPZYeXqYriYg2foLyPng== From: Alexander Stein To: soc@kernel.org, Krzysztof Kozlowski , Conor Dooley , Joel Stanley , Andrew Jeffery , Dinh Nguyen , Andrew Lunn , Gregory Clement , Sebastian Hesselbarth , Avi Fishman , Tomer Maimon , Tali Perry , Patrick Venture , Nancy Yuen , Benjamin Fair , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , Vladimir Zapolskiy , Mark Jackson , Tony Lindgren , Michal Simek , linux-arm-kernel@lists.infradead.org Cc: devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-aspeed@lists.ozlabs.org, linux-kernel@vger.kernel.org, openbmc@lists.ozlabs.org, imx@lists.linux.dev, linux-omap@vger.kernel.org, "Rob Herring (Arm)" Subject: Re: [PATCH] ARM: dts: Fix undocumented LM75 compatible nodes Date: Fri, 09 Aug 2024 09:51:03 +0200 Message-ID: <22384730.EfDdHjke4D@steina-w> Organization: TQ-Systems GmbH In-Reply-To: <20240808164941.1407327-1-robh@kernel.org> References: <20240808164941.1407327-1-robh@kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="iso-8859-1" X-Last-TLS-Session-Version: TLSv1.3 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240809_005118_354905_5B8791C5 X-CRM114-Status: GOOD ( 21.80 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Am Donnerstag, 8. August 2024, 18:49:38 CEST schrieb Rob Herring (Arm): > "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. >=20 > 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. >=20 > "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. >=20 > "nxp,lm75" is most likely an NXP part. NXP makes a LM75A and LM75B. > Both are 11-bit resolution and 100ms sample time, so "national,lm75b" is > the closest match. >=20 > While we're here, fix the node names to use the generic name > "temperature-sensor". >=20 > Signed-off-by: Rob Herring (Arm) > --- > SoC maintainers, Please take this directly. > --- > .../aspeed/aspeed-bmc-facebook-greatlakes.dts | 2 +- > .../socfpga/socfpga_cyclone5_vining_fpga.dts | 4 +-- > .../dts/marvell/armada-385-clearfog-gtr.dtsi | 8 ++--- > .../boot/dts/nuvoton/nuvoton-npcm730-kudo.dts | 32 +++++++++---------- > .../boot/dts/nuvoton/nuvoton-npcm750-evb.dts | 6 ++-- > arch/arm/boot/dts/nxp/imx/imx53-mba53.dts | 4 +-- > arch/arm/boot/dts/nxp/imx/imx53-tqma53.dtsi | 4 +-- > .../dts/nxp/lpc/lpc4357-ea4357-devkit.dts | 4 +-- > .../boot/dts/nxp/lpc/lpc4357-myd-lpc4357.dts | 2 +- > arch/arm/boot/dts/ti/omap/am335x-nano.dts | 2 +- > .../boot/dts/xilinx/zynq-zturn-common.dtsi | 4 +-- > 11 files changed, 36 insertions(+), 36 deletions(-) >=20 < [snip] > diff --git a/arch/arm/boot/dts/nxp/imx/imx53-mba53.dts b/arch/arm/boot/dt= s/nxp/imx/imx53-mba53.dts > index 2117de872703..d155b3ec22ef 100644 > --- a/arch/arm/boot/dts/nxp/imx/imx53-mba53.dts > +++ b/arch/arm/boot/dts/nxp/imx/imx53-mba53.dts > @@ -175,8 +175,8 @@ expander: pca9554@20 { > gpio-controller; > }; > =20 > - sensor2: lm75@49 { > - compatible =3D "lm75"; > + sensor2: temperature-sensor@49 { > + compatible =3D "national,lm75"; I checked the old schematics. This is an NXP LM75A, so 'national,lm75a' would be the correct compatible. > reg =3D <0x49>; > }; > }; > diff --git a/arch/arm/boot/dts/nxp/imx/imx53-tqma53.dtsi b/arch/arm/boot/= dts/nxp/imx/imx53-tqma53.dtsi > index b2d7271d1d24..d01c3aee0272 100644 > --- a/arch/arm/boot/dts/nxp/imx/imx53-tqma53.dtsi > +++ b/arch/arm/boot/dts/nxp/imx/imx53-tqma53.dtsi > @@ -254,8 +254,8 @@ pmic: mc34708@8 { > interrupts =3D <6 4>; /* PATA_DATA6, active high */ > }; > =20 > - sensor1: lm75@48 { > - compatible =3D "lm75"; > + sensor1: temperature-sensor@48 { > + compatible =3D "national,lm75"; I checked the old schematics. This is an NXP LM75A, so 'national,lm75a' would be the correct compatible. Best regards, Alexander > reg =3D <0x48>; > }; > =20 > [snip] =2D-=20 TQ-Systems GmbH | M=FChlstra=DFe 2, Gut Delling | 82229 Seefeld, Germany Amtsgericht M=FCnchen, HRB 105018 Gesch=E4ftsf=FChrer: Detlef Schneider, R=FCdiger Stahl, Stefan Schneider http://www.tq-group.com/