[U-Boot] [PATCH][v2] powerpc/T1040EMU: Add T1040 emulator support

[York Sun]

On 11/10/2013 09:25 PM, Jain Priyanka-B32167 wrote:
> Hi,
> 
> Dependency of below patch marked under --- 
> [U-Boot] powerpc/t1040qds: Add DDR Raw Timing support
>  	http://patchwork.ozlabs.org/patch/286112/
> no longer holds true as thi patch has been deferred. 
> 
> But below emulator patch will work as it is with SPD method.
> No changes required.
> 
> Regards
> Priyanka
> 
>> -----Original Message-----
>> From: Jain Priyanka-B32167
>> Sent: Monday, October 28, 2013 3:53 PM
>> To: u-boot at lists.denx.de; wd at denx.de
>> Cc: sun york-R58495; Jain Priyanka-B32167; Aggrwal Poonam-B10812;
>> Kushwaha Prabhakar-B32579
>> Subject: [PATCH][v2] powerpc/T1040EMU: Add T1040 emulator support
>>
>> Add emulator support for T1040. Emulator has limited peripherals and
>> interfaces.
>> Difference between T1040QDS and emulator includes:
>> -ECC for DDR is disabled due to procedure to load images -Depends on raw
>> timing for DDR initialization -No board FPGA (Qixis) -No PCI -No SPI -No
>> flash support
>>
>> Signed-off-by: Poonam Aggrwal <poonam.aggrwal at freescale.com>
>> Signed-off-by: Prabhakar Kushwaha <prabhakar at freescale.com>
>> Signed-off-by: Priyanka Jain <Priyanka.Jain at freescale.com>
>> ---
>> Changes for v2: Incorporated Wolfgang Denk's review comments
>>
>> Based on u-boot-mpc85xx/next branch.
>>  This patch depends upon following patches:
>>  1)[U-Boot] powerpc/t1040qds: Add DDR Raw Timing support
>> 	http://patchwork.ozlabs.org/patch/286112/
>>  2)[U-Boot] powerpc/t1040qds: Correct Maintainer name in boards.cfg
>> 	http://patchwork.ozlabs.org/patch/286113/
>>
>>  board/freescale/t1040qds/Makefile   |    3 +-
>>  board/freescale/t1040qds/ddr.c      |    3 +
>>  board/freescale/t1040qds/ddr.h      |   13 ++
>>  board/freescale/t1040qds/t1040emu.c |   75 ++++++++
>>  board/freescale/t1040qds/tlb.c      |    4 +
>>  boards.cfg                          |    1 +
>>  include/configs/T1040EMU.h          |  344

This T1040EMU.h should merge with T1040QDS.h and only host the
difference, like the way we did for T4240QDS and T4240EMU.


>> +++++++++++++++++++++++++++++++++++
>>  7 files changed, 442 insertions(+), 1 deletions(-)  create mode 100644
>> board/freescale/t1040qds/t1040emu.c
>>  create mode 100644 include/configs/T1040EMU.h
>>
>> diff --git a/board/freescale/t1040qds/Makefile
>> b/board/freescale/t1040qds/Makefile
>> index 8f0057b..4bd7103 100644
>> --- a/board/freescale/t1040qds/Makefile
>> +++ b/board/freescale/t1040qds/Makefile
>> @@ -8,7 +8,8 @@ include $(TOPDIR)/config.mk
>>
>>  LIB	= $(obj)lib$(BOARD).o
>>
>> -COBJS-y	+= $(BOARD).o
>> +COBJS-$(CONFIG_T1040QDS) += t1040qds.o
>> +COBJS-$(CONFIG_T1040EMU) += t1040emu.o
>>  COBJS-y	+= ddr.o
>>  COBJS-$(CONFIG_PCI)     += pci.o
>>  COBJS-y	+= law.o
>> diff --git a/board/freescale/t1040qds/ddr.c
>> b/board/freescale/t1040qds/ddr.c index 16ab829..d46021b 100644
>> --- a/board/freescale/t1040qds/ddr.c
>> +++ b/board/freescale/t1040qds/ddr.c
>> @@ -123,6 +123,9 @@ phys_size_t initdram(int board_type)
>>  	puts("Initializing....using SPD\n");
>>
>>  	dram_size = fsl_ddr_sdram();
>> +#ifdef CONFIG_T1040EMU
>> +	dram_size = CONFIG_SYS_SDRAM_SIZE * 1024 * 1024; #endif

Why do you set the size to a fixed value if using SPD?

York