[U-Boot] arm: dts: imx53-cx9020: fix packetloss on fec_mxc

From: Patrick Bruenn <p.bruenn@beckhoff.com>

The pinmuxing for i.MX53 FEC ethernet copied from
<kernel>/arch/arm/boot/dts/imx53-qsb-common.dtsi (at least until v4.9)
was bad. It is different from the manual pinmuxing in
<u-boot>/board/freescale/mx53loco/mx53loco.c which was used in
cx9020 implementation previously before mainlining into u-boot.
It seems the bug in imx53-qsb kernel device tree is hidden for so long,
because it was never used, by the kernel driver.

Signed-off-by: Patrick Bruenn <p.bruenn@beckhoff.com>
---

 arch/arm/dts/imx53-cx9020.dts | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

On Mon, Jan 23, 2017 at 03:11:27PM +0100, linux-kernel-dev@beckhoff.com wrote:

> From: Patrick Bruenn <p.bruenn@beckhoff.com>
> 
> The pinmuxing for i.MX53 FEC ethernet copied from
> <kernel>/arch/arm/boot/dts/imx53-qsb-common.dtsi (at least until v4.9)
> was bad. It is different from the manual pinmuxing in
> <u-boot>/board/freescale/mx53loco/mx53loco.c which was used in
> cx9020 implementation previously before mainlining into u-boot.
> It seems the bug in imx53-qsb kernel device tree is hidden for so long,
> because it was never used, by the kernel driver.
> 
> Signed-off-by: Patrick Bruenn <p.bruenn@beckhoff.com>

So in other words, the dts file is correct in current mainline kernel?
Or still pending?  Thanks!

>From: Tom Rini [mailto:trini@konsulko.com]
>Sent: Montag, 23. Januar 2017 16:31
>
>On Mon, Jan 23, 2017 at 03:11:27PM +0100, linux-kernel-dev@beckhoff.com
>wrote:
>
>> From: Patrick Bruenn <p.bruenn@beckhoff.com>
>>
>> The pinmuxing for i.MX53 FEC ethernet copied from
>> <kernel>/arch/arm/boot/dts/imx53-qsb-common.dtsi (at least until v4.9)
>> was bad. It is different from the manual pinmuxing in
>> <u-boot>/board/freescale/mx53loco/mx53loco.c which was used in
>> cx9020 implementation previously before mainlining into u-boot.
>> It seems the bug in imx53-qsb kernel device tree is hidden for so long,
>> because it was never used, by the kernel driver.
>>
>> Signed-off-by: Patrick Bruenn <p.bruenn@beckhoff.com>
>
>So in other words, the dts file is correct in current mainline kernel?
>Or still pending?  Thanks!
Sorry, no it is still pending. I just reported it to the kernel maintainers:
https://mail-archive.com/linux-kernel@vger.kernel.org/msg1316717.html

Patrick

On Mon, Jan 23, 2017 at 05:20:25PM +0000, linux-kernel-dev wrote:
> >From: Tom Rini [mailto:trini@konsulko.com]
> >Sent: Montag, 23. Januar 2017 16:31
> >
> >On Mon, Jan 23, 2017 at 03:11:27PM +0100, linux-kernel-dev@beckhoff.com
> >wrote:
> >
> >> From: Patrick Bruenn <p.bruenn@beckhoff.com>
> >>
> >> The pinmuxing for i.MX53 FEC ethernet copied from
> >> <kernel>/arch/arm/boot/dts/imx53-qsb-common.dtsi (at least until v4.9)
> >> was bad. It is different from the manual pinmuxing in
> >> <u-boot>/board/freescale/mx53loco/mx53loco.c which was used in
> >> cx9020 implementation previously before mainlining into u-boot.
> >> It seems the bug in imx53-qsb kernel device tree is hidden for so long,
> >> because it was never used, by the kernel driver.
> >>
> >> Signed-off-by: Patrick Bruenn <p.bruenn@beckhoff.com>
> >
> >So in other words, the dts file is correct in current mainline kernel?
> >Or still pending?  Thanks!
> Sorry, no it is still pending. I just reported it to the kernel maintainers:
> https://mail-archive.com/linux-kernel@vger.kernel.org/msg1316717.html

Ah, OK.  I think we want get this in the release, but it can wait for
-rc3 or so, unless it gets ack'ed by the kernel folks sooner.  Please
keep us apprised, thanks!

On 23/01/2017 18:45, Tom Rini wrote:
> On Mon, Jan 23, 2017 at 05:20:25PM +0000, linux-kernel-dev wrote:
>>> From: Tom Rini [mailto:trini@konsulko.com]
>>> Sent: Montag, 23. Januar 2017 16:31
>>>
>>> On Mon, Jan 23, 2017 at 03:11:27PM +0100, linux-kernel-dev@beckhoff.com
>>> wrote:
>>>
>>>> From: Patrick Bruenn <p.bruenn@beckhoff.com>
>>>>
>>>> The pinmuxing for i.MX53 FEC ethernet copied from
>>>> <kernel>/arch/arm/boot/dts/imx53-qsb-common.dtsi (at least until v4.9)
>>>> was bad. It is different from the manual pinmuxing in
>>>> <u-boot>/board/freescale/mx53loco/mx53loco.c which was used in
>>>> cx9020 implementation previously before mainlining into u-boot.
>>>> It seems the bug in imx53-qsb kernel device tree is hidden for so long,
>>>> because it was never used, by the kernel driver.
>>>>
>>>> Signed-off-by: Patrick Bruenn <p.bruenn@beckhoff.com>
>>>
>>> So in other words, the dts file is correct in current mainline kernel?
>>> Or still pending?  Thanks!
>> Sorry, no it is still pending. I just reported it to the kernel maintainers:
>> https://mail-archive.com/linux-kernel@vger.kernel.org/msg1316717.html
> 
> Ah, OK.  I think we want get this in the release, but it can wait for
> -rc3 or so, unless it gets ack'ed by the kernel folks sooner.  Please
> keep us apprised, thanks!
>

I saw patch was integrated in kernel - I merge this as well in u-boot-imx.

Regards,
Stefano