linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: marek.vasut@gmail.com (Marek Vasut)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: Add lsxhl machine type to mach-types
Date: Sun, 13 Nov 2011 17:20:22 +0100	[thread overview]
Message-ID: <201111131720.22562.marek.vasut@gmail.com> (raw)
In-Reply-To: <20111113082810.GF27746@n2100.arm.linux.org.uk>

> On Sun, Nov 13, 2011 at 01:29:56AM +0100, Marek Vasut wrote:
> > > On Sun, Nov 13, 2011 at 12:40:42AM +0100, Michael Walle wrote:
> > > > then, i guess, the problem is on the uboot side, at least if they
> > > > want to support both device tree based and old-fashioned setup code
> > > > (eg. backported device support.)
> > > 
> > > If uboot wants to carry the full list, rather than copying it from
> > > the kernel source, they should be going straight to the main source
> > > of the file (which is given inside the file itself.)
> > > 
> > > Otherwise, if they want to be purely dependent on the version shipped
> > > with the kernel, which will have entries for platforms not merged into
> > > the kernel source deleted from it once they're older than 12 months -
> > > but which may be in u-boot, they can continue taking it out of the
> > > kernel tree.  I think that's sub-optimal as they'll see regressions
> > > from time to time when platforms have been merged into u-boot but not
> > > the kernel.
> > 
> > Hey,
> > 
> > U-Boot follows the in-kernel mach-types file. We had a few issues with
> > boards not building due to the removal of old mach types.
> 
> As the kernel version of the file is now specifically customised for
> the kernel, u-boot should _not_ follow the kernel version anymore, as
> I pointed out.  u-boot need to change this policy of theirs.

Why would that be so? If the users want to support the boards and the boards are 
broken/removed from linux, they just define the mach id by hand. That also makes 
it easier to detect such boards. It is also a warning that noone gives a crap 
about such boards and they might as well soon be removed from uboot too.
> 
> The file has always been and remains available on the web.  There's
> absolutely no need for u-boot to take the customised version from the
> kernel source.

  reply	other threads:[~2011-11-13 16:20 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-07 20:54 [PATCH] ARM: Add lsxhl machine type to mach-types Michael Walle
2011-11-12 16:02 ` Michael Walle
2011-11-12 23:01   ` Russell King - ARM Linux
2011-11-12 23:40     ` Michael Walle
2011-11-12 23:55       ` Russell King - ARM Linux
2011-11-13  0:29         ` Marek Vasut
2011-11-13  8:28           ` Russell King - ARM Linux
2011-11-13 16:20             ` Marek Vasut [this message]
2011-11-13 21:06               ` Russell King - ARM Linux

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=201111131720.22562.marek.vasut@gmail.com \
    --to=marek.vasut@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.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;
as well as URLs for NNTP newsgroup(s).