On Thu, Sep 14, 2023 at 05:28:12PM +0530, Chirag Shilwant wrote: > - U-boot recipe in OE-Core supports out-of-tree config fragments that are > passed via SRC_URI and automatically merges all *.cfg files as fragments. > This makes specifying config fragments in the machine configuration a bit > difficult. Hence, add a logic which will ensure we handle u-boot config > fragments using a new variable UBOOT_CONFIG_FRAGMENTS. > > - The u-boot-mergeconfig.inc will allow us to build u-boot with list of > config fragments specified in UBOOT_CONFIG_FRAGMENTS along with the base > defconfig mentioned in UBOOT_MACHINE. > > - Include u-boot-mergeconfig.inc in u-boot-ti.inc
Thanks for addressing majority of my comments! Would be nice to keep the standard formatting for the text and line wrap under 80 characters, instead of writing long paragraphs as a single line. > Signed-off-by: Chirag Shilwant <[email protected]> > --- Also, patch revision logs here are useful to keep track of what's changed between revisions. > meta-ti-bsp/recipes-bsp/u-boot/u-boot-mergeconfig.inc | 7 +++++++ > meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc | 1 + > 2 files changed, 8 insertions(+) > create mode 100644 meta-ti-bsp/recipes-bsp/u-boot/u-boot-mergeconfig.inc > > diff --git a/meta-ti-bsp/recipes-bsp/u-boot/u-boot-mergeconfig.inc > b/meta-ti-bsp/recipes-bsp/u-boot/u-boot-mergeconfig.inc > new file mode 100644 > index 00000000..e2ae5894 > --- /dev/null > +++ b/meta-ti-bsp/recipes-bsp/u-boot/u-boot-mergeconfig.inc > @@ -0,0 +1,7 @@ > +do_configure:append () { > + if [ -n "${UBOOT_CONFIG_FRAGMENTS}" ] Since we are going with the minimal approach for now, would be nice to also check that UBOOT_MACHINE is set before using fragments. > + then > + oe_runmake -C ${S} O=${B} ${UBOOT_MACHINE} ${UBOOT_CONFIG_FRAGMENTS} > + oe_runmake -C ${S} O=${B} olddefconfig > + fi > +} > diff --git a/meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc > b/meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc > index f3285c23..5292517b 100644 > --- a/meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc > +++ b/meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc > @@ -7,6 +7,7 @@ SPL_BINARY ?= "MLO" > > require ${COREBASE}/meta/recipes-bsp/u-boot/u-boot-common.inc > require ${COREBASE}/meta/recipes-bsp/u-boot/u-boot.inc > +require u-boot-mergeconfig.inc > > FILESEXTRAPATHS:prepend := "${THISDIR}/u-boot:" > > -- > 2.34.1
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#16969): https://lists.yoctoproject.org/g/meta-ti/message/16969 Mute This Topic: https://lists.yoctoproject.org/mt/101356499/21656 Group Owner: [email protected] Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/leave/6695321/21656/1393940836/xyzzy [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
