-----Original Message-----
From: Andrew Jeffery <andrew@codeconstruct.com.au>
Sent: Friday, September 27, 2024 2:01 PM
To: Delphine_CC_Chiu/WYHQ/Wiwynn <Delphine_CC_Chiu@wiwynn.com>;
patrick at stwcx.xyz; Rob Herring [off-list ref]; Krzysztof Kozlowski
[off-list ref]; Conor Dooley [off-list ref]; Joel Stanley
[off-list ref]
Cc: Ricky CX Wu <redacted>;
devicetree at vger.kernel.org; linux-arm-kernel at lists.infradead.org;
linux-aspeed at lists.ozlabs.org; linux-kernel at vger.kernel.org
Subject: Re: [PATCH v1] ARM: dts: aspeed: yosemite4: correct the compatible
string of adm1272

 [External Sender]

 [External Sender]

On Thu, 2024-09-26 at 09:34 +0800, Delphine CC Chiu wrote:

quoted

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?

Sure, I'll fixing the wording in v2.

quoted

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

Would like to ask where should I add the Fixes: tag?
Should I add in the patch title like:
Fixes: ARM: dts: aspeed: yosemite4: correct the compatible string of adm1272

Or should I add in the commit message?