Re: [U-Boot] [PATCH 00/11] sandbox: efi_loader support

2018-06-14 Thread Simon Glass
Hi, On 14 June 2018 at 10:33, Alexander Graf wrote: > This patch set augments Simon's patch set for efi_loader support > in sandbox[1], but follows a different memory allocation scheme. > > Instead of keeping U-Boot (physical) addresses in the EFI memory > map, this patch set makes the EFI

[U-Boot] [PATCH v7 06/10] efi: sandbox: Enable EFI loader build for sandbox

2018-06-14 Thread Simon Glass
This allows this feature to build within sandbox. This is for testing purposes only since it is not possible for sandbox to load native code. Signed-off-by: Simon Glass --- Changes in v7: - Update patch subject s/builder/build/ Changes in v6: None Changes in v5: None Changes in v4: None

[U-Boot] [PATCH v7 10/10] efi: Rename bootefi_test_finish() to bootefi_run_finish()

2018-06-14 Thread Simon Glass
This function can be used from do_bootefi_exec() so that we use mostly the same code for a normal EFI application and an EFI test. Rename the function and use it in both places. Signed-off-by: Simon Glass --- Changes in v7: - Drop patch "efi: Init the 'rows' and 'cols' variables" - Drop

[U-Boot] [PATCH v7 04/10] efi: sandbox: Add distroboot support

2018-06-14 Thread Simon Glass
With sandbox these values depend on the host system. Let's assume that it is x86_64 for now. Signed-off-by: Simon Glass --- Changes in v7: - Drop an unnecessary comment Changes in v6: - Warn about building sandbox EFI support on something other than __x86_64__ Changes in v5: None Changes in

[U-Boot] [PATCH v7 05/10] efi: sandbox: Add relocation constants

2018-06-14 Thread Simon Glass
Add these so that we can build the EFI loader for sandbox. The values are for x86_64 so potentially bogus. But we don't support relocation within sandbox anyway. Signed-off-by: Simon Glass --- Changes in v7: - Drop an unnecessary comment Changes in v6: - Warn about building sandbox EFI support

[U-Boot] [PATCH v7 02/10] efi: sandbox: Adjust memory usage for sandbox

2018-06-14 Thread Simon Glass
With sandbox the U-Boot code is not mapped into the sandbox memory range so does not need to be excluded when allocating EFI memory. Update the EFI memory init code to take account of that. Also use mapmem instead of a cast to convert a memory address to a pointer. Signed-off-by: Simon Glass

[U-Boot] [PATCH v7 03/10] sandbox: smbios: Update to support sandbox

2018-06-14 Thread Simon Glass
At present this code casts addresses to pointers so cannot be used with sandbox. Update it to use mapmem instead. Signed-off-by: Simon Glass --- Changes in v7: None Changes in v6: None Changes in v5: None Changes in v4: None Changes in v3: - Drop incorrect map_sysmem() in write_smbios_table()

[U-Boot] [PATCH v7 01/10] efi: Don't allow CMD_BOOTEFI_SELFTEST on sandbox

2018-06-14 Thread Simon Glass
This does not work at present and gives the following error: output: 'ld.bfd: read in flex scanner failed scripts/Makefile.lib:390: recipe for target 'lib/efi_selftest/efi_selftest_miniapp_return_efi.so' failed It may be possible to figure this out with suitable linker magic but it does not

[U-Boot] [PATCH v7 00/10] efi: Enable basic sandbox support for EFI loader

2018-06-14 Thread Simon Glass
A limitation of the EFI loader at present is that it does not build with sandbox. This makes it hard to write tests, since sandbox is used for most testing in U-Boot. This series enables the EFI loader feature. It allows sandbox to build and run a trivial function which calls the EFI API to

Re: [U-Boot] [PATCH v6 03/13] efi: sandbox: Adjust memory usage for sandbox

2018-06-14 Thread Alexander Graf
On 06/14/2018 06:33 PM, Simon Glass wrote: Hi Alex, On 14 June 2018 at 10:26, Alexander Graf wrote: On 06/14/2018 06:13 PM, Simon Glass wrote: Hi Alex, On 14 June 2018 at 10:07, Alexander Graf wrote: On 06/14/2018 05:53 PM, Simon Glass wrote: Hi Alex, On 14 June 2018 at 09:47, Alexander

[U-Boot] [PATCH 08/11] efi_loader: efi_allocate_pages is too restrictive

2018-06-14 Thread Alexander Graf
From: Heinrich Schuchardt When running on the sandbox the stack is not necessarily at a higher memory address than the highest free memory. There is no reason why the checking of the highest memory address should be more restrictive for EFI_ALLOCATE_ANY_PAGES than for EFI_ALLOCATE_MAX_ADDRESS.

Re: [U-Boot] [PATCH v6 03/13] efi: sandbox: Adjust memory usage for sandbox

2018-06-14 Thread Simon Glass
Hi Alex, On 14 June 2018 at 10:26, Alexander Graf wrote: > On 06/14/2018 06:13 PM, Simon Glass wrote: >> >> Hi Alex, >> >> On 14 June 2018 at 10:07, Alexander Graf wrote: >>> >>> On 06/14/2018 05:53 PM, Simon Glass wrote: Hi Alex, On 14 June 2018 at 09:47, Alexander Graf

Re: [U-Boot] [PATCH v2 1/3] net: Prefer command line arguments

2018-06-14 Thread Joe Hershberger
On Thu, Jun 14, 2018 at 5:04 AM, Alexander Graf wrote: > We can call commands like dhcp and bootp without arguments or with > explicit command line arguments that really should tell the code where > to look for files instead. > > Unfortunately, the current code simply overwrites command line

[U-Boot] [PATCH 00/11] sandbox: efi_loader support

2018-06-14 Thread Alexander Graf
This patch set augments Simon's patch set for efi_loader support in sandbox[1], but follows a different memory allocation scheme. Instead of keeping U-Boot (physical) addresses in the EFI memory map, this patch set makes the EFI memory map contain host virtual (virtual) addresses. That way most

[U-Boot] [PATCH 05/11] efi.h: Do not use config options

2018-06-14 Thread Alexander Graf
Currently efi.h determines a few bits of its environment according to config options. This falls apart with the efi stub support which may result in efi.h getting pulled into the stub as well as real U-Boot code. In that case, one may be 32bit while the other one is 64bit. This patch changes the

[U-Boot] [PATCH 11/11] efi: sandbox: Enable EFI loader for sandbox

2018-06-14 Thread Alexander Graf
From: Simon Glass This allows this feature to build within sandbox. This is for testing purposes only since it is not possible for sandbox to load native code. Signed-off-by: Simon Glass Signed-off-by: Alexander Graf --- lib/efi_loader/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1

[U-Boot] [PATCH 09/11] efi_loader: Disable miniapps on sandbox

2018-06-14 Thread Alexander Graf
In the sandbox environment we can not easily build efi stub binaries right now, so let's disable the respective test cases for the efi selftest suite. Signed-off-by: Alexander Graf --- lib/efi_selftest/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[U-Boot] [PATCH 06/11] efi_loader: Allow SMBIOS tables in highmem

2018-06-14 Thread Alexander Graf
We try hard to make sure that SMBIOS tables live in the lower 32bit. However, when we can not find any space at all there, we should not error out but instead just fall back to map them in the full address space instead. Signed-off-by: Alexander Graf --- lib/efi_loader/efi_smbios.c | 11

[U-Boot] [PATCH 02/11] efi: sandbox: Add relocation constants

2018-06-14 Thread Alexander Graf
From: Simon Glass Add these so that we can build the EFI loader for sandbox. The values are for x86_64 so potentially bogus. But we don't support relocation within sandbox anyway. Signed-off-by: Simon Glass Signed-off-by: Alexander Graf --- lib/efi_loader/efi_runtime.c | 12 1

[U-Boot] [PATCH 10/11] efi_loader: Pass virtual address to fs_read()

2018-06-14 Thread Alexander Graf
The fs_read() function wants to get a virtual (u-boot address space in sandbox) address rather than a physical (host address space in sandbox) one. So let's convert from the real pointer back a the physical address to make efi_loader on sandbox happier. Signed-off-by: Alexander Graf ---

[U-Boot] [PATCH 07/11] sandbox: Map host memory for efi_loader

2018-06-14 Thread Alexander Graf
With efi_loader we do not control payload applications, so we can not teach them about the difference between virtual and physical addresses. Instead, let's just always map host virtual addresses in the efi memory map. That way we can be sure that all memory allocation functions always return

[U-Boot] [PATCH 01/11] efi: sandbox: Add distroboot support

2018-06-14 Thread Alexander Graf
From: Simon Glass With sandbox these values depend on the host system. Let's assume that it is x86_64 for now. Signed-off-by: Simon Glass Signed-off-by: Alexander Graf --- include/config_distro_bootcmd.h | 13 + 1 file changed, 13 insertions(+) diff --git

[U-Boot] [PATCH 03/11] efi_loader: Use compiler constants for image loader

2018-06-14 Thread Alexander Graf
The EFI image loader tries to determine which target architecture we're working with to only load PE binaries that match. So far this has worked based on CONFIG defines, because the target CPU was always indicated by a config define. With sandbox however, this is not longer true as all sandbox

[U-Boot] [PATCH 04/11] efi_loader: Use map_sysmem() in bootefi command

2018-06-14 Thread Alexander Graf
The bootefi command gets a few addresses as values passed in. In sandbox, these values are in U-Boot address space, so we need to make sure we explicitly call map_sysmem() on them to be able to access them. Signed-off-by: Alexander Graf --- cmd/bootefi.c | 13 - 1 file changed, 8

Re: [U-Boot] [PATCH v6 03/13] efi: sandbox: Adjust memory usage for sandbox

2018-06-14 Thread Alexander Graf
On 06/14/2018 06:13 PM, Simon Glass wrote: Hi Alex, On 14 June 2018 at 10:07, Alexander Graf wrote: On 06/14/2018 05:53 PM, Simon Glass wrote: Hi Alex, On 14 June 2018 at 09:47, Alexander Graf wrote: Am 14.06.2018 um 17:43 schrieb Simon Glass : Hi Alex, On 14 June 2018 at 08:22,

Re: [U-Boot] [PATCH v6 03/13] efi: sandbox: Adjust memory usage for sandbox

2018-06-14 Thread Simon Glass
Hi Alex, On 14 June 2018 at 10:07, Alexander Graf wrote: > On 06/14/2018 05:53 PM, Simon Glass wrote: >> >> Hi Alex, >> >> On 14 June 2018 at 09:47, Alexander Graf wrote: >>> >>> Am 14.06.2018 um 17:43 schrieb Simon Glass : Hi Alex, > On 14 June 2018 at 08:22, Alexander

Re: [U-Boot] [PATCH v6 03/13] efi: sandbox: Adjust memory usage for sandbox

2018-06-14 Thread Alexander Graf
On 06/14/2018 05:53 PM, Simon Glass wrote: Hi Alex, On 14 June 2018 at 09:47, Alexander Graf wrote: Am 14.06.2018 um 17:43 schrieb Simon Glass : Hi Alex, On 14 June 2018 at 08:22, Alexander Graf wrote: Am 14.06.2018 um 16:12 schrieb Simon Glass : Hi Alex, On 14 June 2018 at

Re: [U-Boot] [PATCH v1 3/3] cmd: usb gadget: Add a command to bind a USB gadget driver to a UDC port

2018-06-14 Thread Stephen Warren
On 06/14/2018 09:11 AM, Simon Glass wrote: On 14 June 2018 at 09:02, Jean-Jacques Hiblot wrote: On 13/06/2018 03:29, Simon Glass wrote: Hi Jean-Jacques, On 12 June 2018 at 03:31, Jean-Jacques Hiblot wrote: Hi, On 08/06/2018 23:59, Simon Glass wrote: Hi, On 7 June 2018 at 01:39,

Re: [U-Boot] [PATCH v6 03/13] efi: sandbox: Adjust memory usage for sandbox

2018-06-14 Thread Simon Glass
Hi Alex, On 14 June 2018 at 09:47, Alexander Graf wrote: > > >> Am 14.06.2018 um 17:43 schrieb Simon Glass : >> >> Hi Alex, >> >>> On 14 June 2018 at 08:22, Alexander Graf wrote: >>> >>> >>> Am 14.06.2018 um 16:12 schrieb Simon Glass : Hi Alex, >> On 14 June 2018 at

Re: [U-Boot] [PATCH v6 03/13] efi: sandbox: Adjust memory usage for sandbox

2018-06-14 Thread Alexander Graf
> Am 14.06.2018 um 17:43 schrieb Simon Glass : > > Hi Alex, > >> On 14 June 2018 at 08:22, Alexander Graf wrote: >> >> >> >>> Am 14.06.2018 um 16:12 schrieb Simon Glass : >>> >>> Hi Alex, >>> > On 14 June 2018 at 07:41, Alexander Graf wrote: > On 06/14/2018 02:58 PM, Simon Glass

Re: [U-Boot] [PATCH v6 03/13] efi: sandbox: Adjust memory usage for sandbox

2018-06-14 Thread Simon Glass
Hi Alex, On 14 June 2018 at 08:22, Alexander Graf wrote: > > > > > Am 14.06.2018 um 16:12 schrieb Simon Glass : > > > > Hi Alex, > > > >> On 14 June 2018 at 07:41, Alexander Graf wrote: > >>> On 06/14/2018 02:58 PM, Simon Glass wrote: > >>> > >>> Hi Alex, > >>> > On 14 June 2018 at 04:12,

Re: [U-Boot] Pull request: u-boot-net.git master

2018-06-14 Thread Tom Rini
On Wed, Jun 13, 2018 at 02:00:15PM -0500, Joe Hershberger wrote: > Hi Tom, > > The following changes since commit acaee30608ce203289a180d664b7f0abb2e64ee7: > > ARM: DTS: resync a3517.dtsi with Linux 4.17 (2018-06-13 07:49:14 -0400) > > are available in the git repository at: > >

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

2018-06-14 Thread Tom Rini
On Wed, Jun 13, 2018 at 10:17:03AM +0800, Bin Meng wrote: > Hi Tom, > > The following changes since commit 7868909ed53ed41a945f7ed95ebb88aa252142ce: > > Merge git://git.denx.de/u-boot-fsl-qoriq (2018-06-12 13:25:24 -0400) > > are available in the git repository at: > >

Re: [U-Boot] [PATCH v5 10/13] efi: sandbox: Add a simple 'bootefi test' command

2018-06-14 Thread Alexander Graf
> Am 14.06.2018 um 17:12 schrieb Simon Glass : > > HI Alex, > >> On 13 June 2018 at 04:08, Alexander Graf wrote: >> >> >>> On 12.06.18 23:57, Simon Glass wrote: >>> Hi Alex, >>> On 12 June 2018 at 08:11, Alexander Graf wrote: > On 12.06.18 15:48, Simon Glass wrote:

[U-Boot] [PATCH 4/4] arm: mvebu: helios4: remove duplicate sdhci pins node

2018-06-14 Thread Baruch Siach
The same pinctrl node appears in the solidrun-microsom dtsi. Use that instead. Cc: Dennis Gilmore Signed-off-by: Baruch Siach --- arch/arm/dts/armada-388-helios4.dts | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/arch/arm/dts/armada-388-helios4.dts

[U-Boot] [PATCH 2/4] arm: mvebu: clearfog: use the microsom .dtsi

2018-06-14 Thread Baruch Siach
Use hardware description from the recently introduced microsom .dtsi file to reduce duplication. Signed-off-by: Baruch Siach --- arch/arm/dts/armada-388-clearfog.dts | 63 +++- 1 file changed, 6 insertions(+), 57 deletions(-) diff --git

[U-Boot] [PATCH 3/4] arm: mvebu: Better align Clearfog dts file with Linux kernel

2018-06-14 Thread Baruch Siach
From: Jon Nettleton This makes changes so the u-boot dts file is structured more similar to the mainline linux dtsi file. It provides a minimal common dts that can work for most boards based on the ClearFog platform. Ethernet support is only supported for eth0 however all devices are left

[U-Boot] [PATCH 1/4] arm: mvebu: solidrun-microsom: update SPI flash compatible

2018-06-14 Thread Baruch Siach
Add the "spi-flash" compatible string so that the generic sf_probe driver can probe the SPI flash on the SolidRun SOM. Signed-off-by: Baruch Siach --- arch/arm/dts/armada-38x-solidrun-microsom.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [U-Boot] [PATCH v5 10/13] efi: sandbox: Add a simple 'bootefi test' command

2018-06-14 Thread Simon Glass
HI Alex, On 13 June 2018 at 04:08, Alexander Graf wrote: > > > On 12.06.18 23:57, Simon Glass wrote: >> Hi Alex, >> >> On 12 June 2018 at 08:11, Alexander Graf wrote: >>> >>> >>> On 12.06.18 15:48, Simon Glass wrote: Hi Alex, On 12 June 2018 at 02:28, Alexander Graf wrote: >

Re: [U-Boot] [PATCH] x86: cherryhill: Fix DTC warning

2018-06-14 Thread Simon Glass
On 12 June 2018 at 21:01, Bin Meng wrote: > Fix warning when compiling cherryhill.dts with latest DTC: > > "Warning (avoid_unnecessary_addr_size): /pci/pch@1f,0: unnecessary >#address-cells/#size-cells without "ranges" or child "reg" property" > > Signed-off-by: Bin Meng > --- > >

Re: [U-Boot] [PATCH v1 3/3] cmd: usb gadget: Add a command to bind a USB gadget driver to a UDC port

2018-06-14 Thread Simon Glass
On 14 June 2018 at 09:02, Jean-Jacques Hiblot wrote: > > > > On 13/06/2018 03:29, Simon Glass wrote: >> >> Hi Jean-Jacques, >> >> On 12 June 2018 at 03:31, Jean-Jacques Hiblot wrote: >>> >>> Hi, >>> >>> >>> On 08/06/2018 23:59, Simon Glass wrote: Hi, On 7 June 2018 at 01:39,

Re: [U-Boot] [PATCH v1 3/3] cmd: usb gadget: Add a command to bind a USB gadget driver to a UDC port

2018-06-14 Thread Jean-Jacques Hiblot
On 13/06/2018 03:29, Simon Glass wrote: Hi Jean-Jacques, On 12 June 2018 at 03:31, Jean-Jacques Hiblot wrote: Hi, On 08/06/2018 23:59, Simon Glass wrote: Hi, On 7 June 2018 at 01:39, Lukasz Majewski wrote: Hi Jean-Jacques, Most of the time the UDC is bound to a driver when a

Re: [U-Boot] [PATCH v6 03/13] efi: sandbox: Adjust memory usage for sandbox

2018-06-14 Thread Alexander Graf
> Am 14.06.2018 um 16:12 schrieb Simon Glass : > > Hi Alex, > >> On 14 June 2018 at 07:41, Alexander Graf wrote: >>> On 06/14/2018 02:58 PM, Simon Glass wrote: >>> >>> Hi Alex, >>> On 14 June 2018 at 04:12, Alexander Graf wrote: > On 06/13/2018 04:37 AM, Simon Glass wrote:

Re: [U-Boot] [PATCH v2 5/9] clk: Add Actions Semi OWL clock support

2018-06-14 Thread Simon Glass
Hi, On 14 June 2018 at 07:13, Manivannan Sadhasivam wrote: > Hi Simon, > > On Thu, Jun 14, 2018 at 06:58:40AM -0600, Simon Glass wrote: >> Hi, >> >> On 12 June 2018 at 22:15, Manivannan Sadhasivam >> wrote: >> > This commit adds Actions Semi OWL family base clock and S900 SoC specific >> >

Re: [U-Boot] [PATCH 4/5] serial: zynq: Check priv pointer get via dev_get_priv()

2018-06-14 Thread Simon Glass
Hi Michal, On 14 June 2018 at 07:05, Michal Simek wrote: > On 14.6.2018 14:58, Simon Glass wrote: >> Hi, >> >> On 14 June 2018 at 03:32, Michal Simek wrote: >>> Make sure that functions are working with proper strcture. >>> >>> Signed-off-by: Michal Simek >>> --- >>> >>> Reported by: Coverity

Re: [U-Boot] [PATCH v6 03/13] efi: sandbox: Adjust memory usage for sandbox

2018-06-14 Thread Simon Glass
Hi Alex, On 14 June 2018 at 07:41, Alexander Graf wrote: > On 06/14/2018 02:58 PM, Simon Glass wrote: >> >> Hi Alex, >> >> On 14 June 2018 at 04:12, Alexander Graf wrote: >>> >>> On 06/13/2018 04:37 AM, Simon Glass wrote: With sandbox the U-Boot code is not mapped into the sandbox

Re: [U-Boot] [PATCH v2 02/13] efi.h: Do not use config options

2018-06-14 Thread Alexander Graf
On 06/14/2018 02:58 PM, Simon Glass wrote: Hi Alex, On 13 June 2018 at 04:17, Alexander Graf wrote: On 13.06.18 03:29, Simon Glass wrote: Hi Bin, Alex, On 12 June 2018 at 09:36, Bin Meng wrote: From: Alexander Graf Currently efi.h determines a few bits of its environment according to

Re: [U-Boot] [PATCH v6 03/13] efi: sandbox: Adjust memory usage for sandbox

2018-06-14 Thread Alexander Graf
On 06/14/2018 02:58 PM, Simon Glass wrote: Hi Alex, On 14 June 2018 at 04:12, Alexander Graf wrote: On 06/13/2018 04:37 AM, Simon Glass wrote: With sandbox the U-Boot code is not mapped into the sandbox memory range so does not need to be excluded when allocating EFI memory. Update the EFI

[U-Boot] Re : Re: Clearfog fails to build with U-Boot v2018.07-rc1

2018-06-14 Thread guillaume . gardet
- Stefan Roese a écrit : > Hi Guillaume, > > On 14.06.2018 14:19, guillaume.gar...@free.fr wrote: > > Clearfog config fails to build with U-Boot v2018.07-rc1, because SPL is too > > big: > > > > LD spl/u-boot-spl > >ld.bfd: SPL image too big > >make[1]: ***

Re: [U-Boot] [PATCH v2 5/9] clk: Add Actions Semi OWL clock support

2018-06-14 Thread Manivannan Sadhasivam
Hi Simon, On Thu, Jun 14, 2018 at 06:58:40AM -0600, Simon Glass wrote: > Hi, > > On 12 June 2018 at 22:15, Manivannan Sadhasivam > wrote: > > This commit adds Actions Semi OWL family base clock and S900 SoC specific > > clock support. For S900 peripheral clock support, only UART clock has been

Re: [U-Boot] [PULL] u-boot-sh/master

2018-06-14 Thread Tom Rini
On Thu, Jun 14, 2018 at 01:35:05PM +0200, Marek Vasut wrote: > On 06/14/2018 01:19 PM, Tom Rini wrote: > > On Wed, Jun 13, 2018 at 06:05:08AM +0200, Marek Vasut wrote: > > > >> The following changes since commit > >> 813d1fb56dc0af9567feb86cd71c49f14662044b: > >> > >> Merge branch 'master' of

Re: [U-Boot] [PATCH 4/5] serial: zynq: Check priv pointer get via dev_get_priv()

2018-06-14 Thread Michal Simek
On 14.6.2018 14:58, Simon Glass wrote: > Hi, > > On 14 June 2018 at 03:32, Michal Simek wrote: >> Make sure that functions are working with proper strcture. >> >> Signed-off-by: Michal Simek >> --- >> >> Reported by: Coverity (local) >> >> --- >> drivers/serial/serial_zynq.c | 27

Re: [U-Boot] [PATCH v6 2/5] include: reset: Change to use CONFIG_IS_ENABLED(DM_RESET)

2018-06-14 Thread Simon Glass
On 14 June 2018 at 04:45, Ley Foon Tan wrote: > Change to use CONFIG_IS_ENABLED(DM_RESET), so this can work in SPL > build (CONFIG_SPL_DM_RESET) and U-boot build (CONFIG_DM_RESET). > > Signed-off-by: Ley Foon Tan > --- > include/reset.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >

Re: [U-Boot] [PATCH v2 02/13] efi.h: Do not use config options

2018-06-14 Thread Simon Glass
Hi Alex, On 13 June 2018 at 04:17, Alexander Graf wrote: > > > On 13.06.18 03:29, Simon Glass wrote: >> Hi Bin, Alex, >> >> On 12 June 2018 at 09:36, Bin Meng wrote: >>> From: Alexander Graf >>> >>> Currently efi.h determines a few bits of its environment according to >>> config options. This

Re: [U-Boot] [PATCH 4/5] serial: zynq: Check priv pointer get via dev_get_priv()

2018-06-14 Thread Simon Glass
Hi, On 14 June 2018 at 03:32, Michal Simek wrote: > Make sure that functions are working with proper strcture. > > Signed-off-by: Michal Simek > --- > > Reported by: Coverity (local) > > --- > drivers/serial/serial_zynq.c | 27 --- > 1 file changed, 24 insertions(+), 3

Re: [U-Boot] [PATCH v2 5/9] clk: Add Actions Semi OWL clock support

2018-06-14 Thread Simon Glass
Hi, On 12 June 2018 at 22:15, Manivannan Sadhasivam wrote: > This commit adds Actions Semi OWL family base clock and S900 SoC specific > clock support. For S900 peripheral clock support, only UART clock has been > added for now. > > Signed-off-by: Manivannan Sadhasivam > --- >

Re: [U-Boot] [PATCH v6 1/5] reset: Rename CONFIG_SPL_RESET_SUPPORT to CONFIG_SPL_DM_RESET

2018-06-14 Thread Simon Glass
On 14 June 2018 at 04:45, Ley Foon Tan wrote: > Rename CONFIG_SPL_RESET_SUPPORT to CONFIG_SPL_DM_RESET, so can use > CONFIG_IS_ENABLED(DM_RESET) checking in reset.h later. > > Signed-off-by: Ley Foon Tan > --- > arch/arm/mach-stm32mp/Kconfig | 2 +- > common/spl/Kconfig| 2 +- >

Re: [U-Boot] [PATCH 2/5] serial: zynq: Write chars till output fifo is full

2018-06-14 Thread Simon Glass
On 14 June 2018 at 03:32, Michal Simek wrote: > Change logic and put char to fifo till there is a space in output fifo. > Origin logic was that output fifo needs to be empty. It means only one > char was in output queue. > Also remove unused ZYNQ_UART_SR_TXEMPTY macro. > > Signed-off-by: Michal

Re: [U-Boot] [PATCH 3/5] serial: zynq: Initialize uart only before relocation

2018-06-14 Thread Simon Glass
On 14 June 2018 at 03:32, Michal Simek wrote: > This issue was found when OF_LIVE was enabled that there are scrambled > chars on the console like this: > Chip ID:zu3eg > Watchdog: Started��j� sdhci@ff16: 0, sdhci@ff17: 1 > In:serial@ff01 > > I found a solution for this

Re: [U-Boot] [PATCH v3 1/2] dm: mdio: add a uclass for MDIO

2018-06-14 Thread Simon Glass
Hi Ken, On 12 June 2018 at 22:33, wrote: > From: Ken Ma > > Add a uclass which provides access to MDIO busses and includes > operations required by MDIO. > The implementation is based on the existing mii/phy/mdio data > structures and APIs. > This patch also adds evice tree binding for MDIO

Re: [U-Boot] [PATCH 1/5] serial: zynq: Use BIT macros instead of shifts and full hex numbers

2018-06-14 Thread Simon Glass
On 14 June 2018 at 03:32, Michal Simek wrote: > Coding style is checking to use BIT macros instead of shifts. > The patch is also fixing the rest of macros which should be BITs instead > of hex numbers. > > Signed-off-by: Michal Simek > --- > > drivers/serial/serial_zynq.c | 16

Re: [U-Boot] [PATCH 5/5] serial: zynq: Make zynq_serial_setbrg static

2018-06-14 Thread Simon Glass
On 14 June 2018 at 03:32, Michal Simek wrote: > This function is used only inside this driver that's why should be > static. > > Signed-off-by: Michal Simek > --- > > drivers/serial/serial_zynq.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Reviewed-by: Simon Glass

Re: [U-Boot] [PATCH 1/4] lib: fdtdec: Fill initial ram top with DDR start value from dt

2018-06-14 Thread Simon Glass
Hi Siva, On 14 June 2018 at 00:53, Siva Durga Prasad Paladugu wrote: > Hi Simon, > >> -Original Message- >> From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon Glass >> Sent: Saturday, June 09, 2018 3:29 AM >> To: Michal Simek >> Cc: Siva Durga Prasad Paladugu ; U-Boot

Re: [U-Boot] [PATCH v6 03/13] efi: sandbox: Adjust memory usage for sandbox

2018-06-14 Thread Simon Glass
Hi Alex, On 14 June 2018 at 04:12, Alexander Graf wrote: > On 06/13/2018 04:37 AM, Simon Glass wrote: >> >> With sandbox the U-Boot code is not mapped into the sandbox memory range >> so does not need to be excluded when allocating EFI memory. Update the EFI >> memory init code to take account

Re: [U-Boot] [PATCH v6 02/13] efi: Init the 'rows' and 'cols' variables

2018-06-14 Thread Simon Glass
Hi Alex, On 14 June 2018 at 04:11, Alexander Graf wrote: > On 06/13/2018 04:37 AM, Simon Glass wrote: >> >> The current code causes a compiler error on gcc 4.8.4 as used by sandbox >> on Ubuntu 14.04, which is fairly recent. Init these variables to fix the >> problem. >> >> Signed-off-by: Simon

Re: [U-Boot] [PATCH 2/4] ARM: Introduce ability to enable invalidate of BTB with ICIALLU on Cortex-A15 for CVE-2017-5715

2018-06-14 Thread Nishanth Menon
On 21:36-20180613, Florian Fainelli wrote: [...] > >>> c) This workaround applies to only the boot processor. It is important > >>> to apply workaround as necessary (context-save-restore) around low > >>> power context loss OR additional processors as necessary in either > >>> firmware

Re: [U-Boot] Clearfog fails to build with U-Boot v2018.07-rc1

2018-06-14 Thread Stefan Roese
Hi Guillaume, On 14.06.2018 14:19, guillaume.gar...@free.fr wrote: Clearfog config fails to build with U-Boot v2018.07-rc1, because SPL is too big: LD spl/u-boot-spl ld.bfd: SPL image too big make[1]: *** [scripts/Makefile.spl:347: spl/u-boot-spl] Error 1 make: ***

[U-Boot] Clearfog fails to build with U-Boot v2018.07-rc1

2018-06-14 Thread guillaume . gardet
Hi Stefan, Clearfog config fails to build with U-Boot v2018.07-rc1, because SPL is too big: LD spl/u-boot-spl ld.bfd: SPL image too big make[1]: *** [scripts/Makefile.spl:347: spl/u-boot-spl] Error 1 make: *** [Makefile:1510: spl/u-boot-spl] Error 2 Guillaume

Re: [U-Boot] [PATCH v3 2/3] efi_loader: ARM: run EFI payloads non-secure

2018-06-14 Thread Marc Zyngier
On 13/06/18 23:41, Mark Kettenis wrote: > If desired (and possible) switch into HYP mode or non-secure SVC mode > before calling the entry point of an EFI application. This allows > U-Boot to provide a usable PSCI implementation and makes it possible > to boot kernels into hypervisor mode using

[U-Boot] [PATCH u-boot 5/5] configs: Update Meson GX configs

2018-06-14 Thread Neil Armstrong
Enable USB on all Amlogic Meson GXL based board. Enable Regulator support on all boards. Enable ADC support on the LibreTech-CC board. Signed-off-by: Neil Armstrong --- configs/khadas-vim_defconfig | 14 ++ configs/libretech-cc_defconfig | 17 +

[U-Boot] [PATCH u-boot 2/5] config: meson-gx-common: Enable USB boot

2018-06-14 Thread Neil Armstrong
Add USB as boot target depending on the configuration. Signed-off-by: Neil Armstrong --- include/configs/meson-gx-common.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/include/configs/meson-gx-common.h b/include/configs/meson-gx-common.h index 6e61b70..7435f34 100644 ---

[U-Boot] [PATCH u-boot 3/5] clk: add Amlogic meson clock driver

2018-06-14 Thread Neil Armstrong
From: Beniamino Galvani Introduce a basic clock driver for Amlogic Meson SoCs which supports enabling/disabling clock gates and getting their frequency. Signed-off-by: Beniamino Galvani Signed-off-by: Neil Armstrong --- arch/arm/include/asm/arch-meson/clock.h | 117 ++

[U-Boot] [PATCH u-boot 4/5] meson: use the clock driver

2018-06-14 Thread Neil Armstrong
From: Beniamino Galvani Use the clk framework to initialize clocks from drivers that need them instead of having hardcoded frequencies and initializations from board code. Signed-off-by: Beniamino Galvani Signed-off-by: Neil Armstrong --- arch/arm/include/asm/arch-meson/gx.h | 10

[U-Boot] [PATCH u-boot 1/5] adc: meson-saradc: fix regmap_init_mem call

2018-06-14 Thread Neil Armstrong
The SARADC driver was merged after the following commit : commit d35812368a59 ("regmap: change regmap_init_mem() to take ofnode instead udevice") Thus breaking build, this patch fixes the regmap_init_mem accordingly. Signed-off-by: Neil Armstrong --- drivers/adc/meson-saradc.c | 2 +- 1 file

[U-Boot] [PATCH u-boot 0/5] Amlogic Meson GX Updates for v2018.07

2018-06-14 Thread Neil Armstrong
This patchset : - fixes meson-saradc build - adds the meson clock driver from Beniamino with it's consent - switches to the clock driver for Ethernet and I2C - Enables boot from USB if config is enabled - finally enables USB, Regulator & ADC drivers for selected boards Beniamino Galvani (2):

Re: [U-Boot] [PULL] u-boot-sh/master

2018-06-14 Thread Marek Vasut
On 06/14/2018 01:19 PM, Tom Rini wrote: > On Wed, Jun 13, 2018 at 06:05:08AM +0200, Marek Vasut wrote: > >> The following changes since commit 813d1fb56dc0af9567feb86cd71c49f14662044b: >> >> Merge branch 'master' of git://git.denx.de/u-boot-ubi (2018-06-08 >> 10:08:20 -0400) >> >> are available

[U-Boot] [PATCH] dts: gpio: Sync header with Linux 4.17

2018-06-14 Thread Marek Vasut
Sync the gpio.h header with Linux 4.17, which contains new macros. Signed-off-by: Marek Vasut Cc: Tom Rini --- include/dt-bindings/gpio/gpio.h | 21 + 1 file changed, 21 insertions(+) diff --git a/include/dt-bindings/gpio/gpio.h b/include/dt-bindings/gpio/gpio.h index

Re: [U-Boot] [PULL] u-boot-sh/master

2018-06-14 Thread Tom Rini
On Wed, Jun 13, 2018 at 06:05:08AM +0200, Marek Vasut wrote: > The following changes since commit 813d1fb56dc0af9567feb86cd71c49f14662044b: > > Merge branch 'master' of git://git.denx.de/u-boot-ubi (2018-06-08 > 10:08:20 -0400) > > are available in the Git repository at: > >

[U-Boot] [PATCH 12/12] rockchip: rk3288-phycore: set flash1 iodomain to 1.8V

2018-06-14 Thread Janine Hagemann
This pin is supplied by 1.8V, but the default iodomain setting is 3.3V. Signed-off-by: Janine Hagemann --- board/phytec/phycore_rk3288/phycore-rk3288.c | 16 1 file changed, 16 insertions(+) diff --git a/board/phytec/phycore_rk3288/phycore-rk3288.c

[U-Boot] [PATCH 09/12] drivers: net: designware: Add reading of DT phy-handle node

2018-06-14 Thread Janine Hagemann
Add the ability to read the phy-handle node of the gmac. Upon reading this handle the phy-id can be stored based on the reg node in the DT. The phy-handle also needs to be stored and passed to the phy to access any phy data that is available. Signed-off-by: Janine Hagemann ---

[U-Boot] [PATCH 11/12] ARM: dts: rockchip: ADD dp83867 CLK_OUT muxing

2018-06-14 Thread Janine Hagemann
The CLK_O_SEL default is synchronus to XI input clock, which is 25 MHz. Set CLK_O_SEL to channel A transmit clock so we have 125 MHz on CLK_OUT. Signed-off-by: Janine Hagemann --- arch/arm/dts/rk3288-phycore-som.dtsi | 1 + 1 file changed, 1 insertion(+) diff --git

[U-Boot] [PATCH 04/12] Net: phy: ti: Fix fifo_depth register write

2018-06-14 Thread Janine Hagemann
The register was not read before the writing, so the previous value was overwritten. Signed-off-by: Janine Hagemann --- drivers/net/phy/ti.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/drivers/net/phy/ti.c b/drivers/net/phy/ti.c index 8f3ed8a..d7ae881 100644

[U-Boot] [PATCH 10/12] net: phy: ti: Add binding for the CLK_OUT pin muxing

2018-06-14 Thread Janine Hagemann
The DP83867 has a muxing option for the CLK_OUT pin. It is possible to set CLK_OUT for different channels. Create a binding to select a specific clock for CLK_OUT pin. Based on commit '9708fb630d19ee51ae3aeb3a533e3010da0e8570' mainline linux kernel. Signed-off-by: Janine Hagemann ---

[U-Boot] [PATCH 08/12] net: gmac_rockchip: Add handeling for RGMII_ID/RXID/TXID

2018-06-14 Thread Janine Hagemann
Using PHY internal delays in combination with the phy-mode rgmii-id/rxid/txid was not possible. Only rgmii was supported. Now we can disable rockchip's gmac delay lines and also use rgmii-id/rxid/txid. Based on commit 'eaf70ad14cbbb99d46b78b1307628a16a3f6075d' from mainline linux kernel.

[U-Boot] [PATCH 07/12] net: phy: ti: add workaround for incorrect RX_CTRL pin strap

2018-06-14 Thread Janine Hagemann
The data manual for DP83867IR/CR, SNLS484E[1], revised march 2017, advises that strapping RX_DV/RX_CTRL pin in mode 1 and 2 is not supported (see note below Table 5 (4-Level Strap Pins)). There are some boards which have the pin strapped this way and need software workaround suggested by the data

[U-Boot] [PATCH 03/12] net: gmac_rockchip: Fix a register write in rk3328_gmac_set_to_rgmii

2018-06-14 Thread Janine Hagemann
We have to use RK3328_RXCLK_DLY_ENA_GMAC_ENABLE instead of RK3328_RXCLK_DLY_ENA_GMAC_MASK in rk3328_gmac_set_to_rgmii() to enable the RX delay. The MASK was used in a wrong way. Signed-off-by: Janine Hagemann --- drivers/net/gmac_rockchip.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[U-Boot] [PATCH 06/12] net: phy: ti: Recover from "port mirroring" N/A MODE4

2018-06-14 Thread Janine Hagemann
The DP83867 when not properly bootstrapped - especially with LED_0 pin - can enter N/A MODE4 for "port mirroring" feature. To provide normal operation of the PHY, one needs not only to explicitly disable the port mirroring feature, but as well stop some IC internal testing (which disables RGMII

[U-Boot] [PATCH 05/12] net: phy: ti: Add lane swapping support in the DP83867 TI's PHY driver

2018-06-14 Thread Janine Hagemann
This patch adds support for enabling or disabling the lane swapping (called "port mirroring" in PHY's CFG4 register) feature of the DP83867 TI's PHY device. One use case is when bootstrap configuration enables this feature (because of e.g. LED_0 wrong wiring) so then one needs to disable it in

[U-Boot] [PATCH 02/12] config: phycore-rk3288_defconfig: add PHY_TI

2018-06-14 Thread Janine Hagemann
add PHY_TI to support dp83867 Signed-off-by: Janine Hagemann --- configs/phycore-rk3288_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/phycore-rk3288_defconfig b/configs/phycore-rk3288_defconfig index 50b0c98..7d0535f 100644 --- a/configs/phycore-rk3288_defconfig +++

[U-Boot] [PATCH 01/12] arch: arm: mach-rockchip: rk3288: Enable regulators in board_init

2018-06-14 Thread Janine Hagemann
At start-up, the regulators have to be enabled. Let's use regulators_enable_boot_on() to enable the regulators needed for boot. Signed-off-by: Wadim Egorov Signed-off-by: Janine Hagemann --- arch/arm/mach-rockchip/rk3288-board.c | 6 ++ 1 file changed, 6 insertions(+) diff --git

[U-Boot] [PATCH] common: bootm: reserve memory bank

2018-06-14 Thread Jason Zhu
Actually the DRAM is not only seperated in one bank. The DRAM bank information is stored in gd->bd->bi_dram, so reserve lmb according to gd->bd->bi_dram. Signed-off-by: Jason Zhu --- common/bootm.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/common/bootm.c

[U-Boot] [PATCH] common: bootm: reserve memory bank

2018-06-14 Thread Jason Zhu
Actually the DRAM is not only seperated in one bank. The DRAM bank information is stored in gd->bd->bi_dram, so reserve lmb according to gd->bd->bi_dram. Signed-off-by: Jason Zhu --- common/bootm.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/common/bootm.c

Re: [U-Boot] [PATCH v6 08/13] efi: sandbox: Enable EFI loader builder for sandbox

2018-06-14 Thread Alexander Graf
On 06/13/2018 04:37 AM, Simon Glass wrote: This allows this feature to build within sandbox. This is for testing purposes only since it is not possible for sandbox to load native code. Signed-off-by: Simon Glass Are you sure you want to enable a "builder", not the "build"? (see subject

Re: [U-Boot] [PATCH v6 07/13] efi: Add a comment about duplicated ELF constants

2018-06-14 Thread Alexander Graf
On 06/13/2018 04:37 AM, Simon Glass wrote: These constants are defined in arch-specific code but redefined here. Add a TODO to clean this up. Signed-off-by: Simon Glass Reviewed-by: Heinrich Schuchardt This patch is part of efi-next already. Alex

Re: [U-Boot] [PATCH v6 06/13] efi: sandbox: Add relocation constants

2018-06-14 Thread Alexander Graf
On 06/13/2018 04:37 AM, Simon Glass wrote: Add these so that we can build the EFI loader for sandbox. The values are for x86_64 so potentially bogus. But we don't support relocation within sandbox anyway. Signed-off-by: Simon Glass --- Changes in v6: - Warn about building sandbox EFI support

Re: [U-Boot] [PATCH v6 05/13] efi: sandbox: Add distroboot support

2018-06-14 Thread Alexander Graf
On 06/13/2018 04:37 AM, Simon Glass wrote: With sandbox these values depend on the host system. Let's assume that it is x86_64 for now. Signed-off-by: Simon Glass --- Changes in v6: - Warn about building sandbox EFI support on something other than __x86_64__ Changes in v5: None Changes in

Re: [U-Boot] [PATCH v6 03/13] efi: sandbox: Adjust memory usage for sandbox

2018-06-14 Thread Alexander Graf
On 06/13/2018 04:37 AM, Simon Glass wrote: With sandbox the U-Boot code is not mapped into the sandbox memory range so does not need to be excluded when allocating EFI memory. Update the EFI memory init code to take account of that. Also use mapmem instead of a cast to convert a memory address

Re: [U-Boot] [PATCH v6 02/13] efi: Init the 'rows' and 'cols' variables

2018-06-14 Thread Alexander Graf
On 06/13/2018 04:37 AM, Simon Glass wrote: The current code causes a compiler error on gcc 4.8.4 as used by sandbox on Ubuntu 14.04, which is fairly recent. Init these variables to fix the problem. Signed-off-by: Simon Glass I thought this patch was obsolete now? Alex

[U-Boot] [PATCH v2 2/3] net: Add option to prefer bootp/dhcp serverip

2018-06-14 Thread Alexander Graf
Currently we can choose between 2 different types of behavior for the serverip variable: 1) Always overwrite it with the DHCP server IP address (default) 2) Ignore what the DHCP server says (CONFIG_BOOTP_SERVERIP) This patch adds a 3rd option: 3) Use serverip from DHCP if no serverip is

[U-Boot] [PATCH v2 1/3] net: Prefer command line arguments

2018-06-14 Thread Alexander Graf
We can call commands like dhcp and bootp without arguments or with explicit command line arguments that really should tell the code where to look for files instead. Unfortunately, the current code simply overwrites command line arguments in the dhcp case with dhcp values. This patch allows the

<    1   2   3   >