On Fri, Feb 10, 2017 at 04:25:58PM +1100, Suraj Jitindar Singh wrote:
> Add a pa-features definition which includes all of the new fields which
> have been added, note we don't claim support for any of these new features
> at this stage.
>
> Signed-off-by: Suraj Jitindar Singh <sjitindarsingh@gmail.com>
Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
> ---
> hw/ppc/spapr.c | 18 ++++++++++++++++++
> 1 file changed, 18 insertions(+)
>
> diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
> index 057adae..44eb014 100644
> --- a/hw/ppc/spapr.c
> +++ b/hw/ppc/spapr.c
> @@ -356,6 +356,20 @@ static void spapr_populate_pa_features(CPUPPCState *env, void *fdt, int offset)
> 0x80, 0x00, 0x00, 0x00, 0x00, 0x00,
> 0x00, 0x00, 0x00, 0x00, 0x80, 0x00,
> 0x80, 0x00, 0x80, 0x00, 0x00, 0x00 };
> + /* Currently we don't advertise any of the "new" ISAv3.00 functionality */
> + uint8_t pa_features_300[] = { 64, 0,
> + 0xf6, 0x1f, 0xc7, 0xc0, 0x80, 0xf0, /* 0 - 5 */
> + 0x80, 0x00, 0x00, 0x00, 0x00, 0x00, /* 6 - 11 */
> + 0x00, 0x00, 0x00, 0x00, 0x80, 0x00, /* 12 - 17 */
> + 0x80, 0x00, 0x80, 0x00, 0x00, 0x00, /* 18 - 23 */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, /* 24 - 29 */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, /* 30 - 35 */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, /* 36 - 41 */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, /* 42 - 47 */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, /* 48 - 53 */
> + 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, /* 54 - 59 */
> + 0x00, 0x00, 0x00, 0x00 }; /* 60 - 63 */
> +
> uint8_t *pa_features;
> size_t pa_size;
>
> @@ -370,6 +384,10 @@ static void spapr_populate_pa_features(CPUPPCState *env, void *fdt, int offset)
> pa_features = pa_features_207;
> pa_size = sizeof(pa_features_207);
> break;
> + case POWERPC_MMU_3_00:
> + pa_features = pa_features_300;
> + pa_size = sizeof(pa_features_300);
> + break;
> default:
> return;
> }
--
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