Re: [U-Boot] [PATCH 083/126] x86: Add a new global_data member for the cache record

2019-10-10 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > At present we reuse the mrc_output char * to also point to the cache > record after it has been set up. This is confusing and doesn't save much > data space. > > Add a new mrc_cache member instead. > > Signed-off-by: Simon Glass > --- > > a

Re: [U-Boot] [PATCH 084/126] x86: Tidy up error handling in mrccache_save()

2019-10-10 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > This function is a bit confusing at present due to the error handling. > Update it to remove the goto, returning errors as they happen. > > While we are here, use hex for the data size since this is the norm in > U-Boot. > > Signed-off-by: Si

Re: [U-Boot] [PATCH 078/126] x86: Define the SPL image start

2019-10-10 Thread Bin Meng
Hi Simon, On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > Define this symbol so that we can use binman symbols correctly. > > Signed-off-by: Simon Glass > --- > > arch/x86/cpu/u-boot-spl.lds | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/arch/x86/cpu/u-boot

Re: [U-Boot] [PATCH 105/126] x86: Reduce resetvec size

2019-10-10 Thread Bin Meng
On Wed, Sep 25, 2019 at 11:02 PM Simon Glass wrote: > > At present this produces a 16-byte file. It is intended to start 16 bytes > before the end of ROM and pads with zeroes to readh the end. > > But binman sometimes wants to add an image-header at the very end of ROM. > Furthermore binman automa

Re: [U-Boot] [PATCH v3 2/4] arm: socfpga: Convert reset manager from struct to defines

2019-10-10 Thread Simon Goldschmidt
On Thu, Oct 10, 2019 at 8:20 AM Ley Foon Tan wrote: > > Convert reset manager for Gen5, Arria 10 and Stratix 10 from struct > to defines. A commit message should describe what the commit does. By leaving out the fact that you're now loading the base address from DTS, you're not describing all cha

Re: [U-Boot] [PATCH v3 3/4] arm: socfpga: Convert system manager from struct to defines

2019-10-10 Thread Simon Goldschmidt
On Thu, Oct 10, 2019 at 8:20 AM Ley Foon Tan wrote: > > Convert system manager for Gen5, Arria 10 and Stratix 10 from struct > to defines. Same as 2/4: please add a description about the "read address from dts" thing. Regards, Simon > > Signed-off-by: Ley Foon Tan > > --- > v3: > - Remove "No

Re: [U-Boot] [PATCH v3 4/4] arm: socfpga: Convert clock manager from struct to defines

2019-10-10 Thread Simon Goldschmidt
On Thu, Oct 10, 2019 at 8:20 AM Ley Foon Tan wrote: > > Convert clock manager for Gen5, Arria 10 and Stratix 10 from struct > to defines. Same as 2/4: please add a description about the "read address from dts" thing. Regards, Simon > > Signed-off-by: Ley Foon Tan > > --- > v3: > - Remove "No f

Re: [U-Boot] [PATCH v3 2/4] arm: socfpga: Convert reset manager from struct to defines

2019-10-10 Thread Ley Foon Tan
On Thu, Oct 10, 2019 at 3:14 PM Simon Goldschmidt wrote: > > On Thu, Oct 10, 2019 at 8:20 AM Ley Foon Tan wrote: > > > > Convert reset manager for Gen5, Arria 10 and Stratix 10 from struct > > to defines. > > A commit message should describe what the commit does. By leaving out the fact > that yo

[U-Boot] [PATCH v4 1/4] arm: dts: socfpga: Add u-boot, dm-pre-reloc for sysmgr and clkmgr nodes

2019-10-10 Thread Ley Foon Tan
Add u-boot,dm-pre-reloc for sysmgr and clkmgr nodes to use it in SPL. In preparation to get base address from DT. Signed-off-by: Ley Foon Tan Reviewed-by: Simon Goldschmidt --- v3: - Fix missing '0' in clkmgr@ffd1 node. --- arch/arm/dts/socfpga-common-u-boot.dtsi | 8 arc

[U-Boot] [PATCH v4 0/4] arm: socfpga: Convert drivers from struct to defines

2019-10-10 Thread Ley Foon Tan
This is 4th version of patchset to convert reset, system and clock manager drivers to use #define instead of struct. This patchset get managers' base address from DT node instead of using #define. Patch 1 unchanged, patch 2,3,4 update commit message. Tested on Cyclone 5, Arria 10 and Stratix 10 d

[U-Boot] [PATCH v4 2/4] arm: socfpga: Convert reset manager from struct to defines

2019-10-10 Thread Ley Foon Tan
Convert reset manager for Gen5, Arria 10 and Stratix 10 from struct to defines. Change to get reset manager base address from DT node instead of using #define. Signed-off-by: Ley Foon Tan --- v4: - Update commit message about get base address from DT node. v3: - Remove "No functional change" i

[U-Boot] [PATCH v4 3/4] arm: socfpga: Convert system manager from struct to defines

2019-10-10 Thread Ley Foon Tan
Convert system manager for Gen5, Arria 10 and Stratix 10 from struct to defines. Change to get system manager base address from DT node instead of using #define. Signed-off-by: Ley Foon Tan --- v4: - Update commit message about get base address from DT node. v3: - Remove "No functional change"

[U-Boot] [PATCH v4 4/4] arm: socfpga: Convert clock manager from struct to defines

2019-10-10 Thread Ley Foon Tan
Convert clock manager for Gen5, Arria 10 and Stratix 10 from struct to defines. Change to get clock manager base address from DT node instead of using #define. Signed-off-by: Ley Foon Tan --- v4: - Update commit message about get base address from DT node. v3: - Remove "No functional change" i

Re: [U-Boot] [PATCH 085/126] x86: Update mrccache to support multiple caches

2019-10-10 Thread Bin Meng
Hi Simon, On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > With apollolake we need to support a normal cache, which almost never > changes and a much smaller 'variable' cache which changes every time. > > Update the code to add a cache type, use an array for the caches and use a > for loop

Re: [U-Boot] [PATCH 086/126] x86: Add mrccache support for a 'variable' cache

2019-10-10 Thread Bin Meng
Hi Simon, On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > Add support for a second cache type, for apollolake. > > Signed-off-by: Simon Glass > --- > > arch/x86/include/asm/mrccache.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/x86/include/asm/mrccache.h b/arch/x86/inc

Re: [U-Boot] [PATCH 087/126] x86: Move fsp_prepare_mrc_cache() to fsp1 directory

2019-10-10 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > This function needs to be different for FSP2, so move the existing > function into the fsp1 directory. Since it is only called from one file, > drop it from the header file. > > Signed-off-by: Simon Glass > --- > > arch/x86/include/asm/fsp/

Re: [U-Boot] [PATCH] ARM: socfpga: Fix FPGA bitstream loading code

2019-10-10 Thread Marek Vasut
On 10/10/19 7:15 AM, Simon Goldschmidt wrote: [...] >>> Have you dropped this? It's assigned to me in patchwork (I'm going >>> through the list of old items assigned to me...). >> >> I don't know, sorry. Apparently there isn't enough information to decide >> whether this patch is correct or not. >>

Re: [U-Boot] [PATCH V2 1/3] watchdog: designware: Migrate CONFIG_DESIGNWARE_WATCHDOG to Kconfig

2019-10-10 Thread Marek Vasut
On 10/10/19 8:28 AM, Ley Foon Tan wrote: > On Fri, Oct 4, 2019 at 7:04 PM Marek Vasut wrote: >> >> On 10/4/19 10:59 AM, Ley Foon Tan wrote: >>> On Thu, Oct 3, 2019 at 9:00 PM Marek Vasut wrote: Migrate CONFIG_DESIGNWARE_WATCHDOG to Kconfig and update the headers accordingly, no funct

Re: [U-Boot] [PATCH v3 3/4] arm: socfpga: Convert system manager from struct to defines

2019-10-10 Thread Anatolij Gustschin
On Thu, 10 Oct 2019 09:16:14 +0200 Simon Goldschmidt simon.k.r.goldschm...@gmail.com wrote: > On Thu, Oct 10, 2019 at 8:20 AM Ley Foon Tan wrote: > > > > Convert system manager for Gen5, Arria 10 and Stratix 10 from struct > > to defines. > > Same as 2/4: please add a description about the "re

Re: [U-Boot] [PATCH 2/4] arm64: xilinx: Move firmware functions from platform to driver

2019-10-10 Thread Luca Ceresoli
Hi Michal, On 04/10/19 16:27, Michal Simek wrote: > versal_pm_request() and invoke_smc() are almost the same. Only one > difference is that versal_pm_request is adding PM_SIP_SVC offset to api_id. > The patch is moving platform implementation to firmware driver code for > synchronization. > > Sig

Re: [U-Boot] [PATCH 2/3] ARM: imx6q_logic: Enable Pin muxing in SPL

2019-10-10 Thread Peng Fan
> Subject: [PATCH 2/3] ARM: imx6q_logic: Enable Pin muxing in SPL > > With the 256KB of OCRAM available to SPL now, there should be enough room > to enable the pinmuxing in SPL from the device tree. > > This patch enables SPL_PINCTRL et al and adds the serial and usdhc pin mux > references to the

Re: [U-Boot] [PATCH] rockchip: rk3399: defconfig: no need to reserve IRAM for SPL

2019-10-10 Thread Peter Robinson
On Thu, Oct 10, 2019 at 2:15 AM Kever Yang wrote: > > Hi Peter, > > On 2019/10/9 下午8:01, Peter Robinson wrote: > > On Wed, Oct 9, 2019 at 11:08 AM Kever Yang > > wrote: > >> We use to reserve IRAM to avoid the SPL text overlap with ATF M0 code, > >> and when we introduce the TPL, the SPL space i

Re: [U-Boot] [PATCH 2/4] arm64: xilinx: Move firmware functions from platform to driver

2019-10-10 Thread Michal Simek
On 10. 10. 19 10:22, Luca Ceresoli wrote: > Hi Michal, > > On 04/10/19 16:27, Michal Simek wrote: >> versal_pm_request() and invoke_smc() are almost the same. Only one >> difference is that versal_pm_request is adding PM_SIP_SVC offset to api_id. >> The patch is moving platform implementation to f

Re: [U-Boot] [PATCH v3 3/4] arm: socfpga: Convert system manager from struct to defines

2019-10-10 Thread Simon Goldschmidt
On Thu, Oct 10, 2019 at 10:09 AM Anatolij Gustschin wrote: > > On Thu, 10 Oct 2019 09:16:14 +0200 > Simon Goldschmidt simon.k.r.goldschm...@gmail.com wrote: > > > On Thu, Oct 10, 2019 at 8:20 AM Ley Foon Tan wrote: > > > > > > Convert system manager for Gen5, Arria 10 and Stratix 10 from struct >

Re: [U-Boot] [PATCH 088/126] x86: Set the DRAM banks to reflect real location

2019-10-10 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > At present with fsp a single DRAM bank is added which extends to the > whole size of memory. However there is typically only 2GB of memory > available below the 4GB boundary, and this is what is used by U-Boot while > running in 32-bit mode.

[U-Boot] [PATCH] firmware: zynqmp: Report error when xilinx_pm_request called from EL3

2019-10-10 Thread Michal Simek
Function should report error when called in EL3 context. Also report it as error instead 0 (success). Reported-by: Luca Ceresoli Signed-off-by: Michal Simek --- Based on https://lists.denx.de/pipermail/u-boot/2019-October/385462.html series. --- drivers/firmware/firmware-zynqmp.c | 6 -- 1

Re: [U-Boot] [PATCH 089/126] x86: Set up the MTRR for SDRAM

2019-10-10 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > Set up MTRRs for the FST SDRAM regions to improve performance. > > Signed-off-by: Simon Glass > --- > > arch/x86/lib/fsp/fsp_dram.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/arch/x86/lib/fsp/fsp_dram.c b/arch/x86/lib/fsp/f

Re: [U-Boot] [PATCH 090/126] x86: Update Kconfig options for FSP1

2019-10-10 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > Many of the Kconfig options do not apply to FSP2. Update them to use the > FSP_VERSION1 condition instead. > > Signed-off-by: Simon Glass > --- > > arch/x86/Kconfig | 14 +++--- > 1 file changed, 7 insertions(+), 7 deletions(-) > R

Re: [U-Boot] [PATCH v2 03/13] mailbox: zynqmp: ipi mailbox driver

2019-10-10 Thread Michal Simek
On 09. 10. 19 17:02, Luca Ceresoli wrote: > Hi Ibai, Michal, > > I had half-written a review of this patch and patch 4. Unfortunately I > didn't finish them before they got applied. I'll send them now anyway, > they are mostly nitpicking but you might consider them for a future > improvement. Sorr

Re: [U-Boot] [PATCH] firmware: zynqmp: Report error when xilinx_pm_request called from EL3

2019-10-10 Thread Luca Ceresoli
Hi, On 10/10/19 11:13, Michal Simek wrote: > Function should report error when called in EL3 context. Also report it as > error instead 0 (success). > > Reported-by: Luca Ceresoli > Signed-off-by: Michal Simek > --- > > Based on https://lists.denx.de/pipermail/u-boot/2019-October/385462.html >

Re: [U-Boot] [PATCH 091/126] x86: Don't imply TPL_OF_LIBFDT

2019-10-10 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > This option pulls in libfdt and therefore precludes getting the full > code-size savings of op-platdata. Drop it since it will be enabled anyway > if needed. > > Signed-off-by: Simon Glass > --- > > arch/Kconfig | 1 - > 1 file changed, 1 d

Re: [U-Boot] [PATCH v2 04/13] firmware: zynqmp: Add zynqmp-power support

2019-10-10 Thread Michal Simek
On 09. 10. 19 17:03, Luca Ceresoli wrote: > Hi, > > On 02/10/19 15:39, Michal Simek wrote: >> From: Ibai Erkiaga >> >> zynqmp-power driver for ZynqMP to handle the communication with the PMU >> firmware. Firmware driver just probes subnodes and power driver handles >> communication with PMU using

Re: [U-Boot] [PATCH 092/126] x86: Allow removal of standard PCH drivers

2019-10-10 Thread Bin Meng
Hi Simon, On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > These drivers are not needed on all platforms. While they are small, it > is useful in TPL to drop then. Add Kconfig control to allow this. > > Signed-off-by: Simon Glass > --- > > drivers/pch/Kconfig | 18 ++ >

Re: [U-Boot] [PATCH 093/126] x86: Allow interrupt to happen once

2019-10-10 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > At present the interrupt table is included in all phases of U-Boot. Allow > it to be omitted, e.g. in TPL, to reduce size. > > Signed-off-by: Simon Glass > --- > > arch/x86/cpu/Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-

Re: [U-Boot] [PATCH v3 3/4] arm: socfpga: Convert system manager from struct to defines

2019-10-10 Thread Anatolij Gustschin
On Thu, 10 Oct 2019 10:43:46 +0200 Simon Goldschmidt simon.k.r.goldschm...@gmail.com wrote: ... > > Why is this required? In the past we have rejected all new code adding > > defines instead of structs for register accesses. Have we changed our mind > > now? > > Who is we? U-Boot maintainers/c

[U-Boot] [PATCH v2] armv8: ls1028a: disable multimedia feature when not supported

2019-10-10 Thread Yuantian Tang
ls1028a has 4 personalities: ls1028a, ls1027a, ls1017a and ls1018a. Both ls1027a and ls1017a personalities are lower functionality version which doesn't support the multimedia subsystems, like LCD, GPU. To disable multimedia feature on non-multimedia version, set the status property to disabled in

[U-Boot] [PATCH] firmware: zynqmp: Clean up zynqmp_power_probe

2019-10-10 Thread Michal Simek
Fix error debug messages to be more accurate and aligned with debug message style in the whole file. And do not initialize ret variable because it is initialized later on and it is just additional step. Fixes: 1327d1678bd2 ("firmware: zynqmp: Add zynqmp-power support") Reported-by: Luca Ceresoli

Re: [U-Boot] [PATCH 095/126] x86: Don't include the BIOS emulator in TPL

2019-10-10 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > We don't generally have enough space to run this, so don't build it into > TPL. This helps reduce the size of TPL. > > Signed-off-by: Simon Glass > --- > > arch/x86/lib/Makefile | 2 ++ > 1 file changed, 2 insertions(+) > Reviewed-by: Bin

Re: [U-Boot] [PATCH v3 3/4] arm: socfpga: Convert system manager from struct to defines

2019-10-10 Thread Marek Vasut
On 10/10/19 11:29 AM, Anatolij Gustschin wrote: > On Thu, 10 Oct 2019 10:43:46 +0200 > Simon Goldschmidt simon.k.r.goldschm...@gmail.com wrote: > ... >>> Why is this required? In the past we have rejected all new code adding >>> defines instead of structs for register accesses. Have we changed our

Re: [U-Boot] [PATCH 096/126] x86: Add an option to include a FIT

2019-10-10 Thread Bin Meng
Hi Simon, On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > Many Intel SoCs require a FIT in order to boot properly. Add an option to > include this and enable it by default. > > Signed-off-by: Simon Glass > --- > > arch/x86/Kconfig | 9 + > arch/x86/dts/u-boot.dtsi | 6 ++

Re: [U-Boot] [PATCH v3 3/4] arm: socfpga: Convert system manager from struct to defines

2019-10-10 Thread Simon Goldschmidt
On Thu, Oct 10, 2019 at 11:29 AM Anatolij Gustschin wrote: > > On Thu, 10 Oct 2019 10:43:46 +0200 > Simon Goldschmidt simon.k.r.goldschm...@gmail.com wrote: > ... > > > Why is this required? In the past we have rejected all new code adding > > > defines instead of structs for register accesses. Ha

Re: [U-Boot] [PATCH 097/126] x86: Add support for newer CAR schemes

2019-10-10 Thread Bin Meng
Hi Simon, On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > Newer Intel SoCs have different ways of setting up cache-as-ram (CAR). > Add support for these along with suitable configuration options. > I wonder why do we need do this in U-Boot. Isn't FSP-T doing the CAR for us? > Signed-off

Re: [U-Boot] [PATCH 098/126] x86: Drop RESET_BASE

2019-10-10 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > Since x86 users binman now, we don't need this compile-time define. > > Signed-off-by: Simon Glass > --- > > arch/x86/cpu/config.mk | 1 - > 1 file changed, 1 deletion(-) > Reviewed-by: Bin Meng ___

Re: [U-Boot] [PATCH 099/126] x86: Drop RESET_SEG_SIZE

2019-10-10 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > This is not used anywhere now, so drop it. > > Signed-off-by: Simon Glass > --- > > arch/x86/Kconfig | 5 - > arch/x86/cpu/config.mk | 1 - > 2 files changed, 6 deletions(-) > Reviewed-by: Bin Meng __

Re: [U-Boot] [PATCH 100/126] x86: Disable microcode section for FSP2

2019-10-10 Thread Bin Meng
Hi Simon, On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > At present we don't support loading microcode with FSP2. The correct way > to do this is by adding it to the FIT. For now, disable including > microcode in the image. > > Signed-off-by: Simon Glass > --- > > arch/x86/Kconfig

Re: [U-Boot] [PATCH v3 3/4] arm: socfpga: Convert system manager from struct to defines

2019-10-10 Thread Anatolij Gustschin
On Thu, 10 Oct 2019 11:39:13 +0200 Marek Vasut ma...@denx.de wrote: ... > > The preferred way for I/O access is documented in [1], see "Use structures > > for I/O access" section. > > This seems to not scale and I keep running into the problem where a few > registers changed between various muta

Re: [U-Boot] [PATCH 101/126] x86: Use fsp command with FSP1

2019-10-10 Thread Bin Meng
Hi Simon, On Wed, Sep 25, 2019 at 11:02 PM Simon Glass wrote: > > The current 'fsp' command only works with FSP1. Update it to avoid trying > to build it with FSP2. > > Signed-off-by: Simon Glass > --- > > cmd/x86/Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a

Re: [U-Boot] [PATCH v2 02/16] doc: dfu: Add dfu documentation

2019-10-10 Thread Lukasz Majewski
Hi Patrick, > Add documentation for dfu stack and "dfu" command. > Reviewed-by: Lukasz Majewski > Signed-off-by: Patrick Delaunay > --- > > Changes in v2: > - Add dfu documentation > > doc/README.dfu | 144 > + 1 file changed, 144 > insertions

Re: [U-Boot] [PATCH 102/126] x86: Update .dtsi file for FSP2

2019-10-10 Thread Bin Meng
Hi Simon, On Wed, Sep 25, 2019 at 11:02 PM Simon Glass wrote: > > Include the IFWI section and the FSP-M binary. The FSP-T binary is not > currently used, as CAR is set up manually. > > Also drop the FSP binary as this relates only to FSP1. > > Signed-off-by: Simon Glass > --- > > arch/x86/dts/

Re: [U-Boot] [PATCH v3 3/4] arm: socfpga: Convert system manager from struct to defines

2019-10-10 Thread Marek Vasut
On 10/10/19 12:00 PM, Anatolij Gustschin wrote: > On Thu, 10 Oct 2019 11:39:13 +0200 > Marek Vasut wrote: > ... >>> The preferred way for I/O access is documented in [1], see "Use structures >>> for I/O access" section. >> >> This seems to not scale and I keep running into the problem where a few

Re: [U-Boot] [PATCH v2 03/16] dfu: sf: add partition support for nor backend

2019-10-10 Thread Lukasz Majewski
Hi Patrick, > Copy the partition support from NAND backend to SF, > support part and partubi option. > In case of ubi partition, erase the rest of the > partition as it is mandatory for UBI. > > for example: > > U-Boot> env set dfu_alt_info "spl part 0 1;\ > u-boot part 0 2;u-boot-env part 0 3;U

Re: [U-Boot] [PATCH] firmware: zynqmp: Clean up zynqmp_power_probe

2019-10-10 Thread Luca Ceresoli
Hi, On 10/10/19 11:33, Michal Simek wrote: > Fix error debug messages to be more accurate and aligned with debug message > style in the whole file. > And do not initialize ret variable because it is initialized later on and > it is just additional step. > > Fixes: 1327d1678bd2 ("firmware: zynqmp:

Re: [U-Boot] [PATCH] cmd: avoid decimal conversion

2019-10-10 Thread Michal Simek
On 09. 10. 19 19:28, Simon Goldschmidt wrote: > Am 09.10.2019 um 18:26 schrieb Tom Rini: >> On Tue, Oct 08, 2019 at 10:48:39AM +0200, Michal Simek wrote: >>> Hi Tom, >>> >>> On 19. 09. 19 15:28, Michal Simek wrote: On 13. 09. 19 17:09, Tom Rini wrote: > On Wed, Sep 11, 2019 at 03:39:53PM +

Re: [U-Boot] [PATCH] cmd: avoid decimal conversion

2019-10-10 Thread Simon Goldschmidt
On Thu, Oct 10, 2019 at 12:46 PM Michal Simek wrote: > > On 09. 10. 19 19:28, Simon Goldschmidt wrote: > > Am 09.10.2019 um 18:26 schrieb Tom Rini: > >> On Tue, Oct 08, 2019 at 10:48:39AM +0200, Michal Simek wrote: > >>> Hi Tom, > >>> > >>> On 19. 09. 19 15:28, Michal Simek wrote: > On 13. 09

[U-Boot] [PATCH v1] usb: ehci-mx6: Fix bus enumeration for iMX7 SoCs

2019-10-10 Thread Igor Opaniuk
From: Igor Opaniuk This fixes the issues with calculation of controller indexes in ehci_usb_bind() for iMX7, as USB controllers on iMX7 SoCs aren't placed next to each other, and their addresses incremented by 0x1000. Example of USB nodes for iMX7S/D: usbotg1: usb@30b1 { compatible = "f

[U-Boot] [RESEND PATCH v1] usb: ehci-mx6: Fix bus enumeration for iMX7 SoCs

2019-10-10 Thread Igor Opaniuk
From: Igor Opaniuk This fixes the issues with calculation of controller indexes in ehci_usb_bind() for iMX7, as USB controllers on iMX7 SoCs aren't placed next to each other, and their addresses incremented by 0x1. Example of USB nodes for iMX7S/D: usbotg1: usb@30b1 { compatible = "

[U-Boot] [PATCH] arm: add initial support for the Phytium FT2004 SoC

2019-10-10 Thread liu hao
This adds platform code and the device tree for the Phytium FT2004 SoC. The initial support comprises the UART and the GMAC. Signed-off-by: Steven Hao --- MAINTAINERS | 6 +++ arch/arm/Kconfig | 8 arch/arm/dts/Makefile| 2 + arch/arm/dts/

Re: [U-Boot] [PATCH] ARM: dts: zynq: enablement of coresight topology

2019-10-10 Thread qwang2
On 10/9/19 10:07 PM, Michal Simek wrote: From: Zumeng Chen This patch is to build the coresight topology structure of zynq-7000 series according to the docs of coresight and userguide of zynq-7000. Signed-off-by: Zumeng Chen Signed-off-by: Quanyang Wang Signed-off-by: Michal Simek --- a

[U-Boot] [PATCH] test/py: hush_if_test: Add tests to cover octal/hex values

2019-10-10 Thread Michal Simek
Extend test suite to cover also automatic octal/hex converstions which haven't been implemented in past. Signed-off-by: Michal Simek --- Depends on https://lists.denx.de/pipermail/u-boot/2019-September/383309.html There are of course other tests which we can run but not sure if make sense to ha

Re: [U-Boot] [PATCH v1] usb: ehci-mx6: Fix bus enumeration for iMX7 SoCs

2019-10-10 Thread Marek Vasut
On 10/10/19 1:25 PM, Igor Opaniuk wrote: [...] >* from which it derives offsets in the PHY and ANATOP register sets. >* >* Here we attempt to calculate these indexes from DT information as > - * well as we can. The USB controllers on all existing iMX6/iMX7 SoCs > -

[U-Boot] [PATCH v1 1/1] imx: support for conga-QMX8 board

2019-10-10 Thread Oliver Graute
Add i.MX8QM qmx8 congatec board support Boot log as below: U-Boot 2019.10-2-gfe063598db (Oct 10 2019 - 10:29:07 +0200) CPU: NXP i.MX8QM RevB A53 at 1200 MHz Model: congatec QMX8 Board: conga-QMX8 Build: SCFW 65afe5f6 Boot: SD2 DRAM: 6 GiB MMC: FSL_SDHC: 0, FSL_SDHC: 1, FSL_SDHC: 2 Load

[U-Boot] [PATCH 0/1] imx: support for conga-QMX8 board

2019-10-10 Thread Oliver Graute
Hello list, this patch adds support for the congatec QMX8 Qseven board. Best regards, Oliver Oliver Graute (1): imx: support for conga-QMX8 board arch/arm/dts/Makefile | 1 + arch/arm/dts/imx8qm-cgtqmx8.dts | 427 + arch/arm/mach-imx/Makefile

Re: [U-Boot] [PATCH v1] usb: ehci-mx6: Fix bus enumeration for iMX7 SoCs

2019-10-10 Thread Igor Opaniuk
Hi Marek On Thu, Oct 10, 2019 at 2:47 PM Marek Vasut wrote: > > On 10/10/19 1:25 PM, Igor Opaniuk wrote: > [...] > >* from which it derives offsets in the PHY and ANATOP register sets. > >* > >* Here we attempt to calculate these indexes from DT information as > > - *

Re: [U-Boot] [PATCH v1] usb: ehci-mx6: Fix bus enumeration for iMX7 SoCs

2019-10-10 Thread Marek Vasut
On 10/10/19 2:29 PM, Igor Opaniuk wrote: > Hi Marek Hi Igor, > On Thu, Oct 10, 2019 at 2:47 PM Marek Vasut wrote: >> >> On 10/10/19 1:25 PM, Igor Opaniuk wrote: >> [...] >>>* from which it derives offsets in the PHY and ANATOP register sets. >>>* >>>* Here we attempt to ca

Re: [U-Boot] [PATCH v1] usb: ehci-mx6: Fix bus enumeration for iMX7 SoCs

2019-10-10 Thread Igor Opaniuk
On Thu, Oct 10, 2019 at 3:43 PM Marek Vasut wrote: > > On 10/10/19 2:29 PM, Igor Opaniuk wrote: > > Hi Marek > > Hi Igor, > > > On Thu, Oct 10, 2019 at 2:47 PM Marek Vasut wrote: > >> > >> On 10/10/19 1:25 PM, Igor Opaniuk wrote: > >> [...] > >>>* from which it derives offsets in the PHY a

Re: [U-Boot] [PATCH v1] usb: ehci-mx6: Fix bus enumeration for iMX7 SoCs

2019-10-10 Thread Marek Vasut
On 10/10/19 2:55 PM, Igor Opaniuk wrote: > On Thu, Oct 10, 2019 at 3:43 PM Marek Vasut wrote: >> >> On 10/10/19 2:29 PM, Igor Opaniuk wrote: >>> Hi Marek >> >> Hi Igor, >> >>> On Thu, Oct 10, 2019 at 2:47 PM Marek Vasut wrote: On 10/10/19 1:25 PM, Igor Opaniuk wrote: [...] >

[U-Boot] [PATCH v2] usb: ehci-mx6: Fix bus enumeration for iMX7 SoCs

2019-10-10 Thread Igor Opaniuk
From: Igor Opaniuk This fixes the issues with calculation of controller indexes in ehci_usb_bind() for iMX7, as USB controllers on iMX7 SoCs aren't placed next to each other, and their addresses incremented by 0x1. Example of USB nodes for iMX7S/D: usbotg1: usb@30b1 { compatible = "

[U-Boot] [PATCH v3 1/9] imx: tpc70: config: Add script commands to update u-boot and OE's wic

2019-10-10 Thread Lukasz Majewski
Signed-off-by: Lukasz Majewski --- Changes in v3: None include/configs/kp_imx6q_tpc.h | 21 + 1 file changed, 21 insertions(+) diff --git a/include/configs/kp_imx6q_tpc.h b/include/configs/kp_imx6q_tpc.h index 2435ebbc7f..fae45cbf3e 100644 --- a/include/configs/kp_imx6q_tpc

[U-Boot] [PATCH v3 0/9] dm: Convert TPC70 to use DM and DTS in SPL and u-boot proper

2019-10-10 Thread Lukasz Majewski
This patch series converts imx6q based tpc70 board to use driver model and device tree description in SPL and U-Boot proper. All the non-DM parts of the code has been removed. This patch series also uses the Common Clock Framework [CCF] as a base for clock management on imx6q board. Those patches

[U-Boot] [PATCH v3 6/9] imx: tpc70: dts: Add TPC70 board (imx6q based) device tree description

2019-10-10 Thread Lukasz Majewski
This commit defines the TPC70 imx6q board with device tree description. Signed-off-by: Lukasz Majewski --- Changes in v3: - Squash 'imx: tpc70: dts: Add TPC70 board (imx6q based) device tree description' with 'imx: tpc70: DTS: Explicitly add imx6q-kp.dtb to Makefile for DTB compilation' arch/

[U-Boot] [PATCH v3 3/9] imx: tpc70: Add board_boot_order() to distinguish between eMMC and SD boot

2019-10-10 Thread Lukasz Majewski
The TPC70 can boot from SD card (debug/development) and eMMC (production). The board_boot_order() function provides a run time check for the device from which one wants to boot (it is selected by GPIO pins setup). Moreover, a fallback to SD card is provided if the detection is not possible or work

[U-Boot] [PATCH v3 4/9] imx: tpc70: cosmetic: Replace magic numbers when setting ENET clock

2019-10-10 Thread Lukasz Majewski
This is a cosmetic change, just to use proper define instead of magic numbers. Signed-off-by: Lukasz Majewski --- Changes in v3: None board/k+p/kp_imx6q_tpc/kp_imx6q_tpc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/board/k+p/kp_imx6q_tpc/kp_imx6q_tpc.c b/board/k+p/k

[U-Boot] [PATCH v3 5/9] imx: tpc70: led: Enable LED default state

2019-10-10 Thread Lukasz Majewski
This change sets the default state of LEDs on TPC70. Signed-off-by: Lukasz Majewski --- Changes in v3: None board/k+p/kp_imx6q_tpc/kp_imx6q_tpc.c | 4 1 file changed, 4 insertions(+) diff --git a/board/k+p/kp_imx6q_tpc/kp_imx6q_tpc.c b/board/k+p/kp_imx6q_tpc/kp_imx6q_tpc.c index 144d2c5

[U-Boot] [PATCH v3 2/9] imx: tpc70: config: Update TPC70 config to support eMMC's boot0 SPL update

2019-10-10 Thread Lukasz Majewski
The TPC70 can boot from eMMC's boot0. This patch allows it to update this HW partition's SPL. Signed-off-by: Lukasz Majewski --- Changes in v3: None include/configs/kp_imx6q_tpc.h | 5 + 1 file changed, 5 insertions(+) diff --git a/include/configs/kp_imx6q_tpc.h b/include/configs/kp_imx6q

[U-Boot] [PATCH v3 7/9] imx: dts: Add u-boot specific set of device tree properties for tpc70

2019-10-10 Thread Lukasz Majewski
This commit adds new file - imx6q-kp-u-boot.dtsi with a set of u-boot specific properties for imx6q KP device. Signed-off-by: Lukasz Majewski --- Changes in v3: None arch/arm/dts/imx6q-kp-u-boot.dtsi | 59 +++ 1 file changed, 59 insertions(+) create mode 100644 arc

[U-Boot] [PATCH v3 8/9] imx: tpc70: Convert TPC70 (imx6q) board to use DM/DTS in SPL and u-boot

2019-10-10 Thread Lukasz Majewski
This patch converts the TPC70 to use driver model and device tree description in both SPL and u-boot proper. Notable changes (DM/DTS conversion): - PINCTRL{_IMX6} - DM_I2C - enable 'regulator' and 'pmic' commands - DM_MMC and BLK (USDHC) - DM_ETH - DM WDT (including SYSRESET) Signed-off-by: Luka

[U-Boot] [PATCH v3 9/9] imx: config: Update KP's TPC70 config to support SWUpdate

2019-10-10 Thread Lukasz Majewski
This patch updates envs responsible for using USB pendrive as a SWUpdate based tool for recovery and update. Signed-off-by: Lukasz Majewski --- Changes in v3: None include/configs/kp_imx6q_tpc.h | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/include/configs/kp_imx6q

Re: [U-Boot] [PATCH] test/py: hush_if_test: Add tests to cover octal/hex values

2019-10-10 Thread Stephen Warren
On 10/10/19 5:44 AM, Michal Simek wrote: Extend test suite to cover also automatic octal/hex converstions which haven't been implemented in past. Acked-by: Stephen Warren ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-b

Re: [U-Boot] [PATCH] test/py: hush_if_test: Add tests to cover octal/hex values

2019-10-10 Thread Simon Goldschmidt
Stephen Warren schrieb am Do., 10. Okt. 2019, 17:56: > On 10/10/19 5:44 AM, Michal Simek wrote: > > Extend test suite to cover also automatic octal/hex converstions which > > haven't been implemented in past. > > Acked-by: Stephen Warren > Reviewed-by: Simon Goldschmidt >

Re: [U-Boot] [PATCH] test/py: hush_if_test: Add tests to cover octal/hex values

2019-10-10 Thread Simon Glass
Hi Michal, On Thu, 10 Oct 2019 at 05:44, Michal Simek wrote: > > Extend test suite to cover also automatic octal/hex converstions which > haven't been implemented in past. > > Signed-off-by: Michal Simek > --- > > Depends on https://lists.denx.de/pipermail/u-boot/2019-September/383309.html > > T

Re: [U-Boot] [PATCH v2 33/38] x86: fsp: Allow the HOBs to be used after relocation

2019-10-10 Thread Simon Glass
Hi Bin, On Thu, 3 Oct 2019 at 20:23, Bin Meng wrote: > > Hi Simon, > > On Wed, Sep 25, 2019 at 10:13 PM Simon Glass wrote: > > > > Allocate memory for the HOBs and copy them before relocation. This ensures > > that they can still be accessed after relocation. > > > > This is needed when relocati

Re: [U-Boot] [PATCH] sandbox: fix build error due to missing struct udevice definition

2019-10-10 Thread Simon Glass
On Thu, 3 Oct 2019 at 21:41, AKASHI Takahiro wrote: > > Without this patch, compiling may potentially fail. > > Signed-off-by: AKASHI Takahiro > --- > arch/sandbox/include/asm/u-boot-sandbox.h | 2 ++ > 1 file changed, 2 insertions(+) Reviewed-by: Simon Glass __

Re: [U-Boot] [PATCH 078/126] x86: Define the SPL image start

2019-10-10 Thread Simon Glass
Hi Bin, On Thu, 10 Oct 2019 at 01:09, Bin Meng wrote: > > Hi Simon, > > On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > > > Define this symbol so that we can use binman symbols correctly. > > > > Signed-off-by: Simon Glass > > --- > > > > arch/x86/cpu/u-boot-spl.lds | 5 - > > 1 fi

Re: [U-Boot] [PATCH 1/2] rockchip: make_fit_atf.py: allow inclusion of a tee binary

2019-10-10 Thread Simon Glass
Hi Heiko, On Tue, 1 Oct 2019 at 14:23, Heiko Stuebner wrote: > > A trusted execution environment should also get loaded as loadable from > a fit image, so add the possibility to present a tee.elf to make_fit_atf.py > that then gets included as additional loadable into the generated its. > > For e

Re: [U-Boot] [PATCH 000/126] x86: Add initial support for apollolake

2019-10-10 Thread Simon Glass
Hi Bin, On Mon, 7 Oct 2019 at 08:31, Bin Meng wrote: > > Hi Simon, > > On Wed, Oct 2, 2019 at 8:34 PM Bin Meng wrote: > > > > Hi Simon, > > > > On Wed, Oct 2, 2019 at 10:15 AM Simon Glass wrote: > > > > > > Hi Bin, > > > > > > On Wed, 25 Sep 2019 at 08:58, Simon Glass wrote: > > > > > > > > Ap

Re: [U-Boot] [PATCH] binman: Use qemu-x86_defconfig in the example

2019-10-10 Thread Simon Glass
On Wed, 2 Oct 2019 at 19:07, Bin Meng wrote: > > The doc currently uses sandbox_defconfig as examples of enabling > debug/verbose output of binman. However during a sandbox build it > does not call binman at all. Change it to qemu-x86_defconfig. > > Signed-off-by: Bin Meng > --- > > tools/binman

Re: [U-Boot] [PATCH 1/3] buildman: Perform tests in /tmp

2019-10-10 Thread Simon Glass
On Fri, 4 Oct 2019 at 09:13, Tom Rini wrote: > > We may not always be able to write to the default output directory but > we will always have a usable /tmp. Make the buildman tests use /tmp > > Cc: Simon Glass > Signed-off-by: Tom Rini > --- > tools/buildman/func_test.py | 24 -

Re: [U-Boot] [PATCH 1/1] tools: provide cbfstool

2019-10-10 Thread Simon Glass
Hi, On Sun, 6 Oct 2019 at 15:06, Tom Rini wrote: > > On Sun, Oct 06, 2019 at 10:50:45PM +0200, Heinrich Schuchardt wrote: > > > 'make tests' requires cbfstool. But this file is not built by U-Boot. If so this is a bug. The tests are supposed to detect that cbfstool is missing and skip the compar

Re: [U-Boot] [PATCH 050/126] x86: timer: Reduce timer code size in TPL on Intel CPUs

2019-10-10 Thread Simon Glass
Hi Bin, On Sat, 5 Oct 2019 at 08:36, Bin Meng wrote: > > Hi Simon, > > On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > > > Most of the timer-calibration methods are not needed on recent Intel CPUs > > and just increase code size. Add an option to use the known-good way to > > get the clo

Re: [U-Boot] [PATCH 079/126] x86: Reduce mrccache record alignment size

2019-10-10 Thread Simon Glass
Hi Bin, On Wed, 9 Oct 2019 at 23:09, Bin Meng wrote: > > Hi Simon, > > On Wed, Sep 25, 2019 at 10:59 PM Simon Glass wrote: > > > > At present the records are 4KB in size. This is unnecessarily large when > > the SPI-flash erase size is 256 bytes. Reduce it so it will be more > > But this will br

Re: [U-Boot] [PATCH 1/1] cbfs: do not pack struct cbfs_cachenode

2019-10-10 Thread Simon Glass
On Sun, 6 Oct 2019 at 15:38, Heinrich Schuchardt wrote: > > With the __packed attribute sandbox_defconfig cannot be compiled with GCC > 9.2.1: > > fs/cbfs/cbfs.c: In function ‘file_cbfs_fill_cache’: > fs/cbfs/cbfs.c:164:16: error: taking address of packed member of > ‘struct cbfs_cachenode’ may re

Re: [U-Boot] [PATCH 1/2] rockchip: make_fit_atf.py: allow inclusion of a tee binary

2019-10-10 Thread Heiko Stübner
Hi Simon, Am Donnerstag, 10. Oktober 2019, 19:06:12 CEST schrieb Simon Glass: > On Tue, 1 Oct 2019 at 14:23, Heiko Stuebner wrote: > > A trusted execution environment should also get loaded as loadable from > > a fit image, so add the possibility to present a tee.elf to make_fit_atf.py > > that t

[U-Boot] Add support for imxrt - Sdram

2019-10-10 Thread Giulio Benetti
Hi Fabio and Stefano, about imxrt porting, would it make sense to add an SDRAM driver to setup ram or is it enough to provide .cfg file with DCD? This question is specific for SDRAM of course. Thanks in advance Best regards -- Giulio Benetti Benetti Engineering sas __

Re: [U-Boot] [PATCH 1/1] tools: provide cbfstool

2019-10-10 Thread Tom Rini
On Thu, Oct 10, 2019 at 11:06:27AM -0600, Simon Glass wrote: > Hi, > > On Sun, 6 Oct 2019 at 15:06, Tom Rini wrote: > > > > On Sun, Oct 06, 2019 at 10:50:45PM +0200, Heinrich Schuchardt wrote: > > > > > 'make tests' requires cbfstool. But this file is not built by U-Boot. > > If so this is a bug

Re: [U-Boot] [PATCH 1/8] power: domain: meson-gx-pwrc: add G12A support

2019-10-10 Thread Anatolij Gustschin
On Fri, 30 Aug 2019 14:09:21 +0200 Neil Armstrong narmstr...@baylibre.com wrote: > Add Amlogic G12A support for the gx-pwrc driver, aligned on the > Linux v5.2 driver. > > Signed-off-by: Neil Armstrong Reviewed-by: Anatolij Gustschin -- Anatolij ___

Re: [U-Boot] [PATCH 2/8] clk: meson: g12a: add support for VPU/HDMI clocks

2019-10-10 Thread Anatolij Gustschin
On Fri, 30 Aug 2019 14:09:22 +0200 Neil Armstrong narmstr...@baylibre.com wrote: > Add necessary clock support to set up clock for the VPU and > HDMI support. > > Signed-off-by: Neil Armstrong Reviewed-by: Anatolij Gustschin -- Anatolij ___ U-Boot m

Re: [U-Boot] [PATCH 3/8] video: meson: remove power domain get

2019-10-10 Thread Anatolij Gustschin
On Fri, 30 Aug 2019 14:09:23 +0200 Neil Armstrong narmstr...@baylibre.com wrote: > Remove getting and enabling the node power domain since it's now handled > by the dm core directly. > > Signed-off-by: Neil Armstrong Reviewed-by: Anatolij Gustschin -- Anatolij

Re: [U-Boot] [PATCH 5/8] video: meson: add compatible for Amlogic G12A

2019-10-10 Thread Anatolij Gustschin
On Fri, 30 Aug 2019 14:09:25 +0200 Neil Armstrong narmstr...@baylibre.com wrote: > Finally add the Amlogic G12A SoC compatible for the VPU driver. > > Signed-off-by: Neil Armstrong Reviewed-by: Anatolij Gustschin -- Anatolij ___ U-Boot mailing list

Re: [U-Boot] [PATCH 6/8] arm: meson: board-g12a: Setup VPU in fdt

2019-10-10 Thread Anatolij Gustschin
On Fri, 30 Aug 2019 14:09:26 +0200 Neil Armstrong narmstr...@baylibre.com wrote: > If VIDEO_MESON is enabled, we need to setup the fdt for the framebuffer. > > Call meson_vpu_rsv_fb() which reserves the framebuffer memory region for > EFI, and sets up simple-framebuffer nodes if simplefb support

Re: [U-Boot] [PATCH 7/8] ARM: dts: meson-g12a: add U-Boot specific DT for graphics

2019-10-10 Thread Anatolij Gustschin
On Fri, 30 Aug 2019 14:09:27 +0200 Neil Armstrong narmstr...@baylibre.com wrote: > Like the meson-gx support, add the U-Boot specific bits in DT > to support graphics on G12A SoCs. > > Signed-off-by: Neil Armstrong Reviewed-by: Anatolij Gustschin -- Anatolij __

  1   2   >