[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 4/6] hw/arm: Add I2C device tree for Quanta GSJ
From: |
Corey Minyard |
Subject: |
Re: [PATCH 4/6] hw/arm: Add I2C device tree for Quanta GSJ |
Date: |
Tue, 26 Jan 2021 17:05:09 -0600 |
On Tue, Jan 26, 2021 at 11:32:35AM -0800, wuhaotsh--- via wrote:
> Add an I2C device tree for Quanta GSJ. We only included devices with
> existing QEMU implementation, including AT24 EEPROM and temperature
> sensors.
>
> Reviewed-by: Doug Evans<dje@google.com>
> Reviewed-by: Tyrong Ting<kfting@nuvoton.com>
> Signed-off-by: Hao Wu <wuhaotsh@google.com>
> ---
> hw/arm/npcm7xx_boards.c | 17 +++++++++++++++++
> 1 file changed, 17 insertions(+)
>
> diff --git a/hw/arm/npcm7xx_boards.c b/hw/arm/npcm7xx_boards.c
> index 2d82f48848..1418629e06 100644
> --- a/hw/arm/npcm7xx_boards.c
> +++ b/hw/arm/npcm7xx_boards.c
> @@ -19,6 +19,7 @@
> #include "exec/address-spaces.h"
> #include "hw/arm/npcm7xx.h"
> #include "hw/core/cpu.h"
> +#include "hw/i2c/smbus_eeprom.h"
> #include "hw/loader.h"
> #include "hw/qdev-properties.h"
> #include "qapi/error.h"
> @@ -112,6 +113,21 @@ static void npcm750_evb_i2c_init(NPCM7xxState *soc)
> i2c_slave_create_simple(npcm7xx_i2c_get_bus(soc, 6), "tmp105", 0x48);
> }
>
> +static void quanta_gsj_i2c_init(NPCM7xxState *soc)
> +{
> + uint8_t *eeprom_buf0 = g_malloc0(32 * 1024);
> + uint8_t *eeprom_buf1 = g_malloc0(32 * 1024);
This is kind of pointless because the smbus eeprom is 256 bytes.
It would be nice to modify the smbus eeprom code to take different
sizes, if you want to submit a patch.
-corey
> +
> + i2c_slave_create_simple(npcm7xx_i2c_get_bus(soc, 1), "tmp105", 0x48);
> + i2c_slave_create_simple(npcm7xx_i2c_get_bus(soc, 2), "tmp105", 0x48);
> + i2c_slave_create_simple(npcm7xx_i2c_get_bus(soc, 3), "tmp105", 0x48);
> + i2c_slave_create_simple(npcm7xx_i2c_get_bus(soc, 4), "tmp105", 0x48);
> + smbus_eeprom_init_one(npcm7xx_i2c_get_bus(soc, 9), 0x55, eeprom_buf0);
> + smbus_eeprom_init_one(npcm7xx_i2c_get_bus(soc, 10), 0x55, eeprom_buf1);
> +
> + /* TODO: Add addtional i2c devices. */
> +}
> +
> static void npcm750_evb_init(MachineState *machine)
> {
> NPCM7xxState *soc;
> @@ -137,6 +153,7 @@ static void quanta_gsj_init(MachineState *machine)
> npcm7xx_load_bootrom(machine, soc);
> npcm7xx_connect_flash(&soc->fiu[0], 0, "mx25l25635e",
> drive_get(IF_MTD, 0, 0));
> + quanta_gsj_i2c_init(soc);
> npcm7xx_load_kernel(machine, soc);
> }
>
> --
> 2.30.0.365.g02bc693789-goog
>
>
- [PATCH 0/6] hw/i2c: Add NPCM7XX SMBus Device, Hao Wu, 2021/01/26
- [PATCH 3/6] hw/arm: Add I2C device tree for NPCM750 eval board, Hao Wu, 2021/01/26
- [PATCH 2/6] hw/i2c: Implement NPCM7XX SMBus Module Single Mode, Hao Wu, 2021/01/26
- [PATCH 4/6] hw/arm: Add I2C device tree for Quanta GSJ, Hao Wu, 2021/01/26
- [PATCH 1/6] hw/arm: Remove GPIO from unimplemented NPCM7XX, Hao Wu, 2021/01/26
- [PATCH 5/6] hw/i2c: Add a QTest for NPCM7XX SMBus Device, Hao Wu, 2021/01/26
- [PATCH 6/6] hw/i2c: Implement NPCM7XX SMBus Module FIFO Mode, Hao Wu, 2021/01/26
- Re: [PATCH 6/6] hw/i2c: Implement NPCM7XX SMBus Module FIFO Mode, Corey Minyard, 2021/01/26
- Re: [PATCH 6/6] hw/i2c: Implement NPCM7XX SMBus Module FIFO Mode, Hao Wu, 2021/01/27
- Re: [PATCH 6/6] hw/i2c: Implement NPCM7XX SMBus Module FIFO Mode, Corey Minyard, 2021/01/27
- Re: [PATCH 6/6] hw/i2c: Implement NPCM7XX SMBus Module FIFO Mode, Hao Wu, 2021/01/27
- Re: [PATCH 6/6] hw/i2c: Implement NPCM7XX SMBus Module FIFO Mode, Corey Minyard, 2021/01/27
- Re: [PATCH 6/6] hw/i2c: Implement NPCM7XX SMBus Module FIFO Mode, Corey Minyard, 2021/01/28