[PATCH] mtd: devices: elm: fix elm_context_save() and elm_context_restore() functions

[Gupta, Pekon]

>From: Ted Juan [mailto:ted.juan at gmail.com]
>
> These two function's switch case lack the 'break' that make them always return error.
>
>Signed-off-by: Ted Juan <ted.juan at gmail.com>
>---
> drivers/mtd/devices/elm.c |    2 ++
> 1 file changed, 2 insertions(+)
>
>diff --git a/drivers/mtd/devices/elm.c b/drivers/mtd/devices/elm.c
>index 7df8694..b4f61c7 100644
>--- a/drivers/mtd/devices/elm.c
>+++ b/drivers/mtd/devices/elm.c
>@@ -475,6 +475,7 @@ static int elm_context_save(struct elm_info *info)
> 					ELM_SYNDROME_FRAGMENT_1 + offset);
> 			regs->elm_syndrome_fragment_0[i] = elm_read_reg(info,
> 					ELM_SYNDROME_FRAGMENT_0 + offset);
>+			break;
> 		default:
> 			return -EINVAL;
> 		}
>@@ -520,6 +521,7 @@ static int elm_context_restore(struct elm_info *info)
> 					regs->elm_syndrome_fragment_1[i]);
> 			elm_write_reg(info, ELM_SYNDROME_FRAGMENT_0 + offset,
> 					regs->elm_syndrome_fragment_0[i]);
>+			break;
> 		default:
> 			return -EINVAL;
> 		}
>--
>1.7.9.5

Thanks for fixing this. Apologies, my bad, I didn't test this nicely.
Also, marking it for stable, as it might break transition to low-power modes

CC: <stable at vger.kernel.org> # 3.12.x+
Acked-by: Pekon Gupta <pekon at ti.com>

with regards, pekon