[meta-xilinx] Custom u-boot-xlnx defconfig breaks spl/u-boot-spl recipe?

[Manjukumar Harthikote Matha]

> -----Original Message-----
> From: meta-xilinx-bounces at yoctoproject.org [mailto:meta-xilinx-
> bounces at yoctoproject.org] On Behalf Of Nathan Rossi
> Sent: Tuesday, November 21, 2017 8:29 AM
> To: Giordon Stark <kratsg at gmail.com>
> Cc: meta-xilinx at yoctoproject.org
> Subject: Re: [meta-xilinx] Custom u-boot-xlnx defconfig breaks spl/u-boot-spl
> recipe?
> 
> On 22 November 2017 at 01:28, Giordon Stark <kratsg at gmail.com> wrote:
> > Indeed. I'm now realizing that `make menuconfig` isn't really
> > inheriting from the xilinx_zynqmp defconfig board at all. So I'm going
> > to copy that file, and disable all the SPL/SDHCI support there. Thanks!
> >
> > [side note: why does bitbake -c menuconfig not work for u-boot-xlnx?]
> 
> It is more that u-boot.inc in oe-core doesn't support menuconfig rather that u-boot-
> xlnx not supporting it. I think this stack overflow answer covers the details and
> solutions.
> https://stackoverflow.com/a/43222625
> 

We are working on a patch to submit to oe-core to support menuconfig for u-boot. However, the upstream recipe for u-boot is little complicated to support menuconfig, since it builds multiple defconfig using the same build area. If you have only one defconfig to support it is easier to add the menuconfig, as we did here
https://github.com/Xilinx/meta-xilinx/blob/rel-v2017.3/recipes-bsp/u-boot/u-boot.inc#L18
https://github.com/Xilinx/meta-xilinx/blob/rel-v2017.3/recipes-bsp/u-boot/u-boot.inc#L76-L95

One of the ideas is to allow u-boot source code to be in a shared directory and build multiple defconfigs in different builds area. Any other ideas before we submit the RFC patch would be highly appreciated :)

> Also keep in mind that u-boot still has not fully switched from header configs to
> kconfig, so you might still have to config with #defines for certain features.
> 

Agreed to this point.

Thanks,
Manju


> Regards,
> Nathan
> 
> >
> > Giordon
> >
> > On Tue, Nov 21, 2017 at 9:25 AM Nathan Rossi <nathan at nathanrossi.com> wrote:
> >>
> >> On 22 November 2017 at 01:12, Giordon Stark <kratsg at gmail.com> wrote:
> >> > Hi,
> >> >
> >> > Here's the log file:
> >> > https://gist.github.com/355ccc2aecb8887a03699d68b5a8c2b5
> >> >
> >> > and here's the defconfig patch that I applied to u-boot-xlnx:
> >> > https://gist.github.com/07ade34baf39b8717d879760a15a274c (adds a
> >> > <machine>_defconfig where machine=gfex-prototype3). I'm not sure
> >> > what went wrong...
> >>
> >> Your config is missing 'CONFIG_ZYNQ_SDHCI=y'. But you enable MMC
> >> options specifically 'CONFIG_SPL_MMC_SUPPORT=y'.
> >>
> >> >
> >> > Right now, my machine configuration is somewhat bare, and I haven't
> >> > actually enabled SPL yet. That said, it's curious I'm seeing a
> >> > problem with that when
> >>
> >> Your config says otherwise 'CONFIG_SPL=y'. :)
> >>
> >> Regards,
> >> Nathan
> >>
> >> > changing out the defconfig?
> >> >
> >> > Giordon
> >> >
> >> > --
> >> > _______________________________________________
> >> > meta-xilinx mailing list
> >> > meta-xilinx at yoctoproject.org
> >> > https://lists.yoctoproject.org/listinfo/meta-xilinx
> >> >
> --
> _______________________________________________
> meta-xilinx mailing list
> meta-xilinx at yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-xilinx