[U-Boot] [PATCH 02/14] spl: mmc: Fix build without LIBCOMMON_SUPPORT
Marek Vasut
marex at denx.de
Mon Nov 28 03:53:33 CET 2016
On 11/28/2016 03:33 AM, Jaehoon Chung wrote:
> On 11/27/2016 05:58 AM, Marek Vasut wrote:
>> 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.
>
> I don't know..but i remembered that i suggested about putting just "break;" end of bracket.
> how about?
Sure, that works too.
--
Best regards,
Marek Vasut
More information about the U-Boot
mailing list