Re: [U-Boot] [PATCH] qemu-arm: Add persistent environment support

On Wed, Dec 12, 2018 at 12:14:27PM +0530, Sumit Garg wrote: > On Wed, 12 Dec 2018 at 07:09, Takahiro Akashi > wrote: > > > > On Tue, Dec 11, 2018 at 06:04:05PM +0530, Sumit Garg wrote: > > > On Mon, 26 Nov 2018 at 16:51, Sumit Garg wrote: > > > > > > > > Currently on qemu-arm platforms

[U-Boot] [PATCH] poplar: fix boot failure caused by serial driver change

Commit 4687919684e0 ("serial: Remove DM_FLAG_PRE_RELOC flag in various drivers") essentially drops flag DM_FLAG_PRE_RELOC from serial_pl01x driver for Poplar platform, because the platform falls into the following strategy category made by the commit. Surround DM_FLAG_PRE_RELOC flag with

[U-Boot] [PATCH v4 18/25] riscv: Do some basic architecture level cpu initialization

In arch_cpu_init_dm() do some basic architecture level cpu initialization, like FPU enable, etc. Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: - only initialize mcounteren CSR for S-mode - only touch satp in M-mode U-Boot - move the implementation to

[U-Boot] [PATCH v4 17/25] riscv: Add indirect stringification to csr_xxx ops

With current csr_xxx ops, we cannot pass a macro to parameter 'csr', hence we need add another level to allow the parameter to be a macro itself, aka indirect stringification. Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: None Changes in v2: - new

[U-Boot] [PATCH v4 16/25] riscv: Update supports_extension() to use desc from cpu driver

This updates supports_extension() implementation to use the desc string from the cpu driver whenever possible, which avoids the reading of misa CSR for S-mode U-Boot. Signed-off-by: Bin Meng --- Changes in v4: - print warnings for S-mode without CONFIG_CPU Changes in v3: - new patch to update

[U-Boot] [PATCH v4 14/25] riscv: Add CSR numbers

The standard RISC-V ISA sets aside a 12-bit encoding space for up to 4096 CSRs. This adds all known CSR numbers as defined in the RISC-V Privileged Architecture Version 1.10. Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: - add sedeleg (0x102) and

[U-Boot] [PATCH v4 15/25] riscv: Add exception codes for xcause register

This adds all exception codes in encoding.h. Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: None Changes in v2: None arch/riscv/include/asm/encoding.h | 15 +++ 1 file changed, 15 insertions(+) diff --git

[U-Boot] [PATCH v4 19/25] riscv: Move trap handler codes to mtrap.S

Currently the M-mode trap handler codes are in start.S. For future extension, move them to a separate file mtrap.S. Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: None Changes in v2: None arch/riscv/cpu/Makefile | 2 +- arch/riscv/cpu/mtrap.S | 111

[U-Boot] [PATCH v4 25/25] riscv: Remove ae350.dts

This is not used by any board. Remove it. Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: - new patch to remove ae350.dts Changes in v2: None arch/riscv/dts/ae350.dts | 229 --- 1 file changed, 229

[U-Boot] [PATCH v4 07/25] riscv: Introduce a Kconfig option for machine mode

From: Anup Patel So far we have a Kconfig option for supervisor mode. This adds an option for the machine mode. Signed-off-by: Anup Patel Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: None Changes in v2: - incorporated and reworked Anup's S-mode

[U-Boot] [PATCH v4 23/25] riscv: Save boot hart id to the global data

At present the hart id passed via a0 in the U-Boot entry is saved to s0 at the beginning but does not preserve later. Save it to the global data structure so that it can be used later. Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: - new patch to save

[U-Boot] [PATCH v4 10/25] riscv: qemu: Add platform-specific Kconfig options

Add the QEMU RISC-V platform-specific Kconfig options, to include CPU and timer drivers. Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: None Changes in v2: - add CMD_CPU as well arch/riscv/Kconfig | 1 + arch/riscv/cpu/qemu/Kconfig

[U-Boot] [PATCH v4 12/25] riscv: Probe cpus during boot

This calls cpu_probe_all() to probe all available cpus. Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: - probe cpus in the pre-relocation stage too Changes in v2: - move to arch/riscv/cpu/cpu.c arch/riscv/cpu/cpu.c| 26

[U-Boot] [PATCH v4 21/25] riscv: Return to previous privilege level after trap handling

At present the trap handler returns to hardcoded M-mode/S-mode. Change to returning to previous privilege level instead. Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: None Changes in v2: None arch/riscv/cpu/mtrap.S | 8 1 file changed, 8

[U-Boot] [PATCH v4 24/25] riscv: bootm: Change to use boot_hart from global data

Avoid reading mhartid CSR directly, instead use the one we saved in the global data structure before. With this patch, BBL no longer needs to be hacked to provide the mhartid CSR emulation for S-mode U-Boot. Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in

[U-Boot] [PATCH v4 20/25] riscv: Fix context restore before returning from trap handler

sp cannot be loaded before restoring other registers. Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: None Changes in v2: None arch/riscv/cpu/mtrap.S | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/riscv/cpu/mtrap.S

[U-Boot] [PATCH v4 22/25] riscv: Adjust the _exit_trap() position to come before handle_trap()

With this change, we can avoid a forward declaration. Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: None Changes in v2: - rebase on u-boot/master - drop the patch "riscv: Pass correct exception code to _exit_trap()" - drop the patch "riscv: Refactor

[U-Boot] [PATCH v4 08/25] riscv: Add a SYSCON driver for SiFive's Core Local Interruptor

This adds U-Boot syscon driver for SiFive's Core Local Interruptor (CLINT). The CLINT block holds memory-mapped control and status registers associated with software and timer interrupts. This driver implements the riscv_get_time() API as required by the generic RISC-V timer driver, as well as

[U-Boot] [PATCH v4 11/25] riscv: Enlarge the default SYS_MALLOC_F_LEN

Increase the heap size for the pre-relocation stage, so that CPU driver can be loaded. Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: None Changes in v2: None arch/riscv/Kconfig | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/riscv/Kconfig

[U-Boot] [PATCH v4 09/25] riscv: Implement riscv_get_time() API using rdtime instruction

From: Anup Patel This adds an implementation of riscv_get_time() API that is using rdtime instruction. This is the case for S-mode U-Boot, and is useful for processors that support rdtime in M-mode too. Signed-off-by: Anup Patel Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes

[U-Boot] [PATCH v4 13/25] riscv: Remove non-DM version of print_cpuinfo()

With DM CPU driver, the non-DM version of print_cpuinfo() is no longer needed. Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: - new patch to remove non-DM version of print_cpuinfo() Changes in v2: None arch/riscv/cpu/cpu.c | 37

[U-Boot] [PATCH v4 06/25] riscv: ax25: Hide the ax25-specific Kconfig option

There is no need to expose RISCV_NDS to the Kconfig menu as it is an ax25-specific option. Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: None Changes in v2: None arch/riscv/cpu/ax25/Kconfig | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-)

[U-Boot] [PATCH v4 05/25] timer: Add generic driver for RISC-V privileged architecture defined timer

RISC-V privileged architecture v1.10 defines a real-time counter, exposed as a memory-mapped machine-mode register - mtime. mtime must run at constant frequency, and the platform must provide a mechanism for determining the timebase of mtime. The mtime register has a 64-bit precision on all RV32,

[U-Boot] [PATCH v4 04/25] cpu: Add a RISC-V CPU driver

This adds a driver for RISC-V CPU. Note the driver will bind a RISC-V timer driver if "timebase-frequency" property is present in the device tree. Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: None Changes in v2: - pass NULL as the timer device to

[U-Boot] [PATCH v4 03/25] riscv: qemu: Create a simple-bus driver for the soc node

To enumerate devices on the /soc/ node, create a "simple-bus" driver to match "riscv-virtio-soc". Signed-off-by: Bin Meng Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: None Changes in v2: - add DM_FLAG_PRE_RELOC flag to the simple-bus driver arch/riscv/cpu/qemu/cpu.c | 14

[U-Boot] [PATCH v4 01/25] riscv: add Kconfig entries for the code model

From: Lukas Auer RISC-V has two code models, medium low (medlow) and medium any (medany). Medlow limits addressable memory to a single 2 GiB range between the absolute addresses -2 GiB and +2 GiB. Medany limits addressable memory to any single 2 GiB address range. By default, medlow is selected

[U-Boot] [PATCH v4 02/25] dm: cpu: Add timebase frequency to the platdata

This adds a timebase_freq member to the 'struct cpu_platdata', to hold the "timebase-frequency" value in the cpu or /cpus node. Signed-off-by: Bin Meng Reviewed-by: Simon Glass Reviewed-by: Lukas Auer --- Changes in v4: None Changes in v3: None Changes in v2: - Use 'Hz' instead of 'HZ'

[U-Boot] [PATCH v4 00/25] riscv: Adding RISC-V CPU and timer driver

This adds DM drivers to support RISC-V CPU and timer, plus some bug fixes. This series is available at u-boot-x86/riscv-working for testing. Changes in v4: - print warnings for S-mode without CONFIG_CPU Changes in v3: - remove 'select RISCV_CLINT' in Kconfig - do not mention the spec version

Re: [U-Boot] [PATCH v3 16/25] riscv: Update supports_extension() to use desc from cpu driver

Hi Lukas, On Wed, Dec 12, 2018 at 7:40 AM Auer, Lukas wrote: > > Hi Bin, > > On Tue, 2018-12-11 at 01:34 -0800, Bin Meng wrote: > > This updates supports_extension() implementation to use the desc > > string from the cpu driver whenever possible, which avoids the > > reading of misa CSR for

Re: [U-Boot] [PATCH] qemu-arm: Add persistent environment support

On Wed, 12 Dec 2018 at 07:09, Takahiro Akashi wrote: > > On Tue, Dec 11, 2018 at 06:04:05PM +0530, Sumit Garg wrote: > > On Mon, 26 Nov 2018 at 16:51, Sumit Garg wrote: > > > > > > Currently on qemu-arm platforms environment is kept in RAM. Instead > > > use pflash device 1 to provide persistent

Re: [U-Boot] [PATCH v2 2/2] riscv: qemu: Imply SIFIVE_SERIAL for emulation

On Tue, Dec 11, 2018 at 11:02 PM Anup Patel wrote: > > This patch enables SiFive UART driver for QEMU RISC-V emulation > by implying SIFIVE_SERIAL on BOARD_SPECIFIC_OPTIONS. > > Signed-off-by: Anup Patel > --- > board/emulation/qemu-riscv/Kconfig | 1 + > 1 file changed, 1 insertion(+) >

Re: [U-Boot] [PATCH v2 1/2] drivers: serial: Add SiFive UART driver

On Tue, Dec 11, 2018 at 11:02 PM Anup Patel wrote: > > This patch adds SiFive UART driver. The driver is 100% DM driver > and it determines input clock using clk framework. > > Signed-off-by: Anup Patel > Reviewed-by: Palmer Dabbelt > --- > drivers/serial/Kconfig | 13 +++ >

Re: [U-Boot] [PATCH v2 0/2] SiFive UART support

On Wed, Dec 12, 2018 at 12:20 AM Anup Patel wrote: > > On Tue, Dec 11, 2018 at 9:45 PM Bin Meng wrote: > > > > Hi Anup, > > > > On Wed, Dec 12, 2018 at 12:07 AM Anup Patel wrote: > > > > > > On Tue, Dec 11, 2018 at 9:03 PM Bin Meng wrote: > > > > > > > > Hi Anup, > > > > > > > > On Tue, Dec

[U-Boot] u-boot: at91: wdt broken for smartweb and taurus board

Hello all, just found out, that since patch: commit 1473f6ac882fde8078826ca828aa3494ff98bf08 Author: Prasanthi Chellakumar Date: Tue Oct 9 11:46:40 2018 -0700 arm: at91: wdt: Convert watchdog driver to dm/dt Convert the Watchdog driver for AT91SAM9x processors to support the

Re: [U-Boot] [PATCH u-boot-marvell v3 09/10] board: turris_mox: Support 1 GB version of Turris Mox

Hi, I have found the bug causing this issue. If I understand the algorithm in get_ram_size correctly, it does approximately this. Suppose A, B, C, D, E, F are different constatnts. X(i) is a value at address 1< wrote: > On 11.12.18 15:53, Marek Behún wrote: > > On Tue, 11 Dec 2018 15:28:11 +0100

[U-Boot] [imx/next V2] tools: imx8m_image: use bash

travis-ci reports "tools/imx8m_image.sh: 15: [: signed_hdmi_imx8m.bin: unexpected operator" After use bash, no error, so use bash instead of sh. Signed-off-by: Peng Fan --- tools/imx8m_image.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/imx8m_image.sh

Re: [U-Boot] [imx/next] tools: imx8m_image: use bash

> -Original Message- > From: Fabio Estevam [mailto:feste...@gmail.com] > Sent: 2018年12月12日 9:48 > To: Peng Fan > Cc: Stefano Babic ; U-Boot-Denx > Subject: Re: [U-Boot] [imx/next] tools: imx8m_image: use bash > > Hi Peng, > > On Tue, Dec 11, 2018 at 11:46 PM Peng Fan wrote: > > > >

Re: [U-Boot] [imx/next] tools: imx8m_image: use bash

Hi Peng, On Tue, Dec 11, 2018 at 11:46 PM Peng Fan wrote: > > Use bash instead of sh Please explain the reason in the commit log. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] [PATCH 1/3] mx7: Do not call lcdif_power_down() in the SPL case

> -Original Message- > From: Otavio Salvador [mailto:ota...@ossystems.com.br] > Sent: 2018年12月12日 2:41 > To: U-Boot Mailing List > Cc: Stefano Babic ; Fabio Estevam > ; Marek Vasut ; Richard Hu > ; John Weber ; > Fabio Estevam ; Otavio Salvador > ; Ye Li ; Peng Fan > ; Bryan O'Donoghue

[U-Boot] [imx/next] tools: imx8m_image: use bash

Use bash instead of sh Signed-off-by: Peng Fan --- V1: CI report tools/imx8m_image.sh: 15: [: signed_hdmi_imx8m.bin: unexpected operator tools/imx8m_image.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/imx8m_image.sh b/tools/imx8m_image.sh index

Re: [U-Boot] [PATCH] qemu-arm: Add persistent environment support

On Tue, Dec 11, 2018 at 06:04:05PM +0530, Sumit Garg wrote: > On Mon, 26 Nov 2018 at 16:51, Sumit Garg wrote: > > > > Currently on qemu-arm platforms environment is kept in RAM. Instead > > use pflash device 1 to provide persistent environment support across > > device reset. > > > > Also

Re: [U-Boot] [PATCH v2 0/6] rk3399-puma: Enable PWM regulator for RK3399-Q7

Christoph, > On 12.12.2018, at 01:57, Christoph Muellner > wrote: > > This patch series allows to tune VDD_LOG on RK3399-Q7 Puma boards > to a voltage level defined in the DTS using a PWM adjustable regulator. > > To do so a reimplemenation of the RK3399 pinctrl driver has been done. >

[U-Boot] [PATCH v2 4/6] dm: pinctrl: Add pinctrl_decode_pin_config_dm().

pinctrl_decode_pin_config_dm() is basically a feature-equivalent implementation of pinctrl_decode_pin_config(), which operates on struct udevice devices and uses the dev_read_*() API. Signed-off-by: Christoph Muellner --- Changes in v2: None drivers/pinctrl/pinctrl-uclass.c | 15

Re: [U-Boot] [PATCH v2 1/6] rockchip: rk3399-puma: Cleanup of vdd_log DTS entry.

> On 12.12.2018, at 01:57, Christoph Muellner > wrote: > > This patch eliminates the non-standard entries "rockchip,pwm_id" > and "rockchip,pwm_voltage". They are neither documented nor > read out by any driver. > > Additionally it introduces the entry regulator-init-microvolt > and sets it

[U-Boot] [PATCH v2 2/6] power: regulator: Allow PWM regulator to be omitted from SPL.

This patch allows to enable the PWM regulator driver independent for U-Boot and SPL. Signed-off-by: Christoph Muellner --- Changes in v2: None drivers/power/regulator/Kconfig | 7 +++ drivers/power/regulator/Makefile | 2 +- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git

[U-Boot] [PATCH v2 6/6] rockchip: rk3399-puma: Set VDD_LOG to 950 mV.

This patch sets VDD_LOG to 950 mV on RK3399-Q7. This is required to address stability issues on Puma in heavy-load use-cases. Reported-by: Assaf Agmon Signed-off-by: Philipp Tomsich Signed-off-by: Christoph Muellner --- Changes in v2: - Changed patches according to review feedback. - Fix

[U-Boot] [PATCH v2 3/6] rockchip: rk3399-puma: enable PWM regulator in Puma defconfig.

This patch enables the PWM regulator driver in the defconfig for the RK3399-Q7. Signed-off-by: Christoph Muellner --- Changes in v2: None configs/puma-rk3399_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/puma-rk3399_defconfig b/configs/puma-rk3399_defconfig index

[U-Boot] [PATCH v2 5/6] rockchip: rk3399: Add improved pinctrl driver.

The current pinctrl driver for the RK3399 has a range of qulity issues. E.g. it only implements the .set_state_simple() callback, it does not parse the available pinctrl information from the DTS (instead uses hardcoded values), is not flexible enough to cover devices without 'interrupt' field in

[U-Boot] [PATCH v2 0/6] rk3399-puma: Enable PWM regulator for RK3399-Q7

This patch series allows to tune VDD_LOG on RK3399-Q7 Puma boards to a voltage level defined in the DTS using a PWM adjustable regulator. To do so a reimplemenation of the RK3399 pinctrl driver has been done. Although the new pinctrl driver is written in a way, that we could merge it with other

[U-Boot] [PATCH v2 1/6] rockchip: rk3399-puma: Cleanup of vdd_log DTS entry.

This patch eliminates the non-standard entries "rockchip,pwm_id" and "rockchip,pwm_voltage". They are neither documented nor read out by any driver. Additionally it introduces the entry regulator-init-microvolt and sets it to 900 mV, which is the default target value for VDD_LOG. Signed-off-by:

Re: [U-Boot] [PATCH v1] rockchip: rk3399-puma: Remove obsolete DTS node 'vcc5v0_host'.

> On 12.12.2018, at 01:32, Christoph Muellner > wrote: > > vcc5v0_host and usbhub_enable share gpio4 RK_PA3, > which is a problem during probing (the second probe > will trigger a -EBUSY, when trying to get the gpio handle). > > An analysis of the situation shows, that both regulators > are

[U-Boot] [PATCH v1] rockchip: rk3399-puma: Remove obsolete DTS node 'vcc5v0_host'.

vcc5v0_host and usbhub_enable share gpio4 RK_PA3, which is a problem during probing (the second probe will trigger a -EBUSY, when trying to get the gpio handle). An analysis of the situation shows, that both regulators are actually describing the same supply. This patch removes the (currenlty

Re: [U-Boot] [PATCH v3 16/25] riscv: Update supports_extension() to use desc from cpu driver

Hi Bin, On Tue, 2018-12-11 at 01:34 -0800, Bin Meng wrote: > This updates supports_extension() implementation to use the desc > string from the cpu driver whenever possible, which avoids the > reading of misa CSR for S-mode U-Boot. > > Signed-off-by: Bin Meng > > --- > > Changes in v3: > -

Re: [U-Boot] [PATCH v3 25/25] riscv: Remove ae350.dts

On Tue, 2018-12-11 at 01:35 -0800, Bin Meng wrote: > This is not used by any board. Remove it. > > Signed-off-by: Bin Meng > > --- > > Changes in v3: > - new patch to remove ae350.dts > > Changes in v2: None > > arch/riscv/dts/ae350.dts | 229 --- >

Re: [U-Boot] [PATCH v3 24/25] riscv: bootm: Change to use boot_hart from global data

On Tue, 2018-12-11 at 01:35 -0800, Bin Meng wrote: > Avoid reading mhartid CSR directly, instead use the one we saved > in the global data structure before. > > With this patch, BBL no longer needs to be hacked to provide the > mhartid CSR emulation for S-mode U-Boot. > > Signed-off-by: Bin Meng

Re: [U-Boot] [PATCH v3 23/25] riscv: Save boot hart id to the global data

On Tue, 2018-12-11 at 01:34 -0800, Bin Meng wrote: > At present the hart id passed via a0 in the U-Boot entry is saved > to s0 at the beginning but does not preserve later. Save it to the > global data structure so that it can be used later. > > Signed-off-by: Bin Meng > > --- > > Changes in

Re: [U-Boot] [PATCH v3 18/25] riscv: Do some basic architecture level cpu initialization

On Tue, 2018-12-11 at 01:34 -0800, Bin Meng wrote: > In arch_cpu_init_dm() do some basic architecture level cpu > initialization, like FPU enable, etc. > > Signed-off-by: Bin Meng > > --- > > Changes in v3: > - only initialize mcounteren CSR for S-mode > - only touch satp in M-mode U-Boot > -

Re: [U-Boot] [PATCH v3 13/25] riscv: Remove non-DM version of print_cpuinfo()

On Tue, 2018-12-11 at 01:34 -0800, Bin Meng wrote: > With DM CPU driver, the non-DM version of print_cpuinfo() is no > longer needed. > > Signed-off-by: Bin Meng > > --- > > Changes in v3: > - new patch to remove non-DM version of print_cpuinfo() > > Changes in v2: None > >

Re: [U-Boot] [PATCH v3 08/25] riscv: Add a SYSCON driver for SiFive's Core Local Interruptor

On Tue, 2018-12-11 at 01:34 -0800, Bin Meng wrote: > This adds U-Boot syscon driver for SiFive's Core Local Interruptor > (CLINT). The CLINT block holds memory-mapped control and status > registers associated with software and timer interrupts. > > This driver implements the riscv_get_time() API

Re: [U-Boot] [PATCH 2/2] mtd: Get rid of board_mtdparts_default()

Hi Boris, On Mon, Dec 10, 2018 at 04:38:50PM +0100, Boris Brezillon wrote: > The only implementer of this function has been patched to use > CONFIG_MTD{IDS,PARTS}_DEFAULT instead. Let's get rid of this function > and the associated CONFIG_SYS_MTDPARTS_RUNTIME option. the only implementer of this

Re: [U-Boot] policy regarding unused code

On Tue, 11 Dec 2018 16:46:01 +0100 Jean-Jacques Hiblot wrote: > On 11/12/2018 16:25, Jean-Jacques Hiblot wrote: > > Hi Tom, > > > > Is there such a policy described somewhere ? > > > > There are quite a pieces of code that are not used/compiled because > > no config enables it. > > > > A random

Re: [U-Boot] [PATCH v5 0/9] Fix CVE-2018-18440 and CVE-2018-18439

Am 11.12.2018 um 21:10 schrieb Tom Rini: On Tue, Dec 11, 2018 at 04:19:44PM +0100, Simon Goldschmidt wrote: Hi Tom, [truncated the CC list a bit since I got "too many recipients" errors last time] Am 11.12.2018 um 14:31 schrieb Tom Rini: On Sun, Dec 09, 2018 at 09:45:13PM +0100, Simon

Re: [U-Boot] [PATCH] test: dm: regmap: Fix the long test delay

Hi Neil, On Tue, 11 Dec 2018 at 01:27, Neil Armstrong wrote: > > On 10/12/2018 01:11, Simon Glass wrote: > > At present one of the regmap tests takes 5 seconds to run since it waits > > for a timeout. This should be handled using sandbox_timer_add_offset() > > which advances time for test

Re: [U-Boot] [PATCH v4 1/6] common: command: Fix command auto-completion

On Tue, 11 Dec 2018 at 04:24, Boris Brezillon wrote: > > Hi Simon, > > On Mon, 10 Dec 2018 18:04:20 -0700 > Simon Glass wrote: > > > Hi Boris, > > > > On Mon, 3 Dec 2018 at 14:54, Boris Brezillon > > wrote: > > > > > > When auto-completing command arguments, the last argument is not > > >

Re: [U-Boot] [PATCH v5 0/9] Fix CVE-2018-18440 and CVE-2018-18439

On Tue, Dec 11, 2018 at 04:19:44PM +0100, Simon Goldschmidt wrote: > Hi Tom, > > [truncated the CC list a bit since I got "too many recipients" errors last > time] > > Am 11.12.2018 um 14:31 schrieb Tom Rini: > >On Sun, Dec 09, 2018 at 09:45:13PM +0100, Simon Goldschmidt wrote: > > > >>This

Re: [U-Boot] [PATCH v2 2/3] efi_loader: enumerate disk devices every time

On 11/15/18 5:58 AM, AKASHI Takahiro wrote: > Currently, efi_init_obj_list() scan disk devices only once, and never > change a list of efi disk devices. This will possibly result in failing > to find a removable storage which may be added later on. See [1]. > > In this patch, called is

[U-Boot] [BUG] build warnings for tools/mtk_image.c

Hello Ryder, commit 3b975a147c3c ("tools: MediaTek: add MTK boot header generation to mkimage") leads to build warnings: In function ‘mtk_image_set_gen_header’, inlined from ‘mtk_image_set_header’ at tools/mtk_image.c:733:3: tools/mtk_image.c:659:2: warning: ‘strncpy’ specified bound 12

Re: [U-Boot] policy regarding unused code

On Tue, 11 Dec 2018, Tom Rini wrote: ... snip ... > > This is not a small list and a lot of them we probably want to keep. > > > > ['CONFIG_MX8M', 'CONFIG_POWER_PMIC_MAX77693', 'CONFIG_TPM2_TIS_SANDBOX', > > 'CONFIG_TPM_ST33ZP24_SPI', 'CONFIG_USB_SL811HS', 'CONFIG_RAM_SUPPORT', > >

Re: [U-Boot] Regression: dm: i2c: Make i2c_get_chip_for_busnum() fail if the chip is not detected

Hi Stephen, On Mon, 10 Dec 2018 at 11:23, Stephen Warren wrote: > > The following commit: > > > dm: i2c: Make i2c_get_chip_for_busnum() fail if the chip is not detected > > > > i2c_get_chip_for_busnum() really should check the presence of the chip > > on > > the bus. Most of the users

Re: [U-Boot] [PATCH 4/4] efi_loader: bootmgr: make BootNext non-volatile

On 11/28/18 7:00 AM, AKASHI Takahiro wrote: > See UEFI specification v2.7a, section 3.3 for details attributes. This patch is not applicable to U-Boot master not to efi-next. There is no preceding patch in https://patchwork.ozlabs.org/project/uboot/list/?submitter=61166 I have no clue what this

Re: [U-Boot] [PATCH 3/4] cmd: efishell: make Boot####/BootOrder variable non-volatile

On 11/28/18 7:00 AM, AKASHI Takahiro wrote: > See UEFI specification v2.7a, section 3.3 for details attributes. I cannot find file cmd/efishell.c in U-Boot. And I cannot find it in https://patchwork.ozlabs.org/project/uboot/list/?submitter=61166 So I have no clue what this patch is based on.

[U-Boot] [PATCH] i2c: tegra: Fix regression by implementing a dummy probe_chip() callback

Commit f32a8007ef0f ("dm: i2c: Make i2c_get_chip_for_busnum() fail if the chip is not detected") introduced a regression for the NVIDIA Jetson TX2. For some reason the xfer callback of the tegra i2c driver doesn't support probing the I2C devices with a 0-length message. Fixing the regression by

Re: [U-Boot] [PATCH 2/4] cmd: efishell: support -nv option to setvar sub-command

On 11/28/18 7:00 AM, AKASHI Takahiro wrote: > With -nv specified, a variable to be created will have NON_VOLATILE > attribute. > > Signed-off-by: AKASHI Takahiro Why would we only make the NON_VOLATILE attribute available and not all the other attributes like EFI_VARIABLE_BOOTSERVICE_ACCESS?

Re: [U-Boot] Regression: dm: i2c: Make i2c_get_chip_for_busnum() fail if the chip is not detected

On 11/12/2018 19:22, Stephen Warren wrote: On 12/11/18 11:12 AM, Stephen Warren wrote: On 12/11/18 10:41 AM, Jean-Jacques Hiblot wrote: On 11/12/2018 18:10, Stephen Warren wrote: On 12/11/18 2:44 AM, Jean-Jacques Hiblot wrote: On 11/12/2018 05:41, Heiko Schocher wrote: Hello Stephen, Am

[U-Boot] [PATCH 3/3] logos: Add the TechNexion's logo

From: Fabio Estevam Add the TechNexion's logo from their internal U-Boot tree. Signed-off-by: Fabio Estevam Signed-off-by: Otavio Salvador --- tools/logos/technexion.bmp | Bin 0 -> 22390 bytes 1 file changed, 0 insertions(+), 0 deletions(-) create mode 100644 tools/logos/technexion.bmp

[U-Boot] [PATCH 2/3] pico-imx7d: Add LCD support

From: Fabio Estevam Add support for the VXT VL050-8048NT-C01 panel connected through the 24 bit parallel LCDIF interface. Signed-off-by: Fabio Estevam Signed-off-by: Otavio Salvador --- board/technexion/pico-imx7d/pico-imx7d.c | 55

[U-Boot] [PATCH 1/3] mx7: Do not call lcdif_power_down() in the SPL case

From: Fabio Estevam Like it was done on imx6 in commit 9236269de57d ("imx: mx6: Fix implementantion reset_misc") Do not call lcdif_power_down() in the SPL case to fix the following build error: LD spl/u-boot-spl MKIMAGE u-boot.img arch/arm/mach-imx/built-in.o: In function

Re: [U-Boot] [PATCH 1/4] efi_loader: support non-volatile variable behavior

On 11/28/18 7:00 AM, AKASHI Takahiro wrote: > An EFI variable is nothing but a wrapper of a corresponding u-boot > environment variable (See efi_variable.c), but under the current > implementation, NON_VOLATILE attribute is not honored while u-boot > environment variables can be saved/restored in

Re: [U-Boot] Regression: dm: i2c: Make i2c_get_chip_for_busnum() fail if the chip is not detected

On 12/11/18 11:12 AM, Stephen Warren wrote: On 12/11/18 10:41 AM, Jean-Jacques Hiblot wrote: On 11/12/2018 18:10, Stephen Warren wrote: On 12/11/18 2:44 AM, Jean-Jacques Hiblot wrote: On 11/12/2018 05:41, Heiko Schocher wrote: Hello Stephen, Am 10.12.2018 um 19:23 schrieb Stephen Warren:

Re: [U-Boot] policy regarding unused code

On Tue, Dec 11, 2018 at 06:01:23PM +0100, Jean-Jacques Hiblot wrote: > > On 11/12/2018 17:35, Tom Rini wrote: > >On Tue, Dec 11, 2018 at 04:46:01PM +0100, Jean-Jacques Hiblot wrote: > >>On 11/12/2018 16:25, Jean-Jacques Hiblot wrote: > >>>Hi Tom, > >>> > >>>Is there such a policy described

Re: [U-Boot] Regression: dm: i2c: Make i2c_get_chip_for_busnum() fail if the chip is not detected

On 12/11/18 10:41 AM, Jean-Jacques Hiblot wrote: On 11/12/2018 18:10, Stephen Warren wrote: On 12/11/18 2:44 AM, Jean-Jacques Hiblot wrote: On 11/12/2018 05:41, Heiko Schocher wrote: Hello Stephen, Am 10.12.2018 um 19:23 schrieb Stephen Warren: The following commit: dm: i2c: Make

Re: [U-Boot] Regression: dm: i2c: Make i2c_get_chip_for_busnum() fail if the chip is not detected

On 11/12/2018 18:10, Stephen Warren wrote: On 12/11/18 2:44 AM, Jean-Jacques Hiblot wrote: On 11/12/2018 05:41, Heiko Schocher wrote: Hello Stephen, Am 10.12.2018 um 19:23 schrieb Stephen Warren: The following commit: dm: i2c: Make i2c_get_chip_for_busnum() fail if the chip is not

[U-Boot] [PATCH v1 2/3] x86: edison: move CONFIG_BOOTCOMMAND from header file to defconfig

Use defconfig instead of header file for CONFIG_BOOTCOMMAND. Signed-off-by: Andy Shevchenko --- include/configs/edison.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/include/configs/edison.h b/include/configs/edison.h index 8e312da2da..7e2f9c129e 100644 --- a/include/configs/edison.h

[U-Boot] [PATCH v1 3/3] x86: edison: Remove staled comments from configuration header

Since some options had been moved to defconfig from header, the leftover comments are not needed anymore. Remove them. Signed-off-by: Andy Shevchenko --- include/configs/edison.h | 4 1 file changed, 4 deletions(-) diff --git a/include/configs/edison.h b/include/configs/edison.h index

[U-Boot] [PATCH v1 1/3] x86: edison: move CONFIG_CMD_PCI from header file to defconfig

Use defconfig instead of header file for CONFIG_CMD_PCI. Signed-off-by: Andy Shevchenko --- configs/edison_defconfig | 1 - include/configs/edison.h | 3 --- 2 files changed, 4 deletions(-) diff --git a/configs/edison_defconfig b/configs/edison_defconfig index b523fe92ce..e376c0cea6 100644 ---

Re: [U-Boot] Regression: dm: i2c: Make i2c_get_chip_for_busnum() fail if the chip is not detected

On 12/11/18 2:44 AM, Jean-Jacques Hiblot wrote: On 11/12/2018 05:41, Heiko Schocher wrote: Hello Stephen, Am 10.12.2018 um 19:23 schrieb Stephen Warren: The following commit: dm: i2c: Make i2c_get_chip_for_busnum() fail if the chip is not detected     i2c_get_chip_for_busnum() really

Re: [U-Boot] Regression: dm: i2c: Make i2c_get_chip_for_busnum() fail if the chip is not detected

On 12/10/18 9:41 PM, Heiko Schocher wrote: Hello Stephen, Am 10.12.2018 um 19:23 schrieb Stephen Warren: The following commit: dm: i2c: Make i2c_get_chip_for_busnum() fail if the chip is not detected     i2c_get_chip_for_busnum() really should check the presence of the chip on     the bus.

Re: [U-Boot] policy regarding unused code

On 11/12/2018 17:35, Tom Rini wrote: On Tue, Dec 11, 2018 at 04:46:01PM +0100, Jean-Jacques Hiblot wrote: On 11/12/2018 16:25, Jean-Jacques Hiblot wrote: Hi Tom, Is there such a policy described somewhere ? There are quite a pieces of code that are not used/compiled because no config

[U-Boot] Armada 8k IO windows

I am trying to find out where the IOB windows are setup. I see that the values do not matched the documented defaults, but I also do not see access from u-boot to these registers. I am especially interested where the values for CP-0/PEX0 are coming from (and how to to properly modify them).

Re: [U-Boot] policy regarding unused code

On Tue, Dec 11, 2018 at 04:46:01PM +0100, Jean-Jacques Hiblot wrote: > > On 11/12/2018 16:25, Jean-Jacques Hiblot wrote: > >Hi Tom, > > > >Is there such a policy described somewhere ? > > > >There are quite a pieces of code that are not used/compiled because no > >config enables it. > > > >A

Re: [U-Boot] [PATCH] arm: ti: boot: Increase system partition size

On 12/11/2018 10:20 AM, Sam Protsenko wrote: Android code base is growing, so since Android "Pie" the size of system.img grew up to be about 740 MiB. Let's increase system.img to 1 GiB to accommodate for those changes and leave some margin for future changes. We don't want to make it more than

[U-Boot] [PATCH v1] scripts/check-config.sh: Add usage()

When arguments are not supplied the error message is misleading and doesn't tell what exactly has to be done. Fix this by adding usage() and call it if above circumstance occurs. Signed-off-by: Andy Shevchenko --- scripts/check-config.sh | 9 + 1 file changed, 9 insertions(+) diff

[U-Boot] Bogus message about SCSI during build

Hi! Since X86 implies SCSI and Intel Edison board does not use it, I have got a = WARNING == This board does not use CONFIG_DM_SCSI. Please update the storage controller to use CONFIG_DM_SCSI before the v2019.07 release. Failure to update by the deadline

Re: [U-Boot] [PATCH v2 0/2] SiFive UART support

On Tue, Dec 11, 2018 at 9:45 PM Bin Meng wrote: > > Hi Anup, > > On Wed, Dec 12, 2018 at 12:07 AM Anup Patel wrote: > > > > On Tue, Dec 11, 2018 at 9:03 PM Bin Meng wrote: > > > > > > Hi Anup, > > > > > > On Tue, Dec 11, 2018 at 11:02 PM Anup Patel wrote: > > > > > > > > This patchset adds

[U-Boot] [PATCH] arm: ti: boot: Increase system partition size

Android code base is growing, so since Android "Pie" the size of system.img grew up to be about 740 MiB. Let's increase system.img to 1 GiB to accommodate for those changes and leave some margin for future changes. We don't want to make it more than 1 GiB, because we should keep userdata partition

Re: [U-Boot] [PATCH v2 0/2] SiFive UART support

Hi Anup, On Wed, Dec 12, 2018 at 12:07 AM Anup Patel wrote: > > On Tue, Dec 11, 2018 at 9:03 PM Bin Meng wrote: > > > > Hi Anup, > > > > On Tue, Dec 11, 2018 at 11:02 PM Anup Patel wrote: > > > > > > This patchset adds SiFive UART driver for SiFive UART > > > found on SiFive boards. > > > > >

Re: [U-Boot] [PATCH v2] efi_loader: Make RTS relocation more robust

Hi Alex, Tested successfully on Khadas VIM board, fix my panic with efivars. Tested-by: Loic Devulder On 12/11/18 10:00 AM, Alexander Graf wrote: > While changing the RTS alignment to 64KB in commit 7a82c3051c8f > ("efi_loader: Align runtime section to 64kb") the relocation code > started to

Re: [U-Boot] [PATCH v2 0/2] SiFive UART support

On Tue, Dec 11, 2018 at 9:03 PM Bin Meng wrote: > > Hi Anup, > > On Tue, Dec 11, 2018 at 11:02 PM Anup Patel wrote: > > > > This patchset adds SiFive UART driver for SiFive UART > > found on SiFive boards. > > > > The driver is tested on QEMU sifive_u machine. In fact, > > with this patchset

Re: [U-Boot] policy regarding unused code

On 11/12/2018 16:25, Jean-Jacques Hiblot wrote: Hi Tom, Is there such a policy described somewhere ? There are quite a pieces of code that are not used/compiled because no config enables it. A random pick among many others: - CONFIG_FPGA_LATTICE - CONFIG_MESON_SPIFC -

Re: [U-Boot] [PATCH v2 0/2] SiFive UART support

Hi Anup, On Tue, Dec 11, 2018 at 11:02 PM Anup Patel wrote: > > This patchset adds SiFive UART driver for SiFive UART > found on SiFive boards. > > The driver is tested on QEMU sifive_u machine. In fact, > with this patchset same U-Boot binary boots on QEMU virt > machine and QEMU sifive_u

  1   2   >