Hi,
Since the merge window is open, any comment / news on the serie ?
--
Antoine
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
Hi,
Any news on the status of this serie ?
--
Antoine
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
>> ---
>> Makefile|2 +-
>> arch/arm/cpu/armv7/Makefile |2 +-
>> arch/arm/cpu/armv7/omap-common/Makefile |2 +-
>> arch/arm/include/asm/arch-am33xx/cpu.h |2 +-
>> arch/arm/include/asm/arch-am33xx/omap.h |2 +-
>> drivers/seri
I forgot to add the description of this patchset. Sorry for the noise.
This serie introduce the support of the TI816X EVM board.
The serie fits into the existing AM33XX SoC support and reuse some
definitions from the TI814X.
Based on the implementation in the TI-PSP-04.00.02.14 vendor tree:
htt
Signed-off-by: Antoine Tenart
---
MAINTAINERS |4 +
board/ti/ti816x/Makefile | 47 +
board/ti/ti816x/evm.c| 229 ++
boards.cfg |1 +
include/configs/ti816x_evm.h | 180 ++
Signed-off-by: Antoine Tenart
---
arch/arm/cpu/armv7/am33xx/Makefile |1 +
arch/arm/cpu/armv7/am33xx/clock_ti814x.c | 95 -
arch/arm/cpu/armv7/am33xx/clock_ti816x.c | 445
arch/arm/cpu/armv7/am33xx/emif4.c |2
Rename some CONFIG_TI814X to a more generic CONFIG_TI81XX
Signed-off-by: Antoine Tenart
---
Makefile|2 +-
arch/arm/cpu/armv7/Makefile |2 +-
arch/arm/cpu/armv7/omap-common/Makefile |2 +-
arch/arm/include/asm/arch-am33xx/cpu.h |2 +-
Changes for v6:
- fix typo in commit message
Changes for v5:
- use wd_timer structure for WDT
- remove some magic
- remove memtest and verify from config
- fix a redefinition warning (VTP1_CTRL_ADDR)
- rebase on top of "ARM: AM33xx: Cleanup clocks and hwinit"
Changes for v
Dear Wolfgang,
> Hmmm... should not any boards that used to set CONFIG_TI814X in their
> board config files (ti814x_evm) now also set CONFIG_TI81XX ?
ti814x_evm already has it.
No other board use CONFIG_TI814X.
> Please also fix the typo in the commit message:
> s/CONFIG_81XX/CONFIG_TI81XX/
Rig
Signed-off-by: Antoine Tenart
---
MAINTAINERS |4 +
board/ti/ti816x/Makefile | 47 +
board/ti/ti816x/evm.c| 229 ++
boards.cfg |1 +
include/configs/ti816x_evm.h | 180 ++
Signed-off-by: Antoine Tenart
---
arch/arm/cpu/armv7/am33xx/Makefile |1 +
arch/arm/cpu/armv7/am33xx/clock_ti814x.c | 95 -
arch/arm/cpu/armv7/am33xx/clock_ti816x.c | 445
arch/arm/cpu/armv7/am33xx/emif4.c |2
Rename some CONFIG_TI814X to a more generic CONFIG_81XX
Signed-off-by: Antoine Tenart
---
Makefile|2 +-
arch/arm/cpu/armv7/Makefile |2 +-
arch/arm/cpu/armv7/omap-common/Makefile |2 +-
arch/arm/include/asm/arch-am33xx/cpu.h |2 +-
ar
This serie introduce the support of the TI816X EVM board.
The serie fits into the existing AM33XX SoC support and reuse some definitions
from the TI814X.
Based on the implementation in the TI-PSP-04.00.02.14 vendor tree:
http://arago-project.org/git/projects/?p=u-boot-omap3.git;a=summary
It has
>> +#define CONFIG_EXTRA_ENV_SETTINGS \
>> +"verify=yes\0"
>
> Do we really want that?
I don't think so. This will be removed.
>> +/*
>> + * Memtest works on 8 MB in DRAM after skipping 32MB
>> + * from start addr of ram disk
>> + */
>> +#define CONFIG_SYS_MEMTEST_START(PHYS_DRAM_1 + (64*
> [snip]
>> +/* WDT related */
>> +#define WDT_WDSC(WDT_BASE + 0x010)
>> +#define WDT_WDST(WDT_BASE + 0x014)
>> +#define WDT_WISR(WDT_BASE + 0x018)
>> +#define WDT_WIER(WDT_BASE + 0x01C)
>> +#define WDT_WWER(WDT_BASE + 0x020)
>> +#define W
Signed-off-by: Antoine Tenart
---
arch/arm/cpu/armv7/am33xx/Makefile |1 +
arch/arm/cpu/armv7/am33xx/clock_ti814x.c | 95
arch/arm/cpu/armv7/am33xx/clock_ti816x.c | 529
arch/arm/cpu/armv7/am33xx/emif4.c |2
Rename some CONFIG_TI814X to a more generic CONFIG_81XX
Signed-off-by: Antoine Tenart
---
Makefile|2 +-
arch/arm/cpu/armv7/Makefile |2 +-
arch/arm/cpu/armv7/omap-common/Makefile |2 +-
arch/arm/include/asm/arch-am33xx/cpu.h |2 +-
ar
This serie introduce the support of the TI816X EVM board.
The serie fits into the existing AM33XX SoC support and reuse some definitions
from the TI814X.
Based on the implementation in the TI-PSP-04.00.02.14 vendor tree:
http://arago-project.org/git/projects/?p=u-boot-omap3.git;a=summary
It has
Signed-off-by: Antoine Tenart
---
MAINTAINERS |4 +
board/ti/ti816x/Makefile | 47
board/ti/ti816x/evm.c| 249 ++
boards.cfg |1 +
include/configs/ti816x_evm.h | 188 +++
> +/*
> + * Values supported 400,531,675,796
> + *
> + * On TI8168 rev C, use 400 or 531 MHz !
>
>
> Why? Is it specific to the EVM or is it general for all rev. C parts.
> A pointer to an errata would be good
I don't know if there is an errata, but I couldn't get U-Boot working
[snip]
> these seem more SOC specific than board specific. Would likely be
> replicated per board, I'd suggest to move out of board specific file.
[snip]
> this function is generic to the SOC, move out of board specific file and
> call with appropriate values or defines
-plus the other ones-
W
Signed-off-by: Antoine Tenart
---
MAINTAINERS|4 +
arch/arm/include/asm/arch-am33xx/spl.h |9 +
board/ti/ti816x/Makefile | 48 ++
board/ti/ti816x/evm.c | 866
boards.cfg
Signed-off-by: Antoine Tenart
---
arch/arm/include/asm/arch-am33xx/clock.h |4 +
arch/arm/include/asm/arch-am33xx/clocks_ti816x.h | 136
arch/arm/include/asm/arch-am33xx/cpu.h |4 +
arch/arm/include/asm/arch-am33xx/cpu_ti816x.h | 194 +++
ar
Rename some CONFIG_TI814X to a more generic CONFIG_81XX
Signed-off-by: Antoine Tenart
---
Makefile|2 +-
arch/arm/cpu/armv7/Makefile |2 +-
arch/arm/cpu/armv7/omap-common/Makefile |2 +-
arch/arm/include/asm/arch-am33xx/cpu.h |2 +-
ar
This serie introduce the support of the TI816X EVM board.
It applies on top of Matt Porter patches introducing the support of the
TI814X EVM board ("Add TI814x EVM Support").
The serie fits into the existing AM33XX SoC support and reuse some definitions
from the TI814X.
Based on the implementatio
Signed-off-by: Antoine Tenart
---
arch/arm/include/asm/arch-am33xx/clock.h |4 +
arch/arm/include/asm/arch-am33xx/clocks_ti816x.h | 136
arch/arm/include/asm/arch-am33xx/cpu.h |4 +
arch/arm/include/asm/arch-am33xx/cpu_ti816x.h | 194 +++
ar
Signed-off-by: Antoine Tenart
---
MAINTAINERS|4 +
arch/arm/include/asm/arch-am33xx/spl.h |9 +
board/ti/ti816x/Makefile | 48 ++
board/ti/ti816x/evm.c | 866
boards.cfg
Rename some CONFIG_TI814X to a more generic CONFIG_81XX
Signed-off-by: Antoine Tenart
---
Makefile|2 +-
arch/arm/cpu/armv7/Makefile |2 +-
arch/arm/cpu/armv7/omap-common/Makefile |2 +-
arch/arm/include/asm/arch-am33xx/cpu.h |2 +-
ar
This serie introduce the support of the TI816X EVM board.
It applies on top of Matt Porter patches introducing the support of the
TI814X EVM board ("Add TI814x EVM Support").
The serie fits into the existing AM33XX SoC support and reuse some definitions
from the TI814X.
Based on the implementatio
> TENART> /* DMM */
> TENART> +#if defined(CONFIG_TI816X)
> TENART> +#define DMM_BASE 0x4E00
> TENART> +#else
> TENART> #define DMM_BASE 0x4E40
> TENART> +#endif
>
> I don't think that's right. Matt's series uses DMM_BASE to refer to the
> Yes, please fix with a v2 of the series which also addreses the other
> feedback as well. Also, patches 2-7 can be squashed into a single "Add
> ti816x support" patch.
>
Thanks for your review, I'll do a v2 following your advices.
___
U-Boot mailing l
> Tom> This is fine except it breaks bisectability, you need to also add
> Tom> CONFIG_TI81XX to ti814x_evm.h at this point.
>
> It afaik already has it.
>
Yes, it was introduced by Matt Porter in "ti814x_evm: add ti814x evm
board support" patch :
+#define CONFIG_TI81XX
+#define CONFIG_TI81
This serie introduce the support of the TI816X EVM board.
It applies on top of Matt Porter patches introducing the support of the
TI814X EVM board ("Add TI814x EVM Support").
The serie fits into the existing AM33XX SoC support and reuse some definitions
from the TI814X.
Based on the implementat
---
MAINTAINERS|4 +
arch/arm/include/asm/arch-am33xx/spl.h |9 +
board/ti/ti816x/Makefile | 48 ++
board/ti/ti816x/evm.c | 753
boards.cfg |1 +
include/configs/t
---
arch/arm/include/asm/arch-am33xx/mmc_host_def.h |2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/include/asm/arch-am33xx/mmc_host_def.h
b/arch/arm/include/asm/arch-am33xx/mmc_host_def.h
index e0a3b8b..09d7a77 100644
--- a/arch/arm/include/asm/arch-am33xx/mmc_host_def.h
+++ b/
Also move some non common defintions from hardware.h
---
arch/arm/include/asm/arch-am33xx/hardware.h|7 +--
arch/arm/include/asm/arch-am33xx/hardware_am33xx.h |5 ++
arch/arm/include/asm/arch-am33xx/hardware_ti814x.h |5 ++
arch/arm/include/asm/arch-am33xx/hardware_ti816x.h |
---
arch/arm/include/asm/arch-am33xx/cpu.h|4 +
arch/arm/include/asm/arch-am33xx/cpu_ti816x.h | 269 +
arch/arm/include/asm/emif.h |4 +
3 files changed, 277 insertions(+)
create mode 100644 arch/arm/include/asm/arch-am33xx/cpu_ti816x.h
---
arch/arm/include/asm/arch-am33xx/ddr_defs.h|4 +
arch/arm/include/asm/arch-am33xx/ddr_defs_ti816x.h | 250
2 files changed, 254 insertions(+)
create mode 100644 arch/arm/include/asm/arch-am33xx/ddr_defs_ti816x.h
diff --git a/arch/arm/include/asm/arch-am33xx/
---
arch/arm/include/asm/arch-am33xx/clock.h |4 +
arch/arm/include/asm/arch-am33xx/clocks_ti816x.h | 173 ++
2 files changed, 177 insertions(+)
create mode 100644 arch/arm/include/asm/arch-am33xx/clocks_ti816x.h
diff --git a/arch/arm/include/asm/arch-am33xx/cloc
---
arch/arm/include/asm/arch-am33xx/mux.h|2 +
arch/arm/include/asm/arch-am33xx/mux_ti816x.h | 363 +
2 files changed, 365 insertions(+)
create mode 100644 arch/arm/include/asm/arch-am33xx/mux_ti816x.h
diff --git a/arch/arm/include/asm/arch-am33xx/mux.h
b/a
Rename some CONFIG_TI814X to a more generic CONFIG_81XX
---
Makefile|2 +-
arch/arm/cpu/armv7/Makefile |2 +-
arch/arm/cpu/armv7/omap-common/Makefile |2 +-
arch/arm/include/asm/arch-am33xx/cpu.h |2 +-
arch/arm/include/asm/arch-am33xx/o
41 matches
Mail list logo
