[Buildroot] [PATCH 1/4] configs: atmel: at91sam9260eknf: update defconfig

[Ludovic Desroches]

On Wed, Nov 09, 2016 at 11:25:50AM +0100, Thomas Petazzoni wrote:
> Hello,
> 
> On Wed, 9 Nov 2016 11:10:09 +0100, Ludovic Desroches wrote:
> 
> > > Why enable both U-Boot and Barebox?
> > >   
> > 
> > I don't know why Barebox was selected but the 'official' bootloader is
> > U-Boot for all our products that's why I add it. I kept Barebox because
> > it was already selected and I don't know if someone is still using it or
> > not.
> 
> But the end result is a defconfig that doesn't make any sense.
> 
> Please provide a defconfig that makes sense, and with an updated
> readme.txt if you decide to change the bootloader (and therefore the
> flashing instructions).
> 
> > > >  # Kernel
> > > >  BR2_LINUX_KERNEL=y
> > > > -BR2_LINUX_KERNEL_CUSTOM_VERSION=y
> > > > -BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="3.9.1"  
> > > 
> > > For the sake of reproducibility you should set this to the version you tested, 
> > > say, 4.8.6. The same goes for other patches in this series.
> > 
> > As Alexandre said, we don't want to spend time for the maintainance of these
> > old boards so sticking to the mainline seems to be the way to go. Giving a
> > version is a kind of commitment but we no longer perform tests on these
> > boards excepting kernel boot with kernelci.
> 
> That's not Buildroot policy. We want defconfigs with fixed kernel and
> bootloader versions, so that we know they have been tested.
> 
> If you are not able/willing to test those defconfigs, then we could
> just as well remove them. But I'm not going to merge a defconfig that
> doesn't comply with our policy of having a fixed kernel and a fixed
> bootloader version.
> 

Can I use a fixed version of the compiler too? It seems it makes sense
because this defconfig was tested but is no more compiling.

Regards

Ludovic

> Best regards,
> 
> Thomas
> -- 
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux and Kernel engineering
> http://free-electrons.com