[U-Boot] [PATCH v1]ppc4xx/Canyonlands added USB board callbacks
Rupjyoti Sarmah
rsarmah at apm.com
Wed Jul 7 12:18:13 CEST 2010
Hi Wolfgang,
Sent: Wednesday, July 07, 2010 1:02 PM
To: Rupjyoti Sarmah
> - if (in_8((void *)(CONFIG_SYS_BCSR_BASE + 3)) &
CONFIG_SYS_BCSR3_PCIE)
> + if (bcsr_data->board_status & CONFIG_SYS_BCSR3_PCIE)
Please use I/O accessors to read the BCSR.
> - printf(", Rev. %X", in_8((void *)(CONFIG_SYS_BCSR_BASE + 0)));
> + printf(", Rev. %X", bcsr_data->cpld_rev);
Ditto. There are more changes like this - please fix all of them.
>>> Done.
> --- a/include/configs/canyonlands.h
> +++ b/include/configs/canyonlands.h
What is "ep460c"? We know 460EX (the CPU) and Canyonlands (the board).
>>> It is the Canyonlands board user manual ( ep460c from embedded
planet).
I changed this comment to make it more clear.
I am submitting [PATCH v2] with the changes.
Regards,
Rup
More information about the U-Boot
mailing list