On 12/01/2016 11:10 AM, Jaehoon Chung wrote:
> Hi Marek,
> 
> On 11/28/2016 11:53 AM, Marek Vasut wrote:
>> 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 <ma...@denx.de>
>>>>>> Cc: Pantelis Antoniou <pa...@antoniou-consulting.com>
>>>>>> Cc: Tom Rini <tr...@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.
> 
> I didn't see this patch in your next version, right?

It doesn't seem to be necessary anymore.

-- 
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