Am 04.12.2012 18:42, schrieb Peter Maydell: > On 2 December 2012 17:14, Antoine Mathys <[email protected]> wrote: >> Per the datasheet, the mapping between 12 and 24 hours modes is: >> 0 <-> 12 PM >> 1-12 <-> 1-12 AM >> 13-23 <-> 1-11 PM >> >> Signed-off-by: Antoine Mathys <[email protected]> > > This looks good as far as the logic goes, but I think we > could use some symbolic constants for the 12-hour and PM > bits rather than all the literal 0x20 0x40 0x60.
What about adding qtests like we have for the x86 RTC? Is I2C as problematic as PCI there? Andreas P.S. Antoine, please make sure you thread your patches to a cover letter so that they stay together on the list. -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
