[U-Boot] [PATCH 1/1] efi_loader: correct signature of GetPosition, SetPosition

2018-10-06 Thread Heinrich Schuchardt
The UEFI spec requires that file positions are passed as u64 in GetPosition() and SetPosition(). Check if the file handle points to a directory in GetPosition(). Provide a unit test for GetPosition() and SetPosition(). Fix Coverity warning CID 184079 (CONSTANT_EXPRESSION_RESULT). Add comments.

Re: [U-Boot] [PATCH 1/3] spi: davinci: Add platdata support

2018-10-06 Thread Adam Ford
On Mon, Oct 1, 2018 at 4:57 PM Adam Ford wrote: > > On Mon, Oct 1, 2018 at 7:54 AM Adam Ford wrote: > > > > On Mon, Sep 3, 2018 at 5:22 PM Adam Ford wrote: > > > > > > On Mon, Sep 3, 2018 at 5:21 PM Adam Ford wrote: > > > > > > > > On Mon, Sep 3, 2018 at 12:30 PM Jagan Teki > > > > wrote: >

Re: [U-Boot] [PATCH 1/4] Enable CONFIG_TIMER_EARLY with bootstage

2018-10-06 Thread Bin Meng
On Tue, Oct 2, 2018 at 9:25 PM Bin Meng wrote: > > Hi Simon, > > On Thu, Sep 27, 2018 at 9:42 PM Simon Glass wrote: > > > > Hi Bin, > > > > On 25 September 2018 at 23:39, Bin Meng wrote: > > > Hi Simon, > > > > > > On Wed, Sep 26, 2018 at 1:42 PM Simon Glass wrote: > > >> > > >> Hi Bin, > > >>

Re: [U-Boot] configs: move CONFIG_SPL_TEXT_BASE to Kconfig

2018-10-06 Thread Tom Rini
On Sun, Sep 30, 2018 at 02:31:53PM +0200, Simon Goldschmidt wrote: > Moved CONFIG_SPL_TEXT_BASE to common/spl/Kconfig with > help from moveconfig.py (only had to prepare socfpga, > stm32f746 and am33x/43x manually) > > Signed-off-by: Simon Goldschmidt > --- > > This patch is in preparation for

Re: [U-Boot] spl: spi: Do not hardcode fixed size for header

2018-10-06 Thread Tom Rini
On Thu, Oct 04, 2018 at 09:30:20AM +0200, Michal Simek wrote: > Find out size directly from header structure. > > Signed-off-by: Michal Simek Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature ___ U-Boot mailing

Re: [U-Boot] spl: ram: Fix u_boot_pos calculation

2018-10-06 Thread Tom Rini
On Thu, Oct 04, 2018 at 09:29:20AM +0200, Michal Simek wrote: > The patch: > "spl: Weed out CONFIG_SYS_TEXT_BASE usage" > (sha1: 04ce5427bd3914cab8be78513275a20ab878520a) > introduced bug where assigning to u_boot_pos variable wasn't done > which end up in situation where SPL wasn't able to find

Re: [U-Boot] [U-Boot, 1/5] ARM: DTS: LogicPD-SOM-LV & Torpedo: Resync DTS with Kernel

2018-10-06 Thread Tom Rini
On Sat, Sep 29, 2018 at 02:10:15PM -0500, Adam Ford wrote: > The device tree entries are from linux-omap's for-next branch > destined to me put into 4.20 (or whatever the version is after 4.19) > > Signed-off-by: Adam Ford > > diff --git a/arch/arm/dts/logicpd-som-lv-37xx-devkit.dts >

Re: [U-Boot] [U-Boot, 5/5] ARM: LogicPD: omap3logic: Remove Torpedo/SOM-LV autodection

2018-10-06 Thread Tom Rini
On Sat, Sep 29, 2018 at 02:10:19PM -0500, Adam Ford wrote: > With there now being four device tree files, and 4 separate > defconfig files, the code necessary to determine which board is > being used is no longer necessary as the corresponding pin-muxing > and board names are determined by the

Re: [U-Boot] [U-Boot, v3] misc: Add support for the Arm Versatile Express config bus

2018-10-06 Thread Tom Rini
On Fri, Sep 28, 2018 at 01:43:31PM +0100, Liviu Dudau wrote: > Add support for the Arm Versatile Express config bus that is > being used for exposing various subsystems via a generic > configuration bus. This driver adds support for generating > transactions on this configuration bus and can be

Re: [U-Boot] [U-Boot, 2/5] ARM: DTS: Remove unnecessary u-boot.dtsi options from omap3/36xx

2018-10-06 Thread Tom Rini
On Sat, Sep 29, 2018 at 02:10:16PM -0500, Adam Ford wrote: > With the introduction of the omap serial driver, the need for some > of these U-Boot specific modifications is gone. This cleans up > this unnneeded stuff. > > Signed-off-by: Adam Ford > > diff --git

Re: [U-Boot] [U-Boot, 4/5] ARM: DTS: Add Logic PD OMAP35/DM37 SOM-LV and OMAP35 Torpedo

2018-10-06 Thread Tom Rini
On Sat, Sep 29, 2018 at 02:10:18PM -0500, Adam Ford wrote: > With the device trees doing most of the work of pin-muxing and > DM doing much of the peripheral initialization, this creates > new defconfig files for each of the Logic PD variants with > proper register settings/pin-muxing. > >

Re: [U-Boot] [U-Boot, 3/5] ARM: DTS: Add support for Logic PD OMAP35 Torpedo & SOM-LV

2018-10-06 Thread Tom Rini
On Sat, Sep 29, 2018 at 02:10:17PM -0500, Adam Ford wrote: > The baseboards and SOM's are virtually identical to their DM37 > counterparts, but OMAP36/37 and OMAP3 have some minor register > differences. With the boards being mostly driven by device trees > now, this synchronizes their

Re: [U-Boot] [U-Boot, 2/2] test/py: test_fs: add docstring comments to helper functions

2018-10-06 Thread Tom Rini
On Thu, Sep 27, 2018 at 04:07:23PM +0900, Akashi, Takahiro wrote: > From: "Akashi Takahiro" > > After Siomon's comment, add a descriptive comment (docstring) to each of > helper functions in conftest.py. No functionality changed. > > Signed-off-by: Akashi Takahiro > Reviewed-by: Simon Glass

Re: [U-Boot] [U-Boot,1/1] fs: fat: fix set_cluster()

2018-10-06 Thread Tom Rini
On Tue, Oct 02, 2018 at 09:30:45AM +0200, Heinrich Schuchardt wrote: > Avoid CoverityScan warning SIGN_EXTENSION CID 184096 by changing the > type of parameter size of set_cluster(). > > Avoid leaking stack content when writing an incomplete last sector. > > Reported-by: Tom Rini >

Re: [U-Boot] [U-Boot,1/1] fs: fat: memory leak in fat_unlink()

2018-10-06 Thread Tom Rini
On Tue, Oct 02, 2018 at 06:58:00AM +0200, Heinrich Schuchardt wrote: > Do not leak filename_copy in case of error. > Catch out of memory when calling strdup. > > This addresses Coverity Scan CID 184086. > > Reported-by: Tom Rini > Signed-off-by: Heinrich Schuchardt Applied to u-boot/master,

Re: [U-Boot] mach-stm32: Set MPU SDRAM size to 512MB for STM32F7/H7

2018-10-06 Thread Tom Rini
On Tue, Oct 02, 2018 at 09:03:10AM +0200, Patrice Chotard wrote: > This allows to boot all STM32F7 and STM32H7 boards independently > of the amount of embedded SDRAM. > > Signed-off-by: Patrice Chotard > Reviewed-by: Vikas Manocha Applied to u-boot/master, thanks! -- Tom signature.asc

Re: [U-Boot] [U-Boot, 1/2] test/py: test_fs: remove fs_type argument from umount_fs()

2018-10-06 Thread Tom Rini
On Thu, Sep 27, 2018 at 04:07:22PM +0900, Akashi, Takahiro wrote: > From: "Akashi Takahiro" > > Since there is no use of fs_type in umount_fs(), just remove it. > > Signed-off-by: Akashi Takahiro > Reviewed-by: Simon Glass Applied to u-boot/master, thanks! -- Tom signature.asc

Re: [U-Boot] configs: drop CONFIG_SYS_EXTRA_ENV_RELOC

2018-10-06 Thread Tom Rini
On Thu, Sep 27, 2018 at 12:38:34PM +0200, Simon Goldschmidt wrote: > Instead of manually specifying CONFIG_SYS_EXTRA_ENV_RELOC > for every board that needs it, it shouldn't hurt to let > initr_reloc_global_data() always relocate gd->env_addr > unless we know this pointer is outside the initial

Re: [U-Boot] [U-Boot, V2] test/py: ignore console read exceptions after test failure

2018-10-06 Thread Tom Rini
On Thu, Sep 20, 2018 at 04:55:03PM -0600, Stephen Warren wrote: > From: Stephen Warren > > After a test has failed, test/py drains the U-Boot console log to ensure > that any relevant output is captured. At this point, we don't care about > detecting any additional errors, since the test is

Re: [U-Boot] [PATCH v2 3/9] dm: uclass: Add uclass_foreach_dev_probe

2018-10-06 Thread Simon Glass
On 1 October 2018 at 05:59, Patrice Chotard wrote: > > Add uclass_foreach_dev_probe() which iterates through > devices of a given uclass. Devices are probed if necessary > and are ready to use. > > Signed-off-by: Patrice Chotard > --- > > Changes in v2: None > > include/dm/uclass.h | 16

Re: [U-Boot] [PATCH] arm: socfpga: fix SPL booting from fpga OnChip RAM

2018-10-06 Thread Tom Rini
On Sat, Oct 06, 2018 at 10:33:42PM +0200, Simon Goldschmidt wrote: > This patch prevents disabling the FPGA bridges when > SPL or U-Boot is executed from FPGA onchip RAM. > > Signed-off-by: Simon Goldschmidt > --- > > > There are other patches required to make boot from > FPGA work correctly:

[U-Boot] [PATCH] arm: socfpga: fix SPL booting from fpga OnChip RAM

2018-10-06 Thread Simon Goldschmidt
This patch prevents disabling the FPGA bridges when SPL or U-Boot is executed from FPGA onchip RAM. Signed-off-by: Simon Goldschmidt --- There are other patches required to make boot from FPGA work correctly: - CONFIG_SPL_TEXT_BASE needs to be set to 0xC000 ideally this is done by moving

Re: [U-Boot] [PATCH] spl: ram: Fix u_boot_pos calculation

2018-10-06 Thread Simon Goldschmidt
On 06.10.2018 17:29, Simon Goldschmidt wrote: On 04.10.2018 09:29, Michal Simek wrote: The patch: "spl: Weed out CONFIG_SYS_TEXT_BASE usage" (sha1: 04ce5427bd3914cab8be78513275a20ab878520a) introduced bug where assigning to u_boot_pos variable wasn't done which end up in situation where SPL

[U-Boot] [PATCH] fs: fat: fix reading non-cluster-aligned root directory

2018-10-06 Thread Anssi Hannula
FAT16 root directory might not start at a cluster boundary (though it always ends on one). However, next_dent() always starts reading the directory at the beginning of the cluster, causing no files to be found (at least if the beginning of the cluster contains zeroes). Fix that by skipping to

Re: [U-Boot] ARM: da850evm: Remove DM_I2C_COMPAT

2018-10-06 Thread Tom Rini
On Sun, Sep 30, 2018 at 09:20:13AM -0500, Adam Ford wrote: > The da850evm does not need this enabled, so this removes a > notice that appears during compile time that says > "Please remove" > > Signed-off-by: Adam Ford NAK, this breaks da850_am18xxevm which, ahem, you're listed as the

Re: [U-Boot] 4 failures with 'ut dm'

2018-10-06 Thread Bin Meng
On Sat, Oct 6, 2018 at 9:14 PM Bin Meng wrote: > > Hi Simon, > > With current u-boot/master, there are 4 failures with 'ut dm'. I was > wondering is this not covered by travis-ci? > > Test: dm_test_bus_children: bus.c > test/dm/bus.c:112, dm_test_bus_children(): num_devices == >

[U-Boot] [PATCH 4/8] dm: core: Respect drivers with the DM_FLAG_PRE_RELOC flag in lists_bind_fdt()

2018-10-06 Thread Bin Meng
Currently the comments of several APIs (eg: dm_init_and_scan()) say: @pre_reloc_only: If true, bind only drivers with the DM_FLAG_PRE_RELOC flag. If false bind all drivers. The 'Pre-Relocation Support' chapter in doc/driver-model/README.txt documents the same. However the implementation only

[U-Boot] [PATCH 8/8] timer: Sort Kconfig driver entries

2018-10-06 Thread Bin Meng
This is currently out of order. Sort it. Signed-off-by: Bin Meng --- drivers/timer/Kconfig | 110 +- 1 file changed, 55 insertions(+), 55 deletions(-) diff --git a/drivers/timer/Kconfig b/drivers/timer/Kconfig index a7d600b..8c0f356 100644 ---

[U-Boot] [PATCH 7/8] test: dm: core: Add a test case for driver marked with DM_FLAG_PRE_RELOC flag

2018-10-06 Thread Bin Meng
Now that we fixed the pre-relocation driver binding for driver marked with DM_FLAG_PRE_RELOC flag, add a test case to cover that scenario. Signed-off-by: Bin Meng --- arch/sandbox/dts/test.dts | 4 test/dm/bus.c | 2 +- test/dm/test-fdt.c| 29

[U-Boot] [PATCH 2/8] cpu: mpc83xx: Remove unnecessary characters in the description string

2018-10-06 Thread Bin Meng
The description string should not contain unnecessary characters, like the ending '\n' or the leading 'CPU:'. Signed-off-by: Bin Meng --- drivers/cpu/mpc83xx_cpu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/cpu/mpc83xx_cpu.c b/drivers/cpu/mpc83xx_cpu.c index

[U-Boot] [PATCH 5/8] dm: Correct pre_reloc_only parameter description in several APIs' comments

2018-10-06 Thread Bin Meng
The pre_reloc_only parameter description currently only mentions drivers with the DM_FLAG_PRE_RELOC flag, but does not mention the special device tree properties. Correct them. Signed-off-by: Bin Meng --- include/dm/device-internal.h | 4 ++-- include/dm/lists.h | 4 ++--

[U-Boot] [PATCH 6/8] dm: core: Mirror the chosen node parse logic in the livetree scanning

2018-10-06 Thread Bin Meng
Commit f2006808f099: ("dm: core: parse chosen node") added a logic to parse the chosen node during dm_scan_fdt_node(), but unfortunately it missed adding the same logic in dm_scan_fdt_live(). This mirrors the logic in the livetree version. The weird thing is that commit f2006808f099 did update

[U-Boot] [PATCH 3/8] dm: util: Add a livetree equivalent API of dm_fdt_pre_reloc()

2018-10-06 Thread Bin Meng
This adds a new API dm_ofnode_pre_reloc(), a livetree equivalent API of dm_fdt_pre_reloc(). Signed-off-by: Bin Meng --- drivers/core/util.c | 25 + include/dm/util.h | 27 ++- 2 files changed, 51 insertions(+), 1 deletion(-) diff --git

[U-Boot] [PATCH 1/8] dm: cpu: Fix print_cpuinfo() output

2018-10-06 Thread Bin Meng
Current output of print_cpuinfo() does not have an ending '\n', neither have a leading 'CPU:' like others. However previously this used to work on QEMU x86 targets. It turns out commit c0434407b595 introduced a unified DM version of print_cpuinfo() that exposed such issue. Fixes: c0434407b595

[U-Boot] [PATCH 0/8] dm: Various fixes in dm core/cpu/timer

2018-10-06 Thread Bin Meng
This fixed several issues identified in dm core/cpu/timer codes. The issues were found during RISC-V cpu/timer driver development for QEMU RISC-V port. This series is available at u-boot-x86/dm-fixes for testing. Bin Meng (8): dm: cpu: Fix print_cpuinfo() output cpu: mpc83xx: Remove

Re: [U-Boot] [PATCH] spl: ram: Fix u_boot_pos calculation

2018-10-06 Thread Simon Goldschmidt
On 04.10.2018 09:29, Michal Simek wrote: The patch: "spl: Weed out CONFIG_SYS_TEXT_BASE usage" (sha1: 04ce5427bd3914cab8be78513275a20ab878520a) introduced bug where assigning to u_boot_pos variable wasn't done which end up in situation where SPL wasn't able to find out image on the selected

[U-Boot] [PATCH] sunxi: fix DRAM gate/reset sequence of H6

2018-10-06 Thread Icenowy Zheng
Currently the DRAM bus gate and reset is changed at the same time in H6 DRAM initialization code, which disobeys the user manual's programming guide. Fix the sequence by follow the sequence suggested by the user manual (ungate the bus clock after release the reset signal). By some experiments it

Re: [U-Boot] [PATCH] Revert "fdt: fdtdec_setup_memory_banksize() use livetree"

2018-10-06 Thread Patrice CHOTARD
Hi I got same issue on STM32F746 Patrice On 10/04/2018 09:24 AM, Michal Simek wrote: > This reverts commit c35a7d375ec8f0a8ee343ae4868be3242172632e. > This commit is breaking SPL on zc706. > > Signed-off-by: Michal Simek > --- > > lib/fdtdec.c | 44

[U-Boot] 4 failures with 'ut dm'

2018-10-06 Thread Bin Meng
Hi Simon, With current u-boot/master, there are 4 failures with 'ut dm'. I was wondering is this not covered by travis-ci? Test: dm_test_bus_children: bus.c test/dm/bus.c:112, dm_test_bus_children(): num_devices == list_count_items(>dev_head): Expected 7, got 6 Test: dm_test_fdt: test-fdt.c

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

2018-10-06 Thread Tom Rini
On Fri, Oct 05, 2018 at 11:26:55PM +0200, Marek Vasut wrote: > The following changes since commit 592cd5defd4f71d34ffcbd8dd3326bc10f662e20: > > Merge branch 'master' of git://git.denx.de/u-boot-spi (2018-10-02 > 17:01:46 -0400) > > are available in the Git repository at: > >

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

2018-10-06 Thread Tom Rini
On Fri, Oct 05, 2018 at 11:27:44PM +0200, Marek Vasut wrote: > The following changes since commit 592cd5defd4f71d34ffcbd8dd3326bc10f662e20: > > Merge branch 'master' of git://git.denx.de/u-boot-spi (2018-10-02 > 17:01:46 -0400) > > are available in the Git repository at: > >

[U-Boot] [PATCH 3/4] sunxi: display: Also use the PWM controller for backlight on sun7i

2018-10-06 Thread Paul Kocialkowski
Using PWM to drive the backlight pin instead of a GPIO provides various advantages, that are described in commit 421c98d7d2 ("sunxi: display: Use PWM to drive backlight where applicable"). Defining SUNXI_PWM_PIN0 triggers the configuration of the PWM controller in the display driver.

[U-Boot] [PATCH 2/4] sunxi: display: Add support for panels with an active-low DE signal

2018-10-06 Thread Paul Kocialkowski
This introduces a Kconfig option for indicating that the panel requires the data enable (DE) signal driven low. The associated display flag is set when converting the mode to display timings. It is later used for setting the relevant bit in the TCON configuration registers. This new option is

[U-Boot] [PATCH 4/4] sunxi: Add defconfig for the BananaPi with BL035 LCD module

2018-10-06 Thread Paul Kocialkowski
This introduces a new defconfig for the BananaPi, which includes support for the BL035 LCD module distributed by Lemaker for use with this board. Signed-off-by: Paul Kocialkowski --- configs/Bananapi_LCD_BL035_defconfig | 29 1 file changed, 29 insertions(+) create

[U-Boot] [PATCH 1/4] sunxi: display: Fix h/vsync TCON polarity bits to indicate active-high

2018-10-06 Thread Paul Kocialkowski
Contrary to what is implemented in the Linux kernel, the U-Boot driver supporting the sunxi TCON was assuming that the TCON0_IO_POL bits indicate active-low signals when set for the hsync and vsync signals. After looking at it with a logic analyzer, it appears that the signals are active-low with

Re: [U-Boot] [PATCH 1/1] efi_loader: set image_base and image_size to correct values

2018-10-06 Thread Heinrich Schuchardt
On 10/06/2018 09:51 AM, Heinrich Schuchardt wrote: > On 09/30/2018 07:26 AM, Heinrich Schuchardt wrote: >> From: AKASHI Takahiro >> >> Currently, image's image_base points to an address where the image was >> temporarily uploaded for further loading. Since efi_loader relocates >> the image to

Re: [U-Boot] [PATCH 1/1] efi_loader: set image_base and image_size to correct values

2018-10-06 Thread Heinrich Schuchardt
On 09/30/2018 07:26 AM, Heinrich Schuchardt wrote: > From: AKASHI Takahiro > > Currently, image's image_base points to an address where the image was > temporarily uploaded for further loading. Since efi_loader relocates > the image to final destination, image_base and image_size should reflect