Re: [U-Boot] [UBOOT] [PATCH] cmd: usb: ignore block devices under mass storage device

2017-08-31 Thread Lothar Waßmann
Hi, On Thu, 10 Aug 2017 22:53:31 -0700 Suneel Garapati wrote: > usb tree and info commands may cause crash otherwise > > Signed-off-by: Suneel Garapati > --- > cmd/usb.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/cmd/usb.c b/cmd/usb.c > index 992d414..81e1a7

Re: [U-Boot] [PATCH 2/2] ARM: Add a new arch + board for QEMU's 'virt' machine

2017-08-31 Thread Bin Meng
Hi Tuomas, On Fri, Sep 1, 2017 at 5:56 AM, Tuomas Tynkkynen wrote: > Hi, > > > On 08/31/2017 09:55 AM, Bin Meng wrote: >> >> Hi Tuomas, >> >> On Wed, Aug 30, 2017 at 4:31 PM, Tuomas Tynkkynen >> wrote: >>> >>> This board builds an U-Boot binary that is bootable with QEMU's 'virt' >>> machine on

Re: [U-Boot] [PATCH 03/23] efi_loader: support 16 protocols per efi_object

2017-08-31 Thread Heinrich Schuchardt
On 08/31/2017 04:01 PM, Alexander Graf wrote: > On 08/27/2017 12:51 AM, Heinrich Schuchardt wrote: >> 8 protocols per efi_object is insufficient for iPXE. >> >> Signed-off-by: Heinrich Schuchardt >> --- >> include/efi_loader.h | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> d

Re: [U-Boot] [PATCH v3 1/8] spi: add support of SPI flash commands

2017-08-31 Thread Wenyou.Yang
Hi Jagan, Thank you for your comments. I will rework this patch set according to your advice. Thank you! Best Regards, Wenyou Yang > -Original Message- > From: Jagan Teki [mailto:jagannadh.t...@gmail.com] > Sent: 2017年8月30日 21:50 > To: Wenyou Yang - A41535 > Cc: U-Boot Mailing List ;

Re: [U-Boot] [PATCH V2] imx: mx7: Add support for USB and normal boot modes

2017-08-31 Thread Troy Kisky
On 8/31/2017 4:04 PM, Eric Nelson wrote: > On 08/31/2017 03:53 PM, Troy Kisky wrote: >> On 8/31/2017 2:28 PM, Troy Kisky wrote: >> >>> >>> Maybe if you change the WDOG pinmux it might work ? >>> >>> >> >> Worked for me >> >> => mm.l 302c >> 302c: 0003 ? 0 >> 302c0004: 0001 ? q >> =>

Re: [U-Boot] [PATCH V2] imx: mx7: Add support for USB and normal boot modes

2017-08-31 Thread Eric Nelson
On 08/31/2017 04:11 PM, Fabio Estevam wrote: Troy, On Thu, Aug 31, 2017 at 7:53 PM, Troy Kisky wrote: On 8/31/2017 2:28 PM, Troy Kisky wrote: Maybe if you change the WDOG pinmux it might work ? Worked for me => mm.l 302c 302c: 0003 ? 0 302c0004: 0001 ? q => bmod usb res

Re: [U-Boot] [PATCH V2] imx: mx7: Add support for USB and normal boot modes

2017-08-31 Thread Fabio Estevam
Troy, On Thu, Aug 31, 2017 at 7:53 PM, Troy Kisky wrote: > On 8/31/2017 2:28 PM, Troy Kisky wrote: > >> >> Maybe if you change the WDOG pinmux it might work ? >> >> > > Worked for me > > => mm.l 302c > 302c: 0003 ? 0 > 302c0004: 0001 ? q > => bmod usb > resetting ... > Ok, but di

Re: [U-Boot] [PATCH] arm64: ls1043ardb: Add distro secure boot support

2017-08-31 Thread York Sun
On 06/04/2017 11:54 PM, Sumit Garg wrote: > Enable validation of boot.scr script prior to its execution dependent > on "secureboot" flag in environment. Disable fall back option to > nor/qspi boot in case of secure boot. Also enable "secureboot=y" > flag in environment for ARM based platforms inste

Re: [U-Boot] [PATCH V2] imx: mx7: Add support for USB and normal boot modes

2017-08-31 Thread Eric Nelson
On 08/31/2017 03:53 PM, Troy Kisky wrote: On 8/31/2017 2:28 PM, Troy Kisky wrote: Maybe if you change the WDOG pinmux it might work ? Worked for me => mm.l 302c 302c: 0003 ? 0 302c0004: 0001 ? q => bmod usb resetting ... Hmm... On SABRE-SD or Nitrogen7? ___

Re: [U-Boot] [PATCH V2] imx: mx7: Add support for USB and normal boot modes

2017-08-31 Thread Eric Nelson
Thanks Troy (and Peng), On 08/31/2017 02:28 PM, Troy Kisky wrote: On 8/31/2017 6:56 AM, Fabio Estevam wrote: On Thu, Aug 31, 2017 at 10:35 AM, Fabio Estevam wrote: On Wed, Aug 30, 2017 at 7:13 PM, Eric Nelson wrote: This adds support for two additional boot modes on the i.MX7D SoC, which is

Re: [U-Boot] [PATCH V2] imx: mx7: Add support for USB and normal boot modes

2017-08-31 Thread Troy Kisky
On 8/31/2017 2:28 PM, Troy Kisky wrote: > > Maybe if you change the WDOG pinmux it might work ? > > Worked for me => mm.l 302c 302c: 0003 ? 0 302c0004: 0001 ? q => bmod usb resetting ... ___ U-Boot mailing list U-Boot@lists.denx.de

Re: [U-Boot] [PATCH 03/23] efi_loader: support 16 protocols per efi_object

2017-08-31 Thread Alexander Graf
On 08/27/2017 12:51 AM, Heinrich Schuchardt wrote: 8 protocols per efi_object is insufficient for iPXE. Signed-off-by: Heinrich Schuchardt --- include/efi_loader.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/efi_loader.h b/include/efi_loader.h index 6f71a6

Re: [U-Boot] [PATCH V2] imx: mx7: Add support for USB and normal boot modes

2017-08-31 Thread Troy Kisky
On 8/31/2017 6:56 AM, Fabio Estevam wrote: > On Thu, Aug 31, 2017 at 10:35 AM, Fabio Estevam wrote: >> Hi Eric, >> >> On Wed, Aug 30, 2017 at 7:13 PM, Eric Nelson wrote: >>> This adds support for two additional boot modes on the i.MX7D SoC, which >>> is most useful when doing U-Boot development o

Re: [U-Boot] [PATCH 01/23] efi_loader: allow return value in EFI_CALL

2017-08-31 Thread Alexander Graf
On 08/31/2017 02:51 PM, Simon Glass wrote: On 27 August 2017 at 06:51, Heinrich Schuchardt wrote: Macro EFI_CALL was introduced to call an UEFI function. Should this be 'an EFI'. Or 'a UEFI'? In a nutshell, EFI is the v1, proprietary, for pay, closed protocol while UEFI is v2 and what edk2

Re: [U-Boot] [PATCH 1/1] usb: gadget: g_dnl: Use serial# variable to set g_dnl_serial

2017-08-31 Thread Łukasz Majewski
On 08/31/2017 08:45 PM, Sam Protsenko wrote: On 31 August 2017 at 21:20, Sam Protsenko wrote: Since 842778a09104 commit, "fastboot devices" stopped to show correct device serial number for TI boards, showing this line instead: fastboot This is because serial# env varia

Re: [U-Boot] [PATCH v2] dm: core: Add livetree documentation

2017-08-31 Thread Łukasz Majewski
On 08/31/2017 02:00 PM, Simon Glass wrote: On 31 August 2017 at 19:59, Simon Glass wrote: Add some documentation for the live device tree support in U-Boot. This was missing from the initial series. Signed-off-by: Simon Glass Suggested-by: Lukasz Majewski Reviewed-by: Łukasz Majewski (I

Re: [U-Boot] [PATCH 1/1] usb: gadget: g_dnl: Use serial# variable to set g_dnl_serial

2017-08-31 Thread Sam Protsenko
On 31 August 2017 at 21:20, Sam Protsenko wrote: > Since 842778a09104 commit, "fastboot devices" stopped to show correct > device serial number for TI boards, showing this line instead: > > fastboot > > This is because serial# env variable could be set after g_dnl gadget wa

Re: [U-Boot] [PATCH v2] imx: imx7d: remove CamelCase from ENET_xMHz macros

2017-08-31 Thread Stefano Babic
Hi Eric, On 31/08/2017 17:34, Eric Nelson wrote: > Update these macros to use all upper-case to avoid checkpatch > warnings: > > ENET_25MHz, > ENET_50MHz, > ENET_125MHz, > > Signed-off-by: Eric Nelson > --- > V2 fixes a couple of references in mx7/clock.c > arch/arm/include/a

[U-Boot] [PATCH 1/1] usb: gadget: g_dnl: Use serial# variable to set g_dnl_serial

2017-08-31 Thread Sam Protsenko
Since 842778a09104 commit, "fastboot devices" stopped to show correct device serial number for TI boards, showing this line instead: fastboot This is because serial# env variable could be set after g_dnl gadget was initialized (e.g. by using env_set() in the board file).

Re: [U-Boot] [ANN] U-Boot v2017.09-rc3 released

2017-08-31 Thread Sam Protsenko
On 29 August 2017 at 05:56, Tom Rini wrote: > Hey all, > > It's release day and v2017.09-rc3 is out. I think there's still a few > outstanding important bugfixes that need to come in, and probably a few > more Kconfig migrations as well. > > If anyone has critical fixes I've missed please speak u

Re: [U-Boot] [ANN] U-Boot v2017.09-rc3 released

2017-08-31 Thread Peter Robinson
On Tue, Aug 29, 2017 at 5:44 AM, Jonathan Gray wrote: > On Mon, Aug 28, 2017 at 10:56:52PM -0400, Tom Rini wrote: >> Hey all, >> >> It's release day and v2017.09-rc3 is out. I think there's still a few >> outstanding important bugfixes that need to come in, and probably a few >> more Kconfig migr

Re: [U-Boot] [PATCH 12/23] efi_loader: implement OpenProtocolInformation

2017-08-31 Thread Heinrich Schuchardt
On 08/31/2017 02:51 PM, Simon Glass wrote: > Hi Heinrich, > > On 27 August 2017 at 06:53, Heinrich Schuchardt wrote: >> efi_open_protocol_information provides the agent and controller >> handles as well as the attributes and open count of an protocol >> on a handle. >> >> Cc: Rob Clark >> Signed

Re: [U-Boot] ** Can't read Driver Desriptor Block **

2017-08-31 Thread Heinrich Schuchardt
On 08/31/2017 02:55 PM, Bin Meng wrote: > Hi Simon, > > On Thu, Aug 31, 2017 at 8:52 PM, Simon Glass wrote: >> Hi Bin, >> >> On 31 August 2017 at 10:53, Bin Meng wrote: >>> Hi Heinrich, >>> >>> On Thu, Aug 31, 2017 at 5:19 AM, Heinrich Schuchardt >>> wrote: On 08/30/2017 06:37 AM, Heinric

Re: [U-Boot] efi_loader: GOP fix for no display

2017-08-31 Thread Alexander Graf
> uclass_first_device() returns 0 if there is no device, but error if > there is a device that failed to probe. > > Signed-off-by: Rob Clark Thanks, applied to efi-next Alex ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/

Re: [U-Boot] [U-Boot, v2] efi_loader: LocateHandle should return EFI_NOT_FOUND if none found

2017-08-31 Thread Alexander Graf
> Spotted this debugging OpenBSD's bootloader in qemu. (Wouldn't really > fix anything, the problem was not having any disks, but we should > probably return the correct error code.) > > Signed-off-by: Rob Clark > Reviewed-by: Heinrich Schuchardt Thanks, applied to efi-next Alex

[U-Boot] [PATCH v2] imx: imx7d: remove CamelCase from ENET_xMHz macros

2017-08-31 Thread Eric Nelson
Update these macros to use all upper-case to avoid checkpatch warnings: ENET_25MHz, ENET_50MHz, ENET_125MHz, Signed-off-by: Eric Nelson --- V2 fixes a couple of references in mx7/clock.c arch/arm/include/asm/arch-mx7/clock.h | 6 +++--- arch/arm/mach-imx/mx7/clock.c

Re: [U-Boot] [PATCH v3 1/2] armv8: fsl-lsch3: Add SD boot support for LS1088ARDB

2017-08-31 Thread York Sun
On 08/31/2017 04:14 AM, Ashish Kumar wrote: > Add SD boot support for LS1088ARDB > > Signed-off-by: Prabhakar Kushwaha > Signed-off-by: Ashish Kumar > Signed-off-by: Raghav Dogra > --- > v3: > Depends upon > http://patchwork.ozlabs.org/patch/794217/ You know your dependency is not going to be

Re: [U-Boot] [PATCH v2 2/2] armv8: mmu: add space around operator

2017-08-31 Thread York Sun
On 08/17/2017 12:56 AM, Andy Yan wrote: > Add space around operator "+", make it > match the coding style. > > Signed-off-by: Andy Yan > --- > > Changes in v2: None Reviewed-by: York Sun ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx

Re: [U-Boot] [PATCH v2 1/2] armv8: mmu: remove unused macro definition

2017-08-31 Thread York Sun
On 08/17/2017 12:55 AM, Andy Yan wrote: > Macro VA_BITS and PTE_BLOCK_BITS are not used > in the code, so remove them. > > Signed-off-by: Andy Yan > Reviewed-by: Philipp Tomsich > > --- > > Changes in v2: > - remove CONFIG_SYS_PTL2_BITS from the whitelist > > arch/arm/include/asm/armv8/mmu.

Re: [U-Boot] [PATCH 00/23] efi_loader implement missing functions

2017-08-31 Thread Leif Lindholm
On Wed, Aug 30, 2017 at 08:59:31AM +0100, Leif Lindholm wrote: > On Wed, Aug 30, 2017 at 12:03:16AM +0200, Heinrich Schuchardt wrote: > > I am able to provide a test application that will cover the API > > functions that I have focused on (events, protocols, (dis)connect > > controllers). To limit

Re: [U-Boot] [PATCH V2] imx: mx7: Add support for USB and normal boot modes

2017-08-31 Thread Eric Nelson
Hi Fabio, On 08/31/2017 06:56 AM, Fabio Estevam wrote: On Thu, Aug 31, 2017 at 10:35 AM, Fabio Estevam wrote: Hi Eric, On Wed, Aug 30, 2017 at 7:13 PM, Eric Nelson wrote: This adds support for two additional boot modes on the i.MX7D SoC, which is most useful when doing U-Boot development on

Re: [U-Boot] [PATCH v2] mtd/spi: Add MT35XU512ABA1G12 NOR flash support

2017-08-31 Thread Jagan Teki
On Thu, Aug 31, 2017 at 10:26 AM, Yogesh Gaur wrote: > Add MT35XU512ABA1G12 parameters to NOR flash parameters array. > > The MT35XU512ABA1G12 only supports 1 bit mode and 8 bits. It can't support > dual and quad. Supports subsector erase with 4KB granularity, have support > of FSR(flag status reg

Re: [U-Boot] [PATCH] sunxi: rename Bananapi M3 dts file name

2017-08-31 Thread Jagan Teki
On Thu, Aug 31, 2017 at 7:35 PM, Maxime Ripard wrote: > On Thu, Aug 31, 2017 at 10:01:40PM +0800, Chen-Yu Tsai wrote: >> The upstream (Linux) device tree file for the Bananapi M3 follows the >> convention of using the well known brand name, instead of the vendor >> name, for naming. The file was r

Re: [U-Boot] [PATCH] mmc: sunxi: Only update timing mode bit when enabling new timing mode

2017-08-31 Thread Jagan Teki
On Thu, Aug 31, 2017 at 7:34 PM, Maxime Ripard wrote: > On Thu, Aug 31, 2017 at 09:57:48PM +0800, Chen-Yu Tsai wrote: >> When enabling the new mmc timing mode, we inadvertently clear all the >> remaining bits in the new timing mode register. The bits cleared >> include a default phase delay on the

Re: [U-Boot] [PATCH] sunxi: rename Bananapi M3 dts file name

2017-08-31 Thread Maxime Ripard
On Thu, Aug 31, 2017 at 10:01:40PM +0800, Chen-Yu Tsai wrote: > The upstream (Linux) device tree file for the Bananapi M3 follows the > convention of using the well known brand name, instead of the vendor > name, for naming. The file was recently added to upstream in commit > 359b5a1e1c2d ("ARM: su

Re: [U-Boot] [PATCH] mmc: sunxi: Only update timing mode bit when enabling new timing mode

2017-08-31 Thread Maxime Ripard
On Thu, Aug 31, 2017 at 09:57:48PM +0800, Chen-Yu Tsai wrote: > When enabling the new mmc timing mode, we inadvertently clear all the > remaining bits in the new timing mode register. The bits cleared > include a default phase delay on the output clock. The BSP kernel > states that the default valu

[U-Boot] [PATCH] sunxi: rename Bananapi M3 dts file name

2017-08-31 Thread Chen-Yu Tsai
The upstream (Linux) device tree file for the Bananapi M3 follows the convention of using the well known brand name, instead of the vendor name, for naming. The file was recently added to upstream in commit 359b5a1e1c2d ("ARM: sun8i: a83t: Add device tree for Sinovoip Bananapi BPI-M3") Rename the

[U-Boot] [PATCH] mmc: sunxi: Only update timing mode bit when enabling new timing mode

2017-08-31 Thread Chen-Yu Tsai
When enabling the new mmc timing mode, we inadvertently clear all the remaining bits in the new timing mode register. The bits cleared include a default phase delay on the output clock. The BSP kernel states that the default values are supposed to be used. Clearing them results in decreased perform

Re: [U-Boot] [PATCH V2] imx: mx7: Add support for USB and normal boot modes

2017-08-31 Thread Fabio Estevam
On Thu, Aug 31, 2017 at 10:35 AM, Fabio Estevam wrote: > Hi Eric, > > On Wed, Aug 30, 2017 at 7:13 PM, Eric Nelson wrote: >> This adds support for two additional boot modes on the i.MX7D SoC, which >> is most useful when doing U-Boot development on this chip. >> >> 1. "bmode usb" can be used to f

Re: [U-Boot] [PATCH V2] imx: mx7: Add support for USB and normal boot modes

2017-08-31 Thread Fabio Estevam
Hi Eric, On Wed, Aug 30, 2017 at 7:13 PM, Eric Nelson wrote: > This adds support for two additional boot modes on the i.MX7D SoC, which > is most useful when doing U-Boot development on this chip. > > 1. "bmode usb" can be used to force the ROM boot loader's serial > 2. "bmode normal" can be used

[U-Boot] [PATCH] arm: am33xx: Make pin multiplexing functions optional

2017-08-31 Thread Felix Brack
Boards using the single-register-pin-controller can configure all pins by means of the device tree. This renders the implementation of the two functions set_uart_mux_conf and set_mux_conf_regs obsolete. Using the weak attribute for these two function declarations allows the omission of the respect

Re: [U-Boot] [PATCH v2 03/40] dm: blk: Add a generic function for block device commands

2017-08-31 Thread Bin Meng
Hi Simon, On Thu, Aug 31, 2017 at 8:51 PM, Simon Glass wrote: > Hi Bin, > > On 1 August 2017 at 12:56, Bin Meng wrote: >> Hi Simon, >> >> On Sun, Jul 30, 2017 at 1:34 AM, Simon Glass wrote: >>> Most block devices provide a command (e.g. 'sata', 'scsi', 'ide') and >>> these commands generally do

Re: [U-Boot] [UBOOT] [PATCH] cmd: usb: ignore block devices under mass storage device

2017-08-31 Thread Simon Glass
Hi Suneel, On 15 August 2017 at 11:06, Suneel Garapati wrote: > Hi Simon, > > > On Sun, Aug 13, 2017 at 2:37 PM, Simon Glass wrote: >> Hi Suneel, >> >> On 10 August 2017 at 23:53, Suneel Garapati wrote: >>> usb tree and info commands may cause crash otherwise >>> >>> Signed-off-by: Suneel Garap

Re: [U-Boot] [PATCH 21/23] efi_loader: fix efi_net_get_status

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:54, Heinrich Schuchardt wrote: > The returned interrupt status was wrong. > > As out transmit buffer is empty we need to always set > EFI_SIMPLE_NETWORK_TRANSMIT_INTERRUPT. > > When we have received a packet we need to set > EFI_SIMPLE_NETWORK_RECEIVE_INTERRUPT. > > Further

Re: [U-Boot] [PATCH 22/23] efi_loader: set parent handle in efi_load_image

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:54, Heinrich Schuchardt wrote: > The parent_handle of the loaded image must be set. > Add the file path protocol from the provided parameter. > Set system table. > Add parameter checks. > > Signed-off-by: Heinrich Schuchardt > --- > lib/efi_loader/efi_boottime.c | 31

Re: [U-Boot] [PATCH v2 00/16] dtoc: Add support for 64-bit addresses

2017-08-31 Thread Simon Glass
Hi Philipp, On 30 August 2017 at 18:12, Dr. Philipp Tomsich wrote: > >> On 30 Aug 2017, at 11:37, Simon Glass wrote: >> >> Hi Philipp, >> >> On 30 August 2017 at 04:31, Dr. Philipp Tomsich >> wrote: >>> Simon, >>> >>> I should be able to give this a test drive on the RK3368 by the end of the >

Re: [U-Boot] [PATCH 08/23] efi_loader: allow creating new handles

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:51, Heinrich Schuchardt wrote: > In efi_install_protocol_interface support creating > a new handle. > > Signed-off-by: Heinrich Schuchardt > --- > lib/efi_loader/efi_boottime.c | 22 -- > 1 file changed, 20 insertions(+), 2 deletions(-) > Reviewed-by:

Re: [U-Boot] [PATCH 16/23] efi_loader: implement DisconnectController

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:53, Heinrich Schuchardt wrote: > Signed-off-by: Heinrich Schuchardt > --- > lib/efi_loader/efi_boottime.c | 77 > ++- > 1 file changed, 76 insertions(+), 1 deletion(-) > > diff --git a/lib/efi_loader/efi_boottime.c b/lib/efi_loader

Re: [U-Boot] [PATCH 23/23] efi_loader: implement SetWatchdogTimer

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:54, Heinrich Schuchardt wrote: > The watchdog is initialized with a 5 minute timeout period. > It can be reset by SetWatchdogTimer. > It is stopped by ExitBoottimeServices. > > Signed-off-by: Heinrich Schuchardt > --- > cmd/bootefi.c | 1 + > include/efi_

Re: [U-Boot] [PATCH 07/23] efi_loader: simplify efi_install_protocol_interface

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:51, Heinrich Schuchardt wrote: > Use function efi_search_obj and efi_search_protocol > to simplify the coding. > > Do away with efi_install_protocol_interface_ext. > We can use EFI_CALL for internal usage. > > Signed-off-by: Heinrich Schuchardt > --- > lib/efi_loader/efi_

Re: [U-Boot] [PATCH 17/23] efi_loader: efi_net: hwaddr_size = 6

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:53, Heinrich Schuchardt wrote: > The length of a MAC address is 6. > We have to set this length in the EFI_SIMPLE_NETWORK_MODE > structure of the EFI_SIMPLE_NETWORK_PROTOCOL. > > Without this patch iPXE fails to initialize the network with > error message > SNP MAC(001e0633

Re: [U-Boot] [PATCH 09/23] efi_loader: simplify efi_uninstall_protocol_interface

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:51, Heinrich Schuchardt wrote: > Use function efi_search_obj and efi_search_protocol > to simplify the coding. > > Do away with efi_uninstall_protocol_interface_ext. > We can use EFI_CALL for internal usage. > > Signed-off-by: Heinrich Schuchardt > --- > lib/efi_loader/ef

Re: [U-Boot] ** Can't read Driver Desriptor Block **

2017-08-31 Thread Simon Glass
Hi Bin, On 31 August 2017 at 10:53, Bin Meng wrote: > Hi Heinrich, > > On Thu, Aug 31, 2017 at 5:19 AM, Heinrich Schuchardt > wrote: >> On 08/30/2017 06:37 AM, Heinrich Schuchardt wrote: >>> >>> >>> On 08/30/2017 03:54 AM, Bin Meng wrote: Hi Heinrich, On Wed, Aug 30, 2017 at 4:26

Re: [U-Boot] [PATCH v2] patman: add support for omitting bouncing addresses

2017-08-31 Thread Simon Glass
Hi Chris, On 30 August 2017 at 05:48, Chris Packham wrote: > Add support for reading a list of bouncing addresses from a in-tree file > (doc/bounces) and from the ~/.patman config file. These addresses are > stripped from the Cc list. > > Signed-off-by: Chris Packham > --- > This version support

Re: [U-Boot] [PATCH 20/23] efi_loader: use events for efi_net_receive

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:54, Heinrich Schuchardt wrote: > A timer event is defined. The timer handler cares for receiving new > packets. > > efi_timer_check is called both in efi_net_transmit and efi_net_receive > to enable events during network communication. > > Calling efi_timer_check in efi_net

Re: [U-Boot] [PATCH] gitignore: add intermediates from libfdt build

2017-08-31 Thread Simon Glass
On 29 August 2017 at 17:47, Philipp Tomsich wrote: > Since ee95d10 (fdt: Build the new python libfdt module), a number of > additional files are auto-generated/installed into the tools directory. > List these in .gitignore to suppress having them listed as untracked. > > Signed-off-by: Philipp Tom

Re: [U-Boot] [PATCH v3] rpi: Enable USB keyboard support

2017-08-31 Thread Simon Glass
Hi Alex, On 28 August 2017 at 18:10, Alexander Graf wrote: > > >> Am 26.08.2017 um 19:43 schrieb Simon Glass : >> >> This is currently disabled, so USB keyboards are not detected in U-Boot. >> Enable this option to fix that. >> >> Signed-off-by: Simon Glass >> Tested-by: Jonathan Gray >> Review

Re: [U-Boot] [PATCH 19/23] efi_loader: correct bits of receive_filters bit mask

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:53, Heinrich Schuchardt wrote: > Remove extraneous commas. > Add comment. > > Signed-off-by: Heinrich Schuchardt > --- > include/efi_api.h | 11 ++- > 1 file changed, 6 insertions(+), 5 deletions(-) Reviewed-by: Simon Glass __

Re: [U-Boot] [PATCH 13/23] efi_loader: non-static efi_open_protocol, efi_close_protocol

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:53, Heinrich Schuchardt wrote: > We need efi_open_protocol and efi_close_protocol for > implementing other functions. So they shouldn't be static. > > Signed-off-by: Heinrich Schuchardt > --- > include/efi_loader.h | 9 + > lib/efi_loader/efi_boottime.c |

Re: [U-Boot] [PATCH 18/23] efi_net: return EFI_UNSUPPORTED where appropriate

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:53, Heinrich Schuchardt wrote: > U-Boot does not implement all functions of the simple network > protocol. The unimplemented functions return either of > EFI_SUCCESS and EFI_INVALID_PARAMETER. > > The UEFI spec foresees to return EFI_UNSUPPORTED in these cases. > > Signed-o

Re: [U-Boot] ** Can't read Driver Desriptor Block **

2017-08-31 Thread Bin Meng
Hi Simon, On Thu, Aug 31, 2017 at 8:52 PM, Simon Glass wrote: > Hi Bin, > > On 31 August 2017 at 10:53, Bin Meng wrote: >> Hi Heinrich, >> >> On Thu, Aug 31, 2017 at 5:19 AM, Heinrich Schuchardt >> wrote: >>> On 08/30/2017 06:37 AM, Heinrich Schuchardt wrote: On 08/30/2017 03:54

Re: [U-Boot] [PATCH 05/23] efi_loader: rework efi_search_obj

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:51, Heinrich Schuchardt wrote: > EFI_HANDLEs are used both in boottime and in runtime services. > efi_search_obj is a function that can be used to validate > handles. So let's make it accessible via efi_loader.h. > > We can simplify the coding using list_for_each_entry. > >

Re: [U-Boot] [PATCH 14/23] efi_loader: pass GUIDs as const efi_guid_t *

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:53, Heinrich Schuchardt wrote: > We need to call some boottime services internally. > Our GUIDs are stored as const efi_guid_t *. > > The boottime services never change GUIDs. > So we can define the parameters as const efi_guid_t *. > > Signed-off-by: Heinrich Schuchardt >

Re: [U-Boot] [PATCH 01/23] efi_loader: allow return value in EFI_CALL

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:51, Heinrich Schuchardt wrote: > Macro EFI_CALL was introduced to call an UEFI function. Should this be 'an EFI'. Or 'a UEFI'? > Unfortunately it does not support return values. > Most UEFI functions have a return value. > > So let's rename EFI_CALL to EFI_CALL_VOID and i

Re: [U-Boot] [PATCH 11/23] efi_loader: open_info in CloseProtocol

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:53, Heinrich Schuchardt wrote: > efi_open_protocol and efi_close_protocol have to keep track of > opened protocols. > > efi_close_protocol has to mark the appropriate entry as > empty. > > Signed-off-by: Heinrich Schuchardt > --- > lib/efi_loader/efi_boottime.c | 33 +

Re: [U-Boot] [PATCH 03/23] efi_loader: support 16 protocols per efi_object

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:51, Heinrich Schuchardt wrote: > 8 protocols per efi_object is insufficient for iPXE. > > Signed-off-by: Heinrich Schuchardt > --- > include/efi_loader.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Reviewed-by: Simon Glass ___

Re: [U-Boot] [PATCH 06/23] efi_loader: new function efi_search_protocol

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:51, Heinrich Schuchardt wrote: > In multiple functions we are searching for the protocol of a handle. > This patch provides a new function efi_search_protocol that we can > use to avoid duplicating code. > > Signed-off-by: Heinrich Schuchardt > --- > lib/efi_loader/efi_bo

Re: [U-Boot] [PATCH 15/23] efi_loader: implement ConnectController

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:53, Heinrich Schuchardt wrote: > Signed-off-by: Heinrich Schuchardt > --- > include/efi_api.h | 22 > include/efi_loader.h | 1 + > lib/efi_loader/efi_boottime.c | 119 > +- > 3 files changed, 141 i

Re: [U-Boot] [PATCH 10/23] efi_loader: open_info in OpenProtocol

2017-08-31 Thread Simon Glass
Hi Heinrich, On 27 August 2017 at 06:53, Heinrich Schuchardt wrote: > efi_open_protocol and close_protocol have to keep track of > opened protocols. > > So we add an array open_info to each protocol of each handle. > > OpenProtocol has enter the agent and controller handle > information into this

Re: [U-Boot] [PATCH 12/23] efi_loader: implement OpenProtocolInformation

2017-08-31 Thread Simon Glass
Hi Heinrich, On 27 August 2017 at 06:53, Heinrich Schuchardt wrote: > efi_open_protocol_information provides the agent and controller > handles as well as the attributes and open count of an protocol > on a handle. > > Cc: Rob Clark > Signed-off-by: Heinrich Schuchardt > --- > lib/efi_loader/e

Re: [U-Boot] [PATCH 02/23] efi_loader: notify when ExitBootServices is invoked

2017-08-31 Thread Simon Glass
On 27 August 2017 at 06:51, Heinrich Schuchardt wrote: > All events of type EVT_SIGNAL_EXIT_BOOT_SERVICES have to be > notified when ExitBootServices is invoked. > > Signed-off-by: Heinrich Schuchardt > --- > lib/efi_loader/efi_boottime.c | 9 + > 1 file changed, 9 insertions(+) Reviewe

Re: [U-Boot] [PATCH 04/23] efi_loader: rework efi_locate_handle

2017-08-31 Thread Simon Glass
Hi Heinrich, On 27 August 2017 at 06:51, Heinrich Schuchardt wrote: > Check the parameters in efi_locate_handle. > > Use list_for_each_entry instead of list_for_each. > > Signed-off-by: Heinrich Schuchardt > --- > lib/efi_loader/efi_boottime.c | 42 +++--- >

Re: [U-Boot] [PATCH v2 03/40] dm: blk: Add a generic function for block device commands

2017-08-31 Thread Simon Glass
Hi Bin, On 1 August 2017 at 12:56, Bin Meng wrote: > Hi Simon, > > On Sun, Jul 30, 2017 at 1:34 AM, Simon Glass wrote: >> Most block devices provide a command (e.g. 'sata', 'scsi', 'ide') and >> these commands generally do the same thing. This makes it harder to >> maintain this code and keep it

Re: [U-Boot] [PATCH] rockchip: timer: fix U_BOOT_DRIVER name

2017-08-31 Thread Simon Glass
On 25 August 2017 at 19:25, Philipp Tomsich wrote: > > When I originally added this driver, I did some careless (and in > retrospect: mindless) copy & paste for the U_BOOT_DRIVER structure > skeletion... unfortunately, the 'arc_timer' string was committed > and slipped through all reviews. > > Thi

Re: [U-Boot] [PATCH] mtd/spi: Add MT35XU512ABA1G12 NOR flash support

2017-08-31 Thread Yogesh Narayan Gaur
> -Original Message- > From: Jagan Teki [mailto:jagannadh.t...@gmail.com] > Sent: Tuesday, August 29, 2017 10:48 PM > To: Yogesh Narayan Gaur > Cc: u-boot@lists.denx.de; York Sun > Subject: Re: [PATCH] mtd/spi: Add MT35XU512ABA1G12 NOR flash support > > On Tue, Aug 29, 2017 at 9:42 PM,

[U-Boot] [PATCH v2] mtd/spi: Add MT35XU512ABA1G12 NOR flash support

2017-08-31 Thread Yogesh Gaur
Add MT35XU512ABA1G12 parameters to NOR flash parameters array. The MT35XU512ABA1G12 only supports 1 bit mode and 8 bits. It can't support dual and quad. Supports subsector erase with 4KB granularity, have support of FSR(flag status register) and flash size is 64MB. Signed-off-by: Yogesh Gaur ---

Re: [U-Boot] [PATCH v2] dm: core: Add livetree documentation

2017-08-31 Thread Simon Glass
On 31 August 2017 at 19:59, Simon Glass wrote: > Add some documentation for the live device tree support in U-Boot. This > was missing from the initial series. > > Signed-off-by: Simon Glass > Suggested-by: Lukasz Majewski Reviewed-by: Łukasz Majewski (I hope this is OK, I just fixed the typo

[U-Boot] [PATCH v2] dm: core: Add livetree documentation

2017-08-31 Thread Simon Glass
Add some documentation for the live device tree support in U-Boot. This was missing from the initial series. Signed-off-by: Simon Glass Suggested-by: Lukasz Majewski --- Changes in v2: - Fix several typos doc/driver-model/livetree.txt | 272 ++ 1 file c

Re: [U-Boot] [PATCH v2 1/2] armv8: mmu: remove unused macro definition

2017-08-31 Thread Andy Yan
ping On 2017年08月17日 15:55, Andy Yan wrote: Macro VA_BITS and PTE_BLOCK_BITS are not used in the code, so remove them. Signed-off-by: Andy Yan Reviewed-by: Philipp Tomsich --- Changes in v2: - remove CONFIG_SYS_PTL2_BITS from the whitelist arch/arm/include/asm/armv8/mmu.h | 9 -

Re: [U-Boot] [U-Boot,v2,2/2] armv8: mmu: add space around operator

2017-08-31 Thread Andy Yan
ping On 2017年08月18日 21:01, Philipp Tomsich wrote: Add space around operator "+", make it match the coding style. Signed-off-by: Andy Yan --- Changes in v2: None arch/arm/include/asm/armv8/mmu.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Philipp Tomsich ___

[U-Boot] [PATCH v3 1/2] armv8: fsl-lsch3: Add SD boot support for LS1088ARDB

2017-08-31 Thread Ashish Kumar
Add SD boot support for LS1088ARDB Signed-off-by: Prabhakar Kushwaha Signed-off-by: Ashish Kumar Signed-off-by: Raghav Dogra --- v3: Depends upon http://patchwork.ozlabs.org/patch/794217/ Rebase to top commit "8b3cec7 mtdparts: Fix uninitialized scalar usage" Enable PPA New memory map arch/a

[U-Boot] [PATCH v3 2/2] armv8: fsl-lsch3: Add SD boot support for ls1088aqds

2017-08-31 Thread Ashish Kumar
Add support for SD boot for QDS for ls1088 paltform Signed-off-by: Ashish Kumar Signed-off-by: Prabhakar Kushwaha Signed-off-by: Raghav Dogra --- v3: Depends upon http://patchwork.ozlabs.org/patch/794217/ Rebase to top commit "8b3cec7 mtdparts: Fix uninitialized scalar usage" Enable PPA New me

[U-Boot] [PATCH v3 1/2] armv8: fsl-layerscape: Support to add RGMII for ls1088aqds

2017-08-31 Thread Ashish Kumar
This patch adds support for RGMII protocol NXP's LDPAA2 support RGMII protocol. LS1088A is the first Soc supporting both RGMII and SGMII. Signed-off-by: Prabhakar Kushwaha Signed-off-by: Amrita Kumari Signed-off-by: Ashish Kumar --- v3: No change arch/arm/cpu/armv8/fsl-layerscape/Kconfig

[U-Boot] [PATCH v3 2/2] armv8: ls1088aqds: Add support of EC1 and EC2

2017-08-31 Thread Ashish Kumar
From: Prabhakar Kushwaha EC1 and EC2 are RGMII interface on ls1088aqds platform. This patch add support of RGMII with PHY and MDIO Signed-off-by: Prabhakar Kushwaha Signed-off-by: Amrita Kumari Signed-off-by: Ashish Kumar --- v3: Add missing singoff board/freescale/ls1088a/eth_ls1088aqds.c

[U-Boot] [PATCH v5 3/3] armv8: ls1088aqds: Add support of LS1088AQDS

2017-08-31 Thread Ashish Kumar
This patch add support of LS1088AQDS platform. The LS1088A QorIQTM Development System (QDS) is a high-performance computing, evaluation, and development platform that supports the LS1088A QorIQ Architecture processor. Signed-off-by: Prabhakar Kushwaha Signed-off-by: Shaohui Xie Signed-off-by: A

[U-Boot] [PATCH v5 2/3] armv8: ls1088ardb: Add support for LS1088ARDB platform

2017-08-31 Thread Ashish Kumar
LS1088A is an ARMv8 implementation. The LS1088ARDB is an evaluatoin platform that supports the LS1088A family SoCs. This patch add basic support of the platform. Signed-off-by: Alison Wang Signed-off-by: Prabhakar Kushwaha Signed-off-by: Ashish Kumar Signed-off-by: Raghav Dogra Signed-off-by:

[U-Boot] [PATCH v5 1/3] armv8: ls1088a: Add NXP LS1088A SoC support

2017-08-31 Thread Ashish Kumar
The QorIQ LS1088A processor is built on the Layerscape architecture combining eight ARM A53 processor cores with advanced, high-performance datapath acceleration and networks, peripheral interfaces required for networking, wireless infrastructure, and general-purpose embedded applications. LS1088A

[U-Boot] mv88e61xx autoneg doesn't work with new driver

2017-08-31 Thread Clément Péron
Hi, I try to update my u-boot with the latest version 2017.07. My board is an Altera SocFPGA CycloneV and I have a mv88e61xx switch. My board is working fine except I got an issue with the new switch driver. If autoneg is set to enable, negotation never complete and i got a timeout. If I let the

Re: [U-Boot] [PATCH] rockchip: clk: rk3368: always run rkclk_init when driver probe

2017-08-31 Thread Andy Yan
Hi Philipp: On 2017年08月28日 04:21, Dr. Philipp Tomsich wrote: On 23 Aug 2017, at 09:26, Andy Yan wrote: commit 4bebf94e8544("rockchip: clk: rk3368: do not change CPLL/GPLL before returning to BROM") limits the pll can only be setup in SPL stage, but there are still some rk3368 based boards hav

Re: [U-Boot] tpl build error "width suffixes are invalid in ARM mode"

2017-08-31 Thread Dr. Philipp Tomsich
I just looked at the codebase and now I am also confused. ;-) At least the usage in the Makefiles seems to not correctly support TPL at this point. For what I have seen, we seem to have multiple problems: 1/ arch/arm/config.mk does test for '$(CONFIG_$(SPL_)SYS_THUMB_BUILD)’ instead of '$(C

Re: [U-Boot] [PATCH] sunxi: Change SUNXI_GMAC to SUN7I_GMAC in sources

2017-08-31 Thread Jagan Teki
On Wed, Aug 30, 2017 at 2:59 PM, Lionel Koenig wrote: > Hi, > I found that in commit-id 4d43d065db3262f9a9918ba72457bf36dfb8e0bb was > changing SUNXI_GMAC config to SUN7I_GMAC. However, after that change, the > PHY was not initialized correctly on my board (Olimex > Olinuxino-A20-Lime2-eMMC). > Th

[U-Boot] [PATCH] pci: layerscape: Fixup iommu-map for LS208xA

2017-08-31 Thread Bharat Bhushan
"pci: layerscape: Fixup device tree node for ls2088a" added support for LS208xA devices but fixing iommu-map property is missing. This patch adds support for fixing iommu-map. Signed-off-by: Bharat Bhushan Signed-off-by: Ioana Ciornei --- drivers/pci/pcie_layerscape_fixup.c | 19 ++-

Re: [U-Boot] [PATCH] usb: dwc2: Align size of invalidating dcache before starting DMA

2017-08-31 Thread Seung-Woo Kim
Hello, This patch was tested with Exynos4412 Odroid-U3 board for THOR usb gadget driver. Best Regards, - Seung-Woo Kim > -Original Message- > From: Seung-Woo Kim [mailto:sw0312@samsung.com] > Sent: Monday, July 31, 2017 6:08 PM > To: u-boot@lists.denx.de; lu...@denx.de; ma...@denx.d

Re: [U-Boot] [PATCH] cmd: set filesize variable in ubi read

2017-08-31 Thread Heiko Schocher
Hello Holger, Am 30.08.2017 um 18:38 schrieb Holger Dengler: After a successful read of a UBI volume, the variable filesize is set to the number of read bytes. To boot linux with a raw initramfs/initrd, you must specify the exact size of the initramfs/initrd image in boot command. If the initram