Re: [U-Boot] [PATCH 2/9] MX51: Add initial support for the Freescale MX51

2010-01-18 Thread Wolfgang Denk
Dear Stefano Babic, In message <4b540829.3080...@denx.de> you wrote: > > >> + unsigned long now, last = readl(&cur_gpt->counter); > >> + long tmo = usec * (CONFIG_MX51_CLK32 / 1000) / 1000; > >> + > >> + if (!tmo) > >> + tmo = 1; > >> + > >> + while (tmo > 0) { > >> + now = r

Re: [U-Boot] [PATCH 2/9] MX51: Add initial support for the Freescale MX51

2010-01-17 Thread Stefano Babic
Wolfgang Denk wrote: > Dear Stefano Babic, > Hi Wolfgang, > It makes little sense to implement arch specific commands that will > most probably repeated - I don;t want to see a "mx51clocks" command > here, and "mx25clocks", "mx27clocks", "mx31clocks", "mx35clocks", ... > commands on other system

Re: [U-Boot] [PATCH 2/9] MX51: Add initial support for the Freescale MX51

2010-01-17 Thread Wolfgang Denk
Dear Stefano Babic, In message <1263212760-27272-3-git-send-email-sba...@denx.de> you wrote: > The patch add initial support for the Freescale i.MX51 processor > (family arm cortex_a8). ... > --- /dev/null > +++ b/cpu/arm_cortexa8/mx51/clock.c ... > +/* > + * Calculate the frequence of this pll. >

Re: [U-Boot] [PATCH 2/9] MX51: Add initial support for the Freescale MX51

2010-01-11 Thread Detlev Zundel
Hi Stefano, > Detlev Zundel wrote: >> Hi Stefano, >> > > Hi Detlev, > >>> --- /dev/null >>> +++ b/cpu/arm_cortexa8/mx51/iomux.c >>> @@ -0,0 +1,205 @@ >>> +/* >>> + * Copyright 2009 Freescale Semiconductor, Inc. All Rights Reserved. >>> + */ >>> + >>> +/* >>> + * The code contained herein is licen

Re: [U-Boot] [PATCH 2/9] MX51: Add initial support for the Freescale MX51

2010-01-11 Thread Detlev Zundel
Hi Stefano, > The patch add initial support for the Freescale i.MX51 processor > (family arm cortex_a8). > > Signed-off-by: Fred Fan fanyef...@gmail.com I just noticed that this format misses the "<>" around the e-mail address, please fix for next round. This also applies to the other patches in

Re: [U-Boot] [PATCH 2/9] MX51: Add initial support for the Freescale MX51

2010-01-11 Thread Stefano Babic
Detlev Zundel wrote: > Hi Stefano, > Hi Detlev, >> --- /dev/null >> +++ b/cpu/arm_cortexa8/mx51/iomux.c >> @@ -0,0 +1,205 @@ >> +/* >> + * Copyright 2009 Freescale Semiconductor, Inc. All Rights Reserved. >> + */ >> + >> +/* >> + * The code contained herein is licensed under the GNU General Publ

Re: [U-Boot] [PATCH 2/9] MX51: Add initial support for the Freescale MX51

2010-01-11 Thread Detlev Zundel
Hi Stefano, > The patch add initial support for the Freescale i.MX51 processor > (family arm cortex_a8). A small comment below. [...] > diff --git a/cpu/arm_cortexa8/mx51/iomux.c b/cpu/arm_cortexa8/mx51/iomux.c > new file mode 100644 > index 000..a42ec3a > --- /dev/null > +++ b/cpu/arm_cort

[U-Boot] [PATCH 2/9] MX51: Add initial support for the Freescale MX51

2010-01-11 Thread Stefano Babic
The patch add initial support for the Freescale i.MX51 processor (family arm cortex_a8). Signed-off-by: Fred Fan fanyef...@gmail.com Signed-off-by: Stefano Babic --- cpu/arm_cortexa8/mx51/Makefile| 48 + cpu/arm_cortexa8/mx51/clock.c | 282 + cp