Re: [PATCH] EDAC, sb_edac: fix missing break in switch

From: Borislav Petkov
Date: Thu Oct 19 2017 - 04:57:51 EST


On Mon, Oct 16, 2017 at 12:40:29PM -0500, Gustavo A. R. Silva wrote:
> Add missing break statement in order to prevent the code from falling
> through.
>
> Signed-off-by: Gustavo A. R. Silva <garsilva@xxxxxxxxxxxxxx>
> ---
> drivers/edac/sb_edac.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/edac/sb_edac.c b/drivers/edac/sb_edac.c
> index 72b98a0..f34430f 100644
> --- a/drivers/edac/sb_edac.c
> +++ b/drivers/edac/sb_edac.c
> @@ -2485,6 +2485,7 @@ static int ibridge_mci_bind_devs(struct mem_ctl_info *mci,
> case PCI_DEVICE_ID_INTEL_IBRIDGE_IMC_HA0_TA:
> case PCI_DEVICE_ID_INTEL_IBRIDGE_IMC_HA1_TA:
> pvt->pci_ta = pdev;
> + break;
> case PCI_DEVICE_ID_INTEL_IBRIDGE_IMC_HA0_RAS:
> case PCI_DEVICE_ID_INTEL_IBRIDGE_IMC_HA1_RAS:
> pvt->pci_ras = pdev;
> --

Applied, thanks.

--
Regards/Gruss,
Boris.

Good mailing practices for 400: avoid top-posting and trim the reply.