[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 6/7] hw/arm: quanta-gbs-bmc add i2c devices
From: |
Peter Maydell |
Subject: |
Re: [PATCH 6/7] hw/arm: quanta-gbs-bmc add i2c devices |
Date: |
Thu, 19 Aug 2021 14:55:52 +0100 |
On Sat, 14 Aug 2021 at 00:34, Hao Wu <wuhaotsh@google.com> wrote:
>
> From: Patrick Venture <venture@google.com>
>
> Adds supported i2c devices to the quanta-gbc-bmc board.
>
> Signed-off-by: Patrick Venture <venture@google.com>
> Reviewed-by: Hao Wu <wuhaotsh@google.com>
> ---
> hw/arm/npcm7xx_boards.c | 79 +++++++++++++++++++++++------------------
> 1 file changed, 45 insertions(+), 34 deletions(-)
>
Does the documentation need updating to mention these newly supported devices ?
> static void quanta_gbs_i2c_init(NPCM7xxState *soc)
> {
> - /*
> - * i2c-0:
> - * pca9546@71
> - *
> - * i2c-1:
> + I2CSlave *i2c_mux;
> +
> + /* i2c-0: */
> + i2c_slave_create_simple(npcm7xx_i2c_get_bus(soc, 0), TYPE_PCA9546, 0x71);
> +
> + /* i2c-1:
> * pca9535@24
> * pca9535@20
> * pca9535@21
This leaves this multiline block comment with the wrong format: QEMU's
coding style wants the leading "/*" on a line of its own. Similarly
for some others in this patch.
thanks
-- PMM
- [PATCH 0/7] Misc NPCM7XX patches, Hao Wu, 2021/08/13
- [PATCH 1/7] hw/i2c: Clear ACK bit in NPCM7xx SMBus module, Hao Wu, 2021/08/13
- [PATCH 2/7] hw/i2c: Read FIFO during RXF_CTL change in NPCM7XX SMBus, Hao Wu, 2021/08/13
- [PATCH 3/7] hw/adc: Fix CONV bit in NPCM7XX ADC CON register, Hao Wu, 2021/08/13
- [PATCH 4/7] hw/adc: Make adci[*] R/W in NPCM7XX ADC, Hao Wu, 2021/08/13
- [PATCH 5/7] hw/nvram: Add a new auxiliary function to init at24c eeprom, Hao Wu, 2021/08/13
- [PATCH 6/7] hw/arm: quanta-gbs-bmc add i2c devices, Hao Wu, 2021/08/13
- Re: [PATCH 6/7] hw/arm: quanta-gbs-bmc add i2c devices,
Peter Maydell <=
- [PATCH 7/7] hw/arm: Use unit number in quanta-gsj eeprom files, Hao Wu, 2021/08/13
- Re: [PATCH 0/7] Misc NPCM7XX patches, Peter Maydell, 2021/08/19