From: "Rob Herring (Arm)" <robh@kernel.org>
To: Peter Yin <peteryin.openbmc@gmail.com>
Cc: Andrew Jeffery <andrew@codeconstruct.com.au>,
patrick@stwcx.xyz, linux-arm-kernel@lists.infradead.org,
Conor Dooley <conor+dt@kernel.org>,
linux-aspeed@lists.ozlabs.org, amithash@meta.com,
linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Joel Stanley <joel@jms.id.au>
Subject: Re: [PATCH v2 00/10] Revise Meta(Facebook) Harma BMC(AST2600)
Date: Thu, 27 Jun 2024 08:48:47 -0600 [thread overview]
Message-ID: <171949936638.3312359.3954253966631367606.robh@kernel.org> (raw)
In-Reply-To: <20240627070013.2509150-1-peteryin.openbmc@gmail.com>
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@gmail.com:
arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-harma.dtb: ftgmac@1e690000: 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@1e78a000/i2c@700/i2c-mux@70/i2c@0/power-monitor@61: failed to match any schema with compatible: ['isil,isl69260']
arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-harma.dtb: /ahb/apb/bus@1e78a000/i2c@700/i2c-mux@70/i2c@0/power-monitor@62: failed to match any schema with compatible: ['isil,isl69260']
arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-harma.dtb: /ahb/apb/bus@1e78a000/i2c@700/i2c-mux@70/i2c@0/power-monitor@63: failed to match any schema with compatible: ['isil,isl69260']
prev parent reply other threads:[~2024-06-27 14:48 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-27 7:00 Peter Yin
2024-06-27 7:00 ` [PATCH v2 01/10] ARM: dts: aspeed: Harma: revise hsc chip Peter Yin
2024-06-27 7:00 ` [PATCH v2 02/10] ARM: dts: aspeed: Harma: add VR device Peter Yin
2024-06-27 7:00 ` [PATCH v2 03/10] ARM: dts: aspeed: Harma: add sgpio name Peter Yin
2024-06-27 7:00 ` [PATCH v2 04/10] ARM: dts: aspeed: Harma: add ina238 Peter Yin
2024-06-27 7:00 ` [PATCH v2 05/10] ARM: dts: aspeed: Harma: add power monitor xdp710 Peter Yin
2024-06-27 7:00 ` [PATCH v2 06/10] ARM: dts: aspeed: Harma: remove multi-host property Peter Yin
2024-06-27 7:00 ` [PATCH v2 07/10] ARM: dts: aspeed: Harma: add fru device Peter Yin
2024-06-27 7:00 ` [PATCH v2 08/10] ARM: dts: aspeed: Harma: add temperature device Peter Yin
2024-06-27 7:00 ` [PATCH v2 09/10] ARM: dts: aspeed: Harma: enable mctp controller Peter Yin
2024-06-27 7:00 ` [PATCH v2 10/10] ARM: dts: aspeed: Harma: fixed dtb_check warning Peter Yin
2024-06-27 14:48 ` Rob Herring (Arm) [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=171949936638.3312359.3954253966631367606.robh@kernel.org \
--to=robh@kernel.org \
--cc=amithash@meta.com \
--cc=andrew@codeconstruct.com.au \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=joel@jms.id.au \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-aspeed@lists.ozlabs.org \
--cc=linux-kernel@vger.kernel.org \
--cc=patrick@stwcx.xyz \
--cc=peteryin.openbmc@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®