Re: [U-Boot] [PATCH] fs/fat: debug-print file read position during file_fat_read_at()

2018-08-13 Thread Heinrich Schuchardt
On 08/14/2018 04:35 AM, Andreas Dannenberg wrote: > In order to make the debug print in file_fat_read_at() a tad more useful, > show the offset the file is being read at alongside the filename. > > Suggested-by: Tero Kristo > Signed-off-by: Andreas Dannenberg > --- > > Small addition but

Re: [U-Boot] [PATCH 1/1] fat: provide position in debug message

2018-08-13 Thread Heinrich Schuchardt
On 08/09/2018 08:37 PM, Heinrich Schuchardt wrote: > The debug message in file_fat_read_at() is not very useful without > indicating the position at which the file is read. > > Suggested-by: Tero Kristo > Signed-off-by: Heinrich Schuchardt > --- > fs/fat/fat.c | 2 +- > 1 file changed, 1

[U-Boot] [RFC PATCH] arm: zynq: read mac address from SPI flash memory

2018-08-13 Thread Luis Araneda
Implement a method for reading the MAC address from an SPI flash memory. In particular, this method is used by the Zybo Z7 board to read the MAC address from the OTP region in the SPI NOR memory Signed-off-by: Luis Araneda --- I'm trying to implement the reading of the MAC address of the Zybo

Re: [U-Boot] [PATCH] Kconfig: Migrate CONFIG_NR_DRAM_BANKS

2018-08-13 Thread Ramon Fried
On Tue, Aug 14, 2018 at 1:11 AM Marek Vasut wrote: > On 08/14/2018 12:00 AM, Ramon Fried wrote: > > Move CONFIG_NR_DRAM_BANKS from headers to Kconfig. > > > > Signed-off-by: Ramon Fried > > Somehow this seems to be missing a lot of cleanups in include/configs/ > ... look at moveconfig.py >

[U-Boot] [PATCH] fs/fat: debug-print file read position during file_fat_read_at()

2018-08-13 Thread Andreas Dannenberg
In order to make the debug print in file_fat_read_at() a tad more useful, show the offset the file is being read at alongside the filename. Suggested-by: Tero Kristo Signed-off-by: Andreas Dannenberg --- Small addition but helpful nevertheless as none of the other debug prints embedded into

Re: [U-Boot] [PATCH] pci: Support parsing PCI controller DT subnodes

2018-08-13 Thread Bin Meng
Hi Tom, On Tue, Aug 14, 2018 at 1:16 AM, Tom Rini wrote: > On Mon, Aug 13, 2018 at 06:07:14PM +0200, Marek Vasut wrote: >> On 08/13/2018 03:39 PM, Tom Rini wrote: >> [...] >> >> Next step is to upstream the DT >> changes to Linux kernel, then sync the changes to U-Boot to satisfy >>

[U-Boot] [PATCH] ARM: da850evm_direct_nor_defconfig: Enable DM_SERIAL

2018-08-13 Thread Adam Ford
With DM enabled, this patch enables DM_SERIAL and removes the NS16550 initialization from da850_lowlevel since the driver will take care of that itself. Signed-off-by: Adam Ford diff --git a/arch/arm/mach-davinci/da850_lowlevel.c b/arch/arm/mach-davinci/da850_lowlevel.c index

Re: [U-Boot] [PATCH] pci: Support parsing PCI controller DT subnodes

2018-08-13 Thread Bin Meng
Hi Marek, On Mon, Aug 13, 2018 at 9:46 PM, Marek Vasut wrote: > On 08/13/2018 04:24 AM, Bin Meng wrote: >> Hi Marek, >> >> On Fri, Aug 10, 2018 at 8:38 PM, Marek Vasut wrote: >>> On 08/10/2018 02:01 PM, Tom Rini wrote: On Wed, Aug 08, 2018 at 09:37:25PM +0200, Marek Vasut wrote: > On

[U-Boot] [PATCH] ARM: da850evm_direct_nor: Enable CONFIG_BLK

2018-08-13 Thread Adam Ford
At least for now, CONFIG_BLK is working, but this variant of the da850evm doesn't need/support SPL so it's OK to enable it here. Signed-off-by: Adam Ford diff --git a/configs/da850evm_direct_nor_defconfig b/configs/da850evm_direct_nor_defconfig index 9d6c47df50..bb8fa3bd6e 100644 ---

[U-Boot] [ANN] U-Boot v2018.09-rc2 released

2018-08-13 Thread Tom Rini
Hey all, It is release day and despite Travis-CI being really bad about network connections over the weekend, it's back to normal today at least. So I'm putting out v2018.09-rc2 now. I plan on looking at Kconfig migrations and similar things that I can size-test. I think there's still a code

[U-Boot] [PATCH 7/7] common: avb_verify: Fix division by zero in mmc_byte_io()

2018-08-13 Thread Eugeniu Rosca
Compiling U-Boot with ubsan/asan libraries and running it in sandbox may lead to below backtrace: => avb init 0 => avb verify ## Android Verified Boot 2.0 version 1.1.0 read_is_device_unlocked not supported yet common/avb_verify.c:407:31: runtime error: division by zero

[U-Boot] [PATCH 6/7] common: avb_verify: Fix never-occurring avb_free(ops_data)

2018-08-13 Thread Eugeniu Rosca
Cppcheck (v1.85) reports w/o this patch: [common/avb_verify.c:738] -> [common/avb_verify.c:741]: (warning) \ Either the condition 'ops' is redundant or there is possible null \ pointer dereference: ops. Signed-off-by: Eugeniu Rosca --- common/avb_verify.c | 2 +- 1 file changed, 1

[U-Boot] [PATCH 5/7] common: avb_verify: Fix memory leaks

2018-08-13 Thread Eugeniu Rosca
Cppcheck (v1.85) reports w/o this patch: [common/avb_verify.c:351]: (error) Memory leak: part [common/avb_verify.c:356]: (error) Memory leak: part [common/avb_verify.c:361]: (error) Memory leak: part [common/avb_verify.c:366]: (error) Memory leak: part Signed-off-by: Eugeniu Rosca ---

[U-Boot] [PATCH 3/7] common: kconfig: Mark AVB_VERIFY as dependent on PARTITION_UUIDS

2018-08-13 Thread Eugeniu Rosca
Avoid below compiler [1] errors, reproduced with configuration [2]: common/avb_verify.c: In function ‘get_unique_guid_for_partition’: common/avb_verify.c:692:31: error: ‘disk_partition_t {aka struct disk_partition}’ has no member named ‘uuid’ uuid_size = sizeof(part->info.uuid);

[U-Boot] [PATCH 2/7] common: avb_verify: Fix invalid 'for' loop condition

2018-08-13 Thread Eugeniu Rosca
Fix below compiler [1] warning: common/avb_verify.c: In function ‘avb_find_dm_args’: common/avb_verify.c:179:30: warning: left-hand operand of comma expression has no effect [-Wunused-value] for (i = 0; i < AVB_MAX_ARGS, args[i]; ++i) { [1] aarch64-linux-gnu-gcc (Linaro GCC 7.2-2017.11)

[U-Boot] [PATCH 1/7] libavb: Handle wrong hashtree_error_mode in avb_append_options()

2018-08-13 Thread Eugeniu Rosca
From: Ievgen Maliarenko Exit with AVB_SLOT_VERIFY_RESULT_ERROR_INVALID_ARGUMENT when hashtree_error_mode value passed to avb_append_options() is unknown (not from AvbHashtreeErrorMode enum). Otherwise, default value is not handled in the switch(hashtree_error_mode), which causes below compile

[U-Boot] [PATCH 4/7] common: avb_verify: Make local data static

2018-08-13 Thread Eugeniu Rosca
Fix sparse complaint: common/avb_verify.c:14:21: warning: \ symbol 'avb_root_pub' was not declared. Should it be static? Signed-off-by: Eugeniu Rosca --- common/avb_verify.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/common/avb_verify.c b/common/avb_verify.c index

Re: [U-Boot] [U-Boot, v1, 10/11] configs: stm32f469-discovery: Add DISTRO_DEFAULT support

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 11:46:20AM +0200, Patrice Chotard wrote: > Add DISTRO_DEFAULT support to be able to boot on mmc > by default on boot. > > Signed-off-by: Patrice Chotard Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [U-Boot] ARM: dts: stm32mp157: Add ADC DT node

2018-08-13 Thread Tom Rini
On Mon, Aug 06, 2018 at 09:54:04AM +0200, Patrice Chotard wrote: > Add ADC device tree node. This allows to get analog conversions on > stm32mp157. > > Signed-off-by: Fabrice Gasnier > Signed-off-by: Patrice Chotard Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP

Re: [U-Boot] [U-Boot, 1/2] ARM: dts: stm32: remove cd-inverted for stm32f769-disco

2018-08-13 Thread Tom Rini
On Mon, Aug 06, 2018 at 09:38:17AM +0200, Patrice Chotard wrote: > As cd-inverted property is no more used by arm_pl180_mmci driver, > remove it. Update cd-gpios active level accordingly. > > Reported-by: Tuomas Tynkkynen > Signed-off-by: Patrice Chotard Applied to u-boot/master, thanks! --

Re: [U-Boot] [U-Boot, v1, 3/3] db410: alter WLAN/BT MAC address fixup

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 04:25:37PM +0300, Ramon Fried wrote: > Change the way MAC address fixup is done: > 1. Stop using LK handed device-tree and calculate >the MAC address our own. > 2. Allow overriding the generated MACS with environment variables: >"wlanaddr" and "btaddr". > >

Re: [U-Boot] ARM: dts: stm32f4: Fix DT dtc warnings

2018-08-13 Thread Tom Rini
On Mon, Aug 06, 2018 at 11:25:42AM +0200, Patrice Chotard wrote: > From: Patrick Delaunay > > This patch fix the following warnings for for stm32f429 > evaluation and discovery boards: > > unnecessary #address-cells/#size-cells without "ranges" or > child "reg" property > > Signed-off-by:

Re: [U-Boot] [U-Boot, v1, 2/3] snapdragon: added MAC generation functions

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 04:25:36PM +0300, Ramon Fried wrote: > Add support for generation of unique MAC address > that is derived from board serial. > Algorithm for generation of MAC taken from LK. > > Signed-off-by: Ramon Fried Applied to u-boot/master, thanks! -- Tom signature.asc

Re: [U-Boot] [U-Boot, v1, 1/3] snapdragon: added msm_board_serial() func

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 04:25:35PM +0300, Ramon Fried wrote: > This commit adds a function to get the board > serial number. > In snapdragon it's actually the eMMC serial number. > > Function added in a new file misc.c that will > include further snapdragon miscellaneous functions. > >

Re: [U-Boot] [U-Boot, v1, 04/11] configs: stm32f4xx: Enable ICACHE and DCACHE

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 11:46:14AM +0200, Patrice Chotard wrote: > Enable instruction and data caches. > Fix boot_sd command as since commit d409c962169b ("armv7m: disable > icache before linux booting"), instruction cache is automatically > disable before linux booting. "icache off" from

Re: [U-Boot] [U-Boot,1/2] tpm: sandbox: fix wrong check on pcr_map

2018-08-13 Thread Tom Rini
On Sun, Aug 05, 2018 at 06:53:06PM +0200, Miquel Raynal wrote: > The second check on pcr_map in sandbox_tpm2_xfer() is wrong. It should > check for pcr_map not being empty. Instead, it is a pure copy/paste of > the first check which is redundant. > > This has been found thanks to a Coverity Scan

Re: [U-Boot] [U-Boot, v1, 06/11] configs: stm32h7xx: Migrate CONFIG_CMD_CACHE to defconfig

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 11:46:16AM +0200, Patrice Chotard wrote: > Remove CONFIG_CMD_CACHE from include/configs/stm32h7xx.h > and enable it in stm32h7xx_defconfig > > Signed-off-by: Patrice Chotard Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [U-Boot] MAINTAINERS: Add more sources to Arch Snapdragon

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 08:56:27PM +0300, Ramon Fried wrote: > Add scattered driver files around the source tree > that belongs to Snapdragon arch. Not sure why they > were not included in the first place. > > Signed-off-by: Ramon Fried Applied to u-boot/master, thanks! -- Tom

Re: [U-Boot] arm: bcm7445: Move config defines to bcm7445.h

2018-08-13 Thread Tom Rini
On Thu, Jul 26, 2018 at 11:02:47PM -0400, Thomas Fitzsimmons wrote: > Move some configuration #defines that do not apply to other bcmstb > boards from bcmstb.h to bcm7445.h. > > Signed-off-by: Thomas Fitzsimmons Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP

Re: [U-Boot] [U-Boot, 2/2] ARM: dts: stm32: remove cd-inverted for stm32f746-disco

2018-08-13 Thread Tom Rini
On Mon, Aug 06, 2018 at 09:38:18AM +0200, Patrice Chotard wrote: > As cd-inverted property is no more used by arm_pl180_mmci driver, > remove it. Update cd-gpios active level accordingly. > > Reported-by: Tuomas Tynkkynen > Signed-off-by: Patrice Chotard Applied to u-boot/master, thanks! --

Re: [U-Boot] [U-Boot, v1, 3/3] db410: alter WLAN/BT MAC address fixup

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 04:25:37PM +0300, Ramon Fried wrote: > Change the way MAC address fixup is done: > 1. Stop using LK handed device-tree and calculate >the MAC address our own. > 2. Allow overriding the generated MACS with environment variables: >"wlanaddr" and "btaddr". > >

Re: [U-Boot] [U-Boot, v1, 11/11] configs: stm32f429-evaluation: Add DISTRO_DEFAULT support

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 11:46:21AM +0200, Patrice Chotard wrote: > Add DISTRO_DEFAULT support to be able to boot on mmc > by default on boot. > > Signed-off-by: Patrice Chotard Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [U-Boot] [U-Boot, v1, 02/11] configs: stm32fxxx: Remove CONFIG_SYS_CLK_FREQ

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 11:46:12AM +0200, Patrice Chotard wrote: > Since commit aa5e3e22f4d6 ("board: stm32: switch to DM STM32 timer") > SYS_CLK_FREQ is useless, remove it from stm32f4 and stm32f7 boards. > > Signed-off-by: Patrice Chotard Applied to u-boot/master, thanks! -- Tom

Re: [U-Boot] ARM: omap3: evm: Enable CONFIG_BLK and misc. cleanup

2018-08-13 Thread Tom Rini
On Sun, Aug 05, 2018 at 11:51:31PM -0500, Derald D. Woods wrote: > This commit enables CONFIG_BLK and removes USB_STORAGE which is awaiting > proper implementation for current U-Boot interfaces. Additionally the > console selection is now handled by Kconfig and no longer needs to be in > the

Re: [U-Boot] db410c: add FIT support

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 04:31:02PM +0300, Ramon Fried wrote: > 1. Add FIT support for DB410c defconfig. > 2. Don't overwrite bootargs (they're already >defined in Linux device tree for DB410c. > > Signed-off-by: Ramon Fried Applied to u-boot/master, thanks! -- Tom signature.asc

Re: [U-Boot] [U-Boot, v1, 03/11] configs: stm32f429-disco: Remove CONFIG_SYS_RAM_CS

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 11:46:13AM +0200, Patrice Chotard wrote: > This flag is not used, remove it. > > Signed-off-by: Patrice Chotard Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature ___ U-Boot mailing list

Re: [U-Boot] [U-Boot, v1, 07/11] configs: stm32f4xx: Remove CONFIG_SYS_RAM_FREQ_DIV

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 11:46:17AM +0200, Patrice Chotard wrote: > Since commit bfea69ad2793 ("stm32f7: sdram: correct sdram > configuration as per micron sdram"), CONFIG_SYS_RAM_FREQ_DIV > flag is no more used, remove it. > > Signed-off-by: Patrice Chotard Applied to u-boot/master, thanks!

Re: [U-Boot] [U-Boot, v1, 05/11] configs: stm32f746-disco: Migrate CONFIG_CMD_CACHE to defconfig

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 11:46:15AM +0200, Patrice Chotard wrote: > Remove CONFIG_CMD_CACHE from include/configs/stm32f746-disco.h > and enable it in stm32f746-disco_defconfig > > Signed-off-by: Patrice Chotard Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP

Re: [U-Boot] MAINTAINERS: Update STM32MP fragments

2018-08-13 Thread Tom Rini
On Mon, Aug 06, 2018 at 11:52:23AM +0200, Patrice Chotard wrote: > Add new drivers > Add Christophe Kerello and myself as maintainers > > Signed-off-by: Patrice Chotard Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [U-Boot] [U-Boot, v1, 08/11] configs: stm32h743-evaluation: Add DISTRO_DEFAULT support

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 11:46:18AM +0200, Patrice Chotard wrote: > Add DISTRO_DEFAULT support to be able to boot on mmc > by default on boot. > > Signed-off-by: Patrice Chotard Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [U-Boot] [U-Boot,v2] stm32f7: board: Fix memory init

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 01:09:55PM +0200, Patrice Chotard wrote: > Commit 1473b12ad0b3 ("lib: fdtdec: Update ram_base to store ram start > adddress") brings regression on STM32F7 which can't boot. > > Use fdtdec_setup_mem_size_base() to setup memory base and size. > Use

Re: [U-Boot] [U-Boot, v1, 09/11] configs: stm32h743-discovery: Add DISTRO_DEFAULT support

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 11:46:19AM +0200, Patrice Chotard wrote: > Add DISTRO_DEFAULT support to be able to boot on > mmc by default on boot. > > Signed-off-by: Patrice Chotard Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [U-Boot] [U-Boot, 2/2] tpm: sandbox: fix wrong assignment with a simplification

2018-08-13 Thread Tom Rini
On Sun, Aug 05, 2018 at 06:53:07PM +0200, Miquel Raynal wrote: > The recv variable in sandbox_tpm2_fill_buf() is a pointer on a pointer > of a char array. It means accessing *recv is the char array pointer > itself while **recv is the first character of that array. There is no > need for such

Re: [U-Boot] arm: bcm7445: Fix parallel make race condition

2018-08-13 Thread Tom Rini
On Thu, Jul 26, 2018 at 10:55:37PM -0400, Thomas Fitzsimmons wrote: > Move the contents of prior_stage.h into bcmstb.h to prevent a build > failure when bcmstb.h is #include'ed before the asm/arch symbolic link > is present. > > Signed-off-by: Thomas Fitzsimmons Applied to u-boot/master,

Re: [U-Boot] [U-Boot, v1, 01/11] board: stm32: use bi_dram[0].start instead of hardcoded value

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 11:46:11AM +0200, Patrice Chotard wrote: > Use gd->bd->bi_dram[0].start initialized from DT instead of using > hardcoded CONFIG_SYS_SDRAM_BASE from config file. > > Remove unused CONFIG_SYS_RAM_BASE and CONFIG_SYS_SDRAM_BASE defines. > > Signed-off-by: Patrice Chotard

Re: [U-Boot] clk: at91: utmi: add timeout for utmi lock

2018-08-13 Thread Tom Rini
On Fri, Aug 03, 2018 at 12:10:49PM +0300, Eugen Hristev wrote: > In case the slow clock is not properly configured, the UTMI clock > cannot lock the PLL, because UPLLCOUNT will "wait X slow clock cycles". > In this case U-boot will loop indefinitely. > Added a timeout in this case, to start

Re: [U-Boot] [U-Boot,v2] rsa: Fix LibreSSL before v2.7.0

2018-08-13 Thread Tom Rini
On Wed, Jul 25, 2018 at 10:13:03PM -0400, nom...@palism.com wrote: > From: Caliph Nomble > > Fix LibreSSL compilation for versions before v2.7.0. > > Signed-off-by: Caliph Nomble > Reviewed-by: Jonathan Gray Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP

Re: [U-Boot] [U-Boot,1/1] fs: fix typo 'dumm'

2018-08-13 Thread Tom Rini
On Sat, Aug 11, 2018 at 03:52:14PM +0200, Heinrich Schuchardt wrote: > %s/dumm /dummy / > > Signed-off-by: Heinrich Schuchardt Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature ___ U-Boot mailing list

Re: [U-Boot] Please pull u-boot-fsl-qoriq master

2018-08-13 Thread Tom Rini
On Mon, Aug 13, 2018 at 04:19:02PM +, York Sun wrote: > Tom, > > The following changes since commit 373413cce6260acdf14de762f94010b627a77a3b: > > Merge branch 'master' of git://git.denx.de/u-boot-video (2018-08-07 > 07:15:20 -0400) > > are available in the git repository at: > >

Re: [U-Boot] [U-Boot,v3,1/1] avb2.0: add get_size_of_partition()

2018-08-13 Thread Tom Rini
On Fri, Aug 10, 2018 at 04:59:59PM +0300, Igor Opaniuk wrote: > Implement get_size_of_partition() operation, > which is required by the latest upstream libavb [1]. > > [1] > https://android.googlesource.com/platform/external/avb/+/android-p-preview-5 > > Signed-off-by: Igor Opaniuk >

Re: [U-Boot] [PATCH] spi: davinci: Full dm conversion

2018-08-13 Thread Adam Ford
On Mon, Aug 13, 2018 at 5:09 PM Adam Ford wrote: > > On Mon, Aug 13, 2018 at 1:29 PM Simon Goldschmidt > wrote: > > > > On Mon, Aug 13, 2018 at 3:46 PM Alex Kiernan wrote: > > > > > > On Mon, Aug 13, 2018 at 1:40 PM Adam Ford wrote: > > > > > > > > On Sat, Aug 11, 2018 at 3:09 PM Adam Ford

Re: [U-Boot] [PATCH] Kconfig: Migrate CONFIG_NR_DRAM_BANKS

2018-08-13 Thread Marek Vasut
On 08/14/2018 12:00 AM, Ramon Fried wrote: > Move CONFIG_NR_DRAM_BANKS from headers to Kconfig. > > Signed-off-by: Ramon Fried Somehow this seems to be missing a lot of cleanups in include/configs/ ... look at moveconfig.py > --- > Kconfig | 6 ++ > 1 file changed, 6 insertions(+) > >

Re: [U-Boot] [PATCH] spi: davinci: Full dm conversion

2018-08-13 Thread Adam Ford
On Mon, Aug 13, 2018 at 1:29 PM Simon Goldschmidt wrote: > > On Mon, Aug 13, 2018 at 3:46 PM Alex Kiernan wrote: > > > > On Mon, Aug 13, 2018 at 1:40 PM Adam Ford wrote: > > > > > > On Sat, Aug 11, 2018 at 3:09 PM Adam Ford wrote: > > > > > > > > > > > > > > > > On Sat, Aug 11, 2018, 1:24 PM

Re: [U-Boot] [PATCH v4 5/5] malloc_simple: calloc: don't call memset if malloc failed

2018-08-13 Thread Marek Vasut
On 08/13/2018 09:34 PM, Simon Goldschmidt wrote: > malloc_simple() can return 0 if out of memory. Don't call memset > from calloc() in this case but rely on the caller checking > the return value. > > Signed-off-by: Simon Goldschmidt > Reviewed-by: Marek Vasut Separate this from the patchset,

Re: [U-Boot] [PATCH v4 2/5] arm: socfpga: fix U-Boot running from fpga OnChip RAM

2018-08-13 Thread Marek Vasut
On 08/13/2018 09:34 PM, Simon Goldschmidt wrote: > gd->env_addr points to pre-relocation address even after > relocation. This leads to an abort in env_callback_init > when loading the environment. > > Fix this by enabling CONFIG_SYS_EXTRA_ENV_RELOC. > > Signed-off-by: Simon Goldschmidt > --- >

Re: [U-Boot] [PATCH v4 4/5] arm: socfpga: fix SPL booting from fpga OnChip RAM

2018-08-13 Thread Marek Vasut
On 08/13/2018 09:34 PM, Simon Goldschmidt wrote: > To boot from fpga OnChip RAM, some changes are required in SPL > to ensure the code is linked to the correct address (in contrast > to QSPI and MMC boot, FPGA boot executes SPL in place instead of > copying it to SRAM) and that fpga OnChip RAM

Re: [U-Boot] [PATCH] fdt_support: Use CONFIG_NR_DRAM_BANKS if necessary

2018-08-13 Thread Lukasz Majewski
Hi Ramon, > If CONFIG_NR_DRAM_BANKS is bigger than the default > value (4) define MEMORY_BANKS_MAX as CONFIG_NR_DRAM_BANKS. Also the Odroid XU3 uses 8 banks (but this is the fault of the board as it has 4x512MiB probably). I would also prefer to have the CONFIG_NR_DRAM_BANKS set to 4 (as it

[U-Boot] [PATCH v4 4/5] arm: socfpga: fix SPL booting from fpga OnChip RAM

2018-08-13 Thread Simon Goldschmidt
To boot from fpga OnChip RAM, some changes are required in SPL to ensure the code is linked to the correct address (in contrast to QSPI and MMC boot, FPGA boot executes SPL in place instead of copying it to SRAM) and that fpga OnChip RAM stays accessible while SPL runs (don't disable fpga

[U-Boot] [PATCH v4 5/5] malloc_simple: calloc: don't call memset if malloc failed

2018-08-13 Thread Simon Goldschmidt
malloc_simple() can return 0 if out of memory. Don't call memset from calloc() in this case but rely on the caller checking the return value. Signed-off-by: Simon Goldschmidt Reviewed-by: Marek Vasut --- Changes in v4: None Changes in v3: None Changes in v2: None common/malloc_simple.c | 3

[U-Boot] [PATCH v4 3/5] arm: socfpga: gen5: combine some init code for SPL and U-Boot

2018-08-13 Thread Simon Goldschmidt
Some of the code for low level system initialization in SPL's board_init_f() and U-Boot's arch_early_init_r() is the same, so let's combine it into a single function called from both. Signed-off-by: Simon Goldschmidt --- Changes in v4: - rename socfpga_init_bus_mapping() to

[U-Boot] [PATCH v4 2/5] arm: socfpga: fix U-Boot running from fpga OnChip RAM

2018-08-13 Thread Simon Goldschmidt
gd->env_addr points to pre-relocation address even after relocation. This leads to an abort in env_callback_init when loading the environment. Fix this by enabling CONFIG_SYS_EXTRA_ENV_RELOC. Signed-off-by: Simon Goldschmidt --- Changes in v4: enable this fix for all socfpga, not for gen5 only

[U-Boot] [PATCH v4 0/5] Get socfpga gen5 SPL working again.

2018-08-13 Thread Simon Goldschmidt
Socfpga gen5 SPL has been broken since moving to DM serial with v2018.07. Also, U-Boot console output has been broken since then. This series fixes this and makes some related small improvements. Changes in v4: - dropped already merged patches 1, 3 and 4 - enable the env relocatation for all

[U-Boot] [PATCH v4 1/5] arm: socfpga: fix device trees to work with DM serial

2018-08-13 Thread Simon Goldschmidt
Device trees need to have the serial console device available before relocation and require a stdout-path in chosen at least for SPL to have a console. Signed-off-by: Simon Goldschmidt --- Changes in v4: None Changes in v3: - moved uart0's "u-boot,dm-pre-reloc;" from socfpga.dtsi to board

Re: [U-Boot] [U-Boot,6/8] efi_loader: Pass file path to payload

2018-08-13 Thread Heinrich Schuchardt
On 04/11/2016 04:16 PM, Alexander Graf wrote: > The payload gets information on where it got loaded from. This includes > the device as well as file path. > > So far we've treated both as the same thing and always gave it the device > name. However, in some situations grub2 actually wants to find

[U-Boot] [PATCH] ARM: socfpga: Register the FPGA on A10 in SPL again

2018-08-13 Thread Marek Vasut
The restructuring of the SPL dropped registration of the FPGA in SPL, readd it. Signed-off-by: Marek Vasut Cc: Chin Liang See Cc: Dinh Nguyen Cc: Ley Foon Tan Fixes: c859f2a77d98 ("arm: socfpga: Restructure the SPL file") --- arch/arm/mach-socfpga/spl_a10.c | 6 ++ 1 file changed, 6

[U-Boot] [PATCH] ARM: socfpga: Enable DM ethernet on A10

2018-08-13 Thread Marek Vasut
Enable DM ethernet framework on Arria10, so that the designware GMAC can be probed from DT as it should be. Signed-off-by: Marek Vasut Cc: Chin Liang See Cc: Dinh Nguyen Cc: Ley Foon Tan --- configs/socfpga_arria10_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git

[U-Boot] [PATCH] Kconfig: Migrate CONFIG_NR_DRAM_BANKS

2018-08-13 Thread Ramon Fried
Move CONFIG_NR_DRAM_BANKS from headers to Kconfig. Signed-off-by: Ramon Fried --- Kconfig | 6 ++ 1 file changed, 6 insertions(+) diff --git a/Kconfig b/Kconfig index db0f545e45..d96e3373c1 100644 --- a/Kconfig +++ b/Kconfig @@ -104,6 +104,12 @@ config ENV_VARS_UBOOT_CONFIG -

[U-Boot] [PATCH 3/3] ARM: socfpga: Remove adhoc ethernet reset and configuration

2018-08-13 Thread Marek Vasut
Remove ad-hoc ethernet syscon registers configuration and reset support. Reset is now handled by the reset framework and the syscon registers are set in the dwmac_socfpga.c driver. Signed-off-by: Marek Vasut Cc: Chin Liang See Cc: Dinh Nguyen Cc: Ley Foon Tan --- arch/arm/Kconfig

[U-Boot] [PATCH 2/3] ARM: socfpga: Zap unused reset code

2018-08-13 Thread Marek Vasut
Remove code from the reset manager that is never called. Signed-off-by: Marek Vasut Cc: Chin Liang See Cc: Dinh Nguyen Cc: Ley Foon Tan --- .../include/mach/reset_manager_arria10.h | 3 - arch/arm/mach-socfpga/reset_manager_arria10.c | 123 - 2 files

[U-Boot] [PATCH] ARM: socfpga: Enable DM reset framework on A10

2018-08-13 Thread Marek Vasut
Enable the DM reset framework and DM reset driver on Arria10 both in U-Boot and in SPL. This lets U-Boot parse reset control from DT. Signed-off-by: Marek Vasut Cc: Chin Liang See Cc: Dinh Nguyen Cc: Ley Foon Tan --- arch/arm/mach-socfpga/Kconfig | 2 ++ 1 file changed, 2 insertions(+) diff

[U-Boot] [PATCH 5/5] ARM: dts: socfpga: Add i2c alias to A10 SoCDK

2018-08-13 Thread Marek Vasut
The A10 SoCDK is missing the I2C bus alias, so DM I2C cannot assign the I2C bus a bus number. Add the missing alias. Signed-off-by: Marek Vasut Cc: Chin Liang See Cc: Dinh Nguyen Cc: Ley Foon Tan --- arch/arm/dts/socfpga_arria10_socdk.dtsi | 1 + 1 file changed, 1 insertion(+) diff --git

[U-Boot] [PATCH] ARM: socfpga: Zap all the UART handling complexity

2018-08-13 Thread Marek Vasut
The UART reset handling is now done via reset framework using the SoCFPGA reset driver. The UART console assignment is done using the DM and console framework. Nuke all this comlexity, since it is just duplicating the same functionality, badly. Signed-off-by: Marek Vasut Cc: Chin Liang See Cc:

[U-Boot] [PATCH 1/3] net: designware: socfpga: Add Arria10 extras

2018-08-13 Thread Marek Vasut
Add wrapper around the designware MAC driver to handle the SoCFPGA specific configuration bits. On Arria10, this is configuration of syscon phy_intf. Signed-off-by: Marek Vasut Cc: Chin Liang See Cc: Dinh Nguyen Cc: Ley Foon Tan Cc: Joe Hershberger --- NOTE: This driver is not enabled on

[U-Boot] [PATCH] ARM: socfpga: Enable DM I2C framework on A10

2018-08-13 Thread Marek Vasut
Enable the DM I2C framework on Arria10, so that the DM capable Designware I2C driver can handle the reset via DM reset framework. Signed-off-by: Marek Vasut Cc: Chin Liang See Cc: Dinh Nguyen Cc: Ley Foon Tan --- arch/arm/mach-socfpga/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git

[U-Boot] [PATCH 4/5] ARM: dts: socfpga: Add missing I2C resets

2018-08-13 Thread Marek Vasut
The I2Cx resets are missing from DT, so the reset manager cannot control them. Add the missing DT reset entries. Signed-off-by: Marek Vasut Cc: Chin Liang See Cc: Dinh Nguyen Cc: Ley Foon Tan --- arch/arm/dts/socfpga_arria10.dtsi | 10 ++ 1 file changed, 10 insertions(+) diff --git

[U-Boot] [PATCH 3/5] ARM: dts: socfpga: Fix Arria10 GMAC resets

2018-08-13 Thread Marek Vasut
Add the GMAC0,1 OCP resets, which must also be ungated for those GMACs to work and add GMAC2 reset and OCP resets which were missing altogether. Signed-off-by: Marek Vasut Cc: Chin Liang See Cc: Dinh Nguyen Cc: Ley Foon Tan --- arch/arm/dts/socfpga_arria10.dtsi | 10 ++ 1 file

[U-Boot] [PATCH 2/5] ARM: dts: socfpga: Add missing UART resets

2018-08-13 Thread Marek Vasut
The UART0 and UART1 resets are missing from DT, so the reset manager cannot control them. Add the missing DT reset entries. Signed-off-by: Marek Vasut Cc: Chin Liang See Cc: Dinh Nguyen Cc: Ley Foon Tan --- arch/arm/dts/socfpga_arria10.dtsi | 2 ++ 1 file changed, 2 insertions(+) diff --git

[U-Boot] [PATCH 1/5] ARM: dts: socfpga: Flag reset manager on A10 as pre-reloc

2018-08-13 Thread Marek Vasut
The Altera reset manager block must be available very early on, since it controls ie. UART resets. Flag it as pre-reloc. Signed-off-by: Marek Vasut Cc: Chin Liang See Cc: Dinh Nguyen Cc: Ley Foon Tan --- arch/arm/dts/socfpga_arria10.dtsi | 2 ++ 1 file changed, 2 insertions(+) diff --git

[U-Boot] [PATCH] fdt_support: Use CONFIG_NR_DRAM_BANKS if necessary

2018-08-13 Thread Ramon Fried
If CONFIG_NR_DRAM_BANKS is bigger than the default value (4) define MEMORY_BANKS_MAX as CONFIG_NR_DRAM_BANKS. Fixes: 2a1f4f1758b5 ("Revert "fdt_support: Use CONFIG_NR_DRAM_BANKS if defined"") Signed-off-by: Ramon Fried --- common/fdt_support.c | 4 1 file changed, 4 insertions(+) diff

Re: [U-Boot] [PATCH] spi: davinci: Full dm conversion

2018-08-13 Thread Simon Goldschmidt
On Mon, Aug 13, 2018 at 3:46 PM Alex Kiernan wrote: > > On Mon, Aug 13, 2018 at 1:40 PM Adam Ford wrote: > > > > On Sat, Aug 11, 2018 at 3:09 PM Adam Ford wrote: > > > > > > > > > > > > On Sat, Aug 11, 2018, 1:24 PM Jagan Teki > > > wrote: > > >> > > >> On Sat, Aug 11, 2018 at 6:12 PM, Adam

[U-Boot] [PATCH] configs: sun7i: Fix to use emmc dts for OLinuXino_MICRO-eMMC

2018-08-13 Thread Jagan Teki
A20 OLinuXino Micro eMMC board has emmc with mmc2 slot so use proper dts, sun7i-a20-olinuxino-micro-emmc.dts Cc: Stefan Mavrodiev Cc: Hans de Goede Signed-off-by: Jagan Teki --- configs/A20-OLinuXino_MICRO-eMMC_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [U-Boot] [PATCH] fdt_support: Use VLA instead of MEMORY_BANKS_MAX

2018-08-13 Thread Ramon Fried
On August 13, 2018 7:59:05 PM GMT+03:00, Tom Rini wrote: >On Mon, Aug 13, 2018 at 10:55:03PM +0300, Ramon Fried wrote: >> On Mon, Aug 13, 2018 at 7:22 PM Ramon Fried >wrote: >> >> > On August 13, 2018 7:15:14 PM GMT+03:00, Tom Rini > >> > wrote: >> > >On Mon, Aug 13, 2018 at 07:14:00PM +0300,

Re: [U-Boot] [PATCH] pci: Support parsing PCI controller DT subnodes

2018-08-13 Thread Tom Rini
On Mon, Aug 13, 2018 at 06:07:14PM +0200, Marek Vasut wrote: > On 08/13/2018 03:39 PM, Tom Rini wrote: > [...] > > Next step is to upstream the DT > changes to Linux kernel, then sync the changes to U-Boot to satisfy > this obsession - using exactly the same DT as Linux. > >>> >

Re: [U-Boot] [PATCH] fdt_support: Use VLA instead of MEMORY_BANKS_MAX

2018-08-13 Thread Tom Rini
On Mon, Aug 13, 2018 at 10:55:03PM +0300, Ramon Fried wrote: > On Mon, Aug 13, 2018 at 7:22 PM Ramon Fried wrote: > > > On August 13, 2018 7:15:14 PM GMT+03:00, Tom Rini > > wrote: > > >On Mon, Aug 13, 2018 at 07:14:00PM +0300, Ramon Fried wrote: > > >> On August 13, 2018 7:08:22 PM GMT+03:00,

Re: [U-Boot] [PATCH] fdt_support: Use VLA instead of MEMORY_BANKS_MAX

2018-08-13 Thread Ramon Fried
On Mon, Aug 13, 2018 at 7:22 PM Ramon Fried wrote: > On August 13, 2018 7:15:14 PM GMT+03:00, Tom Rini > wrote: > >On Mon, Aug 13, 2018 at 07:14:00PM +0300, Ramon Fried wrote: > >> On August 13, 2018 7:08:22 PM GMT+03:00, Tom Rini > > wrote: > >> >On Mon, Aug 13, 2018 at 09:54:30PM +0300, Ramon

Re: [U-Boot] [PATCH] cmd: led.c: fix coding style

2018-08-13 Thread Tom Rini
On Sun, Aug 12, 2018 at 03:06:03PM +0800, Akee Huang wrote: > Fix coding style according to checkpatch.pl > > Signed-off-by: Akee Huang > --- > cmd/led.c | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/cmd/led.c b/cmd/led.c > index fc07ca95a3..b339c98dfb

Re: [U-Boot] [PATCH v4 1/2] armv8: layerscape: move ns_dev[] define from h to c file.

2018-08-13 Thread York Sun
On 08/10/2018 12:00 AM, Ran Wang wrote: > Since more c files will include ns_access.h, this move will fix some > compiling warnings and make it sense. > > Signed-off-by: Ran Wang > --- > Change in v4: > - Apply same move for ls102xa > > Change in v3: > - New file > This set is

Re: [U-Boot] [PATCH 1/1] mtd: nand: fsl_ifc: Fix handling of bitflips in erased pages

2018-08-13 Thread York Sun
On 08/02/2018 01:03 AM, Kurt Kanzenbach wrote: > From: Darwin Dingel > > This is a fix made for the fsl_ifc_nand driver on linux kernel by > Pavel Machek and is applied to uboot. It is currently on applied on > linux-mtd. > >

Re: [U-Boot] [PATCH v8 1/8] armv8: fsl-layerscape: add missing register blocks base address defines

2018-08-13 Thread York Sun
On 08/09/2018 05:20 AM, laurentiu.tu...@nxp.com wrote: > From: Laurentiu Tudor > > Add defines for the edma and qdma register block base addresses. > > Reviewed-by: Bharat Bhushan > Signed-off-by: Laurentiu Tudor > --- This set is applied to fsl-qoriq master, awaiting upstream. Thanks. York

Re: [U-Boot] [PATCH 2/2] net: Increase ethernet name string size to 20 chars

2018-08-13 Thread York Sun
On 08/01/2018 10:31 PM, Pankaj Bansal wrote: > The 16 char ethernet name size is inadequate to hold the name of ethernet > name "DPMAC17@rgmii-id", which is a valid name in LX2160AQDS/LX2160ARDB. > > Therefore, increase the name string size to 20 chars. > > Reported-by: Ioana Ciornei >

Re: [U-Boot] [PATCH 1/2] fsl/mc: Limit the ethernet name to ETH_NAME_LEN

2018-08-13 Thread York Sun
On 08/01/2018 10:31 PM, Pankaj Bansal wrote: > The ethernet name should be within the ETH_NAME_LEN, as this > is the buffer space allocated to ethernet name. > > Otherwise, this causes buffer overflow. > > Reported-by: Ioana Ciornei > Signed-off-by: Pankaj Bansal > --- Applied to fsl-qoriq

Re: [U-Boot] [PATCH] FSL PCI: Configure PCIe reference ratio

2018-08-13 Thread York Sun
On 09/12/2017 10:56 AM, Joakim Tjernlund wrote: > Most FSL PCIe controllers expects 333 MHz PCI reference clock. > This clock is derived from the CCB but in many cases the ref. > clock is not 333 MHz and a divisor needs to be configured. > > This adds PEX_CCB_DIV #define which can be defined for

Re: [U-Boot] [PATCH v2] Board: ls1088ardb: Enable PCIe config for Secure boot defconfigs

2018-08-13 Thread York Sun
On 07/16/2018 11:29 PM, Vinitha V Pillai wrote: > Enable PCIe config options in LS1088 SD Secure Boot defconfig > > Signed-off-by: Vinitha V Pillai > --- > > Changes in v2: > Changed the commit message and added reviewers Applied to fsl-qoriq master, awaiting upstream. Thanks. York

Re: [U-Boot] [PATCH] fdt_support: Use VLA instead of MEMORY_BANKS_MAX

2018-08-13 Thread Ramon Fried
On August 13, 2018 7:15:14 PM GMT+03:00, Tom Rini wrote: >On Mon, Aug 13, 2018 at 07:14:00PM +0300, Ramon Fried wrote: >> On August 13, 2018 7:08:22 PM GMT+03:00, Tom Rini > wrote: >> >On Mon, Aug 13, 2018 at 09:54:30PM +0300, Ramon Fried wrote: >> >> On Mon, Aug 13, 2018 at 5:52 PM Tom Rini

Re: [U-Boot] [PATCH] arm: ls102xa: Fix RGMII phy-connection-type FDT fixup

2018-08-13 Thread York Sun
On 07/16/2018 01:28 PM, Brendan Shanks wrote: > In ft_fixup_enet_phy_connect_type(), use strlen() instead of sizeof() on > the pointer result of phy_string_for_interface(). > sizeof() was returning the size of the pointer (4 bytes), resulting in > the phy-connection-type being set to "rgmi" rather

Re: [U-Boot] [PATCH] board: freescale: ls1012ardb: Add command to switch QSPI bank

2018-08-13 Thread York Sun
On 04/11/2018 11:40 PM, Jagdish Gediya wrote: > Add command "boot_bank X" to switch the boot bank to either > 1 or 2. > > Signed-off-by: Jagdish Gediya > --- Applied to fsl-qoriq master, awaiting upstream. Thanks. York ___ U-Boot mailing list

[U-Boot] Please pull u-boot-fsl-qoriq master

2018-08-13 Thread York Sun
Tom, The following changes since commit 373413cce6260acdf14de762f94010b627a77a3b: Merge branch 'master' of git://git.denx.de/u-boot-video (2018-08-07 07:15:20 -0400) are available in the git repository at: git://git.denx.de/u-boot-fsl-qoriq.git for you to fetch changes up to

Re: [U-Boot] [PATCH] fdt_support: Use VLA instead of MEMORY_BANKS_MAX

2018-08-13 Thread Tom Rini
On Mon, Aug 13, 2018 at 07:14:00PM +0300, Ramon Fried wrote: > On August 13, 2018 7:08:22 PM GMT+03:00, Tom Rini wrote: > >On Mon, Aug 13, 2018 at 09:54:30PM +0300, Ramon Fried wrote: > >> On Mon, Aug 13, 2018 at 5:52 PM Tom Rini wrote: > >> > >> > On Mon, Aug 13, 2018 at 08:20:03AM +0100,

Re: [U-Boot] [PATCH] fdt_support: Use VLA instead of MEMORY_BANKS_MAX

2018-08-13 Thread Ramon Fried
On August 13, 2018 7:08:22 PM GMT+03:00, Tom Rini wrote: >On Mon, Aug 13, 2018 at 09:54:30PM +0300, Ramon Fried wrote: >> On Mon, Aug 13, 2018 at 5:52 PM Tom Rini wrote: >> >> > On Mon, Aug 13, 2018 at 08:20:03AM +0100, Peter Robinson wrote: >> > >> > > On Sun, Aug 12, 2018 at 9:37 PM, Ramon

Re: [U-Boot] [PATCH] fdt_support: Use VLA instead of MEMORY_BANKS_MAX

2018-08-13 Thread Tom Rini
On Mon, Aug 13, 2018 at 09:54:30PM +0300, Ramon Fried wrote: > On Mon, Aug 13, 2018 at 5:52 PM Tom Rini wrote: > > > On Mon, Aug 13, 2018 at 08:20:03AM +0100, Peter Robinson wrote: > > > > > On Sun, Aug 12, 2018 at 9:37 PM, Ramon Fried > > wrote: > > > > From: Ramon Fried > > > > > > > >

  1   2   >