Re: [PATCH 1/1] dt-bindings: net: convert qca,qca7000.txt yaml format

[Stefan Wahren <wahrenst@gmx.net>]

Hi Frank,

Am 02.06.25 um 17:35 schrieb Frank Li:
> On Fri, May 30, 2025 at 10:33:28AM +0200, Stefan Wahren wrote:
>> Hi Frank,
>>
>> thanks for this patch.
>>
>> Am 29.05.25 um 21:17 schrieb Frank Li:
...
>>> +  - Frank Li <Frank.Li@nxp.com>
>>> +
>>> +description: |
>>> +  The QCA7000 is a serial-to-powerline bridge with a host interface which could
>>> +  be configured either as SPI or UART slave. This configuration is done by
>>> +  the QCA7000 firmware.
>>> +
>>> +  (a) Ethernet over SPI
>>> +
>>> +  In order to use the QCA7000 as SPI device it must be defined as a child of a
>>> +  SPI master in the device tree.
>> Could you please add the dropped "(b) Ethernet over UART" description here?
> Okay, I will add back.
>
>>> +
>>> +properties:
>>> +  compatible:
>>> +    const: qca,qca7000
>>> +
>>> +  reg:
>>> +    maxItems: 1
>>> +
>>> +  interrupts:
>>> +    maxItems: 1
>>> +
>>> +  spi-cpha: true
>>> +
>>> +  spi-cpol: true
>> In case of a SPI setup these properties should be required. Unfortunately
>> i'm not sure how to enforce this. Maybe depending on the presence of "reg"?
> But It think depend on reg is not good idea, which too obscure.
I understand, but this was the first idea which comes to my mind.
>   Ideally it
> should be use two compatible strings. It should treat as two kinds device.
Back in the days there was a discussion about different compatible 
strings and it was rejected [1].
> It is really old devices and not worth to update compatible string.
Yes, this powerline communication chip was introduced in 2012 but it's 
very widespread. Most of the electric vehicles and DC charging stations 
use this chip.

Regards

[1] - https://lkml.org/lkml/2017/5/19/390
>
> Maybe some one in dt team can provide suggestion!
>
> Rob and Krzysztof Kozlowski:
>    any idea about this?
>
> Frank