Re: [U-Boot] [RFC 3/6] efi_loader: support convert_pointer at runtime

2019-07-12 Thread Heinrich Schuchardt
On 6/17/19 7:41 AM, Heinrich Schuchardt wrote: On 6/17/19 3:15 AM, AKASHI Takahiro wrote: On Sat, Jun 15, 2019 at 09:41:31PM +0200, Heinrich Schuchardt wrote: On 6/5/19 6:21 AM, AKASHI Takahiro wrote: With this patch, ConvertPointer runtime service is enabled. This function will be useful only

[U-Boot] [PATCH 0/2] efi_loader: EVT_SIGNAL_VIRTUAL_ADDRESS_CHANGE

2019-07-12 Thread Heinrich Schuchardt
Implement the EVT_SIGNAL_VIRTUAL_ADDRESS_CHANGE event. This requires allocating the event and the event list from runtime data. Provide a unit test for SetVirtualAddressMap() and ConvertPointer(). As ConvertPointer() is not implemented yet this will spit out some warnings. Heinrich Schuchardt (2)

[U-Boot] [PATCH 1/2] efi_loader: EVT_SIGNAL_VIRTUAL_ADDRESS_CHANGE

2019-07-12 Thread Heinrich Schuchardt
Implement the EVT_SIGNAL_VIRTUAL_ADDRESS_CHANGE event. This requires allocating the event and the event list from runtime data. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_boottime.c | 25 +++-- lib/efi_loader/efi_runtime.c | 14 -- 2 files changed

[U-Boot] [PATCH 2/2] efi_selftest: EVT_SIGNAL_VIRTUAL_ADDRESS_CHANGE

2019-07-12 Thread Heinrich Schuchardt
Provide a unit test for SetVirtualAddressMap() and ConvertPointer(). As ConvertPointer() is not implemented yet this will spit out some warnings. Signed-off-by: Heinrich Schuchardt --- lib/efi_selftest/Makefile | 1 + .../efi_selftest_set_virtual_address_map.c| 192

Re: [U-Boot] [PATCH 8/8] Add support for the NXP LS1021A-TSN board

2019-07-12 Thread Bin Meng
Hi Vladimir, On Mon, Jun 24, 2019 at 1:50 AM Vladimir Oltean wrote: > > From: Jianchao Wang > > The LS1021A-TSN is a development board built by VVDN/Argonboards in > partnership with NXP. > > It features the LS1021A SoC and the first-generation SJA1105T Ethernet > switch for prototyping implemen

Re: [U-Boot] [PATCH 4/8] net: tsec: Make errors visible

2019-07-12 Thread Bin Meng
On Mon, Jun 24, 2019 at 1:50 AM Vladimir Oltean wrote: > > This replaces debug() calls with printf() so that it is immediately > obvious from the console that something is wrong. > > Signed-off-by: Vladimir Oltean > --- > drivers/net/tsec.c | 8 > 1 file changed, 4 insertions(+), 4 dele

Re: [U-Boot] [PATCH 3/8] net: tsec: Reverse Christmas tree notation

2019-07-12 Thread Bin Meng
On Mon, Jun 24, 2019 at 1:50 AM Vladimir Oltean wrote: > > This is a cosmetic patch that reorders variable definitions in the > inverse order of their line length, where possible. > > Signed-off-by: Vladimir Oltean > --- > drivers/net/tsec.c | 10 +- > 1 file changed, 5 insertions(+), 5

Re: [U-Boot] [PATCH 7/8] configs: ls1021atwr: Fix distro_bootcmd for QSPI boot

2019-07-12 Thread Bin Meng
On Mon, Jun 24, 2019 at 1:50 AM Vladimir Oltean wrote: > > Due to a typo, "run qspi_bootcmd" and "env exists secureboot" got > concatenated instead of being separated by a semicolon. > > Signed-off-by: Vladimir Oltean > --- > include/configs/ls1021atwr.h | 2 +- > 1 file changed, 1 insertion(+),

Re: [U-Boot] [PATCH 5/8] net: tsec: Common handling of MAC station address for DM_ETH

2019-07-12 Thread Bin Meng
On Mon, Jun 24, 2019 at 1:50 AM Vladimir Oltean wrote: > > In tsec_init, the MAC address is retrieved from 2 different structures > depending on whether DM_ETH is enabled or not. > > But since the field name is the same inside both structures, we can > conditionally define the structure of the cor

Re: [U-Boot] [PATCH 2/8] net: tsec: Fix offset of MDIO registers for DM_ETH

2019-07-12 Thread Bin Meng
Hi Vladimir, On Mon, Jun 24, 2019 at 1:50 AM Vladimir Oltean wrote: > > By convention, the eTSEC MDIO controller nodes are defined in DT at > 0x2d24000 and 0x2d5, but actually U-boot does not touch the nits: U-Boot, not U-boot > interrupt portion of the register map (MDIO_IEVENTM, MDIO_IMAS

Re: [U-Boot] [PATCH 1/8] net: tsec: Refactor the readout of the tbi-handle property

2019-07-12 Thread Bin Meng
On Mon, Jun 24, 2019 at 1:50 AM Vladimir Oltean wrote: > > The point of this patch is to eliminate the use of the locally-defined > "reg" variable (which interferes with next patch) and simplify the > fallback to the default CONFIG_SYS_TBIPA_VALUE in case "tbi-handle" is > missing. > > Signed-off-

Re: [U-Boot] [PATCH 1/2] drivers: net: driver for MDIO muxes controlled over I2C

2019-07-12 Thread Bin Meng
Hi Alex, On Fri, Jul 12, 2019 at 10:21 PM Alex Marginean wrote: > > This driver is used for MDIO muxes driven over I2C. This is currently > used on Freescale LS1028A QDS board, on which the physical MDIO MUX is > controlled by an on-board FPGA which in turn is configured through I2C. > > Signed-

Re: [U-Boot] [EXT] Re: [u-boot][xhci] Dealing with XHCI controllers with spurious success

2019-07-12 Thread Aaron Williams
No, it's our Marvell OcteonTX2 SoC. I put in what I thought was a fix, but I'm still working on it. This issue only shows up when using a USB Ethernet device where the rx buffer crosses a 64K boundary and the packet is received in the first buffer. I don't know how other hardware behaves in this

Re: [U-Boot] [PATCH 2/2] configs: ls1028aqds: enable I2C and GPIO DM

2019-07-12 Thread Bin Meng
On Fri, Jul 12, 2019 at 7:25 PM Alex Marginean wrote: > > LS1028A was already converted to I2C/GPIO DM, move the QDS board over too. > > Signed-off-by: Alex Marginean > --- > configs/ls1028aqds_tfa_SECURE_BOOT_defconfig | 5 - > configs/ls1028aqds_tfa_defconfig | 5 - > 2 fil

Re: [U-Boot] [PATCH 1/2] armv8: ls1028aqds: switch LS1028A QDS board to I2C DM

2019-07-12 Thread Bin Meng
On Fri, Jul 12, 2019 at 7:25 PM Alex Marginean wrote: > > Replace legacy I2C_EARLY_INIT with dm-pre-reloc dts property. > > Signed-off-by: Alex Marginean > --- > > Depends on this patch series: > https://patchwork.ozlabs.org/project/uboot/list/?series=118778 > > arch/arm/dts/fsl-ls1028a-qds.dts

Re: [U-Boot] [u-boot 2/2] drivers/fsl-mc: Support DPSPARSER object and apply spb command

2019-07-12 Thread Joe Hershberger
On Tue, Jul 9, 2019 at 3:33 PM Joe Hershberger wrote: > > On Wed, May 15, 2019 at 4:10 AM Florinel Iordache > wrote: > > > > Add support for DPSPARSER object (create/destroy, open/close, apply spb) > > required to configure Soft Parser by using MC. Also add uboot command to > > apply a Soft Parse

Re: [U-Boot] [PATCH] net/macb: increase RX buffer size for GEM

2019-07-12 Thread Joe Hershberger
On Mon, Jun 24, 2019 at 2:02 PM Ramon Fried wrote: > > Macb Ethernet controller requires a RX buffer of 128 bytes. It is > highly sub-optimal for Gigabit-capable GEM that is able to use > a bigger DMA buffer. Change this constant and associated macros > with data stored in the private structure. >

Re: [U-Boot] [PATCH] net: designware: remove mdio bus on probe failure

2019-07-12 Thread Joe Hershberger
On Fri, Jul 12, 2019 at 2:07 PM Simon Goldschmidt wrote: > > The designware eth driver registers an mdio bus during probe, but if no > PHY is found, this bus is never removed although probe failes and the > driver is shown as not probed in the dm tree. > > This later leads to errors when e.g. the

Re: [U-Boot] [PATCH v4] gpio: add gpio-hog support

2019-07-12 Thread Simon Glass
Hi Heiko, On Wed, 26 Jun 2019 at 22:12, Heiko Schocher wrote: > > Hello Simon, > > Am 26.06.2019 um 17:07 schrieb Simon Glass: > > Hi Heiko, > > > > On Mon, 24 Jun 2019 at 00:16, Heiko Schocher wrote: > >> > >> Hello Simon, > >> > >> Am 22.06.2019 um 21:09 schrieb Simon Glass: > >>> Hi Heiko, >

Re: [U-Boot] [PATCH 1/2] dm: core: device: switch off power domain after device removal

2019-07-12 Thread Simon Glass
Hi Anatolij, On Wed, 10 Jul 2019 at 08:09, Anatolij Gustschin wrote: > > The power domain associated with a device is enabled when probing, > but currently the domain remains enabled when the device is removed. > Some boards started to disable power domains for selected devices > via custom board

Re: [U-Boot] [PATCH] dm: uclass: fix comment copy/paste error

2019-07-12 Thread Simon Glass
On Thu, 11 Jul 2019 at 02:23, Baruch Siach wrote: > > The uclass_next_device() routine continues a previously started device > iteration. Change the description that is copied from > uclass_first_device(). > > Signed-off-by: Baruch Siach > --- > include/dm/uclass.h | 2 +- > 1 file changed, 1 in

Re: [U-Boot] [PATCH v2] board/BuR/brsmarc1: initial commit

2019-07-12 Thread Tom Rini
On Thu, May 02, 2019 at 02:09:14PM +0200, Hannes Schmelzer wrote: > This commit adds support for the B&R brsmarc1 SoM. > > The SoM is based on TI's AM335x SoC. > Mainly vxWorks 6.9.4.x is running on the board, > doing some PLC stuff on various carrier boards. > > Signed-off-by: Hannes Schmelzer

[U-Boot] [PATCH 1/1] efi_loader: efi_set_virtual_address_map_runtime()

2019-07-12 Thread Heinrich Schuchardt
efi_set_virtual_address_map_runtime() must be of type __efi_runtime. Fixes: ee8ebedc ("efi_loader: split off detaching SetVirtualAddress()") Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_runtime.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/efi_loader/

Re: [U-Boot] [PATCH 6/8] arm: ls1021atwr: Convert to use driver model TSEC driver

2019-07-12 Thread Joe Hershberger
On Sun, Jun 23, 2019 at 12:53 PM Vladimir Oltean wrote: > > From: Bin Meng > > Now that we have added driver model support to the TSEC driver, > convert ls1021atwr board to use it. > > This depends on previous DM series for ls1021atwr: > http://patchwork.ozlabs.org/patch/561855/ > > Signed-off-by

[U-Boot] [PATCH] arm: mach-keystone: Use appended original image size in image processing

2019-07-12 Thread Andrew F. Davis
When an image is signed/encrypted on K2 devices the image may get padded to meet alignment requirements for the signature/encryption type. The original size is appended as 4 bytes little-endian to the end of the final image. Normally the trailing extra bytes are ignored and so restoring the exact

[U-Boot] [PATCH] net: designware: remove mdio bus on probe failure

2019-07-12 Thread Simon Goldschmidt
The designware eth driver registers an mdio bus during probe, but if no PHY is found, this bus is never removed although probe failes and the driver is shown as not probed in the dm tree. This later leads to errors when e.g. the mii or mdio commands try to use available mdio buses because the mdio

Re: [U-Boot] [PATCH 5/8] net: tsec: Common handling of MAC station address for DM_ETH

2019-07-12 Thread Joe Hershberger
On Sun, Jun 23, 2019 at 12:52 PM Vladimir Oltean wrote: > > In tsec_init, the MAC address is retrieved from 2 different structures > depending on whether DM_ETH is enabled or not. > > But since the field name is the same inside both structures, we can > conditionally define the structure of the co

Re: [U-Boot] [PATCH 4/8] net: tsec: Make errors visible

2019-07-12 Thread Joe Hershberger
On Sun, Jun 23, 2019 at 12:52 PM Vladimir Oltean wrote: > > This replaces debug() calls with printf() so that it is immediately > obvious from the console that something is wrong. > > Signed-off-by: Vladimir Oltean Acked-by: Joe Hershberger ___ U-Boot

Re: [U-Boot] [PATCH 3/8] net: tsec: Reverse Christmas tree notation

2019-07-12 Thread Joe Hershberger
On Sun, Jun 23, 2019 at 12:53 PM Vladimir Oltean wrote: > > This is a cosmetic patch that reorders variable definitions in the > inverse order of their line length, where possible. > > Signed-off-by: Vladimir Oltean I've never heard of anyone optimizing for this before, but sure. Acked-by: Joe

[U-Boot] [PATCH] cmd: mdio: prevent data abort when no mdio bus is found

2019-07-12 Thread Simon Goldschmidt
Calling 'mdio read ...' currently leads to a data abort when no mdio bus is found. To fix this, check if 'bus' is a valid pointer before accessing it. Signed-off-by: Simon Goldschmidt --- cmd/mdio.c | 5 + 1 file changed, 5 insertions(+) diff --git a/cmd/mdio.c b/cmd/mdio.c index 5e219f69

Re: [U-Boot] [PATCH 1/8] net: tsec: Refactor the readout of the tbi-handle property

2019-07-12 Thread Joe Hershberger
On Sun, Jun 23, 2019 at 12:50 PM Vladimir Oltean wrote: > > The point of this patch is to eliminate the use of the locally-defined > "reg" variable (which interferes with next patch) and simplify the > fallback to the default CONFIG_SYS_TBIPA_VALUE in case "tbi-handle" is > missing. > > Signed-off

Re: [U-Boot] [PATCH v3] drivers: net: phy: Use Aquantia driver for AQR112, AQR412

2019-07-12 Thread Joe Hershberger
On Wed, Jun 19, 2019 at 7:54 AM Alexandru Marginean wrote: > > adds AQR112 and AQR412 to the list of supported PHYs using existing AQR > code. > > Signed-off-by: Alex Marginean > Reviewed-By: Ramon Fried Acked-by: Joe Hershberger ___ U-Boot mailing l

Re: [U-Boot] [PATCH v3 2/4] sysreset: socfpga: gen5: add sysreset driver

2019-07-12 Thread Simon Goldschmidt
Am 12.07.2019 um 07:23 schrieb Simon Goldschmidt: On Fri, Jul 12, 2019 at 7:15 AM Marek Vasut wrote: On 7/10/19 9:22 PM, Simon Goldschmidt wrote: Am 09.05.2019 um 21:53 schrieb Marek Vasut: On 5/9/19 8:55 PM, Simon Goldschmidt wrote: On 09.05.19 20:08, Simon Goldschmidt wrote: This adds

Re: [U-Boot] [PATCH v5 1/3] common: Move bootm_decomp_image() to image.c (as image_decomp())

2019-07-12 Thread Julius Werner
> I'd like to review this, but maybe you can help me speed up the process and > tell > us if the move has been a 1:1 code move or if you had to adapt things to the > new > location (other than the function being renamed)? print_decomp_msg() is a 1:1 move. bootm_decomp_image() is mostly 1:1 move,

Re: [U-Boot] [PATCH v5 2/3] fit: Support compression for non-kernel components (e.g. FDT)

2019-07-12 Thread Simon Goldschmidt
Am 12.07.2019 um 20:25 schrieb Julius Werner: If there haven't been any changes from v4, why did you drop Simon Glass's Reviewed-by tag? Sorry, I wasn't aware I was supposed to preserve those manually. Yes, there were no changes so this should still be considered I was suprised by that, too,

Re: [U-Boot] [PATCH 2/3] spl: ubi: Add support for loading simple fitImage

2019-07-12 Thread Tom Rini
On Tue, May 28, 2019 at 03:49:14AM +0200, Marek Vasut wrote: > Add code to load simple fitImage from the kernel volume, the args > volume is unnecessary. > > Signed-off-by: Marek Vasut > Cc: Ladislav Michl > Cc: Tom Rini > --- > common/spl/spl_ubi.c | 22 +- > 1 file chang

Re: [U-Boot] [PATCH v5 0/3] fit: Image node compression

2019-07-12 Thread Julius Werner
> This kind of interleaved change log is kind of hard to read. A higher-level, > manually written changelog for the cover letter would be better I think. Sorry, will rewrite the cover letter next time. For v5 I just added the new patch and didn't change the other two. _

Re: [U-Boot] [PATCH v5 3/3] fit: Support compat string property in configuration node

2019-07-12 Thread Julius Werner
> If there haven't been any changes from v4, why did you drop Simon Glass's > Reviewed-by tag? Same here: Reviewed-by: Simon Glass ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH v5 2/3] fit: Support compression for non-kernel components (e.g. FDT)

2019-07-12 Thread Julius Werner
> If there haven't been any changes from v4, why did you drop Simon Glass's > Reviewed-by tag? Sorry, I wasn't aware I was supposed to preserve those manually. Yes, there were no changes so this should still be considered Reviewed-by: Simon Glass ___ U

[U-Boot] [PATCH v3 2/2] board: Add support for iMX8QXP AI_ML board

2019-07-12 Thread Manivannan Sadhasivam
This commit adds initial board support for iMX8QXP AI_ML board from Einfochips. This board is one of the 96Boards Consumer Edition and AI boards of the 96Boards family based on i.MX8QXP SoC from NXP/Freescale. This initial supports contains following peripherals which are tested and known to work:

[U-Boot] [PATCH v3 1/2] arm: dts: Add devicetree support for iMXQXP AI_ML board

2019-07-12 Thread Manivannan Sadhasivam
Add devicetree support for iMXQXP AI_ML board from Einfochips. Signed-off-by: Manivannan Sadhasivam Reviewed-by: Peng Fan --- arch/arm/dts/Makefile | 1 + arch/arm/dts/fsl-imx8qxp-ai_ml-u-boot.dtsi | 117 + arch/arm/dts/fsl-imx8qxp-ai_ml.dts | 181

[U-Boot] [PATCH v3 0/2] Add board support for iMX8QXP AI_ML board

2019-07-12 Thread Manivannan Sadhasivam
Hello, This patchset adds initial board support for iMX8QXP AI_ML board from Einfochips. This board is one of the Consumer Edition and AI boards of the 96Boards family. This initial supports contains following peripherals which are tested and known to work: 1. Debug serial via UART2 2. SD ca

[U-Boot] [PATCH v2] arm: socfpga: gen5: don't zero bss in board_init_f()

2019-07-12 Thread Simon Goldschmidt
The socfpga gen5 SPL manually zeroed bss in board_init_f(). Now that the DDR driver does not use bss any more, bss is not used before board_init_r() and we can remove this hack. bss is normally zeroed by crt0.S, but after board_init_f(), before board_init_r(). socfpga just had this double-zeroing

Re: [U-Boot] [PATCH v2 2/2] board: Add support for iMXQXP AI_ML board

2019-07-12 Thread Manivannan Sadhasivam
Hi Fabio, On Fri, Jul 12, 2019 at 08:37:24AM -0300, Fabio Estevam wrote: > Hi Manivannan, > > On Thu, Jul 11, 2019 at 3:07 PM Manivannan Sadhasivam > wrote: > > > > This commit adds initial board support for iMXQXP AI_ML board from > > iMXQXP --> i.MX8QXP > Ack. > > > Einfochips. This board

[U-Boot] [PATCH] env: ti: boot: Use ttyS2 instead of ttyO2

2019-07-12 Thread Sam Protsenko
ttyO2 console enables legacy CONFIG_SERIAL_OMAP driver in kernel. Nowadays it's preferred to use the generic CONFIG_SERIAL_8250_OMAP driver, which being enabled via ttyS2 console. Both drivers are enabled in multi_v7_defconfig and in omap2plus_defconfig, for compatibility reasons. Let's switch to t

Re: [U-Boot] [PATCH] env: ti: boot: Use ttyS2 instead of ttyO2

2019-07-12 Thread Sam Protsenko
Superseded by v2. On Tue, Jul 9, 2019 at 5:54 PM Sam Protsenko wrote: > > + Dan Rue > > On Tue, Jul 9, 2019 at 5:49 PM Sam Protsenko > wrote: > > > > ttyO2 console enables legacy CONFIG_SERIAL_OMAP driver in kernel. > > Nowadays it's preferred to use the generic CONFIG_SERIAL_8250_OMAP > > driv

[U-Boot] [PATCH v2] env: ti: boot: Handle reboot reason from BCB

2019-07-12 Thread Sam Protsenko
In case of Android boot, reboot reason can be written into BCB (usually it's an area in 'misc' partition). U-Boot then can obtain that reboot reason from BCB and handle it accordingly to achieve correct Android boot flow, like it was suggested in [1]: - if it's empty: perform normal Android boot

Re: [U-Boot] [PATCH] env: ti: boot: Handle reboot reason from BCB

2019-07-12 Thread Sam Protsenko
Superseded by v2. On Tue, Jul 9, 2019 at 5:45 PM Sam Protsenko wrote: > > In case of Android boot, reboot reason can be written into BCB (usually > it's an area in 'misc' partition). U-Boot then can obtain that reboot > reason from BCB and handle it accordingly to achieve correct Android > boot f

Re: [U-Boot] [PATCH v2 3/5] cmd: bcb: Fix duplicated handling in two case-branches

2019-07-12 Thread Sam Protsenko
On Fri, Jul 12, 2019 at 6:49 PM Eugeniu Rosca wrote: > > On Fri, Jul 12, 2019 at 05:01:09PM +0300, Sam Protsenko wrote: > > > > diff --git a/cmd/bcb.c b/cmd/bcb.c > > > > index 2bd5a744deb5..3b1c7434e287 100644 > > > > --- a/cmd/bcb.c > > > > +++ b/cmd/bcb.c > > > > @@ -46,9 +46,6 @@ static int bc

[U-Boot] [PULL] u-boot-stm32/master for v2019.10-rc1: u-boot-stm32-20190712

2019-07-12 Thread Patrick DELAUNAY
Hi Tom please pull the STM32 related patches for v2019.10-rc1 = u-boot-stm32-20190712 Travis CI status: https://travis-ci.org/patrickdelaunay/u-boot/builds/557765098 the 4 issues are not related to the patchsets : test_ut[ut_dm_pci_ep_base] Thanks, Patrick The following

Re: [U-Boot] [PATCH v2 3/5] cmd: bcb: Fix duplicated handling in two case-branches

2019-07-12 Thread Eugeniu Rosca
On Fri, Jul 12, 2019 at 05:01:09PM +0300, Sam Protsenko wrote: > > > diff --git a/cmd/bcb.c b/cmd/bcb.c > > > index 2bd5a744deb5..3b1c7434e287 100644 > > > --- a/cmd/bcb.c > > > +++ b/cmd/bcb.c > > > @@ -46,9 +46,6 @@ static int bcb_is_misused(int argc, char *const argv[]) > > > > > > switc

Re: [U-Boot] [PATCH] sf: Add support for Macronix mx25u12835f

2019-07-12 Thread Vladimir Vid
Sorry for the late reply, I got caught up working on another project and overlooked this. Yes, per my testing mx25u12835f is working stable without the SECT_4K option. I also found that Xilinx is using the same flash and also without the SECT_4K, so I don't think this should not be an issue.

Re: [U-Boot] [PATCH 3/3] spl: fit: Place OS_BOOT DTB at CONFIG_SYS_SPL_ARGS_ADDR if defined

2019-07-12 Thread Tom Rini
On Tue, May 28, 2019 at 03:49:15AM +0200, Marek Vasut wrote: > If SPL OS_BOOT is enabled and CONFIG_SYS_SPL_ARGS_ADDR is defined, > place the kernel DTB at that location, so the rest of the OS_BOOT > machinery in SPL has the DTB blob at the correct location. > > Signed-off-by: Marek Vasut > Cc:

Re: [U-Boot] [PATCH v3 1/7] usb_kdb: only process events succesfully received

2019-07-12 Thread Michal Suchánek
On Fri, 12 Jul 2019 06:05:45 +0200 Marek Vasut wrote: > On 7/10/19 5:47 PM, Michal Suchánek wrote: > > On Fri, 5 Jul 2019 14:12:36 +0200 > > Marek Vasut wrote: > > > >> On 7/5/19 12:44 PM, Michal Suchanek wrote: > >>> Causes unbound key repeat on error otherwise. > >>> > >>> Signed-off-by:

[U-Boot] [PATCH v2 2/3] rockchip: rk3399: Enable CONFIG_MISC_INIT_R for the Rock PI 4

2019-07-12 Thread Rohan Garg
This enables us to set a static MAC address Signed-off-by: Rohan Garg --- configs/rock-pi-4-rk3399_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/rock-pi-4-rk3399_defconfig b/configs/rock-pi-4-rk3399_defconfig index be670df23f..dc84ded86c 100644 --- a/configs/rock-pi-4-r

[U-Boot] [PATCH v2 3/3] board: puma: Use rockchip_* helpers to setup cpuid and macaddr

2019-07-12 Thread Rohan Garg
We should use the shared helpers to setup the necessary parts Signed-off-by: Rohan Garg --- .../puma_rk3399/puma-rk3399.c | 98 ++- 1 file changed, 8 insertions(+), 90 deletions(-) diff --git a/board/theobroma-systems/puma_rk3399/puma-rk3399.c b/board/theobroma

[U-Boot] [PATCH v2 1/3] rockchip: rk3399: derive ethaddr from cpuid

2019-07-12 Thread Rohan Garg
Generate a MAC address based on the cpuid available in the efuse block: Use the first 6 byte of the cpuid's SHA256 hash and set the locally administered bits. Also ensure that the multicast bit is cleared. The MAC address is only generated and set if there is no ethaddr present in the saved enviro

[U-Boot] [PATCH 2/2] doc: bindings: Add binding for register driven MDIO muxes

2019-07-12 Thread Alex Marginean
This binding documents two properties that describe the registers used to perform MUX selection. Signed-off-by: Alex Marginean --- doc/device-tree-bindings/net/mdio-mux-reg.txt | 64 +++ 1 file changed, 64 insertions(+) create mode 100644 doc/device-tree-bindings/net/mdio-mux-re

[U-Boot] [PATCH 1/2] drivers: net: driver for MDIO muxes controlled over I2C

2019-07-12 Thread Alex Marginean
This driver is used for MDIO muxes driven over I2C. This is currently used on Freescale LS1028A QDS board, on which the physical MDIO MUX is controlled by an on-board FPGA which in turn is configured through I2C. Signed-off-by: Alex Marginean --- Depends on https://patchwork.ozlabs.org/project/

Re: [U-Boot] [PATCH 20/20] spi: stm32: Fix warnings when compiling with W=1

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves the following warnings: > > drivers/spi/stm32_spi.c: In function 'stm32_spi_write_txfifo': > drivers/spi/stm32_spi.c:116:20: warning: comparison between signed and > unsigned integer expressions [-Wsign-compa

Re: [U-Boot] [PATCH 19/20] psci: Fix warnings when compiling with W=1

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves the following warnings: > arch/arm/mach-stm32mp/psci.c: > > warning: no previous prototype for ‘psci_set_state’ [-Wmissing-prototypes] > warning: no previous prototype for ‘psci_arch_cpu_entry’ > [-Wmissing-

Re: [U-Boot] [PATCH 05/20] stm32mp1: bsec: Fix warnings when compiling with W=1

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves the following warnings: > > arch/arm/mach-stm32mp/bsec.c: In function 'stm32mp_bsec_read': > warning: comparison between signed and unsigned integer expressions [-Wsign- > compare] > if (offset >= STM32_BSE

Re: [U-Boot] [PATCH 18/20] cmd: pinmux: Fix warnings when compiling with W=1

2019-07-12 Thread Patrick DELAUNAY
Hi > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves the following warnings: > > cmd/pinmux.c: In function 'do_dev': > cmd/pinmux.c:26:6: warning: this statement may fall through [-Wimplicit- > fallthrough=] >if (ret) { > ^ > cmd/pinmux.c:30:2: note: h

Re: [U-Boot] [PATCH 11/20] power: regulator: stm32: Fix warnings when compiling with W=1

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves the following warnings: > > drivers/power/regulator/stm32-vrefbuf.c: In function > 'stm32_vrefbuf_set_value': > warning: comparison between signed and unsigned integer expressions [-Wsign- > compare] >if

Re: [U-Boot] [PATCH 10/20] clk: clk_stm32mp1: Fix warnings when compiling with W=1

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves the following warnings: > > drivers/clk/clk_stm32mp1.c: In function 'stm32mp1_clk_get_parent': > warning: comparison between signed and unsigned integer expressions [-Wsign- > compare] > for (i = 0; i < ARR

Re: [U-Boot] [PATCH 17/20] spi: stm32_qspi: avoid warnings when building with W=1 option

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves warnings detected by setting W=1 when building. > > Warnings type detected: > - [-Wtype-limits] > - [-Wsign-compare] > > Signed-off-by: Christophe Kerello > Signed-off-by: Patrick Delaunay > --- > > dr

Re: [U-Boot] [PATCH 13/20] ram: stm32mp1_ram: Fix warnings when compiling with W=1

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves the following warnings: > > drivers/ram/stm32mp1/stm32mp1_ram.c: In function 'stm32mp1_ddr_clk_enable': > warning: comparison between signed and unsigned integer expressions [-Wsign- > compare] > for (idx =

Re: [U-Boot] [PATCH 15/20] power: stpmic1: Fix warnings when compiling with W=1

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves the following warnings: > warning: this statement may fall through [-Wimplicit-fallthrough=] > > Signed-off-by: Patrick Delaunay > --- > > drivers/power/regulator/stpmic1.c | 1 + > 1 file changed, 1 inser

Re: [U-Boot] [PATCH 16/20] mtd: rawnand: stm32_fmc2: avoid warnings when building with W=1 option

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves warnings detected by setting W=1 when building. > > Warnings type detected: > - [-Wsign-compare] > - [-Wtype-limits] > > Signed-off-by: Christophe Kerello > Signed-off-by: Patrick Delaunay > --- > > dr

Re: [U-Boot] [PATCH 09/20] i2c: stm32f7_i2c: Fix warnings when compiling with W=1

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves the following warnings: > > drivers/i2c/stm32f7_i2c.c: In function 'stm32_i2c_compute_solutions': > warning: comparison between signed and unsigned integer expressions [-Wsign- > compare] > if (scldel < s

Re: [U-Boot] [PATCH 01/20] stm32mp1: Fix warnings when compiling with W=1

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves the following warnings: > > arch/arm/mach-stm32mp/cpu.c:378:16: warning: comparison between signed and > unsigned integer expressions [-Wsign-compare] >if (instance > ARRAY_SIZE(serial_addr)) >

Re: [U-Boot] [PATCH 08/20] gpio: stm32_gpio: Fix warnings when compiling with W=1

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves the following warnings: > > drivers/gpio/stm32_gpio.c: In function 'stm32_offset_to_index': > : comparison between signed and unsigned integer expressions [-Wsign-compare] > if (idx == offset) >

Re: [U-Boot] [PATCH 06/20] adc: stm32-adc: Fix warnings when compiling with W=1

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves the following warnings: > > drivers/adc/stm32-adc.c: In function 'stm32_adc_chan_of_init': > warning: comparison between signed and unsigned integer expressions [-Wsign- > compare] > if (num_channels > adc-

Re: [U-Boot] [PATCH v2 3/5] cmd: bcb: Fix duplicated handling in two case-branches

2019-07-12 Thread Sam Protsenko
On Fri, Jul 12, 2019 at 4:37 PM Sam Protsenko wrote: > > On Fri, Jul 12, 2019 at 3:50 PM Eugeniu Rosca wrote: > > > > Fix warning V1037 reported by PVS-Studio Static Analyzer: > > Two or more case-branches perform the same actions. Check lines: 49, 53 > > > > Fixes: db7b7a05b267 ("cmd: Add 'bcb'

Re: [U-Boot] [PATCH 07/20] adc: stm32: Fix warnings when compiling with W=1

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves the following warnings: > > drivers/adc/stm32-adc-core.c: In function 'stm32h7_adc_clk_sel': > warning: comparison between signed and unsigned integer expressions [-Wsign- > compare] >for (i = 0; i < ARRA

Re: [U-Boot] [PATCH 14/20] pinctrl: pinctrl_stm32: Fix warnings when compiling with W=1

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves the following warnings: > > warning: comparison of unsigned expression < 0 is always false [-Wtype-limits] > if (*idx < 0) > ^ > drivers/pinctrl/pinctrl_stm32.c: At top level: > warning: no p

Re: [U-Boot] [PATCH 12/20] misc: stm32_fuse: Fix warnings when compiling with W=1

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves the following warnings: > > warning: no previous prototype for 'fuse_read' [-Wmissing-prototypes] int > fuse_read(u32 bank, u32 word, u32 *val) > ^ > CC cmd/sf.o > warning: no previous pr

Re: [U-Boot] [PATCH 04/20] mmc: stm32_sdmmc2: avoid warnings when building with W=1 option

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves warnings detected by setting W=1 when building. > > Warnings type detected: > - [-Wmissing-prototypes] > - [-Wimplicit-fallthrough=] > > Signed-off-by: Christophe Kerello > Signed-off-by: Patrick Delaunay

Re: [U-Boot] [PATCH 03/20] serial: stm32: Fix warnings when compiling with W=1

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves the following warnings: > > drivers/serial/serial_stm32.c: In function 'stm32_serial_probe': > warning: comparison of unsigned expression < 0 is always false [-Wtype-limits] > if (plat->clock_rate < 0) { >

Re: [U-Boot] [PATCH 02/20] board: stm32mp1: Fix warnings when compiling with W=1

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrick DELAUNAY > Sent: vendredi 21 juin 2019 15:27 > > This patch solves the following warnings: > > warning: no previous prototype for 'board_quiesce_devices' [-Wmissing- > prototypes] void board_quiesce_devices(void) > ^ > > Signed-off-by: Patrice Cho

Re: [U-Boot] [PATCH 2/2] mx6: mx6sabre common: Add distro boot config

2019-07-12 Thread Fabio Estevam
Hi Sjoerd, Subject says 2/2, but I don't see a 1/2. On Thu, Jun 20, 2019 at 5:35 PM Sjoerd Simons wrote: > > Allow mx6 sabreauto and sabresd to boot via the default distro boot > commands to make it easier to support general distributions. The > previous boot setup is still supported via legacy

Re: [U-Boot] [PATCH v2 5/5] cmd: bcb: Apply non-functional refinements

2019-07-12 Thread Sam Protsenko
On Fri, Jul 12, 2019 at 3:52 PM Eugeniu Rosca wrote: > > These have been reported by Simon in [1] and fixed in [2]. > However, since [1] has already been pushed to u-boot/master, the > improvements incorporated in [2] are now extracted and resubmitted. > > The changes are in the area of coding sty

Re: [U-Boot] [PATCH v2 4/5] cmd: bcb: Use strcmp() instead of strncmp() for string literals

2019-07-12 Thread Sam Protsenko
On Fri, Jul 12, 2019 at 3:50 PM Eugeniu Rosca wrote: > > Quote from https://patchwork.ozlabs.org/patch/1104244/#2210814: > > --8<--- > strncmp() is chosen for the sake of paranoid/defensive programming. > Indeed, strncmp() is not really needed when comparing a variable > with a st

Re: [U-Boot] [PATCH v2 3/5] cmd: bcb: Fix duplicated handling in two case-branches

2019-07-12 Thread Sam Protsenko
On Fri, Jul 12, 2019 at 3:50 PM Eugeniu Rosca wrote: > > Fix warning V1037 reported by PVS-Studio Static Analyzer: > Two or more case-branches perform the same actions. Check lines: 49, 53 > > Fixes: db7b7a05b267 ("cmd: Add 'bcb' command to read/modify/write BCB fields") > Signed-off-by: Eugeniu R

Re: [U-Boot] [PATCH v2 2/5] treewide: Fix stale references of Android docs

2019-07-12 Thread Sam Protsenko
On Fri, Jul 12, 2019 at 3:50 PM Eugeniu Rosca wrote: > > Commit 9bdf0e8fef86 ("doc: relocate/rename Android README and add BCB > overview") left some obsolete references of Android documents/paths. > > This has been pointed out by Sam (thanks!) in: > https://patchwork.ozlabs.org/patch/1104245/#220

Re: [U-Boot] [PATCH v2 1/5] doc: Move README.android-fastboot-protocol to doc/android/

2019-07-12 Thread Sam Protsenko
Reviewed-by: Sam Protsenko On Fri, Jul 12, 2019 at 3:50 PM Eugeniu Rosca wrote: > > Commit 9bdf0e8fef86 ("doc: relocate/rename Android README and add BCB > overview") missed relocating README.android-fastboot-protocol. Fix it. > > Fixes: 9bdf0e8fef86 ("doc: relocate/rename Android README and add

Re: [U-Boot] [RFC 4/4] colibri_vf: enable DM_VIDEO

2019-07-12 Thread Oleksandr Suvorov
On Thu, 11 Jul 2019 at 17:28, Igor Opaniuk wrote: > > + Oleksandr, Philippe > > On Mon, Jun 10, 2019 at 2:48 PM Igor Opaniuk wrote: > > > > From: Igor Opaniuk > > > > Enable DM_VIDEO for Colibri VF. > > > > Signed-off-by: Igor Opaniuk Reviewed-by: Oleksandr Suvorov > > --- > > configs/colib

Re: [U-Boot] [RFC 3/4] ARM: dts: colibri_vf: Add dcu0 node

2019-07-12 Thread Oleksandr Suvorov
On Thu, 11 Jul 2019 at 17:27, Igor Opaniuk wrote: > > + Oleksandr, Philippe > > On Mon, Jun 10, 2019 at 2:48 PM Igor Opaniuk wrote: > > > > From: Igor Opaniuk > > > > Add dumb node for NXP Display Control Unit0(DCU), which permits DM_ENABLED > > converted driver to be probed. Currently no displa

Re: [U-Boot] [RFC 2/4] video: fsl_dcu_fb: add DM_VIDEO support

2019-07-12 Thread Oleksandr Suvorov
On Thu, 11 Jul 2019 at 17:27, Igor Opaniuk wrote: > > + Oleksandr, Philippe > > On Mon, Jun 10, 2019 at 2:47 PM Igor Opaniuk wrote: > > > > From: Igor Opaniuk > > > > Extend the driver to build with DM_VIDEO enabled. DTS files > > must additionally include 'u-boot,dm-pre-reloc' property in > > s

Re: [U-Boot] Please pull u-boot-dm

2019-07-12 Thread Tom Rini
On Thu, Jul 11, 2019 at 11:18:43AM -0600, Simon Glass wrote: > Hi Tom, > > I picked up a different patch for the unicode handling. > > Passing build here: > https://travis-ci.org/sglass68/u-boot/builds/557041198 > > > The following changes since commit ef8b7e045ec744dce385cac4b1438c9be6e2bbc8:

Re: [U-Boot] Please pull u-boot-marvell/master

2019-07-12 Thread Tom Rini
On Thu, Jul 11, 2019 at 06:01:47PM +0200, Stefan Roese wrote: > Hi Tom, > > please pull the following Marvell related patches: > Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature ___ U-Boot mailing list U-Boot@lists

Re: [U-Boot] [GIT PULL] UniPhier SoC updates for v2019.10-rc1

2019-07-12 Thread Tom Rini
On Thu, Jul 11, 2019 at 11:19:43PM +0900, Masahiro Yamada wrote: > Hi Tom, > > Here is a pretty big bulk of changes for -rc1. Please pull! > > > > The following changes since commit b4944e25744aa42dc23c6912e1cd4ccad270c4e0: > > Merge tag 'u-boot-atmel-2019.10-a' of > https://gitlab.denx.de/

Re: [U-Boot] [PATCH 2/5] treewide: Fix stale references of Android docs

2019-07-12 Thread Eugeniu Rosca
Hi Sam, On Fri, Jul 12, 2019 at 04:25:11PM +0300, Sam Protsenko wrote: > Reviewed-by: Sam Protsenko Please, discard the v1 [*] and consider v2 [**] instead. There should be a clear protocol to communicate that to the maintainers, such that we avoid seeing the v1 applied to master. I placed a co

Re: [U-Boot] [PATCH 1/1] efi_selftest: do not call CloseEvent() after ExitBootServices()

2019-07-12 Thread Oleksandr Suvorov
On Fri, 12 Jul 2019 at 14:04, Heinrich Schuchardt wrote: > > Boot services are not available after calling ExitBootServices(). So we > should not try to close an event here. > > Signed-off-by: Heinrich Schuchardt Reviewed-by: Oleksandr Suvorov > --- > .../efi_selftest_exitbootservices.c

Re: [U-Boot] [PATCH] env: ti: boot: Handle reboot reason from BCB

2019-07-12 Thread Eugeniu Rosca
Hi Sam, On Fri, Jul 12, 2019 at 04:18:20PM +0300, Sam Protsenko wrote: > Let's finalize modern Android boot flow via scripting (as we need all > those commands anyway), then we can think if we need to wrap the whole > thing into boot_android command. There is a lot of stuff happening > during the

Re: [U-Boot] [PATCH 1/5] doc: Move README.android-fastboot-protocol to doc/android/

2019-07-12 Thread Sam Protsenko
On Fri, Jul 12, 2019 at 2:23 PM Igor Opaniuk wrote: > > Hi Eugeniu, > > On Fri, Jul 12, 2019 at 2:21 PM Eugeniu Rosca wrote: > > > > Commit 9bdf0e8fef86 ("doc: relocate/rename Android README and add BCB > > overview") missed relocating README.android-fastboot-protocol. Fix it. > > > > Fixes: 9bdf

Re: [U-Boot] [PATCH 2/5] treewide: Fix stale references of Android docs

2019-07-12 Thread Sam Protsenko
On Fri, Jul 12, 2019 at 2:25 PM Igor Opaniuk wrote: > > On Fri, Jul 12, 2019 at 2:21 PM Eugeniu Rosca wrote: > > > > Commit 9bdf0e8fef86 ("doc: relocate/rename Android README and add BCB > > overview") left some obsolete references of Android documents/paths. > > > > This has been pointed out by

Re: [U-Boot] [PATCH v4 0/4] Add 'bcb' command to read/modify/write Android BCB

2019-07-12 Thread Sam Protsenko
Hi Eugeniu, On Fri, Jul 12, 2019 at 12:20 PM Eugeniu Rosca wrote: > > Hi Sam, > > On Tue, Jul 09, 2019 at 03:20:01PM +0300, Sam Protsenko wrote: > > Tom, Simon, > > > > Can we please handle this series and apply it to master? A lot of > > Android-related patches depend on this, so if all comments

Re: [U-Boot] [PATCH] env: ti: boot: Handle reboot reason from BCB

2019-07-12 Thread Sam Protsenko
Hi Eugeniu, On Fri, Jul 12, 2019 at 12:39 AM Eugeniu Rosca wrote: > > Hi Sam, > > On Tue, Jul 09, 2019 at 05:45:43PM +0300, Sam Protsenko wrote: > > "emmc_android_boot=" \ > > + "if bcb load " __stringify(CONFIG_FASTBOOT_FLASH_MMC_DEV) " misc; " \ > > + "then " \ > > + "

Re: [U-Boot] [PATCH] board: st: add missing gpio_request() for stm32f429-discovery

2019-07-12 Thread Patrick DELAUNAY
Hi, > From: Patrice CHOTARD > Sent: mercredi 26 juin 2019 10:27 > > This fixes the following error message: > > U-Boot 2019.07-rc4-00103-g5eea874b5e (Jun 25 2019 - 15:09:31 +0200) > > DRAM: 8 MiB > gpio@40021800: dir_output: error: gpio GPIOG14 not reserved > gpio@40021800: dir_output: error:

  1   2   3   >