Re: [Qemu-devel] [PATCH v2 0/4] ppc/pnv: XSCOM fixes and unit tests

[David Gibson <david@gibson.dropbear.id.au>]

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

On Mon, Nov 14, 2016 at 10:12:54AM +0100, Cédric Le Goater wrote:
> Hello,
> 
> Here is a little serie adding some fixes for the XSCOM registers of
> the POWER9 cores and a unit test. 
> 
> Changes since v1 :
> 
>  - fixed pnv_xscom_addr() for 32bit host systems
>  - replace hweight_long() by ctpop64()
> 
> Tested on a 32-bit LE system and on 64-bit LE and BE systems.

Applied to ppc-for-2.8.

I fixed a trivial nit in 4/4 - you had a 5 space indent, instead of a
4 space indent in one place.

> 
> Thanks,
> 
> C.
> 
> Cédric Le Goater (3):
>   ppc/pnv: add a 'xscom_core_base' field to PnvChipClass
>   ppc/pnv: fix xscom address translation for POWER9
>   tests: add XSCOM tests for the PowerNV machine
> 
> David Gibson (1):
>   ppc/pnv: Fix fatal bug on 32-bit hosts
> 
>  hw/ppc/pnv.c               |  10 +++-
>  hw/ppc/pnv_xscom.c         |   8 +--
>  include/hw/ppc/pnv.h       |   1 +
>  include/hw/ppc/pnv_xscom.h |   5 +-
>  tests/Makefile.include     |   1 +
>  tests/pnv-xscom-test.c     | 140 +++++++++++++++++++++++++++++++++++++++++++++
>  6 files changed, 156 insertions(+), 9 deletions(-)
>  create mode 100644 tests/pnv-xscom-test.c
> 

-- 
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: 819 bytes --]