[U-Boot] [PATCH v2 0/7]Timer: Simplify API
This patch series is the first 'logical' half of the previous series titled '(WIP) [Timer]API Rewrite' which was a 16 part series This half (the first half) of the original series is a simplification of the existing API designed to correct a few 'implementation errors' and prepare the U-Boot tree for the new Timer API (which is still under active discussion). Changes since v1 ((WIP) [Timer]API Rewrite): - Rebased to 68d4230c3ccce96a72c5b99e48399bf1796fe3c6 - Tweaks to some patch titles - Changed title prefix from '[Timer]' to 'Timer:' (git eats [..]) - Introduce new #define CONFIG_SYS_LOW_RES_TIMER - Dropping of the entire second half of the series (the new Timer API) Graeme Russ (7): Timer: Fix misuse of ARM *timer_masked() functions outside arch/arm Timer: Remove calls to set_timer outside arch/ Timer: Remove set_timer completely Timer: Allow reset_timer() only for systems with low resolution timers Timer: Remove reset_timer() for non-Nios2 arches Timer: Fix at91rm9200/spi.c timer usage Timer: Remove reset_timer_masked() arch/arm/cpu/arm1136/mx31/timer.c| 17 - arch/arm/cpu/arm1136/mx35/timer.c|9 - arch/arm/cpu/arm1136/omap24xx/timer.c| 24 +++- arch/arm/cpu/arm1176/s3c64xx/timer.c | 17 - arch/arm/cpu/arm1176/tnetv107x/timer.c | 14 -- arch/arm/cpu/arm720t/interrupts.c| 17 - arch/arm/cpu/arm920t/a320/timer.c| 12 arch/arm/cpu/arm920t/at91/timer.c| 19 --- arch/arm/cpu/arm920t/at91rm9200/spi.c|5 +++-- arch/arm/cpu/arm920t/at91rm9200/timer.c | 18 -- arch/arm/cpu/arm920t/ep93xx/timer.c | 15 +++ arch/arm/cpu/arm920t/imx/timer.c | 21 +++-- arch/arm/cpu/arm920t/ks8695/timer.c | 25 + arch/arm/cpu/arm920t/s3c24x0/timer.c | 18 -- arch/arm/cpu/arm925t/timer.c | 23 +++ arch/arm/cpu/arm926ejs/armada100/timer.c | 20 ++-- arch/arm/cpu/arm926ejs/davinci/timer.c |5 - arch/arm/cpu/arm926ejs/kirkwood/timer.c | 20 ++-- arch/arm/cpu/arm926ejs/mb86r0x/timer.c | 19 +++ arch/arm/cpu/arm926ejs/mx25/timer.c | 19 --- arch/arm/cpu/arm926ejs/mx27/timer.c | 19 --- arch/arm/cpu/arm926ejs/nomadik/timer.c | 12 +--- arch/arm/cpu/arm926ejs/omap/timer.c | 11 --- arch/arm/cpu/arm926ejs/orion5x/timer.c | 20 ++-- arch/arm/cpu/arm926ejs/pantheon/timer.c | 20 ++-- arch/arm/cpu/arm926ejs/spear/timer.c | 22 +++--- arch/arm/cpu/arm926ejs/versatile/timer.c | 11 --- arch/arm/cpu/armv7/mx5/timer.c | 20 arch/arm/cpu/armv7/omap-common/timer.c | 21 +++-- arch/arm/cpu/armv7/s5p-common/timer.c| 10 -- arch/arm/cpu/armv7/tegra2/timer.c| 17 - arch/arm/cpu/armv7/u8500/timer.c |5 - arch/arm/cpu/ixp/timer.c | 10 -- arch/arm/cpu/lh7a40x/timer.c | 11 --- arch/arm/cpu/pxa/timer.c | 18 +- arch/arm/cpu/s3c44b0/timer.c | 18 -- arch/arm/cpu/sa1100/timer.c | 16 arch/avr32/cpu/interrupts.c | 23 --- arch/blackfin/cpu/interrupts.c |5 - arch/m68k/cpu/mcf547x_8x/slicetimer.c|9 - arch/m68k/lib/board.c|2 -- arch/m68k/lib/time.c | 21 ++--- arch/microblaze/cpu/timer.c | 12 +--- arch/mips/cpu/mips32/time.c | 12 arch/nios2/cpu/interrupts.c |6 -- arch/powerpc/lib/board.c |2 -- arch/powerpc/lib/interrupts.c| 10 -- arch/sh/lib/time.c | 12 arch/sh/lib/time_sh2.c | 15 +-- arch/sparc/lib/board.c |2 -- arch/sparc/lib/interrupts.c | 12 +--- arch/x86/lib/board.c |2 -- arch/x86/lib/timer.c | 10 -- board/BuS/EB+MCF-EV123/flash.c | 10 ++ board/armadillo/flash.c |6 -- board/armltd/integrator/timer.c | 24 board/armltd/vexpress/ca9x4_ct_vxp.c |5 - board/atmel/at91rm9200dk/flash.c | 10 ++ board/cerf250/flash.c| 14 ++ board/cm4008/flash.c | 10 ++ board/cm41xx/flash.c | 10 ++ board/cmc_pu2/flash.c|
Re: [U-Boot] [PATCH v2 0/7]Timer: Simplify API
Hi Wolfgang, On 15/07/11 22:15, Graeme Russ wrote: > This patch series is the first 'logical' half of the previous series titled > '(WIP) [Timer]API Rewrite' which was a 16 part series > > This half (the first half) of the original series is a simplification of > the existing API designed to correct a few 'implementation errors' and > prepare the U-Boot tree for the new Timer API (which is still under active > discussion). > > Changes since v1 ((WIP) [Timer]API Rewrite): > - Rebased to 68d4230c3ccce96a72c5b99e48399bf1796fe3c6 > - Tweaks to some patch titles > - Changed title prefix from '[Timer]' to 'Timer:' (git eats [..]) > - Introduce new #define CONFIG_SYS_LOW_RES_TIMER > - Dropping of the entire second half of the series (the new Timer API) > > Graeme Russ (7): > Timer: Fix misuse of ARM *timer_masked() functions outside arch/arm > Timer: Remove calls to set_timer outside arch/ > Timer: Remove set_timer completely > Timer: Allow reset_timer() only for systems with low resolution timers > Timer: Remove reset_timer() for non-Nios2 arches > Timer: Fix at91rm9200/spi.c timer usage > Timer: Remove reset_timer_masked() Oh dear, it looks like Patchwork does not honour 'in-reply-to'. It has created new patches rather than updating the existing ones :( I've marked the old ones 'Superseded' in Patchwork Regards, Graeme ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 0/7]Timer: Simplify API
Hi Greame, Le 15/07/2011 15:08, Graeme Russ a écrit : > Oh dear, it looks like Patchwork does not honour 'in-reply-to'. It has > created new patches rather than updating the existing ones :( I'm not sure patchwork is supposed to honor the in-reply between patch versions. BTBW, is there a rationale or other documentation on the behavior of patchwork somewhere? > I've marked the old ones 'Superseded' in Patchwork Thanks! Amicalement, -- Albert. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 0/7]Timer: Simplify API
Hi Albert, On 16/07/11 18:36, Albert ARIBAUD wrote: > Hi Greame, > > Le 15/07/2011 15:08, Graeme Russ a écrit : > >> Oh dear, it looks like Patchwork does not honour 'in-reply-to'. It has >> created new patches rather than updating the existing ones :( > > I'm not sure patchwork is supposed to honor the in-reply between patch > versions. BTBW, is there a rationale or other documentation on the behavior > of patchwork somewhere? I should! - The whole point of me painstakingly getting the in-reply-to correct for each individual patch was so they would track correctly everywhere (inbox, list archive, patchwork). Oh Well Regards, Graeme ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 0/7]Timer: Simplify API
Le 16/07/2011 11:01, Graeme Russ a écrit : > Hi Albert, > > On 16/07/11 18:36, Albert ARIBAUD wrote: >> Hi Greame, >> >> Le 15/07/2011 15:08, Graeme Russ a écrit : >> >>> Oh dear, it looks like Patchwork does not honour 'in-reply-to'. It has >>> created new patches rather than updating the existing ones :( >> >> I'm not sure patchwork is supposed to honor the in-reply between patch >> versions. BTBW, is there a rationale or other documentation on the behavior >> of patchwork somewhere? > > I should! - The whole point of me painstakingly getting the in-reply-to > correct for each individual patch was so they would track correctly > everywhere (inbox, list archive, patchwork). Oh Well Two out of three ain't bad. :) > Regards, > > Graeme Amicalement, -- Albert. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 0/7]Timer: Simplify API
Dear Graeme Russ, In message <4e215370.2090...@gmail.com> you wrote: > > > I'm not sure patchwork is supposed to honor the in-reply between patch > > versions. BTBW, is there a rationale or other documentation on the behavior > > of patchwork somewhere? > > I should! - The whole point of me painstakingly getting the in-reply-to > correct for each individual patch was so they would track correctly > everywhere (inbox, list archive, patchwork). Oh Well As far as I understand the threading is only given in the mailing list. Patchwork is not clever enough to recognize a reposting. Old, superseded patches have to be manually marked as such (as long as they are not already set to "Changes Requested"). Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Disobedience: The silver lining to the cloud of servitude. - Ambrose Bierce ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
