qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: David Gibson <david@gibson.dropbear.id.au>
To: "Cédric Le Goater" <clg@kaod.org>
Cc: qemu-devel@nongnu.org, qemu-ppc@nongnu.org,
	Greg Kurz <groug@kaod.org>,
	bala24@linux.ibm.com
Subject: Re: [PATCH 0/2] ppc/pnv: HOMER fixes and improvements
Date: Thu, 12 Dec 2019 16:11:36 +1100	[thread overview]
Message-ID: <20191212051136.GW207300@umbus.fritz.box> (raw)
In-Reply-To: <20191211082912.2625-1-clg@kaod.org>

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

On Wed, Dec 11, 2019 at 09:29:10AM +0100, Cédric Le Goater wrote:
> Hello,
> 
> The first patch introduces a new XSCOM region for the PBA bridge unit
> (Power Bus Access) which connects the OCC (On Chip Controller) to the
> Power bus and System Memory. The PBA is used to gather sensor data,
> for power management, for sleep states, for initial boot, among other
> things. This first patch also fixes :
> 
>  - BAR sizes and BAR masks
>  - The mapping of the OCC common area, which is common to all chips
>    and should be mapped once.  
>  - OCC common area is in BAR 3 on P8
> 
> and more globally, it fixes multichip support which is currently
> broken because of wrong mappings.
> 
> The OCC common area is mapped at a unique address on the system and
> each OCC is assigned a segment to expose its sensor data. This is
> fixed in the second patch.
> 
> Thanks,

Applied to ppc-for-5.0.


> 
> C. 
> 
> Cédric Le Goater (2):
>   ppc/pnv: Introduce PBA registers
>   ppc/pnv: Fix OCC common area region mapping
> 
>  include/hw/ppc/pnv.h       |  20 +++----
>  include/hw/ppc/pnv_homer.h |   3 +
>  include/hw/ppc/pnv_occ.h   |   8 ++-
>  include/hw/ppc/pnv_xscom.h |   6 ++
>  hw/ppc/pnv.c               |  12 +++-
>  hw/ppc/pnv_homer.c         | 109 +++++++++++++++++++++++++++++++++++++
>  hw/ppc/pnv_occ.c           |  11 ++--
>  hw/ppc/pnv_xscom.c         |  32 -----------
>  8 files changed, 148 insertions(+), 53 deletions(-)
> 

-- 
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: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

      parent reply	other threads:[~2019-12-12  5:14 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-12-11  8:29 [PATCH 0/2] ppc/pnv: HOMER fixes and improvements Cédric Le Goater
2019-12-11  8:29 ` [PATCH 1/2] ppc/pnv: Introduce PBA registers Cédric Le Goater
2019-12-11  8:29 ` [PATCH 2/2] ppc/pnv: Fix OCC common area region mapping Cédric Le Goater
2019-12-12  5:11 ` David Gibson [this message]

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=20191212051136.GW207300@umbus.fritz.box \
    --to=david@gibson.dropbear.id.au \
    --cc=bala24@linux.ibm.com \
    --cc=clg@kaod.org \
    --cc=groug@kaod.org \
    --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 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).