Re: [U-Boot] [PATCH 013/126] dm: test: Correct a stray backslash in dm_test_destroy()

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > This should perhaps be a period but it is not necessary. Remove it. > > Signed-off-by: Simon Glass > --- > > test/dm/test-main.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/test/dm/test-main.c

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

2019-10-04 Thread Marek Vasut
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 functional change. The S10 enables the WDT only in >> SPL, but does not enable it in U-Boot itself, hence

Re: [U-Boot] [PATCH] MAINTAINERS, git-mailrc: Update the maintainer for socfpga

2019-10-04 Thread Marek Vasut
On 10/4/19 10:43 AM, Ley Foon Tan wrote: > This updates MAINTAINERS and git-mailrc to add me as maintainer for > socfpga. > > Signed-off-by: Ley Foon Tan Acked-by: Marek Vasut ___ U-Boot mailing list U-Boot@lists.denx.de

Re: [U-Boot] [PATCH 007/126] dm: test: Fix running of multiple test from command line

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > At present when multiple 'ut dm' commands are executed, all but the first > is run with a flat tree, even if live tree is enabled. This is because the > live tree node pointer is set to NULL and never restored. > > This does not affect

Re: [U-Boot] [PATCH V2 2/3] watchdog: designware: Convert to DM and DT probing

2019-10-04 Thread Ley Foon Tan
On Thu, Oct 3, 2019 at 9:00 PM Marek Vasut wrote: > > Convert the designware watchdog timer driver to DM and add DT probing > support. Perform minor coding style clean up, like drop superfluous > braces. There ought to be no functional change. All watchdog DT nodes with compatible "snps,dw-wdt"

[U-Boot] [PATCH 1/2] mx28evk: README: Fix location for the generic mxs README

2019-10-04 Thread Fabio Estevam
doc/README.mxs no longer exists. It has been renamed doc/imx/common/mxs.txt, so fix the mx28evk README accordingly. Signed-off-by: Fabio Estevam --- board/freescale/mx28evk/README | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/board/freescale/mx28evk/README

[U-Boot] [PATCH 2/2] imx: mxs: Fix location for the elftosb tool

2019-10-04 Thread Fabio Estevam
The Denx FTP location is no longer reachable. Switch to the Timesys repository instead. Signed-off-by: Fabio Estevam --- doc/imx/common/mxs.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/imx/common/mxs.txt b/doc/imx/common/mxs.txt index e23ab9cc6d..372062c0ea

[U-Boot] [PATCH] MAINTAINERS, git-mailrc: Update the maintainer for socfpga

2019-10-04 Thread Ley Foon Tan
This updates MAINTAINERS and git-mailrc to add me as maintainer for socfpga. Signed-off-by: Ley Foon Tan --- MAINTAINERS| 1 + doc/git-mailrc | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index e752e4b3de..a9a1049eaf 100644 ---

Re: [U-Boot] [PATCH] efi_loader: device_path: allow for arbitrary length of file path

2019-10-04 Thread Heinrich Schuchardt
On 10/4/19 6:45 AM, AKASHI Takahiro wrote: This patch will lift the upper limit of maximum path length. Signed-off-by: AKASHI Takahiro Thanks for addressing this issue. Please, have a look at the side effects to our implementation of the device path to text protocol. Here I assumed that a

Re: [U-Boot] [PATCH 020/126] sandbox: pci: Drop the get_devfn() method

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > This method is not used anymore since the bus/device/function of PCI > devices can be obtained from their (parent's per-child) platform data. > Drop it. > > Signed-off-by: Simon Glass > --- > > drivers/misc/swap_case.c | 8 >

Re: [U-Boot] [PATCH 019/126] sandbox: swap_case: Use statics where possible

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > Some functions and a struct should be marked static since they are not > used outside this file. Update them. > > Signed-off-by: Simon Glass > --- > > drivers/misc/swap_case.c | 10 +- > 1 file changed, 5 insertions(+), 5

Re: [U-Boot] [PATCH 017/126] sandbox: Rename PCI ID for swap_case to be more specific

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > Rename this ID to SANDBOX_PCI_SWAP_CASE_EMUL_ID since it is more > descriptive and allows us to add new PCI emulators without any conflict or > confusion. > > Signed-off-by: Simon Glass > --- > > arch/sandbox/include/asm/test.h | 2 +- >

Re: [U-Boot] [PATCH 015/126] sandbox: spmi: Add ranges property for address translation

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > At present address translation does not work since there is no ranges > property in the spmi nodes. Add empty ranges properties and a little more > logging so that this shows the error: > >/tmp/b/sandbox/u-boot -d

Re: [U-Boot] [PATCH 018/126] sandbox: Add support for clrsetio_32() and friends

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > These functions are available on x86 but not sandbox. They are useful > shortcuts and clarify the code, so add them to sandbox. > > Signed-off-by: Simon Glass > --- > > arch/sandbox/include/asm/io.h | 42 ++-

Re: [U-Boot] [PATCH 024/126] sandbox: pci: Increase the memory space

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > Increase the memory space so we can support the p2sb bus which needs > multiples of 1MB. > > Signed-off-by: Simon Glass > --- > > arch/sandbox/dts/test.dts | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Reviewed-by: Bin Meng

Re: [U-Boot] [PATCH 016/126] sandbox: mmc: Fix up MMC emulator for valgrind

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > At present running sandbox with valgrind produces some warnings due to the > MMC emulator not filling in all the expected fields. Fix it. > > Signed-off-by: Simon Glass > --- > > drivers/mmc/sandbox_mmc.c | 4 > 1 file changed, 4

Re: [U-Boot] [PATCH 021/126] sandbox: net: Suppress the MAC-address warnings

2019-10-04 Thread Bin Meng
Hi Simon, On Sat, Oct 5, 2019 at 10:16 AM Bin Meng wrote: > > Hi Simon, > > On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > > > These warnings appear every thing sandbox is run (see below) and dwarf the > > actual useful output. Suppress them. > > > > Hopefully at some point a way can

Re: [U-Boot] [PATCH 021/126] sandbox: net: Suppress the MAC-address warnings

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > These warnings appear every thing sandbox is run (see below) and dwarf the > actual useful output. Suppress them. > > Hopefully at some point a way can be found to fix the root cause. > > U-Boot 2019.10-rc2 > > Model: sandbox > DRAM: 128

Re: [U-Boot] [PATCH 023/126] sandbox: Add a -T flag to use the test device tree

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > U-Boot already supports using -D to indicate that it should use the normal > device tree. It is sometimes useful to run with the test device tree, e.g. > when running a test. Add a -T option for this along with some > documentation. > > It

Re: [U-Boot] [PATCH 026/126] pci: sandbox: Move the emulators into their own node

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > Sandbox i2c works using emulation drivers which are currently children of pci > the i2c device: pci > > pci-controller { > pci@1f,0 { > compatible = "pci-generic"; >

Re: [U-Boot] [PATCH 028/126] pci: Show the result of binding a device

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > Update the debugging info a little to show the result of trying to bind > a PCI device. > > Signed-off-by: Simon Glass > --- > > drivers/pci/pci-uclass.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > Reviewed-by: Bin Meng

Re: [U-Boot] [PATCH 029/126] pci: Disable autoconfig in SPL

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > At present U-Boot runs autoconfig in SPL but this is best left to U-Boot > proper. For TPL and SPL we can normally used fixed BARs and save code size > and time. > > Signed-off-by: Simon Glass > --- > > drivers/pci/pci-uclass.c | 2 +- > 1

Re: [U-Boot] [PATCH 027/126] pci: sandbox: Probe PCI emulation devices when used

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > At present PCI emulation devices are not probed before use, since they > used to be children of the device that used them, and children cannot be > probed before their parents. > > Now that PCI emulation devices are attached to the root

Re: [U-Boot] [PATCH 030/126] pci: Correct 'specifified' and 'Plese' typos

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > Fix these spelling errors the header file and documentation. in the header > > Fix a small typo in the PCI documentation. > > Signed-off-by: Simon Glass > --- > > doc/driver-model/pci-info.rst | 2 +- > include/pci.h | 2

Re: [U-Boot] [PATCH 2/8] ARM: socfpga: arria10: Sync A10 SoCDK devicetrees

2019-10-04 Thread Marek Vasut
On 10/5/19 12:30 AM, Dalon Westergreen wrote: > From: Dalon Westergreen > > Sync devicetree from 5.2 kernel. Linux 5.3 was already released, can you update this ? Also, make sure to list the exact commit ID in the commit message when importing stuff from Linux.

Re: [U-Boot] [PATCH 3/8] ARM: socfpga: arria10: Add common u-boot devicetree include

2019-10-04 Thread Marek Vasut
On 10/5/19 12:30 AM, Dalon Westergreen wrote: > From: Dalon Westergreen > > Add a common u-boot devicetree include file for the SocFPGA > Arria10 device. Isn't arch/arm/dts/socfpga_arria10_handoff_u-boot.dtsi doing basically the same thing, except more fine-grained ? > diff --git

Re: [U-Boot] [PATCH 1/8] ARM: socfpga: arria10: Add qts-filter for arria10 socfpga

2019-10-04 Thread Marek Vasut
On 10/5/19 12:30 AM, Dalon Westergreen wrote: > From: Dalon Westergreen > > Add a script to process hps handoff data and generate a header > for inclusion in u-boot specific devicetree addons. The header > should be included in the top level u-boot.dtsi. A10 should already be completely DT

Re: [U-Boot] [PATCH 4/8] ARM: socfpga: arria10: Add generic handoff devicetree include

2019-10-04 Thread Marek Vasut
On 10/5/19 12:30 AM, Dalon Westergreen wrote: > From: Dalon Westergreen > > Generic handoff devicetree include uses a header generated by > the qts-filter-a10.sh script in mach-socfpga. The script > creates the header based on design specific implementations > for clock and pinmux

Re: [U-Boot] [PATCH 025/126] sandbox: Allow use of real I/O with readl(), etc.

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > At present these functions are stubbed out. For more comprehensive testing > with PCI devices it is useful to be able to fully emulate I/O access. Add > simple implementations for these. > > Signed-off-by: Simon Glass > --- > >

[U-Boot] [PATCH 3/8] ARM: socfpga: arria10: Add common u-boot devicetree include

2019-10-04 Thread Dalon Westergreen
From: Dalon Westergreen Add a common u-boot devicetree include file for the SocFPGA Arria10 device. Signed-off-by: Dalon Westergreen --- .../dts/socfpga_arria10-common-u-boot.dtsi| 206 ++ 1 file changed, 206 insertions(+) create mode 100644

[U-Boot] [PATCH 0/8] ARM: socfpga: arria10: Cleanup devicetree and

2019-10-04 Thread Dalon Westergreen
From: Dalon Westergreen This series sync the arria10 devicetree to the kernel devicetree and cleans up the inclusion of u-boot specific requirements. It also adds a new qts-filter-a10.sh script to allow for generation of project specific settings required for configuration of the device clocks,

[U-Boot] [PATCH 1/8] ARM: socfpga: arria10: Add qts-filter for arria10 socfpga

2019-10-04 Thread Dalon Westergreen
From: Dalon Westergreen Add a script to process hps handoff data and generate a header for inclusion in u-boot specific devicetree addons. The header should be included in the top level u-boot.dtsi. Signed-off-by: Dalon Westergreen --- arch/arm/mach-socfpga/qts-filter-a10.sh | 141

[U-Boot] [PATCH 2/8] ARM: socfpga: arria10: Sync A10 SoCDK devicetrees

2019-10-04 Thread Dalon Westergreen
From: Dalon Westergreen Sync devicetree from 5.2 kernel. Signed-off-by: Dalon Westergreen --- arch/arm/dts/socfpga_arria10.dtsi| 104 ++- arch/arm/dts/socfpga_arria10_socdk.dtsi | 75 +++-- arch/arm/dts/socfpga_arria10_socdk_sdmmc.dts | 53 +-

[U-Boot] [PATCH 5/8] ARM: socfpga: arria10: Add handoff header for A10 SoCDK SDMMC

2019-10-04 Thread Dalon Westergreen
From: Dalon Westergreen Add the qts-filter-a10.sh generated handoff data for the arria10 socdk sdmmc uboot devicetree. Signed-off-by: Dalon Westergreen --- .../dts/socfpga_arria10_socdk_sdmmc_handoff.h | 305 ++ 1 file changed, 305 insertions(+) create mode 100644

[U-Boot] [PATCH 4/8] ARM: socfpga: arria10: Add generic handoff devicetree include

2019-10-04 Thread Dalon Westergreen
From: Dalon Westergreen Generic handoff devicetree include uses a header generated by the qts-filter-a10.sh script in mach-socfpga. The script creates the header based on design specific implementations for clock and pinmux configurations. Signed-off-by: Dalon Westergreen ---

[U-Boot] [PATCH 8/8] ARM: socfpga: Update README.socfpga to add qts-filter-a10

2019-10-04 Thread Dalon Westergreen
From: Dalon Westergreen Update the readme to add a simple description of using the qts-filter-a10.sh script. Signed-off-by: Dalon Westergreen --- doc/README.socfpga | 37 + 1 file changed, 33 insertions(+), 4 deletions(-) diff --git a/doc/README.socfpga

[U-Boot] [PATCH 7/8] ARM: socfpga: arria10: Remove old A10 SoCDK Handoff dtsi

2019-10-04 Thread Dalon Westergreen
From: Dalon Westergreen This file is no longer needed and has been replaced with socfpga_arria10_handoff_u-boot.dtsi and a generated header. Signed-off-by: Dalon Westergreen --- .../socfpga_arria10_socdk_sdmmc_handoff.dtsi | 329 -- 1 file changed, 329 deletions(-) delete

[U-Boot] [PATCH 6/8] ARM: socfpga: arria10: Add u-boot include for A10 SoCDK SDMMC devicetree

2019-10-04 Thread Dalon Westergreen
From: Dalon Westergreen Rather then modifying the devicetree to add u-boot specific requirements, use the -u-boot.dtsi convention to allow binman to merge the devicetree appropriately. Signed-off-by: Dalon Westergreen --- .../socfpga_arria10_socdk_sdmmc-u-boot.dtsi | 34 +++

Re: [U-Boot] [PATCH v3 2/2] efi: device path for nvme

2019-10-04 Thread Heinrich Schuchardt
On 10/3/19 4:24 PM, Patrick Wildt wrote: > This allows our EFI API to create a device path node for NVMe > devices. It adds the necessary device path struct, uses the > nvme namespace accessor to retrieve the id and eui64, and also > provides support for the device path text protocol. > >

Re: [U-Boot] [PATCH 021/126] sandbox: net: Suppress the MAC-address warnings

2019-10-04 Thread Bin Meng
Hi Simon, On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > These warnings appear every thing sandbox is run (see below) and dwarf the > actual useful output. Suppress them. > > Hopefully at some point a way can be found to fix the root cause. Does removing ethaddr env from

Re: [U-Boot] [PATCH 022/126] sandbox: pci: Move pci_offset_to_barnum() to pci.h

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > This function is useful in PCI emulators. More it into the header file to > avoid duplicating it in other drivers. > > Signed-off-by: Simon Glass > --- > > drivers/misc/swap_case.c | 7 ++- > include/pci.h| 4 > 2

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

2019-10-04 Thread Ley Foon Tan
On Thu, Oct 3, 2019 at 9:00 PM Marek Vasut wrote: > > Migrate CONFIG_DESIGNWARE_WATCHDOG to Kconfig and update the headers > accordingly, no functional change. The S10 enables the WDT only in > SPL, but does not enable it in U-Boot itself, hence disable it in > the config again. > >

Re: [U-Boot] [PATCH 011/126] dm: core: Correct the return value for uclass_find_first_device()

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > This function returns -ENODEV when there is no device. This is > inconsistent with other functions, such as uclass_find_next_device(), > which returns 0. > > Update it and tidy up the incorrect '-1' values in the comments. > > Signed-off-by:

Re: [U-Boot] [PATCH 014/126] fdt: Show the preprocessed .dts file on error

2019-10-04 Thread Bin Meng
Hi Simon, On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > When device-tree compilation fails it is sometimes tricky to see which > line is broken, since the input file to dtc is a pre-processed version > of the device tree. > > Add a line that points to the file that needs to be checked:

Re: [U-Boot] [PATCH 012/126] dm: core: Add device_foreach_child()

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > We have a 'safe' version of this function but sometimes it is not needed. > Add a normal version too and update a few places that can use it. > > Signed-off-by: Simon Glass > --- > > arch/arm/mach-uniphier/pinctrl-glue.c | 4 ++-- >

Re: [U-Boot] [PATCH 009/126] dm: core: Call ofdata_to_platdata() with of-platdata

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > At present this function is never called when of-platdata is enabled since > we never have a device tree. However, this function is responsible for > copying over the of-platdata, so we must call it. Otherwise the probe() > method would have

Re: [U-Boot] [PATCH 010/126] dm: doc: Correct of-platdata CONFIG_IS_ENABLED() condition

2019-10-04 Thread Bin Meng
Hi Simon, On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > This condition is not correct since it should not include an SPL_ prefix. > Fix it and also add a note about the driver name in the same file. > > Signed-off-by: Simon Glass > --- > > doc/driver-model/of-plat.rst | 2 +- > 1

Re: [U-Boot] [PATCH 008/126] dm: test: Don't fail when tests are skipped due to build

2019-10-04 Thread Bin Meng
On Wed, Sep 25, 2019 at 10:58 PM Simon Glass wrote: > > At present tests that are marked as only for livetree fail when executed > on sandbox_flattree. They cannot actually be executed, but we should not > resport them as 'not found', since this causes errors. Instead, they > should be silently

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

2019-10-04 Thread Michal Simek
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. Signed-off-by: Michal Simek --- arch/arm/mach-versal/cpu.c

[U-Boot] [PATCH 4/4] arm64: zynqmp: Convert invoke_smc() to xilinx_pm_request()

2019-10-04 Thread Michal Simek
Remove macros which use PM_SIP_SVC offset and convert invoke_smc() to xilinx_pm_request() which do calculation with PM_SIP_SVC already. Signed-off-by: Michal Simek --- arch/arm/mach-zynqmp/cpu.c | 11 --- board/xilinx/zynqmp/cmds.c | 4 ++--

[U-Boot] [PATCH 0/4] xilinx: firmware: Synchronization firmware functions

2019-10-04 Thread Michal Simek
Hi, Versal and ZynqMP are using the same firmware interface that's why some synchronization should be done. Still there are zynqmp_mmio_read/write which should be converted. Patches are done in steps for easier reviewing. Thanks, Michal Michal Simek (4): arm64: versal: Clean pm_api_id usage

[U-Boot] [PATCH 3/3] gitlab-ci: Switch to Ubuntu 18.04 image

2019-10-04 Thread Tom Rini
In order to run all filesystem tests we need to have newer ext4 tools, move up to Ubuntu 18.04 'bionic' for our base. We need to change slightly how we invoke the provided grub-mkimage. This will also make future python3 work easier. Signed-off-by: Tom Rini --- .gitlab-ci.yml | 7 --- 1

[U-Boot] [PATCH 2/3] gitlab-ci: Have buildman use /tmp for output

2019-10-04 Thread Tom Rini
When running as another user we might not be able to use '..' for certain directories and this is the default for buildman. Specify an output directory instead. Signed-off-by: Tom Rini --- .gitlab-ci.yml | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff

Re: [U-Boot] [PATCH 2/3] imx: add the i.MX8M reset controller node

2019-10-04 Thread Fabio Estevam
Hi Patrick, On Fri, Oct 4, 2019 at 12:50 PM Patrick Wildt wrote: > yes, that's true. It does make sense to sync the device trees and > do the name change. Can the diff with the reset controller driver > (patch 1/3) be merged anyway, and I'll follow up with the DTS change? > Would be easier

[U-Boot] [gitlab-ci-runner][PATCH 2/2] Dockerfile: Update to current bionic tag

2019-10-04 Thread Tom Rini
As part of this we stop installing python-pytest (we enforce a specific version via pip in tests) as well as python-coverage (this can and should be done via pip as well). Signed-off-by: Tom Rini --- Dockerfile | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git

[U-Boot] [gitlab-ci-runner][PATCH 1/2] Dockerfile: Switch to non-root user

2019-10-04 Thread Tom Rini
Add a 'uboot' user / group, allow them sudo access and make use of them in the container. Signed-off-by: Tom Rini --- Dockerfile | 5 + 1 file changed, 5 insertions(+) diff --git a/Dockerfile b/Dockerfile index 1d7a3dae39d0..c7b35026a171 100644 --- a/Dockerfile +++ b/Dockerfile @@ -136,6

Re: [U-Boot] [PATCH 2/3] imx: add the i.MX8M reset controller node

2019-10-04 Thread Patrick Wildt
On Fri, Oct 04, 2019 at 09:39:29AM -0300, Fabio Estevam wrote: > Hi Patrick, > > On Thu, Oct 3, 2019 at 11:12 AM Patrick Wildt wrote: > > > > This patch adds the reset controller node to the i.MX8MQ SoC > > device tree. > > > > Signed-off-by: Patrick Wildt > > --- > >

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

2019-10-04 Thread Tom Rini
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 1 file changed, 12 insertions(+), 12 deletions(-)

[U-Boot] [PATCH 1/4] arm64: versal: Clean pm_api_id usage

2019-10-04 Thread Michal Simek
Copy enum values from platform code to firmware code. IDs are shared between ZynqMP and Versal. Signed-off-by: Michal Simek --- arch/arm/mach-versal/cpu.c| 1 + arch/arm/mach-versal/include/mach/sys_proto.h | 55 --- drivers/clk/clk_versal.c

[U-Boot] [PATCH 3/4] arm64: versal: Rename versal_pm_request to xilinx_pm_request

2019-10-04 Thread Michal Simek
Use generic name instead of Versal specific because this should be also used on ZynqMP. Signed-off-by: Michal Simek --- drivers/clk/clk_versal.c | 14 +++--- drivers/firmware/firmware-zynqmp.c | 2 +- drivers/fpga/versalpl.c| 2 +- include/zynqmp_firmware.h

Re: [U-Boot] [PATCH 2/3] imx: add the i.MX8M reset controller node

2019-10-04 Thread Fabio Estevam
Hi Patrick, On Thu, Oct 3, 2019 at 11:12 AM Patrick Wildt wrote: > > This patch adds the reset controller node to the i.MX8MQ SoC > device tree. > > Signed-off-by: Patrick Wildt > --- > arch/arm/dts/fsl-imx8mq.dtsi | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git

Re: [U-Boot] [RESEND PATCH v3 1/2] Makefile: Add target to generate hex output for combined spl and dtb

2019-10-04 Thread Tom Rini
On Fri, Sep 27, 2019 at 11:27:56AM -0700, Dalon Westergreen wrote: > From: Dalon Westergreen > > Stratix10 requires a hex image of the spl plus spl devicetree offset to > the Stratix10 onchip memory located at SPL_TEXT_BASE. This patch adds > a target to generate a hex file from the u-boot-spl

Re: [U-Boot] [PATCH 1/3] imx: add support for i.MX7/i.MX8MQ reset controller

2019-10-04 Thread Fabio Estevam
Hi Patrick, On Thu, Oct 3, 2019 at 11:08 AM Patrick Wildt wrote: > > Add support for the reset controller that's used on the i.MX7D > and i.MX8MQ. This will be needed to be able to assert the PCIe > reset pins. Bindings taken from Linux, driver implementation > mostly taken from Linux and

[U-Boot] [PATCH] arm64: versal: Enable clock driver

2019-10-04 Thread Michal Simek
Enable clock driver for Versal. Signed-off-by: Michal Simek --- configs/xilinx_versal_virt_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/xilinx_versal_virt_defconfig b/configs/xilinx_versal_virt_defconfig index 75f8b983080d..f924d3e54221 100644 ---

Re: [U-Boot] [PATCH v3 1/4] fs: export fs_close()

2019-10-04 Thread Heinrich Schuchardt
On 10/4/19 5:05 AM, AKASHI Takahiro wrote: > This function is always paired with either fs_set_blk_desc() or > fs_set_blk_desc_with_part(). So just export it. > > Signed-off-by: AKASHI Takahiro > --- > fs/fs.c | 2 +- > include/fs.h | 7 +++ > 2 files changed, 8 insertions(+), 1

Re: [U-Boot] [PATCH 2/3] gitlab-ci: Have buildman use /tmp for output

2019-10-04 Thread Stephen Warren
On 10/4/19 10:12 AM, Tom Rini wrote: When running as another user we might not be able to use '..' for certain directories and this is the default for buildman. Specify an output directory instead. What if multiple invocations run in parallel on the same machine? They'll use the same path

Re: [U-Boot] [PATCH 2/3] gitlab-ci: Have buildman use /tmp for output

2019-10-04 Thread Tom Rini
On Fri, Oct 04, 2019 at 10:34:05AM -0600, Stephen Warren wrote: > On 10/4/19 10:12 AM, Tom Rini wrote: > > When running as another user we might not be able to use '..' for > > certain directories and this is the default for buildman. Specify an > > output directory instead. > > What if multiple

Re: [U-Boot] [PATCH v3 2/4] fs: add fs_get_type() for current filesystem type

2019-10-04 Thread Heinrich Schuchardt
On 10/4/19 5:05 AM, AKASHI Takahiro wrote: > This function is a variant of fs_get_type_name() and returns a filesystem > type with which the current device is associated. > We don't want to export fs_type variable directly because we have to take > care of it consistently within fs.c. > >

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

2019-10-04 Thread Stephen Warren
On 10/4/19 10:12 AM, Tom Rini wrote: We may not always be able to write to the default output directory but we will always have a usable /tmp. Is that a valid assumption? Surely $TMPDIR should be used if set in the environment, falling back to a hard-coded /tmp if that isn't set?

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

2019-10-04 Thread Tom Rini
On Fri, Oct 04, 2019 at 10:32:42AM -0600, Stephen Warren wrote: > On 10/4/19 10:12 AM, Tom Rini wrote: > > We may not always be able to write to the default output directory but > > we will always have a usable /tmp. > > Is that a valid assumption? Surely $TMPDIR should be used if set in the >

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

2019-10-04 Thread Stephen Warren
On 10/4/19 11:26 AM, Tom Rini wrote: On Fri, Oct 04, 2019 at 10:32:42AM -0600, Stephen Warren wrote: On 10/4/19 10:12 AM, Tom Rini wrote: We may not always be able to write to the default output directory but we will always have a usable /tmp. Is that a valid assumption? Surely $TMPDIR

Re: [U-Boot] [PATCH v3 4/4] efi_loader: disk: install file system protocol to a whole disk

2019-10-04 Thread Heinrich Schuchardt
On 10/4/19 5:05 AM, AKASHI Takahiro wrote: > Currently, a whole disk without any partitions is not associated > with EFI_SIMPLE_FILE_SYSTEM_PROTOCOL. So even if it houses some > file system, there is a chance that we may not be able to access > it, particularly, when accesses are to be attempted

[U-Boot] [PATCH 1/1] Revert "env: solve compilation error in SPL"

2019-10-04 Thread Tom Rini
This reverts commit 7d4776545b0f8a8827e5d061206faf61c9ba6ea9. The changes here break environment validation and furthermore do not seem to be required. Signed-off-by: Tom Rini --- Here is what, sadly, is going to I feel be the most frustraiting part of the regression here. The problem that I'm

Re: [U-Boot] [PATCH v3 3/4] efi_loader: disk: install FILE_SYSTEM_PROTOCOL only if available

2019-10-04 Thread Heinrich Schuchardt
On 10/4/19 5:05 AM, AKASHI Takahiro wrote: > In the current implementation, EFI_SIMPLEFILE_SYSTEM_PROTOCOL is always > installed to all the partitions even if some of them may house no file > system. > > With this patch, that protocol will be installed only if any file system > exists. I

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

2019-10-04 Thread Tom Rini
On Fri, Oct 04, 2019 at 12:40:02PM -0600, Stephen Warren wrote: > On 10/4/19 11:26 AM, Tom Rini wrote: > > On Fri, Oct 04, 2019 at 10:32:42AM -0600, Stephen Warren wrote: > > > On 10/4/19 10:12 AM, Tom Rini wrote: > > > > We may not always be able to write to the default output directory but > > >