[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 02/24] hw/misc/mps2-scc: Support configurable number of OS
From: |
Peter Maydell |
Subject: |
Re: [PATCH v2 02/24] hw/misc/mps2-scc: Support configurable number of OSCCLK values |
Date: |
Mon, 15 Feb 2021 13:06:40 +0000 |
On Mon, 15 Feb 2021 at 12:02, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>
> On 2/15/21 12:51 PM, Peter Maydell wrote:
> > Currently the MPS2 SCC device implements a fixed number of OSCCLK
> > values (3). The variant of this device in the MPS3 AN524 board has 6
> > OSCCLK values. Switch to using a PROP_ARRAY, which allows board code
> > to specify how large the OSCCLK array should be as well as its
> > values.
> >
> > With a variable-length property array, the SCC no longer specifies
> > default values for the OSCCLKs, so we must set them explicitly in the
> > board code. This defaults are actually incorrect for the an521 and
> > an505; we will correct this bug in a following patch.
> >
> > This is a migration compatibility break for all the mps boards.
> >
> > Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
>
> R-b stands :)
> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Yeah, it's down here...
> > It would be possible to avoid the compat break, but we've already
> > broken compat for the mps boards this release cycle (eg in commit
> > eeae0b2bf4e69de2) when we added Clock support to the armsse code, so
> > there's no point in trying to keep compat for this change.
> > Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
...below the '---' material, because the script I use to add
the r-by tags isn't clever enough to figure out that it ought to
add them above the '---' line rather than just at the bottom
of the commit message. Sorry about that.
-- PMM
- [PATCH v2 00/24] hw/arm: New board model mps3-an524, Peter Maydell, 2021/02/15
- [PATCH v2 01/24] hw/arm/mps2-tz: Make SYSCLK frequency board-specific, Peter Maydell, 2021/02/15
- [PATCH v2 02/24] hw/misc/mps2-scc: Support configurable number of OSCCLK values, Peter Maydell, 2021/02/15
- [PATCH v2 03/24] hw/arm/mps2-tz: Correct the OSCCLK settings for mps2-an505 and mps2-an511, Peter Maydell, 2021/02/15
- [PATCH v2 04/24] hw/arm/mps2-tz: Make the OSCCLK settings be configurable per-board, Peter Maydell, 2021/02/15
- [PATCH v2 05/24] hw/misc/mps2-fpgaio: Make number of LEDs configurable by board, Peter Maydell, 2021/02/15
- [PATCH v2 06/24] hw/misc/mps2-fpgaio: Support SWITCH register, Peter Maydell, 2021/02/15
- [PATCH v2 07/24] hw/arm/mps2-tz: Make FPGAIO switch and LED config per-board, Peter Maydell, 2021/02/15