On 22 August 2017 at 06:37, Alistair Francis <alistai...@gmail.com> wrote: > On Mon, Aug 21, 2017 at 2:20 AM, Nathan Rossi <nat...@nathanrossi.com> wrote: >> Limit the appending/enabling of the zynqmp-pmu BBCLASSEXTEND to only >> specific recipes which are used for the building of pmu-firmware. This >> is just binutils, gcc, newlib, libgloss and pmu-firmware itself. >> >> The limiting is done based on the BPN of the recipe, which is not >> provided as an override so the conditional is tested with an inline >> python function. >> >> This change removes the need for the extender class to be enabled >> globally which can conflict with certain recipes that were not written >> to be extended by the BBCLASSEXTEND mechanism. This change also improves >> the parse time avoiding the need to parse all recipes for both target >> and zynqmp-pmu. >> >> Also remove the duplicate PACKAGE_EXTRA_ARCHS setting, this is already >> done in zynqmp-pmu.bbclass itself. >> >> Signed-off-by: Nathan Rossi <nat...@nathanrossi.com> > > After applying this series (on top of my ATF changes) I see these warnings: > > WARNING: core-image-minimal-1.0-r0 do_image_complete: Manifest > /scratch/alistai/yocto/oe-master/build/tmp-glibc/sstate-control/manifest-allarch-zynqmp-pmu-newlib.populate_sysroot > not found? > WARNING: core-image-minimal-1.0-r0 do_image_complete: Manifest > /scratch/alistai/yocto/oe-master/build/tmp-glibc/sstate-control/manifest-allarch-zynqmp-pmu-libgloss.populate_sysroot > not found? > WARNING: core-image-minimal-1.0-r0 do_image_complete: Manifest > /scratch/alistai/yocto/oe-master/build/tmp-glibc/sstate-control/manifest-allarch-zynqmp-pmu-libgcc.populate_sysroot > not found? > > It looks like it happens only after this patch.
Yer this is introduced because of the removal of the "PACKAGE_EXTRA_ARCHS" setting. More details on why it occurs described here: https://lists.yoctoproject.org/pipermail/meta-xilinx/2017-May/002934.html Forgot it was working around the warning :|, I will remove that part of the change v2 incoming. Regards, Nathan > > Thanks, > Alistair > >> --- >> conf/machine/include/zynqmp-pmu-config.inc | 14 +++++++++++--- >> 1 file changed, 11 insertions(+), 3 deletions(-) >> >> diff --git a/conf/machine/include/zynqmp-pmu-config.inc >> b/conf/machine/include/zynqmp-pmu-config.inc >> index aa854cdcf6..f1ee275459 100644 >> --- a/conf/machine/include/zynqmp-pmu-config.inc >> +++ b/conf/machine/include/zynqmp-pmu-config.inc >> @@ -1,5 +1,13 @@ >> >> -# Enable the use of zynqmp-pmu specific recipe builds >> -BBCLASSEXTEND_append = " zynqmp-pmu" >> -PACKAGE_EXTRA_ARCHS_append = " microblazeel-v9.2-bs-cmp" >> +# Enable the use of zynqmp-pmu for specific recipes only >> +def zynqmp_pmu_enable_for_bpns(d): >> + bpns = [ >> + "binutils", >> + "gcc", "libgcc", >> + "newlib", "libgloss", >> + "pmu-firmware", >> + ] >> + return " zynqmp-pmu" if (d.getVar("BPN") in bpns) else "" >> + >> +BBCLASSEXTEND_append = "${@zynqmp_pmu_enable_for_bpns(d)}" >> >> -- >> 2.14.1 >> >> -- >> _______________________________________________ >> meta-xilinx mailing list >> meta-xilinx@yoctoproject.org >> https://lists.yoctoproject.org/listinfo/meta-xilinx -- _______________________________________________ meta-xilinx mailing list meta-xilinx@yoctoproject.org https://lists.yoctoproject.org/listinfo/meta-xilinx