mbox series

[0/6] net: phy: relax PHY and MDIO reset handling

Message ID 20200626155325.7021-1-brgl@bgdev.pl
Headers show
Series net: phy: relax PHY and MDIO reset handling | expand

Message

Bartosz Golaszewski June 26, 2020, 3:53 p.m. UTC
From: Bartosz Golaszewski <bgolaszewski@baylibre.com>

Previously these patches were submitted as part of a larger series[1]
but since the approach in it will have to be reworked I'm resending
the ones that were non-controversial and have been reviewed for upstream.

Florian suggested a better solution for managing multiple resets. While
I will definitely try to implement something at the driver model's bus
level (together with regulator support), the 'resets' and 'reset-gpios'
DT property is a stable ABI defined in mdio.yaml so improving its support
is in order as we'll have to stick with it anyway. Current implementation
contains an unnecessary limitation where drivers without probe() can't
define resets.

Changes from the previous version:
- order forward declarations in patch 4 alphabetically
- collect review tags

[1] https://lkml.org/lkml/2020/6/22/253

Bartosz Golaszewski (6):
  net: phy: arrange headers in mdio_bus.c alphabetically
  net: phy: arrange headers in mdio_device.c alphabetically
  net: phy: arrange headers in phy_device.c alphabetically
  net: mdio: add a forward declaration for reset_control to mdio.h
  net: phy: reset the PHY even if probe() is not implemented
  net: phy: mdio: reset MDIO devices even if probe() is not implemented

 drivers/net/phy/mdio_bus.c    | 32 +++++++++++-----------
 drivers/net/phy/mdio_device.c | 15 +++++------
 drivers/net/phy/phy_device.c  | 51 ++++++++++++++++++-----------------
 include/linux/mdio.h          |  1 +
 4 files changed, 50 insertions(+), 49 deletions(-)

Comments

David Miller June 26, 2020, 8:40 p.m. UTC | #1
From: Bartosz Golaszewski <brgl@bgdev.pl>
Date: Fri, 26 Jun 2020 17:53:19 +0200

> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
> 
> Previously these patches were submitted as part of a larger series[1]
> but since the approach in it will have to be reworked I'm resending
> the ones that were non-controversial and have been reviewed for upstream.
> 
> Florian suggested a better solution for managing multiple resets. While
> I will definitely try to implement something at the driver model's bus
> level (together with regulator support), the 'resets' and 'reset-gpios'
> DT property is a stable ABI defined in mdio.yaml so improving its support
> is in order as we'll have to stick with it anyway. Current implementation
> contains an unnecessary limitation where drivers without probe() can't
> define resets.
> 
> Changes from the previous version:
> - order forward declarations in patch 4 alphabetically
> - collect review tags
> 
> [1] https://lkml.org/lkml/2020/6/22/253

Series applied, thank you.