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
| ||
|
Message-ID: <MW5PR12MB5598ED29E01601585963D5D0876C9@MW5PR12MB5598.namprd12.prod.outlook.com> Date: Wed, 3 May 2023 09:30:37 +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: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org> > Sent: Tuesday, May 2, 2023 3:56 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 28/03/2023 14:52, Gaddam, Sarath Babu Naidu wrote: > > > > > >> -----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 ? > > 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/MW5PR12MB559880B0E220BDBD64E06D2487889@MW5PR12MB5598.namprd12.prod.outlook.com/ https://lore.kernel.org/all/MW5PR12MB5598678BB9AB6EC2FFC424F487889@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. Thanks, Sarath
Powered by blists - more mailing lists