Re: [PATCH 5/5] hw/i2c/versatile_i2c: Rename versatile_i2c -> arm_sbcon_i2c

[Richard Henderson]

On 1/9/23 01:29, Philippe Mathieu-Daudé wrote:
> On 9/1/23 10:17, Philippe Mathieu-Daudé wrote:
> > This device model started with the Versatile board, named
> > TYPE_VERSATILE_I2C, then ended up renamed TYPE_ARM_SBCON_I2C
> > as per the official "ARM SBCon two-wire serial bus interface"
> > description from:
> > https://developer.arm.com/documentation/dui0440/b/programmer-s-reference/two-wire-serial-bus-interface--sbcon
> >
> > Use the latter name as a better description.
> >
> > Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> > ---
> >   hw/arm/Kconfig                              |  4 ++--
> >   hw/i2c/Kconfig                              |  2 +-
> >   hw/i2c/{versatile_i2c.c => arm_sbcon_i2c.c} | 24 ++++++++++-----------
> >   hw/i2c/meson.build                          |  2 +-
> >   4 files changed, 16 insertions(+), 16 deletions(-)
> >   rename hw/i2c/{versatile_i2c.c => arm_sbcon_i2c.c} (81%)
>
>
> > diff --git a/hw/i2c/versatile_i2c.c b/hw/i2c/arm_sbcon_i2c.c
> > similarity index 81%
> > rename from hw/i2c/versatile_i2c.c
> > rename to hw/i2c/arm_sbcon_i2c.c
>
> Sorry I forgot this hunk:
>
> -- >8 --
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 7a40d4d865..a3c7748505 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -940,6 +940,7 @@ M: Peter Maydell <peter.maydell@linaro.org>
>   L: qemu-arm@nongnu.org
>   S: Maintained
>   F: hw/*/versatile*
> +F: hw/i2c/arm_sbcon_i2c.c
>   F: include/hw/i2c/arm_sbcon_i2c.h
>   F: hw/misc/arm_sysctl.c
>   F: docs/system/arm/versatile.rst
>
> ---

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~