From mboxrd@z Thu Jan  1 00:00:00 1970
From: Pragnesh Patel <pragnesh.patel@sifive.com>
Date: Fri, 24 Jan 2020 08:06:23 +0000
Subject: [PATCH v3 03/10] lib: Makefile: build crc7.c when CONFIG_MMC_SPI
In-Reply-To: <CAMty3ZAXoNBkp=YBuMgvTzwbbmMxmdFodLiou1+vRLLBZct8qA@mail.gmail.com>
References: <20200124055026.30787-1-pragnesh.patel@sifive.com>
 <20200124055026.30787-4-pragnesh.patel@sifive.com>
 <CAMty3ZAXoNBkp=YBuMgvTzwbbmMxmdFodLiou1+vRLLBZct8qA@mail.gmail.com>
Message-ID: <MN2PR13MB36164AB4B65BE5E712DD227FE50E0@MN2PR13MB3616.namprd13.prod.outlook.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
To: u-boot@lists.denx.de


>-----Original Message-----
>From: Jagan Teki <jagan@amarulasolutions.com>
>Sent: 24 January 2020 11:58
>To: Pragnesh Patel <pragnesh.patel@sifive.com>
>Cc: U-Boot-Denx <u-boot@lists.denx.de>; Atish Patra
><atish.patra@wdc.com>; palmerdabbelt at google.com; Bin Meng
><bmeng.cn@gmail.com>; Paul Walmsley ( Sifive)
><paul.walmsley@sifive.com>; Troy Benjegerdes ( Sifive)
><troy.benjegerdes@sifive.com>; Anup Patel <anup.patel@wdc.com>; Sagar
>Kadam <sagar.kadam@sifive.com>; Simon Glass <sjg@chromium.org>;
>AKASHI Takahiro <takahiro.akashi@linaro.org>; Marek Beh=C3=BAn
><marek.behun@nic.cz>; Philippe Reynes <philippe.reynes@softathome.com>;
>Alexander Graf <agraf@csgraf.de>; Peng Fan <peng.fan@nxp.com>
>Subject: Re: [PATCH v3 03/10] lib: Makefile: build crc7.c when
>CONFIG_MMC_SPI
>
>On Fri, Jan 24, 2020 at 11:21 AM Pragnesh Patel <pragnesh.patel@sifive.com>
>wrote:
>>
>> When build U-boot SPL, meet an issue of undefined reference to 'crc7'
>> for drivers/mmc/mmc_spi.c, so let's compile crc7.c when CONFIG_MMC_SPI
>> selected.
>>
>> Signed-off-by: Pragnesh Patel <pragnesh.patel@sifive.com>
>> Reviewed-by: Anup Patel <anup.patel@wdc.com>
>> ---
>>  lib/Makefile | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/lib/Makefile b/lib/Makefile index 51eba80b89..d21e0a6b3a
>> 100644
>> --- a/lib/Makefile
>> +++ b/lib/Makefile
>> @@ -78,6 +78,7 @@ endif
>>
>>  ifdef CONFIG_SPL_BUILD
>>  obj-$(CONFIG_SPL_YMODEM_SUPPORT) +=3D crc16.o
>> +obj-$(CONFIG_MMC_SPI) +=3D crc7.o
>
>It is invalid to mark crc7 to MMC_SPI, try to add Kconfig option for CRC7.=
 like
>this.
>https://paste.ubuntu.com/p/kSnkDWpTgS/

Thanks for pointing me. How about if I will add like this:

config SPL_CRC7_SUPPORT
       bool "Support CRC7 hashing drivers"
       default y if MMC_SPI
       help
        Enable CRC7 hashing for drivers which are using in SPL.

Any comments?