On Fri, May 5, 2023 at 6:38 PM Randolph Sapp via lists.openembedded.org <rs= [email protected]> wrote:
> From: Randolph Sapp <[email protected]> > > Fedora/Redhat and Arch are somewhat standardized on their dtb directory > structure. Let's add some flags to configure yocto to mimic that > behavior. > > Add the following variables to the kernel class: > - KERNEL_DTBDEST (controls the destination directory for dtbs) > - KERNEL_DTBVENDORED (controls if vendor subdirectories are to > be respected) > > Currently KERNEL_DTBDEST is expected to be a subdir of KERNEL_IMAGEDEST > and KERNEL_DTBVENDORED is expected to be "true"/"false". This only > applies to the package directory structure. The deploydir structure is > purposely left untouched for compatibility with existing recipes. > > By default this is configured to behave the same as the current recipe > and produce a flat dtb directory at KERNEL_IMAGEDEST. > > Signed-off-by: Randolph Sapp <[email protected]> > --- > > Well, suppose I was breaking things by submitting this to kirkstone > first. This is just the master version of the following patchset: > https://lists.openembedded.org/g/openembedded-core/message/180754 > > I'd love to get that series merged as well if this patch is acceptable. > > meta/classes-recipe/kernel-devicetree.bbclass | 22 ++++++++++++++----- > meta/classes-recipe/kernel.bbclass | 2 ++ > 2 files changed, 19 insertions(+), 5 deletions(-) > > diff --git a/meta/classes-recipe/kernel-devicetree.bbclass > b/meta/classes-recipe/kernel-devicetree.bbclass > index 4d0ecb1032..a6c6c5f227 100644 > --- a/meta/classes-recipe/kernel-devicetree.bbclass > +++ b/meta/classes-recipe/kernel-devicetree.bbclass > ... > - dtb_base_name=`basename $dtb .$dtb_ext` > dtb_path=`get_real_dtb_path_in_kernel "$dtb"` > - install -m 0644 $dtb_path > ${D}/${KERNEL_IMAGEDEST}/$dtb_base_name.$dtb_ext > + if [ ${KERNEL_DTBVENDORED} == "false" ]; then > dash doesn't like this: /bin/dash -c "if [ false == "false" ]; then echo foo; fi" add quotes or use single '='. + dtb_ext=${dtb##*.} > + dtb_base_name=`basename $dtb .$dtb_ext` > + dtb=$dtb_base_name.$dtb_ext > + fi > + install -Dm 0644 $dtb_path ${D}/${KERNEL_DTBDEST}/$dtb > done > } > > @@ -88,7 +97,10 @@ do_deploy:append() { > dtb_ext=${dtb##*.} > dtb_base_name=`basename $dtb .$dtb_ext` > install -d $deployDir > - install -m 0644 > ${D}/${KERNEL_IMAGEDEST}/$dtb_base_name.$dtb_ext > $deployDir/$dtb_base_name-${KERNEL_DTB_NAME}.$dtb_ext > + if [ ${KERNEL_DTBVENDORED} == "false" ]; then > Same here > + dtb=$dtb_base_name.$dtb_ext > + fi > + install -m 0644 ${D}/${KERNEL_DTBDEST}/$dtb > $deployDir/$dtb_base_name-${KERNEL_DTB_NAME}.$dtb_ext > if [ "${KERNEL_IMAGETYPE_SYMLINK}" = "1" ] ; then > ln -sf $dtb_base_name-${KERNEL_DTB_NAME}.$dtb_ext > $deployDir/$dtb_base_name.$dtb_ext > fi >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#180981): https://lists.openembedded.org/g/openembedded-core/message/180981 Mute This Topic: https://lists.openembedded.org/mt/98709532/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
