> -----Original Message----- > From: linux-arch-owner@xxxxxxxxxxxxxxx [mailto:linux-arch-owner@xxxxxxxxxxxxxxx] On Behalf Of Guan Xuetao > Sent: Sunday, January 02, 2011 5:48 PM > To: 'john stultz' > Cc: linux-arch@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx > Subject: RE: [PATCHv1 09/12] unicore32 core architecture: timer and time > > > > > -----Original Message----- > > From: johnstul.lkml@xxxxxxxxx [mailto:johnstul.lkml@xxxxxxxxx] On Behalf Of john stultz > > Sent: Tuesday, December 28, 2010 5:27 AM > > To: Guan Xuetao > > Cc: linux-arch@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx > > Subject: Re: [PATCHv1 09/12] unicore32 core architecture: timer and time > > > > 2010/12/25 Guan Xuetao <guanxuetao@xxxxxxxxxxxxxxx>: > > > From: Guan Xuetao <guanxuetao@xxxxxxxxxxxxxxx> > > > > > > Patch 9 implements timer and time. RTC and PWM device drivers are also here. > > > > > > Signed-off-by: Guan Xuetao <guanxuetao@xxxxxxxxxxxxxxx> > > > > Just a small request: > > > > > +static struct clocksource cksrc_puv3_oscr = { > > > + .name = "oscr", > > > + .rating = 200, > > > + .read = puv3_read_oscr, > > > + .mask = CLOCKSOURCE_MASK(32), > > > + .shift = 18, > > > > Drop the shift assignment here. > > > > > + cksrc_puv3_oscr.mult = > > > + clocksource_hz2mult(CLOCK_TICK_RATE, cksrc_puv3_oscr.shift); > > > + > > Drop the mult assignment here. > > > > > > > + clocksource_register(&cksrc_puv3_oscr); > > > > Use "clocksource_register_hz(&cksrc_puv3_oscr, CLOCK_TICK_RATE );" and > > the kernel will fill in mult/shift appropriately. > > > > If you have any troubles after doing so, let me know. > > When using clocksource_register_hz(&cksrc_puv3_oscr, CLOCK_TICK_RATE ), > the shift value become 0, and mult value become 0x838a91a7, and the system will be broken > after printing "Switching to clocksource oscr". > > When using clocksource_register_khz(&cksrc_puv3_oscr, CLOCK_TICK_RATE ), > the shift value become 0, and mult value become 0x4359611, and the system will run smoothly, > but timer interrupt seems too frequently. Perhaps it is 4 times faster. > > For comparison, originally, when shift value is set to 0x12, the mult value become 1175e5e. > The error is the implementation of do_div() function. When replacing the arch-specific div64.h with asm-generic/div64.h, clocksource_rigister_hz() works. Thanks. > Which one is better? > > > > > thanks > > -john > > Thanks john. > > Guan Xuetao > Guan Xuetao -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
