Comments

Several map-related functions look like a serie of ifs, checking
widths of map. Those functions do not have any handling for default
case. Instead of fiddling with uninitialized_var in those functions,
let's just add a (correct) BUG() to the default case on those maps. This
will also allow us to catch potential errors in maps setup in future.
Signed-off-by: Dmitry Eremin-Solenikov <dmitry_eremin@mentor.com>
Cc: Artem Bityutskiy <dedekind1@gmail.com>
Cc: David Woodhouse <dwmw2@infradead.org>
---
include/linux/mtd/map.h | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)

On Thu, 2013-01-31 at 21:06 +0400, Dmitry Eremin-Solenikov wrote:
> Several map-related functions look like a serie of ifs, checking> widths of map. Those functions do not have any handling for default> case. Instead of fiddling with uninitialized_var in those functions,> let's just add a (correct) BUG() to the default case on those maps. This> will also allow us to catch potential errors in maps setup in future.> > Signed-off-by: Dmitry Eremin-Solenikov <dmitry_eremin@mentor.com>
Pushed to l2-mtd.git, thanks!