From: Spencer Oliver <spen@spen-soft.co.uk>
Date: Fri, 10 Aug 2012 15:17:24 +0000 (+0100)
Subject: flash: fix FC_FLEX_RAM class code path
X-Git-Tag: v0.6.0-rc2~27
X-Git-Url: https://review.openocd.org/gitweb?p=openocd.git;a=commitdiff_plain;h=fb525cdd76982b1abf69974f3ed9327876df0036;ds=sidebyside

flash: fix FC_FLEX_RAM class code path

If the flash class was defined as FC_FLEX_RAM then this would always drop
through to the default handler.

This bug was found by clang, so untested.

Change-Id: I2d9fe6415dd216728a145519400f7b9ef1bd3c3a
Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk>
Reviewed-on: http://openocd.zylin.com/773
Tested-by: jenkins
Reviewed-by: Freddie Chopin <freddie.chopin@gmail.com>
Reviewed-by: Mathias Küster <kesmtp@freenet.de>
---

diff --git a/src/flash/nor/kinetis.c b/src/flash/nor/kinetis.c
index cf36104a44..e0bb5c20ba 100644
--- a/src/flash/nor/kinetis.c
+++ b/src/flash/nor/kinetis.c
@@ -587,6 +587,8 @@ static int kinetis_read_part_info(struct flash_bank *bank)
 					LOG_DEBUG("FlexRAM bank %d already configured okay",
 						  kinfo->bank_ordinal);
 				}
+				break;
+
 			default:
 				LOG_WARNING("Unknown or inconsistent flash class");
 				reassign = 1;