[Buildroot] [PATCH 1/4] configs: atmel: unify and update defconfig for Xplained boards

Ludovic Desroches ludovic.desroches at atmel.com
Thu Mar 24 08:54:06 UTC 2016


On Thu, Mar 24, 2016 at 09:46:55AM +0100, Thomas Petazzoni wrote:
> Hello,
> 
> On Thu, 24 Mar 2016 09:43:44 +0100, Ludovic Desroches wrote:
> 
> > On Thu, Mar 24, 2016 at 09:40:06AM +0100, Thomas Petazzoni wrote:
> > > Hello,
> > > 
> > > I'm adding in Cc: Alexandre Belloni, who has worked on Atmel defconfigs
> > > in the past.
> > 
> > He was in Cc.
> 
> Gah, my bad. Alex must have chosen in his mailing list preferences to
> remove duplicates or something like that, because on the mailing list,
> your Cc to Alex was clearly not visible. Hence my comment.
> 
> > > > - use bootstrap as first stage bootloader
> > > 
> > > As opposed to? Did you mean at91bootstrap3 rather than "bootstrap"?
> > 
> > As opposed to u-boot spl which is less tested than at91bootstrap3.
> 
> OK. I think Alex has a somewhat different opinion on the matter, but I
> guess that if the vendor recommends to use at91bootstrap3, we should
> use this solution in our defconfig.
> 

I know he doesn't share the same opinion. At the moment u-boot spl is not
working on SAMA5D2. Moreover, latest enhancements for at91bootstrap are
probably not ported to u-boot spl.

> > > This is not good. linux-4.1-at91 is a branch name, which means that the
> > > actual source behind this branch name will change over time, making
> > > this defconfig non-reproducible. You must stick to a tag or a commit id.
> > 
> > In fact it was the plan in order to prevent the need to update these
> > defconfig files. If I must stick to a tag then I will use linux4sam
> > releases.
> 
> We really want our defconfigs to be reproducible in terms of
> bootloader/kernel versions. Otherwise, what people will build/test is
> not something that the person submitting the defconfig has tested.

There is pros and cons for this choice but I understand. I will resend
the patches with a tag.

> Thanks!
> 
> Thomas

Regards

Ludovic



More information about the buildroot mailing list