Re: [PATCH v1] ARM: dts: aspeed: yosemite4: correct the compatible string of adm1272
From: Andrew Jeffery <andrew@codeconstruct.com.au>
Date: 2024-09-27 06:00:38
Also in:
linux-arm-kernel, linux-aspeed, linux-devicetree
On Thu, 2024-09-26 at 09:34 +0800, Delphine CC Chiu wrote:
From: Ricky CX Wu <redacted> Remove the redundant space in the compatible string of adm1272.
In this case the space is not redundant, it's just incorrect, the compatible string doesn't match any specified. Do you mind fixing the wording?
Signed-off-by: Ricky CX Wu <redacted> Signed-off-by: Delphine CC Chiu <Delphine_CC_Chiu@wiwynn.com>
Can you please add an appropriate Fixes: tag? Thanks, Andrew
quoted hunk ↗ jump to hunk
--- arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dts index 98477792aa00..7ed76cd4fd2d 100644 --- a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dts +++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dts@@ -284,12 +284,12 @@ &i2c10 { &i2c11 { status = "okay"; power-sensor@10 { - compatible = "adi, adm1272"; + compatible = "adi,adm1272"; reg = <0x10>; }; power-sensor@12 { - compatible = "adi, adm1272"; + compatible = "adi,adm1272"; reg = <0x12>; };