Re: [PATCH 1/3] hw/gpio: add PCA6414 i2c GPIO expander

2023-02-08 Thread Philippe Mathieu-Daudé
On 8/2/23 23:40, Titus Rwantare wrote: On Mon, 6 Feb 2023 at 13:38, Philippe Mathieu-Daudé wrote: Hi Titus, On 6/2/23 20:49, Titus Rwantare wrote: This is a simple i2c device that allows i2c capable devices to have GPIOs. Reviewed-by: Hao Wu Signed-off-by: Titus Rwantare ---

Re: [PATCH 1/3] hw/gpio: add PCA6414 i2c GPIO expander

2023-02-08 Thread Titus Rwantare
On Mon, 6 Feb 2023 at 14:27, Corey Minyard wrote: > > On Mon, Feb 06, 2023 at 07:49:34PM +, Titus Rwantare wrote: > > This is a simple i2c device that allows i2c capable devices to have > > GPIOs. > > > > Reviewed-by: Hao Wu > > Signed-off-by: Titus Rwantare > > --- > > hw/arm/Kconfig

Re: [PATCH 1/3] hw/gpio: add PCA6414 i2c GPIO expander

2023-02-08 Thread Titus Rwantare
On Mon, 6 Feb 2023 at 13:38, Philippe Mathieu-Daudé wrote: > > Hi Titus, > > On 6/2/23 20:49, Titus Rwantare wrote: > > This is a simple i2c device that allows i2c capable devices to have > > GPIOs. > > > > Reviewed-by: Hao Wu > > Signed-off-by: Titus Rwantare > > --- > > hw/arm/Kconfig

Re: [PATCH 1/3] hw/gpio: add PCA6414 i2c GPIO expander

2023-02-08 Thread Titus Rwantare
23 11:50 AM > > To: peter.mayd...@linaro.org > > Cc: qemu-...@nongnu.org; qemu-devel@nongnu.org; Titus Rwantare > > ; Hao Wu > > Subject: [PATCH 1/3] hw/gpio: add PCA6414 i2c GPIO expander > > > > This is a simple i2c device that allows i2c capable devices to have G

Re: [PATCH 1/3] hw/gpio: add PCA6414 i2c GPIO expander

2023-02-08 Thread Titus Rwantare
On Mon, 6 Feb 2023 at 19:43, Joel Stanley wrote: > > On Mon, 6 Feb 2023 at 19:51, Titus Rwantare wrote: > > > > This is a simple i2c device that allows i2c capable devices to have > > GPIOs. > > Nice. > > In Linux this is supported by a driver called pca953x. Would it make > sense to name your

Re: [PATCH 1/3] hw/gpio: add PCA6414 i2c GPIO expander

2023-02-06 Thread Joel Stanley
On Mon, 6 Feb 2023 at 19:51, Titus Rwantare wrote: > > This is a simple i2c device that allows i2c capable devices to have > GPIOs. Nice. In Linux this is supported by a driver called pca953x. Would it make sense to name your model similarly (both the file and the prefixes you use)? If we do

RE: [PATCH 1/3] hw/gpio: add PCA6414 i2c GPIO expander

2023-02-06 Thread Dong, Eddie
vel@nongnu.org; Titus Rwantare > ; Hao Wu > Subject: [PATCH 1/3] hw/gpio: add PCA6414 i2c GPIO expander > > This is a simple i2c device that allows i2c capable devices to have GPIOs. This patch is to implement a device model of I2C to GPIO for PCA_xxx, right? Or do you implement a gene

Re: [PATCH 1/3] hw/gpio: add PCA6414 i2c GPIO expander

2023-02-06 Thread Corey Minyard
On Mon, Feb 06, 2023 at 07:49:34PM +, Titus Rwantare wrote: > This is a simple i2c device that allows i2c capable devices to have > GPIOs. > > Reviewed-by: Hao Wu > Signed-off-by: Titus Rwantare > --- > hw/arm/Kconfig | 1 + > hw/gpio/meson.build | 1 + >

Re: [PATCH 1/3] hw/gpio: add PCA6414 i2c GPIO expander

2023-02-06 Thread Philippe Mathieu-Daudé
Hi Titus, On 6/2/23 20:49, Titus Rwantare wrote: This is a simple i2c device that allows i2c capable devices to have GPIOs. Reviewed-by: Hao Wu Signed-off-by: Titus Rwantare --- hw/arm/Kconfig | 1 + hw/gpio/meson.build | 1 + hw/gpio/pca_i2c_gpio.c

[PATCH 1/3] hw/gpio: add PCA6414 i2c GPIO expander

2023-02-06 Thread Titus Rwantare
This is a simple i2c device that allows i2c capable devices to have GPIOs. Reviewed-by: Hao Wu Signed-off-by: Titus Rwantare --- hw/arm/Kconfig | 1 + hw/gpio/meson.build | 1 + hw/gpio/pca_i2c_gpio.c | 362