[PATCH 07/17] net: mdio: mscc-miim: Handle the switch reset

[Sai Krishna Gajula]

> -----Original Message-----
> From: Herve Codina <herve.codina at bootlin.com>
> Sent: Tuesday, April 30, 2024 2:07 PM
> To: Herve Codina <herve.codina at bootlin.com>; Thomas Gleixner
> <tglx at linutronix.de>; Rob Herring <robh at kernel.org>; Krzysztof Kozlowski
> <krzk+dt at kernel.org>; Conor Dooley <conor+dt at kernel.org>; David S. Miller
> <davem at davemloft.net>; Eric Dumazet <edumazet at google.com>; Jakub
> Kicinski <kuba at kernel.org>; Paolo Abeni <pabeni at redhat.com>; Lee Jones
> <lee at kernel.org>; Arnd Bergmann <arnd at arndb.de>; Horatiu Vultur
> <horatiu.vultur at microchip.com>; UNGLinuxDriver at microchip.com; Andrew
> Lunn <andrew at lunn.ch>; Heiner Kallweit <hkallweit1 at gmail.com>; Russell
> King <linux at armlinux.org.uk>; Saravana Kannan <saravanak at google.com>;
> Bjorn Helgaas <bhelgaas at google.com>; Philipp Zabel
> <p.zabel at pengutronix.de>; Lars Povlsen <lars.povlsen at microchip.com>;
> Steen Hegelund <Steen.Hegelund at microchip.com>; Daniel Machon
> <daniel.machon at microchip.com>; Alexandre Belloni
> <alexandre.belloni at bootlin.com>
> Cc: linux-kernel at vger.kernel.org; devicetree at vger.kernel.org;
> netdev at vger.kernel.org; linux-pci at vger.kernel.org; linux-arm-
> kernel at lists.infradead.org; Allan Nielsen <allan.nielsen at microchip.com>;
> Steen Hegelund <steen.hegelund at microchip.com>; Luca Ceresoli
> <luca.ceresoli at bootlin.com>; Thomas Petazzoni
> <thomas.petazzoni at bootlin.com>
> Subject: [PATCH 07/17] net: mdio: mscc-miim: Handle the switch
> reset
> 
> The mscc-miim device can be impacted by the switch reset, at least when this
> device is part of the LAN966x PCI device.
> 
> Handle this newly added (optional) resets property.
> 
> Signed-off-by: Herve Codina <herve.codina at bootlin.com>
> ---
>  drivers/net/mdio/mdio-mscc-miim.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/drivers/net/mdio/mdio-mscc-miim.c b/drivers/net/mdio/mdio-
> mscc-miim.c
> index c29377c85307..6a6c1768f533 100644
> --- a/drivers/net/mdio/mdio-mscc-miim.c
> +++ b/drivers/net/mdio/mdio-mscc-miim.c
> @@ -19,6 +19,7 @@
>  #include <linux/platform_device.h>
>  #include <linux/property.h>
>  #include <linux/regmap.h>
> +#include <linux/reset.h>
> 
>  #define MSCC_MIIM_REG_STATUS		0x0
>  #define		MSCC_MIIM_STATUS_STAT_PENDING	BIT(2)
> @@ -270,11 +271,18 @@ static int mscc_miim_probe(struct platform_device
> *pdev)  {
>  	struct device_node *np = pdev->dev.of_node;
>  	struct regmap *mii_regmap, *phy_regmap;
> +	struct reset_control *reset;

Please follow reverse x-mass tree order

>  	struct device *dev = &pdev->dev;
>  	struct mscc_miim_dev *miim;
>  	struct mii_bus *bus;
>  	int ret;
> 
> +	reset = devm_reset_control_get_optional_shared(dev, "switch");
> +	if (IS_ERR(reset))
> +		return dev_err_probe(dev, PTR_ERR(reset), "Failed to get
> reset\n");
> +
> +	reset_control_reset(reset);
> +
>  	mii_regmap = ocelot_regmap_from_resource(pdev, 0,
> 
> &mscc_miim_regmap_config);
>  	if (IS_ERR(mii_regmap))
> --
> 2.44.0
>