Re: [U-Boot] [PATCH 4/7] mmc: pci: fix uninitialized pointer deref on probe

2019-08-13 Thread Simon Glass
On Mon, 5 Aug 2019 at 20:48, Peng Fan wrote: > > Commit 3d296365e4e8 ("mmc: sdhci: Add support for > sdhci-caps-mask") sdhci_setup_cfg() expects a valid sdhci_host mmc > field. > > Fixes: 3d296365e4e8 ("mmc: sdhci: Add support for sdhci-caps-mask") > Cc: Faiz Abbas > Cc: Simon Glass > Cc: Bin

[U-Boot] Travis test/py sandbox_spl test fail

2019-08-13 Thread Kever Yang
Hi Simon,     I got fail in test/py sandbox_spl, and the log says: E OSError: [Errno 5] Input/output error I have no idea about what's wrong in source code, could you help to take a look? Travis job: https://travis-ci.org/keveryang/u-boot/jobs/571125119 Thanks, - Kever

Re: [U-Boot] [PATCH PATCH v4 00/15] Add support for applications of overlays in SPL

2019-08-13 Thread Simon Glass
Hi Jean-Jacques, On Mon, 5 Aug 2019 at 03:44, Jean-Jacques Hiblot wrote: > > > The purpose of this series is to provide the SPL with ability to apply > overlays for u-boot. this is only a RFC so far, to get a feedback on the > approach. > > Our use-case is the support of the daughter boards of

Re: [U-Boot] [PATCH PATCH v4 04/15] spl: fit: allocate a temporary buffer to load the overlays

2019-08-13 Thread Simon Glass
Hi Jean-Jacques, On Mon, 5 Aug 2019 at 03:44, Jean-Jacques Hiblot wrote: > > If the node describing an overlay does not specify a load address, it will > be loaded at the address previously used. > Fixing it by allocating a temporary 64kB region that will be used as a > default load address.

Re: [U-Boot] [PATCH] dm: pci: Change to config device according to Header Type

2019-08-13 Thread Simon Glass
Hi Hou, On Mon, 5 Aug 2019 at 02:05, Hou Zhiqiang wrote: > > The configuration space layouts of normal device, bridge and > CardBus bridge are distinct, and it is indicated by Header > Type instead of Class Code. So change it to config device > according to Header Type. > > Signed-off-by: Hou

Re: [U-Boot] [PATCH 1/2] usb: kbd: simplify coding for arrow keys

2019-08-13 Thread Simon Glass
Hi Heinrich, On Fri, 9 Aug 2019 at 15:36, Heinrich Schuchardt wrote: > > On 6/24/19 8:49 PM, Simon Glass wrote: > > Hi Heinrich, > > > > On Mon, 24 Jun 2019 at 11:36, Heinrich Schuchardt > > wrote: > >> > >> On 6/24/19 3:56 PM, Simon Glass wrote: > >>> On Sun, 16 Jun 2019 at 22:44, Heinrich

Re: [U-Boot] [U-Boot-Custodians] [RFC] enforce CONFIG_DM=y

2019-08-13 Thread Tom Rini
On Tue, Aug 13, 2019 at 11:53:59AM +0200, Marek Vasut wrote: > On 8/13/19 11:34 AM, Simon Glass wrote: > > On Fri, 9 Aug 2019 at 12:39, Heinrich Schuchardt wrote: > >> > >> The time lines set force in doc/driver-model/migration.rst for different > >> subsystems to move to the driver model have

Re: [U-Boot] [RESEND PATCH] core: ofnode: Add ofnode_get_addr_size_index

2019-08-13 Thread Simon Glass
Hi Keethy, On Tue, 9 Jul 2019 at 13:38, Eugeniu Rosca wrote: > > Hi All, > > On Wed, Apr 24, 2019 at 05:19:53PM +0530, Keerthy wrote: > > Add ofnode_get_addr_size_index function to fetch the address > > and size of the reg space based on index. > > > > Signed-off-by: Keerthy > > Reviewed-by:

Re: [U-Boot] [PATCH PATCH v4 01/15] spl: fit: don't load the firmware twice

2019-08-13 Thread Simon Glass
On Mon, 5 Aug 2019 at 03:44, Jean-Jacques Hiblot wrote: > > When u-boot.img is a FIT image generated automatically by mkimage, the > configuration node has the following structure: > conf-1 { >description = "k3-am654-base-board"; >firmware = "firmware-1"; >loadables = "firmware-1"; >

Re: [U-Boot] x86 FSP 2

2019-08-13 Thread Simon Glass
Hi Bin, On Tue, 30 Jul 2019 at 21:03, Bin Meng wrote: > > Hi Simon, > > On Wed, Jul 31, 2019 at 10:55 AM Simon Glass wrote: > > > > Hi Bin, > > > > On Thu, 23 May 2019 at 08:04, Bin Meng wrote: > > > > > > Hi Simon, > > > > > > On Thu, May 23, 2019 at 8:20 PM Simon Glass wrote: > > > > > > >

Re: [U-Boot] [PATCH v2 4/5] usb: kbd: move USB_KBD_BOOT_REPORT_SIZE to usb.h

2019-08-13 Thread Simon Glass
On Sat, 10 Aug 2019 at 03:24, Heinrich Schuchardt wrote: > > Move constant USB_KBD_BOOT_REPORT_SIZE. This allows us to reuse it. > > Signed-off-by: Heinrich Schuchardt > --- > v2 > new patch > --- > common/usb_kbd.c | 6 -- > include/usb.h| 6 ++ > 2 files changed, 6

Re: [U-Boot] [PATCH PATCH v4 12/15] drivers: board: Make the board drivers available in SPL

2019-08-13 Thread Simon Glass
Hi Jean-Jacques, Missing commit message. On Mon, 5 Aug 2019 at 03:44, Jean-Jacques Hiblot wrote: > > Signed-off-by: Jean-Jacques Hiblot > --- > > Changes in v4: New > > drivers/Makefile | 2 +- > drivers/board/Kconfig | 2 ++ > drivers/board/Makefile | 2 +- > 3 files changed, 4

Re: [U-Boot] [RESEND PATCH] core: ofnode: Add ofnode_get_addr_size_index

2019-08-13 Thread Keerthy
On 13/08/19 3:03 PM, Simon Glass wrote: Hi Keethy, On Tue, 9 Jul 2019 at 13:38, Eugeniu Rosca wrote: Hi All, On Wed, Apr 24, 2019 at 05:19:53PM +0530, Keerthy wrote: Add ofnode_get_addr_size_index function to fetch the address and size of the reg space based on index. Signed-off-by:

Re: [U-Boot] [PATCH 1/2] i.MX6: nand: extend nandbcb command for imx6UL(L)

2019-08-13 Thread Shyam Saini
Hi Parthiban, Thanks a lot for working on this. Please see comments below. > Firmware Configuration Block(FCB) for imx6ul(l) needs to be > BCH encoded. This patch depends on [1]. > > [1]: https://patchwork.ozlabs.org/project/uboot/list/?series=113810 > > Signed-off-by: Parthiban Nallathambi >

[U-Boot] [PATCH v2 1/4] kconfig: doc: Update comment regarding CONFIG_IS_ENABLED(FOO) for TPL

2019-08-13 Thread Lukasz Majewski
This patch adds some commit info for CONFIG_IS_ENABLED(FOO) when used in TPL context. Signed-off-by: Lukasz Majewski --- Changes in v2: None include/linux/kconfig.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/linux/kconfig.h b/include/linux/kconfig.h index

[U-Boot] [PATCH v2 3/4] spi: Move DM_SPI_FLASH and SPI_FLASH_DATAFLASH to Kconfig (for ls1021aXXX)

2019-08-13 Thread Lukasz Majewski
This patch moves the CONFIG_DM_SPI_FLASH and CONFIG_SPI_FLASH_DATAFLASH to be defined in Kconfig, not in board specific header file (include/configs/.h). Before this change the CONFIG_DM_SPI_FLASH was not set in .config (so it was not possible to use CONFIG_IS_ENABLED(DM_SPI_FLASH) in SPI DM/DTS

[U-Boot] [PATCH v2 4/4] spi: Convert CONFIG_DM_SPI* to CONFIG_$(SPL_TPL_)DM_SPI*

2019-08-13 Thread Lukasz Majewski
This change allows more fine tuning of driver model based SPI support in SPL and TPL. It is now possible to explicitly enable/disable the DM_SPI support in SPL and TPL via Kconfig option. Before this change it was necessary to use: /* SPI Flash Configs */ #if defined(CONFIG_SPL_BUILD)

[U-Boot] [PATCH v2 2/4] spi: Move DM_SPI_FLASH to Kconfig (for NXP's ls1043a)

2019-08-13 Thread Lukasz Majewski
This patch fixes issue with defining the DM_SPI_FLASH in the configs/include/ instead of enabling this option in Kconfig. The problem is that CONFIG_IS_ENABLED(DM_SPI_FLASH) shows false as there is no DM_SPI_FLASH=y in .config (but the define is set in u-boot.cfg). As a result conversion of

Re: [U-Boot] [PATCH 1/2] i.MX6: nand: extend nandbcb command for imx6UL(L)

2019-08-13 Thread Parthiban Nallathambi
Hi Shyam, On 8/13/19 3:40 PM, Shyam Saini wrote: Hi Parthiban, Thanks a lot for working on this. Still enabling SECURE_BOOT fails to boot. Am yet to figure out this. Do you have secure boot working from NAND? Please see comments below. Firmware Configuration Block(FCB) for imx6ul(l)

Re: [U-Boot] [PATCH v9 8/9] board: intel: Add new slimbootloader board

2019-08-13 Thread Park, Aiden
Hi Bin, > On Aug 13, 2019, at 9:03 PM, Bin Meng wrote: > > Hi Aiden, > >> On Thu, Aug 8, 2019 at 10:42 PM Bin Meng wrote: >> >> Hi Aiden, >> >>> On Thu, Aug 8, 2019 at 10:35 PM Bin Meng wrote: >>> On Sat, Aug 3, 2019 at 4:31 PM Park, Aiden wrote: Add slimbootloader board

[U-Boot] [PATCH v2] rockchip: ram: rk3399: fix setting for some type of dram

2019-08-13 Thread Kever Yang
We need to store all the ram related cap/map info back to register after all the init has been done in case some of register was reset during the process. Signed-off-by: YouMin Chen Signed-off-by: Kever Yang --- Changes in v2: - fix compile error for non lpddr4 driver

[U-Boot] [PATCH] ARM: da850evm: Remove dead SPI Code

2019-08-13 Thread Adam Ford
With SPL now supporting DM_SPI, the need for compiler directives and hard-coded addresses is obsolete. This patch removes some dead legacy code defining the SPI base address Signed-off-by: Adam Ford diff --git a/include/configs/da850evm.h b/include/configs/da850evm.h index

[U-Boot] [PATCH] ARM: da850evm: Remove dead code

2019-08-13 Thread Adam Ford
Now that SPL supports DM_SERIAL and the direct NOR boot supports DM_SERIAL, the check to see if DM_SERIAL is defined can go away, because all da850evm variants now support DM_SERIAL. This patch simply removes some dead precompiler defines. Signed-off-by: Adam Ford diff --git

[U-Boot] [RFC PATCH v1] common: fix building image.c

2019-08-13 Thread Igor Opaniuk
From: Igor Opaniuk Wrap image_decomp() and all dependent functions with !ifdef(USE_HOSTCC) macro, as this function isn't used by any tool from /tools. Without this it leads to compilation issues (because of CONFIG_LZMA from newly added #include statement, which pulls all these definitions):

Re: [U-Boot] [RFC PATCH v1] common: fix building image.c

2019-08-13 Thread Igor Opaniuk
Hi all, On Tue, Aug 13, 2019 at 3:44 PM Igor Opaniuk wrote: > > From: Igor Opaniuk > > Wrap image_decomp() and all dependent functions with > !ifdef(USE_HOSTCC) macro, as this function isn't used by > any tool from /tools. > > Without this it leads to compilation issues (because of >

Re: [U-Boot] [PATCH v9 8/9] board: intel: Add new slimbootloader board

2019-08-13 Thread Bin Meng
Hi Aiden, On Thu, Aug 8, 2019 at 10:42 PM Bin Meng wrote: > > Hi Aiden, > > On Thu, Aug 8, 2019 at 10:35 PM Bin Meng wrote: > > > > On Sat, Aug 3, 2019 at 4:31 PM Park, Aiden wrote: > > > > > > Add slimbootloader board to run U-boot as a Slim Bootloader payload > > > - Add new

Re: [U-Boot] [RFC PATCH v1] common: fix building image.c

2019-08-13 Thread Igor Opaniuk
Hi all, On Tue, Aug 13, 2019 at 3:44 PM Igor Opaniuk wrote: > > From: Igor Opaniuk > > Wrap image_decomp() and all dependent functions with > !ifdef(USE_HOSTCC) macro, as this function isn't used by > any tool from /tools. > > Without this it leads to compilation issues (because of >

<    1   2   3