[PATCH net-next v6 5/5] ARM: dts: aspeed: ast2600-evb: Configure RGMII delay for MAC

[Jacky Chou]

> > Could you please help confirm how we should proceed with this project?
> 
> That is up to you.
> 
> If you do lots of small patches with good comments, you probably can do it in
> Linux. But you need to convince reviewers you are not breaking backwards
> compatibility.
> 
> I think the bootloader change is simpler. But i've not worked on bootloaders for
> a long time.
> 
> > 3. Send a patch to correct the phy-mode settings in
> > aspeed-ast2600-evb.dts so that they match the bootloader after the fix.
> >
> > mac0: "rgmii-rxid" -> "rgmii-id"
> > mac1: "rgmii-rxid" -> "rgmii-id"
> > mac2: "rgmii" -> "rgmii-id"
> > mac3: "rgmii" -> "rgmii-id"
> 
> You will do this anyway. However, just so that backwards comparability gets
> tested all the time, i would suggest you only change mac0 and mac3. Leave
> mac1 and mac2 with old settings, and add comments that old values are being
> used for testing, and they should not be cut/paste into new .dts files. And add a
> comment that mac0 and mac3 require the new bootloader, if you decide to
> solve it that way.
> 

Thank you for your patience and for the detailed guidance.

I will discuss the possible approaches internally with the team and decide on the 
direction for the fix. 
Thanks again for your feedback.

Thanks,
Jacky