From mboxrd@z Thu Jan 1 00:00:00 1970 From: Scott Wood Date: Mon, 13 Aug 2012 18:34:42 -0500 Subject: [U-Boot] [PATCH 6/6] nand_spl: change out_be32 to raw_writel and depend on subsequent sync In-Reply-To: References: <1344881442-22671-1-git-send-email-msm@freescale.com> <1344881442-22671-6-git-send-email-msm@freescale.com> <50298C6B.9090907@freescale.com> Message-ID: <50298F12.7060807@freescale.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 08/13/2012 06:31 PM, McClintock Matthew-B29882 wrote: > On Mon, Aug 13, 2012 at 6:23 PM, Scott Wood wrote: >> On 08/13/2012 01:10 PM, Matthew McClintock wrote: >>> This change reduces the SPL size by removing the redundant syncs produced >>> by out_be32 and just replies on one final sync >>> >>> Done with: >>> >>> sed -r '/in_be32/b; s/(out_be32)\(([^,]*),\s+(.*)\)/__raw_writel(\3, \2)/g' -i `git grep --name-only sdram_init nand_spl/` >>> >>> Signed-off-by: Matthew McClintock >>> --- >>> nand_spl/board/freescale/p1010rdb/nand_boot.c | 54 ++++++++++----------- >>> nand_spl/board/freescale/p1023rds/nand_boot.c | 42 ++++++++-------- >>> nand_spl/board/freescale/p1_p2_rdb_pc/nand_boot.c | 48 +++++++++--------- >>> 3 files changed, 71 insertions(+), 73 deletions(-) >> >> This should come first if the other patches break without it, to >> preserve bisectability. > > I think they all work on at least some compilers.... not that that's > extremely helpful. Was this patch meant to free up room for the other patches, or was it just bundled in because it's touching the same code? -Scott