On Mon, 28 Sept 2026 at 10:31, Philippe Mathieu-Daudé
<[email protected]> wrote:
>
> On 26/9/26 19:56, Jacob Whitaker Abrams wrote:
> > Official STM32CubeL4 drivers use and require support for 16-bit writes to
> > UART registers for proper function despite datasheet claims.
> >
> > Signed-off-by: Jacob Whitaker Abrams <[email protected]>
> > ---
> > v3:
> >   - Add suggested comment and set only valid.min_access_size to 4
> > v2:
> >   - Remove all changes other than supporting 16-bit writes UART regs
> >
> >   hw/char/stm32l4x5_usart.c          |  7 ++++++-
> >   tests/qtest/stm32l4x5_usart-test.c | 22 +++++++++++++++++++++-
> >   2 files changed, 27 insertions(+), 2 deletions(-)
>
> Reviewed-by: Philippe Mathieu-Daudé <[email protected]>

Applied to target-arm.next, thanks. I expanded the commit message
slightly, to add:

"As the only 16 bit accesses we need to handle are ones to a
register's base address for registers that have no defined fields in
their top 16 bits, the emulation of 16-bit accesses provided by the
memory system core is good enough, and we can implement this by
updating .valid.min_access_size."

-- PMM

Reply via email to