Hi Andrew,
Andrew Lunn <and...@lunn.ch> writes:
> Marvell Ethernet switches contain temperature sensors. They are inside
> the embedded PHYs. Move the code into the PHY driver, so that discrete
> PHY drivers also export there temperature sensor.
This message is not correct. The Marvell Ethernet switches contain only
one temperature sensor for the entire chip (please adjust the cover
letter and commit messages when you respin.)
The temperature and threshold are accessed through the embedded PHY
registers of any port, as long as the port is not disabled.
Even unlikely to be used, an interrupt can be generated when the
temperature exceeds a certain threshold. It should be enabled on only
one port at a time since there is only one temperature sensor.
To sum up briefly, the temperature chip is physically inside the switch
chip, but its access is via the embedded PHYs of the switch.
I'd prefer to see the temperature code in the PHY driver as well, but I
prefer we make sure that privileging the data access over the physical
representation is the correct thing to do before moving code around.
Thanks,
Vivien
