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:crosthwaitepe...@gmail.com] > Sent: Thursday, 28 January 2016 23:31 > > On Fri, Jan 15, 2016 at 3:58 PM, Andrew Baumann > > <andrew.baum...@microsoft.com> 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
