public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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

  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