| 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: <04112f5c-231d-559a-39f9-d183e8985a87@linaro.org>
Date: Tue, 2 May 2023 12:28:02 +0200
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: "Gaddam, Sarath Babu Naidu" <sarath.babu.naidu.gaddam@....com>,
"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
On 02/05/2023 12:09, Gaddam, Sarath Babu Naidu wrote:
>
>
>> -----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.
The DTS you pointed obviously cannot work with the binding - it has
obvious mistakes. Starting with phy-mode. So whatever you did, was not
correct testing. Since nothing from your code is upstream, I cannot
verify it.
Upstream your DTS first.
Best regards,
Krzysztof
Powered by blists - more mailing lists