From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] Multiple device support - none at all?
Date: Thu, 12 Mar 2009 22:29:10 +0100 [thread overview]
Message-ID: <20090312212910.12DBB832E8B8@gemini.denx.de> (raw)
In-Reply-To: <Pine.LNX.4.64ksi.0903121203270.8874@home-gw.koi8.net>
Dear ksi at koi8.net,
In message <Pine.LNX.4.64ksi.0903121203270.8874@home-gw.koi8.net> you wrote:
>
> That's true but... It is not that unusual to have several similar interfaces
> on a board. There is nothing obscure or weird in that. But allowing for only
> one of those interfaces (whichever one chooses) supported by a given binary
> is what is weird.
When U-Boot was started, it was for MPC860 only. Well, actually for
MPC8xx.
It took until Stefan Roese submitted the firt port to another pro-
cessor family (4xx) - before that, supporting other processors was
never a goal.
You see the same happen in another area.
> And it is _NOT_ that existing U-Boot does not provide a solution for a
> particular board. It is that it does not provide _MEANS_ to make a solution.
You mean the code cannot be changed? C'me on...
> > Wrong. You can switch console devices on the fly. Including assigning
> > it to the null device. Of course you better know exactly what you are
> > doing.
>
> So what? Let's say I'm trying to read a file from a USB device with some
> interactive command. I do NOT have a serial console and I want to get a
> result somehow...
Use stdin on USB keyboard, if you like (I'd prefer netconsole any-
way); then define an environment variable that 1) switches stdin to
nulldev; 2) reads the file from USB ; 3) switches stdin to USB key-
board.
I do not claim that this is especially elegant, but it's a simple
workaround that allows you to do what you ask for and takes less than
3 seconds to implement.
You are welcoem to provide patches for a more elegant (and more
expensive) solution.
> > > controller permanently enabled if we use USB keyboard that in turn means the
> > > boot device absolutely must reside on that same controller in current
> > > architecture.
> >
> > Wrong. You can switch on the fly.
>
> Yes, I can. Will it do any good is totally different question.
It can be used as a quick (even if considered somewhat dirty)
workaround.
> > As such, I wonder why you waste time for the messages in this current
> > thread.
>
> It was _NOT_ a discussion. It ceased to be one after a couple of days. You
> guys somehow got scared by innocent CPP tricks and then discussion degraded
> to junk. I didn't even tell that there _IS_ a whole bunch of very similar
> CPP-generated code already in U-Boot source. Look at e.g. drivers/pci/pci.c
> or drivers/pci/pci_indirect.c...
Well, I think you can blame both sides. You also provided your share
of unproven claims, ignoring other opinions, etc.
I suggest we let this rest for now, and start again in the next
release cycle, hoping the minds are quieter tehn.
> > Submit a patch? Then we can see the code, the size impact, etc.
>
> I did it for I2C, it got nothing. There is absolutely no reason to make a
> whole bunch of similar patches for other interfaces, they are almost
> identical. The design is exactly the same, there is nothing unique there
> that was not in I2C.
OK, so let's wait what will come out the I2C discussion later, and
then take the next step.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
1000 pains = 1 Megahertz
next prev parent reply other threads:[~2009-03-12 21:29 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-03-10 20:41 [U-Boot] Multiple device support - none at all? ksi at koi8.net
2009-03-10 21:15 ` Jerry Van Baren
2009-03-10 21:30 ` ksi at koi8.net
2009-03-12 15:25 ` Detlev Zundel
2009-03-12 16:00 ` ksi at koi8.net
2009-03-12 17:25 ` Detlev Zundel
2009-03-12 18:07 ` ksi at koi8.net
2009-03-12 19:01 ` Wolfgang Denk
2009-03-12 19:57 ` ksi at koi8.net
2009-03-12 21:29 ` Wolfgang Denk [this message]
2009-03-12 22:11 ` ksi at koi8.net
2009-03-13 8:24 ` Heiko Schocher
2009-03-13 6:45 ` Heiko Schocher
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=20090312212910.12DBB832E8B8@gemini.denx.de \
--to=wd@denx.de \
--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