linux-kernel - Re: [PATCH net-next v2 4/9] dt-bindings: net: add OPEN Alliance 10BASE-T1x MAC-PHY Serial Interface

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: <90b77b7e-4424-4436-8cec-055f41fa7470@microchip.com>
Date:   Tue, 31 Oct 2023 12:59:47 +0000
From:   <Parthiban.Veerasooran@...rochip.com>
To:     <krzysztof.kozlowski@...aro.org>
CC:     <netdev@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>, <linux-doc@...r.kernel.org>,
        <Horatiu.Vultur@...rochip.com>, <Woojung.Huh@...rochip.com>,
        <Nicolas.Ferre@...rochip.com>, <UNGLinuxDriver@...rochip.com>,
        <Thorsten.Kummermehr@...rochip.com>, <davem@...emloft.net>,
        <edumazet@...gle.com>, <kuba@...nel.org>, <pabeni@...hat.com>,
        <robh+dt@...nel.org>, <krzysztof.kozlowski+dt@...aro.org>,
        <conor+dt@...nel.org>, <corbet@....net>,
        <Steen.Hegelund@...rochip.com>, <rdunlap@...radead.org>,
        <horms@...nel.org>, <casper.casan@...il.com>, <andrew@...n.ch>
Subject: Re: [PATCH net-next v2 4/9] dt-bindings: net: add OPEN Alliance
 10BASE-T1x MAC-PHY Serial Interface

Hi Krzysztof

As per the comments in the other email of this patch, this DT bindings 
are going to be removed.

Best Regards,
Parthiban V

On 24/10/23 1:14 pm, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> On 23/10/2023 17:46, Parthiban Veerasooran wrote:
>> Add DT bindings OPEN Alliance 10BASE-T1x MACPHY Serial Interface
>> parameters. These are generic properties that can apply to any 10BASE-T1x
>> MAC-PHY which uses OPEN Alliance TC6 specification.
> 
> Except that it was not tested at all few more issues.
> 
>>
>> Signed-off-by: Parthiban Veerasooran <Parthiban.Veerasooran@...rochip.com>
>> ---
>>   .../devicetree/bindings/net/oa-tc6.yaml       | 72 +++++++++++++++++++
>>   MAINTAINERS                                   |  1 +
>>   2 files changed, 73 insertions(+)
>>   create mode 100644 Documentation/devicetree/bindings/net/oa-tc6.yaml
>>
>> diff --git a/Documentation/devicetree/bindings/net/oa-tc6.yaml b/Documentation/devicetree/bindings/net/oa-tc6.yaml
>> new file mode 100644
>> index 000000000000..9f442fa6cace
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/net/oa-tc6.yaml
> 
> Filename based on compatible.
> 
>> @@ -0,0 +1,72 @@
>> +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
>> +%YAML 1.2
>> +---
>> +$id: http://devicetree.org/schemas/net/oa-tc6.yaml#
>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>> +
>> +title: OPEN Alliance 10BASE-T1x MAC-PHY Specification Common Properties
>> +
>> +maintainers:
>> +  - Parthiban Veerasooran <parthiban.veerasooran@...rochip.com>
>> +
>> +description:
>> +  These are generic properties that can apply to any 10BASE-T1x MAC-PHY
>> +  which uses OPEN Alliance TC6 specification.
>> +
>> +  10BASE-T1x MAC-PHY Serial Interface Specification can be found at:
>> +    https://opensig.org/about/specifications/
>> +
>> +properties:
>> +  $nodename:
>> +    pattern: "^oa-tc6(@.*)?"
> 
> Drop
> 
>> +
>> +  "#address-cells":
>> +    const: 1
>> +
>> +  "#size-cells":
>> +    const: 0
> 
> Why?
> 
>> +
>> +  oa-cps:
>> +    maxItems: 1
>> +    description:
>> +      Chunk Payload Size. Configures the data chunk payload size to 2^N,
>> +      where N is the value of this bitfield. The minimum possible data
>> +      chunk payload size is 8 bytes or N = 3. The default data chunk
>> +      payload size is 64 bytes, or N = 6. The minimum supported data chunk
>> +      payload size for this MAC-PHY device is indicated in the CPSMIN
>> +      field of the CAPABILITY register. Valid values for this parameter
>> +      are 8, 16, 32 and 64. All other values are reserved.
>> +
>> +  oa-txcte:
>> +    maxItems: 1
>> +    description:
>> +      Transmit Cut-Through Enable. When supported by this MAC-PHY device,
>> +      this bit enables the cut-through mode of frame transfer through the
>> +      MAC-PHY device from the SPI host to the network.
>> +
>> +  oa-rxcte:
>> +    maxItems: 1
>> +    description:
>> +      Receive Cut-Through Enable. When supported by this MAC-PHY device,
>> +      this bit enables the cut-through mode of frame transfer through the
>> +      MAC-PHY device from the network to the SPI host.
>> +
>> +  oa-prote:
>> +    maxItems: 1
>> +    description:
>> +      Control data read/write Protection Enable. When set, all control
>> +      data written to and read from the MAC-PHY will be transferred with
>> +      its complement for detection of bit errors.
>> +
>> +additionalProperties: true
>> +
>> +examples:
>> +  - |
>> +    oa-tc6 {
>> +        #address-cells = <1>;
>> +     #size-cells = <0>;
> 
> That's some total mess in indentation.
> 
>> +     oa-cps = <64>;
>> +     oa-txcte;
>> +     oa-rxcte;
>> +     oa-prote;
>> +    };
> Best regards,
> Krzysztof
>