public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Russell King - ARM Linux <linux@arm.linux.org.uk>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [PATCH] make the machid configurable via theenvironment (Was: [PATCH] Changed load address and entrypoint for uImage to ZRELADDR + 6MB)
Date: Wed, 30 Jan 2008 15:47:40 +0000	[thread overview]
Message-ID: <20080130154740.GA29330@flint.arm.linux.org.uk> (raw)
In-Reply-To: <000201c8634b$546ee570$8901a8c0@atmel.com>

On Wed, Jan 30, 2008 at 10:33:46AM +0100, Ulf Samuelsson wrote:
> Nicolas Pitre wrote:
> > > But does U-Boot properly provide machine information to the kernel these
> > > days?  I have been and still being burned by U-Boot hardcoding the
> > > _wrong_ machine ID, and therefore custom built kernels from k.o simply
> > > won't boot on those boards without hacking the kernel source.
> > With "our" U-Boot you can override the compiled-in machid with an
> > environment variable.  I didn't implement this patch but I think it's
> > not that difficult.  I will check tomorrow if I can easily get a patch
> > out of our scm.
> 
> I proposed this patch for a long time, but it was rejected by Wolfgang.

One solution to that problem is to have someone who maintains a separate
tree for uboot which does have the feature in.  Then it becomes a matter
of community choice whether they use the official tree or the tree with
sensible machine-id configurability.

That way, those who want the configurability can have it, and Wolfgang can
avoid having his tree polluted with useful features he doesn't want.

  parent reply	other threads:[~2008-01-30 15:47 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1200469629509-git-send-email-vivek.kutal@celunite.com>
     [not found] ` <20080116081303.GA25437@uranus.ravnborg.org>
     [not found]   ` <20080116100904.GA4487@digi.com>
     [not found]     ` <alpine.LFD.1.00.0801160855140.25841@xanadu.home>
     [not found]       ` <20080116163434.GA2955@digi.com>
     [not found]         ` <alpine.LFD.1.00.0801161233050.25841@xanadu.home>
     [not found]           ` <20080116181027.GB29568@uranus.ravnborg.org>
     [not found]             ` <alpine.LFD.1.00.0801161318000.25841@xanadu.home>
     [not found]               ` <20080116205131.GA3605@digi.com>
2008-01-30  8:08                 ` [U-Boot-Users] [PATCH] make the machid configurable via the environment (Was: [PATCH] Changed load address and entry point for uImage to ZRELADDR + 6MB) Uwe Kleine-König
2008-01-30  9:33                   ` [U-Boot-Users] [PATCH] make the machid configurable via theenvironment (Was: [PATCH] Changed load address and entrypoint " Ulf Samuelsson
2008-01-30 14:53                     ` Nicolas Pitre
2008-01-30 15:47                     ` Russell King - ARM Linux [this message]
2008-01-31  0:05                       ` Wolfgang Denk
2008-01-31  8:43                         ` Russell King - ARM Linux
2008-01-30 23:49                     ` Wolfgang Denk
2008-01-30 23:41                   ` [U-Boot-Users] [PATCH] make the machid configurable via the environment (Was: [PATCH] Changed load address and entry point " Wolfgang Denk
2008-02-14 21:25                   ` Wolfgang Denk

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=20080130154740.GA29330@flint.arm.linux.org.uk \
    --to=linux@arm.linux.org.uk \
    --cc=u-boot@lists.denx.de \
    /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