Re: [PATCH][net-next] cxgb4: fix missing break in switch and indent return statements

2017-10-14 Thread David Miller
From: Colin King Date: Fri, 13 Oct 2017 17:29:00 +0100 > From: Colin Ian King > > The break statement for the Macronix case is missing and will > fall through to the Winbond case and re-assign the size setting. > Fix this by adding the missing break statement. Also correctly > indent the retur

[PATCH][net-next] cxgb4: fix missing break in switch and indent return statements

2017-10-13 Thread Colin King
From: Colin Ian King The break statement for the Macronix case is missing and will fall through to the Winbond case and re-assign the size setting. Fix this by adding the missing break statement. Also correctly indent the return statements. Detected by CoverityScan, CID#1458020 ("Missing break