Re: [Qemu-arm] [PATCH v4 8/8] raspi: add raspberry pi 2 machine

[Andrew Baumann]

Hi Peter,

Thanks for all the reviews. I should have a respun version on the list shortly. 
There's one minor change to this last patch:

> From: Peter Crosthwaite [mailto:address@hidden
> Sent: Thursday, 28 January 2016 23:31
> > On Fri, Jan 15, 2016 at 3:58 PM, Andrew Baumann <address@hidden> wrote:
 [...]
> > +typedef struct RaspiState {
> 
> A quick google search, I see the camel case form for rpi is usually
> "RasPi". Should we follow?

Ok.

> > +    union {
> 
> union not needed.

I know it's not needed now, but it will be as soon as we add pi1, which I hope 
to address in the next patch series. It will make that diff cleaner if we keep 
this here now, so I'm going to leave it as-is. I hope that's ok with you.

Thanks,
Andrew