Re: PHY firmware update method

[Michael Walle <michael@walle.cc>]

Am 2023-01-24 21:42, schrieb Andrew Lunn:
>> So if you'd do this during the PHY probe, it might try to update the
>> firmware on every boot and fail. Would that be acceptable?
> 
> Do you have a feeling how long that takes?

As a ballpark, with my PoC it takes about 11s. The binary itself
is around 128k and the MDC has a frequency of 2.5MHz. Although,
I didn't do any testing for now with slower or faster clock
frequency. I'm polling for a ready bit most of the time.

> Also, is it possible to put the firmware into RAM and run it from
> there, rather than put it into the EEPROM?

Not that I'm aware of.

>> How long could can a firmware update during probe run? Do we need
>> to do it in the background with the PHY being offline. Sounds like
>> not something we want.
> 
> One device being slow to probe will slow down the probe of that
> bus. But probe of other busses should be unaffected. I _guess_ it
> might have a global affect on EPROBE_DEFER, the next cycle could be
> delayed?  Probably a question for GregKH, or reading the code.
> 
> If it going to be really slow, then i would suggest making use of
> devlink and it being a user initiated operation.

One concern which raised internally was that you'll always do
the update (unconditionally) if there is a newer version. You seem
to make life easier for the user, because the update just runs
automatically. OTHO, what if a user doesn't want to update (for
whatever reason) to the particular version in linux-firmware.git.
I'm undecided on that.

It's different than a firmware which is loaded into RAM and which
*needs* to be loaded anyway. In this case the update is voluntary.

-michael