qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v6 3/5] sam460ex: Add RTC device


From: David Gibson
Subject: Re: [Qemu-devel] [PATCH v6 3/5] sam460ex: Add RTC device
Date: Mon, 2 Jul 2018 14:04:36 +1000
User-agent: Mutt/1.10.0 (2018-05-17)

On Fri, Jun 29, 2018 at 02:47:32PM +1000, David Gibson wrote:
> On Fri, Jun 29, 2018 at 12:38:33AM +0200, BALATON Zoltan wrote:
> > The Sam460ex has an M41T80 serial RTC chip on I2C bus 0 at address 0x68.
> > 
> > Signed-off-by: BALATON Zoltan <address@hidden>
> > Reviewed-by: Cédric Le Goater <address@hidden>
> 
> With Cédric's review, I've applied patches 1..3 to ppc-for-3.0.

So, I discovered after this that 3/3 breaks make check for the
ppcemb-softmmu target, because ppcemb included sam460ex, but not the
i2c bits it now requires.

I've fixed that up by removing sam460ex from the ppcemb build by
adding a new config option.

Please remember to do an all-targets make and make check before
posting, though.

> 
> > ---
> >  hw/ppc/sam460ex.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/hw/ppc/sam460ex.c b/hw/ppc/sam460ex.c
> > index bdc53d2..dc730cc 100644
> > --- a/hw/ppc/sam460ex.c
> > +++ b/hw/ppc/sam460ex.c
> > @@ -457,6 +457,7 @@ static void sam460ex_init(MachineState *machine)
> >      object_property_set_bool(OBJECT(dev), true, "realized", NULL);
> >      smbus_eeprom_init(i2c[0]->bus, 8, smbus_eeprom_buf, smbus_eeprom_size);
> >      g_free(smbus_eeprom_buf);
> > +    i2c_create_slave(i2c[0]->bus, "m41t80", 0x68);
> >  
> >      dev = sysbus_create_simple(TYPE_PPC4xx_I2C, 0x4ef600800, uic[0][3]);
> >      i2c[1] = PPC4xx_I2C(dev);
> 



-- 
David Gibson                    | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au  | minimalist, thank you.  NOT _the_ _other_
                                | _way_ _around_!
http://www.ozlabs.org/~dgibson

Attachment: signature.asc
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]