mbox series

[v4,0/2] Extend Sparx5 switch reset driver for lan966x

Message ID 20211018091522.1113510-1-horatiu.vultur@microchip.com
Headers show
Series Extend Sparx5 switch reset driver for lan966x | expand

Message

Horatiu Vultur Oct. 18, 2021, 9:15 a.m. UTC
This patch serie extends the Microchip Sparx5 reset driver to support
lan966x

v3->v4:
  - drop all the changes regarding the phy reset.
  - use enum instead oneOf in dt-bindings

v2->v3:
  - rename variable reset_gpio to phy_reset_gpio
  - rename gpios property in documentation to phy-reset-gpios

v1->v2:
  - add reviewed-by tag
  - extend driver to be able to release the reset also for external PHYs

Horatiu Vultur (2):
  dt-bindings: reset: Add lan966x support
  reset: mchp: sparx5: Extend support for lan966x

 .../bindings/reset/microchip,rst.yaml         |  4 +-
 drivers/reset/Kconfig                         |  2 +-
 drivers/reset/reset-microchip-sparx5.c        | 40 +++++++++++++++----
 3 files changed, 36 insertions(+), 10 deletions(-)

Comments

Philipp Zabel Oct. 18, 2021, 10:43 a.m. UTC | #1
On Mon, 2021-10-18 at 11:15 +0200, Horatiu Vultur wrote:
> This patch extends sparx5 driver to support also the lan966x. The
> process to reset the switch is the same only it has different offsets.
> Therefore make the driver more generic and add support for lan966x.
> 
> Signed-off-by: Horatiu Vultur <horatiu.vultur@microchip.com>
> Reviewed-by: Andrew Lunn <andrew@lunn.ch>

Reviewed-by: Philipp Zabel <p.zabel@pengutronix.de>

regards
Philipp
Steen Hegelund Oct. 18, 2021, 12:01 p.m. UTC | #2
On Mon, 2021-10-18 at 11:15 +0200, Horatiu Vultur wrote:
> This patch extends sparx5 driver to support also the lan966x. The
> process to reset the switch is the same only it has different offsets.
> Therefore make the driver more generic and add support for lan966x.
> 
> Signed-off-by: Horatiu Vultur <horatiu.vultur@microchip.com>
> Reviewed-by: Andrew Lunn <andrew@lunn.ch>
> ---

Reviewed-by: Steen Hegelund <steen.hegelund@microchip.com>
Philipp Zabel Oct. 18, 2021, 3:01 p.m. UTC | #3
On Mon, 2021-10-18 at 11:15 +0200, Horatiu Vultur wrote:
> This patch serie extends the Microchip Sparx5 reset driver to support
> lan966x
> 
> v3->v4:
>   - drop all the changes regarding the phy reset.
>   - use enum instead oneOf in dt-bindings
> 
> v2->v3:
>   - rename variable reset_gpio to phy_reset_gpio
>   - rename gpios property in documentation to phy-reset-gpios
> 
> v1->v2:
>   - add reviewed-by tag
>   - extend driver to be able to release the reset also for external PHYs
> 
> Horatiu Vultur (2):
>   dt-bindings: reset: Add lan966x support
>   reset: mchp: sparx5: Extend support for lan966x
> 
>  .../bindings/reset/microchip,rst.yaml         |  4 +-
>  drivers/reset/Kconfig                         |  2 +-
>  drivers/reset/reset-microchip-sparx5.c        | 40 +++++++++++++++----
>  3 files changed, 36 insertions(+), 10 deletions(-)

Thank you, both applied to reset/next.

regards
Philipp