[Buildroot] [PATCH 1/3] imx-mkimage: new package

[Erik Larsson]

Hello,

2018-02-19 9:02 GMT+01:00 Thomas Petazzoni <thomas.petazzoni at bootlin.com>:

> Hello Erik,
>
> On Mon, 19 Feb 2018 06:43:15 +0100, Erik Larsson wrote:
>
> > > > diff --git a/package/freescale-imx/imx-mkimage/0001-make-bl33-read-
> env-variables.patch
> > > b/package/freescale-imx/imx-mkimage/0001-make-bl33-read-
> > > env-variables.patch
> > > > new file mode 100644
> > > > index 0000000..490eb05
> > > > --- /dev/null
> > > > +++ b/package/freescale-imx/imx-mkimage/0001-make-bl33-read-
> > > env-variables.patch
> > > > @@ -0,0 +1,37 @@
> > > > +From 5ca549356ef889286c12b4d736d9026efafb8fff Mon Sep 17 00:00:00
> 2001
> > > > +From: Erik Larsson <erik.larsson at combitech.se>
> > > > +Date: Mon, 29 Jan 2018 08:00:57 +0100
> > > > +Subject: [PATCH] Add support for overriding BL32 and BL33 not only
> BL31
> > > > +
> > > > +Signed-off-by: Erik Larsson <karl.erik.larsson at gmail.com>
> > > > +Signed-off-by: Christopher Dahlberg <crille.dahlberg at gmail.com>
> > > > +Signed-off-by: Marcus Folkesson <marcus.folkesson at gmail.com>
> > >
> > > What is the upstream status of this patch ?
> > >
> > I've sent the patch to some of the developers, but still waiting for a
> > reaction.
>
> OK. Then who are those other people who have Signed-off on your patch ?
>
My colleagues :)

>
> > > Why are you building just the tool in iMX8M ? There is also one in src/
> > > and in iMX8dv. What are the differences ? Should we build/install all
> > > of them ? Conditionally ?
> > >
> > I'm currently only building for IMX8M because that's the only hardware I
> > have access to. Preferably this should be conditional. Do you think I
> shall
> > add it as a conditional option right away event though I can't test it
> > against other hardware?
>
> It's fine to support just what you need today. However, it would be
> nice to explain it in the commit log and/or in a comment above those
> lines in the .mk file.
>
Okay, I will fix that.

>
> Thanks a lot!
>
> Thomas
> --
> Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
> Embedded Linux and Kernel engineering
> http://bootlin.com
>

Thanks,
Erik
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20180219/4f7c0a5a/attachment.html>