From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: mvebu-mbus: defining a DT binding
Date: Sat, 6 Apr 2013 10:39:56 +0200 [thread overview]
Message-ID: <201304061039.56241.arnd@arndb.de> (raw)
In-Reply-To: <20130405212143.GA16221@obsidianresearch.com>
On Friday 05 April 2013, Jason Gunthorpe wrote:
> On Fri, Apr 05, 2013 at 11:01:27PM +0200, Arnd Bergmann wrote:
>
> > > - The bridge ranges would be offset 0 length 4G-1 in the DT since
> > > the value is not known. However firmware could do PCI address
> > > assignment and fill in corrected values.
> >
> > I don't undestand this part. It would make the topmost byte in the
> > 4GB bus space unadressable, which seems strange. Why can't we use
> > the entire 4GB? Maybe we should leave at least a page?
>
> Oh, sorry, that was just a mostly arbitary hacky choice to fit within
> 1 size cell. Using 2 size cells requires an upgrade to skeleton64 for
> all mvebu platforms, eg kirkwood.
>
> Realistically this size should never be used so it doesn't matter if
> it is 4G or 4G-1 - though obviously 4G is preferred in cases where
> size cells is 2. :)
But the PCI bus already is required to have #size-cells=<2>, and
we would only use this in the ranges property of the PCI node, right?
> So.. I don't think it matters today for mvebu, but something to think
> about - shuffling the firmware's address map could be dangerous.
I would at least expect a secure-mode firmware to prevent the OS
from changing any mappings that the firmware relies on, but I see
what you mean.
Arnd
next prev parent reply other threads:[~2013-04-06 8:39 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-05 13:02 mvebu-mbus: defining a DT binding Thomas Petazzoni
2013-04-05 13:17 ` Arnd Bergmann
2013-04-05 13:51 ` Thomas Petazzoni
2013-04-05 14:36 ` Arnd Bergmann
2013-04-05 17:07 ` Jason Gunthorpe
2013-04-05 17:28 ` Arnd Bergmann
2013-04-05 17:48 ` Jason Gunthorpe
2013-04-05 19:49 ` Arnd Bergmann
2013-04-05 20:36 ` Jason Gunthorpe
2013-04-05 21:01 ` Arnd Bergmann
2013-04-05 21:21 ` Jason Gunthorpe
2013-04-06 8:39 ` Arnd Bergmann [this message]
2013-04-08 17:03 ` Jason Gunthorpe
2013-04-05 16:27 ` Jason Gunthorpe
2013-04-05 16:58 ` Arnd Bergmann
2013-04-05 17:29 ` Jason Gunthorpe
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=201304061039.56241.arnd@arndb.de \
--to=arnd@arndb.de \
--cc=linux-arm-kernel@lists.infradead.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.