linux-kernel - Re: [PATCH v8 2/2] ARM: dts: aspeed: clemente: add Meta Clemente BMC

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <c980c6c55f3a4914f1393498763bdf9cfb109ad2.camel@codeconstruct.com.au>
Date: Mon, 21 Jul 2025 13:22:19 +0930
From: Andrew Jeffery <andrew@...econstruct.com.au>
To: Leo Wang <leo.jt.wang@...il.com>, Rob Herring <robh@...nel.org>, 
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, Joel Stanley <joel@....id.au>, Kees Cook
 <kees@...nel.org>, Tony Luck <tony.luck@...el.com>,  "Guilherme G. Piccoli"
 <gpiccoli@...lia.com>, Geert Uytterhoeven <geert+renesas@...der.be>, Magnus
 Damm <magnus.damm@...il.com>
Cc: devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, 
	linux-aspeed@...ts.ozlabs.org, linux-kernel@...r.kernel.org, 
	linux-hardening@...r.kernel.org, linux-renesas-soc@...r.kernel.org, 
	leo.jt.wang@...-foxconn.com, george.kw.lee@...-foxconn.com, 
	bruce.jy.hung@...-foxconn.com
Subject: Re: [PATCH v8 2/2] ARM: dts: aspeed: clemente: add Meta Clemente BMC

On Thu, 2025-07-17 at 22:59 +0800, Leo Wang wrote:
> From: Leo Wang <leo.jt.wang@...il.com>
> 
> Add linux device tree entry for Meta Clemente compute-tray
> BMC using AST2600 SoC.
> 
> Signed-off-by: Leo Wang <leo.jt.wang@...il.com>
> 

[...]

> +       };
> +
> +// PDB TEMP SENSOR

This comment should be indented appropriately?

> +       temperature-sensor@4e {
> +               compatible = "ti,tmp1075";
> +               reg = <0x4e>;
> +       };
> 

[...]

> +
> +&i2c11 {
> +       status = "okay";
> +       aspeed,enable-byte;

This is not a property specified in any accepted binding, please drop
it.

   arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-clemente.dtb: i2c@600 (aspeed,ast2600-i2c-bus): Unevaluated properties are not allowed ('aspeed,enable-byte' was unexpected)
   	from schema $id: http://devicetree.org/schemas/i2c/aspeed,i2c.yaml#

> +
> +       ssif-bmc@10 {
> +               compatible = "ssif-bmc";
> +               reg = <0x10>;
> +       };
> +};
> 

[...]

> +
> diff --git a/arch/arm/boot/dts/aspeed/aspeed-g6-pinctrl.dtsi b/arch/arm/boot/dts/aspeed/aspeed-g6-pinctrl.dtsi
> index 289668f051eb4271ac48ae3ce9b82587911548ee..61b1d1c5040c820f8c995132739becde80e069bb 100644
> --- a/arch/arm/boot/dts/aspeed/aspeed-g6-pinctrl.dtsi
> +++ b/arch/arm/boot/dts/aspeed/aspeed-g6-pinctrl.dtsi
> @@ -412,6 +412,16 @@ pinctrl_mdio4_default: mdio4_default {
>                 groups = "MDIO4";
>         };
>  
> +       pinctrl_ncsi3_default: ncsi3_default {
> +               function = "RMII3";
> +               groups = "NCSI3";
> +       };
> +
> +       pinctrl_ncsi4_default: ncsi4_default {
> +               function = "RMII4";
> +               groups = "NCSI4";
> +       };
> +

Can you please make this a separate patch?

Thanks,

Andrew