[PATCH] spi: fsl-dspi: Add missing breaks for switch cases

[PATCH] spi: fsl-dspi: Add missing breaks for switch cases

[Axel Lin]

Signed-off-by: Axel Lin <axel.lin-8E1dMatC8ynQT0dZR+AlfA@public.gmane.org>
---
 drivers/spi/spi-fsl-dspi.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/spi/spi-fsl-dspi.c b/drivers/spi/spi-fsl-dspi.c
index 8641b03..a37f156 100644
--- a/drivers/spi/spi-fsl-dspi.c
+++ b/drivers/spi/spi-fsl-dspi.c
@@ -320,8 +320,10 @@ static void dspi_chipselect(struct spi_device *spi, int value)
 	switch (value) {
 	case BITBANG_CS_ACTIVE:
 		pushr |= SPI_PUSHR_CONT;
+		break;
 	case BITBANG_CS_INACTIVE:
 		pushr &= ~SPI_PUSHR_CONT;
+		break;
 	}
 
 	writel(pushr, dspi->base + SPI_PUSHR);
-- 
1.8.1.2



--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] spi: fsl-dspi: Add missing breaks for switch cases

[Axel Lin]

Hi Chao and Alison,

I'm wondering if this driver really work without this patch?
Any chance to test this patch?

Regards,
Axel

2014/1/8 Axel Lin <axel.lin-8E1dMatC8ynQT0dZR+AlfA@public.gmane.org>:
> Signed-off-by: Axel Lin <axel.lin-8E1dMatC8ynQT0dZR+AlfA@public.gmane.org>
> ---
>  drivers/spi/spi-fsl-dspi.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/spi/spi-fsl-dspi.c b/drivers/spi/spi-fsl-dspi.c
> index 8641b03..a37f156 100644
> --- a/drivers/spi/spi-fsl-dspi.c
> +++ b/drivers/spi/spi-fsl-dspi.c
> @@ -320,8 +320,10 @@ static void dspi_chipselect(struct spi_device *spi, int value)
>         switch (value) {
>         case BITBANG_CS_ACTIVE:
>                 pushr |= SPI_PUSHR_CONT;
> +               break;
>         case BITBANG_CS_INACTIVE:
>                 pushr &= ~SPI_PUSHR_CONT;
> +               break;
>         }
>
>         writel(pushr, dspi->base + SPI_PUSHR);
> --
> 1.8.1.2
>
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] spi: fsl-dspi: Add missing breaks for switch cases

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 264 bytes --]

On Fri, Jan 10, 2014 at 12:19:41PM +0800, Axel Lin wrote:
> Hi Chao and Alison,
> 
> I'm wondering if this driver really work without this patch?
> Any chance to test this patch?

I applied the patch already - sorry, seems I must've forgotten
to mail out.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

RE: [PATCH] spi: fsl-dspi: Add missing breaks for switch cases

[Chao Fu]

> > Hi Chao and Alison,
> >
> > I'm wondering if this driver really work without this patch?
> > Any chance to test this patch?
> 
> I applied the patch already - sorry, seems I must've forgotten to mail
> out.


[Chao Fu] Hi Axel , Mark,
Sorry for this careless omission. But no break it is really working in former debugging,
make me can not find this omission. I will take more care of my code, thank you!
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html