Re: [PATCH v2] fdt: Use phandle to distinguish DT nodes with same name

2020-12-02 Thread Aswath Govindraju
Hi Simon, On 03/12/20 3:51 am, Simon Glass wrote: > Hi Aswath, > > On Wed, 2 Dec 2020 at 08:42, Aswath Govindraju wrote: >> >> While assigning the sequence number to subsystem instances by reading the >> aliases property, only DT nodes names are compared and not the complete >> path. This

[PATCH v3] fdt: Use phandle to distinguish DT nodes with same name

2020-12-02 Thread Aswath Govindraju
While assigning the sequence number to subsystem instances by reading the aliases property, only DT nodes names are compared and not the complete path. This causes a problem when there are two DT nodes with same name but have different paths. In arch/arm/dts/k3-am65-main.dtsi there are two USB

[RESEND,PATCH v3] cmd: Add a pwm command

2020-12-02 Thread Pragnesh Patel
Add the command "pwm" for controlling the pwm channels. This command provides pwm invert/config/enable/disable functionalities via PWM uclass drivers Signed-off-by: Pragnesh Patel Reviewed-by: Simon Glass --- Changes in v3: - Replace goto with return - Print return value for error - Change the

RE: [PATCH v3] cmd: Add a pwm command

2020-12-02 Thread Pragnesh Patel
Forgot to add "Reviewed-by: Simon Glass " tag, will resend this patch. >-Original Message- >From: Pragnesh Patel >Sent: 03 December 2020 10:21 >To: u-boot@lists.denx.de >Cc: atish.pa...@wdc.com; palmerdabb...@google.com; bmeng...@gmail.com; >Paul Walmsley ( Sifive) ; anup.pa...@wdc.com;

[PATCH v3] cmd: Add a pwm command

2020-12-02 Thread Pragnesh Patel
Add the command "pwm" for controlling the pwm channels. This command provides pwm invert/config/enable/disable functionalities via PWM uclass drivers Signed-off-by: Pragnesh Patel --- Changes in v3: - Replace goto with return - Print return value for error - Change the assert condition for

[PATCH] powerpc: mpc85xx: Allow boards to override CONFIG_USB_MAX_CONTROLLER_COUNT

2020-12-02 Thread Chris Packham
If the board isn't strapped to enable USB1 then attempting to access it will result in a hang. Avoid this by allowing boards to define CONFIG_USB_MAX_CONTROLLER_COUNT. Signed-off-by: Chris Packham --- arch/powerpc/include/asm/config_mpc85xx.h | 2 ++ 1 file changed, 2 insertions(+) diff --git

Re: [u-boot, v2019.04-aspeed, v1 0/1] Common:fdt: Check for error return

2020-12-02 Thread Joel Stanley
Hello Hongwei, On Wed, 2 Dec 2020 at 19:48, Hongwei Zhang wrote: > > Hello Reviewer, > > Check for negative return value of fdt_noffset from calling > boot_get_fdt_fit(). Otherwise, when fdt subimage is corrupted, the > u-boot report bad hash value but continue loading kernel image and > get

Re: [PATCH 00/27] dm: Change the way sequence numbers are implemented

2020-12-02 Thread Simon Glass
Hi Heinrich, On Tue, 1 Dec 2020 at 01:32, Heinrich Schuchardt wrote: > > On 11/30/20 2:53 AM, Simon Glass wrote: > > At present each device has two sequence numbers, with 'req_seq' being > > set up at bind time and 'seq' at probe time. The idea is that devices > > can 'request' a sequence number

Re: [SPECIFICATION RFC] The firmware and bootloader log specification

2020-12-02 Thread Julius Werner
Standardizing in-memory logging sounds like an interesting idea, especially with regards to components that can run on top of different firmware stacks (things like GRUB or TF-A). But I would be a bit wary of creating a "new standard to rule them all" and then expecting all projects to switch what

Re: [PATCH v2] cmd: Add a pwm command

2020-12-02 Thread Simon Glass
Hi Pragnesh, On Tue, 1 Dec 2020 at 00:05, Pragnesh Patel wrote: > > Hi Simon, > > >-Original Message- > >From: U-Boot On Behalf Of Pragnesh Patel > >Sent: 01 December 2020 11:17 > >To: Simon Glass > >Cc: U-Boot Mailing List ; Atish Patra > >; Palmer Dabbelt ; Bin > >Meng ; Paul

Re: [PATCH v1] x86: edison: Add CPU to compatible string

2020-12-02 Thread Simon Glass
On Wed, 2 Dec 2020 at 03:35, Andy Shevchenko wrote: > > Like in the rest of x86 boards append CPU to the board compatible string. > > Signed-off-by: Andy Shevchenko > --- > arch/x86/dts/edison.dts | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Simon Glass

Re: [PATCH 1/1] efi_loader: replace printf by log in efi_uclass.c

2020-12-02 Thread Simon Glass
On Tue, 1 Dec 2020 at 01:24, Heinrich Schuchardt wrote: > > Use logging functions instead of printf() and debug(). > > Change logging messages for uclass creation and destruction to log_debug(). > > Reported-by: Simon Glass > Signed-off-by: Heinrich Schuchardt > --- >

Re: [PATCH v3 1/2] cmd: load: Fix code indentation

2020-12-02 Thread Simon Glass
On Wed, 2 Dec 2020 at 00:45, Michal Simek wrote: > > Reported by smatch like: > cmd/load.c:228 read_record() warn: inconsistent indenting > > Signed-off-by: Michal Simek > --- > > Changes in v3: > - Remove additional dt changes > > Changes in v2: > - One } was missing. Get it back. > >

[PATCH v5 7/7] doc: board: Add Microchip MPFS Icicle Kit doc

2020-12-02 Thread Padmarao Begari
This doc describes the procedure to build, flash and boot Linux using U-boot on Microchip MPFS Icicle Kit. Signed-off-by: Padmarao Begari Reviewed-by: Anup Patel --- doc/board/index.rst | 1 + doc/board/microchip/index.rst | 9 + doc/board/microchip/mpfs_icicle.rst |

[PATCH v5 5/7] riscv: dts: Add device tree for Microchip Icicle Kit

2020-12-02 Thread Padmarao Begari
Add device tree for Microchip PolarFire SoC Icicle Kit. Signed-off-by: Padmarao Begari Reviewed-by: Anup Patel --- arch/riscv/dts/Makefile | 1 + .../dts/microchip-mpfs-icicle-kit-u-boot.dtsi | 14 + arch/riscv/dts/microchip-mpfs-icicle-kit.dts | 421

[PATCH v5 6/7] riscv: Add Microchip MPFS Icicle Kit support

2020-12-02 Thread Padmarao Begari
This patch adds Microchip MPFS Icicle Kit support. For now, only NS16550 Serial, Microchip clock, Cadence eMMC and MACB drivers are only enabled. The Microchip MPFS Icicle defconfig by default builds U-Boot for S-Mode because U-Boot on Microchip PolarFire SoC will run in S-Mode as payload of HSS +

[PATCH v5 2/7] net: macb: Add DMA 64-bit address support for macb

2020-12-02 Thread Padmarao Begari
Enable 32-bit or 64-bit DMA in the macb driver based on the macb compatible string of the device tree node. Signed-off-by: Padmarao Begari Reviewed-by: Anup Patel --- drivers/net/macb.c | 131 +++-- drivers/net/macb.h | 6 +++ 2 files changed, 120

[PATCH v5 3/7] net: macb: Add phy address to read it from device tree

2020-12-02 Thread Padmarao Begari
Read phy address from device tree and use it to find the phy device if not found then search in the range of 0 to 31. Signed-off-by: Padmarao Begari Reviewed-by: Anup Patel --- drivers/net/macb.c | 13 + 1 file changed, 13 insertions(+) diff --git a/drivers/net/macb.c

[PATCH v5 1/7] riscv: Add DMA 64-bit address support

2020-12-02 Thread Padmarao Begari
dma_addr_t holds any valid DMA address. If the DMA API only uses 32/64-bit addresses, dma_addr_t need only be 32/64 bits wide. Signed-off-by: Padmarao Begari Reviewed-by: Anup Patel --- arch/riscv/Kconfig | 4 arch/riscv/include/asm/types.h | 4 2 files changed, 8

[PATCH v5 4/7] clk: Add Microchip PolarFire SoC clock driver

2020-12-02 Thread Padmarao Begari
Add clock driver code for the Microchip PolarFire SoC. This driver handles reset and clock control of the Microchip PolarFire SoC device. Signed-off-by: Padmarao Begari Reviewed-by: Anup Patel --- drivers/clk/Kconfig | 1 + drivers/clk/Makefile

[PATCH v5 0/7] Microchip PolarFire SoC support

2020-12-02 Thread Padmarao Begari
This patch set adds Microchip PolarFire SoC Icicle Kit support to RISC-V U-Boot. The patches are based upon latest U-Boot tree (https://gitlab.denx.de/u-boot/u-boot.git) at commit id 80cbd731df50b9ab646940ea862b70bcaff37225 All drivers namely: NS16550 Serial, Microchip clock, Cadence eMMC and

Re: [PATCH v4] usb: xhci: fix lack of short packet event trb handling

2020-12-02 Thread Marek Vasut
On 11/18/20 8:49 AM, Ran Wang wrote: For bulk IN transfer, the codes will set ISP flag to request event TRB being generated by xHC for the case of short packet. So when encountering buffer-cross-64K-boundary (which we will divide payload and enqueuqe more than 1 transfer TRB), and the first TRB

Re: [PATCH] console: cosmetics: remove #if 0

2020-12-02 Thread Simon Glass
On Wed, 2 Dec 2020 at 07:16, Patrick Delaunay wrote: > > Remove the #if 0 present since the first version of console.c > > Signed-off-by: Patrick Delaunay > --- > > common/console.c | 10 -- > 1 file changed, 10 deletions(-) > Reviewed-by: Simon Glass

Re: [PATCH v2] fdt: Use phandle to distinguish DT nodes with same name

2020-12-02 Thread Simon Glass
Hi Aswath, On Wed, 2 Dec 2020 at 08:42, Aswath Govindraju wrote: > > While assigning the sequence number to subsystem instances by reading the > aliases property, only DT nodes names are compared and not the complete > path. This causes a problem when there are two DT nodes with same name but >

RE: [dwi2c PATCH v1] dwi2c add offsets to reads

2020-12-02 Thread Duffin, CooperX
-Original Message- From: Heiko Schocher Sent: Tuesday, November 17, 2020 11:42 PM To: Duffin, CooperX Cc: Simon Glass ; U-Boot Mailing List ; uboot-snps-...@synopsys.com; Tom Rini ; Robert Beckett ; Wolgang Denk ; Ian Ray Subject: Re: [dwi2c PATCH v1] dwi2c add offsets to reads

Re: [Uboot-stm32] [PATCH v2 7/9] console: allow to record console output before ready

2020-12-02 Thread Simon Glass
On Wed, 2 Dec 2020 at 03:55, Patrick DELAUNAY wrote: > > Hi Simon, > > > From: Uboot-stm32 On > > Behalf Of Simon Glass > > > > Hi Patrick, > > > > On Fri, 27 Nov 2020 at 03:21, Patrick Delaunay > > wrote: > > > > > > Allow to record the console output before before U-Boot has a console > > >

Re: [PATCH 10/10] setexpr: Add support for strings

2020-12-02 Thread Tom Rini
On Sun, Nov 01, 2020 at 02:15:44PM -0700, Simon Glass wrote: > Add support for dealing with string operands, including reading a string > from memory into an environment variable and concatenating two strings. > > Signed-off-by: Simon Glass > Acked-by: Marek Behún Applied to u-boot/next,

Re: [PATCH 09/10] setexpr: Convert to use a struct for values

2020-12-02 Thread Tom Rini
On Sun, Nov 01, 2020 at 02:15:43PM -0700, Simon Glass wrote: > At present a ulong is used to hold operand values. This means that > strings cannot be used. While most operations are not useful for strings, > concatenation is. As a starting point to supporting strings, convert the > code to use a

Re: [PATCH 08/10] setexpr: Correct buffer overflow bug and enable tests

2020-12-02 Thread Tom Rini
On Sun, Nov 01, 2020 at 02:15:42PM -0700, Simon Glass wrote: > At present when more than one substitution is made this function > overwrites its buffers. Fix this bug and update the tests now that they > can pass. > > Also update the debug code to show all substrings, since at present it > omits

Re: [PATCH 07/10] setexpr: Correct dropping of final unmatched string

2020-12-02 Thread Tom Rini
On Sun, Nov 01, 2020 at 02:15:41PM -0700, Simon Glass wrote: > At present the 'nlen' variable increases with each loop. If the previous > loop had back references, then subsequent loops without back references > use the wrong value of nlen. The value is larger, meaning that the string >

Re: [PATCH 06/10] setexpr: Add some tests for buffer overflow and backref

2020-12-02 Thread Tom Rini
On Sun, Nov 01, 2020 at 02:15:40PM -0700, Simon Glass wrote: > Add tests to check for buffer overflow using simple replacement as well > as back references. At present these don't fully pass. > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc Description:

Re: [PATCH 05/10] setexpr: Split the core logic into its own function

2020-12-02 Thread Tom Rini
On Sun, Nov 01, 2020 at 02:15:39PM -0700, Simon Glass wrote: > At present this function always allocates a large amount of stack, and > selects its own size for buffers. This makes it hard to test the code > for buffer overflow. > > Separate out the inner logic of the substitution so that tests

Re: [PATCH 04/10] test: Add some setexpr regex tests

2020-12-02 Thread Tom Rini
On Sun, Nov 01, 2020 at 02:15:38PM -0700, Simon Glass wrote: > Add tests for the setexpr regex commands. > > Note that these tests currently crash on sandbox due to an existing bug in > the setexpr implementation, so two of the tests are commented out. > > Signed-off-by: Simon Glass Applied

Re: [PATCH 03/10] setexpr: Add explicit support for 32- and 64-bit ints

2020-12-02 Thread Tom Rini
On Sun, Nov 01, 2020 at 02:15:37PM -0700, Simon Glass wrote: > At present this function assumes that a size of 4 refers to a ulong. This > is true on 32-bit machines but not commonly on 64-bit machines. > > This means that the 'l' specify does not work correctly with setexpr. > > Add an

Re: [PATCH 02/10] command: Add constants for cmd_get_data_size string / error

2020-12-02 Thread Tom Rini
On Sun, Nov 01, 2020 at 02:15:36PM -0700, Simon Glass wrote: > At present these values are open-coded in a few places. Add constants so > the meaning is clear. > > Also add a comment to cmd_get_data_size() > > Signed-off-by: Simon Glass Applied to u-boot/next, thanks! -- Tom signature.asc

Re: [PATCH 01/10] test: Add some tests for setexpr

2020-12-02 Thread Tom Rini
On Sun, Nov 01, 2020 at 02:15:35PM -0700, Simon Glass wrote: > This command currently has no tests. Add some for basic assignment and the > integer operations. > > Note that the default size for setexpr is ulong, which varies depending on > the build machine. So for sandbox on a 64-bit host,

Re: [PATCH] km/arm: coding style clean up

2020-12-02 Thread Tom Rini
On Fri, Oct 30, 2020 at 12:45:49PM +0100, Holger Brunck wrote: > Address most of the checkpatch issues we found in km_arm and common km > code. > > CC: Stefan Roese > CC: Valentin Longchamp > Signed-off-by: Holger Brunck > Reviewed-by: Stefan Roese Applied to u-boot/next, thanks! -- Tom

Re: [PATCH] spl: spl_fit.c: enable check of signature for config node in spl/tpl

2020-12-02 Thread Tom Rini
On Thu, Oct 29, 2020 at 06:50:29PM +0100, Philippe Reynes wrote: > This commit add the support of signature check for config node > in spl/tpl when the function spl_load_simple_fit is used. > > Signed-off-by: Philippe Reynes > Reviewed-by: Simon Glass Applied to u-boot/next, thanks! -- Tom

Re: [PATCH 4/4] IPQ40xx: clk: add USB clock handling

2020-12-02 Thread Tom Rini
On Wed, Oct 28, 2020 at 01:56:26PM +0100, Robert Marko wrote: > USB clocks were completely forgotten as driver would always return 0 even if > clock ID was unknown. > > This behaviour changed with "IPQ40xx: clk: dont always return 0" and this > will now causes the USB-s to fail probing as

Re: [PATCH 3/4] IPQ40xx: clk: dont always return 0

2020-12-02 Thread Tom Rini
On Wed, Oct 28, 2020 at 01:56:25PM +0100, Robert Marko wrote: > Currently the driver will go through the clock ID-s and set/enable them as > needed. > But if the ID is unknown it will fall through the switch case to the default > case which will always return 0. > > This is not correct and

Re: [PATCH 2/4] IPQ40xx: clk: drop breaks in switch case

2020-12-02 Thread Tom Rini
On Wed, Oct 28, 2020 at 01:56:24PM +0100, Robert Marko wrote: > There is no point in having break statements in the switch case as there is > already a return before break. > > So lets drop them from the driver. > > Signed-off-by: Robert Marko > Cc: Luka Perkov Applied to u-boot/next,

Re: [PATCH 1/4] IPQ40xx: clk: use dev_read_addr()

2020-12-02 Thread Tom Rini
On Wed, Oct 28, 2020 at 01:56:23PM +0100, Robert Marko wrote: > Lets convert the driver to use dev_read_addr() instead of the > devfdt_get_addr(). > > While we are here, lets also alphabetise the includes. > > Signed-off-by: Robert Marko > Cc: Luka Perkov Applied to u-boot/next, thanks! --

Re: [PATCH] Makefile: Correctly propagate failure when removing target

2020-12-02 Thread Tom Rini
On Mon, Oct 26, 2020 at 02:10:49PM +0100, Pali Rohár wrote: > On more places is used pattern 'command > $@ || rm -f $@'. But it does not > propagate failure from 'command' as 'rm -f' returns success. > > Fix it by calling 'false' to correctly propagate failure after 'rm -f'. > > Signed-off-by:

Re: [PATCH 2/2] cmd: Kconfig: migrate CONFIG_SYS_PROMPT_HUSH_PS2

2020-12-02 Thread Tom Rini
On Mon, Oct 26, 2020 at 09:31:42AM +0100, Patrick Delaunay wrote: > Move CONFIG_SYS_PROMPT_HUSH_PS2 in Kconfig, depending > on CONFIG_HUSH_PARSER, and remove the default value defined > in cli_hush.c under __U_BOOT__. > > Signed-off-by: Patrick Delaunay > Reviewed-by: Simon Glass Applied to

Re: [PATCH 1/1] net: sntp: remove CONFIG_TIMESTAMP constraint

2020-12-02 Thread Tom Rini
On Fri, Oct 23, 2020 at 12:58:27PM +0200, Heinrich Schuchardt wrote: > CONFIG_TIMESTAMP is not related to the RTC drivers. It does not make any > sense to let the updating of the RTC by the sntp command depend on it. > > Drop the CONFIG_TIMESTAMP checks. > > Furthermore function dm_rtc_set() is

Re: [PATCH] gpio: Convert to use APIs which support live DT

2020-12-02 Thread Tom Rini
On Wed, Sep 09, 2020 at 06:26:16PM +0200, Patrick Delaunay wrote: > Use ofnode_ or dev_ APIs instead of fdt_ and fdtdec_ APIs so that the > driver can support live DT. > > Signed-off-by: Patrick Delaunay > Reviewed-by: Simon Glass > Reviewed-by: Heiko Schocher Applied to u-boot/next, thanks!

Re: [U-Boot] [PATCH v2] cmd: pxe: Use internal FDT if retrieving from FDTDIR fails

2020-12-02 Thread Tom Rini
On Tue, Sep 03, 2019 at 10:52:24AM +0300, Anton Leontiev wrote: > From: Anton Leontiev > > As FDTDIR label doesn't specify exact file to be loaded, it should > not fail if no file exists in the directory. In this case try to boot > with internal FDT if it exists. > > Signed-off-by: Anton

Re: [U-Boot] [PATCH] Adds basic support for ProxyDHCP

2020-12-02 Thread Tom Rini
On Mon, Aug 05, 2019 at 06:23:42AM -0400, lylejfrank...@gmail.com wrote: > From: Lyle Franklin > > - ProxyDHCP allows a second DHCP server to exist alongside your main > DHCP server and supply additional BOOTP related options > - When u-boot sends out a DHCP request, the real DHCP server will

[u-boot, v2019.04-aspeed, v1 1/1] Common:fdt: Check for error return value

2020-12-02 Thread Hongwei Zhang
Check for negative return value of fdt_noffset from calling boot_get_fdt_fit(). Signed-off-by: Hongwei Zhang --- common/image-fdt.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/common/image-fdt.c b/common/image-fdt.c index 01186aeac7..52ada56fc1 100644 --- a/common/image-fdt.c +++

[u-boot,v2019.04-aspeed, v1 0/1] Common:fdt: Check for error return

2020-12-02 Thread Hongwei Zhang
Hello Reviewer, Check for negative return value of fdt_noffset from calling boot_get_fdt_fit(). Otherwise, when fdt subimage is corrupted, the u-boot report bad hash value but continue loading kernel image and get hanged later. Hongwei Zhang (1): Common:fdt: Check for error return value

Re: [PATCH v1] linux/compat.h: Remove debug() from spin_lock_irqsave()

2020-12-02 Thread Andy Shevchenko
On Wed, Dec 02, 2020 at 01:37:35PM -0500, Tom Rini wrote: > On Wed, Dec 02, 2020 at 08:28:55PM +0200, Andy Shevchenko wrote: > > On Thu, Nov 19, 2020 at 10:21:39PM +0200, Andy Shevchenko wrote: > > > Update Tom's address > > > > Tom, should I resend with your address fixed? > > No, I grab things

Re: [PATCH v1] linux/compat.h: Remove debug() from spin_lock_irqsave()

2020-12-02 Thread Tom Rini
On Wed, Dec 02, 2020 at 08:28:55PM +0200, Andy Shevchenko wrote: > On Thu, Nov 19, 2020 at 10:21:39PM +0200, Andy Shevchenko wrote: > > Update Tom's address > > Tom, should I resend with your address fixed? No, I grab things from patchwork. This is on my TODO list, btw. -- Tom signature.asc

Re: [PATCH v1] linux/compat.h: Remove debug() from spin_lock_irqsave()

2020-12-02 Thread Andy Shevchenko
On Thu, Nov 19, 2020 at 10:21:39PM +0200, Andy Shevchenko wrote: > Update Tom's address Tom, should I resend with your address fixed? -- With Best Regards, Andy Shevchenko

[PATCH v2 1/3] MAINTAINERS: Update ARM STI and ARM STM STM32MP Arch maintainers emails

2020-12-02 Thread Patrice Chotard
Update Patrick and my email address with the one dedicated to upstream activities. Signed-off-by: Patrice Chotard --- (no changes since v1) MAINTAINERS | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 874cf2c0e5..ed5d7c3ab6 100644 ---

[PATCH v2 2/3] treewide: Update email address Patrick Delaunay and Patrice Chotard

2020-12-02 Thread Patrice Chotard
Update Patrick and my email address with the one dedicated to upstream activities. Signed-off-by: Patrice Chotard --- (no changes since v1) arch/arm/include/asm/arch-stih410/sdhci.h| 2 +- arch/arm/include/asm/arch-stih410/sys_proto.h| 2 +-

[PATCH v2 3/3] .mailmap: map Patrick Delaunay and my email address

2020-12-02 Thread Patrice Chotard
Add our new email address dedicated for upstream activities. Signed-off-by: Patrice Chotard --- Changes in v2: - Add .mailmap update .mailmap | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.mailmap b/.mailmap index 8250015453..33001f1e01 100644 --- a/.mailmap +++ b/.mailmap @@ -31,6

Re: [PATCH 1/2] MAINTAINERS: Update ARM STI and ARM STM STM32MP Arch maintainers emails

2020-12-02 Thread Patrice CHOTARD
Hi Heinrich On 12/2/20 6:31 PM, Heinrich Schuchardt wrote: > On 12/2/20 6:21 PM, Patrice Chotard wrote: >> Update Patrick and my email address with the one dedicated to >> upstream activities. > > It might make sense to add entries to .mailmap too if you have > contributed to files where you are

[PATCH 1/2] MAINTAINERS: Update ARM STI and ARM STM STM32MP Arch maintainers emails

2020-12-02 Thread Patrice Chotard
Update Patrick and my email address with the one dedicated to upstream activities. Signed-off-by: Patrice Chotard --- MAINTAINERS | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 874cf2c0e5..ed5d7c3ab6 100644 --- a/MAINTAINERS +++

[PATCH 2/2] treewide: Update email address Patrick Delaunay and Patrice Chotard

2020-12-02 Thread Patrice Chotard
Update Patrick and my email address with the one dedicated to upstream activities. Signed-off-by: Patrice Chotard --- arch/arm/include/asm/arch-stih410/sdhci.h| 2 +- arch/arm/include/asm/arch-stih410/sys_proto.h| 2 +- arch/arm/include/asm/arch-stm32/stm32f.h |

[PATCH v2 2/2] ARM: dts: imx8m: add UHS or HS400/HS400ES properties

2020-12-02 Thread Andrey Zhizhikin
i.MX8M series provide support for high speed grades in their usdhc controllers, which has eMMC and SDHC connected to them. Enable this support across the entire i.MX8M family by providing quirks to usdhc controllers designated by storage media connected to them. Signed-off-by: Andrey Zhizhikin

[PATCH v2 1/2] ARM: dts: imx8m: increase off-on delay on the SD Vcc regulator

2020-12-02 Thread Andrey Zhizhikin
Some SD Card controller and power circuitry has increased capacitance, which keeps the internal logic remains powered after regulator is switch off. This is generally the case when card is switched to SD104 mode, where a power cycle should be performed. In case if the card internal logic remains

Re: [PATCH 1/2] MAINTAINERS: Update ARM STI and ARM STM STM32MP Arch maintainers emails

2020-12-02 Thread Heinrich Schuchardt
On 12/2/20 6:21 PM, Patrice Chotard wrote: Update Patrick and my email address with the one dedicated to upstream activities. It might make sense to add entries to .mailmap too if you have contributed to files where you are not the maintainer. Best regards Heinrich Signed-off-by: Patrice

[PATCH v3] tools: add a simple script to generate EFI variables

2020-12-02 Thread Paulo Alcantara
This script generates EFI variables for U-Boot variable store format. A few examples: - Generating secure boot keys $ openssl x509 -in foo.crt -outform DER -out foo.der $ efisiglist -a -c foo.der -o foo.esl $ ./efivar.py -i ubootefi.var set -n db -d foo.esl -t file $

Re: [PATCH v10 11/11] sandbox: enable capsule update for testing

2020-12-02 Thread Tom Rini
On Wed, Dec 02, 2020 at 03:37:54PM +0100, Heinrich Schuchardt wrote: > On 11/30/20 10:12 AM, AKASHI Takahiro wrote: > > Add more configuration options to allow for efi capsule update > > on sandbox. > > > > Signed-off-by: AKASHI Takahiro > > --- > > configs/sandbox64_defconfig | 6 ++ > >

Re: [PATCH 2/3 v2] efi_loader: Introduce eventlog support for TCG2_PROTOCOL

2020-12-02 Thread Ilias Apalodimas
Hi Heinrich [...] > > + rc = tpm2_pcr_extend(dev, pcr_index, alg, > > This does not compile with sandbox_defconfig > > lib/efi_loader/efi_tcg2.c: In function ‘tcg2_pcr_extend’: > lib/efi_loader/efi_tcg2.c:152:40: warning: passing argument 3 of > ‘tpm2_pcr_extend’ makes pointer from

[PATCH v2] fdt: Use phandle to distinguish DT nodes with same name

2020-12-02 Thread Aswath Govindraju
While assigning the sequence number to subsystem instances by reading the aliases property, only DT nodes names are compared and not the complete path. This causes a problem when there are two DT nodes with same name but have different paths. In arch/arm/dts/k3-am65-main.dtsi there are two USB

[PATCH] clk: at91: sam9x60: remove the parsing of atmel, main-osc-bypass

2020-12-02 Thread Claudiu Beznea
Remove the parsing of atmel,main-osc-bypass DT property as the SAM9X60 have no support for crystal oscillator bypass. Setting this bit might affect the device functionality. Fixes: a64862284f65 ("clk: at91: sam9x60: add support compatible with CCF") Signed-off-by: Claudiu Beznea ---

Re: [PATCH 2/3 v2] efi_loader: Introduce eventlog support for TCG2_PROTOCOL

2020-12-02 Thread Heinrich Schuchardt
On 11/30/20 10:47 AM, Ilias Apalodimas wrote: > In the previous patches we only introduced a minimal subset of the > EFI_TCG2_PROTOCOL protocol implementing GetCapability(). > So let's continue adding features to it, introducing the > GetEventLog() and HashLogExtendEvent() functions. > > In order

[PATCH 1/1] sandbox: implement runtime system reset

2020-12-02 Thread Heinrich Schuchardt
Implement a reset function that we can call after ExitBootServices(), when all driver model devices are gone. Signed-off-by: Heinrich Schuchardt --- arch/sandbox/cpu/start.c | 10 ++ lib/efi_loader/Kconfig | 3 ++- 2 files changed, 12 insertions(+), 1 deletion(-) diff --git

Re: [sw-dev] Booting Linux from U-Boot in qemu-system-riscv64?

2020-12-02 Thread Michael Opdenacker
Hi Anup, On 12/2/20 3:28 PM, Anup Patel wrote: > root=/dev/vda2 rootwait console=ttyS0 earlycon=sbi Many thanks for the "earlycon=sbi" tip. It doesn't solve the problem yet but definitely helps to get further details: [    0.00] Linux version 5.10.0-rc5-00357-gaae5ab854e38

Re: [PATCH v10 11/11] sandbox: enable capsule update for testing

2020-12-02 Thread Heinrich Schuchardt
On 11/30/20 10:12 AM, AKASHI Takahiro wrote: > Add more configuration options to allow for efi capsule update > on sandbox. > > Signed-off-by: AKASHI Takahiro > --- > configs/sandbox64_defconfig | 6 ++ > configs/sandbox_defconfig | 6 ++ > 2 files changed, 12 insertions(+) This still

Re: [PATCH 4/4] arm: mvebu: Espressobin: Detect presence of emmc at runtime

2020-12-02 Thread Andre Heider
On 25/11/2020 19:20, Pali Rohár wrote: Try to initialize emmc in board_late_init() and if it fails then we know that emmc device is not connected. This allows to use in U-Boot just one DTS file for all Espressobin variants and also to correctly set fdtfile env variable for Linux kernel.

Re: [sw-dev] Booting Linux from U-Boot in qemu-system-riscv64?

2020-12-02 Thread Anup Patel
Hi Michael, Set bootargs to "root=/dev/vda2 rootwait console=ttyS0 earlycon=sbi" in U-Boot before doing bootm Regards, Anup On Wed, 2 Dec, 2020, 12:51 Michael Opdenacker, < michael.opdenac...@bootlin.com> wrote: > Greetings, > > To prepare an embedded Linux demo (on RiscV), I'm trying to boot

Re: [PATCH 0/4] Use just one DTS file for all Espressobin variants

2020-12-02 Thread Philipp Tomsich
> On 02.12.2020, at 13:37, Andre Heider wrote: > > On 02/12/2020 11:35, Stefan Roese wrote: >> On 02.12.20 10:12, Pali Rohár wrote: >>> On Wednesday 02 December 2020 09:09:15 Stefan Roese wrote: On 02.12.20 01:33, Pali Rohár wrote: > On Wednesday 25 November 2020 19:20:06 Pali Rohár

[PATCH] console: cosmetics: remove #if 0

2020-12-02 Thread Patrick Delaunay
Remove the #if 0 present since the first version of console.c Signed-off-by: Patrick Delaunay --- common/console.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/common/console.c b/common/console.c index 3348436da6..d9dfd61ac3 100644 --- a/common/console.c +++

RE: [PATCH 2/2] console: sandbox: remove unnecessary sandbox code

2020-12-02 Thread Patrick DELAUNAY
Hi Simon, > From: Simon Glass > Sent: lundi 30 novembre 2020 21:12 > > Hi Patrick, > > On Fri, 27 Nov 2020 at 03:49, Patrick Delaunay > wrote: > > > > Remove the specific sandbox code in console.c, as the config > > CONFIG_DEBUG_UART is already supported in drivers/serial/sandbox.c and > >

[PATCH v3 1/1] riscv: reset after crash

2020-12-02 Thread Heinrich Schuchardt
If an exception occurs on ARM or x86, we call panic() which will try to reset the board. Do the same on RISC-V. To avoid -Werror=format-zero-length move a '\n' to the string passed to panic. We don't need a message here as depending on CONFIG_PANIC_HANG we will either see ### ERROR ###

Re: [PATCH] ARM: dts: stm32: Switch to MCO2 for PHY 50 MHz clock

2020-12-02 Thread Patrice CHOTARD
Hi Marek On 12/1/20 11:34 AM, Marek Vasut wrote: > The LAN8710i PHY currently uses 50 MHz clock direct from PLL4P. > To permit PLL4P to run at faster frequency, use MCO2 as a divider. > The PLL4P runs at 100 MHz, supplies MCO2 which divides it by 2 to > 50MHz, and supplies the PHY with 50 MHz via

[PATCHv2 1/2] arm: dts: add imx8mm-cl-iot-gate dts file

2020-12-02 Thread Ying-Chun Liu
From: "Ying-Chun Liu (PaulLiu)" Add board dts for imx8mm-cl-iot-gate Signed-off-by: Kirill Kapranov Signed-off-by: Uri Mashiach Signed-off-by: Valentin Raevsky Signed-off-by: Ying-Chun Liu (PaulLiu) --- arch/arm/dts/Makefile | 2 +

[PATCHv2 0/2] add support for Compulab iot-gate-imx8

2020-12-02 Thread Ying-Chun Liu
From: "Ying-Chun Liu (PaulLiu)" Add initial support for Compulab iot-gate-imx8 board (imx8mm-cl-iot-gate). The initial support includes: - MMC - eMMC - I2C - FEC - Serial console Ying-Chun Liu (PaulLiu) (2): arm: dts: add imx8mm-cl-iot-gate dts file arm: imx8m: add support for Compulab

Re: [PATCH 0/4] Use just one DTS file for all Espressobin variants

2020-12-02 Thread Andre Heider
On 02/12/2020 11:35, Stefan Roese wrote: On 02.12.20 10:12, Pali Rohár wrote: On Wednesday 02 December 2020 09:09:15 Stefan Roese wrote: On 02.12.20 01:33, Pali Rohár wrote: On Wednesday 25 November 2020 19:20:06 Pali Rohár wrote: This patch series change Espressobin code to use in U-Boot

Re: [PATCH v6 17/28] fdt: translate address if #size-cells = <0>

2020-12-02 Thread Heinrich Schuchardt
On 12/1/20 9:48 PM, dario...@libero.it wrote: Dear Heinrich On 11/22/20 5:11 PM, Dario Binacchi wrote: > The __of_translate_address routine translates an address from the > device tree into a CPU physical address. A note in the description of > the routine explains that the crossing of any

RE: [PATCH v2 2/9] log: don't build the trace buffer when log is not ready

2020-12-02 Thread Patrick DELAUNAY
Hi Sean, > From: Sean Anderson > Sent: vendredi 27 novembre 2020 15:51 > > On 11/27/20 5:20 AM, Patrick Delaunay wrote: > > Update _log function to drop any traces when log is yet initialized: > > vsnprintf is no more executed in this case. > > > > This patch allows to reduce the cost for the

RE: [Uboot-stm32] [PATCH v2 7/9] console: allow to record console output before ready

2020-12-02 Thread Patrick DELAUNAY
Hi Simon, > From: Uboot-stm32 On > Behalf Of Simon Glass > > Hi Patrick, > > On Fri, 27 Nov 2020 at 03:21, Patrick Delaunay > wrote: > > > > Allow to record the console output before before U-Boot has a console > > ready. > > > > This patch allows to test the console output in sandbox test

RE: [PATCH v2 4/9] log: use console puts to output trace before LOG init

2020-12-02 Thread Patrick DELAUNAY
Hi Sean, > From: Sean Anderson > Sent: vendredi 27 novembre 2020 16:02 > > On 11/27/20 5:20 AM, Patrick Delaunay wrote: > > Use the console puts functions to output the traces before the log > > initialization (when CONFIG_LOG is not activated). > > > > This patch allows to display the first

Re: [PATCH v2] tools: add a simple script to generate EFI variables

2020-12-02 Thread Heinrich Schuchardt
On 12/1/20 11:58 PM, Paulo Alcantara wrote: This script generates EFI variables for U-Boot variable store format. A few examples: - Generating secure boot keys $ openssl x509 -in foo.crt -outform DER -out foo.der $ efisiglist -a -c foo.der -o foo.esl $ ./efivar.py -i

[PATCH v1] x86: edison: Add CPU to compatible string

2020-12-02 Thread Andy Shevchenko
Like in the rest of x86 boards append CPU to the board compatible string. Signed-off-by: Andy Shevchenko --- arch/x86/dts/edison.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/dts/edison.dts b/arch/x86/dts/edison.dts index bc84bc892e71..97cc6ec386c2 100644 ---

Re: [PATCH 0/4] Use just one DTS file for all Espressobin variants

2020-12-02 Thread Stefan Roese
On 02.12.20 10:12, Pali Rohár wrote: On Wednesday 02 December 2020 09:09:15 Stefan Roese wrote: On 02.12.20 01:33, Pali Rohár wrote: On Wednesday 25 November 2020 19:20:06 Pali Rohár wrote: This patch series change Espressobin code to use in U-Boot just one DTS file for all Espressobin

Re: [PATCH 1/2] ARM: dts: imx8m: increase off-on delay on the SD Vcc regulator

2020-12-02 Thread Peter Robinson
On Wed, Dec 2, 2020 at 10:08 AM ZHIZHIKIN Andrey wrote: > > Hello Peter, > > > -Original Message- > > From: Peter Robinson > > Sent: Wednesday, December 2, 2020 9:45 AM > > To: ZHIZHIKIN Andrey > > Cc: u-boot@lists.denx.de; sba...@denx.de > > Subject: Re: [PATCH 1/2] ARM: dts: imx8m:

RE: [PATCH 1/2] ARM: dts: imx8m: increase off-on delay on the SD Vcc regulator

2020-12-02 Thread ZHIZHIKIN Andrey
Hello Peter, > -Original Message- > From: Peter Robinson > Sent: Wednesday, December 2, 2020 9:45 AM > To: ZHIZHIKIN Andrey > Cc: u-boot@lists.denx.de; sba...@denx.de > Subject: Re: [PATCH 1/2] ARM: dts: imx8m: increase off-on delay on the SD Vcc > regulator > > > On Tue, Dec 1, 2020

Re: [PATCH 0/4] Use just one DTS file for all Espressobin variants

2020-12-02 Thread Pali Rohár
On Wednesday 02 December 2020 09:09:15 Stefan Roese wrote: > On 02.12.20 01:33, Pali Rohár wrote: > > On Wednesday 25 November 2020 19:20:06 Pali Rohár wrote: > > > This patch series change Espressobin code to use in U-Boot just one DTS > > > file for all Espressobin variants. Therefore DT

Re: [PATCH 1/2] ARM: dts: imx8m: increase off-on delay on the SD Vcc regulator

2020-12-02 Thread Peter Robinson
On Tue, Dec 1, 2020 at 1:31 PM Andrey Zhizhikin wrote: > > Some SD Card controller and power circuitry has increased capacitance, > which keeps the internal logic remains powered after regulator is switch > off. This is generally the case when card is switched to SD104 mode, > where a power cycle

Re: [PATCH 0/4] Use just one DTS file for all Espressobin variants

2020-12-02 Thread Stefan Roese
On 02.12.20 01:33, Pali Rohár wrote: On Wednesday 25 November 2020 19:20:06 Pali Rohár wrote: This patch series change Espressobin code to use in U-Boot just one DTS file for all Espressobin variants. Therefore DT compatible string globalscale,espressobin-emmc is not used anymore as it is not

[PATCH] arm64: zynqmp: Sync DT with Linux kernel

2020-12-02 Thread Michal Simek
All changes are recorded in lore.kernel.org. Here are links to that patches for the record. Link: https://lore.kernel.org/r/f59a63d8cb941592de6d2dee8afa6f120b2e40c8.1601379794.git.michal.si...@xilinx.com Link: