On 7/18/24 08:49, Jamin Lin wrote:
> Fix coding style issues from checkpatch.pl
>
> Test command:
> ./scripts/checkpatch.pl --no-tree -f hw/arm/aspeed.c
>
> Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Thanks,
C.
> ---
> hw/arm/aspeed.c | 21 ++++++++++++++-------
> 1 file changed, 14 insertions(+), 7 deletions(-)
>
> diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c
> index 53a4f665d0..f8766ea462 100644
> --- a/hw/arm/aspeed.c
> +++ b/hw/arm/aspeed.c
> @@ -265,7 +265,8 @@ static void write_boot_rom(BlockBackend *blk, hwaddr addr, size_t rom_size,
> g_autofree void *storage = NULL;
> int64_t size;
>
> - /* The block backend size should have already been 'validated' by
> + /*
> + * The block backend size should have already been 'validated' by
> * the creation of the m25p80 object.
> */
> size = blk_getlength(blk);
> @@ -463,8 +464,10 @@ static void palmetto_bmc_i2c_init(AspeedMachineState *bmc)
> DeviceState *dev;
> uint8_t *eeprom_buf = g_malloc0(32 * 1024);
>
> - /* The palmetto platform expects a ds3231 RTC but a ds1338 is
> - * enough to provide basic RTC features. Alarms will be missing */
> + /*
> + * The palmetto platform expects a ds3231 RTC but a ds1338 is
> + * enough to provide basic RTC features. Alarms will be missing
> + */
> i2c_slave_create_simple(aspeed_i2c_get_bus(&soc->i2c, 0), "ds1338", 0x68);
>
> smbus_eeprom_init_one(aspeed_i2c_get_bus(&soc->i2c, 0), 0x50,
> @@ -555,8 +558,10 @@ static void romulus_bmc_i2c_init(AspeedMachineState *bmc)
> {
> AspeedSoCState *soc = bmc->soc;
>
> - /* The romulus board expects Epson RX8900 I2C RTC but a ds1338 is
> - * good enough */
> + /*
> + * The romulus board expects Epson RX8900 I2C RTC but a ds1338 is
> + * good enough
> + */
> i2c_slave_create_simple(aspeed_i2c_get_bus(&soc->i2c, 11), "ds1338", 0x32);
> }
>
> @@ -664,8 +669,10 @@ static void witherspoon_bmc_i2c_init(AspeedMachineState *bmc)
> i2c_slave_create_simple(aspeed_i2c_get_bus(&soc->i2c, 9), TYPE_TMP105,
> 0x4a);
>
> - /* The witherspoon board expects Epson RX8900 I2C RTC but a ds1338 is
> - * good enough */
> + /*
> + * The witherspoon board expects Epson RX8900 I2C RTC but a ds1338 is
> + * good enough
> + */
> i2c_slave_create_simple(aspeed_i2c_get_bus(&soc->i2c, 11), "ds1338", 0x32);
>
> smbus_eeprom_init_one(aspeed_i2c_get_bus(&soc->i2c, 11), 0x51,