From: Steen Hegelund <steen.hegelund@microchip.com> To: Philipp Zabel <p.zabel@pengutronix.de>, "David S. Miller" <davem@davemloft.net>, Jakub Kicinski <kuba@kernel.org> Cc: Andrew Lunn <andrew@lunn.ch>, Russell King <linux@armlinux.org.uk>, Microchip Linux Driver Support <UNGLinuxDriver@microchip.com>, "Alexandre Belloni" <alexandre.belloni@bootlin.com>, Madalin Bucur <madalin.bucur@oss.nxp.com>, Mark Einon <mark.einon@gmail.com>, "Masahiro Yamada" <masahiroy@kernel.org>, Arnd Bergmann <arnd@arndb.de>, Simon Horman <simon.horman@netronome.com>, <netdev@vger.kernel.org>, <linux-kernel@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org>, Bjarni Jonasson <bjarni.jonasson@microchip.com>, Lars Povlsen <lars.povlsen@microchip.com> Subject: Re: [PATCH net-next v4 02/10] net: sparx5: add the basic sparx5 driver Date: Wed, 16 Jun 2021 08:20:24 +0200 [thread overview] Message-ID: <cb6ae093f6db85dd62ac9cafc14362ce18987e6c.camel@microchip.com> (raw) In-Reply-To: <2800a872ff4260fc7a942f2f2ea1c9e3603b13d7.camel@pengutronix.de> Hi Philipp, Thanks for your comments, On Tue, 2021-06-15 at 15:22 +0200, Philipp Zabel wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > 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 <bjarni.jonasson@microchip.com> > > Signed-off-by: Lars Povlsen <lars.povlsen@microchip.com> > [...] > > 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); Yes, I will add an error message. > > regards > Philipp -- BR Steen -=-=-=-=-=-=-=-=-=-=-=-=-=-= steen.hegelund@microchip.com
WARNING: multiple messages have this Message-ID (diff)
From: Steen Hegelund <steen.hegelund@microchip.com> To: Philipp Zabel <p.zabel@pengutronix.de>, "David S. Miller" <davem@davemloft.net>, Jakub Kicinski <kuba@kernel.org> Cc: Andrew Lunn <andrew@lunn.ch>, Russell King <linux@armlinux.org.uk>, Microchip Linux Driver Support <UNGLinuxDriver@microchip.com>, "Alexandre Belloni" <alexandre.belloni@bootlin.com>, Madalin Bucur <madalin.bucur@oss.nxp.com>, Mark Einon <mark.einon@gmail.com>, "Masahiro Yamada" <masahiroy@kernel.org>, Arnd Bergmann <arnd@arndb.de>, Simon Horman <simon.horman@netronome.com>, <netdev@vger.kernel.org>, <linux-kernel@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org>, Bjarni Jonasson <bjarni.jonasson@microchip.com>, Lars Povlsen <lars.povlsen@microchip.com> Subject: Re: [PATCH net-next v4 02/10] net: sparx5: add the basic sparx5 driver Date: Wed, 16 Jun 2021 08:20:24 +0200 [thread overview] Message-ID: <cb6ae093f6db85dd62ac9cafc14362ce18987e6c.camel@microchip.com> (raw) In-Reply-To: <2800a872ff4260fc7a942f2f2ea1c9e3603b13d7.camel@pengutronix.de> Hi Philipp, Thanks for your comments, On Tue, 2021-06-15 at 15:22 +0200, Philipp Zabel wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > 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 <bjarni.jonasson@microchip.com> > > Signed-off-by: Lars Povlsen <lars.povlsen@microchip.com> > [...] > > 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); Yes, I will add an error message. > > regards > Philipp -- BR Steen -=-=-=-=-=-=-=-=-=-=-=-=-=-= steen.hegelund@microchip.com _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2021-06-16 6:20 UTC|newest] Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-06-15 8:50 [PATCH net-next v4 00/10] Adding the Sparx5 Switch Driver Steen Hegelund 2021-06-15 8:50 ` Steen Hegelund 2021-06-15 8:50 ` [PATCH net-next v4 01/10] dt-bindings: net: sparx5: Add sparx5-switch bindings Steen Hegelund 2021-06-15 8:50 ` Steen Hegelund 2021-06-15 8:50 ` [PATCH net-next v4 02/10] net: sparx5: add the basic sparx5 driver Steen Hegelund 2021-06-15 13:22 ` Philipp Zabel 2021-06-15 13:22 ` Philipp Zabel 2021-06-16 6:20 ` Steen Hegelund [this message] 2021-06-16 6:20 ` Steen Hegelund 2021-06-15 8:50 ` [PATCH net-next v4 03/10] net: sparx5: add hostmode with phylink support Steen Hegelund 2021-06-15 8:50 ` Steen Hegelund 2021-06-21 14:26 ` Russell King (Oracle) 2021-06-21 14:26 ` Russell King (Oracle) 2021-06-23 13:04 ` Steen Hegelund 2021-06-23 13:04 ` Steen Hegelund 2021-06-15 8:50 ` [PATCH net-next v4 04/10] net: sparx5: add port module support Steen Hegelund 2021-06-15 8:50 ` Steen Hegelund 2021-06-21 14:33 ` Russell King (Oracle) 2021-06-21 14:33 ` Russell King (Oracle) 2021-06-23 13:05 ` Steen Hegelund 2021-06-23 13:05 ` Steen Hegelund 2021-06-15 8:50 ` [PATCH net-next v4 05/10] net: sparx5: add mactable support Steen Hegelund 2021-06-15 8:50 ` Steen Hegelund 2021-06-15 8:50 ` [PATCH net-next v4 06/10] net: sparx5: add vlan support Steen Hegelund 2021-06-15 8:50 ` Steen Hegelund 2021-06-15 8:50 ` [PATCH net-next v4 07/10] net: sparx5: add switching support Steen Hegelund 2021-06-15 8:50 ` Steen Hegelund 2021-06-15 8:50 ` [PATCH net-next v4 08/10] net: sparx5: add calendar bandwidth allocation support Steen Hegelund 2021-06-15 8:50 ` Steen Hegelund 2021-06-15 8:50 ` [PATCH net-next v4 09/10] net: sparx5: add ethtool configuration and statistics support Steen Hegelund 2021-06-15 8:50 ` Steen Hegelund 2021-06-15 8:50 ` [PATCH net-next v4 10/10] arm64: dts: sparx5: Add the Sparx5 switch node Steen Hegelund 2021-06-15 8:50 ` Steen Hegelund
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=cb6ae093f6db85dd62ac9cafc14362ce18987e6c.camel@microchip.com \ --to=steen.hegelund@microchip.com \ --cc=UNGLinuxDriver@microchip.com \ --cc=alexandre.belloni@bootlin.com \ --cc=andrew@lunn.ch \ --cc=arnd@arndb.de \ --cc=bjarni.jonasson@microchip.com \ --cc=davem@davemloft.net \ --cc=kuba@kernel.org \ --cc=lars.povlsen@microchip.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux@armlinux.org.uk \ --cc=madalin.bucur@oss.nxp.com \ --cc=mark.einon@gmail.com \ --cc=masahiroy@kernel.org \ --cc=netdev@vger.kernel.org \ --cc=p.zabel@pengutronix.de \ --cc=simon.horman@netronome.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.