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

2020-04-30 Thread Philippe Mathieu-Daudé
On 4/30/20 5:59 PM, Peter Maydell wrote: On Tue, 28 Apr 2020 at 18:26, 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/cpu

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

2020-04-30 Thread Peter Maydell
On Tue, 28 Apr 2020 at 18:26, 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 from

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

2020-04-28 Thread Laurent Desnogues
On Tue, Apr 28, 2020 at 7:26 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

[PATCH v3] 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’}