From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jagan Teki <jagan@amarulasolutions.com>
Date: Mon, 27 Jan 2020 13:28:40 +0530
Subject: [PATCH v3 03/10] lib: Makefile: build crc7.c when CONFIG_MMC_SPI
In-Reply-To: <MN2PR13MB36164AB4B65BE5E712DD227FE50E0@MN2PR13MB3616.namprd13.prod.outlook.com>
References: <20200124055026.30787-1-pragnesh.patel@sifive.com>
 <20200124055026.30787-4-pragnesh.patel@sifive.com>
 <CAMty3ZAXoNBkp=YBuMgvTzwbbmMxmdFodLiou1+vRLLBZct8qA@mail.gmail.com>
 <MN2PR13MB36164AB4B65BE5E712DD227FE50E0@MN2PR13MB3616.namprd13.prod.outlook.com>
Message-ID: <CAMty3ZAFwkFEry2=K95APcSSvqX3T0yNZj9gneDcdT562mg4TA@mail.gmail.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

On Fri, Jan 24, 2020 at 1:36 PM Pragnesh Patel
<pragnesh.patel@sifive.com> wrote:
>
>
> >-----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.c=
om>
> >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 CRC=
7. 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?

Make sense.