linux-kernel - 回覆: [PATCH net-next v4 1/7] dt-bindings: net: ftgmac100: support for AST2700

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] [thread-next>] [day] [month] [year] [list]

Message-ID:
 <SEYPR06MB5134245024FB21E4D823ADCE9D312@SEYPR06MB5134.apcprd06.prod.outlook.com>
Date: Fri, 6 Dec 2024 01:57:33 +0000
From: Jacky Chou <jacky_chou@...eedtech.com>
To: Krzysztof Kozlowski <krzk@...nel.org>
CC: "andrew+netdev@...n.ch" <andrew+netdev@...n.ch>, "davem@...emloft.net"
	<davem@...emloft.net>, "edumazet@...gle.com" <edumazet@...gle.com>,
	"kuba@...nel.org" <kuba@...nel.org>, "pabeni@...hat.com" <pabeni@...hat.com>,
	"robh@...nel.org" <robh@...nel.org>, "krzk+dt@...nel.org"
	<krzk+dt@...nel.org>, "conor+dt@...nel.org" <conor+dt@...nel.org>,
	"p.zabel@...gutronix.de" <p.zabel@...gutronix.de>, "netdev@...r.kernel.org"
	<netdev@...r.kernel.org>, "devicetree@...r.kernel.org"
	<devicetree@...r.kernel.org>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, Conor Dooley <conor.dooley@...rochip.com>
Subject:
 回覆: [PATCH net-next v4 1/7] dt-bindings: net: ftgmac100: support for AST2700

Hi Krzysztof,

Thank you for your reply.

> On Thu, Dec 05, 2024 at 03:20:42PM +0800, Jacky Chou wrote:
> > The AST2700 is the 7th generation SoC from Aspeed.
> > Add compatible support and resets property for AST2700 in yaml.
> >
> > Signed-off-by: Jacky Chou <jacky_chou@...eedtech.com>
> > Acked-by: Conor Dooley <conor.dooley@...rochip.com>
> 
> 
> Your changelog in cover letter does not mention received ack. When did it
> happen?

I am really sorry for forgetting to remove this 'acked-by'.
I will drop it.

> 
> > ---
> >  .../bindings/net/faraday,ftgmac100.yaml         | 17
> ++++++++++++++++-
> >  1 file changed, 16 insertions(+), 1 deletion(-)
> >
> > diff --git
> > a/Documentation/devicetree/bindings/net/faraday,ftgmac100.yaml
> > b/Documentation/devicetree/bindings/net/faraday,ftgmac100.yaml
> > index 9bcbacb6640d..3bba8eee83d6 100644
> > --- a/Documentation/devicetree/bindings/net/faraday,ftgmac100.yaml
> > +++ b/Documentation/devicetree/bindings/net/faraday,ftgmac100.yaml
> > @@ -21,6 +21,7 @@ properties:
> >                - aspeed,ast2400-mac
> >                - aspeed,ast2500-mac
> >                - aspeed,ast2600-mac
> > +              - aspeed,ast2700-mac
> >            - const: faraday,ftgmac100
> >
> >    reg:
> > @@ -33,7 +34,7 @@ properties:
> >      minItems: 1
> >      items:
> >        - description: MAC IP clock
> > -      - description: RMII RCLK gate for AST2500/2600
> > +      - description: RMII RCLK gate for AST2500/2600/2700
> >
> >    clock-names:
> >      minItems: 1
> > @@ -73,6 +74,20 @@ required:
> >
> >  unevaluatedProperties: false
> >
> > +if:
> > +  properties:
> > +    compatible:
> > +      contains:
> > +        const: aspeed,ast2700-mac
> 
> 1. That's a signigicant change. *Drop ack.*
> 
> 2. Test your bindings.
> 3. Put if: block under allOf: and move entire allOf just above your
> unevaluatedProperties... if this stays.
> 4. But you cannot define properties in if:then. They must be defined in top level.
> You can disallow them for variants in if:then: with :false"
> 
> Even exmaple schema has exactly this case:
> https://elixir.bootlin.com/linux/v5.19/source/Documentation/devicetree/bindin
> gs/example-schema.yaml#L212

Thank you for the information.
I will correct my patch and hold it before the DTS is ready.

Thanks,
Jacky