From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Fri, 30 Apr 2021 06:40:02 +0200 Subject: [PATCH v1 08/10] net: mvpp2: fix missing switch case break In-Reply-To: References: <20210427132718.645043-1-sr@denx.de> <20210427152713.v1.8.I90a9ae45ab51c6f846de0ed1499bf60ddafebb83@changeid> Message-ID: <79ff6204-883d-cce4-a713-9deb30b414d9@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 29.04.21 22:24, Ramon Fried wrote: > On Tue, Apr 27, 2021 at 4:28 PM Stefan Roese wrote: >> >> From: Ben Peled >> >> Signed-off-by: Ben Peled >> Reviewed-by: Stefan Chulski >> Reviewed-by: Kostya Porotchkin >> Tested-by: sa_ip-sw-jenkins > Please remove this, Jenkins is not a person. I've left these lines in because most of these patches are plain cherry-picked from the Marvell SDK. Should I resubmit this patch only or the complete patchset? Or could you perhaps remove this line while collecting the patches for a pull request? Thanks, Stefan >> Signed-off-by: Stefan Roese >> --- >> >> drivers/net/mvpp2.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/drivers/net/mvpp2.c b/drivers/net/mvpp2.c >> index 61a0ea0894a7..3d920e85ffef 100644 >> --- a/drivers/net/mvpp2.c >> +++ b/drivers/net/mvpp2.c >> @@ -3247,9 +3247,11 @@ static int gop_gmac_mode_cfg(struct mvpp2_port *port) >> >> case PHY_INTERFACE_MODE_1000BASEX: >> gop_gmac_1000basex_cfg(port); >> + break; >> >> case PHY_INTERFACE_MODE_2500BASEX: >> gop_gmac_2500basex_cfg(port); >> + break; >> >> case PHY_INTERFACE_MODE_RGMII: >> case PHY_INTERFACE_MODE_RGMII_ID: >> -- >> 2.31.1 >> Viele Gr??e, Stefan -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr at denx.de