From: Becky Bruce <beckyb@kernel.crashing.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 00/12] Go to common initdram() on 85xx-based boards
Date: Mon, 13 Dec 2010 15:06:40 -0600 [thread overview]
Message-ID: <1292274412-11340-1-git-send-email-beckyb@kernel.crashing.org> (raw)
This patch series consists of a bunch of cleanups that allow us to use
a common initdram() on all of the non-corenet 85xx-based boards. Also,
switch to using phys_size_t to represent the size of memory returned.
Most of these patches are just code rearranges or renaming things to
get a common scheme in place. There are also lots of cleanups -
the various initdram() implementations had a variety of issues that
I have hopefully fixed.
This is a large patch series, and I've had to hack on a bunch of different
boards that I know nothing about, so I'd appreciate any review commentary.
I have tested this on 8572DS and confirmed that the results of a MAKEALL
powerpc are the same before and after this series.
Changes from V1:
- Removed prints from initdram() as suggested by Peter
- Changed spacing of 36-bit DRAM size warning messages to better line
up after the removal of the excess prints from initdram()
- Switched sdram_init() to be a weak function
- renamed sdram_init() to lbc_sdram_init()
Diffstat below.
Cheers,
Becky
arch/powerpc/cpu/mpc85xx/cmd_errata.c | 3 +
arch/powerpc/cpu/mpc85xx/cpu.c | 51 ++++++++++++++++
arch/powerpc/cpu/mpc85xx/tlb.c | 6 ++-
arch/powerpc/cpu/mpc8xxx/ddr/main.c | 5 +-
arch/powerpc/cpu/mpc8xxx/fsl_lbc.c | 10 +++
arch/powerpc/include/asm/fsl_ddr_sdram.h | 13 ++++
arch/powerpc/include/asm/fsl_lbc.h | 4 +
board/atum8548/atum8548.c | 25 +--------
board/freescale/mpc8536ds/mpc8536ds.c | 21 -------
board/freescale/mpc8540ads/mpc8540ads.c | 56 +-----------------
board/freescale/mpc8540ads/tlb.c | 19 ------
board/freescale/mpc8541cds/mpc8541cds.c | 46 +--------------
board/freescale/mpc8544ds/mpc8544ds.c | 17 ------
board/freescale/mpc8548cds/mpc8548cds.c | 41 +-------------
board/freescale/mpc8555cds/mpc8555cds.c | 48 +---------------
board/freescale/mpc8560ads/mpc8560ads.c | 56 +-----------------
board/freescale/mpc8560ads/tlb.c | 19 ------
board/freescale/mpc8568mds/mpc8568mds.c | 41 +-------------
board/freescale/mpc8569mds/mpc8569mds.c | 36 ------------
board/freescale/mpc8572ds/mpc8572ds.c | 19 ------
board/freescale/p1022ds/p1022ds.c | 13 ----
board/freescale/p1_p2_rdb/ddr.c | 15 +-----
board/freescale/p2020ds/p2020ds.c | 34 ++---------
board/mpc8540eval/mpc8540eval.c | 52 +---------------
board/pm854/law.c | 5 +-
board/pm854/pm854.c | 62 +-------------------
board/pm854/tlb.c | 18 +-----
board/pm856/law.c | 5 +-
board/pm856/pm856.c | 57 +------------------
board/pm856/tlb.c | 18 +-----
board/sbc8548/sbc8548.c | 48 +---------------
board/sbc8548/tlb.c | 34 ++++-------
board/sbc8560/sbc8560.c | 93 +-----------------------------
board/socrates/sdram.c | 15 +-----
board/stx/stxgp3/stxgp3.c | 32 ----------
board/stx/stxssa/stxssa.c | 33 -----------
board/tqc/tqm85xx/sdram.c | 70 +++++++----------------
board/tqc/tqm85xx/tlb.c | 47 ---------------
board/xes/common/Makefile | 2 -
board/xes/common/fsl_8xxx_ddr.c | 46 ---------------
board/xes/xpedite517x/xpedite517x.c | 13 ++++
doc/README.mpc85xxads | 4 +-
include/configs/MPC8536DS.h | 1 -
include/configs/MPC8540EVAL.h | 2 +-
include/configs/MPC8548CDS.h | 2 +-
include/configs/MPC8568MDS.h | 1 -
include/configs/MPC8569MDS.h | 7 --
include/configs/MPC8572DS.h | 1 -
include/configs/P1_P2_RDB.h | 1 -
include/configs/PM854.h | 8 +--
include/configs/PM856.h | 8 +--
include/configs/SBC8540.h | 2 +-
include/configs/TQM85xx.h | 6 ++-
include/configs/sbc8560.h | 2 +-
include/configs/stxgp3.h | 2 +-
include/configs/stxssa.h | 1 -
56 files changed, 186 insertions(+), 1110 deletions(-)
next reply other threads:[~2010-12-13 21:06 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-13 21:06 Becky Bruce [this message]
2010-12-13 21:06 ` [U-Boot] [PATCH V2 01/12] mpc8540eval: Split initdram() into initdram() and sdram_init() Becky Bruce
2010-12-13 21:06 ` [U-Boot] [PATCH V2 02/12] tqm85xx: create fixed_sdram() to do sdram setup Becky Bruce
2010-12-13 21:06 ` [U-Boot] [PATCH V2 03/12] mpc85xx/tlb.c: Allow platforms to specify wimge bits Becky Bruce
2010-12-13 21:06 ` [U-Boot] [PATCH V2 04/12] socrates: rename sdram_setup fixed_sdram() Becky Bruce
2010-12-13 21:06 ` [U-Boot] [PATCH V2 05/12] mpc8569mds: Remove unnecessary CONFIG_SYS_LBC_SDRAM_BASE definition Becky Bruce
2010-12-13 21:06 ` [U-Boot] [PATCH V2 06/12] PM854: rename CONFIG_SYS_LBC_SDRAM_BASE CONFIG_SYS_LBC_BASE Becky Bruce
2010-12-13 21:06 ` [U-Boot] [PATCH V2 07/12] PM856: Rename " Becky Bruce
2010-12-13 21:06 ` [U-Boot] [PATCH V2 08/12] MPC8568/MPC8569: Remove CONFIG_DDR_DLL define Becky Bruce
2010-12-13 21:06 ` [U-Boot] [PATCH V2 09/12] mpc85xx boards: initdram() cleanup/bugfix Becky Bruce
2010-12-13 21:06 ` [U-Boot] [PATCH V2 10/12] 85xx boards: Rename CONFIG_DDR_DLL to CONFIG_SYS_FSL_ERRATUM_DDR_MSYNC_IN Becky Bruce
2010-12-13 21:06 ` [U-Boot] [PATCH V2 11/12] mpc85xx: rename sdram_init() lbc_sdram_init() Becky Bruce
2010-12-13 21:06 ` [U-Boot] [PATCH V2 12/12] MPC8xxx DDR: align informational prints Becky Bruce
2010-12-14 2:12 ` [U-Boot] [PATCH V2 09/12] mpc85xx boards: initdram() cleanup/bugfix Paul Gortmaker
2010-12-14 16:38 ` Becky Bruce
2010-12-15 23:25 ` Paul Gortmaker
2010-12-15 23:27 ` Becky Bruce
2010-12-14 4:21 ` [U-Boot] [PATCH V2 08/12] MPC8568/MPC8569: Remove CONFIG_DDR_DLL define Kumar Gala
2010-12-14 4:28 ` [U-Boot] [PATCH V2 07/12] PM856: Rename CONFIG_SYS_LBC_SDRAM_BASE CONFIG_SYS_LBC_BASE Kumar Gala
2010-12-14 4:28 ` [U-Boot] [PATCH V2 06/12] PM854: rename " Kumar Gala
2010-12-14 4:21 ` [U-Boot] [PATCH V2 05/12] mpc8569mds: Remove unnecessary CONFIG_SYS_LBC_SDRAM_BASE definition Kumar Gala
2010-12-14 4:21 ` [U-Boot] [PATCH V2 04/12] socrates: rename sdram_setup fixed_sdram() Kumar Gala
2010-12-14 4:19 ` [U-Boot] [PATCH V2 03/12] mpc85xx/tlb.c: Allow platforms to specify wimge bits Kumar Gala
2010-12-14 4:28 ` [U-Boot] [PATCH V2 01/12] mpc8540eval: Split initdram() into initdram() and sdram_init() Kumar Gala
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1292274412-11340-1-git-send-email-beckyb@kernel.crashing.org \
--to=beckyb@kernel.crashing.org \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox