[meta-xilinx] [PATCH v2 1/2] zynqmp-pmu-config.inc: Limit to specific recipes

Nathan Rossi nathan at nathanrossi.com
Sun Aug 27 03:49:17 PDT 2017


On 26 August 2017 at 02:07, Alistair Francis <alistair23 at gmail.com> wrote:
> On Fri, Aug 25, 2017 at 7:02 AM, Nathan Rossi <nathan at nathanrossi.com> wrote:
>> On 23 August 2017 at 04:53, Alistair Francis <alistair23 at gmail.com> wrote:
>>> On Tue, Aug 22, 2017 at 7:33 AM, Nathan Rossi <nathan at 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.
>>>>
>>>> Add comment for the PACKAGE_EXTRA_ARCHS setting, so it is clear why it
>>>> is set.
>>>>
>>>> Signed-off-by: Nathan Rossi <nathan at nathanrossi.com>
>>>
>>> Acked-by: Alistair Francis <alistair.francis at xilinx.com>
>>> Tested-by: Alistair Francis <alistair.francis at xilinx.com>
>>
>> Thanks for testing this Alistair, did you also test/ack the 2/2 patch
>> in this series? Or only this one, I am just checking for when I merge
>> them.
>
> Sorry I should have clarified this.
>
> I tested both and both look good, although I don't understand the
> second one as well as this one.
>
> You can apply the ACK/Tested to both.

Thanks, I've added it for both.

I've merged this series to both master and pyro.

Thanks,
Nathan



More information about the meta-xilinx mailing list