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 > > Reviewed-by: Cédric Le Goater > > 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