[PATCH net-next V7] dt-bindings: net: xlnx,axi-ethernet: convert bindings document to yaml

[Gaddam, Sarath Babu Naidu]

> -----Original Message-----
> From: Gaddam, Sarath Babu Naidu
> Sent: Wednesday, May 3, 2023 3:01 PM
> To: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>;
> davem at davemloft.net; edumazet at google.com; kuba at kernel.org;
> pabeni at redhat.com; robh+dt at kernel.org;
> krzysztof.kozlowski+dt at linaro.org
> Cc: michal.simek at xilinx.com; radhey.shyam.pandey at xilinx.com;
> netdev at vger.kernel.org; devicetree at vger.kernel.org; linux-arm-
> kernel at lists.infradead.org; linux-kernel at vger.kernel.org; Sarangi,
> Anirudha <anirudha.sarangi at amd.com>; Katakam, Harini
> <harini.katakam at amd.com>; git (AMD-Xilinx) <git at amd.com>
> Subject: RE: [PATCH net-next V7] dt-bindings: net: xlnx,axi-ethernet:
> convert bindings document to yaml
> 
> 
> 
> > -----Original Message-----
> > From: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
> > Sent: Tuesday, May 2, 2023 3:56 PM
> > To: Gaddam, Sarath Babu Naidu
> > <sarath.babu.naidu.gaddam at amd.com>; davem at davemloft.net;
> > edumazet at google.com; kuba at kernel.org; pabeni at redhat.com;
> > robh+dt at kernel.org; krzysztof.kozlowski+dt at linaro.org
> > Cc: michal.simek at xilinx.com; radhey.shyam.pandey at xilinx.com;
> > netdev at vger.kernel.org; devicetree at vger.kernel.org; linux-arm-
> > kernel at lists.infradead.org; linux-kernel at vger.kernel.org; Sarangi,
> > Anirudha <anirudha.sarangi at amd.com>; Katakam, Harini
> > <harini.katakam at amd.com>; git (AMD-Xilinx) <git at amd.com>
> > Subject: Re: [PATCH net-next V7] dt-bindings: net: xlnx,axi-ethernet:
> > convert bindings document to yaml
> >
> > On 28/03/2023 14:52, Gaddam, Sarath Babu Naidu wrote:
> > >
> > >
> > >> -----Original Message-----
> > >> From: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
> > >> Sent: Tuesday, March 14, 2023 9:22 PM
> > >> To: Gaddam, Sarath Babu Naidu
> > >> <sarath.babu.naidu.gaddam at amd.com>; davem at davemloft.net;
> > >> edumazet at google.com; kuba at kernel.org; pabeni at redhat.com;
> > >> robh+dt at kernel.org; krzysztof.kozlowski+dt at linaro.org
> > >> Cc: michal.simek at xilinx.com; radhey.shyam.pandey at xilinx.com;
> > >> netdev at vger.kernel.org; devicetree at vger.kernel.org; linux-arm-
> > >> kernel at lists.infradead.org; linux-kernel at vger.kernel.org; Sarangi,
> > >> Anirudha <anirudha.sarangi at amd.com>; Katakam, Harini
> > >> <harini.katakam at amd.com>; git (AMD-Xilinx) <git at amd.com>
> > >> Subject: Re: [PATCH net-next V7] dt-bindings: net: xlnx,axi-ethernet:
> > >> convert bindings document to yaml
> > >>
> > >> On 08/03/2023 07:12, Sarath Babu Naidu Gaddam wrote:
> > >>> From: Radhey Shyam Pandey <radhey.shyam.pandey at xilinx.com>
> > >>>
> > >>> Convert the bindings document for Xilinx AXI Ethernet Subsystem
> > from
> > >>> txt to yaml. No changes to existing binding description.
> > >>>
> > >>
> > >> (...)
> > >>
> > >>> +properties:
> > >>> +  compatible:
> > >>> +    enum:
> > >>> +      - xlnx,axi-ethernet-1.00.a
> > >>> +      - xlnx,axi-ethernet-1.01.a
> > >>> +      - xlnx,axi-ethernet-2.01.a
> > >>> +
> > >>> +  reg:
> > >>> +    description:
> > >>> +      Address and length of the IO space, as well as the address
> > >>> +      and length of the AXI DMA controller IO space, unless
> > >>> +      axistream-connected is specified, in which case the reg
> > >>> +      attribute of the node referenced by it is used.
> > >>
> > >> Did you test it with axistream-connected? The schema and
> > >> description feel contradictory and tests would point the issue.
> > >
> > > Thanks for review comments. We tested with axistream-connected
> and
> > did
> > > not observe any errors. Do you anticipate any issues/errors ?
> >
> > Yes, I anticipate errors. What you wrote here looks incorrect based on
> > the schema.
> >
> > Also, See also my further comments (or you ignored them?).
> >
> > You can come many months after my review to ask about details, to be
> > sure I will forget the topic.
> 
> 
> Hi Krzysztof, Apologies for miscommunication. I replied to this thread on
> March 28 and said that I would address remaining review comments in
> the next version.
> 
> Lore link:
> https://lore.kernel.org/all/MW5PR12MB559880B0E220BDBD64E06D24
> 87889 at MW5PR12MB5598.namprd12.prod.outlook.com/
> https://lore.kernel.org/all/MW5PR12MB5598678BB9AB6EC2FFC424F48
> 7889 at MW5PR12MB5598.namprd12.prod.outlook.com/
> 
> I planned to send next version with phy-mode and pcs-handle maxItems
> fixed.but I wanted to close on the axistream-connected discussion before
> doing so.
> 
> Related to axistream-connected discussion:
> I already ran dt binding check for schema and dts node validation. I
> assume this should point any errors on it.
> 

Hi Krzysztof, Could you please comment on this ? Please let me if I missed
any changes or comments.

Thanks,
Sarath