[PATCH v14 00/12] net: pcs: Introduce support for fwnode PCS
Christian Marangi (Ansuel)
ansuelsmth at gmail.com
Wed Aug 19 08:55:51 PDT 2026
Il giorno lun 17 ago 2026 alle ore 23:02 Christian Marangi
<ansuelsmth at gmail.com> ha scritto:
>
> On Mon, Aug 17, 2026 at 01:47:27PM -0700, Jakub Kicinski wrote:
> > On Thu, 13 Aug 2026 10:35:05 +0200 Christian Marangi wrote:
> > > This series introduce a most awaited feature that is correctly
> > > provide PCS with fwnode without having to use specific export symbol
> > > and additional handling of PCS in phylink.
> > >
> > > At times there were 2 different implementation (this and the one
> > > from Sean) but Sean agreed that this can be picked and used in favor
> > > of his implementation as long as his case with race condition is
> > > correctly handled.
> >
> > Coccicheck says:
> >
> > drivers/net/phy/phylink.c:1039:31-34:
> > ERROR: invalid reference to the index variable of the iterator on line 1028
> >
> > drivers/net/phy/phylink.c:1364:26-29:
> > ERROR: invalid reference to the index variable of the iterator on line 1346
> >
> > drivers/net/phy/phylink.c:582:7-10:
> > ERROR: invalid reference to the index variable of the iterator on line 569
> >
> > I vaguely recall that you're supposed to make a copy of the pointer,
> > not just add a "found" boolean. Dunno if it ever happened but Linus
> > was trying to make the list iterators override the pointer on exit.
>
> Oh! I wasn't aware of this intention... I have seen lots of pattern where
> the last value of the iterator was directly used (if detected valid)
>
> It's and easy fix... I can send a new revision right away if you want,
> I'm already on it fixing this in the meantime.
>
Sooo should I post a v15 with the fix? It would be great to have this for 7.3.
More information about the Linux-mediatek
mailing list