Re: [PATCH net-next v3 12/12] dt-bindings: net: add Microchip's LAN865X 10BASE-T1S MACPHY

Parthiban.Veerasooran
2024-03-25T07:11:16Z

Hi Krzysztof,

On 23/03/24 3:54 pm, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>
> On 22/03/2024 09:28, Parthiban.Veerasooran@microchip.com wrote:
>>>>
>>>> - const: microchip,lan8650
>>>>
>>>> - items:
>>>>
>>>> - const: microchip,lan8651
>>>>
>>>> - const: microchip,lan8650
>>>>
>>>> Executed dt_binding_check with the above update and it was successful.
>>>> Hope this is OK?
>>>
>>> This is the third time you ask us. None of the previous cases were
>>> actually tested. Maybe this one was, maybe not. I assume the latter.
>>>
>>> First, test your code.
>> As I mentioned in the previous email itself, I tested this case and the
>> previous case both in my RPI 4 setup before replying to the comment. The
>
> I don't understand how one can test bindings and DTS on RPI 4. Testing
> is with dt_bindings_check and dtbs_check.
Ok, may be I had a misunderstanding here. Every time I used to test the
driver with the generated dts in RPI 4. I thought you are asking about
it. I used the below commands to check the dt bindings and dtbs.

$ make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf-
DT_SCHEMA_FILES=Documentation/devicetree/bindings/net/microchip,lan865x.yaml
dt_binding_check

$ make ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf-
DT_SCHEMA_FILES=Documentation/devicetree/bindings/net/microchip,lan865x.yaml
dtbs_check

They didn't report any error.

Best regards,
Parthiban V
>
>
>
> Best regards,
> Krzysztof
>
>