lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 21 Oct 2009 01:02:31 +0200 From: Mikael Pettersson <mikpe@...uu.se> To: Linus Walleij <linus.walleij@...ricsson.com> Cc: Thomas Gleixner <tglx@...utronix.de>, linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, linux-mips@...ux-mips.org, Mikael Pettersson <mikpe@...uu.se>, Ralf Baechle <ralf@...ux-mips.org> Subject: Re: [PATCH] Make MIPS dynamic clocksource/clockevent clock code generic v2 Linus Walleij writes: > This moves the clocksource_set_clock() and clockevent_set_clock() > from the MIPS timer code into clockchips and clocksource where > it belongs. The patch was triggered by code posted by Mikael > Pettersson duplicating this code for the IOP ARM system. The > function signatures where altered slightly to fit into their > destination header files, unsigned int changed to u32 and inlined. > > Signed-off-by: Linus Walleij <linus.walleij@...ricsson.com> > Cc: Thomas Gleixner <tglx@...utronix.de> > Tested-by: Mikael Pettersson <mikpe@...uu.se> > Reviewed-by: Ralf Baechle <ralf@...ux-mips.org> > --- > Changes v1->v2: > - Fixed Mikaels comments: spelling, terminology. > - Kept the functions inline: all uses and foreseen uses > are once per kernel and all are in __init or __cpuinit sections. > - Unable to break out common code - the code is not common and > implementing two execution paths will be more awkward. > - Hoping the tglx likes it anyway. Very minor spelling nits below. > --- a/include/linux/clockchips.h > +++ b/include/linux/clockchips.h > @@ -115,6 +115,28 @@ static inline unsigned long div_sc(unsigned long ticks, unsigned long nsec, > return (unsigned long) tmp; > } > > +/** > + * clockevent_set_clock - calculates an appropriate shift > + * and mult values for a clockevent given a can't have 'an' and a plural form, so s/an // > --- a/include/linux/clocksource.h > +++ b/include/linux/clocksource.h > @@ -257,6 +257,28 @@ static inline u32 clocksource_hz2mult(u32 hz, u32 shift_constant) > } > > /** > + * clocksource_set_clock - calculates an appropriate shift > + * and mult values for a clocksource given a ditto -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists