[PATCH 1/1] board: fsl: lx2160a: unused variable gic_lpi_base

2020-04-11 Thread Heinrich Schuchardt
If the board is configured without CONFIG_GIC_V3_ITS, an error occurs: board/freescale/lx2160a/lx2160a.c: In function ‘ft_board_setup’: board/freescale/lx2160a/lx2160a.c:673:6: error: unused variable ‘gic_lpi_base’ [-Werror=unused-variable] 673 | u64 gic_lpi_base; | ^~~~ Let

[PATCH v3] mmc: zynq: parse dt when probing

2020-04-11 Thread Benedikt Grassl
Currently, the entry "bus-width = <8>" in the ZynqMP's sdhci nodes is not evaluated. This results in the bus width staying at its default value (4 bit in HS200 mode). Fix this by calling mmc_of_parse. This function also checks for the "no-1-8-v" and "max-frequency" entries. Remove the handling of t

Re: [PATCH v2 0/6] rockchip: rk3328: sync dts and add ROC-RK3328-CC board

2020-04-11 Thread Kurt Miller
On Sun, 2020-04-05 at 10:25 +0800, Chen-Yu Tsai wrote: > From: Chen-Yu Tsai > > Hi everyone, > > This is v2 of my ROC-RK3328-CC series. Changes from v1 are mainly > dropping the custom board target, and dealing with the pinmuxing > through proper use of DM regulators / GPIO / pinctrl in SPL. >

Fix for broken gigabit ethernet in Olimex A20-OLinuXino-LIME2

2020-04-11 Thread Dario
Hello, On 2017/02/07, commit af2cbfd6 broke gigabit ethernet on Olimex A20-OLinuXino-LIME2. This commit likely prevents the activation of the option CONFIG_RTL8211X_PHY_FORCE_MASTER needed for proper gigabit networking. Also in a later revision this option was removed altogether from both A20-

Re: [PATCH v3 03/23] buildman: Document the members of BuilderJob

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:41AM -0600, Simon Glass wrote: > This class has a few more members now. Add documentation for them and fix > a nit in the 'commits' comment. > > Signed-off-by: Simon Glass Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v3 02/23] main: Drop show_boot_progress() prototype

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:40AM -0600, Simon Glass wrote: > This is defined in bootstage.h and is not called in this file anyway. Drop > it. > > Signed-off-by: Simon Glass > Reviewed-by: Tom Rini Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v3 04/23] bulidman: Add support for a simple build

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:42AM -0600, Simon Glass wrote: > It is useful to run a simple build and put all the output in a single > directory. Add a -w option to support this. > > Signed-off-by: Simon Glass Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v3 01/23] sandbox: Add documentation about required/useful packages

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:39AM -0600, Simon Glass wrote: > Quite a few packages are used by sandbox or tools. Add a list of these to > help people setting up for the first time. > > Signed-off-by: Simon Glass Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signat

Re: [PATCH v3 05/23] buildman: Update help for -d

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:43AM -0600, Simon Glass wrote: > This help is a bit ambiguous. It only does anything if asked to show size > changes with -S. Update the help and the function comments. > > Signed-off-by: Simon Glass Applied to u-boot/master, thanks! -- Tom signature.asc Descri

Re: [PATCH v3 06/23] buildman: Allow ignoring warnings in the return code

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:44AM -0600, Simon Glass wrote: > Sometimes we don't want buildman to return failure if it seems warnings. > Add a -W option to support this. If buildman detects warnings (and no > errors) it will return an exit code of 0 (success). > > Note that the definition of 'wa

Re: [PATCH v3 09/23] buildman: Drop the -a option

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:47AM -0600, Simon Glass wrote: > There is no point in setting the ARCH environment variable since the > U-Boot build system no-longer uses it. > > It seems safe to drop this feature since it was only recently added. > > Signed-off-by: Simon Glass > Reviewed-by: Tom

Re: [PATCH v3 10/23] travis: Don't copy files into .bm-work/

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:48AM -0600, Simon Glass wrote: > At present if TEST_PY_BD is empty the script copies various files into a > directory, to no purpose. This happens because UBOOT_TRAVIS_BUILD_DIR is > set before TEST_PY_BD is tested. > > Move the 'if' to fix this. > > Signed-off-by:

Re: [PATCH v3 07/23] buildman: Be more selective about which directories to remove

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:45AM -0600, Simon Glass wrote: > At present buildman removes any directory it doesn't intend to write > output into. This is overly expansive since if the output directory > happens to be somewhere with existing files, they may be removed. Using > an existing director

Re: [PATCH v3 08/23] buildman: Allow building within a subdir of the current dir

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:46AM -0600, Simon Glass wrote: > This is useful in some situations, in particular with -w and when building > in-tree. Now that we are more careful about what we remove in > _PrepareOutputSpace(), it should be safe to relax this restriction. > > Update the progress i

Re: [PATCH v3 12/23] gitlab/azure: Use the -w option for sandbox_spl

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:50AM -0600, Simon Glass wrote: > Avoid needing to know about the internal .bm-work directory, by passing > the -w flag to buildman. > > This does not affect travis since the previous commit already used the -w > flag. > > Signed-off-by: Simon Glass > Reviewed-by: T

Re: [PATCH v3 11/23] travis: Split the building into two parts

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:49AM -0600, Simon Glass wrote: > Buildman is used in two ways: > > - to build a selection of boards (with no testing) > - to build a single board (and run pytest) > > The gitlab and azure scrips do this in separate places, but travis does > not. To aid the refactori

Re: [PATCH v3 14/23] travis/gitlab/azure: Drop BUILDMAN variable with test.py

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:52AM -0600, Simon Glass wrote: > This is not needed in the test.py part of the config, now since we use the > same name as the pytests. > > Drop BUILDMAN, retaining it only for the 'build' parts of the config, i.e. > where we build multiple boards and don't run any t

Re: [PATCH v3 15/23] travis/gitlab/azure: Drop the buildman -d flag

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:53AM -0600, Simon Glass wrote: > This has no effect since -S is not given also. Drop it. > > Signed-off-by: Simon Glass > Reviewed-by: Tom Rini Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v3 13/23] travis/gitlab/azure: Use --board buildman flag with test.py

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:51AM -0600, Simon Glass wrote: > The current method of selecting the board to build with test.py is a bit > error-prone, e.g. with "^sandbox$" it actually builds 5 boards (all of > those in the sandbox architecture). > > Use the (newish) --board flag instead, to get

Re: [PATCH v3 18/23] travis/gitlab/azure: Use bash to avoid a_test_which_does_not_exist

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:56AM -0600, Simon Glass wrote: > Bash allows for variables to expand only if non-empty: > > $ var=test > $ echo ${var:+"$var"} > test > $ echo ${var:+"-k $var"} > -k test > $ var= > $ echo ${var:+"-k $var"} > > Use this feat

Re: [PATCH v3 16/23] gitlab/azure: Drop unnecessary if..fi when using test.py

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:54AM -0600, Simon Glass wrote: > Since TEST_PY_BD is always defined we can drop this check. > > This does not affect travis since it has a single, unified script. > > Signed-off-by: Simon Glass > Reviewed-by: Tom Rini Applied to u-boot/master, thanks! -- Tom

Re: [PATCH v3 17/23] gitlab/azure: Use -w flag for all test.py builds

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:55AM -0600, Simon Glass wrote: > Avoid needing to know about the internal .bm-work directory, by passing > the -w flag to buildman. > > This is not needed on travis since the -w flag is already used (from a > previous patch). > > Drop the -P flag since this has no e

Re: [PATCH v3 20/23] travis/gitlab/azure: Enable test_handoff

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:58AM -0600, Simon Glass wrote: > Ensure that this SPL test runs on gitlab. > > Signed-off-by: Simon Glass > Reviewed-by: Tom Rini Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v3 21/23] travis/gitlab/azure: Simplify the exit code for test.py

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:59AM -0600, Simon Glass wrote: > It seems unnecessary to read the exit code and then check it again. Drop > this and just let the test.py provide the exit code directly. > > Signed-off-by: Simon Glass > Reviewed-by: Tom Rini Applied to u-boot/master, thanks! --

Re: [PATCH v3 22/23] travis/gitlab/azure: Drop repeated buildman call with test.py

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:43:00AM -0600, Simon Glass wrote: > It does not seem to be necessary to run buildman again to show errors, > since any errors can be shown by the first invocation and there is only > a single board being built. Update this to simplify the code, using the > -e flag to mak

Re: [PATCH v3 19/23] travis/gitlab/azure: Use -W to avoid warnings check

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:42:57AM -0600, Simon Glass wrote: > We can use the -W flag to tell buildman to ignore warnings. Since we also > have -E defined, compiler warnings are promoted to errors, so they will > still cause a failure. But migration warnings of the form: > > =

Re: [PATCH] travis/gitlab/azure: Run cppcheck in parallel

2020-04-11 Thread Tom Rini
On Sun, Apr 05, 2020 at 02:35:43PM -0600, Simon Glass wrote: > This takes ages to run single-threaded. Adjust it to use all available > processors. > > Signed-off-by: Simon Glass Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PATCH v3 23/23] test/py: Allow using buildman to build U-Boot

2020-04-11 Thread Tom Rini
On Wed, Mar 18, 2020 at 09:43:01AM -0600, Simon Glass wrote: > It is a pain to have to set the CROSS_COMPILE environment variable when > using test.py's --build option. It is possible to get this using the -A > option from buildman. But it seems better to just use buildman to do the > build when i

[PATCH 1/1] tools: image-host.c: use correct output format

2020-04-11 Thread Heinrich Schuchardt
When building on a 32bit host the following warning occurs: tools/image-host.c: In function ‘fit_image_read_data’: tools/image-host.c:310:42: warning: format ‘%ld’ expects argument of type ‘long int’, but argument 3 has type ‘ssize_t’ {aka ‘int’} [-Wformat=] printf("Can't read all file %s (read

Re: [PATCH 2/2] dm: core: Walk the tree to find address- and size-cells properties

2020-04-11 Thread Matthias Brugger
Hi Simon, On 4/9/20 6:25 PM, Simon Glass wrote: > Hi Matthias, > > On Wed, 8 Apr 2020 at 03:35, wrote: >> >> From: Matthias Brugger >> >> Walk the tree when reading size-cells or address-cells properties. >> >> Reported-by: Robin Randhawa >> Signed-off-by: Matthias Brugger >> >> --- >> >> dr

[PATCH 2/5] fdtdec: Add weak function to patch U-Boot DT right after fdtdec_setup()

2020-04-11 Thread Marek Vasut
Add weak function which is called right after fdtdec_setup() configured the U-Boot DT. This permits board-specific adjustments to the U-Boot DT before U-Boot starts parsing the DT. This could be used e.g. to patch in various custom nodes or merge in DT fragments from prior-stage firmware. Signed-o

[PATCH 1/5] libfdt: Export overlay_apply_node() as fdt_overlay_apply_node()

2020-04-11 Thread Marek Vasut
This function is useful to merge a subset of DT into another DT, for example if some prior-stage firmware passes a DT fragment to U-Boot and U-Boot needs to merge it into its own DT. Export this function to permit implementing such functionality. Signed-off-by: Marek Vasut Cc: Nobuhiro Iwamatsu

[PATCH 3/5] ARM: dts: rmobile: Reserve space in R-Car Gen3 DTs

2020-04-11 Thread Marek Vasut
Reserve 4 kiB of space in R-Car Gen3 DTs when those DTs are compiled to permit patching in OpTee-OS /firmware node, /reserved-memory node and possibly also additional /memory@ nodes. Signed-off-by: Marek Vasut Cc: Nobuhiro Iwamatsu Cc: Simon Glass Cc: Tom Rini --- arch/arm/dts/Makefile | 4 ++

[PATCH 5/5] ARM: rmobile: Enable support for OpTee on Gen3

2020-04-11 Thread Marek Vasut
Enable OpTee support on R-Car Gen3, so that U-Boot would copy the OpTee /firmware and /reserved-memory nodes into the Linux DT. Signed-off-by: Marek Vasut Cc: Nobuhiro Iwamatsu Cc: Simon Glass Cc: Tom Rini --- configs/r8a77970_eagle_defconfig | 2 ++ configs/r8a77980_condor_defconfig | 2 +

[PATCH 4/5] ARM: rmobile: Merge prior-stage firmware DT fragment into U-Boot DT on Gen3

2020-04-11 Thread Marek Vasut
The prior-stage firmware generates DT fragment containing the /firmware node, /reserved-memory node and /memory@ nodes. Merge these nodes into the U-Boot DT, so U-Boot can use this information. Signed-off-by: Marek Vasut Cc: Nobuhiro Iwamatsu Cc: Simon Glass Cc: Tom Rini --- board/renesas/rca

Re: [PATCH] usb: max3420: add the gadget driver

2020-04-11 Thread Jassi Brar
On Sun, Mar 29, 2020 at 7:47 PM wrote: > > From: Jassi Brar > > MAX3420 implements FullSpeed USB Device over SPI. > Another version MAX3421, also implements USB Host mode. > This driver should be good for the device mode of max3421 as well. > A polite reminder please Thanks.

Re: [PATCH] usb: max3420: add the gadget driver

2020-04-11 Thread Heinrich Schuchardt
Am April 11, 2020 11:47:06 PM UTC schrieb Jassi Brar : >On Sun, Mar 29, 2020 at 7:47 PM wrote: >> >> From: Jassi Brar >> >> MAX3420 implements FullSpeed USB Device over SPI. >> Another version MAX3421, also implements USB Host mode. >> This driver should be good for the device mode of max3421 as

Re: [PATCH] usb: max3420: add the gadget driver

2020-04-11 Thread Marek Vasut
On 4/12/20 2:04 AM, Heinrich Schuchardt wrote: > Am April 11, 2020 11:47:06 PM UTC schrieb Jassi Brar > : >> On Sun, Mar 29, 2020 at 7:47 PM wrote: >>> >>> From: Jassi Brar >>> >>> MAX3420 implements FullSpeed USB Device over SPI. >>> Another version MAX3421, also implements USB Host mode. >>> T

Re: [PATCH] usb: max3420: add the gadget driver

2020-04-11 Thread Jassi Brar
On Sat, Apr 11, 2020 at 9:31 PM Marek Vasut wrote: > > On 4/12/20 2:04 AM, Heinrich Schuchardt wrote: > > Am April 11, 2020 11:47:06 PM UTC schrieb Jassi Brar > > : > >> On Sun, Mar 29, 2020 at 7:47 PM wrote: > >>> > >>> From: Jassi Brar > >>> > >>> MAX3420 implements FullSpeed USB Device over

RE: [PATCH] Revert "mpc85xx: ddr: Always start DDR RAM in Self Refresh mode"

2020-04-11 Thread Priyanka Jain
>-Original Message- >From: Joakim Tjernlund >Sent: Friday, April 10, 2020 5:21 PM >To: Priyanka Jain ; Biwen Li (OSS) > >Cc: u-boot@lists.denx.de; Biwen Li ; Jiafei Pan > >Subject: Re: [PATCH] Revert "mpc85xx: ddr: Always start DDR RAM in Self >Refresh mode" > >On Fri, 2020-04-10 at 11:40

pull request of u-boot-mpc85xx for v2020.04

2020-04-11 Thread Priyanka Jain
Dear Tom, Please find my pull-request for u-boot-mpc85xx/master https://travis-ci.org/github/p-priyanka-jain/u-boot/builds/673382817 Summary Fixes DDR initialization failure on PowerPC boards like P3041DS, P4080DS priyankajain - The following changes since commit 31232de07ef2bd97