All of lore.kernel.org
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@au1.ibm.com>
To: "Cédric Le Goater" <clg@kaod.org>,
	"David Gibson" <david@gibson.dropbear.id.au>
Cc: qemu-ppc@nongnu.org, qemu-devel@nongnu.org,
	Alexander Graf <agraf@suse.de>
Subject: Re: [Qemu-devel] [PATCH v2 3/7] ppc/pnv: Add XSCOM infrastructure
Date: Wed, 07 Sep 2016 07:49:10 +1000	[thread overview]
Message-ID: <1473198550.8689.40.camel@au1.ibm.com> (raw)
In-Reply-To: <1473198436.8689.38.camel@kernel.crashing.org>

On Wed, 2016-09-07 at 07:47 +1000, Benjamin Herrenschmidt wrote:
> d be an extra op in the xscom device model I guess. 
> 
> No. If you split the XSCOM bus from the MMIO -> XSCOM bridge (the
> ADU)
> then the conversion only happens in the former. You don't directly
> route the MMIOs over ! You intercept the MMIOs and use use the
> address_space_rw to "generate" the XSCOM accesses on the other side,
> like I do for the LPC bus.
> 
> We need that anyway because of the way XSCOMs can manipulate the HMER
> etc...
> 
> > 
> > Also, the main purpose of the XscomBus is to loop on the devices 
> > to populate the device tree. I am wondering if we could just use 
> > a simple list under the chip for that purpose.

In fact, if you do the above, you no longer need a XSCOM device...

A number of "devices" can exist below a chip, all they need to have
XSCOMs is to register memory regions that are child of that chip's
xscom_region.

For device-tree, well, we could have a generic interface that anything
that can populate DT has and iterate through them. Or make a "chiplet"
class or something.

Cheers,
Ben.

  reply	other threads:[~2016-09-06 21:49 UTC|newest]

Thread overview: 62+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-31 16:34 [Qemu-devel] [PATCH v2 0/7] ppc/pnv: add a minimal platform Cédric Le Goater
2016-08-31 16:34 ` [Qemu-devel] [PATCH v2 1/7] ppc/pnv: add skeleton PowerNV platform Cédric Le Goater
2016-09-01 16:31   ` [Qemu-devel] [Qemu-ppc] " Greg Kurz
2016-09-02  6:32     ` Cédric Le Goater
2016-09-02  6:39       ` Benjamin Herrenschmidt
2016-09-05  2:48   ` [Qemu-devel] " David Gibson
2016-09-05  6:06     ` Cédric Le Goater
2016-08-31 16:34 ` [Qemu-devel] [PATCH v2 2/7] ppc/pnv: add a PnvChip object Cédric Le Goater
2016-09-01 17:21   ` [Qemu-devel] [Qemu-ppc] " Greg Kurz
2016-09-02  6:34     ` Cédric Le Goater
2016-09-05  2:58   ` [Qemu-devel] " David Gibson
2016-09-05  6:59     ` Benjamin Herrenschmidt
2016-09-05  7:41       ` Cédric Le Goater
2016-09-05  8:28         ` Benjamin Herrenschmidt
2016-09-06  0:49           ` David Gibson
2016-09-06  6:21             ` Cédric Le Goater
2016-09-05  7:41       ` David Gibson
2016-09-05  9:10         ` Cédric Le Goater
2016-09-06  0:50           ` David Gibson
2016-09-05  7:56     ` Cédric Le Goater
2016-09-06  0:52       ` David Gibson
2016-08-31 16:34 ` [Qemu-devel] [PATCH v2 3/7] ppc/pnv: Add XSCOM infrastructure Cédric Le Goater
2016-09-05  3:39   ` David Gibson
2016-09-05  7:11     ` Benjamin Herrenschmidt
2016-09-06  0:48       ` David Gibson
2016-09-06 14:42         ` Cédric Le Goater
2016-09-06 21:47           ` Benjamin Herrenschmidt
2016-09-06 21:49             ` Benjamin Herrenschmidt [this message]
2016-09-07 15:55               ` Cédric Le Goater
2016-09-07 19:48                 ` Benjamin Herrenschmidt
2016-09-07  2:03             ` David Gibson
2016-09-07 15:47             ` Cédric Le Goater
2016-09-07 21:53               ` Benjamin Herrenschmidt
2016-09-08  8:52                 ` Cédric Le Goater
2016-09-07  2:01           ` David Gibson
2016-09-06 14:42     ` Cédric Le Goater
2016-09-06 21:45       ` Benjamin Herrenschmidt
2016-09-07  2:02         ` David Gibson
2016-09-07 16:40         ` Cédric Le Goater
2016-09-07  1:59       ` David Gibson
2016-09-07  5:27         ` Benjamin Herrenschmidt
2016-09-07  5:46           ` David Gibson
2016-09-07  8:29             ` Benjamin Herrenschmidt
2016-09-05  4:16   ` [Qemu-devel] [Qemu-ppc] " Sam Bobroff
2016-09-06 14:51     ` Cédric Le Goater
2016-08-31 16:34 ` [Qemu-devel] [PATCH v2 4/7] ppc/pnv: add a core mask to PnvChip Cédric Le Goater
2016-09-02  8:03   ` Cédric Le Goater
2016-09-05  3:42   ` David Gibson
2016-09-05 11:13     ` Cédric Le Goater
2016-09-05 11:30       ` Benjamin Herrenschmidt
2016-08-31 16:34 ` [Qemu-devel] [PATCH v2 5/7] ppc/pnv: add a PnvCore object Cédric Le Goater
2016-09-05  4:02   ` David Gibson
2016-09-06  6:14     ` Cédric Le Goater
2016-09-07  1:48       ` David Gibson
2016-08-31 16:34 ` [Qemu-devel] [PATCH v2 6/7] ppc/pnv: add a XScomDevice to PnvCore Cédric Le Goater
2016-09-05  4:19   ` David Gibson
2016-09-06 13:54     ` Cédric Le Goater
2016-09-07  1:51       ` David Gibson
2016-08-31 16:34 ` [Qemu-devel] [PATCH v2 7/7] monitor: fix crash for platforms without a CPU 0 Cédric Le Goater
2016-09-05  4:27   ` David Gibson
2016-09-06  6:28     ` Cédric Le Goater
2016-09-07  1:49       ` 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=1473198550.8689.40.camel@au1.ibm.com \
    --to=benh@au1.ibm.com \
    --cc=agraf@suse.de \
    --cc=clg@kaod.org \
    --cc=david@gibson.dropbear.id.au \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.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.