Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] board: add support for ARC AXS101 and AXS103 Software Development Platforms
Date: Tue, 4 Aug 2015 15:24:28 +0200	[thread overview]
Message-ID: <20150804152428.0c0d4de7@free-electrons.com> (raw)
In-Reply-To: <1438694226.22313.12.camel@synopsys.com>

Dear Alexey Brodkin,

On Tue, 4 Aug 2015 13:17:06 +0000, Alexey Brodkin wrote:

> Opps dummy copy-paste from Busybox's example.
> Please pardon that silly thing.

No problem.

> Agree, that's a long-standing problem and if I'm not mistaken you guys
> advise to go for fs overlay if that kind of "feature" is required.
> 
> That said I'm not happy with that solution but having no other way I have to swallow

Yes, no problem with that.

> > Please force the version of the kernel headers, so that they match the
> > kernel being built.
> 
> Well you see the problem here is we do need yet to be released Linux v4.2
> because that's where both boards are finally exist.
> 
> So once v4.2 is released (in a couple of weeks now) and Buildroot is updated
> I would assume v4.2 headers will become default version, right?
> So then we're golden and there's no need to update these defconfigs.

You're wrong. After 4.2 becomes the default version, 4.3 will become
the default version. And then your defconfigs are broken.

So you *must* specify a custom kernel headers version. Just use 4.1,
this is good enough for a 4.2-rc kernel.

> As for kernel version itself as of today I HAVE TO specify custom version
> (v4.2-rc5) just to have sources with our boards supported. In case of Buildroot
> that means these defconfigs will be build successfully.
> 
> Still once v4.2 gets released (and I think it will happen well between some
> Buildroots RCs) I will send an update that will effectively remove 
> ----->8-----
> BR2_LINUX_KERNEL_CUSTOM_VERSION=y
> BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="4.2-rc5"
> ----->8-----

No, you will certainly *not* send such an update, because doing this is
wrong. 4.2 might be the default version for a while, but then it will
be 4.3, and you will not have tested this.

All defconfigs *must* use a fixed version of the kernel headers, kernel
and bootloader, so that we are relatively sure that they will continue
to build and boot even if we change the default kernel headers, kernel
or bootloader versions.

Please look at all other defconfigs. The qemu_* defconfigs are good
examples of doing things right since they are nicely maintained by
Gustavo.

Best regards,

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

  reply	other threads:[~2015-08-04 13:24 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-04 12:00 [Buildroot] [PATCH] board: add support for ARC AXS101 and AXS103 Software Development Platforms Alexey Brodkin
2015-08-04 12:57 ` Thomas Petazzoni
2015-08-04 13:17   ` Alexey Brodkin
2015-08-04 13:24     ` Thomas Petazzoni [this message]
2015-08-04 13:37       ` Alexey Brodkin
2015-08-04 13:55         ` Thomas Petazzoni
2015-08-04 14:12           ` Alexey Brodkin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150804152428.0c0d4de7@free-electrons.com \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox