[U-Boot] [PATCH 02/14] spl: mmc: Fix build without LIBCOMMON_SUPPORT

Marek Vasut marex at denx.de
Sat Nov 26 21:58:06 CET 2016


On 11/26/2016 09:45 PM, Tom Rini wrote:
> On Fri, Nov 25, 2016 at 11:32:23PM +0100, Marek Vasut wrote:
>> If CONFIG_SPL_LIBCOMMON_SUPPORT is undefined, the following error
>> will happen, so fix it.
>>
>> In file included from common/spl/spl_mmc.c:11:0:
>> common/spl/spl_mmc.c: In function ‘spl_mmc_load_image’:
>> include/spl.h:18:30: error: label at end of compound statement
>>  #define MMCSD_MODE_UNDEFINED 0
>>                               ^
>> common/spl/spl_mmc.c:335:7: note: in expansion of macro ‘MMCSD_MODE_UNDEFINED’
>>   case MMCSD_MODE_UNDEFINED:
>>        ^
>> Signed-off-by: Marek Vasut <marex at denx.de>
>> Cc: Pantelis Antoniou <panto at antoniou-consulting.com>
>> Cc: Tom Rini <trini at konsulko.com>
>> ---
>>  common/spl/spl_mmc.c | 4 +++-
>>  1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/common/spl/spl_mmc.c b/common/spl/spl_mmc.c
>> index 0b681c2..43c1030 100644
>> --- a/common/spl/spl_mmc.c
>> +++ b/common/spl/spl_mmc.c
>> @@ -343,10 +343,12 @@ static int spl_mmc_load_image(struct spl_image_info *spl_image,
>>  
>>  		break;
>>  	case MMCSD_MODE_UNDEFINED:
>> -#ifdef CONFIG_SPL_LIBCOMMON_SUPPORT
>>  	default:
>> +		{
>> +#ifdef CONFIG_SPL_LIBCOMMON_SUPPORT
>>  		puts("spl: mmc: wrong boot mode\n");
>>  #endif
>> +		}
>>  	}
>>  
>>  	return err;
> 
> I think we have something slightly off here.  Is
> CONFIG_SPL_SERIAL_SUPPORT also enabled?  That would turn this into a
> no-op as puts becomes a do { } while(0), if disabled.  So I think we
> should just remove the ifdef here.

No, series does not fit into SPL on this platform.

-- 
Best regards,
Marek Vasut


More information about the U-Boot mailing list