Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Arnout Vandecappelle <arnout@mind.be>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 0 of 2 v2] Add Xenomai real-time framework
Date: Thu, 15 Sep 2011 07:44:46 +0200	[thread overview]
Message-ID: <201109150744.47121.arnout@mind.be> (raw)
In-Reply-To: <patchbomb.1316004973@localhost6.localdomain6>


On Wednesday 14 September 2011 14:56:13, Thomas De Schampheleire wrote:
> Xenomai comes with a Linux Kernel and user-space part.
> I added a Linux extensions sub-menu (The first patch) which handles
> the kernel modification provided by Adeos/Xenomai. The advantage of
> this way is for maintenance purpose. (Xenomai do not provide a full
> kernel patch, but use a script called "prepare-kernel.sh")
> Also, this split the user-space from the kernel in a nice way instead
> of adding a serie of patches.

 I don't agree with this split.  When you're configuring Xenomai, I think you 
want the Adeos patch and the other Xenomai configuration in a single place, 
not spread over two separate menu items.  And the same reasoning goes for 
maintaining the buildroot parts of Xenomai.  

 There is only one advantage to this approach: the Adeos patch and the kernel 
version are intimately related, so if you change one you also want to change 
the other.  The same reasoning however goes for the relationship between 
kernel headers and kernel itself, which are currently miles apart in 
buildroot.


 Regards,
 Arnout


-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
Essensium/Mind                                     http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium                BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  31BB CF53 8660 6F88 345D  54CC A836 5879 20D7 CF43

  parent reply	other threads:[~2011-09-15  5:44 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-14 12:56 [Buildroot] [PATCH 0 of 2 v2] Add Xenomai real-time framework Thomas De Schampheleire
2011-09-14 12:56 ` [Buildroot] [PATCH 1 of 2 v2] linux: Add Linux Kernel extensions menu Thomas De Schampheleire
2011-09-14 12:56 ` [Buildroot] [PATCH 2 of 2 v2] Add xenomai real-time Framework to buildroot Thomas De Schampheleire
2011-09-15 22:58   ` Arnout Vandecappelle
2011-09-15 23:03     ` Arnout Vandecappelle
2011-09-14 13:39 ` [Buildroot] [PATCH 0 of 2 v2] Add Xenomai real-time framework Sven Neumann
2011-09-15  5:44 ` Arnout Vandecappelle [this message]
2011-09-15  7:13   ` Thomas Petazzoni
2011-09-15  8:02     ` Arnout Vandecappelle

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=201109150744.47121.arnout@mind.be \
    --to=arnout@mind.be \
    --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