linux-kernel - Re: [PATCH v2 00/10] Revise Meta(Facebook) Harma BMC(AST2600)

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: <171949936638.3312359.3954253966631367606.robh@kernel.org>
Date: Thu, 27 Jun 2024 08:48:47 -0600
From: "Rob Herring (Arm)" <robh@...nel.org>
To: Peter Yin <peteryin.openbmc@...il.com>
Cc: Andrew Jeffery <andrew@...econstruct.com.au>, patrick@...cx.xyz, 
 linux-arm-kernel@...ts.infradead.org, Conor Dooley <conor+dt@...nel.org>, 
 linux-aspeed@...ts.ozlabs.org, amithash@...a.com, 
 linux-kernel@...r.kernel.org, devicetree@...r.kernel.org, 
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Joel Stanley <joel@....id.au>
Subject: Re: [PATCH v2 00/10] Revise Meta(Facebook) Harma BMC(AST2600)


On Thu, 27 Jun 2024 15:00:02 +0800, Peter Yin wrote:
> Summary:
> Revise linux device tree entry related to Meta(Facebook) Harma
> specific devices connected to BMC(AST2600) SoC.
> 
> Base on: https://lore.kernel.org/all/CAPSyxFRj0twCJG6Lr5UZpznrUHyd_L0Reo=kZSFwCw3FNQ+x+A@mail.gmail.com/
> 
> Change log:
> 
> v1 -> v2
>   - Patch 0007  - add fru device
>   - Patch 0008  - add temperature device
>   - Patch 0009  - enable mctp controller
>   - Patch 00010 - fixed dtb_check warning
> 
> v1
>   - Patch 0001 - revise hsc chip
>   - Patch 0002 - add VR device
>   - Patch 0003 - add sgpio name
>   - Patch 0004 - add ina238
>   - Patch 0005 - add power monitor xdp710
>   - Patch 0006 - remove multi-host property
>   - Patch 0007 - remove pca9546
> 
> Peter Yin (10):
>   ARM: dts: aspeed: Harma: revise hsc chip
>   ARM: dts: aspeed: Harma: add VR device
>   ARM: dts: aspeed: Harma: add sgpio name
>   ARM: dts: aspeed: Harma: add ina238
>   ARM: dts: aspeed: Harma: add power monitor xdp710
>   ARM: dts: aspeed: Harma: remove multi-host property
>   ARM: dts: aspeed: Harma: add fru device
>   ARM: dts: aspeed: Harma: add temperature device
>   ARM: dts: aspeed: Harma: enable mctp controller
>   ARM: dts: aspeed: Harma: fixed dtb_check warning
> 
>  .../dts/aspeed/aspeed-bmc-facebook-harma.dts  | 146 ++++++++++++++++--
>  1 file changed, 132 insertions(+), 14 deletions(-)
> 
> --
> 2.25.1
> 
> 
> 


My bot found new DTB warnings on the .dts files added or changed in this
series.

Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
are fixed by another series. Ultimately, it is up to the platform
maintainer whether these warnings are acceptable or not. No need to reply
unless the platform maintainer has comments.

If you already ran DT checks and didn't see these error(s), then
make sure dt-schema is up to date:

  pip3 install dtschema --upgrade


New warnings running 'make CHECK_DTBS=y aspeed/aspeed-bmc-facebook-harma.dtb' for 20240627070013.2509150-1-peteryin.openbmc@...il.com:

arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-harma.dtb: ftgmac@...90000: Unevaluated properties are not allowed ('#address-cells', '#size-cells' were unexpected)
	from schema $id: http://devicetree.org/schemas/net/faraday,ftgmac100.yaml#
arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-harma.dtb: /ahb/apb/bus@...8a000/i2c@.../i2c-mux@...i2c@...ower-monitor@61: failed to match any schema with compatible: ['isil,isl69260']
arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-harma.dtb: /ahb/apb/bus@...8a000/i2c@.../i2c-mux@...i2c@...ower-monitor@62: failed to match any schema with compatible: ['isil,isl69260']
arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-harma.dtb: /ahb/apb/bus@...8a000/i2c@.../i2c-mux@...i2c@...ower-monitor@63: failed to match any schema with compatible: ['isil,isl69260']