Hello Tom,

Tom Rini wrote:
> This introduces 200MHz Micron parts timing information based on x-loader
> to <asm/arch-omap3/mem.h>.  The memory init logic is also based on what
> x-loader does in these cases.  Note that while previously u-boot would
> be flashed in with SW ECC in this case it now must be flashed with HW
> ECC.  We also change CONFIG_SYS_TEXT_BASE to 0x80100000.
> 
> Cc: Dirk Behme <dirk.be...@gmail.com>
> Beagleboard rev C5, xM rev A:
> Tested-by: Tom Rini <tr...@ti.com>
> Beagleboard xM rev C:
> Tested-by: Matt Ranostay <mranos...@gmail.com>
> Beagleboard rev B7, C2, xM rev B:
> Tested-by: Matt Porter <mpor...@ti.com>
> Signed-off-by: Tom Rini <tr...@ti.com>
> ---
>  arch/arm/include/asm/arch-omap3/mem.h |   23 ++++++++++
>  board/ti/beagle/beagle.c              |   72 
> ++++++++++++++++++++++++++++++++-
>  board/ti/beagle/config.mk             |   33 ---------------
>  include/configs/omap3_beagle.h        |   58 +++++++++++++++++++++++++-
>  4 files changed, 148 insertions(+), 38 deletions(-)
>  delete mode 100644 board/ti/beagle/config.mk
[...]

> diff --git a/board/ti/beagle/beagle.c b/board/ti/beagle/beagle.c
> index 9482c5e..1c8f995 100644
> --- a/board/ti/beagle/beagle.c
> +++ b/board/ti/beagle/beagle.c
[...]
> +void get_board_mem_timings(u32 *mcfg, u32 *ctrla, u32 *ctrlb, u32 *rfr_ctrl,
> +             u32 *mr)
> +{
> +     int pop_mfr, pop_id;
> +
> +     /*
> +      * We need to identify what PoP memory is on the board so that
> +      * we know what timings to use.  If we can't identify it then
> +      * we know it's an xM.
> +      */
> +     identify_nand_chip(&pop_mfr, &pop_id);
> +
> +     /*
> +      * We cannot use the MICRON_MCFG_165 as it relies on
> +      * PHYS_SDRAM_1_SIZE being defined to the correct value, and we
> +      * don't know that value until runtime.
> +      */
> +     *mr = MICRON_V_MR_165;
> +     switch (get_board_revision()) {
> +     case REVISION_C4:
> +             if (pop_mfr == NAND_MFR_STMICRO && pop_id == 0xba) {
                                                             ^
Please use a define for this here, thanks!

> +                     *mcfg = 0x04590099;
                                ^
here too.

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to