All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.