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:[email protected]] > Sent: Thursday, 28 January 2016 23:31 > > On Fri, Jan 15, 2016 at 3:58 PM, Andrew Baumann > > <[email protected]> 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
