[PATCH v3 1/3] dt-bindings: can: xilinx_can: Add ECC property 'xlnx,has-ecc'

[Goud, Srinivas]

Hi Krzysztof,

>-----Original Message-----
>From: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
>Sent: Wednesday, August 30, 2023 12:43 PM
>To: Goud, Srinivas <srinivas.goud at amd.com>; Rob Herring <robh at kernel.org>
>Cc: wg at grandegger.com; mkl at pengutronix.de; davem at davemloft.net;
>edumazet at google.com; kuba at kernel.org; pabeni at redhat.com;
>krzysztof.kozlowski+dt at linaro.org; conor+dt at kernel.org;
>p.zabel at pengutronix.de; git (AMD-Xilinx) <git at amd.com>; Simek, Michal
><michal.simek at amd.com>; linux-can at vger.kernel.org; linux-arm-
>kernel at lists.infradead.org; linux-kernel at vger.kernel.org;
>netdev at vger.kernel.org; devicetree at vger.kernel.org;
>appana.durga.rao at xilinx.com; naga.sureshkumar.relli at xilinx.com
>Subject: Re: [PATCH v3 1/3] dt-bindings: can: xilinx_can: Add ECC property
>'xlnx,has-ecc'
>
>On 30/08/2023 08:06, Goud, Srinivas wrote:
>>>> +
>>>>  required:
>>>>    - compatible
>>>>    - reg
>>>> @@ -137,6 +141,7 @@ examples:
>>>>          interrupts = <GIC_SPI 59 IRQ_TYPE_EDGE_RISING>;
>>>>          tx-fifo-depth = <0x40>;
>>>>          rx-fifo-depth = <0x40>;
>>>> +        xlnx,has-ecc
>>>
>>> Obviously not tested.
>> Will fix it.
>>
>
>Fix it by fixing error or by testing? Can you do both?
Tested with v1 and missed in v2 due to copy paste, will test and send next version.

Thanks.
Srinivas