[U-Boot] [PATCH v2 3/3] rockchip: rk3288: Move rockchip_get_cru() out of the driver

2016-10-01 Thread Simon Glass
This function is called from outside the driver. It should be placed into common SoC code. Move it. Signed-off-by: Simon Glass --- Changes in v2: - Rebase to mainline and fix resulting build error - Add a similar change for rk3036 and rk3399

[U-Boot] [PATCH v2 1/3] rockchip: rk3036: Move rockchip_get_cru() out of the driver

2016-10-01 Thread Simon Glass
This function is called from outside the driver. It should be placed into common SoC code. Move it. Also rename the driver symbol to be more consistent with the other rockchip clock drivers. Signed-off-by: Simon Glass --- Changes in v2: None

[U-Boot] [PATCH v2 2/3] rockchip: rk3399: Move rockchip_get_cru() out of the driver

2016-10-01 Thread Simon Glass
This function is called from outside the driver. It should be placed into common SoC code. Move it. Also rename the driver symbol to be more consistent with the other rockchip clock drivers. Signed-off-by: Simon Glass --- Changes in v2: None

[U-Boot] S5PV210 boot problem

2016-10-01 Thread nader last
Good day, I have a custom HW around an s5pv210 soc. It's based on mini210 board and uses u-boot v1.3.4 as its bootloader. I want to use a newer version of u-boot to boot my HW. AFAIK in recent version of u-boot (e.g. v2016.09) GONI board is the only supported HW which is based on s5pv210 too. I

Re: [U-Boot] [PATCH v2 11/12] RFC: Use binman for a sunxi board

2016-10-01 Thread Tom Rini
On Sat, Oct 01, 2016 at 06:29:42PM -0600, Simon Glass wrote: > Hi Tom, > > On 1 October 2016 at 18:15, Tom Rini wrote: > > On Wed, Sep 28, 2016 at 09:46:25AM -0600, Simon Glass wrote: > >> Hi Tom, > >> > >> On 27 September 2016 at 19:55, Tom Rini wrote: >

Re: [U-Boot] [PATCH 3/4] rockchip: move common function from board-file to rk3036-board.c

2016-10-01 Thread Simon Glass
On 22 September 2016 at 20:58, Simon Glass wrote: > On 19 September 2016 at 04:46, Jacob Chen wrote: >> To keep it same with 3288 >> >> Signed-off-by: Jacob Chen >> --- >> >> arch/arm/mach-rockchip/Makefile|

Re: [U-Boot] [PATCH 2/4] rockchip: rename board.c to rk3288-board.c

2016-10-01 Thread Simon Glass
On 22 September 2016 at 20:58, Simon Glass wrote: > On 19 September 2016 at 04:46, Jacob Chen wrote: >> Signed-off-by: Jacob Chen >> --- >> >> arch/arm/mach-rockchip/Makefile| 2 +- >>

Re: [U-Boot] [PATCH 4/4] rockchip: add boot-mode support for rk3288, rk3036

2016-10-01 Thread Simon Glass
On 22 September 2016 at 20:58, Simon Glass wrote: > On 19 September 2016 at 04:46, Jacob Chen wrote: >> rockchip platform have a protocol to pass the the kernel reboot mode to >> bootloader >> by some special registers when system reboot. In

Re: [U-Boot] [PATCH 1/4] rockchip: move partitons define from 3036-kylin to 3036-common

2016-10-01 Thread Simon Glass
On 22 September 2016 at 20:58, Simon Glass wrote: > On 19 September 2016 at 04:46, Jacob Chen wrote: >> To keep it same with 3288. >> >> Signed-off-by: Jacob Chen >> --- >> >> include/configs/kylin_rk3036.h | 24

Re: [U-Boot] [PATCH v2] cros_ec: Fix issue with cros_ec_flash_write command

2016-10-01 Thread Simon Glass
On 22 September 2016 at 22:15, Simon Glass wrote: > On 12 September 2016 at 13:57, Moritz Fischer > wrote: >> This commit fixes an issue where data is written to an >> invalid memory location. >> The issue has been introduced in commit >> (88364387

Re: [U-Boot] [PATCH 1/2] cros_ec: Add function to read back flash parameters

2016-10-01 Thread Simon Glass
On 18 September 2016 at 18:58, Simon Glass wrote: > On 13 September 2016 at 15:44, Moritz Fischer > wrote: >> Add support for reading back flash parameters as reported by >> the ec. >> >> Signed-off-by: Moritz Fischer >>

Re: [U-Boot] [PATCH 2/2] cros_ec: Add crosec flashinfo command

2016-10-01 Thread Simon Glass
On 18 September 2016 at 18:58, Simon Glass wrote: > Hi Moritz, > > On 13 September 2016 at 15:44, Moritz Fischer > wrote: >> Add command to print out the flash info as reported by the >> ec. The data read back includes size, write block size, >>

Re: [U-Boot] [PATCH 12/13] buildman: Don't show a stacktrace on Ctrl-C

2016-10-01 Thread Simon Glass
On 18 September 2016 at 16:48, Simon Glass wrote: > When Ctrl-C is pressed, just exited quietly. There is no sense in displaying > a stack trace since buildman will always be in the same place: waiting for > threads to complete building all the jobs on the queue. > >

Re: [U-Boot] [PATCH 11/13] buildman: Drop the 'active' flag in the builder

2016-10-01 Thread Simon Glass
On 18 September 2016 at 16:48, Simon Glass wrote: > This serves no real purpose, since when we are not active, we exit. Drop it. > > Signed-off-by: Simon Glass > --- > > tools/buildman/builder.py | 7 --- > tools/buildman/builderthread.py | 10

Re: [U-Boot] [PATCH v2 11/12] RFC: Use binman for a sunxi board

2016-10-01 Thread Simon Glass
Hi Tom, On 1 October 2016 at 18:15, Tom Rini wrote: > On Wed, Sep 28, 2016 at 09:46:25AM -0600, Simon Glass wrote: >> Hi Tom, >> >> On 27 September 2016 at 19:55, Tom Rini wrote: >> > On Sun, Sep 25, 2016 at 03:52:27PM -0600, Simon Glass wrote: >> > >> >>

Re: [U-Boot] [PATCH 13/13] buildman: Drop the 'alive' flag in BuilderThread

2016-10-01 Thread Simon Glass
On 18 September 2016 at 16:48, Simon Glass wrote: > This is not used, so drop it. > > Signed-off-by: Simon Glass > --- > > tools/buildman/builderthread.py | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) Applied to u-boot-dm.

Re: [U-Boot] [PATCH 09/13] buildman: Put our local libraries first in the path

2016-10-01 Thread Simon Glass
On 18 September 2016 at 16:48, Simon Glass wrote: > If patman is installed on the machine (e.g. in the standard dist-packages > directory), it will find libraries from there in preference to our local > libraries. Adjust the order of the path to ensure that local libraries are

Re: [U-Boot] [PATCH 10/13] buildman: Allow builds to terminate cleanly

2016-10-01 Thread Simon Glass
Hi Tom, On 19 September 2016 at 13:16, Tom Rini wrote: > On Sun, Sep 18, 2016 at 04:48:35PM -0600, Simon Glass wrote: > >> It is annoying that buildman does not respond cleanly to Ctrl-C or SIGINT, >> particularly on machines with lots of CPUS. Unfortunately queue.join() >>

Re: [U-Boot] [PATCH 08/13] buildman: Print a message indicating the build is starting

2016-10-01 Thread Simon Glass
On 18 September 2016 at 16:48, Simon Glass wrote: > Make it clear when buildman actually starts building. This happens when it > has prepared the threads, working directory and output directories. > > Signed-off-by: Simon Glass > --- > >

Re: [U-Boot] [PATCH 07/13] buildman: Print a message when removing old directories

2016-10-01 Thread Simon Glass
On 18 September 2016 at 16:48, Simon Glass wrote: > When buildman starts, it prepares its output directory by removing any old > build directories which will not be used this time. This can happen if a > previous build left directories around for commit hashes which are

Re: [U-Boot] [PATCH 05/13] patman: Flush output when there is no newline

2016-10-01 Thread Simon Glass
On 18 September 2016 at 16:48, Simon Glass wrote: > Output which does not include a newline will not be displayed unless > flushed. Add a flush to ensure that it becomes visible. > > Signed-off-by: Simon Glass > --- > > tools/patman/terminal.py | 2 ++ > 1

Re: [U-Boot] [PATCH 06/13] buildman: Tidy up the 'cloning' message

2016-10-01 Thread Simon Glass
On 18 September 2016 at 16:48, Simon Glass wrote: > On a machine with a lot of CPUs this prints a lot of useless lines of the > form: > >Cloning repo for thread > > Adjust the output so that these all appear on one line, and disappear when > the cloning is complete. > >

Re: [U-Boot] [PATCH v2 11/12] RFC: Use binman for a sunxi board

2016-10-01 Thread Tom Rini
On Wed, Sep 28, 2016 at 09:46:25AM -0600, Simon Glass wrote: > Hi Tom, > > On 27 September 2016 at 19:55, Tom Rini wrote: > > On Sun, Sep 25, 2016 at 03:52:27PM -0600, Simon Glass wrote: > > > >> Add an example usage of binman for a sunxi board. This involves adding the > >>

[U-Boot] [PATCH v3 5/7] efi_loader: Readd freed pages to memory pool

2016-10-01 Thread Stefan Brüns
Currently each allocation creates a new mapping. Readding the mapping as free memory (EFI_CONVENTIONAL_MEMORY) potentially allows to hand out an existing mapping, thus limiting the number of mapping descriptors in the memory map. Mitigates a problem with current (4.8rc7) linux kernels when doing

[U-Boot] [PATCH v3 7/7] efi_loader: Do not leak memory when unlinking a mapping

2016-10-01 Thread Stefan Brüns
As soon as a mapping is unlinked from the list, there are no further references to it, so it should be freed. If it not unlinked, update the start address and length. Signed-off-by: Stefan Brüns Reviewed-by: Alexander Graf ---

[U-Boot] [PATCH v2 0/6] efi_loader: implement BS.FreePages()/BS.FreePool, some cleanup

2016-10-01 Thread Stefan Brüns
Linux 4.8 no longer boots, as it runs out of descriptor space. Readding the mapping allows reuse, and the next AllocatePool will likely not add a new mapping. Also fix some small issues found while debugging. v2, updated: efi_loader: Fix memory map size check to avoid out-of-bounds access -

[U-Boot] [PATCH v3 3/7] efi_loader: Move efi_allocate_pool implementation to efi_memory.c

2016-10-01 Thread Stefan Brüns
Implementation essentially unchanged, but use EFI_PAGE_MASK/SHIFT instead of numeric constants. Signed-off-by: Stefan Brüns --- include/efi_loader.h | 3 +++ lib/efi_loader/efi_boottime.c | 11 +-- lib/efi_loader/efi_memory.c | 11 +++ 3

[U-Boot] [PATCH v3 4/7] efi_loader: Track size of pool allocations to allow freeing

2016-10-01 Thread Stefan Brüns
allocate_pool has to return a buffer which is 8-byte aligned. Shift the region returned by allocate_pages by 8 byte and store the size in the headroom. The 8 byte overhead is neglegible, but provides the required size when freeing the allocation later. Signed-off-by: Stefan Brüns

[U-Boot] [PATCH v3 1/7] efi_loader: Update description of internal efi_mem_carve_out

2016-10-01 Thread Stefan Brüns
In 74c16acce30bb882ad5951829d8dafef8eea564c the return values where changed, but the description was kept. Signed-off-by: Stefan Brüns --- lib/efi_loader/efi_memory.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git

[U-Boot] [PATCH v3 6/7] efi_loader: Keep memory mapping sorted when splitting an entry

2016-10-01 Thread Stefan Brüns
The code assumes sorted mappings in descending address order. When splitting a mapping, insert the new part next to the current mapping. Signed-off-by: Stefan Brüns Reviewed-by: Alexander Graf --- lib/efi_loader/efi_memory.c | 3 ++- 1 file changed,

[U-Boot] [PATCH v3 2/7] efi_loader: Fix memory map size check to avoid out-of-bounds access

2016-10-01 Thread Stefan Brüns
Do not overwrite the specified size of the provided buffer without having checked it is sufficient. If the buffer is to small, memory_map_size is updated to indicate the required size, and an error code is returned. Signed-off-by: Stefan Brüns ---

[U-Boot] [PATCH v3 1/3] dm: mmc: Support erase

2016-10-01 Thread Simon Glass
At present erase is not suported with CONFIG_DM_OPS. Add it so that MMC devices can be erased. Signed-off-by: Simon Glass --- Changes in v3: None drivers/mmc/mmc-uclass.c | 1 + drivers/mmc/mmc_private.h | 5 ++--- drivers/mmc/mmc_write.c | 10 -- 3 files

[U-Boot] [PATCH v3 3/3] dm: mmc: Enable DM_MMC_OPS by default with DM_MMC

2016-10-01 Thread Simon Glass
These two options go together and it is best to do the conversion in one step. So enable DM_MMC_OPS by default if DM_MMC is enabled. Signed-off-by: Simon Glass --- Changes in v3: - Updates to remove build errors from tegra and other boards

[U-Boot] [PATCH v3 2/3] dm: blk: Enable CONFIG_BLK if DM_MMC is enabled

2016-10-01 Thread Simon Glass
To speed up conversion to CONFIG_BLK, enable it by default when DM_MMC is enabled. Signed-off-by: Simon Glass --- Changes in v3: None configs/am335x_boneblack_vboot_defconfig | 1 + configs/am335x_evm_defconfig | 1 + configs/am43xx_evm_defconfig

[U-Boot] [PATCH v3 0/3] dm: mmc: Default to DM_MMC_OPS and BLK

2016-10-01 Thread Simon Glass
Two of these patches have been around for a while, but I've been waiting to apply them early in the release cycle to permit testing. The goal is to make CONFIG_DM_MMC_OPS and CONFIG_BLK enabled by default when driver model is used for mmc. Then these options can be removed. All mmc drivers should

[U-Boot] [PATCH] sandbox: Use the address in readl/writel() functions

2016-10-01 Thread Simon Glass
At present these functions do not touch addr, which can raising warnings about unused variables. This fixes the following warnings: sandbox_spl defconfig drivers/core/regmap.c: In function ‘regmap_read’: drivers/core/regmap.c:125:12: warning: unused variable ‘ptr’ [-Wunused-variable] uint32_t

Re: [U-Boot] [PATCH v3 03/29] Convert CONSOLE_PRE_CONSOLE_BUFFER options to Kconfig

2016-10-01 Thread Simon Glass
Hi Siarhei, On 30 September 2016 at 00:00, Siarhei Siamashka wrote: > > Hello Simon, > > On Thu, 29 Sep 2016 14:23:02 -0600 > Simon Glass wrote: > > > Move these option to Kconfig and tidy up existing uses. > > > > Signed-off-by: Simon Glass

[U-Boot] [PATCH 4/5] sandbox/fs: Set correct filetype for unknown filetype

2016-10-01 Thread Stefan Brüns
Signed-off-by: Stefan Brüns --- arch/sandbox/cpu/os.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/sandbox/cpu/os.c b/arch/sandbox/cpu/os.c index 16af3f5..df2bd4c 100644 --- a/arch/sandbox/cpu/os.c +++ b/arch/sandbox/cpu/os.c @@ -363,6 +363,8 @@ int

[U-Boot] [PATCH 5/5] sandbox/fs: Use readdir instead of deprecated readdir_r

2016-10-01 Thread Stefan Brüns
Using readdir_r limits the maximum file name length and may even be unsafe, and is thus deprecated in since glibc 2.24. Signed-off-by: Stefan Brüns --- arch/sandbox/cpu/os.c | 19 +++ 1 file changed, 11 insertions(+), 8 deletions(-) diff --git

[U-Boot] [PATCH 0/5] Fix some trivial issues in sandbox/fs

2016-10-01 Thread Stefan Brüns
Stefan Brüns (5): sandbox/fs: Free memory allocated by os_dirent_ls sandbox/fs: Make linking of nodes in os_dirent_ls more obvious sandbox/fs: Use correct size path name buffer sandbox/fs: Set correct filetype for unknown filetype sandbox/fs: Use readdir instead of deprecated readdir_r

[U-Boot] [PATCH 3/5] sandbox/fs: Use correct size path name buffer

2016-10-01 Thread Stefan Brüns
The readdir linux manpage explicitly states (quoting POSIX.1) that sizeof(d_name) is not correct for determining the required size, but to always use strlen. Grow the buffer if needed. Signed-off-by: Stefan Brüns --- arch/sandbox/cpu/os.c | 13 ++--- 1 file

[U-Boot] [PATCH 2/5] sandbox/fs: Make linking of nodes in os_dirent_ls more obvious

2016-10-01 Thread Stefan Brüns
Previously, after reading/creating the second dirent, the second entry would be chained to the first entry and the first entry would be linked to head. Instead, immediately link the first entry to head. Signed-off-by: Stefan Brüns --- arch/sandbox/cpu/os.c | 4 ++--

[U-Boot] [PATCH 1/5] sandbox/fs: Free memory allocated by os_dirent_ls

2016-10-01 Thread Stefan Brüns
Signed-off-by: Stefan Brüns --- fs/sandbox/sandboxfs.c | 1 + include/os.h | 11 ++- 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/fs/sandbox/sandboxfs.c b/fs/sandbox/sandboxfs.c index 2703eed..cd10fd6 100644 ---

Re: [U-Boot] [PATCH v2 3/6] efi_loader: Track size of pool allocations to allow freeing

2016-10-01 Thread Alexander Graf
On 01.10.16 19:31, Stefan Brüns wrote: > allocate_pool has to return a buffer which is 8-byte aligned. Shift the > region returned by allocate_pages by 8 byte and store the size in the > headroom. The 8 byte overhead is neglegible, but provides the required > size when freeing the allocation

Re: [U-Boot] [PATCH v2 2/6] efi_loader: Fix memory map size check to avoid out-of-bounds access

2016-10-01 Thread Alexander Graf
On 01.10.16 19:31, Stefan Brüns wrote: > Do not overwrite the specified size of the provided buffer without > having checked it is sufficient. > > If the buffer is to small, memory_map_size is updated to indicate the > required size, and an error code is returned. > > Signed-off-by: Stefan

[U-Boot] [PATCH v2 5/6] efi_loader: Keep memory mapping sorted when splitting an entry

2016-10-01 Thread Stefan Brüns
The code assumes sorted mappings in descending address order. When splitting a mapping, insert the new part next to the current mapping. Signed-off-by: Stefan Brüns Reviewed-by: Alexander Graf --- lib/efi_loader/efi_memory.c | 3 ++- 1 file changed,

[U-Boot] [PATCH v2 3/6] efi_loader: Track size of pool allocations to allow freeing

2016-10-01 Thread Stefan Brüns
allocate_pool has to return a buffer which is 8-byte aligned. Shift the region returned by allocate_pages by 8 byte and store the size in the headroom. The 8 byte overhead is neglegible, but provides the required size when freeing the allocation later. Signed-off-by: Stefan Brüns

[U-Boot] [PATCH v2 1/6] efi_loader: Update description of internal efi_mem_carve_out

2016-10-01 Thread Stefan Brüns
In 74c16acce30bb882ad5951829d8dafef8eea564c the return values where changed, but the description was kept. Signed-off-by: Stefan Brüns --- lib/efi_loader/efi_memory.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git

[U-Boot] [PATCH v2 2/6] efi_loader: Fix memory map size check to avoid out-of-bounds access

2016-10-01 Thread Stefan Brüns
Do not overwrite the specified size of the provided buffer without having checked it is sufficient. If the buffer is to small, memory_map_size is updated to indicate the required size, and an error code is returned. Signed-off-by: Stefan Brüns ---

[U-Boot] [PATCH v2 4/6] efi_loader: Readd freed pages to memory pool

2016-10-01 Thread Stefan Brüns
Currently each allocation creates a new mapping. Readding the mapping as free memory (EFI_CONVENTIONAL_MEMORY) potentially allows to hand out an existing mapping, thus limiting the number of mapping descriptors in the memory map. Mitigates a problem with current (4.8rc7) linux kernels when doing

[U-Boot] [PATCH v2 6/6] efi_loader: Do not leak memory when unlinking a mapping

2016-10-01 Thread Stefan Brüns
As soon as a mapping is unlinked from the list, there are no further references to it, so it should be freed. If it not unlinked, update the start address and length. Signed-off-by: Stefan Brüns Reviewed-by: Alexander Graf ---

[U-Boot] [PATCH v2 0/6] efi_loader: implement BS.FreePages()/BS.FreePool, some cleanup

2016-10-01 Thread Stefan Brüns
Linux 4.8 no longer boots, as it runs out of descriptor space. Readding the mapping allows reuse, and the next AllocatePool will likely not add a new mapping. Also fix some small issues found while debugging. v2, updated: efi_loader: Fix memory map size check to avoid out-of-bounds access -

Re: [U-Boot] [PATCH 07/23] pci: Handle MIPS systems with virtual CONFIG_SYS_SDRAM_BASE

2016-10-01 Thread Paul Burton
On Tuesday, 27 September 2016 12:01:32 BST Daniel Schwierzeck wrote: > 2016-09-27 2:35 GMT+02:00 Simon Glass : > > Hi Paul, > > > > On 26 September 2016 at 12:29, Paul Burton wrote: > >> The decode_regions() function in the PCI code presumes that > >>

[U-Boot] [PATCH 26/27] mips: Use a physical CONFIG_SYS_SDRAM_BASE for remaining boards

2016-10-01 Thread Paul Burton
README states that CONFIG_SYS_SDRAM_BASE should the physical base address of SDRAM, whilst up until now various pieces of generic code have presumed that it can be directly accessed by the CPU & MIPS has provided a virtual address for CONFIG_SYS_SDRAM_BASE. Other generic code expects

[U-Boot] [PATCH 27/27] mips: Remove virt_to_phys call on bi_memstart

2016-10-01 Thread Paul Burton
CONFIG_SYS_SDRAM_BASE & by extension bi_memstart are now physical addresses, so there's no need to call virt_to_phys in arch_fixup_fdt. Remove the call. Signed-off-by: Paul Burton Cc: Daniel Schwierzeck --- arch/mips/lib/bootm.c | 2 +- 1

[U-Boot] [PATCH 24/27] malta: Use a physical CONFIG_SYS_SDRAM_BASE

2016-10-01 Thread Paul Burton
README states that CONFIG_SYS_SDRAM_BASE should the physical base address of SDRAM, whilst up until now various pieces of generic code have presumed that it can be directly accessed by the CPU & MIPS has provided a virtual address for CONFIG_SYS_SDRAM_BASE. Other generic code expects

[U-Boot] [PATCH 25/27] xilfpga: Use a physical CONFIG_SYS_SDRAM_BASE

2016-10-01 Thread Paul Burton
README states that CONFIG_SYS_SDRAM_BASE should the physical base address of SDRAM, whilst up until now various pieces of generic code have presumed that it can be directly accessed by the CPU & MIPS has provided a virtual address for CONFIG_SYS_SDRAM_BASE. Other generic code expects

[U-Boot] [PATCH 22/27] mips: Ensure stack is at a virtual address

2016-10-01 Thread Paul Burton
In preparation for making CONFIG_SYS_SDRAM_BASE a physical address on MIPS as README says it should be, ensure that our default CONFIG_SYS_INIT_SP_ADDR is placed in (c)kseg0 rather than being a simple offset addition to CONFIG_SYS_SDRAM_BASE. Signed-off-by: Paul Burton

[U-Boot] [PATCH 23/27] boston: Provide physical CONFIG_SYS_SDRAM_BASE

2016-10-01 Thread Paul Burton
README states that CONFIG_SYS_SDRAM_BASE should the physical base address of SDRAM, whilst up until now various pieces of generic code have presumed that it can be directly accessed by the CPU & MIPS has provided a virtual address for CONFIG_SYS_SDRAM_BASE. Other generic code expects

[U-Boot] [PATCH 21/27] mips: Use ram_top, not bi_memsize, in arch_lmb_reserve

2016-10-01 Thread Paul Burton
When calculating the region to reserve for the stack in arch_lmb_reserve, make use of ram_top instead of adding bi_memsize to CONFIG_SYS_SDRAM_BASE. This avoids overflow if the system has enough memory to reach the end of the address space. Signed-off-by: Paul Burton Cc:

[U-Boot] [PATCH 20/27] image: Use ram_top, not bi_memsize, in getenv_bootm_size

2016-10-01 Thread Paul Burton
When determining the region of memory to allow for use by bootm, using bi_memstart & adding bi_memsize can cause problems if that leads to an integer overflow. For example on some MIPS systems bi_memstart would be 0x8000 (ie. the start of the MIPS ckseg0 region) and if the system has

[U-Boot] [PATCH 19/27] image: Account for CONFIG_SYS_SDRAM_BASE being physical

2016-10-01 Thread Paul Burton
README declares that CONFIG_SYS_SDRAM_BASE is meant to be the physical address of SDRAM, but right now that is not the case on MIPS systems, where it is instead a virtual address. In preparation for making it physical, use phys_to_virt to translate CONFIG_SYS_SDRAM_BASE & the associated

[U-Boot] [PATCH 18/27] board_f: Account for CONFIG_SYS_SDRAM_BASE being physical

2016-10-01 Thread Paul Burton
README declares that CONFIG_SYS_SDRAM_BASE is meant to be the physical address of SDRAM, but right now that is not the case on MIPS systems. In preparation for making it so, use phys_to_virt to translate CONFIG_SYS_SDRAM_BASE to the ram_top field of struct global_data which is then used to

[U-Boot] [PATCH 17/27] sandbox: Use asm-generic/io.h

2016-10-01 Thread Paul Burton
Convert the sandbox architecture to make use of the new asm-generic/io.h to provide address mapping functions. As sandbox actually performs non-identity mapping between physical & virtual addresses we can't simply make use of the generic mapping functions, but are able to implement phys_to_virt()

[U-Boot] [PATCH 16/27] powerpc: Use asm-generic/io.h

2016-10-01 Thread Paul Burton
Convert the powerpc architecture to make use of the new asm-generic/io.h to provide address mapping functions. As powerpc can actually perform non-identity mapping between physical & virtual addresses we can't simply make use of the generic phys_to_virt() & virt_to_phys() functions. However since

[U-Boot] [PATCH 15/27] nios2: Use asm-generic/io.h

2016-10-01 Thread Paul Burton
Convert the nios2 architecture to make use of the new asm-generic/io.h to provide address mapping functions. As nios2 actually performs non-identity mapping between physical & virtual addresses we can't simply make use of the generic functions, with the exception of being able to drop our no-op

[U-Boot] [PATCH 14/27] mips: Fix map_physmem for cached mappings

2016-10-01 Thread Paul Burton
map_physmem should return a pointer that can be used by the CPU to access the given memory - on MIPS simply returning the physical address as it does prior to this patch doesn't achieve that. Instead return a pointer to the memory within (c)kseg0, which matches up consistently with the (c)kseg1

[U-Boot] [PATCH 13/27] mips: Use asm-generic/io.h

2016-10-01 Thread Paul Burton
Convert the mips architecture to make use of the new asm-generic/io.h to provide address mapping functions. As mips actually performs non-identity mapping between physical & virtual addresses we can't simply make use of the generic functions, with the exception of being able to drop our no-op

[U-Boot] [PATCH 12/27] xtensa: Use asm-generic/io.h

2016-10-01 Thread Paul Burton
Convert the xtensa architecture to make use of the new asm-generic/io.h to provide address mapping functions. As the generic implementations are suitable for xtensa this is primarily a matter of moving code. This has only been build-tested, feedback from architecture maintainers is welcome.

[U-Boot] [PATCH 11/27] x86: Use asm-generic/io.h

2016-10-01 Thread Paul Burton
Convert the x86 architecture to make use of the new asm-generic/io.h to provide address mapping functions. As the generic implementations are suitable for x86 this is primarily a matter of moving code. This has only been build-tested, feedback from architecture maintainers is welcome.

[U-Boot] [PATCH 08/27] openrisc: Use asm-generic/io.h

2016-10-01 Thread Paul Burton
Convert the openrisc architecture to make use of the new asm-generic/io.h to provide address mapping functions. As the generic implementations are suitable for openrisc this is primarily a matter of removing code. Feedback from architecture maintainers is welcome. Signed-off-by: Paul Burton

[U-Boot] [PATCH 09/27] sh: Use asm-generic/io.h

2016-10-01 Thread Paul Burton
Convert the sh architecture to make use of the new asm-generic/io.h to provide address mapping functions. As the generic implementations are suitable for sh this is primarily a matter of moving code. Feedback from architecture maintainers is welcome. Signed-off-by: Paul Burton

[U-Boot] [PATCH 10/27] sparc: Use asm-generic/io.h

2016-10-01 Thread Paul Burton
Convert the sparc architecture to make use of the new asm-generic/io.h to provide address mapping functions. As the generic implementations are suitable for sparc this is primarily a matter of moving code. Feedback from architecture maintainers is welcome. Signed-off-by: Paul Burton

[U-Boot] [PATCH 07/27] nds32: Use asm-generic/io.h

2016-10-01 Thread Paul Burton
Convert the nds32 architecture to make use of the new asm-generic/io.h to provide address mapping functions. As the generic implementations are suitable for nds32 this is primarily a matter of removing code. Feedback from architecture maintainers is welcome. Signed-off-by: Paul Burton

[U-Boot] [PATCH 06/27] microblaze: Use asm-generic/io.h

2016-10-01 Thread Paul Burton
Convert the microblaze architecture to make use of the new asm-generic/io.h to provide address mapping functions. As the generic implementations are suitable for microblaze this is primarily a matter of removing code. Feedback from architecture maintainers is welcome. Signed-off-by: Paul Burton

[U-Boot] [PATCH 04/27] blackfin: Use asm-generic/io.h

2016-10-01 Thread Paul Burton
Convert the blackfin architecture to make use of the new asm-generic/io.h to provide address mapping functions. As the generic implementations are suitable for blackfin this is primarily a matter of removing code. Feedback from architecture maintainers is welcome. Signed-off-by: Paul Burton

[U-Boot] [PATCH 05/27] m68k: Use asm-generic/io.h

2016-10-01 Thread Paul Burton
Convert the m68k architecture to make use of the new asm-generic/io.h to provide address mapping functions. As the generic implementations are suitable for m68k this is primarily a matter of emoving code. Feedback from architecture maintainers is welcome. Signed-off-by: Paul Burton

[U-Boot] [PATCH 03/27] arm: Use asm-generic/io.h

2016-10-01 Thread Paul Burton
Convert the arm architecture to make use of the new asm-generic/io.h to provide address mapping functions. As the generic implementations are suitable for arm this is primarily a matter of removing code. This has only been build-tested, feedback from architecture maintainers is welcome.

[U-Boot] [PATCH 02/27] arc: Use asm-generic/io.h

2016-10-01 Thread Paul Burton
Convert the arc architecture to make use of the new asm-generic/io.h to provide address mapping functions. As the generic implementations are suitable for arc this is primarily a matter of removing code. Feedback from architecture maintainers is welcome. Signed-off-by: Paul Burton

[U-Boot] [PATCH 00/27] Clean up address mapping functions & CONFIG_SYS_SDRAM_BASE

2016-10-01 Thread Paul Burton
README states that CONFIG_SYS_SDRAM_BASE should be the physical address of the base of SDRAM memory. This is expected by some code such as the PCI layer, which uses CONFIG_SYS_SDRAM_BASE to set up a region for system memory. Other code such as the image loading code used by bootm or the generic

[U-Boot] [PATCH 01/27] Provide a generic io.h & address mapping functions

2016-10-01 Thread Paul Burton
Most architectures currently supported by U-Boot use trivial implementations of map_to_physmem & virt_to_phys which simply cast a physical address to a pointer for use a virtual address & vice-versa. This results in a lot of duplicate implementations of these mapping functions. The functions

Re: [U-Boot] [PATCH v3 03/29] Convert CONSOLE_PRE_CONSOLE_BUFFER options to Kconfig

2016-10-01 Thread Soeren Moch
>> > Hello Simon, >> > >> > On Thu, 29 Sep 2016 14:23:02 -0600 >> > Simon Glass wrote: >> > >>> > > Move these option to Kconfig and tidy up existing uses. >>> > > >>> > > Signed-off-by: Simon Glass >>> > > --- >>> > > >>> > > Changes in v3: None >>> >

Re: [U-Boot] Uboot send pull request

2016-10-01 Thread Tom Rini
On Thu, Sep 29, 2016 at 04:22:45PM +0800, ub...@andestech.com wrote: > Hi Tom, > > Please pull the following patch from u-boot-nds32 into your tree. > Thanks! > > The following changes since commit 06572f0301c18e63b887efc91803bb9467e55dbe: > > Merge git://www.denx.de/git/u-boot-ppc4xx

Re: [U-Boot] Please pull u-boot-85xx master

2016-10-01 Thread Tom Rini
On Wed, Sep 28, 2016 at 05:49:26PM +, york sun wrote: > Tom, > > The following changes since commit 06572f0301c18e63b887efc91803bb9467e55dbe: > >Merge git://www.denx.de/git/u-boot-ppc4xx (2016-09-27 12:48:18 -0400) > > are available in the git repository at: > >

Re: [U-Boot] [PULL] u-boot-usb/master

2016-10-01 Thread Tom Rini
On Wed, Sep 28, 2016 at 07:53:52PM +0200, Marek Vasut wrote: > The following changes since commit cbe7706ab8aab06c18edaa9b120371f9c8012728: > > Merge git://git.denx.de/u-boot-fsl-qoriq (2016-09-26 17:10:56 -0400) > > are available in the git repository at: > >

Re: [U-Boot] is there any issue with creating and using more than one hashtable?

2016-10-01 Thread Wolfgang Denk
Dear Robert, In message you wrote: > > > 244 /* If there is a callback, call it */ > > 245 if (htab->table[idx].entry.callback && > > 246

Re: [U-Boot] is there any issue with creating and using more than one hashtable?

2016-10-01 Thread Robert P. J. Day
On Sat, 1 Oct 2016, Wolfgang Denk wrote: > Dear Robert, ... snip ... > Actually the TFTP will not access the environment directly to > determine the boot parameters like bootfile, ipaddr, gatewayip, > netmask, serverip, etc.; instead, it uses internal variables. So > your environment settings

Re: [U-Boot] is there any issue with creating and using more than one hashtable?

2016-10-01 Thread Robert P. J. Day
On Sat, 1 Oct 2016, Wolfgang Denk wrote: > Dear Robert, > > In message you > wrote: > > > > in misc_init_r(), i create a new hashtable to just grab the > > contents of that bootline as is. i then create a *second* > > hashtable and

Re: [U-Boot] x86: Booting Linux v4.7+ broken

2016-10-01 Thread Stefan Roese
On 01.10.2016 05:52, Yinghai Lu wrote: On Tue, Sep 27, 2016 at 2:28 AM, Stefan Roese wrote: Hi! I just tried to boot a recent Linux kernel on a BayTrail based x86 board and noticed that booting Linux kernel version v4.7+ does not work. git bisecting leads to this kernel commit

Re: [U-Boot] is there any issue with creating and using more than one hashtable?

2016-10-01 Thread Wolfgang Denk
Dear Robert, In message you wrote: > > in misc_init_r(), i create a new hashtable to just grab the contents > of that bootline as is. i then create a *second* hashtable and > selectively move over just those keys i want from the

Re: [U-Boot] x86: Booting Linux v4.7+ broken

2016-10-01 Thread Yinghai Lu
On Tue, Sep 27, 2016 at 2:28 AM, Stefan Roese wrote: > Hi! > > I just tried to boot a recent Linux kernel on a BayTrail based > x86 board and noticed that booting Linux kernel version v4.7+ > does not work. git bisecting leads to this kernel commit > changing the compressed image