Dear Poonam Aggrwal,

In message <1277185601-4939-1-git-send-email-poonam.aggr...@freescale.com> you 
wrote:
> Signed-off-by: Dipen Dudhat <dipen.dud...@freescale.com>
> Signed-off-by: Poonam Aggrwal <poonam.aggr...@freescale.com>
> ---
>  board/freescale/p1_p2_rdb/p1_p2_rdb.c |   25 ++++++++-----------------
>  1 files changed, 8 insertions(+), 17 deletions(-)
> 
> diff --git a/board/freescale/p1_p2_rdb/p1_p2_rdb.c 
> b/board/freescale/p1_p2_rdb/p1_p2_rdb.c
> index 31cdf9a..d870a3b 100644
> --- a/board/freescale/p1_p2_rdb/p1_p2_rdb.c
> +++ b/board/freescale/p1_p2_rdb/p1_p2_rdb.c
> @@ -52,32 +52,23 @@ DECLARE_GLOBAL_DATA_PTR;
>  
>  #define SYSCLK_MASK  0x00200000
>  #define BOARDREV_MASK        0x10100000
> -#define BOARDREV_B   0x10100000
>  #define BOARDREV_C   0x00100000
> +#define BOARDREV_D   0x00000000
>  
>  #define SYSCLK_66    66666666
> -#define SYSCLK_50    50000000
>  #define SYSCLK_100   100000000

Does that mean thea board rev. B and 50 MHz sys clk are not longer
supported? Have these configurations been used in the past?Then we
should rather keep them?

>  unsigned long get_board_sys_clk(ulong dummy)
>  {
>       volatile ccsr_gpio_t *pgpio = (void *)(CONFIG_SYS_MPC85xx_GPIO_ADDR);

Please use I/O accessors to read from peripherals!


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
"I find this a nice feature but it is not according to  the  documen-
tation. Or is it a BUG?"   "Let's call it an accidental feature. :-)"
                       - Larry Wall in <6...@jpl-devvax.jpl.nasa.gov>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to