lkml.org 
[lkml]   [2022]   [Feb]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH net v3] net: phy: intel-xway: enable integrated led functions
From


On 2/2/2022 5:01 PM, Andrew Lunn wrote:
>> As a person responsible for boot firmware through kernel for a set of
>> boards I continue to do the following to keep Linux from mucking with
>> various PHY configurations:
>> - remove PHY reset pins from Linux DT's to keep Linux from hard resetting PHY's
>> - disabling PHY drivers
>>
>> What are your thoughts about this?
>
> Hi Tim
>
> I don't like the idea that the bootloader is controlling the hardware,
> not linux.

This is really trying to take advantage of the boot loader setting
things up in a way that Linux can play dumb by using the Generic PHY
driver and being done with it. This works... until it stops, which
happens very very quickly in general. The perfect counter argument to
using the Generic PHY driver is when your system implements a low power
mode where the PHY loses its power/settings, comes up from suspend and
the strap configuration is insufficient and the boot loader is not part
of the resume path *prior* to Linux. In that case Linux needs to restore
the settings, but it needs a PHY driver for that.

If your concern Tim is with minimizing the amount of time the link gets
dropped and re-established, then there is not really much that can be
done that is compatible with Linux setting things up, short of
minimizing the amount of register writes that do need the "commit phase"
via BMCR.RESET.

I do agree that blindly imposing LED settings that are different than
those you want is not great, and should be remedied. Maybe you can
comment this part out in your downstream tree for a while until the LED
binding shows up (we have never been so close I am told).
--
Florian

\
 
 \ /
  Last update: 2022-02-03 04:13    [W:3.436 / U:0.004 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site