[U-Boot] [PATCH v2 4/5] riscv: sifive: fu540: Enable SiFive SPI and MMC SPI drivers

2019-06-28 Thread Anup Patel
From: Bhargav Shah This patch enables SiFive SPI and MMC SPI drivers for the SiFive Unleashed board. Signed-off-by: Bhargav Shah Signed-off-by: Anup Patel --- board/sifive/fu540/Kconfig | 8 1 file changed, 8 insertions(+) diff --git a/board/sifive/fu540/Kconfig b/board/sifive/fu540

[U-Boot] [PATCH v2 3/5] mmc: mmc_spi: Re-write driver using DM framework

2019-06-28 Thread Anup Patel
From: Bhargav Shah This patch rewrites MMC SPI driver using U-Boot DM framework and get it's working on SiFive Unleashed board. Signed-off-by: Bhargav Shah Signed-off-by: Anup Patel --- drivers/mmc/Kconfig | 18 ++ drivers/mmc/mmc_spi.c | 469 +++--- 2 f

[U-Boot] [PATCH v2 1/5] spi: Add SiFive SPI driver

2019-06-28 Thread Anup Patel
From: Bhargav Shah This patch adds SiFive SPI driver. The driver is 100% DM driver and it determines input clock using clk framework. The SiFive SPI block is found on SiFive FU540 SOC and is used to access flash and MMC devices on SiFive Unleashed board. This driver implementation is inspired f

[U-Boot] [PATCH v2 2/5] mmc: skip select_mode_and_width for MMC SPI host

2019-06-28 Thread Anup Patel
The MMC mode and width are fixed for MMC SPI host hence we skip sd_select_mode_and_width() and mmc_select_mode_and_width() for MMC SPI host. Signed-off-by: Anup Patel --- drivers/mmc/mmc.c | 14 ++ 1 file changed, 14 insertions(+) diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c i

[U-Boot] [PATCH v2 0/5] SiFive SPI MMC Support

2019-06-28 Thread Anup Patel
This patchset adds: 1. SiFive SPI driver 2. New MMC SPI driver based on DM_MMC and DM_SPI 3. Enables SiFive SPI driver and MMC SPI driver for SiFive Unleashed board With this patch series, we can now load files from SD card on SiFive Unleashed board. Many thanks to Bhargav for porting SiFive SPI d

[U-Boot] [PATCH v2 1/3] board_r: move initr_watchdog to be called, after initr_serial

2019-06-28 Thread Suniel Mahesh
Hi, May I know if this patch is still under review ? I have converted OMAP3 watchdog driver into driver model for TI AM33XX based SOC and tested corresponding patches on BeagleBone Black board. Build was successful but board goes into a loop when turned on/runtime, problem identified and is in

Re: [U-Boot] [PATCH 3/4] mmc: mmc_spi: Re-write driver using DM framework

2019-06-28 Thread Anup Patel
On Fri, Jun 28, 2019 at 2:08 PM Anup Patel wrote: > > From: Bhargav Shah > > This patch rewrites MMC SPI driver using U-Boot DM > framework and get it's working on SiFive Unleashed > board. > > Signed-off-by: Bhargav Shah > Signed-off-by: Anup Patel > --- > drivers/mmc/Kconfig | 18 ++ > dr

Re: [U-Boot] [PATCH 0/4] SiFive SPI MMC Support

2019-06-28 Thread Anup Patel
On Sat, Jun 29, 2019 at 6:01 AM Atish Patra wrote: > > On Fri, 2019-06-28 at 08:38 +, Anup Patel wrote: > > This patchset adds: > > 1. SiFive SPI driver > > 2. New MMC SPI driver based on DM_MMC and DM_SPI > > 3. Enables SiFive SPI driver and MMC SPI driver for SiFive Unleashed > > board > > >

[U-Boot] [PATCH 1/1] efi_loader: correct signature of ConvertPointer()

2019-06-28 Thread Heinrich Schuchardt
ConvertPointer() must be EFIAPI. The first parameter should be of type efi_uint_t. Use the same parameter name as the UEFI specification. Signed-off-by: Heinrich Schuchardt --- include/efi_api.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/include/efi_api.h b/include/efi

Re: [U-Boot] rpi2: u-boot misaligned operation

2019-06-28 Thread Alexander von Gluck IV
June 28, 2019 8:49 PM, "Alexander von Gluck IV" wrote: > June 26, 2019 3:22 AM, "Matthias Brugger" wrote: > >> On 25/06/2019 02:42, Alexander von Gluck IV wrote: >> >>> Was rebuilding u-boot binary for the Raspberry Pi 2 (from master), and ran >>> into the following >>> issue: >>> " >>> U-Boot

Re: [U-Boot] rpi2: u-boot misaligned operation

2019-06-28 Thread Alexander von Gluck IV
June 26, 2019 3:22 AM, "Matthias Brugger" wrote: > On 25/06/2019 02:42, Alexander von Gluck IV wrote: > >> Was rebuilding u-boot binary for the Raspberry Pi 2 (from master), and ran >> into the following >> issue: >> " >> U-Boot 2019.07-rc4-00179-g77f6e2dd05 (Jun 24 2019 - 14:25:14 -0500) >> >>

Re: [U-Boot] [PATCH 1/3] board_r: move initr_serial to be called before initr_watchdog

2019-06-28 Thread Suniel Mahesh
Hi, Is this patch under review ? facing the same problem on TI Based AM33XX platform. -- Sent from: http://u-boot.10912.n7.nabble.com/ ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 0/4] SiFive SPI MMC Support

2019-06-28 Thread Atish Patra
On Fri, 2019-06-28 at 08:38 +, Anup Patel wrote: > This patchset adds: > 1. SiFive SPI driver > 2. New MMC SPI driver based on DM_MMC and DM_SPI > 3. Enables SiFive SPI driver and MMC SPI driver for SiFive Unleashed > board > > With this patch series, we can now load files from SD card on SiFi

Re: [U-Boot] Pull request: u-boot-rockchip/tags/rockchip-for-v2019.07-rc5

2019-06-28 Thread Tom Rini
On Thu, Jun 27, 2019 at 08:46:39AM +0800, Kever Yang wrote: > Hi Tom, > > Please pull some rockchip updates: > - new board: rk3328 rock64 > - rk3328 SPL support > - rk3399 spl/tpl board init cleanup > - use environment to get ATF binary for rk3399 > - build u-boot.itb target by default for rockch

Re: [U-Boot] [GIT PULL] Pull request: u-boot-imx u -boot-imx-20190628

2019-06-28 Thread Tom Rini
4: > > Merge tag 'efi-2019-07-rc5-2' of git://git.denx.de/u-boot-efi > (2019-06-15 13:03:00 -0400) > > are available in the Git repository at: > > git://www.denx.de/git/u-boot-imx.git u-boot-imx-20190628 The GitLab URI works too ;) > > for you to

Re: [U-Boot] [PULL] u-boot-stm32 for v2019.07​ (round 7)

2019-06-28 Thread Tom Rini
in the Git repository at: > >   https://gitlab.denx.de/u-boot/custodians/u-boot-stm > tags/u-boot-stm32-20190628 > > for you to fetch changes up to 36cb793b35333e9a281445313de6db1ca4a6ab3a: > >   mach-stm32: Fix MPU region size dedicated to SDRAM for STM32F4 (2019-06-28 &

Re: [U-Boot] [PATCH v6 1/3] i.MX6: nand: add nandbcb command for imx

2019-06-28 Thread Parthiban Nallathambi
Hello Shyam, On 6/18/19 9:54 AM, Shyam Saini wrote: > Hi Igor, > >> Hi Shyam, >> >> On Fri, Jun 14, 2019 at 10:37 AM Shyam Saini >> wrote: >>> >>> Writing/updating boot image in nand device is not >>> straight forward in i.MX6 platform and it requires >>> boot control block(BCB) to be configured

[U-Boot] [PATCH 1/1] efi_loader: bump UEFI specification number to 2.8

2019-06-28 Thread Heinrich Schuchardt
We are implementing UEFI variable RuntimeServicesSupported and set the unimplemented runtime functions return EFI_UNSUPPORTED as described in UEFI specification 2.8. So let's also advertise this specification version in our system table. Signed-off-by: Heinrich Schuchardt --- include/efi_api.h |

Re: [U-Boot] [PATCH 1/1] imx6: wandboard: allow booting from MMC 2

2019-06-28 Thread Simon Glass
Hi Heinrich On Fri, 28 Jun 2019 at 09:23, Heinrich Schuchardt wrote: > > On 6/28/19 3:55 PM, Simon Glass wrote: > > On Sun, 23 Jun 2019 at 07:33, Fabio Estevam wrote: > >> > >> Hi Heinrich, > >> > >> On Sun, Jun 23, 2019 at 4:55 AM Heinrich Schuchardt > >> wrote: > >>> > >>> One of the SD-CARD

[U-Boot] [PATCH 2/6] ARM: uniphier: remove unused SC_DPLLOSCCTRL

2019-06-28 Thread Masahiro Yamada
Signed-off-by: Masahiro Yamada --- arch/arm/mach-uniphier/sc-regs.h | 4 1 file changed, 4 deletions(-) diff --git a/arch/arm/mach-uniphier/sc-regs.h b/arch/arm/mach-uniphier/sc-regs.h index b105335f69..28de19c039 100644 --- a/arch/arm/mach-uniphier/sc-regs.h +++ b/arch/arm/mach-uniphier/s

[U-Boot] [PATCH 4/6] ARM: uniphier: include from dram_init.c

2019-06-28 Thread Masahiro Yamada
This file calls readl(), so needs to include . Currently, it relies on someone else including it. Signed-off-by: Masahiro Yamada --- arch/arm/mach-uniphier/dram_init.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/mach-uniphier/dram_init.c b/arch/arm/mach-uniphier/dram_init.c in

[U-Boot] [PATCH 1/6] ARM: uniphier: fix build error for CONFIG_DEBUG_LL=y

2019-06-28 Thread Masahiro Yamada
Commit e27d6c7d328c ("ARM: uniphier: simplify SoC ID get function") accidentally removed the macros needed to compile debug_ll.S Revive them. Fixes: e27d6c7d328c ("ARM: uniphier: simplify SoC ID get function") Signed-off-by: Masahiro Yamada --- arch/arm/mach-uniphier/arm32/debug_ll.S | 2 ++ 1

[U-Boot] [PATCH 6/6] ARM: uniphier: move sg_set_{pinsel, iectrl} to more relevant places

2019-06-28 Thread Masahiro Yamada
Move the sg_set_pinsel macro to arch/arm/mach-uniphier/arm32/debug_ll.S since it is not used anywhere else. Move the C functions sg_set_{pinsel,iectrl} to debug-uart.c since they are not used anywhere else. Signed-off-by: Masahiro Yamada --- arch/arm/mach-uniphier/arm32/debug_ll.S | 8 +

[U-Boot] [PATCH 5/6] ARM: uniphier: remove unused init code for CONFIG_DEBUG_UART

2019-06-28 Thread Masahiro Yamada
debug_uart_init() is called from spl_board_init(), which is only compiled for SPL. For U-boot proper, _debug_uart_init() is unreachable, so dropped by the dead code elimination. Now that 64-bit SoCs of this SoC family no longer support SPL, debug-uart-ld20.c is never compiled. Signed-off-by: Masa

[U-Boot] [PATCH 3/6] ARM: uniphier: remove unused sg_set_iectrl_range()

2019-06-28 Thread Masahiro Yamada
Signed-off-by: Masahiro Yamada --- arch/arm/mach-uniphier/sg-regs.h | 8 1 file changed, 8 deletions(-) diff --git a/arch/arm/mach-uniphier/sg-regs.h b/arch/arm/mach-uniphier/sg-regs.h index 0497655fb5..8aed826c96 100644 --- a/arch/arm/mach-uniphier/sg-regs.h +++ b/arch/arm/mach-uniphi

[U-Boot] Watchdog DM - ENOMEM error identified - need suggestions

2019-06-28 Thread Suniel Mahesh
Looks like a detailed explanation of the problem and a patch has already been submitted. http://u-boot.10912.n7.nabble.com/PATCH-1-3-board-r-move-initr-serial-to-be-called-before-initr-watchdog-tc369705.html Regards -- Suniel Mahesh Embedded Linux, Kernel & U-Boot engineer https://github.com/su

Re: [U-Boot] [PATCH 1/1] imx6: wandboard: allow booting from MMC 2

2019-06-28 Thread Heinrich Schuchardt
On 6/28/19 3:55 PM, Simon Glass wrote: > On Sun, 23 Jun 2019 at 07:33, Fabio Estevam wrote: >> >> Hi Heinrich, >> >> On Sun, Jun 23, 2019 at 4:55 AM Heinrich Schuchardt >> wrote: >>> >>> One of the SD-CARD slots on the Wandboard Quad B1 is MMC 2. Enable it as a >>> boot device. >>> >>> Signed-of

Re: [U-Boot] [PATCH v1 06/11] mmc: if possible, poll the busy state using DAT0

2019-06-28 Thread Marek Vasut
On 6/28/19 4:26 PM, Jean-Jacques Hiblot wrote: > > On 28/06/2019 15:38, Marek Vasut wrote: >> On 6/28/19 10:28 AM, Jean-Jacques Hiblot wrote: >>> On 28/06/2019 02:07, Marek Vasut wrote: On 6/27/19 12:31 PM, Jean-Jacques Hiblot wrote: > Using the DAT0 line as a rdy/busy line is an alternat

[U-Boot] [RESEND PATCH v2] usb: gadget: f_dfu.c: fix memory leak

2019-06-28 Thread Ralph Siemsen
dfu_prepare_function() allocates N+1 descriptor header structures, the last one being the "DFU Functional Descriptor". dfu_unbind() handles de-allocation, but fails to free the final one (eg. "DFU Functional Descriptor"), leading to memory leak. Fixed by incrementing counter, as in dfu_prepare_fu

Re: [U-Boot] [PATCH v1 06/11] mmc: if possible, poll the busy state using DAT0

2019-06-28 Thread Jean-Jacques Hiblot
On 28/06/2019 15:38, Marek Vasut wrote: On 6/28/19 10:28 AM, Jean-Jacques Hiblot wrote: On 28/06/2019 02:07, Marek Vasut wrote: On 6/27/19 12:31 PM, Jean-Jacques Hiblot wrote: Using the DAT0 line as a rdy/busy line is an alternative to reading the status register of the card. It especially us

[U-Boot] Watchdog DM - ENOMEM error identified - need suggestions

2019-06-28 Thread Suniel Mahesh
Hi, I converted watchdog driver(omap3_wdt) on TI AM33XX platform to Driver Model. I am getting the following error when tested on BeagleBone Black: U-Boot 2019.07-rc4-00179-g77f6e2d-dirty (Jun 27 2019 - 15:11:15 +0530) CPU : AM335X-GP rev 2.1 Model: TI AM335x BeagleBone Black DRAM: 512 MiB WD

[U-Boot] [PATCH 24/37] binman: Drop unnecessary debug handling

2019-06-28 Thread Simon Glass
The -D option enables debug mode, but we only need to add -D to the command line once. Drop the duplicate code. Also drop the comment about enabling debugging since this can be done with -D. Signed-off-by: Simon Glass --- tools/binman/binman.py | 5 - 1 file changed, 5 deletions(-) diff -

[U-Boot] [PATCH 37/37] binman: Add support for fixed-offset files in CBFS

2019-06-28 Thread Simon Glass
A feature of CBFS is that it allows files to be positioned at particular offset (as with binman in general). This is useful to support execute-in-place (XIP) code, since this may not be relocatable. Add a new cbfs-offset property to control this. Signed-off-by: Simon Glass --- tools/binman/REA

[U-Boot] [PATCH 26/37] binman: Correct comment in u_boot_spl_elf

2019-06-28 Thread Simon Glass
This comment mentions the wrong default filename. Fix it. Signed-off-by: Simon Glass --- tools/binman/README.entries | 2 +- tools/binman/etype/u_boot_spl_elf.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/binman/README.entries b/tools/binman/README.ent

[U-Boot] [PATCH 30/37] binman: Allow preserving test directories

2019-06-28 Thread Simon Glass
Sometimes when debugging tests it is useful to keep the input and output directories so they can be examined later. Add an option for this and update the binman tests to support it. This affects both the test class and the tearDown() function called after each test. Signed-off-by: Simon Glass ---

[U-Boot] [PATCH 29/37] binman: Allow verbosity control when running tests

2019-06-28 Thread Simon Glass
At present the -v flag is ignored with tests, so that (for example) -v2 does not have any effect. Update binman to pass this flag through to tests so that they work just like running binman normally, except in a few special cases where we are actually testing behaviour with different levels of verb

[U-Boot] [PATCH 36/37] binman: Pad empty areas of the CBFS with files

2019-06-28 Thread Simon Glass
When there is lots of open space in a CBFS it is normally padded with 'empty' files so that sequentially scanning the CBFS can skip from one to the next without a break. Add support for this. Signed-off-by: Simon Glass --- tools/binman/cbfs_util.py | 68 --

[U-Boot] [PATCH 15/37] binman: Add a function to create a sample ELF file

2019-06-28 Thread Simon Glass
It is useful to create an ELF file for testing purposes, with just the right attributes used by the test. Add a function to handle this, along with a test that it works correctly. Signed-off-by: Simon Glass --- tools/binman/elf.py | 98 tools/binman

[U-Boot] [PATCH 33/37] binman: Add a utility library for coreboot CBFS

2019-06-28 Thread Simon Glass
Coreboot uses a simple flash-based filesystem called Coreboot Filesystem (CBFS) to organise files used during boot. This allows files to be named and their position in the flash to be set. It has special features for dealing with x86 devices which typically memory-map their SPI flash to the top of

[U-Boot] [PATCH 11/37] binman: Add missing comments toentry

2019-06-28 Thread Simon Glass
At present GetOffsets() lacks a function comment. Add one. Signed-off-by: Simon Glass --- tools/binman/entry.py | 15 +++ 1 file changed, 15 insertions(+) diff --git a/tools/binman/entry.py b/tools/binman/entry.py index d842d89dd66..e8d0adec1e9 100644 --- a/tools/binman/entry.py ++

[U-Boot] [PATCH 35/37] binman: Add support for Intel IFWI entries

2019-06-28 Thread Simon Glass
An Integrated Firmware Image is used to hold various binaries used for booting with Apollolake and some later devices. Add support for this. Signed-off-by: Simon Glass --- tools/binman/etype/intel_descriptor.py| 6 +- tools/binman/etype/intel_ifwi.py | 100 +++

[U-Boot] [PATCH 27/37] binman: Support ELF files for TPL

2019-06-28 Thread Simon Glass
We currenty support using the ELF file in U-Boot proper and SPL, but not TPL. Add this as it is useful both with sandbox and for CBFS to allow adding TPL as a 'stage'. Signed-off-by: Simon Glass --- tools/binman/README.entries | 11 +++ tools/binman/etype/u_boot_tpl_elf.py | 24

[U-Boot] [PATCH 16/37] binman: Add a function to decode an ELF file

2019-06-28 Thread Simon Glass
Add a function which decodes an ELF file, working out where in memory each part of the data should be written. Signed-off-by: Simon Glass --- tools/binman/README | 4 +++ tools/binman/elf.py | 76 tools/binman/elf_test.py | 21 +++ 3 f

[U-Boot] [PATCH 22/37] patman: Add functions to compress and decompress data

2019-06-28 Thread Simon Glass
Add utility functions to compress and decompress using lz4 and lzma algorithms. In the latter case these use the legacy lzma support favoured by coreboot's CBFS. No tests are provided as these functions will be tested by the CBFS tests in a separate patch. Signed-off-by: Simon Glass --- tools/

[U-Boot] [PATCH 34/37] binman: Add support for CBFS entries

2019-06-28 Thread Simon Glass
Add support for putting CBFSs (Coreboot Filesystems) in an image. This allows binman to produce firmware images used by coreboot to boot. Signed-off-by: Simon Glass --- tools/binman/README.entries | 142 ++ tools/binman/control.py | 2 + tools/b

[U-Boot] [PATCH 32/37] patman: Add a function to write ifwitool

2019-06-28 Thread Simon Glass
This tool has quite a few arguments and options, so put the functionality in a function so that we call it from one place and hopefully get it right. Signed-off-by: Simon Glass --- tools/patman/tools.py | 33 + 1 file changed, 33 insertions(+) diff --git a/tools

[U-Boot] [PATCH 25/37] binman: Use tools compression function for blob handling

2019-06-28 Thread Simon Glass
Avoid duplicate code here by using the new compression function in the tools module. Signed-off-by: Simon Glass --- tools/binman/etype/blob.py | 16 1 file changed, 4 insertions(+), 12 deletions(-) diff --git a/tools/binman/etype/blob.py b/tools/binman/etype/blob.py index f56a

[U-Boot] [PATCH 09/37] binman: Add a --toolpath option to set the tool search path

2019-06-28 Thread Simon Glass
Sometimes tools used by binman may not be in the normal PATH search path, such as when the tool is built by the U-Boot build itself (e.g. mkimage). Provide a way to specify an additional search path for tools. The flag can be used multiple times. Update the help to describe this option. Signed-of

[U-Boot] [PATCH 08/37] patman: Add a way to set the search path for tools

2019-06-28 Thread Simon Glass
Sometimes tools can be located by looking in other locations. Add a way to direct the search. Signed-off-by: Simon Glass --- tools/patman/tools.py | 42 +- 1 file changed, 37 insertions(+), 5 deletions(-) diff --git a/tools/patman/tools.py b/tools/patman

[U-Boot] [PATCH 06/37] binman: Correct two typos in function names in ftest

2019-06-28 Thread Simon Glass
Two functions have incorrect names. Fix them. Signed-off-by: Simon Glass --- tools/binman/ftest.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/binman/ftest.py b/tools/binman/ftest.py index cc57ef3e04a..46f669e73b4 100644 --- a/tools/binman/ftest.py +++ b/tools/

[U-Boot] [PATCH 23/37] binman: Use the tools.Decompress method

2019-06-28 Thread Simon Glass
Update the compression test to use the tools module to decompress the output data. Signed-off-by: Simon Glass --- tools/binman/ftest.py | 13 ++--- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/tools/binman/ftest.py b/tools/binman/ftest.py index c74e12d13c8..6ff871b3c16

[U-Boot] [PATCH 28/37] binman: Fix up the _DoTestFile() function -u argument

2019-06-28 Thread Simon Glass
This should be -u, not -up, since we don't need to preserve the output directory in this case. Signed-off-by: Simon Glass --- tools/binman/ftest.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/binman/ftest.py b/tools/binman/ftest.py index 9cec5f42fa3..b1780854cfe 10

[U-Boot] [PATCH 31/37] binman: Pass the toolpath to tests

2019-06-28 Thread Simon Glass
Tools like ifwitool may not be available in the PATH, but are available in the build. These tools may be needed by tests, so allow tests to use the --toolpath flag. Also use this flag with travis. Signed-off-by: Simon Glass --- .travis.yml| 2 +- test/run | 9 ++

[U-Boot] [PATCH 05/37] binman: Fix comment in bsection.GetEntries()

2019-06-28 Thread Simon Glass
This comment is out of date as it does not correctly describe the return value. Fix it. Signed-off-by: Simon Glass --- tools/binman/bsection.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/binman/bsection.py b/tools/binman/bsection.py index 03dfa2f805c..a3026718

[U-Boot] [PATCH 21/37] binman: Allow text directly in the node

2019-06-28 Thread Simon Glass
At present text entries use an indirect method to specify the text to use, with a label pointing to the text itself. Allow the text to be directly written into the node. This is more convenient in cases where the text is constant. Signed-off-by: Simon Glass --- tools/binman/README.entries|

[U-Boot] [PATCH 10/37] binman: Add missing comments to bsection

2019-06-28 Thread Simon Glass
Some functions lack comments in this file. Add comments to cover this functionality. Signed-off-by: Simon Glass --- tools/binman/bsection.py | 23 +++ 1 file changed, 23 insertions(+) diff --git a/tools/binman/bsection.py b/tools/binman/bsection.py index a3026718794..e0e370

[U-Boot] [PATCH 04/37] tools: Drop duplicate raise_on_error argument

2019-06-28 Thread Simon Glass
If kwargs contains raise_on_error then this function generates an error due to a duplicate argument. Fix this. Signed-off-by: Simon Glass --- tools/patman/command.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/patman/command.py b/tools/patman/command.py index 1

[U-Boot] [PATCH 12/37] binman: Tidy up help for --indir

2019-06-28 Thread Simon Glass
The current help is confusing. Adjust it to indicate what the flag actually does. Signed-off-by: Simon Glass --- tools/binman/cmdline.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/binman/cmdline.py b/tools/binman/cmdline.py index ee19c5e33fe..39b835666ea 100644 --

[U-Boot] [PATCH 07/37] binman: Add coverage tools info for Python 3

2019-06-28 Thread Simon Glass
Test coverage with Python 3 requires a new package. Add details about this. Signed-off-by: Simon Glass --- tools/binman/README | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/binman/README b/tools/binman/README index ac193f16cf7..decca47bbf3 100644 --- a/tools/binman/R

[U-Boot] [PATCH 13/37] binman: Use a better error for missing Intel descriptor

2019-06-28 Thread Simon Glass
FD is a bit confusing so write this out in full. Also avoid splitting the string so that people can grep for the error message more easily. Signed-off-by: Simon Glass --- tools/binman/etype/intel_descriptor.py | 2 +- tools/binman/ftest.py | 4 ++-- 2 files changed, 3 insertion

[U-Boot] [PATCH 02/37] cbfs: Add an enum and comment for the magic number

2019-06-28 Thread Simon Glass
This field is not commented in the original file. Add a comment. Signed-off-by: Simon Glass --- include/cbfs.h | 11 +++ 1 file changed, 11 insertions(+) diff --git a/include/cbfs.h b/include/cbfs.h index bd1bf75bbfc..f2ede25f517 100644 --- a/include/cbfs.h +++ b/include/cbfs.h @@ -40,

[U-Boot] [PATCH 03/37] cbfs: Rename checksum to attributes_offset

2019-06-28 Thread Simon Glass
It seems that this field has been renamed in later version of coreboot. Update it. Signed-off-by: Simon Glass --- fs/cbfs/cbfs.c | 4 ++-- include/cbfs.h | 5 +++-- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/fs/cbfs/cbfs.c b/fs/cbfs/cbfs.c index 7b2513cb24b..af4d3c5e564 1006

[U-Boot] [PATCH 18/37] binman: Assume Intel descriptor is at the start of the image

2019-06-28 Thread Simon Glass
At present binman requires that the Intel descriptor has an explicit offset. Generally this is 0 since the descriptor is at the start of the image. Add a default to handle this, so users don't need to specify the offset. Signed-off-by: Simon Glass --- tools/binman/bsection.py | 8

[U-Boot] [PATCH 19/37] binman: Don't assume there is an ME region

2019-06-28 Thread Simon Glass
At present having a descriptor means that there is an ME (Intel Management Engine) entry as well. The descriptor provides the ME location and assumes that it is present. For some SoCs this is not true. Before providing the location of a potentially non-existent entry, check if it is present. Upda

[U-Boot] [PATCH 17/37] binman: Ensure that coverage has access to site packages

2019-06-28 Thread Simon Glass
Code coverage tests fail on binman due to dist-packages being dropped from the python path on Ubuntu 16.04. Add them in so that we can find the elffile module, which is required by binman. Signed-off-by: Simon Glass --- tools/binman/binman.py | 8 1 file changed, 8 insertions(+) diff

[U-Boot] [PATCH 01/37] x86: Add ifwitool for Intel Integrated Firmware Image

2019-06-28 Thread Simon Glass
Some Intel SoCs from about 2016 boot using an internal microcontroller via an 'IFWI' image. This is a special format which can hold firmware images. In U-Boot's case it holds u-boot-tpl.bin. Add this tool, taken from coreboot, so that we can build bootable images on apollolake SoCs. This tool its

[U-Boot] [PATCH 14/37] binman: Detect skipped tests

2019-06-28 Thread Simon Glass
If tests are skipped we should ideally exit with an error, since there may be a missing dependency. However at present this is not desirable since it breaks travis tests. For now, just report the skips. Signed-off-by: Simon Glass --- tools/binman/binman.py | 9 +++-- 1 file changed, 7 inser

[U-Boot] [PATCH 20/37] binman: Update entry.SetOffsetSize to be optional

2019-06-28 Thread Simon Glass
At present this function always sets both the offset and the size of entries. But in some cases we want to set only one or the other, for example with the forthcoming ifwi entry, where we only set the offset. Update the function to handle this. Signed-off-by: Simon Glass --- tools/binman/bsecti

[U-Boot] [PATCH 00/37] binman: Add CBFS support

2019-06-28 Thread Simon Glass
CBFS (Coreboot Filesystem) is a simple ROM-based filesystem used for locating data needed during booting. It has particular features for x86 such as defining a boot block at the top of the ROM image. At run-time, boot loaders can locate files in the ROM by searching for a header and magic strings i

Re: [U-Boot] [PATCH 1/1] trace: trace buffer may exceed 2GiB

2019-06-28 Thread Simon Glass
On Sat, 22 Jun 2019 at 13:10, Simon Glass wrote: > > On Fri, 14 Jun 2019 at 20:52, Heinrich Schuchardt wrote: > > > > Correct the debug output for the trace buffer size to accommodate trace > > buffers exceeding 2GiB. > > > > Signed-off-by: Heinrich Schuchardt > > --- > > common/board_f.c | 4 +

Re: [U-Boot] [PATCH 3/4 v3] test: dm: Add a test for PCI Enhanced Allocation

2019-06-28 Thread Simon Glass
On Fri, 7 Jun 2019 at 02:24, Alex Marginean wrote: > > This test is built on top of the existing swap_case driver. It adds EA > capability structure support to swap_case and uses that to map BARs. > BAR1 works as it used to, swapping upper/lower case. BARs 2,4 map to a > couple of magic values.

Re: [U-Boot] [PATCH v3 0/3] fdt: Allow indicating a node is for U-Boot proper only

2019-06-28 Thread Simon Glass
On Thu, 13 Jun 2019 at 10:33, Simon Glass wrote: > > Hi Patrick, > > On Thu, 13 Jun 2019 at 09:12, Patrick DELAUNAY > wrote: > > > > Hi Simon, > > > > > > > > Hi, > > > > > > I create this serie with: > > > > > > 1/ documentation update for previous patch > > >[U-Boot,v2] dm: remove pre relo

Re: [U-Boot] [PATCH 1/1] trace: conserve gd register

2019-06-28 Thread Simon Glass
On Sat, 22 Jun 2019 at 13:10, Simon Glass wrote: > > On Sun, 2 Jun 2019 at 12:05, Heinrich Schuchardt wrote: > > > > An UEFI application may change the value of the register that gd lives in. > > But some of our functions like get_ticks() access this register. So we > > have to set the gd registe

Re: [U-Boot] [PATCH 1/1] trace: make call depth limit customizable

2019-06-28 Thread Simon Glass
On Sat, 22 Jun 2019 at 13:10, Simon Glass wrote: > > On Sun, 2 Jun 2019 at 12:30, Heinrich Schuchardt wrote: > > > > Up to now we had hard coded values for the call depth up to which trace > > records are created: 200 for early tracing, 15 thereafter. UEFI > > applications reach a call depth of 8

Re: [U-Boot] [PATCH 1/1] trace: do not limit trace buffer to 2GiB

2019-06-28 Thread Simon Glass
On Sat, 22 Jun 2019 at 13:10, Simon Glass wrote: > > On Fri, 14 Jun 2019 at 20:51, Heinrich Schuchardt wrote: > > > > There is no good reason to limit the trace buffer to 2GiB on a 64bit > > system. Adjust the types of the relevant parameters. > > > > Signed-off-by: Heinrich Schuchardt > > --- >

Re: [U-Boot] [PATCH 1/4 v2] pci: fixed dm_pci_map_bar comment

2019-06-28 Thread Simon Glass
On Wed, 5 Jun 2019 at 04:05, Bin Meng wrote: > > On Tue, Jun 4, 2019 at 8:46 PM Alex Marginean wrote: > > > > The comment now indicates that the input argument bar is a register offset, > > not a BAR index. > > It also mentions which BARs are supported for type 0/1 and that the > > function can r

Re: [U-Boot] [PATCH 4/4 v3] drivers: pci: add API to issue FLR on a PCI function if supported

2019-06-28 Thread Simon Glass
On Fri, 7 Jun 2019 at 02:24, Alex Marginean wrote: > > Adds dm_pci_flr API that issues a Function Level reset on a PCI-e function, > if FLR is supported. > > Signed-off-by: Alex Marginean > Reviewed-by: Bin Meng > --- > > Changes in v2: > - Use kernel PCI_EXP macros for register offsets

Re: [U-Boot] [PATCH v3 3/3] dm: doc: add documentation for pre-reloc properties in SPL and TPL

2019-06-28 Thread Simon Glass
On Tue, 21 May 2019 at 11:19, Patrick Delaunay wrote: > > Add documentation for the pre-reloc property in SPL and TPL device-tree: > - u-boot,dm-pre-proper > - u-boot,dm-pre-reloc > - u-boot,dm-spl > - u-boot,dm-tpl > > Signed-off-by: Patrick Delaunay > Reviewed-by: Simon Glass > --- > > Changes

Re: [U-Boot] [PATCH 2/4 v3] drivers: pci: add map_bar support for Enhanced Allocation

2019-06-28 Thread Simon Glass
On Fri, 7 Jun 2019 at 02:24, Alex Marginean wrote: > > Makes dm_pci_map_bar API available for integrated PCI devices that > support Enhanced Allocation instead of the original PCI BAR mechanism. > > Signed-off-by: Alex Marginean > Reviewed-by: Bin Meng > --- > > Changes in v2: > - fixed

Re: [U-Boot] [PATCH 1/1] trace: undefined reference to `trace_early_init'

2019-06-28 Thread Simon Glass
On Sat, 22 Jun 2019 at 13:10, Simon Glass wrote: > > On Sat, 1 Jun 2019 at 23:59, Heinrich Schuchardt wrote: > > > > Compiling with TRACE but without TRACE_EARLY results in an error > > aarch64-linux-gnu-ld.bfd: > > common/built-in.o:(.rodata.init_sequence_f+0x10): > > undefined reference to `tra

Re: [U-Boot] [PATCH 1/1] imx6: wandboard: allow booting from MMC 2

2019-06-28 Thread Simon Glass
On Sun, 23 Jun 2019 at 07:33, Fabio Estevam wrote: > > Hi Heinrich, > > On Sun, Jun 23, 2019 at 4:55 AM Heinrich Schuchardt > wrote: > > > > One of the SD-CARD slots on the Wandboard Quad B1 is MMC 2. Enable it as a > > boot device. > > > > Signed-off-by: Heinrich Schuchardt > > Reviewed-by: Fa

Re: [U-Boot] [PATCH v3 1/3] test: check u-boot properties in SPL device tree

2019-06-28 Thread Simon Glass
On Tue, 21 May 2019 at 18:53, Simon Glass wrote: > > On Tue, 21 May 2019 at 11:19, Patrick Delaunay > wrote: > > > > Add a test to check the management of the U-boot relocation properties > > for device tree SPL generation (fdtgrep result) and platdata: > > - 'dm-pre-proper' and 'dm-tpl' not inc

Re: [U-Boot] [PATCH 1/2] fdtdec: test: Fix memory leak

2019-06-28 Thread Simon Glass
On Tue, 21 May 2019 at 10:43, Simon Glass wrote: > > Hi Thierry, > > On Tue, 21 May 2019 at 04:21, Thierry Reding wrote: > > > > On Mon, May 20, 2019 at 02:51:08PM -0600, Simon Glass wrote: > > > Hi Thierry, > > > > > > On Mon, 20 May 2019 at 10:05, Thierry Reding > > > wrote: > > > > > > > > F

Re: [U-Boot] [PATCH 00/17] sandbox: Various bug fixes

2019-06-28 Thread Simon Glass
Hi, On Sat, 18 May 2019 at 11:59, Simon Glass wrote: > > This series collects together a lot of fixes for sandbox (mostly) that > were discovered in the last release. Most of them are minor. > > > Simon Glass (17): > spl: misc: Allow misc drivers in SPL and TPL > Add an empty stdint.h file >

Re: [U-Boot] python3 support for pylibfdt

2019-06-28 Thread Simon Glass
Hi Tom, On Fri, 28 Jun 2019 at 06:51, Tom Rini wrote: > > On Fri, Jun 28, 2019 at 01:38:01PM +0100, Peter Robinson wrote: > > > > > > With the EOL of python2 soon I've been looking at the Fedora U-Boot > > > > > > builds to see what it would take to move over to python3. There's a > > > > > > cou

Re: [U-Boot] [PATCH v1 06/11] mmc: if possible, poll the busy state using DAT0

2019-06-28 Thread Marek Vasut
On 6/28/19 10:28 AM, Jean-Jacques Hiblot wrote: > > On 28/06/2019 02:07, Marek Vasut wrote: >> On 6/27/19 12:31 PM, Jean-Jacques Hiblot wrote: >>> Using the DAT0 line as a rdy/busy line is an alternative to reading the >>> status register of the card. It especially useful in situation where the >>

Re: [U-Boot] U-Boot/EBBR plugfest at ELC-EU?

2019-06-28 Thread Neil Armstrong
On 28/06/2019 10:19, Grant Likely wrote: > Quick poll: who would be interested in a U-Boot/EBBR plugfest event > collocates with ELC-EU this year (week of 28th Oct)? > > In the EBBR meetings we’ve tossed around the idea of an U-Boot/EBBR plugfest > to work out compatibility issues between OS dis

[U-Boot] [PATCH 0/4] Align QSPI compatible string with kernel one:

2019-06-28 Thread Patrice Chotard
Initially, QSPI compatible string was first introduced in U-boot. On kernel side a different one was used, so align U-boot with kernel. Patrice Chotard (4): ARM: dts: stm32: Use kernel qspi compatible string for stm32f7-uboot.dtsi ARM: dts: stm32: Use kernel qspi compatible string for

[U-Boot] [PATCH 3/4] spi: stm32_qspi: Remove "st, stm32-qspi" compatible string

2019-06-28 Thread Patrice Chotard
"st,stm32-qspi" is no more used, remove it. Signed-off-by: Patrice Chotard --- drivers/spi/stm32_qspi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/spi/stm32_qspi.c b/drivers/spi/stm32_qspi.c index bb1067ff4a..c5878e4058 100644 --- a/drivers/spi/stm32_qspi.c +++ b/drivers/spi/stm

[U-Boot] [PATCH 1/4] ARM: dts: stm32: Use kernel qspi compatible string for stm32f7-uboot.dtsi

2019-06-28 Thread Patrice Chotard
For STM32 QSPI driver, "st,stm32-qspi" compatible string was first introduced in U-boot. But later in kernel side, "st,stm32f469-qspi" was used. To simplify, align U-boot QSPI compatible string with kernel one. Signed-off-by: Patrice Chotard --- arch/arm/dts/stm32f7-u-boot.dtsi | 2 +- 1 file c

[U-Boot] [PATCH 4/4] doc: device-tree-bindings: alignment with v5.2-rc6 for spi-stm32-qspi.txt

2019-06-28 Thread Patrice Chotard
Align doc/device-tree-bindings/spi/spi-stm32-qspi.txt with kernel v5.2-rc6 Signed-off-by: Patrice Chotard --- .../spi/spi-stm32-qspi.txt| 71 ++- 1 file changed, 38 insertions(+), 33 deletions(-) diff --git a/doc/device-tree-bindings/spi/spi-stm32-qspi.txt

[U-Boot] [PATCH 2/4] ARM: dts: stm32: Use kernel qspi compatible string for stm32f469-disco-uboot.dtsi

2019-06-28 Thread Patrice Chotard
For STM32 QSPI driver, "st,stm32-qspi" compatible string was first introduced in U-boot. But later in kernel side, "st,stm32f469-qspi" was used. To simplify, align U-boot QSPI compatible string with kernel one. Signed-off-by: Patrice Chotard --- arch/arm/dts/stm32f469-disco-u-boot.dtsi | 2 +-

Re: [U-Boot] python3 support for pylibfdt

2019-06-28 Thread Tom Rini
On Fri, Jun 28, 2019 at 01:38:01PM +0100, Peter Robinson wrote: > > > > > With the EOL of python2 soon I've been looking at the Fedora U-Boot > > > > > builds to see what it would take to move over to python3. There's a > > > > > couple of issues building the bundled pylibfdt, the first is the > >

Re: [U-Boot] [PATCH v3 2/2] xilinx_xiic: Fix transfer initialisation

2019-06-28 Thread Heiko Schocher
Hello Tomas, Am 28.06.2019 um 14:13 schrieb Melin Tomas: Hi, On 6/28/19 3:08 PM, Melin Tomas wrote: Prior to starting a new transfer, conditionally wait for bus to not be busy. Reinitialise controller as otherwise operation is not stable. For reference, see linux kernel commit: 9656eeebf3f1 (

[U-Boot] [PATCH v2 3/5] fpga: altera: cyclon2: Fix indentation

2019-06-28 Thread Alexander Dahl
Some code parts stood too far left … Signed-off-by: Alexander Dahl --- drivers/fpga/cyclon2.c | 32 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/drivers/fpga/cyclon2.c b/drivers/fpga/cyclon2.c index 99a64d437c..ab979b017a 100644 --- a/drivers/f

[U-Boot] [PATCH v2 4/5] fpga: altera: cyclon2: Check function pointer before calling

2019-06-28 Thread Alexander Dahl
As already done for the 'pre' function, a check is added to not follow a NULL pointer, if somebody has not assigned a 'post' function. Signed-off-by: Alexander Dahl --- drivers/fpga/cyclon2.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/fpga/cyclon2.c b/driver

[U-Boot] [PATCH v2 2/5] fpga: altera: cyclon2: Fix most checkpatch warnings

2019-06-28 Thread Alexander Dahl
Nothing special, but done before further cleanup. * spacing * braces * __FUNCTION__ → __func__ Suggested-by: Michal Simek Signed-off-by: Alexander Dahl --- drivers/fpga/cyclon2.c | 81 -- 1 file changed, 39 insertions(+), 42 deletions(-) diff --

[U-Boot] [PATCH v2 5/5] cmd: fpga: Change return value to avoid printing usage text

2019-06-28 Thread Alexander Dahl
In cmd/fpga.c the commands should return enum command_ret_t, e.g. CMD_RET_USAGE, CMD_RET_SUCCESS, or CMD_RET_FAILURE. What they actually do is passing a return value from different 'fpga_' functions. Passing on a return value of -1 from a called function leads to printing out usage text. In case o

[U-Boot] [PATCH v2 0/5] fpga: collected patches

2019-06-28 Thread Alexander Dahl
Hei hei, after some Feedback from Michal Simek, I made a v2 of this patch series, and based it onto v2019.07-rc4 now. The patch number 2 is new, fixing all kinds of checkpatch warnings in 'drivers/fpga/cyclon2.c' before fixing the indentation in that file in patch number 3 (which was part of v1 a

[U-Boot] [PATCH v2 1/5] fpga: altera: Add some more device sizes

2019-06-28 Thread Alexander Dahl
There seems to be only one place, where this is checked against: `altera_validate()`. It should be non zero. Otherwise it is only used to display it, so it probably does not really matter at the moment. But we had the datasheet open anyway … Sizes in datasheet are bit counts, display here is in by

  1   2   >