Thread (2 messages) flat view 2 messages, 2 authors, 2021-06-16

Re: [PATCH net-next v4 02/10] net: sparx5: add the basic sparx5 driver

From: Philipp Zabel <p.zabel@pengutronix.de>
Date: 2021-06-15 13:22:53
Also in: lkml, netdev

Hi Steen,

On Tue, 2021-06-15 at 10:50 +0200, Steen Hegelund wrote:
This adds the Sparx5 basic SwitchDev driver framework with IO range
mapping, switch device detection and core clock configuration.

Support for ports, phylink, netdev, mactable etc. are in the following
patches.

Signed-off-by: Steen Hegelund <steen.hegelund@microchip.com>
Signed-off-by: Bjarni Jonasson <redacted>
Signed-off-by: Lars Povlsen <redacted>
[...]
quoted hunk
diff --git a/drivers/net/ethernet/microchip/sparx5/sparx5_main.c b/drivers/net/ethernet/microchip/sparx5/sparx5_main.c
new file mode 100644
index 000000000000..892bbbaacbd6
--- /dev/null
+++ b/drivers/net/ethernet/microchip/sparx5/sparx5_main.c
@@ -0,0 +1,743 @@
[...]
+static int mchp_sparx5_probe(struct platform_device *pdev)
+{
[...]
+	/* Do switch core reset if available */
+	reset = devm_reset_control_get_optional_shared(&pdev->dev, "switch");
Please don't ignore errors here. For example:

	if (IS_ERR(reset))
		return dev_err_probe(&pdev->dev, PTR_ERR(reset),
				     "Failed to get reset.\n");
	reset_control_reset(reset);
regards
Philipp

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help