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

[Richard Henderson <richard.henderson@linaro.org>]

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~