Re: [PATCH v2] target/arm: Use correct variable for setting 'max' cpu's MIDR_EL1

2020-04-28 Thread Laurent Desnogues
On Tue, Apr 28, 2020 at 5:50 PM Philippe Mathieu-Daudé wrote: > > MIDR_EL1 a 64-bit system register with the top 32-bit being RES0. > > This fixes when compiling with -Werror=conversion: > > target/arm/cpu64.c: In function ‘aarch64_max_initfn’: > target/arm/cpu64.c:628:21: error: conversion fr

Re: [PATCH v2] target/arm: Use correct variable for setting 'max' cpu's MIDR_EL1

2020-04-28 Thread Peter Maydell
On Tue, 28 Apr 2020 at 16:50, Philippe Mathieu-Daudé wrote: Looks like you forgot to edit the commit message subject line. > MIDR_EL1 a 64-bit system register with the top 32-bit being RES0. > > This fixes when compiling with -Werror=conversion: > > target/arm/cpu64.c: In function ‘aarch64_max

[PATCH v2] target/arm: Use correct variable for setting 'max' cpu's MIDR_EL1

2020-04-28 Thread Philippe Mathieu-Daudé
MIDR_EL1 a 64-bit system register with the top 32-bit being RES0. This fixes when compiling with -Werror=conversion: target/arm/cpu64.c: In function ‘aarch64_max_initfn’: target/arm/cpu64.c:628:21: error: conversion from ‘uint64_t’ {aka ‘long unsigned int’} to ‘uint32_t’ {aka ‘unsigned int’}