On 10/15/2016 07:10 PM, Max Krummenacher wrote:
> If CONFIG_SPL_LIBCOMMON_SUPPORT is not defined there is a lone case statement
> at the end of the switch leading to a compile error.
> Remove the offending case statement.
> 
> | common/spl/spl_mmc.c:339:7: error: label at end of compound statement
> 
> Signed-off-by: Max Krummenacher <max.krummenac...@toradex.com>
> ---
> 
>  common/spl/spl_mmc.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/common/spl/spl_mmc.c b/common/spl/spl_mmc.c
> index c674e61..367b4e4 100644
> --- a/common/spl/spl_mmc.c
> +++ b/common/spl/spl_mmc.c
> @@ -342,7 +342,6 @@ static int spl_mmc_load_image(struct spl_image_info 
> *spl_image,
>                       return err;
>  
>               break;
> -     case MMCSD_MODE_UNDEFINED:

This patch is wrong -- in case CONFIG_SPL_LIBCOMMON_SUPPORT is enabled
and mode is MMCSD_MODE_UNDEFINED, the message in the puts() below would
be printed. After applying this change, the message won't be printed .

The fix is probably something like:

case foo:
default:
#ifdef CONFIG_BAR
    puts();
#endif
    break;

>  #ifdef CONFIG_SPL_LIBCOMMON_SUPPORT
>       default:
>               puts("spl: mmc: wrong boot mode\n");
> 


-- 
Best regards,
Marek Vasut
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to