Hi Bin, >-----Original Message----- >From: Bin Meng <bmeng...@gmail.com> >Sent: 11 March 2020 19:22 >To: Pragnesh Patel <pragnesh.pa...@sifive.com> >Cc: U-Boot Mailing List <u-boot@lists.denx.de>; Atish Patra ><atish.pa...@wdc.com>; Palmer Dabbelt <palmerdabb...@google.com>; Paul >Walmsley <paul.walms...@sifive.com>; Jagan Teki ><ja...@amarulasolutions.com>; Troy Benjegerdes ><troy.benjeger...@sifive.com>; Anup Patel <anup.pa...@wdc.com>; Sagar >Kadam <sagar.ka...@sifive.com>; Rick Chen <r...@andestech.com>; Lukasz >Majewski <lu...@denx.de>; Simon Goldschmidt ><simon.k.r.goldschm...@gmail.com>; Simon Glass <s...@chromium.org>; >Markus Klotzbuecher <markus.klotzbuec...@kistler.com>; Baruch Siach ><bar...@tkos.co.il>; Anatolij Gustschin <ag...@denx.de>; Thomas Hebb ><tommyh...@gmail.com>; AKASHI Takahiro <takahiro.aka...@linaro.org>; >Marek BehĂșn <marek.be...@nic.cz>; Philippe Reynes ><philippe.rey...@softathome.com>; Heinrich Schuchardt ><xypron.g...@gmx.de>; Peng Fan <peng....@nxp.com> >Subject: Re: [PATCH v5 04/14] lib: Makefile: build crc7.c when >CONFIG_MMC_SPI > >On Wed, Mar 11, 2020 at 3:04 PM Pragnesh Patel ><pragnesh.pa...@sifive.com> wrote: >> >> When build U-boot SPL, meet an issue of undefined reference to > >nits: U-Boot
Will update. > >> 'crc7' for drivers/mmc/mmc_spi.c, so let's compile crc7.c when >> CONFIG_MMC_SPI selected. >> >> Signed-off-by: Pragnesh Patel <pragnesh.pa...@sifive.com> >> --- >> common/spl/Kconfig | 7 +++++++ >> lib/Makefile | 1 + >> 2 files changed, 8 insertions(+) >> >> diff --git a/common/spl/Kconfig b/common/spl/Kconfig index >> b03a476b9f..f93f552f5e 100644 >> --- a/common/spl/Kconfig >> +++ b/common/spl/Kconfig >> @@ -401,6 +401,13 @@ config SPL_CRC32_SUPPORT >> for detected accidental image corruption. For secure applications >> you >> should consider SHA1 or SHA256. >> >> +config SPL_CRC7_SUPPORT >> + bool "Support CRC7" >> + default y if MMC_SPI > >This dependency should be set in the MMPC_SPI driver with a "select". >The driver cannot work without it. Okay, will update in v6. Thanks for the review. > >> + help >> + Enable CRC7 hashing for drivers which are using in SPL. >> + This is a 32-bit checksum value that can be used to verify images. >> + >> config SPL_MD5_SUPPORT >> bool "Support MD5" >> depends on SPL_FIT >> diff --git a/lib/Makefile b/lib/Makefile index 15259d0473..7a50aa56ef >> 100644 >> --- a/lib/Makefile >> +++ b/lib/Makefile >> @@ -78,6 +78,7 @@ endif >> >> ifdef CONFIG_SPL_BUILD >> obj-$(CONFIG_SPL_YMODEM_SUPPORT) += crc16.o >> +obj-$(CONFIG_SPL_CRC7_SUPPORT) += crc7.o >> obj-$(CONFIG_$(SPL_TPL_)HASH_SUPPORT) += crc16.o obj-y += >> net_utils.o endif >> -- > >Regards, >Bin