Re: [U-Boot] [PATCH v1 (WIP) 01/16] [Timer]Fix misuse of ARM *timer_masked() functions outside arch/arm
Dear Graeme Russ, Am 28.06.2011 13:40, schrieb Graeme Russ: > > Signed-off-by: Graeme Russ > --- .. > board/atmel/at91rm9200dk/flash.c | 10 ++ don't mind at91rm9200dk, it uses deprecated arm920t/at91rm9200 api and will therefore removed in this merge window. regards Andreas Bießmann ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v1 (WIP) 01/16] [Timer]Fix misuse of ARM *timer_masked() functions outside arch/arm
Dear Graeme Russ, In message <1309261269-4363-2-git-send-email-graeme.r...@gmail.com> you wrote: > > Signed-off-by: Graeme Russ I consider this a bug fix, independent from the other dicussion. This should be applied in any case. Acked-by: Wolfgang Denk 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 "We don't care. We don't have to. We're the Phone Company." ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v1 (WIP) 01/16] [Timer]Fix misuse of ARM *timer_masked() functions outside arch/arm
Hi Wolfgang, Le 11/07/2011 23:57, Wolfgang Denk a écrit : > Dear Graeme Russ, > > In message<1309261269-4363-2-git-send-email-graeme.r...@gmail.com> you wrote: >> >> Signed-off-by: Graeme Russ > > I consider this a bug fix, independent from the other dicussion. This > should be applied in any case. > > Acked-by: Wolfgang Denk > > Best regards, > > Wolfgang Denk Patch applies on u-boot-arm/master except for 4 files which are not found: board/ixdp425/flash.c, removed by commit 973af335e61eede3578371330abd3971805887f5, "update/fix IXDP425 / IXDPG425 boards" board/mpl/vcma9/flash.c: removed by commit 6d754843ff62312999a265162c67588913cab991, "VCMA9: use CFI driver (and remove the old one)" board/trab/cmd_trab.c and board/trab/flash.c: removed by commit 566e5cf451ae7e33e31bb62ae5b9b258e33f8609, "ARM: drop unsupported 'trab' board" If this is fine with everyone, I can pull this patch in u-boot-arm/master despite the four errors. Amicalement, -- Albert. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v1 (WIP) 01/16] [Timer]Fix misuse of ARM *timer_masked() functions outside arch/arm
Dear Albert ARIBAUD, In message <4e1f2100.2030...@aribaud.net> you wrote: > > Patch applies on u-boot-arm/master except for 4 files which are not found: > > board/ixdp425/flash.c, removed by commit > 973af335e61eede3578371330abd3971805887f5, "update/fix IXDP425 / IXDPG425 > boards" > > board/mpl/vcma9/flash.c: removed by commit > 6d754843ff62312999a265162c67588913cab991, "VCMA9: use CFI driver (and > remove the old one)" > > board/trab/cmd_trab.c and board/trab/flash.c: removed by commit > 566e5cf451ae7e33e31bb62ae5b9b258e33f8609, "ARM: drop unsupported 'trab' > board" > > If this is fine with everyone, I can pull this patch in > u-boot-arm/master despite the four errors. Please do. These files were all correctly removed recently. 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 In the bathtub of history the truth is harder to hold than the soap, and much more difficult to find ... - Terry Pratchett, _Sourcery_ ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v1 (WIP) 01/16] [Timer]Fix misuse of ARM *timer_masked() functions outside arch/arm
Hi Wolfgang, On 15/07/11 05:50, Wolfgang Denk wrote: > Dear Albert ARIBAUD, > > In message <4e1f2100.2030...@aribaud.net> you wrote: >> >> Patch applies on u-boot-arm/master except for 4 files which are not found: >> >> board/ixdp425/flash.c, removed by commit >> 973af335e61eede3578371330abd3971805887f5, "update/fix IXDP425 / IXDPG425 >> boards" >> >> board/mpl/vcma9/flash.c: removed by commit >> 6d754843ff62312999a265162c67588913cab991, "VCMA9: use CFI driver (and >> remove the old one)" >> >> board/trab/cmd_trab.c and board/trab/flash.c: removed by commit >> 566e5cf451ae7e33e31bb62ae5b9b258e33f8609, "ARM: drop unsupported 'trab' >> board" >> >> If this is fine with everyone, I can pull this patch in >> u-boot-arm/master despite the four errors. > > Please do. These files were all correctly removed recently. I'm OK with this, but I'm about to rebase - After that, I will rename the series, truncate it to only include the ack'd patches, rev them to V2, set their respective in-reply-to to the WIP patches and set the remainder to RFC in patchwork So I could either: - Send a revised version as a separate patch for ARM and remove it from the series - Include it in the truncated series - Remove it from the series and let Albert process it as is Which do you prefer? Regards, Graeme ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v1 (WIP) 01/16] [Timer]Fix misuse of ARM *timer_masked() functions outside arch/arm
Dear Graeme Russ, In message <4e1f7ac5.2080...@gmail.com> you wrote: > > So I could either: > - Send a revised version as a separate patch for ARM and remove it from > the series > - Include it in the truncated series > - Remove it from the series and let Albert process it as is > > Which do you prefer? I'm fine with each of these - I just like to see this go in :-) 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 C++ is the best example of second-system effect since OS/360. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v1 (WIP) 01/16] [Timer]Fix misuse of ARM *timer_masked() functions outside arch/arm
Hi Wolfgang, On 15/07/11 16:31, Wolfgang Denk wrote: > Dear Graeme Russ, > > In message <4e1f7ac5.2080...@gmail.com> you wrote: >> >> So I could either: >> - Send a revised version as a separate patch for ARM and remove it from >> the series >> - Include it in the truncated series >> - Remove it from the series and let Albert process it as is >> >> Which do you prefer? > > I'm fine with each of these - I just like to see this go in :-) OK, I'll go with option #2 and rename the series 'Simplify Timer API'. I'll post this in the next day or two (hopefully tonight) This will get the existing Timer API into a state where I can proceed with the outcome of our latest round of discussion. Regards, Graeme ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot