From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jaehoon Chung Date: Thu, 01 Dec 2016 19:10:53 +0900 Subject: [U-Boot] [PATCH 02/14] spl: mmc: Fix build without LIBCOMMON_SUPPORT In-Reply-To: <5de3da32-d695-85a1-0bad-b588a06eed83@denx.de> References: <20161125223235.3434-1-marex@denx.de> <20161125223235.3434-2-marex@denx.de> <20161126204547.GR2546@bill-the-cat> <3634960b-afa4-15e6-e7c4-ee346a644159@samsung.com> <5de3da32-d695-85a1-0bad-b588a06eed83@denx.de> Message-ID: <669f9649-277a-6a1b-81d4-128d1d66dbf3@samsung.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 >>>>> Cc: Pantelis Antoniou >>>>> Cc: Tom Rini >>>>> --- >>>>> 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? Best Regards, Jaehoon Chung >