From: Marcin Juszkiewicz <marcin@juszkiewicz.com.pl>
To: openembedded-devel@lists.openembedded.org
Subject: Re: Cleaning up the "linux" directory
Date: Sun, 23 Aug 2009 12:35:37 +0200 [thread overview]
Message-ID: <200908231235.39555.marcin@juszkiewicz.com.pl> (raw)
In-Reply-To: <4A8FBCA1.9050601@atmel.com>
Dnia sobota, 22 sierpnia 2009 o 11:38:41 Ulf Samuelsson napisał(a):
> As everyone knows, the kernel directory is a mess:
> I have been thinking about what to do about this,
> and with the ability to include files in subdirectories,
> I think I have found a nice solution.
I had a look at this and for me it adds lot of things to care when kernel
needs to be updated/changed.
With current linux_X.Y.Z(.A).bb recipes it is easy to add new device - just
one file needs to be edited and thats all. If new patch has to be added then
still one file to edit. New board to be added - same amount of work. If
patches common for few targets needs to be added then you can create new
variable for them:
AT91_PATCHES = "\
file://linux-2.6.30/at91/2.6.30.2/patch-sets/linux-2.6.30.2-at91-001-
andrew-victor.patch;patch=1 \
file://linux-2.6.30/at91/2.6.30.2/patch-sets/linux-2.6.30.2-at91-002-
experimental.patch;patch=1 \
file://linux-2.6.30/at91/2.6.30.2/patch-sets/linux-2.6.30.2-at91-003-
ac97-0001-sam9g45-r1.patch;patch=1 \
file://linux-2.6.30/at91/2.6.30.2/patch-sets/linux-2.6.30.2-at91-003-
ac97-0002-ac97-sam9rlek-r1.patch;patch=1 \
file://linux-2.6.30/at91/2.6.30.2/patch-sets/linux-2.6.30.2-at91-004-
NAND-001-configurable-partition-size.patch;patch=1 \
file://linux-2.6.30/at91/2.6.30.2/patch-sets/linux-2.6.30.2-at91-005-
touchscreen-001-sam9g45ek.patch;patch=1 \
file://linux-2.6.30/at91/2.6.30.2/patch-sets/linux-2.6.30.2-at91-006-
EHCI-001.sam9g45ek.patch;patch=1 \
file://linux-2.6.30/at91/2.6.30.2/patch-sets/linux-2.6.30.2-at91-007-
CLK-001-debug-clocks.patch;patch=1 \
"
And use it in SRC_URI for all boards. If patches do not touch files used by
other devices then adding them directly to default SRC_URI should also be
possible (thats how at91 patch was added in linux_2.6.30.bb recipe).
With your solution I have to check about 10 files in many directories or
regenerate all of them by some script (what about caring of manual changes in
those files?).
Maybe Buildroot uses such way (I do not know because last time I used it
before OE) but for me as long time OE developer it just adds lot of possible
errors to do.
Regards,
--
JID: hrw@jabber.org
Website: http://marcin.juszkiewicz.com.pl/
LinkedIn: http://www.linkedin.com/in/marcinjuszkiewicz
next prev parent reply other threads:[~2009-08-23 10:53 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-22 9:38 Cleaning up the "linux" directory Ulf Samuelsson
2009-08-22 13:59 ` Ulf Samuelsson
2009-08-22 14:59 ` Cleaning up the "linux" directory - Available in origin/ulf/linux-2.6.30.2 Ulf Samuelsson
2009-08-23 10:35 ` Marcin Juszkiewicz [this message]
2009-08-23 12:33 ` Cleaning up the "linux" directory Ulf Samuelsson
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=200908231235.39555.marcin@juszkiewicz.com.pl \
--to=marcin@juszkiewicz.com.pl \
--cc=openembedded-devel@lists.openembedded.org \
/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