mbox series

[net-next,v2,0/2] net: phy: marvell-88q2xxx: Enable temperature measurement in probe again

Message ID 20250220-marvell-88q2xxx-hwmon-enable-at-probe-v2-0-78b2838a62da@gmail.com (mailing list archive)
Headers show
Series net: phy: marvell-88q2xxx: Enable temperature measurement in probe again | expand

Message

Dimitri Fedrau Feb. 20, 2025, 8:11 a.m. UTC 
Patchset fixes these:
- Enable temperature measurement in probe again
- Prevent reading temperature with asserted reset

Signed-off-by: Dimitri Fedrau <dima.fedrau@gmail.com>
---
Changes in v2:
- Add comment in mv88q2xxx_config_init why the temperature sensor is
  enabled again (Stefan)
- Fix commit message by adding the information why the PHY reset might
  be asserted. (Andrew)
- Remove fixes tags (Andrew)
- Switch to net-next (Andrew)
- Return ENETDOWN instead of EIO when PHYs reset is asserted in
  mv88q2xxx_hwmon_read (Andrew)
- Add check if PHYs reset is asserted in mv88q2xxx_hwmon_write as it was
  done in mv88q2xxx_hwmon_read
- Link to v1: https://lore.kernel.org/r/20250218-marvell-88q2xxx-hwmon-enable-at-probe-v1-0-999a304c8a11@gmail.com

---
Dimitri Fedrau (2):
      net: phy: marvell-88q2xxx: Enable temperature measurement in probe again
      net: phy: marvell-88q2xxx: Prevent hwmon access with asserted reset

 drivers/net/phy/marvell-88q2xxx.c | 26 +++++++++++++++++++++++++-
 1 file changed, 25 insertions(+), 1 deletion(-)
---
base-commit: 13260df23f5c0097f632c36fcd568ee33aa6a621
change-id: 20250217-marvell-88q2xxx-hwmon-enable-at-probe-2a2666325985

Best regards,

Comments

Kory Maincent Feb. 20, 2025, 9:19 a.m. UTC | #1 
On Thu, 20 Feb 2025 09:11:10 +0100
Dimitri Fedrau <dima.fedrau@gmail.com> wrote:

> Patchset fixes these:
> - Enable temperature measurement in probe again
> - Prevent reading temperature with asserted reset
> 
> Signed-off-by: Dimitri Fedrau <dima.fedrau@gmail.com>

Reviewed-by: Kory Maincent <kory.maincent@bootlin.com>

Thank you!