Received-SPF: pass (google.com: domain of linux-kernel+bounces-79800-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Date: Sat, 24 Feb 2024 19:16:43 +0000
From: Conor Dooley <conor@kernel.org>
To: Zev Weiss <zev@bewilderbeest.net>
Cc: Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>, Joel Stanley <joel@jms.id.au>,
	Andrew Jeffery <andrew@codeconstruct.com.au>,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-aspeed@lists.ozlabs.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] ARM: dts: aspeed: Add vendor prefixes to lm25066 compat
 strings
Message-ID: <20240224-coyness-civic-d583b863c614@spud>
References: <20240224103712.20864-2-zev@bewilderbeest.net>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
	protocol="application/pgp-signature"; boundary="uJA6p5mfN/szSZK4"
Content-Disposition: inline
In-Reply-To: <20240224103712.20864-2-zev@bewilderbeest.net>


--uJA6p5mfN/szSZK4
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sat, Feb 24, 2024 at 02:37:07AM -0800, Zev Weiss wrote:
> Due to the way i2c driver matching works (falling back to the driver's
> id_table if of_match_table fails) this didn't actually cause any
> misbehavior, but let's add the vendor prefixes so things actually work
> the way they were intended to.
>=20
> Signed-off-by: Zev Weiss <zev@bewilderbeest.net>

Oh sweet, I was just about to go off and write this patch - thanks!

Reviewed-by: Conor Dooley <conor.dooley@microchip.com>

Cheers,
Conor.

> ---
>  arch/arm/boot/dts/aspeed/aspeed-bmc-asrock-romed8hm3.dts | 4 ++--
>  arch/arm/boot/dts/aspeed/aspeed-bmc-delta-ahe50dc.dts    | 2 +-
>  2 files changed, 3 insertions(+), 3 deletions(-)
>=20
> diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-asrock-romed8hm3.dts b/a=
rch/arm/boot/dts/aspeed/aspeed-bmc-asrock-romed8hm3.dts
> index 4554abf0c7cd..9aa2de3723b5 100644
> --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-asrock-romed8hm3.dts
> +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-asrock-romed8hm3.dts
> @@ -98,14 +98,14 @@ w83773g@4c {
> =20
>  	/* IPB PMIC */
>  	lm25066@40 {
> -		compatible =3D "lm25066";
> +		compatible =3D "ti,lm25066";
>  		reg =3D <0x40>;
>  		shunt-resistor-micro-ohms =3D <1000>;
>  	};
> =20
>  	/* 12VSB PMIC */
>  	lm25066@41 {
> -		compatible =3D "lm25066";
> +		compatible =3D "ti,lm25066";
>  		reg =3D <0x41>;
>  		shunt-resistor-micro-ohms =3D <10000>;
>  	};
> diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-delta-ahe50dc.dts b/arch=
/arm/boot/dts/aspeed/aspeed-bmc-delta-ahe50dc.dts
> index 6600f7e9bf5e..e830fec0570f 100644
> --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-delta-ahe50dc.dts
> +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-delta-ahe50dc.dts
> @@ -14,7 +14,7 @@ efuse##n {					\
> =20
>  #define EFUSE(hexaddr, num)							\
>  	efuse@##hexaddr {							\
> -		compatible =3D "lm25066";						\
> +		compatible =3D "ti,lm25066";					\
>  		reg =3D <0x##hexaddr>;						\
>  		shunt-resistor-micro-ohms =3D <675>;				\
>  		regulators {							\
> --=20
> 2.43.2
>=20

--uJA6p5mfN/szSZK4
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZdpAmwAKCRB4tDGHoIJi
0jbhAQC7tzouPoNHtO1cCyXVSSpZ/vASOJWjOSJwRAbbKh4SBwEAnrxhOohokD5+
Pe/1QIdCSRpY6Zoi8eqpqGy4XCcI5gw=
=yxIC
-----END PGP SIGNATURE-----

--uJA6p5mfN/szSZK4--