Re: [PATCH net-next v3 1/3] phy: open_alliance_helpers: Add defines for link quality metrics

From: Andrew Lunn
Date: Wed Aug 28 2024 - 16:51:26 EST


On Wed, Aug 28, 2024 at 01:34:28PM -0700, Jakub Kicinski wrote:
> On Wed, 28 Aug 2024 06:50:46 +0200 Oleksij Rempel wrote:
> > Considering that you've requested a change to the uAPI, the work has now become
> > more predictable. I can plan for it within the task and update the required
> > time budget accordingly. However, it's worth noting that while this work is
> > manageable, the time spent on this particular task could be seen as somewhat
> > wasted from a budget perspective, as it wasn't part of the original scope.
>
> I can probably take a stab at the kernel side, since I know the code
> already shouldn't take me more more than an hour. Would that help?
> You'd still need to retest, fix bugs. And go thru review.. so all
> the not-so-fun parts
>
> > > Especially that we're talking about uAPI, once we go down
> > > the string path I presume they will stick around forever.
> >
> > Yes, I agree with it. I just needed this feedback as early as possible.
>
> Andrew? Do you want to decide? :)

I agree about avoiding free test strings. Something more structures
would be good.

I can definitely help out with review, but i don't have any time at
the moment for writing code.

Andrew