| 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
| ||
|
Date: Tue, 2 May 2023 10:09:17 +0000
From: "Gaddam, Sarath Babu Naidu" <sarath.babu.naidu.gaddam@....com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
"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+dt@...nel.org" <robh+dt@...nel.org>,
"krzysztof.kozlowski+dt@...aro.org"
<krzysztof.kozlowski+dt@...aro.org>
CC: "michal.simek@...inx.com" <michal.simek@...inx.com>,
"radhey.shyam.pandey@...inx.com" <radhey.shyam.pandey@...inx.com>,
"netdev@...r.kernel.org" <netdev@...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>,
"Sarangi, Anirudha" <anirudha.sarangi@....com>,
"Katakam, Harini" <harini.katakam@....com>,
"git (AMD-Xilinx)" <git@....com>
Subject: RE: [PATCH net-next V7] dt-bindings: net: xlnx,axi-ethernet: convert
bindings document to yaml
> -----Original Message-----
> From: Gaddam, Sarath Babu Naidu
> <sarath.babu.naidu.gaddam@....com>
> Sent: Tuesday, March 28, 2023 9:31 PM
> To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>;
> davem@...emloft.net; edumazet@...gle.com; kuba@...nel.org;
> pabeni@...hat.com; robh+dt@...nel.org;
> krzysztof.kozlowski+dt@...aro.org
> Cc: michal.simek@...inx.com; radhey.shyam.pandey@...inx.com;
> netdev@...r.kernel.org; devicetree@...r.kernel.org; linux-arm-
> kernel@...ts.infradead.org; linux-kernel@...r.kernel.org; Sarangi,
> Anirudha <anirudha.sarangi@....com>; Katakam, Harini
> <harini.katakam@....com>; git (AMD-Xilinx) <git@....com>
> Subject: RE: [PATCH net-next V7] dt-bindings: net: xlnx,axi-ethernet:
> convert bindings document to yaml
>
>
>
> > -----Original Message-----
> > From: Gaddam, Sarath Babu Naidu
> > <sarath.babu.naidu.gaddam@....com>
> > Sent: Tuesday, March 28, 2023 6:22 PM
> > To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>;
> > davem@...emloft.net; edumazet@...gle.com; kuba@...nel.org;
> > pabeni@...hat.com; robh+dt@...nel.org;
> > krzysztof.kozlowski+dt@...aro.org
> > Cc: michal.simek@...inx.com; radhey.shyam.pandey@...inx.com;
> > netdev@...r.kernel.org; devicetree@...r.kernel.org; linux-arm-
> > kernel@...ts.infradead.org; linux-kernel@...r.kernel.org; Sarangi,
> > Anirudha <anirudha.sarangi@....com>; Katakam, Harini
> > <harini.katakam@....com>; git (AMD-Xilinx) <git@....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@...aro.org>
> > > Sent: Tuesday, March 14, 2023 9:22 PM
> > > To: Gaddam, Sarath Babu Naidu
> > > <sarath.babu.naidu.gaddam@....com>; davem@...emloft.net;
> > > edumazet@...gle.com; kuba@...nel.org; pabeni@...hat.com;
> > > robh+dt@...nel.org; krzysztof.kozlowski+dt@...aro.org
> > > Cc: michal.simek@...inx.com; radhey.shyam.pandey@...inx.com;
> > > netdev@...r.kernel.org; devicetree@...r.kernel.org; linux-arm-
> > > kernel@...ts.infradead.org; linux-kernel@...r.kernel.org; Sarangi,
> > > Anirudha <anirudha.sarangi@....com>; Katakam, Harini
> > > <harini.katakam@....com>; git (AMD-Xilinx) <git@....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@...inx.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 ?
>
> Just to add more details, we have tested it using below dt node
>
> axienet@0 {
> axistream-connected = <&dma>;
> reg = <0x00 0x80000000 0x00 0x40000>;
> compatible = "xlnx,axi-ethernet-2.01.a";
> clock-names = "s_axi_lite_clk\0axis_clk\0ref_clk";
> clocks = <0x03 0x47 0x03 0x47 0x18>;
> phy-mode = "sgmii";
> xlnx,rxcsum = <0x02>;
> xlnx,rxmem = <0x1000>;
> xlnx,txcsum = <0x02>;
> pcs-handle = <0x19>;
> phy-handle = <0x78>;
> dmas = <0x17 0x00 0x17 0x01>;
> dma-names = "tx_chan0\0rx_chan0";
> mac-address = [ff ff ff ff ff ff];
> managed = "in-band-status";
> phandle = <0x79>;
> mdio {
> #address-cells = <0x01>;
> #size-cells = <0x00>;
>
> phy@0 {
> compatible = "ethernet-phy-ieee802.3-c22";
> reg = <0x00>;
> phandle = <0x78>;
> };
>
> ethernet-phy@2 {
> device_type = "ethernet-phy";
> reg = <0x02>;
> phandle = <0x19>;
> };
> };
> };
> This DT node works with our board. "&dma" is the dma DT node and to
> test the second case where dma address and length included in the
> axienet reg's property as below "reg = <0x00 0x80000000 0x00 0x40000
> 0x0 0x80040000 0x0 0x1000>;"
>
> I did not observe any issue with above two cases. Used below command
> to validate the yaml using above DT node.
> make dtbs_check
> DT_SCHEMA_FILES=Documentation/devicetree/bindings/net/xlnx,axi-
> ethernet.yaml
>
Hi Krzysztof, Can you please comment If above explanation is acceptable ?
I will address remaining review comments and send the next version.
Thanks,
Sarath
Powered by blists - more mailing lists