[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID:
<SEYPR06MB5134F8A2030833C17176A8E69D5DA@SEYPR06MB5134.apcprd06.prod.outlook.com>
Date: Mon, 21 Jul 2025 03:47:01 +0000
From: Jacky Chou <jacky_chou@...eedtech.com>
To: Krzysztof Kozlowski <krzk@...nel.org>
CC: "bhelgaas@...gle.com" <bhelgaas@...gle.com>, "lpieralisi@...nel.org"
<lpieralisi@...nel.org>, "kwilczynski@...nel.org" <kwilczynski@...nel.org>,
"mani@...nel.org" <mani@...nel.org>, "robh@...nel.org" <robh@...nel.org>,
"krzk+dt@...nel.org" <krzk+dt@...nel.org>, "conor+dt@...nel.org"
<conor+dt@...nel.org>, "joel@....id.au" <joel@....id.au>,
"andrew@...econstruct.com.au" <andrew@...econstruct.com.au>,
"linux-aspeed@...ts.ozlabs.org" <linux-aspeed@...ts.ozlabs.org>,
"linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>, "linux-kernel@...r.kernel.org"
<linux-kernel@...r.kernel.org>, "openbmc@...ts.ozlabs.org"
<openbmc@...ts.ozlabs.org>, "linux-gpio@...r.kernel.org"
<linux-gpio@...r.kernel.org>, "linus.walleij@...aro.org"
<linus.walleij@...aro.org>, "p.zabel@...gutronix.de"
<p.zabel@...gutronix.de>, BMC-SW <BMC-SW@...eedtech.com>
Subject:
回覆: [PATCH v2 01/10] dt-bindings: soc: aspeed: Add ASPEED PCIe Config support
Hi Krzysztof,
Thank you for your reply.
> > +maintainers:
> > + - Jacky Chou <jacky_chou@...eedtech.com>
> > +
> > +description: |
>
> Drop |
>
Agreed.
> > + The ASPEED PCIe configuration syscon block provides a set of
> > + registers shared by multiple PCIe-related devices within the SoC.
> > + This node represents the common configuration space that allows
> > + these devices to coordinate and manage shared PCIe settings,
> > + including address mapping, control, and status registers. The
> > + syscon interface enables for various PCIe devices to access and modify
> these shared registers in a consistent and centralized manner.
> > +
> > +properties:
> > + compatible:
> > + items:
> > + - enum:
> > + - aspeed,pcie-cfg
>
> NAK, see writing bindings. You already received comments about generic
> compatible in the past.
>
I understand the generic aspeed,pcie-cfg is not acceptable per the binding guidelines.
I will update it in the next version to use a more specific name like aspeed,ast2600-pciecfg.
Thanks again for your guidance.
Thanks,
Jacky
Powered by blists - more mailing lists