[v3] Add PowerNV I2C Controller Model

[PATCH v3 0/2] Add PowerNV I2C Controller Model

Posted by Glenn Miles 7 months ago

Upstreams the PowerNV I2C controller model originally
authored by Cédric Le Goater with minor changes by
myself to split the actual addition of the model from
wiring it up to a power processor model.

This series only attaches the controller to the powernv9
chip model, but is expected to eventually also be attached
to the powernv10 chip model.

Cédric Le Goater (2):
  ppc/pnv: Add an I2C controller model
  ppc/pnv: Connect I2C controller model to powernv9 chip

 hw/ppc/meson.build         |   1 +
 hw/ppc/pnv.c               |  28 ++
 hw/ppc/pnv_i2c.c           | 697 +++++++++++++++++++++++++++++++++++++
 include/hw/ppc/pnv_chip.h  |   8 +
 include/hw/ppc/pnv_i2c.h   |  38 ++
 include/hw/ppc/pnv_xscom.h |   3 +
 6 files changed, 775 insertions(+)
 create mode 100644 hw/ppc/pnv_i2c.c
 create mode 100644 include/hw/ppc/pnv_i2c.h

-- 
2.31.1

Re: [PATCH v3 0/2] Add PowerNV I2C Controller Model

Posted by Daniel Henrique Barboza 6 months, 1 week ago

Queued in gitlab.com/danielhb/qemu/tree/ppc-next. Thanks,


Daniel

On 10/16/23 19:20, Glenn Miles wrote:
> Upstreams the PowerNV I2C controller model originally
> authored by Cédric Le Goater with minor changes by
> myself to split the actual addition of the model from
> wiring it up to a power processor model.
> 
> This series only attaches the controller to the powernv9
> chip model, but is expected to eventually also be attached
> to the powernv10 chip model.
> 
> Cédric Le Goater (2):
>    ppc/pnv: Add an I2C controller model
>    ppc/pnv: Connect I2C controller model to powernv9 chip
> 
>   hw/ppc/meson.build         |   1 +
>   hw/ppc/pnv.c               |  28 ++
>   hw/ppc/pnv_i2c.c           | 697 +++++++++++++++++++++++++++++++++++++
>   include/hw/ppc/pnv_chip.h  |   8 +
>   include/hw/ppc/pnv_i2c.h   |  38 ++
>   include/hw/ppc/pnv_xscom.h |   3 +
>   6 files changed, 775 insertions(+)
>   create mode 100644 hw/ppc/pnv_i2c.c
>   create mode 100644 include/hw/ppc/pnv_i2c.h
>

Re: [PATCH v3 0/2] Add PowerNV I2C Controller Model

Posted by Cédric Le Goater 6 months, 4 weeks ago

On 10/17/23 00:20, Glenn Miles wrote:
> Upstreams the PowerNV I2C controller model originally
> authored by Cédric Le Goater with minor changes by
> myself to split the actual addition of the model from
> wiring it up to a power processor model.
> 
> This series only attaches the controller to the powernv9
> chip model, but is expected to eventually also be attached
> to the powernv10 chip model.
> 
> Cédric Le Goater (2):
>    ppc/pnv: Add an I2C controller model
>    ppc/pnv: Connect I2C controller model to powernv9 chip


Reviewed-by: Cédric Le Goater <clg@kaod.org>

Thanks for taking care of upstreaming Glenn. Waiting for the P10 part now !

C.


> 
>   hw/ppc/meson.build         |   1 +
>   hw/ppc/pnv.c               |  28 ++
>   hw/ppc/pnv_i2c.c           | 697 +++++++++++++++++++++++++++++++++++++
>   include/hw/ppc/pnv_chip.h  |   8 +
>   include/hw/ppc/pnv_i2c.h   |  38 ++
>   include/hw/ppc/pnv_xscom.h |   3 +
>   6 files changed, 775 insertions(+)
>   create mode 100644 hw/ppc/pnv_i2c.c
>   create mode 100644 include/hw/ppc/pnv_i2c.h
>

Re: [PATCH v3 0/2] Add PowerNV I2C Controller Model

Posted by Miles Glenn 6 months, 4 weeks ago

On Tue, 2023-10-17 at 09:01 +0200, Cédric Le Goater wrote:
> On 10/17/23 00:20, Glenn Miles wrote:
> > Upstreams the PowerNV I2C controller model originally
> > authored by Cédric Le Goater with minor changes by
> > myself to split the actual addition of the model from
> > wiring it up to a power processor model.
> > 
> > This series only attaches the controller to the powernv9
> > chip model, but is expected to eventually also be attached
> > to the powernv10 chip model.
> > 
> > Cédric Le Goater (2):
> >    ppc/pnv: Add an I2C controller model
> >    ppc/pnv: Connect I2C controller model to powernv9 chip
> 
> Reviewed-by: Cédric Le Goater <clg@kaod.org>
> 
> Thanks for taking care of upstreaming Glenn. Waiting for the P10 part
> now !
> 
> C.

No problem!  Thanks for all of your help as well.

Glenn

> 
> 
> >   hw/ppc/meson.build         |   1 +
> >   hw/ppc/pnv.c               |  28 ++
> >   hw/ppc/pnv_i2c.c           | 697
> > +++++++++++++++++++++++++++++++++++++
> >   include/hw/ppc/pnv_chip.h  |   8 +
> >   include/hw/ppc/pnv_i2c.h   |  38 ++
> >   include/hw/ppc/pnv_xscom.h |   3 +
> >   6 files changed, 775 insertions(+)
> >   create mode 100644 hw/ppc/pnv_i2c.c
> >   create mode 100644 include/hw/ppc/pnv_i2c.h
> >