Re: [EXT] Re: [PATCH] usb: ehci-hcd: Add IAA handshake for removing async QH

2021-03-09 Thread Marek Vasut
On 3/9/21 4:18 AM, Ye Li wrote: Hi Marek, Hi, [...] + 10 * 1000); /* 10ms timeout */ + if (ret < 0) + printf("EHCI fail timeout STS_IAA set\n"); Shouldn't there be some abort ^ if ret < 0 ? Also, add the return value into the printf, it is useful for deb

Re: [EXT] Re: [PATCH] usb: ehci-hcd: Add IAA handshake for removing async QH

2021-03-09 Thread Ye Li
Hi Marek, On Tue, 2021-03-09 at 09:08 +0100, Marek Vasut wrote: > Caution: EXT Email > > On 3/9/21 4:18 AM, Ye Li wrote: > > > > Hi Marek, > Hi, > > [...] > > > > > > > > > > > > > > > > > > + 10 * 1000); /* 10ms timeout */ > > > > + if (ret < 0) > > > > +  

Re: [PATCH] disk: gpt: verify alternate LBA points to last usable LBA

2021-03-09 Thread Stefan Herbrechtsmeier
Hi Heinrich, Am 08.03.2021 um 18:38 schrieb Heinrich Schuchardt: On 08.03.21 17:07, Stefan Herbrechtsmeier wrote: From: Stefan Herbrechtsmeier The gpt command require the GPT backup header at the standard location at the end of the device.Check the alternate LBA value before reading the GPT b

bug report : /drivers/pci/pcie_layerscape_rc.c "pcie->ctrl used without being initialised"

2021-03-09 Thread florian.man...@siemens.com
Hi, I would like to report a bug I got, while updating my u-boot version. Configuration : CPU : NXP LS1043a u-boot version : v2020.10 (I didn't see a fix for this issue in the latest u-boot version) What was happening : Function "unsigned int ctrl_readl(struct ls_pcie *pcie, unsigned int offs

Re: [EXT] Re: [PATCH] usb: ehci-hcd: Add IAA handshake for removing async QH

2021-03-09 Thread Marek Vasut
On 3/9/21 10:32 AM, Ye Li wrote: Hi Marek, Hi, [...] + 10 * 1000); /* 10ms timeout */ + if (ret < 0) + printf("EHCI fail timeout STS_IAA set\n"); Shouldn't there be some abort ^ if ret < 0 ? Also, add the return value into the printf, it is useful for deb

Re: [PATCH u-boot 16/39] build: use thin archives instead of incremental linking

2021-03-09 Thread Marek Behun
On Tue, 9 Mar 2021 11:42:08 +0800 Bin Meng wrote: > Hi Marek, > > On Tue, Mar 9, 2021 at 9:24 AM Bin Meng wrote: > > > > Hi Marek, > > > > On Mon, Mar 8, 2021 at 11:22 PM Marek Behún wrote: > > > > > > On Mon, 8 Mar 2021 22:30:17 +0800 > > > Bin Meng wrote: > > > > > > > Hi Marek, > > > >

Re: [PATCH 1/4] watchdog: Set/unset GD_FLG_WDT_READY flag in wdt_start()/wdt_stop()

2021-03-09 Thread Stefan Roese
On 05.03.21 22:36, Pali Rohár wrote: Watchdog is ready after successful call of ops->start() callback in wdt_start() function. And is stopped after successful call of ops->stop() callback in wdt_stop function. So move setting of GD_FLG_WDT_READY flag from initr_watchdog() function to wdt_start()

Re: [PATCH 2/4] watchdog: Show error message when initr_watchdog() cannot start watchdog

2021-03-09 Thread Stefan Roese
On 05.03.21 22:36, Pali Rohár wrote: Function wdt_start() may fail. So in initr_watchdog() function check return value of wdt_start() call and print error message when watchdog starting failed. Signed-off-by: Pali Rohár Reviewed-by: Stefan Roese Thanks, Stefan --- drivers/watchdog/wdt-u

Re: [PATCH 3/4] watchdog: Allow to use CONFIG_WDT without starting watchdog

2021-03-09 Thread Stefan Roese
On 05.03.21 22:36, Pali Rohár wrote: In some cases it is useful to compile support for U-Boot command 'wdt' without starting HW watchdog in early U-Boot phase. For example when user Nitpicking: when "the" user want to start watchdog only on demand by some boot script. This change adds a ne

Re: [PATCH 4/4] arm: mvebu: Espressobin: Enable watchdog support but do not start it

2021-03-09 Thread Stefan Roese
On 05.03.21 22:36, Pali Rohár wrote: Espressobin had disabled watchdog support (CONFIG_WDT) because older stable Linux kernel versions (which are used by current stable OpenWRT and Debian versions) do not have support for Armada 3700 watchdog driver. Therefore they are not able to periodically ki

Re: [PATCH 1/2] board: gateworks: venice: fix gsc_get_dev

2021-03-09 Thread Fabio Estevam
Hi Tim, On Mon, Mar 8, 2021 at 6:52 PM Tim Harvey wrote: > > use dm_i2c_probe instead of i2c_get_chip which appears to be more > reliable. > > Signed-off-by: Tim Harvey Reviewed-by: Fabio Estevam

Re: [PATCH 2/2] board: gateworks: venice: increase CONFIG_SYS_SPL_MALLOC_SIZE

2021-03-09 Thread Fabio Estevam
On Mon, Mar 8, 2021 at 6:52 PM Tim Harvey wrote: > > commit 03f1f78a9b44 ("spl: fit: Prefer a malloc()'d buffer for loading > images")' > changed the way buffer allocation worked for SPL to a more flexible > method. > > For venice this caused breakage that is resolved by increasing the size > of

[PATCH 00/16] TI/Cadence: Add Sierra/Torrent SERDES driver

2021-03-09 Thread Kishon Vijay Abraham I
Patch series adds Sierra and Torrent SERDES driver for the SERDES used in TI's K3 platforms. This SERDES is used by USB3, PCIe and Ethernet. This series is mostly an adaptation of drivers added in upstream Linux kernel. Alan Douglas (1): phy: cadence: Add driver for Sierra PHY Aswath Govindraju

[PATCH 01/16] dm: core: Add helper to compare node names

2021-03-09 Thread Kishon Vijay Abraham I
Add helper to compare node names. Signed-off-by: Kishon Vijay Abraham I --- drivers/core/ofnode.c | 13 + include/dm/ofnode.h | 9 + 2 files changed, 22 insertions(+) diff --git a/drivers/core/ofnode.c b/drivers/core/ofnode.c index fa0bd2a9c4..4e196d680e 100644 --- a/driv

[PATCH 02/16] drivers: reset: Handle gracefully NULL pointers

2021-03-09 Thread Kishon Vijay Abraham I
From: Jean-Jacques Hiblot Prepare the way for a managed reset API by handling NULL pointers without crashing nor failing. Signed-off-by: Jean-Jacques Hiblot Signed-off-by: Vignesh Raghavendra Signed-off-by: Kishon Vijay Abraham I --- drivers/reset/reset-uclass.c | 30 +---

[PATCH 03/16] dt-bindings: phy: Add definitions for additional phy types

2021-03-09 Thread Kishon Vijay Abraham I
From: Aswath Govindraju Add definitions for additional phy types that's used specifically for Torrent SERDES. Signed-off-by: Aswath Govindraju Signed-off-by: Kishon Vijay Abraham I --- include/dt-bindings/phy/phy.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/dt-bindings/phy/ph

[PATCH 04/16] phy: cadence: Add driver for Sierra PHY

2021-03-09 Thread Kishon Vijay Abraham I
From: Alan Douglas Add a Sierra PHY driver with PCIe and USB support. This driver is a port from the mainline linux driver. The PHY has multiple lanes, which can be configured into groups, and a generic PHY device is created for each group. There are two resets controlling the overall PHY block

[PATCH 07/16] usb: cdns3: cdns3-ti: Fix clk_get_by_name() to get the correct name

2021-03-09 Thread Kishon Vijay Abraham I
Upstream device tree got updated to use clock name as "ref" instead of "usb2_refclk". Fix cdns3-ti.c to use the correct name. Signed-off-by: Kishon Vijay Abraham I --- drivers/usb/cdns3/cdns3-ti.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/cdns3/cdns3-ti.c b/

[PATCH 05/16] phy: cadence: Add driver for Torrent SERDES

2021-03-09 Thread Kishon Vijay Abraham I
From: Aswath Govindraju Add driver for Torrent SERDES. Signed-off-by: Aswath Govindraju Signed-off-by: Kishon Vijay Abraham I --- drivers/phy/cadence/Kconfig |6 + drivers/phy/cadence/Makefile |1 + drivers/phy/cadence/phy-cadence-torrent.c | 2456 ++

[PATCH 06/16] phy: ti: j721e-wiz: Add support for WIZ module present in TI J721E SoC

2021-03-09 Thread Kishon Vijay Abraham I
From: Jean-Jacques Hiblot Add support for WIZ module present in TI's J721E SoC. WIZ is a SERDES wrapper used to configure some of the input signals to the SERDES. It is used with both Sierra(16G) and Torrent(10G) SERDES. This driver configures three clock selects (pll0, pll1, dig) and supports re

[PATCH 08/16] board: ti: j721e: Add support for probing and configuring Torrent serdes on J7200

2021-03-09 Thread Kishon Vijay Abraham I
From: Aswath Govindraju Add support for probing and configuring Torrent serdes on J7200. Signed-off-by: Aswath Govindraju Signed-off-by: Kishon Vijay Abraham I --- board/ti/j721e/evm.c | 34 +- 1 file changed, 33 insertions(+), 1 deletion(-) diff --git a/board

[PATCH 09/16] ARM: dts: k3-j721e: Add the entries required for USB3 support on USB0

2021-03-09 Thread Kishon Vijay Abraham I
Partially sync with Linux's dts to add the entries required for USB3 support on USB0. Note that the default mode is still "peripheral" not "host". USB3 is supported only for the host mode. Signed-off-by: Jean-Jacques Hiblot Signed-off-by: Vignesh Raghavendra Signed-off-by: Kishon Vijay Abraham I

[PATCH 10/16] arm: dts: k3-j7200-main: Add DT node for torrent serdes

2021-03-09 Thread Kishon Vijay Abraham I
From: Aswath Govindraju Add DT node for torrent serdes. Signed-off-by: Aswath Govindraju Signed-off-by: Kishon Vijay Abraham I --- arch/arm/dts/k3-j7200-main.dtsi | 63 + 1 file changed, 63 insertions(+) diff --git a/arch/arm/dts/k3-j7200-main.dtsi b/arch/arm/

[PATCH 11/16] arm: dts: k3-j7200-common-proc-board: Enable SERDES DT

2021-03-09 Thread Kishon Vijay Abraham I
From: Aswath Govindraju Add default lane function for torrent serdes. Signed-off-by: Aswath Govindraju Signed-off-by: Kishon Vijay Abraham I --- arch/arm/dts/k3-j7200-common-proc-board.dts | 23 + 1 file changed, 23 insertions(+) diff --git a/arch/arm/dts/k3-j7200-common-

[PATCH 12/16] arm: dts: k3-j7200-common-proc-board-u-boot: Add u-boot tags for torrent serdes

2021-03-09 Thread Kishon Vijay Abraham I
From: Aswath Govindraju Add u-boot tags for torrent serdes. Signed-off-by: Aswath Govindraju Signed-off-by: Kishon Vijay Abraham I --- arch/arm/dts/k3-j7200-common-proc-board-u-boot.dtsi | 12 1 file changed, 12 insertions(+) diff --git a/arch/arm/dts/k3-j7200-common-proc-board-

[PATCH 13/16] configs: j721e_evm_a72: Enable the drivers required for the USB3 support

2021-03-09 Thread Kishon Vijay Abraham I
From: Jean-Jacques Hiblot Enable the mmio mux driver, the J721E-wiz PHy driver and the cadence sierra phy driver. All of them are required for USB3 support Signed-off-by: Jean-Jacques Hiblot Signed-off-by: Vignesh Raghavendra Signed-off-by: Kishon Vijay Abraham I --- configs/j721e_evm_a72_de

[PATCH 14/16] configs: j7200_evm_a72_defconfig: Add config for torrent serdes and common clock framework

2021-03-09 Thread Kishon Vijay Abraham I
From: Aswath Govindraju Add config for torrent serdes and common clock framework. Signed-off-by: Aswath Govindraju Signed-off-by: Kishon Vijay Abraham I --- configs/j7200_evm_a72_defconfig | 7 +++ 1 file changed, 7 insertions(+) diff --git a/configs/j7200_evm_a72_defconfig b/configs/j72

[PATCH 15/16] env: ti: j721e-evm: Add env variable to power on & reset QSGMII PHY in J7200 EVM

2021-03-09 Thread Kishon Vijay Abraham I
MAIN CPSW0 requires the PHY to be powered on and reset for QSGMII operation. Add a env variable to configure driving "0" on ENET_EXP_PWRDN controlled by GPIO EXPANDER2 (I2C Addr: 0x22), PIN: 17 and driving "1" on ENET_EXP_RESETZ controlled by GPIO EXPANDER2 (I2C Addr: 0x22), PIN: 18. Signed-off-by

[PATCH 16/16] configs: j7200_evm_a72: Enhance bootcmd to configure ethernet PHY

2021-03-09 Thread Kishon Vijay Abraham I
Update the default BOOTCOMMAND to provide an automatic and easier way to configure ethernet PHY before loading the firmware. Signed-off-by: Kishon Vijay Abraham I --- configs/j7200_evm_a72_defconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configs/j7200_evm_a72_defconf

Re: [PATCH 1/6] ARM: dts: stm32mp1: explicit clock reference needed by RCC clock driver

2021-03-09 Thread Patrice CHOTARD
Hi Patrick On 2/24/21 11:19 AM, Patrick Delaunay wrote: > From: Etienne Carriere > > Define in the RCC clock provider node which root clocks the driver > depends on. These are root oscillators, which may be present or > not, upon FDT content. > > This update binding is introduced in Linux kerne

Re: [PATCH u-boot 16/39] build: use thin archives instead of incremental linking

2021-03-09 Thread Bin Meng
On Sun, Mar 7, 2021 at 12:26 PM Marek Behún wrote: > > Currently we use incremental linking (ld -r) to link several object > files from one directory into one built-in.o object file containing the > linked code from that directory (and its subdirectories). > > Linux has, some time ago, moved to th

Re: [PATCH 2/6] clk: stm32mp1: gets root clocks from fdt

2021-03-09 Thread Patrice CHOTARD
Hi Patrick On 2/24/21 11:19 AM, Patrick Delaunay wrote: > From: Etienne Carriere > > This change makes stm32mp1 clock driver to get the root clocks > reference from the device node in the FDT rather than fetching > straight these clocks by their name. Driver now stores the > clock reference and

Re: [Uboot-stm32] [PATCH 3/6] scmi: Include device_compat.h

2021-03-09 Thread Patrice CHOTARD
Hi Patrick On 2/24/21 11:19 AM, Patrick Delaunay wrote: > Include the file needed for log function prototype, this patch solves the > compilation issue for undefined reference to `dev_err'. > > Signed-off-by: Patrick Delaunay > --- > > drivers/firmware/scmi/smccc_agent.c | 1 + > 1 file change

Re: [Uboot-stm32] [PATCH 4/6] scmi: define LOG_CATEGORY

2021-03-09 Thread Patrice CHOTARD
Hi Patrick On 2/24/21 11:19 AM, Patrick Delaunay wrote: > Define LOG_CATEGORY to allow filtering with log command. > > Signed-off-by: Patrick Delaunay > --- > > drivers/firmware/scmi/mailbox_agent.c| 2 ++ > drivers/firmware/scmi/sandbox-scmi_agent.c | 2 ++ > drivers/firmware/scmi/s

Re: [Uboot-stm32] [PATCH 3/6] scmi: Include device_compat.h

2021-03-09 Thread Patrice CHOTARD
Hi Patrick On 2/24/21 11:19 AM, Patrick Delaunay wrote: > Include the file needed for log function prototype, this patch solves the > compilation issue for undefined reference to `dev_err'. > > Signed-off-by: Patrick Delaunay > --- > > drivers/firmware/scmi/smccc_agent.c | 1 + > 1 file change

Re: [Uboot-stm32] [PATCH 5/6] scmi: cosmetic: reorder include files

2021-03-09 Thread Patrice CHOTARD
Hi Patrick On 2/24/21 11:19 AM, Patrick Delaunay wrote: > Reorder include files in expected order. > > Signed-off-by: Patrick Delaunay > --- > > drivers/firmware/scmi/mailbox_agent.c | 2 +- > drivers/firmware/scmi/scmi_agent-uclass.c | 3 +-- > 2 files changed, 2 insertions(+), 3 deletion

Re: [PATCH 6/6] configs: stm32mp1_trusted_defconfig rely on SCMI support

2021-03-09 Thread Patrice CHOTARD
Hi Patrick On 2/24/21 11:19 AM, Patrick Delaunay wrote: > Enable SCMI clock and reset domain support for stm32mp1 platform > and ARM SMC mailbox driver used as communication channel for > SCMI messages between non-secure world and secure SCMI server. > > Signed-off-by: Patrick Delaunay > --- >

Re: [PATCH] board: st: remove the nand MTD configuration for NOR boot in stm32mp1 board

2021-03-09 Thread Patrice CHOTARD
Hi Patrick On 2/25/21 11:49 AM, Patrick Delaunay wrote: > Since commit d5d726d3cc47 ("configs: stm32mp1: only support SD card after > NOR in bootcmd_stm32mp"), the stm32mp1 boards only support SD card after > NOR boot device, so the MTD partitions for nand0 or spi-nand0 are useless > (no need of "

Re: [PATCH 1/4] stm32mp: stm32prog: Add Kconfig file for stm32prog command

2021-03-09 Thread Patrice CHOTARD
Hi Patrick On 2/25/21 1:37 PM, Patrick Delaunay wrote: > Move CONFIG_CMD_STM32PROG in a specific Kconfig file for stm32prog command. > > Signed-off-by: Patrick Delaunay > --- > > arch/arm/mach-stm32mp/Kconfig | 18 +- > arch/arm/mach-stm32mp/cmd_stm32prog/Kconfig

Re: [PATCH 2/4] stm32mp: stm32prog: Add CONFIG_CMD_STM32PROG_SERIAL and _USB

2021-03-09 Thread Patrice CHOTARD
Hi Patrick On 2/25/21 1:37 PM, Patrick Delaunay wrote: > Add CONFIG_CMD_STM32PROG_SERIAL and CONFIG_CMD_STM32PROG_USB to > independently select the support of UART or USB communication for > STM32CubeProgrammer. > > For serial boot over UART, user can deactivate CONFIG_CMD_STM32PROG_SERIAL > to u

Re: [PATCH 3/4] stm32mp: stm32prog: reactivate console and display serial error

2021-03-09 Thread Patrice CHOTARD
Hi Patrick On 2/25/21 1:37 PM, Patrick Delaunay wrote: > When serial instance is not found in device tree, the console > should be enabled and the error should be indicated. > > Signed-off-by: Patrick Delaunay > --- > > arch/arm/mach-stm32mp/cpu.c | 14 ++ > 1 file changed, 10 inse

Re: [PATCH 4/4] stm32mp: stm32prog: replace alias by serial device sequence number

2021-03-09 Thread Patrice CHOTARD
Hi Patrick On 2/25/21 1:37 PM, Patrick Delaunay wrote: > The command "stm32prog serial " can directly use the device sequence > number of serial uclass as this sequence number is egual to alias when it > exist; this assumption simplify the code and avoid access to gd->fdt_blob > and the device tre

[PATCH v2 1/4] watchdog: Set/unset GD_FLG_WDT_READY flag in wdt_start()/wdt_stop()

2021-03-09 Thread Pali Rohár
Watchdog is ready after successful call of ops->start() callback in wdt_start() function. And is stopped after successful call of ops->stop() callback in wdt_stop function. So move setting of GD_FLG_WDT_READY flag from initr_watchdog() function to wdt_start() and ensure that GD_FLG_WDT_READY flag

Re: [PATCH] stm32mp: bsec: manage clock when present in device tree

2021-03-09 Thread Patrice CHOTARD
Hi Patrick On 2/25/21 1:43 PM, Patrick Delaunay wrote: > Enable the clocks during bsec probe when they are present in device tree. > > Signed-off-by: Patrick Delaunay > --- > > arch/arm/mach-stm32mp/bsec.c | 10 ++ > 1 file changed, 10 insertions(+) > > diff --git a/arch/arm/mach-stm3

[PATCH v2 4/4] arm: mvebu: Espressobin: Enable watchdog support but do not start it

2021-03-09 Thread Pali Rohár
Espressobin had disabled watchdog support (CONFIG_WDT) because older stable Linux kernel versions (which are used by current stable OpenWRT and Debian versions) do not have support for Armada 3700 watchdog driver. Therefore they are not able to periodically kick watchdog so Espressobin enter into b

[PATCH v2 2/4] watchdog: Show error message when initr_watchdog() cannot start watchdog

2021-03-09 Thread Pali Rohár
Function wdt_start() may fail. So in initr_watchdog() function check return value of wdt_start() call and print error message when watchdog starting failed. Signed-off-by: Pali Rohár --- drivers/watchdog/wdt-uclass.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/driv

[PATCH v2 3/4] watchdog: Allow to use CONFIG_WDT without starting watchdog

2021-03-09 Thread Pali Rohár
In some cases it is useful to compile support for U-Boot command 'wdt' without starting HW watchdog in early U-Boot phase. For example when the user want to start the watchdog only on demand by some boot script. This change adds a new compile option WATCHDOG_AUTOSTART to control whether U-Boot sho

Re: [PATCH 3/4] watchdog: Allow to use CONFIG_WDT without starting watchdog

2021-03-09 Thread Pali Rohár
On Tuesday 09 March 2021 12:33:24 Stefan Roese wrote: > On 05.03.21 22:36, Pali Rohár wrote: > > In some cases it is useful to compile support for U-Boot command 'wdt' > > without starting HW watchdog in early U-Boot phase. For example when user > > Nitpicking: > when "the" user > > > want to s

Re: [PATCH u-boot 17/39] build: support building with Link Time Optimizations

2021-03-09 Thread Bin Meng
On Sun, Mar 7, 2021 at 12:26 PM Marek Behún wrote: > > Add plumbing for building U-Boot with Link Time Optimizations. > > Signed-off-by: Marek Behún > --- > Kbuild | 2 ++ > Kconfig | 19 +++ > Makefile | 36

Re: [PATCH 1/9] ARM: dts: stm32: Add Engicam i.Core STM32MP1 SoM

2021-03-09 Thread Patrice CHOTARD
Hi Jagan On 2/28/21 4:52 PM, Jagan Teki wrote: > i.Core STM32MP1 is an EDIMM SoM based on STM32MP157A from Engicam. > > General features: > - STM32MP157A > - Up to 1GB DDR3L > - 4GB eMMC > - 10/100 Ethernet > - USB 2.0 Host/OTG > - I2S > - MIPI DSI to LVDS > - rest of STM32MP157A features > > i.

Re: [PATCH 2/9] ARM: dts: stm32: Add Engicam i.Core STM32MP1 1X4Gb DDR3

2021-03-09 Thread Patrice CHOTARD
Hi Jagan On 2/28/21 4:52 PM, Jagan Teki wrote: > Engicam i.Core STM32MP1 SODIMM SoM has mounted 1x4Gb DDR3 > which has 32bits width 528000Khz frequency. > > Add DDR configuration via dtsi. > > Signed-off-by: Jagan Teki > --- > .../stm32mp15-ddr3-icore-1x4Gb-1066-binG.dtsi | 119 +++

[PATCH] rtc: add support for rv3028 rtc

2021-03-09 Thread Heiko Schocher
add support for rtc3028 rtc from microcrystal. based on linux dirver: commit a38fd8748464: ("Linux 5.12-rc2") Signed-off-by: Heiko Schocher --- driver is based on code in linux, but with already corrected weekday usage. linux codes the weekday bitwise, while the weekday register has only 3 valid

Re: [PATCH 3/9] ARM: stm32: Imply SPL_SPI_LOAD

2021-03-09 Thread Patrice CHOTARD
Hi Jagan On 2/28/21 4:52 PM, Jagan Teki wrote: > SPI Load isn't mandatory for STM32 builds. > > Let's imply instead of select it to get rid of build > issues for non-SPI defconfigs. > > Signed-off-by: Jagan Teki > --- > arch/arm/mach-stm32mp/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1

Re: [PATCH u-boot-marvell 02/39] ddr: marvell: axp: fix array types have different bounds warning

2021-03-09 Thread Tom Rini
On Mon, Mar 08, 2021 at 07:50:59AM +0100, Stefan Roese wrote: > On 08.03.21 07:45, Marek Behun wrote: > > > Reviewed-by: Stefan Roese > > > > Thanks, Stefan. > > Do you want to merge this into your repo u-boot-marvell, or shall Tom > > merge this once this series is mature? > > AFAIU, your patch

Re: [PATCH 4/9] board: stm32: Add Engicam i.Core STM32MP1 EDIMM2.2 Starter Kit

2021-03-09 Thread Patrice CHOTARD
Hi Jagan one remark below On 2/28/21 4:52 PM, Jagan Teki wrote: > Engicam EDIMM2.2 Starter Kit is an EDIMM 2.2 Form Factor Capacitive > Evaluation Board. > > Genaral features: > - LCD 7" C.Touch > - microSD slot > - Ethernet 1Gb > - Wifi/BT > - 2x LVDS Full HD interfaces > - 3x USB 2.0 > - 1x US

Re: [PATCH 5/9] board: stm32: Add Engicam i.Core STM32MP1 C.TOUCH 2.0

2021-03-09 Thread Patrice CHOTARD
Hi Jagan On 2/28/21 4:52 PM, Jagan Teki wrote: > Engicam C.TOUCH 2.0 is an EDIMM compliant general purpose Carrier > board. > > Genaral features: > - Ethernet 10/100 > - Wifi/BT > - USB Type A/OTG > - Audio Out > - CAN > - LVDS panel connector > > i.Core STM32MP1 is an EDIMM SoM based on STM32MP

Re: [PATCH 6/9] ARM: dts: stm32: Add Engicam MicroGEA STM32MP1 Micro SoM

2021-03-09 Thread Patrice CHOTARD
Hi Jagan On 2/28/21 4:52 PM, Jagan Teki wrote: > MicroGEA STM32MP1 is a STM32MP157A based Micro SoM. > > General features: > - STM32MP157AAC > - Up to 1GB DDR3L-800 > - 512MB Nand flash > - I2S > > MicroGEA STM32MP1 needs to mount on top of Engicam MicroDev carrier > boards for creating complete

Re: [PATCH 7/9] ARM: dts: stm32: Add Engicam MicroGEA STM32MP1 1X4Gb DDR3

2021-03-09 Thread Patrice CHOTARD
Hi Jagan On 2/28/21 4:52 PM, Jagan Teki wrote: > Engicam MicroGEA STM32MP1 Micro SOM has mounted 1x4Gb DDR3 > which has 16bits width 533Mhz frequency. > > Add DDR configurations via dtsi. > > Signed-off-by: Jagan Teki > --- > ...m32mp15-ddr3-microgea-1x4Gb-1066-binG.dtsi | 121

Re: [PATCH 8/9] board: stm32: Add Engicam MicroGEA STM32MP1 MicroDev 2.0 board

2021-03-09 Thread Patrice CHOTARD
Hi Jagan one minor remard below On 2/28/21 4:52 PM, Jagan Teki wrote: > MicroDev 2.0 is a general purpose miniature carrier board with CAN, > LTE and LVDS panel interfaces. > > Genaral features: > - Ethernet 10/100 > - USB Type A > - Audio Out > - microSD > - LVDS panel connector > - Wifi/BT (op

Re: [PATCH 9/9] board: stm32: Add Engicam MicroGEA STM32MP1 MicroDev 2.0 7" OF

2021-03-09 Thread Patrice CHOTARD
Hi Jagan On 2/28/21 4:52 PM, Jagan Teki wrote: > 7" OF is a capacitive touch 7" Open Frame panel solutions with > - 7" AUO B101AW03 LVDS panel > - EDT, FT5526 Touch > > MicroGEA STM32MP1 is a STM32MP157A based Micro SoM. > > MicroDev 2.0 is a general purpose miniature carrier board with CAN, > L

Re: [PATCH 9/9] board: stm32: Add Engicam MicroGEA STM32MP1 MicroDev 2.0 7" OF

2021-03-09 Thread Patrice CHOTARD
Hi Jagan On 2/28/21 4:52 PM, Jagan Teki wrote: > 7" OF is a capacitive touch 7" Open Frame panel solutions with > - 7" AUO B101AW03 LVDS panel > - EDT, FT5526 Touch > > MicroGEA STM32MP1 is a STM32MP157A based Micro SoM. > > MicroDev 2.0 is a general purpose miniature carrier board with CAN, > L

Re: [PATCH] board: stm32mp1: use CONFIG_SYS_MMC_ENV_DEV when available

2021-03-09 Thread Patrice CHOTARD
Hi Patrick On 3/1/21 1:17 PM, Patrick Delaunay wrote: > Check whether user has explicitly defined the mmc device to use > in mmc_get_env_dev() with CONFIG_SYS_MMC_ENV_DEV. > > On STMicroelectronics boards the used mmc device for environment is > the instance of boot device provided by the ROM cod

Re: [PATCH] video: stm32: remove all child of DSI bridge when its probe failed

2021-03-09 Thread Patrice CHOTARD
Hi Patrick On 3/4/21 1:10 PM, Patrick Delaunay wrote: > From: Patrick Delaunay > > Remove the child device of the STM32 DSI bridge when the driver probe > failed to stop futher probe request on panels used with STMicroelectronics > board (orisetech_otm8009a.c or raydium-rm68200.c driver). > > T

Re: [PATCH] configs: stm32mp15: move bootdelay configuration in defconfig

2021-03-09 Thread Patrice CHOTARD
Hi Patrick On 3/1/21 7:40 PM, Patrick Delaunay wrote: > The STM32MP15 boards have no reason to configure bootdelay in stm32mp1.h > as it is already done with CONFIG_BOOTDELAY (default = 2) and in > include/env_default.h. > > This patch allows configuration for customers which reuse stm32mp1.h > a

Re: [Uboot-stm32] [PATCH 1/2] video: dw_mipi_dsi: missing device to log debug

2021-03-09 Thread Patrice CHOTARD
Hi Patrick On 3/4/21 1:14 PM, Patrick Delaunay wrote: > From: Yannick Fertre > > Missing udevice to struct dw_mipi_dsi to log trace. > > Signed-off-by: Yannick Fertre > Signed-off-by: Patrick Delaunay > --- > > drivers/video/dw_mipi_dsi.c | 1 + > 1 file changed, 1 insertion(+) > > diff --

Re: [Uboot-stm32] [PATCH 2/2] video: dw_mipi_dsi: update log of dphy_enable

2021-03-09 Thread Patrice CHOTARD
HI Patrick On 3/4/21 1:14 PM, Patrick Delaunay wrote: > From: Yannick Fertre > > The DSI phy can be turned on from the DSI digital interface in > the dphy_enable() function or from a dedicated DSI phy "wrapper" > in phy_ops->init() function. If the STM32MP1 case, the wrapper > is used then the d

Re: [PATCH] arm: stm32mp1: Set soc_type, soc_pkg, soc_rev env variables

2021-03-09 Thread Patrice CHOTARD
Hi Marek On 3/5/21 3:18 PM, Marek Vasut wrote: > Split up get_soc_name(), clean the decoding up a bit, and set up > environment variables which contain the SoC type, package, revision. > This is useful on SoMs, where multiple SoC options are populated. > > Signed-off-by: Marek Vasut > Cc: Patric

Re: [PATCH] doc: stm32mp1: Use u-boot.itb if CONFIG_SPL_LOAD_FIT=y

2021-03-09 Thread Patrice CHOTARD
Hi Marek On 3/6/21 9:44 PM, Marek Vasut wrote: > For systems where SPL loads fitImage, i.e. CONFIG_SPL_LOAD_FIT=y, use > u-boot.itb in the relevant documentation parts. Otherwise use u-boot.img. > > Signed-off-by: Marek Vasut > Cc: Patrice Chotard > Cc: Patrick Delaunay > --- > doc/board/st/s

Re: [PATCH u-boot 18/39] build: LTO: move platform libs into --start-group list

2021-03-09 Thread Bin Meng
On Sun, Mar 7, 2021 at 12:26 PM Marek Behún wrote: > > When building with LTO, move $(PLATFORM_LIBS) into the --start-group / > --end-group list. This should be --whole-archive > Otherwise some functions declared in assembly may not be resolved and > linking may fail. I am not sure if this is c

Re: [PATCH u-boot 19/39] sandbox: errno: avoid conflict with libc's errno

2021-03-09 Thread Bin Meng
On Sun, Mar 7, 2021 at 12:26 PM Marek Behún wrote: > > When building with LTO, the system libc's `errno` variable used in > arch/sandbox/cpu/os.c conflicts with U-Boot's `errno` (defined in > lib/errno.c) with the following error: > .../ld: errno@@GLIBC_PRIVATE: TLS definition in /lib64/libc.so.6

Re: [PATCH] rtc: add support for rv3028 rtc

2021-03-09 Thread Stefan Roese
Hi Heiko, On 09.03.21 14:45, Heiko Schocher wrote: add support for rtc3028 rtc from microcrystal. based on linux dirver: commit a38fd8748464: ("Linux 5.12-rc2") Nitpicking: You might want to start a sentence in upper-case? ;) Another minor comment below... Signed-off-by: Heiko Schocher --

Re: [PATCH 3/4] watchdog: Allow to use CONFIG_WDT without starting watchdog

2021-03-09 Thread Stefan Roese
On 09.03.21 14:27, Pali Rohár wrote: On Tuesday 09 March 2021 12:33:24 Stefan Roese wrote: On 05.03.21 22:36, Pali Rohár wrote: In some cases it is useful to compile support for U-Boot command 'wdt' without starting HW watchdog in early U-Boot phase. For example when user Nitpicking: when

Re: [PATCH v2 1/4] watchdog: Set/unset GD_FLG_WDT_READY flag in wdt_start()/wdt_stop()

2021-03-09 Thread Stefan Roese
On 09.03.21 14:26, Pali Rohár wrote: Watchdog is ready after successful call of ops->start() callback in wdt_start() function. And is stopped after successful call of ops->stop() callback in wdt_stop function. So move setting of GD_FLG_WDT_READY flag from initr_watchdog() function to wdt_start()

Re: [PATCH v2 2/4] watchdog: Show error message when initr_watchdog() cannot start watchdog

2021-03-09 Thread Stefan Roese
On 09.03.21 14:26, Pali Rohár wrote: Function wdt_start() may fail. So in initr_watchdog() function check return value of wdt_start() call and print error message when watchdog starting failed. Signed-off-by: Pali Rohár Reviewed-by: Stefan Roese Thanks, Stefan --- drivers/watchdog/wdt-u

Re: [PATCH v2 3/4] watchdog: Allow to use CONFIG_WDT without starting watchdog

2021-03-09 Thread Stefan Roese
On 09.03.21 14:26, Pali Rohár wrote: In some cases it is useful to compile support for U-Boot command 'wdt' without starting HW watchdog in early U-Boot phase. For example when the user want to start the watchdog only on demand by some boot script. This change adds a new compile option WATCHDOG_

Re: [PATCH v2 4/4] arm: mvebu: Espressobin: Enable watchdog support but do not start it

2021-03-09 Thread Stefan Roese
On 09.03.21 14:26, Pali Rohár wrote: Espressobin had disabled watchdog support (CONFIG_WDT) because older stable Linux kernel versions (which are used by current stable OpenWRT and Debian versions) do not have support for Armada 3700 watchdog driver. Therefore they are not able to periodically ki

Re: [PATCH 3/4] watchdog: Allow to use CONFIG_WDT without starting watchdog

2021-03-09 Thread Pali Rohár
On Tuesday 09 March 2021 17:12:40 Stefan Roese wrote: > On 09.03.21 14:27, Pali Rohár wrote: > > On Tuesday 09 March 2021 12:33:24 Stefan Roese wrote: > > > On 05.03.21 22:36, Pali Rohár wrote: > > > > In some cases it is useful to compile support for U-Boot command 'wdt' > > > > without starting H

Re: [PATCH] rtc: add support for rv3028 rtc

2021-03-09 Thread Alexandre Belloni
On 09/03/2021 17:10:21+0100, Stefan Roese wrote: > Hi Heiko, > > On 09.03.21 14:45, Heiko Schocher wrote: > > add support for rtc3028 rtc from microcrystal. > > based on linux dirver: > > commit a38fd8748464: ("Linux 5.12-rc2") > > Nitpicking: You might want to start a sentence in upper-case? ;)

Re: [PATCH] disk: gpt: verify alternate LBA points to last usable LBA

2021-03-09 Thread Stefan Herbrechtsmeier
Hi Heinrich, Am 08.03.2021 um 18:38 schrieb Heinrich Schuchardt: On 08.03.21 17:07, Stefan Herbrechtsmeier wrote: From: Stefan Herbrechtsmeier The gpt command require the GPT backup header at the standard location at the end of the device.Check the alternate LBA value before reading the GPT b

Re: [PATCH] disk: gpt: verify alternate LBA points to last usable LBA

2021-03-09 Thread Heinrich Schuchardt
On 09.03.21 17:24, Stefan Herbrechtsmeier wrote: > Hi Heinrich, > > Am 08.03.2021 um 18:38 schrieb Heinrich Schuchardt: >> On 08.03.21 17:07, Stefan Herbrechtsmeier wrote: >>> From: Stefan Herbrechtsmeier >>> >>> The gpt command require the GPT backup header at the standard location >>> at the end

[PATCH 2/2] configs: am65x_evm_r5: Enable checks for spl and stack sizes

2021-03-09 Thread Lokesh Vutla
Enable relevant configs that checks for the size of image and stack: BSS: 3KB Initial MALLOC: ~22KB Initial Stack: 8K SPL Image size can be: ~215KB Signed-off-by: Lokesh Vutla --- configs/am65x_evm_r5_defconfig | 5 + 1 file changed, 5 insertions(+) diff --git a/configs/am65x_evm_r5_de

[PATCH 1/2] include: configs: am65x_evm: Optimize size of SPL BSS

2021-03-09 Thread Lokesh Vutla
Current BSS allocation of SPL is as below: $ size spl/u-boot-spl textdata bss dec hex filename 13236978521496 141717 22995 spl/u-boot-spl But 20KB is allocated currently for BSS. Reduce it to 3KB and save some space for stack. Signed-off-by: Lokesh Vutla --- inclu

[PATCH v2] Nokia RX-51: Enable CONFIG_WDT to remove deprecation warning

2021-03-09 Thread Pali Rohár
Also convert CONFIG_HW_WATCHDOG to CONFIG_WATCHDOG. Signed-off-by: Pali Rohár --- This patch increase u-boot.bin binary size above maximal limit. So this patch cannot be applied yet. But it can be applied on top of the LTO patches. So please put this patch into the queue and include it into U-Boo

[PATCH] cmd: fs: Use part_get_info_by_dev_and_name_or_num to parse partitions

2021-03-09 Thread Sean Anderson
This allows using dev#partlabel syntax. Signed-off-by: Sean Anderson --- fs/fs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/fs.c b/fs/fs.c index 900928c394..b7936fd4cf 100644 --- a/fs/fs.c +++ b/fs/fs.c @@ -385,8 +385,8 @@ int fs_set_blk_dev(const char *ifname, c

Re: [PATCH] cmd: fs: Use part_get_info_by_dev_and_name_or_num to parse partitions

2021-03-09 Thread Heinrich Schuchardt
Am 9. März 2021 21:56:24 MEZ schrieb Sean Anderson : >This allows using dev#partlabel syntax. Allowing more widespread use of that syntax makes sense to me. Unfortunately you do not mention which commands and devices are affected. > >Signed-off-by: Sean Anderson >--- > > fs/fs.c | 4 ++-- > 1 fi

Re: [PATCH] cmd: fs: Use part_get_info_by_dev_and_name_or_num to parse partitions

2021-03-09 Thread Sean Anderson
On 3/9/21 4:21 PM, Heinrich Schuchardt wrote: > Am 9. März 2021 21:56:24 MEZ schrieb Sean Anderson : >> This allows using dev#partlabel syntax. > > Allowing more widespread use of that syntax makes sense to me. > > Unfortunately you do not mention which commands and devices are affected. I ena

Re: [PATCH v4 1/5] firmware: scmi: voltage regulator

2021-03-09 Thread Jaehoon Chung
On 3/9/21 6:38 AM, Etienne Carriere wrote: > Implement voltage regulators interfaced by the SCMI voltage domain > protocol. The DT bindings are defined in the Linux kernel since > SCMI voltage domain and regulators patches [1] and [2] integration > in v5.11-rc7. > > Link: [1] > https://protect2.f

[PATCH v2] spl: Add callback for preprocessing loaded FIT header before parsing

2021-03-09 Thread Farhan Ali
This change adds a callback for preprocessing the FIT header before it is parsed. There are 3 main reasons for this callback: (1) If a vulnerability is discovered in the FIT parsing/loading code, or libfdt, this callback allows users to scan the FIT header for specific exploit signatures and preve

Re: [PATCH u-boot 37/39] ARM: don't use -ffunction-sections/-fdata-sections with LTO build

2021-03-09 Thread Bin Meng
On Sun, Mar 7, 2021 at 12:26 PM Marek Behún wrote: > > When building with LTO, using -ffunction-sections/-fdata-sections is not > useful anymore. > > Signed-off-by: Marek Behún > --- > arch/arm/config.mk | 8 ++-- > 1 file changed, 6 insertions(+), 2 deletions(-) > I believe we should also

Re: Please pull u-boot-x86

2021-03-09 Thread Tom Rini
On Tue, Mar 09, 2021 at 01:15:22PM +0800, Bin Meng wrote: > Hi Tom, > > This PR includes the following x86 changes for v2021.04: > > - Various minor fixes for x86 > > Azure results: PASS > https://dev.azure.com/bmeng/GitHub/_build/results?buildId=337&view=results > > The following changes sinc

Re: [PATCH] rtc: add support for rv3028 rtc

2021-03-09 Thread Heiko Schocher
Hi stefan, On 09.03.21 17:10, Stefan Roese wrote: > Hi Heiko, > > On 09.03.21 14:45, Heiko Schocher wrote: >> add support for rtc3028 rtc from microcrystal. >> based on linux dirver: >> commit a38fd8748464: ("Linux 5.12-rc2") > > Nitpicking: You might want to start a sentence in upper-case? ;)

Re: [PATCH u-boot 35/39] ata: ahci: fix ahci_link_up() type mismatch for LTO

2021-03-09 Thread Bin Meng
On Sun, Mar 7, 2021 at 12:26 PM Marek Behún wrote: > > When building highbank_defconfig with LTO, the compiler complains about > type mismatch of function ahci_link_up(). > > The third parameter of this function is of type u8 in > drivers/ata/ahci.c, but of type int in board/highbank/ahci.c. > > T

Re: [PATCH u-boot 32/39] ARM: fix LTO for rockchip and samsung

2021-03-09 Thread Bin Meng
On Sun, Mar 7, 2021 at 12:26 PM Marek Behún wrote: > > When building with LTO, the compiler complains about type mismatch of > function usb_gadget_handle_interrupts(). This function is defined > without parameters in files > arch/arm/mach-rockchip/board.c > board/samsung/common/exynos5-dt.c >

Re: [PATCH u-boot 30/39] ARM: imx6m: fix imx_eqos_txclk_set_rate() type mismatch for LTO

2021-03-09 Thread Bin Meng
On Sun, Mar 7, 2021 at 12:26 PM Marek Behún wrote: > > When building imx8mp_evk_defconfig with LTO, the compiler complains > about type mismatch of function imx_eqos_txclk_set_rate() in file > drivers/net/dwc_eth_qos.c:845:12 > which contains a weak definition of this function, vs file > arch/

Re: [PATCH u-boot 29/39] ARM: kona: fix clk_bsc_enable() type mismatch for LTO

2021-03-09 Thread Bin Meng
On Sun, Mar 7, 2021 at 12:26 PM Marek Behún wrote: > > When building with LTO, the compiler complains about type mismatch of > function clk_bsc_enable() in file: > arch/arm/cpu/armv7/kona-common/clk-stubs.c > vs other files that define or use this function: > warning: type of ‘clk_bsc_enable’

Re: [PATCH u-boot 26/39] ARM: fix LTO for imx28_xea

2021-03-09 Thread Bin Meng
On Sun, Mar 7, 2021 at 12:26 PM Marek Behún wrote: > > When imx28_xea_defconfig is built with LTO, the compiler complains about > the two different declarations of _start: >include/asm-generic/sections.has extern void _start(void); >arch/arm/cpu/arm926ejs/mxs/mxs.c as extern uint32_

Re: [PATCH u-boot 22/39] sandbox: enable LTO by default

2021-03-09 Thread Bin Meng
On Sun, Mar 7, 2021 at 12:26 PM Marek Behún wrote: > > Build sandbox targets with LTO by default. > > Signed-off-by: Marek Behún > --- > arch/Kconfig | 1 + > 1 file changed, 1 insertion(+) > Reviewed-by: Bin Meng

Re: [PATCH u-boot 21/39] sandbox: make LTO available

2021-03-09 Thread Bin Meng
On Sun, Mar 7, 2021 at 12:26 PM Marek Behún wrote: > > Make LTO available for sandbox architecture. > > Signed-off-by: Marek Behún > --- > arch/Kconfig | 1 + > arch/sandbox/config.mk | 2 ++ > 2 files changed, 3 insertions(+) > Reviewed-by: Bin Meng

[PATCH v2] rtc: add support for rv3028 rtc

2021-03-09 Thread Heiko Schocher
Add support for rtc3028 rtc from microcrystal. based on linux dirver: commit a38fd8748464: ("Linux 5.12-rc2") Signed-off-by: Heiko Schocher Reviewed-by: Stefan Roese --- Driver is based on code in linux, but with already corrected weekday usage. Linux codes the weekday bitwise, while the weekda

Re: [IMPORTANT] gitlab relocation / rename

2021-03-09 Thread Wolfgang Denk
Dear Henning, In message <20210308083457.76db0...@md1za8fc.ad001.siemens.net> you wrote: > > I do not understand why this is not just gitlab (proper) or github. I > hope this discussion was held with the communities of the affected > projects involved. While being disruptive i would suggest for Xe

  1   2   >