October 2015 - U-Boot

[U-Boot] [PATCH] arm: ls1021a: Ensure LS1021 ARM Generic Timer CompareValue Set 64-bit
by Alison Wang 30 Nov '15

30 Nov '15

This patch addresses a problem mentioned recently on this mailing list: [1]. In that posting a LS1021 based system was locking up at about 5 minutes after boot, but the problem was mysteriously related to the toolchain used for building u-boot. Debugging the problem reveals a stuck interrupt 29 on the GIC. It appears Freescale's LS1021 support in u-boot erroneously sets the 64-bit ARM generic PL1 physical time CompareValue register to all-ones with a 32-bit value. This causes the timer compare to fire 344 seconds after u-boot configures it. Depending on how fast u-boot gets the kernel booted, this amounts to about 5-minutes of Linux uptime before locking up. Apparently the bug is masked by some toolchains. Perhaps this is explained by default compiler options, word sizes, or binutils versions. At any rate this patch makes the manipulation explicitly 64-bit which alleviates the issue. [1] https://lists.yoctoproject.org/pipermail/meta-freescale/2015-June/014400.ht… Signed-off-by: Chris Kilgour <techie(a)whiterocker.com> Signed-off-by: Alison Wang <alison.wang(a)freescale.com> --- arch/arm/cpu/armv7/ls102xa/timer.c | 3 ++- arch/arm/include/asm/arch-ls102xa/immap_ls102xa.h | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/arch/arm/cpu/armv7/ls102xa/timer.c b/arch/arm/cpu/armv7/ls102xa/timer.c index 11b17b2..e6a32ca 100644 --- a/arch/arm/cpu/armv7/ls102xa/timer.c +++ b/arch/arm/cpu/armv7/ls102xa/timer.c @@ -56,7 +56,8 @@ static inline unsigned long long us_to_tick(unsigned long long usec) int timer_init(void) { struct sctr_regs *sctr = (struct sctr_regs *)SCTR_BASE_ADDR; - unsigned long ctrl, val, freq; + unsigned long ctrl, freq; + unsigned long long val; /* Enable System Counter */ writel(SYS_COUNTER_CTRL_ENABLE, &sctr->cntcr); diff --git a/arch/arm/include/asm/arch-ls102xa/immap_ls102xa.h b/arch/arm/include/asm/arch-ls102xa/immap_ls102xa.h index ee547fb..34854da 100644 --- a/arch/arm/include/asm/arch-ls102xa/immap_ls102xa.h +++ b/arch/arm/include/asm/arch-ls102xa/immap_ls102xa.h @@ -31,7 +31,7 @@ #define RCWSR4_SRDS1_PRTCL_SHIFT 24 #define RCWSR4_SRDS1_PRTCL_MASK 0xff000000 -#define TIMER_COMP_VAL 0xffffffff +#define TIMER_COMP_VAL 0xffffffffffffffffull #define ARCH_TIMER_CTRL_ENABLE (1 << 0) #define SYS_COUNTER_CTRL_ENABLE (1 << 24) -- 2.1.0.27.g96db324

6 9

[U-Boot] [PATCH] checkpatch.pl: Fix checkpatch.pl warnings with Perl 5.22
by Moritz Fischer 30 Nov '15

30 Nov '15

This patch fixes a bunch of deprecation warings that show up when using checkpatch.pl with Perl 5.22. Unescaped left brace in regex is deprecated, passed through in regex; marked by <-- HERE in m/\#\s*define.*do\s{ <-- HERE / at ./scripts/checkpatch.pl line 2829. Unescaped left brace in regex is deprecated, passed through in regex; marked by <-- HERE in m/$.*${ <-- HERE / at ./scripts/checkpatch.pl line 3262. Unescaped left brace in regex is deprecated, passed through in regex; marked by <-- HERE in m/do{ <-- HERE / at ./scripts/checkpatch.pl line 3263. Unescaped left brace in regex is deprecated, passed through in regex; marked by <-- HERE in m/^${ <-- HERE / at ./scripts/checkpatch.pl line 3636. Unescaped left brace in regex is deprecated, passed through in regex; marked by <-- HERE in m/(?^x: Signed-off-by: Moritz Fischer <moritz.fischer(a)ettus.com> --- scripts/checkpatch.pl | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 74db2e2..3a6b9fa 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -2825,8 +2825,8 @@ sub process { # function brace can't be on same line, except for #defines of do while, # or if closed on same line - if (($line=~/$Type\s*$Ident\(.*$.*\s{/) and - !($line=~/\#\s*define.*do\s{/) and !($line=~/}/)) { + if (($line=~/$Type\s*$Ident$.*$.*\s\{/) and + !($line=~/\#\s*define.*do\s\{/) and !($line=~/\}/)) { ERROR("OPEN_BRACE", "open brace '{' following function declarations go on the next line\n" . $herecurr); } @@ -3259,12 +3259,12 @@ sub process { ## } #need space before brace following if, while, etc - if (($line =~ /$.*${/ && $line !~ /$$Type${/) || - $line =~ /do{/) { + if (($line =~ /$.*$\{/ && $line !~ /$$Type$\{/) || + $line =~ /do\{/) { if (ERROR("SPACING", "space required before the open brace '{'\n" . $herecurr) && $fix) { - $fixed[$linenr - 1] =~ s/^(\+.*(?:do|\))){/$1 {/; + $fixed[$linenr - 1] =~ s/^(\+.*(?:do|\)))\{/$1 {/; } } @@ -3633,7 +3633,7 @@ sub process { $dstat !~ /^for\s*$Constant$/ && # for (...) $dstat !~ /^for\s*$Constant\s+(?:$Ident|-?$Constant)$/ && # for (...) bar() $dstat !~ /^do\s*{/ && # do {... - $dstat !~ /^\({/ && # ({... + $dstat !~ /^\(\{/ && # ({... $ctx !~ /^.\s*#\s*define\s+TRACE_(?:SYSTEM|INCLUDE_FILE|INCLUDE_PATH)\b/) { $ctx =~ s/\n*$//; -- 2.5.1

5 9

[U-Boot] [PATCH] cfi_flash: give default CONFIG_SYS_MAX_FLASH_SECT in flash.h
by Thomas Chou 30 Nov '15

30 Nov '15

Give default CONFIG_SYS_MAX_FLASH_SECT in flash.h, so that the header can be included regardless of the present of flash. The value 512 is the most used. Signed-off-by: Thomas Chou <thomas(a)wytron.com.tw> --- include/flash.h | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/include/flash.h b/include/flash.h index 5754cf9..dc0645e 100644 --- a/include/flash.h +++ b/include/flash.h @@ -8,7 +8,10 @@ #ifndef _FLASH_H_ #define _FLASH_H_ -#ifndef CONFIG_SYS_NO_FLASH +#ifndef CONFIG_SYS_MAX_FLASH_SECT +#define CONFIG_SYS_MAX_FLASH_SECT 512 +#endif + /*----------------------------------------------------------------------- * FLASH Info: contains chip specific data, per FLASH bank */ @@ -500,6 +503,4 @@ extern flash_info_t *flash_get_info(ulong base); #define FLASH_ERASE_TIMEOUT 120000 /* timeout for erasing in ms */ #define FLASH_WRITE_TIMEOUT 500 /* timeout for writes in ms */ -#endif /* !CONFIG_SYS_NO_FLASH */ - #endif /* _FLASH_H_ */ -- 2.5.0

2 1

[U-Boot] [PATCH] ppc4xx: Remove remnants from ocotea, taishan, ebony and taihu
by Stefan Roese 30 Nov '15

30 Nov '15

The removal of some PPC4xx boards did not catch all references to these boards. This patch now removes all remnants still left. Signed-off-by: Stefan Roese <sr(a)denx.de> Cc: Stefan Roese <sr(a)denx.de> --- arch/powerpc/cpu/ppc4xx/Kconfig | 16 ---------------- arch/powerpc/cpu/ppc4xx/start.S | 23 ----------------------- 2 files changed, 39 deletions(-) diff --git a/arch/powerpc/cpu/ppc4xx/Kconfig b/arch/powerpc/cpu/ppc4xx/Kconfig index ce58d86..945e771 100644 --- a/arch/powerpc/cpu/ppc4xx/Kconfig +++ b/arch/powerpc/cpu/ppc4xx/Kconfig @@ -28,9 +28,6 @@ config TARGET_CANYONLANDS select DM select DM_SERIAL -config TARGET_EBONY - bool "Support ebony" - config TARGET_KATMAI bool "Support katmai" @@ -43,21 +40,12 @@ config TARGET_LUAN config TARGET_MAKALU bool "Support makalu" -config TARGET_OCOTEA - bool "Support ocotea" - config TARGET_REDWOOD bool "Support redwood" config TARGET_SEQUOIA bool "Support sequoia" -config TARGET_TAIHU - bool "Support taihu" - -config TARGET_TAISHAN - bool "Support taishan" - config TARGET_WALNUT bool "Support walnut" @@ -142,16 +130,12 @@ source "board/amcc/acadia/Kconfig" source "board/amcc/bamboo/Kconfig" source "board/amcc/bubinga/Kconfig" source "board/amcc/canyonlands/Kconfig" -source "board/amcc/ebony/Kconfig" source "board/amcc/katmai/Kconfig" source "board/amcc/kilauea/Kconfig" source "board/amcc/luan/Kconfig" source "board/amcc/makalu/Kconfig" -source "board/amcc/ocotea/Kconfig" source "board/amcc/redwood/Kconfig" source "board/amcc/sequoia/Kconfig" -source "board/amcc/taihu/Kconfig" -source "board/amcc/taishan/Kconfig" source "board/amcc/walnut/Kconfig" source "board/amcc/yosemite/Kconfig" source "board/amcc/yucca/Kconfig" diff --git a/arch/powerpc/cpu/ppc4xx/start.S b/arch/powerpc/cpu/ppc4xx/start.S index 77d4040..3dd0557 100644 --- a/arch/powerpc/cpu/ppc4xx/start.S +++ b/arch/powerpc/cpu/ppc4xx/start.S @@ -1795,34 +1795,11 @@ ppc405ep_init: ..no_pllset: #endif /* CONFIG_BUBINGA */ -#ifdef CONFIG_TAIHU - mfdcr r4, CPC0_BOOT - andi. r5, r4, CPC0_BOOT_SEP@l - bne strap_1 /* serial eeprom present */ - addis r5,0,CPLD_REG0_ADDR@h - ori r5,r5,CPLD_REG0_ADDR@l - andi. r5, r5, 0x10 - bne _pci_66mhz -#endif /* CONFIG_TAIHU */ - addis r3,0,PLLMR0_DEFAULT@h /* PLLMR0 default value */ ori r3,r3,PLLMR0_DEFAULT@l /* */ addis r4,0,PLLMR1_DEFAULT@h /* PLLMR1 default value */ ori r4,r4,PLLMR1_DEFAULT@l /* */ -#ifdef CONFIG_TAIHU - b 1f -_pci_66mhz: - addis r3,0,PLLMR0_DEFAULT_PCI66@h - ori r3,r3,PLLMR0_DEFAULT_PCI66@l - addis r4,0,PLLMR1_DEFAULT_PCI66@h - ori r4,r4,PLLMR1_DEFAULT_PCI66@l - b 1f -strap_1: - mfdcr r3, CPC0_PLLMR0 - mfdcr r4, CPC0_PLLMR1 -#endif /* CONFIG_TAIHU */ - 1: b pll_write /* Write the CPC0_PLLMR with new value */ -- 2.6.2

1 1

[U-Boot] [PATCH 1/2] usb: host: xhci-omap: fix build break
by Felipe Balbi 28 Nov '15

28 Nov '15

Fix the following build break: drivers/usb/host/xhci-omap.c:35:5: error: ‘board_usb_init’ aliased to external symbol ‘__board_usb_init’ int board_usb_init(int index, enum usb_init_type init) ^ Signed-off-by: Felipe Balbi <balbi(a)ti.com> --- drivers/usb/host/xhci-omap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/host/xhci-omap.c b/drivers/usb/host/xhci-omap.c index 104e7a7282cf..fd19f79f0fc5 100644 --- a/drivers/usb/host/xhci-omap.c +++ b/drivers/usb/host/xhci-omap.c @@ -27,7 +27,7 @@ DECLARE_GLOBAL_DATA_PTR; static struct omap_xhci omap; -inline int __board_usb_init(int index, enum usb_init_type init) +__weak int __board_usb_init(int index, enum usb_init_type init) { return 0; } -- 2.5.3

4 11

[U-Boot] Disabling/Enabling the Data Cache
by Fabio Estevam 24 Nov '15

24 Nov '15

Hi, On a mx6q (armv7) board when I disable and enable the Data Cache the following issue is observed: => dcache Data (writethrough) Cache is ON => dcache off => dcache on data abort pc : [<4ff3d340>] lr : [<4ff3b598>] reloc pc : [<17802340>] lr : [<17800598>] sp : 4f538d50 ip : 00000000 fp : 4f841188 r10: 4ffa4fd4 r9 : 4f538eb0 r8 : 00000002 r7 : 4f539de8 r6 : 00000000 r5 : 00000000 r4 : 00000000 r3 : 00a02000 r2 : 32450000 r1 : 4ff94732 r0 : 00000001 Flags: nZCv IRQs off FIQs off Mode SVC_32 Resetting CPU ... resetting ... U-Boot 2015.10-rc3-24232-g324714b-dirty (Sep 22 2015 - 15:08:16 -0300) CPU: Freescale i.MX6DL rev1.1 996 MHz (running at 792 MHz) Does anyone have any ideas about this? Thanks, Fabio Estevam

3 4

[U-Boot] [PATCH 0/2] Enable DFU for RAM on Allwinner devices
by Siarhei Siamashka 20 Nov '15

20 Nov '15

Hello, DFU allows to transfer large files (such as initrd images) much faster than FEL. Siarhei Siamashka (2): sunxi: Enable DFU for RAM musb: sunxi: Implement dfu_usb_get_reset() drivers/usb/musb-new/sunxi.c | 12 ++++++++++++ include/configs/sunxi-common.h | 30 +++++++++++++++++++++++++----- 2 files changed, 37 insertions(+), 5 deletions(-) -- 2.4.10

6 23

[U-Boot] [PATCH] arm: socfpga: Allow board specific config values for env
by Stefan Roese 19 Nov '15

19 Nov '15

This patch enables the usage of board-specific values for the environment offset and size in SPI NOR flash. Signed-off-by: Stefan Roese <sr(a)denx.de> Cc: Marek Vasut <marex(a)denx.de> Cc: Pavel Machek <pavel(a)denx.de> Cc: Dinh Nguyen <dinguyen(a)opensource.altera.com> --- include/configs/socfpga_common.h | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/include/configs/socfpga_common.h b/include/configs/socfpga_common.h index cece095..ebf0c48 100644 --- a/include/configs/socfpga_common.h +++ b/include/configs/socfpga_common.h @@ -281,7 +281,9 @@ unsigned int cm_get_qspi_controller_clk_hz(void); #define CONFIG_SYS_CONSOLE_IS_IN_ENV #define CONFIG_SYS_CONSOLE_OVERWRITE_ROUTINE #define CONFIG_SYS_CONSOLE_ENV_OVERWRITE +#ifndef CONFIG_ENV_SIZE #define CONFIG_ENV_SIZE 4096 +#endif /* * SPL @@ -327,8 +329,10 @@ unsigned int cm_get_qspi_controller_clk_hz(void); #define CONFIG_DM_SEQ_ALIAS 1 #define CONFIG_SPL_SPI_FLASH_SUPPORT #define CONFIG_SPL_SPI_LOAD +#ifndef CONFIG_SYS_SPI_U_BOOT_OFFS #define CONFIG_SYS_SPI_U_BOOT_OFFS 0x40000 #endif +#endif /* * Stack setup -- 2.6.2

4 14

[U-Boot] [PATCH] common: add CMD_GPIO to Kconfig
by Thomas Chou 18 Nov '15

18 Nov '15

Add CMD_GPIO to Kconfig. Signed-off-by: Thomas Chou <thomas(a)wytron.com.tw> --- common/Kconfig | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/common/Kconfig b/common/Kconfig index 2c42b8e..f2034c2 100644 --- a/common/Kconfig +++ b/common/Kconfig @@ -344,6 +344,11 @@ config CMD_FPGA help FPGA support. +config CMD_GPIO + bool "gpio" + help + GPIO support. + endmenu -- 2.1.4

4 8

[U-Boot] [PATCH] omap3: cm-t3517: add board specific get_board_rev()
by Dmitry Lifshitz 18 Nov '15

18 Nov '15

CM-T3517 has several HW revisions. Add board specific get_board_rev() callback to retrieve revision number. Signed-off-by: Dmitry Lifshitz <lifshitz(a)compulab.co.il> --- board/compulab/cm_t3517/cm_t3517.c | 9 +++++++++ 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/board/compulab/cm_t3517/cm_t3517.c b/board/compulab/cm_t3517/cm_t3517.c index 03b2bad..bf7dec2 100644 --- a/board/compulab/cm_t3517/cm_t3517.c +++ b/board/compulab/cm_t3517/cm_t3517.c @@ -98,6 +98,15 @@ int board_init(void) return 0; } +/* + * Routine: get_board_rev + * Description: read system revision + */ +u32 get_board_rev(void) +{ + return cl_eeprom_get_board_rev(); +}; + int misc_init_r(void) { cl_print_pcb_info(); -- 1.7.5.4

3 4