diff mbox

hw/omap_gpmc.c: Add missing 'break's to fix 8 bit NAND writes

Message ID 1320860543-9596-1-git-send-email-peter.maydell@linaro.org
State Accepted
Commit c0465d1a1d4c9562cfa7e91f6c31ea1bce22052c
Headers show

Commit Message

Peter Maydell Nov. 9, 2011, 5:42 p.m. UTC
Add missing 'break' statements which would have meant that writing
to an 8 bit NAND device was broken. Spotted by Coverity (see bug
887883).

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
This is an embarrassing bug...

 hw/omap_gpmc.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

Comments

Anthony Liguori Nov. 9, 2011, 7:45 p.m. UTC | #1
On 11/09/2011 11:42 AM, Peter Maydell wrote:
> Add missing 'break' statements which would have meant that writing
> to an 8 bit NAND device was broken. Spotted by Coverity (see bug
> 887883).
>
> Signed-off-by: Peter Maydell<peter.maydell@linaro.org>

Applied.  Thanks.

Regards,

Anthony Liguori

> ---
> This is an embarrassing bug...
>
>   hw/omap_gpmc.c |    2 ++
>   1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/hw/omap_gpmc.c b/hw/omap_gpmc.c
> index 7fc82a2..414f9f5 100644
> --- a/hw/omap_gpmc.c
> +++ b/hw/omap_gpmc.c
> @@ -180,6 +180,7 @@ static void omap_nand_setio(DeviceState *dev, uint64_t value,
>               nand_setio(dev, (value>>  24)&  0xff);
>               break;
>           }
> +        break;
>       case OMAP_GPMC_16BIT:
>           switch (size) {
>           case 1:
> @@ -195,6 +196,7 @@ static void omap_nand_setio(DeviceState *dev, uint64_t value,
>               nand_setio(dev, (value>>  16)&  0xffff);
>               break;
>           }
> +        break;
>       }
>   }
>
diff mbox

Patch

diff --git a/hw/omap_gpmc.c b/hw/omap_gpmc.c
index 7fc82a2..414f9f5 100644
--- a/hw/omap_gpmc.c
+++ b/hw/omap_gpmc.c
@@ -180,6 +180,7 @@  static void omap_nand_setio(DeviceState *dev, uint64_t value,
             nand_setio(dev, (value >> 24) & 0xff);
             break;
         }
+        break;
     case OMAP_GPMC_16BIT:
         switch (size) {
         case 1:
@@ -195,6 +196,7 @@  static void omap_nand_setio(DeviceState *dev, uint64_t value,
             nand_setio(dev, (value >> 16) & 0xffff);
             break;
         }
+        break;
     }
 }