lkml.org 
[lkml]   [2021]   [Sep]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: Re: Re: [PATCH] arm64: dts: sun50i: h5: NanoPI Neo 2: phy-mode rgmii-id
Date
Dne torek, 31. avgust 2021 ob 22:53:24 CEST je Andrew Lunn napisal(a):
> > True, but then again, DT is a bit special, since it's not used only by
Linux.
> > It's shared at least with BSDs and U-Boot, which most likely have
independent
> > driver implementation. That's why it's good to have DT fixes referencing DT
> > related commits. As you said, DT described HW wrong.
> >
> > Looking at past Allwinner related DT commits regarding this issue, we were
not
> > totally consistent. Most of them reference commit where emac node was
> > introduced in DT. But I also found a couple of commits which indeed have
fixes
> > tag for bbc4d71d6354.
>
> All true, and it is not a big deal if it does go back further. I don't
> care, we can let the Allwinner Maintainer decide. Ah, that is you :-)
>
> I just want to try to keep it simple for somebody who is contributing
> their first Linux kernel patch. If it is good enough, i say accept it,
> and we can do a bit more education on the second contribution.

Maxime, Chen-Yu, thoughts?

Since there are both approaches for fixes tag already in, I guess this is also
fine.

Best regards,
Jernej




\
 
 \ /
  Last update: 2021-09-01 17:32    [W:0.086 / U:0.324 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site