devicetree-compiler.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: David Gibson <david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org>
To: Benjamin Herrenschmidt
	<benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
Cc: Rob Herring <robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Grant Likely
	<grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	devicetree-compiler-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Scott Wood <scottwood-KZfg59tc24xl57MIdRCFDg@public.gmane.org>,
	Kim Phillips
	<kim.phillips-KZfg59tc24xl57MIdRCFDg@public.gmane.org>,
	Kumar Gala
	<galak-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
Subject: Re: [PATCH] libfdt: add address translation support
Date: Wed, 14 May 2014 14:20:24 +1000	[thread overview]
Message-ID: <20140514042024.GE28789@voom> (raw)
In-Reply-To: <1399932065.17624.139.camel@pasglop>

[-- Attachment #1: Type: text/plain, Size: 1181 bytes --]

On Tue, May 13, 2014 at 08:01:05AM +1000, Benjamin Herrenschmidt wrote:
> On Mon, 2014-05-12 at 14:27 -0500, Rob Herring wrote:
> > But what you wrote was for unflattened tree. For early FDT uses, do we
> > really need to worry about PCI or other special cases? The current FDT
> > address translation code in arch/powerpc/boot/devtree.c (yes, now we
> > have 3 implementations) does not for example.
> 
> This is specific to a bootloader case yes and it's ... fishy.
> 
> PCI on ARM is here now. PCI is everywhere (well, pseudo-PCI but that's
> the basic issue here) on Intel embedded.
> 
> Some embedded systems will probably want to enumerate from the firmware
> and Linux will need to translate. For example because the UART or some
> critical GPIO that needs to be handled early on is behind a pseudo-PCI
> interface of some sort.
> 
> I think we need to handle these cases rather than fail in obscure ways.

Yeah, I think PCI is way to common to just ignore.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]

  reply	other threads:[~2014-05-14  4:20 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-01 16:57 [PATCH] libfdt: add address translation support Rob Herring
     [not found] ` <1396371463-7516-1-git-send-email-robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-04-01 18:59   ` Warner Losh
     [not found]     ` <4073722A-7162-4041-B1A2-8A992871FEC0-uzTCJ5RojNnQT0dZR+AlfA@public.gmane.org>
2014-04-02  5:43       ` David Gibson
     [not found]         ` <20140402054325.GC3496-1s0os16eZneny3qCrzbmXA@public.gmane.org>
2014-04-02 14:53           ` Warner Losh
     [not found]             ` <AB306DF0-D50C-476A-8314-CFCE5C57BB2E-uzTCJ5RojNnQT0dZR+AlfA@public.gmane.org>
2014-04-02 23:37               ` David Gibson
2014-04-01 23:02   ` Scott Wood
     [not found]     ` <1396393358.32034.40.camel-88ow+0ZRuxG2UiBs7uKeOtHuzzzSOjJt@public.gmane.org>
2014-04-02 19:24       ` Kim Phillips
     [not found]         ` <20140402142403.9d3316f14520252a4a3474e0-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
2014-04-02 21:01           ` Rob Herring
2014-04-15 14:03       ` Rob Herring
     [not found]         ` <CAL_JsqKMUHiPeU7iQDHZ=E6GzMxvXBB4L8Arg+m0JrfQsRkhtg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-04-28 18:12           ` Rob Herring
2014-04-15 21:57   ` Grant Likely
     [not found]     ` <CACxGe6uPerPT_cT+p1zvL20ERwuguDJWDHasxz5gAUe5kPQHNg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-04-15 22:55       ` Rob Herring
2014-05-12  6:00       ` David Gibson
2014-05-12  6:10         ` Benjamin Herrenschmidt
2014-05-12  6:14           ` Benjamin Herrenschmidt
2014-05-14  4:19             ` David Gibson
2014-05-12 19:27           ` Rob Herring
     [not found]             ` <CAL_Jsq+zi72PAjQ+RGvUddLtHocgphPQQ9x2X0S0dYGOKz+Hgg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-12 22:01               ` Benjamin Herrenschmidt
2014-05-14  4:20                 ` David Gibson [this message]
2014-05-14  4:18           ` David Gibson
2014-05-14  5:23             ` Benjamin Herrenschmidt
2014-05-14  6:51               ` David Gibson

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=20140514042024.GE28789@voom \
    --to=david-xt8fgy+axnrb3ne2bgzf6laj5h9x9tb+@public.gmane.org \
    --cc=benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org \
    --cc=devicetree-compiler-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=galak-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org \
    --cc=grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=kim.phillips-KZfg59tc24xl57MIdRCFDg@public.gmane.org \
    --cc=robherring2-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=scottwood-KZfg59tc24xl57MIdRCFDg@public.gmane.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).