linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Grant Likely <grant.likely@secretlab.ca>
Cc: devicetree-discuss@ozlabs.org, miltonm@bga.com,
	linuxppc-dev@ozlabs.org, Paul Mackerras <paulus@samba.org>,
	David Miller <davem@davemloft.net>
Subject: Re: [RFC/PATCH 2/3] of: add of_lookup_stdout() utility function
Date: Fri, 08 Aug 2008 08:35:38 +1000	[thread overview]
Message-ID: <1218148538.24157.311.camel@pasglop> (raw)
In-Reply-To: <fa686aa40808060631k278e187ej965f0ac08c837937@mail.gmail.com>


> It's not what we do with flattened device trees blobs though.  In the
> flattened tree we're not using a /chosen/stdout property, just the
> linux,stdout-path one.
> 
> The question that remains is; should there be?  Should the dt blobs
> use /chosen/stdout also?  (I'm not familiar enough with real OF to
> know the answer.  I'm assuming that an instance value is not the same
> as a phandle).

Yup, there are two issues there:

 - The instance value would have to be converted to a phandle while
OF is still alive. I initially did that and added a stdout-node or so
property, but that still hit the next issue.

 - IBM machines has this weird distinction between the real phandle
and the ibm,phandle, the later being the same except for things
that get hotplugged ... and some of the vdevices. We got really
confused trying to sort that out with the output device.

So in the end, I decided to just convert the ihandle to a path and
stick that path in the device-tree.

Cheers,
Ben.

  parent reply	other threads:[~2008-08-07 22:35 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-06  6:02 [RFC/PATCH 0/3] Attempt at making 32bit BAT assignment more intelligent Grant Likely
2008-08-06  6:02 ` [RFC/PATCH 1/3] powerpc: add ioremap_bat() function for setting up BAT translated IO regions Grant Likely
2008-08-06 14:07   ` Kumar Gala
2008-08-06 21:54     ` Grant Likely
2008-08-06 22:11       ` Kumar Gala
2008-08-06 22:28         ` Benjamin Herrenschmidt
2008-08-06 22:55           ` Brad Boyer
2008-08-06 23:11           ` Grant Likely
2008-08-07  1:49           ` Kumar Gala
2008-08-07 22:13             ` Benjamin Herrenschmidt
2008-08-08  0:04               ` Kumar Gala
2008-08-12 19:50                 ` Grant Likely
2008-08-06 22:31         ` Scott Wood
2008-08-06 23:02         ` Grant Likely
2008-08-07  1:52           ` Kumar Gala
2008-08-06 22:26   ` Benjamin Herrenschmidt
2008-08-06 23:11     ` Grant Likely
2008-08-07 16:45       ` Scott Wood
2008-08-07 18:21         ` Kumar Gala
2008-08-07 22:09       ` Benjamin Herrenschmidt
2008-08-06  6:02 ` [RFC/PATCH 2/3] of: add of_lookup_stdout() utility function Grant Likely
2008-08-06  6:14   ` Michael Ellerman
2008-08-06  6:34     ` Grant Likely
2008-08-06  7:42       ` Stephen Rothwell
2008-08-06  7:44         ` David Miller
2008-08-06  7:57         ` Stephen Rothwell
2008-08-06  6:32   ` David Miller
2008-08-06  6:35     ` Grant Likely
2008-08-06  7:19       ` David Miller
2008-08-07 22:37       ` Benjamin Herrenschmidt
2008-08-06 10:21     ` Paul Mackerras
2008-08-06 10:52       ` David Miller
2008-08-06 13:31       ` Grant Likely
2008-08-06 16:25         ` Segher Boessenkool
2008-08-06 17:09           ` Mitch Bradley
2008-08-07  0:40           ` David Gibson
2008-08-07 22:35         ` Benjamin Herrenschmidt [this message]
2008-08-06 16:46   ` Timur Tabi
2008-08-07  6:12     ` David Gibson
2008-08-07 17:28       ` Yoder Stuart
2008-08-07 18:11         ` Timur Tabi
2008-08-13  5:41       ` Grant Likely
2008-08-13 14:32         ` Timur Tabi
2008-08-07 22:36     ` Benjamin Herrenschmidt
2008-08-06  6:02 ` [RFC/PATCH 3/3] powerpc/52xx: add udbg and early debug support for PSC serial console Grant Likely
  -- strict thread matches above, loose matches on Subject: below --
2008-08-07 19:20 [RFC/PATCH 2/3] of: add of_lookup_stdout() utility function Milton Miller
2008-08-07 19:22 ` Timur Tabi
2008-08-07 19:38   ` Scott Wood

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=1218148538.24157.311.camel@pasglop \
    --to=benh@kernel.crashing.org \
    --cc=davem@davemloft.net \
    --cc=devicetree-discuss@ozlabs.org \
    --cc=grant.likely@secretlab.ca \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=miltonm@bga.com \
    --cc=paulus@samba.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).