[EXT] Re: [PATCH 1/3] dt-bings: net: fsl,fec: update compatible item

[Wei Fang]

Hi Krzysztof,

	Thanks for your suggestion, I will resubmit the patch.

-----Original Message-----
From: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org> 
Sent: 2022年7月5日 15:27
To: Wei Fang <wei.fang at nxp.com>; davem at davemloft.net; edumazet at google.com; kuba at kernel.org; pabeni at redhat.com; robh+dt at kernel.org; krzysztof.kozlowski+dt at linaro.org; shawnguo at kernel.org; s.hauer at pengutronix.de
Cc: netdev at vger.kernel.org; devicetree at vger.kernel.org; linux-kernel at vger.kernel.org; kernel at pengutronix.de; festevam at gmail.com; dl-linux-imx <linux-imx at nxp.com>; Peng Fan <peng.fan at nxp.com>; Jacky Bai <ping.bai at nxp.com>; sudeep.holla at arm.com; linux-arm-kernel at lists.infradead.org; Aisheng Dong <aisheng.dong at nxp.com>
Subject: Re: [EXT] Re: [PATCH 1/3] dt-bings: net: fsl,fec: update compatible item

Caution: EXT Email

On 05/07/2022 04:47, Wei Fang wrote:
> Hi Krzysztof,
>
>       Sorry, I'm still a little confused. Do you mean to modify as follows?
>> +      - items:
>> +          - enum:
>> +              - fsl,imx8ulp-fec
>> +          - const: fsl,imx6ul-fec
>> +          - const: fsl,imx6q-fec

Yes

>
> And as far as I know, the imx8ulp's fec is reused from imx6ul, they both have the same features. However, the fec of imx8ulp(and imx6ul) is a little different from imx6q, therefore, the functions supported by the driver are also somewhat different.

I understand. But if imx8ulp is the same as imx6ul and imx6ul is compatible with imx6q, then I expect imx8ulp to be compatible with imx6ul and with imx6q.

Best regards,
Krzysztof