Re: [PATCH v7 00/14] mtd: spi-nor: Add support for Cypress s25hl-t/s25hs-t

2021-04-29 Thread Takahiro Kuwano
On 4/28/2021 3:44 PM, Takahiro Kuwano wrote: > On 4/27/2021 7:14 PM, Jagan Teki wrote: >> On Tue, Apr 27, 2021 at 7:54 AM wrote: >>> >>> From: Takahiro Kuwano >>> >>> The S25HL-T/S25HS-T family is the Cypress Semper Flash with Quad SPI. >>> >>> The summary datasheets can be found in the following

Re: [PATCH v3 5/7] image-fdt: save no-map parameter of reserve-memory

2021-04-29 Thread Ard Biesheuvel
On Thu, 29 Apr 2021 at 18:11, Simon Glass wrote: > > Hi Patrick, > > On Wed, 28 Apr 2021 at 03:23, Patrick Delaunay > wrote: > > > > Save the no-map information present in 'reserved-memory' node to allow > > correct handling when the MMU is configured in board to avoid > > speculative access. > >

Re: [PATCH v1 08/10] net: mvpp2: fix missing switch case break

2021-04-29 Thread Stefan Roese
On 29.04.21 22:24, Ramon Fried wrote: On Tue, Apr 27, 2021 at 4:28 PM Stefan Roese wrote: From: Ben Peled Signed-off-by: Ben Peled Reviewed-by: Stefan Chulski Reviewed-by: Kostya Porotchkin Tested-by: sa_ip-sw-jenkins Please remove this, Jenkins is not a person. I've left these lines

Re: [PATCH v2 04/13] gpio: Introduce CONFIG_ONLY_GENERIC_GPIO to cleanup #ifdefs

2021-04-29 Thread Masami Hiramatsu
Hi Simon, 2021年4月30日(金) 1:10 Simon Glass : > > Hi Masami, > > On Fri, 16 Apr 2021 at 16:38, Masami Hiramatsu > wrote: > > > > Many architecture do not have specific asm/arch/gpio.h, so instead > > of adding !defined(CONFIG_ARCH_xxx), introduce CONFIG_ONLY_GENERIC_GPIO > > This seems OK, but I thi

Re: [PATCH v3 5/7] image-fdt: save no-map parameter of reserve-memory

2021-04-29 Thread Bin Meng
Hi Simon, On Fri, Apr 30, 2021 at 12:13 AM Simon Glass wrote: > > Hi Patrick, > > On Wed, 28 Apr 2021 at 03:23, Patrick Delaunay > wrote: > > > > Save the no-map information present in 'reserved-memory' node to allow > > correct handling when the MMU is configured in board to avoid > > speculati

Please pull u-boot-dm

2021-04-29 Thread Simon Glass
Hi Tom, https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/7344 The following changes since commit 79b0f08d6af498e6fda8cd257d62e2095764410c: configs: Resync with savedefconfig (2021-04-27 08:28:38 -0400) are available in the Git repository at: g...@source.denx.de:u-boot/custod

Re: [PATCH] Makefile: Fix generation of flash.bin u-boot.itb with binman

2021-04-29 Thread Bin Meng
On Sat, Apr 24, 2021 at 8:57 PM Marek Vasut wrote: > > On 2/25/21 9:50 PM, Marek Vasut wrote: > > In case binman is enabled, the u-boot.itb is generated using this tool > > and there is no direct u-boot.itb target, but instead the binman tool > > must be invoked. Add support for this case. > > > >

Re: [PATCH 03/17] x86: Allow coreboot serial driver to guess the UART

2021-04-29 Thread Bin Meng
Hi Simon, On Fri, Apr 30, 2021 at 12:10 AM Simon Glass wrote: > > Hi Bin, > > On Sun, 25 Apr 2021 at 18:21, Bin Meng wrote: > > > > Hi Simon, > > > > On Sun, Apr 25, 2021 at 10:10 AM Simon Glass wrote: > > > > > > Hi Bin, > > > > > > On Sun, 25 Apr 2021 at 13:49, Bin Meng wrote: > > > > > > >

Re: Qemu U-boot Standalone Application Crash

2021-04-29 Thread Bin Meng
Hi, On Fri, Apr 30, 2021 at 1:20 AM prashant basva wrote: > > Hi, > i am trying to execute standalone application from qemu u-boot, but when i > trued to execute using go or bootm , its getting crashed saying trap > Below is command used to launch qemu ppc u-boot > $ sudo qemu-system-ppc -m 1024

Re: [PATCH] net: phy: micrel: Get phy node from phy-handle

2021-04-29 Thread Ramon Fried
On Mon, Apr 26, 2021 at 6:43 AM Ley Foon Tan wrote: > > If can't find ethernet-phy subnode, try to get phy node from "phy-handle". > Lastly, only use Ethernet node if can't find phy node from ethernet-phy > subnode and phy-handle. > > Signed-off-by: Ley Foon Tan > --- > drivers/net/phy/micrel_ks

Re: [PATCH v1 09/10] net: mvpp2: allow MDIO registration for fixed links

2021-04-29 Thread Ramon Fried
On Tue, Apr 27, 2021 at 4:29 PM Stefan Roese wrote: > > From: Stefan Chulski > > Currently, there are 2 valid cases for interface, PHY > and mdio relation: > - If an interface has PHY handler, it'll call > mdio_mii_bus_get_from_phy(), which will register > MDIO bus. > - If we want to

Re: [PATCH v1 10/10] net: mvpp2: add explicit sgmii-2500 support

2021-04-29 Thread Ramon Fried
On Tue, Apr 27, 2021 at 4:29 PM Stefan Roese wrote: > > From: Marcin Wojtas > > Until now the mvpp2 driver used an extra 'phy-speed' > DT property in order to differentiate between the > SGMII and SGMII @2.5GHz. As there is a dedicated > PHY_INTERFACE_MODE_SGMII_2500 flag to mark the latter > sta

Re: [PATCH v1 08/10] net: mvpp2: fix missing switch case break

2021-04-29 Thread Ramon Fried
On Tue, Apr 27, 2021 at 4:28 PM Stefan Roese wrote: > > From: Ben Peled > > Signed-off-by: Ben Peled > Reviewed-by: Stefan Chulski > Reviewed-by: Kostya Porotchkin > Tested-by: sa_ip-sw-jenkins Please remove this, Jenkins is not a person. > Signed-off-by: Stefan Roese > --- > > drivers/net/

Re: [PATCH v1 04/10] net: mvpp2: remove redundant SMI address configuration

2021-04-29 Thread Ramon Fried
On Tue, Apr 27, 2021 at 4:28 PM Stefan Roese wrote: > > From: Marcin Wojtas > > Because the mvpp2 driver now relies on the PHYLIB and > the external MDIO driver, configuring low level > SMI bus settings is redundant. > > Signed-off-by: Marcin Wojtas > Tested-by: sa_ip-sw-jenkins > Reviewed-by:

Re: [PATCH v1 03/10] net: mvpp2: add 1000BaseX and 2500BaseX ppv2 support

2021-04-29 Thread Ramon Fried
On Tue, Apr 27, 2021 at 4:28 PM Stefan Roese wrote: > > From: Stefan Chulski > > Signed-off-by: Stefan Chulski > Signed-off-by: Stefan Roese > --- > > drivers/net/mvpp2.c | 117 ++-- > 1 file changed, 112 insertions(+), 5 deletions(-) > > diff --git a/dr

Re: [PATCH v1 02/10] net: mvpp2: add CP115 port1 10G/5G SFI support

2021-04-29 Thread Ramon Fried
On Tue, Apr 27, 2021 at 4:27 PM Stefan Roese wrote: > > From: Stefan Chulski > > 1. Differ between Port1 RGMII and SFI modes in Netcomplex config. > 2. Remove XPCS config from SFI mode. >Port1 doesn't XPCS domain, XPCS config should be removed. >Access to Port1 XPCS can cause stall. > 3.

Re: [PATCH 4/4] net: mvpp2: Remove PHY_INTERFACE_MODE_SGMII_2500

2021-04-29 Thread Ramon Fried
On Tue, Apr 27, 2021 at 12:49 PM Stefan Roese wrote: > > As was discussed on the list, PHY_INTERFACE_MODE_SGMII_2500 is used > incorrectly in the Marvell mvpp2 network driver and the Marvell PHY > code. This patch removes the references to this macro in the mvpp2 > network driver for now. > > The

Re: [PATCH 3/4] net: phy: marvell: Remove PHY_INTERFACE_MODE_SGMII_2500

2021-04-29 Thread Ramon Fried
On Tue, Apr 27, 2021 at 12:49 PM Stefan Roese wrote: > > As was discussed on the list, PHY_INTERFACE_MODE_SGMII_2500 is used > incorrectly in the Marvell mvpp2 network driver and the Marvell PHY > code. This patch removes the references to this macro in the Marvell > PHY driver for now. > > The co

Re: [PATCH v2 6/6] net: dwc: add a common empty ops eqos_null_ops

2021-04-29 Thread Ramon Fried
On Mon, Apr 26, 2021 at 6:47 PM Patrick Delaunay wrote: > > Add a common empty ops: eqos_null_ops() to remove the duplicated empty > functions and reduce the driver size for stm32 and imx config. > > This patch also aligns the prototype of ops 'eqos_stop_clks' with other > eqos ops by adding retur

Re: [PATCH v2 5/6] net: dwc_eth_qos: use generic ethernet phy for stm32 variant

2021-04-29 Thread Ramon Fried
On Mon, Apr 26, 2021 at 6:47 PM Patrick Delaunay wrote: > > Use the generic ethernet phy which already manages the correct binding > for gpio reset, including the assert an deassert delays. > > Signed-off-by: Patrick Delaunay > --- > > Changes in v2: > - use generic ethernet phy for stm32 variant

Re: [PATCH v2 4/6] net: dwc_eth_qos: remove the field phyaddr of the struct eqos_priv

2021-04-29 Thread Ramon Fried
On Mon, Apr 26, 2021 at 6:47 PM Patrick Delaunay wrote: > > Since the commit commit 6a895d039ba7 ("net: Update eQos driver and FEC > driver to use eth phy interfaces") the field phyaddr of driver private data > struct eqos_priv is no more used in eqos_start() for the phy_connect() > parameter. > >

Re: [PATCH v2 3/6] net: eth-phy: manage subnode mdio0

2021-04-29 Thread Ramon Fried
On Mon, Apr 26, 2021 at 6:47 PM Patrick Delaunay wrote: > > Bind any subnode with name beginning by mdio, mdio0 for example, > and not only the "mdio" as namei of subnode. > > Signed-off-by: Patrick Delaunay > --- > > Changes in v2: > - update eth-phy driver to support STM32 binding with a mdio0

Re: [PATCH v2 2/6] net: eth-phy: use dev_dbg and log_notice

2021-04-29 Thread Ramon Fried
On Mon, Apr 26, 2021 at 6:47 PM Patrick Delaunay wrote: > > Replace debug trace and printf to log macros: > - debug() replaced by dev_dbg() when device is available, this macro > indicate the device name since commit ceb70bb870ac ("dm: Print device > name in dev_xxx like Linux") > - printf() repla

Re: [PATCH v2 1/6] net: eth-phy: add support of device tree configuration for gpio reset

2021-04-29 Thread Ramon Fried
On Mon, Apr 26, 2021 at 6:47 PM Patrick Delaunay wrote: > > The gpio reset and the assert or deassert delay are defined in generic > binding of the ethernet phy in Linux: > Documentation/devicetree/bindings/net/ethernet-phy.yaml > > reset-gpios: > maxItems: 1 > description: > The G

Re: [PATCH 2/3] net: dwc_eth_qos: define LOG_CATEGORY

2021-04-29 Thread Ramon Fried
On Mon, Apr 26, 2021 at 6:25 PM Patrick Delaunay wrote: > > Define LOG_CATEGORY to allow filtering with log command. > > Signed-off-by: Patrick Delaunay > --- > > drivers/net/dwc_eth_qos.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/net/dwc_eth_qos.c b/drivers/net/dwc_eth_

Re: [PATCH 3/3] net: define LOG_CATEGORY

2021-04-29 Thread Ramon Fried
On Mon, Apr 26, 2021 at 6:25 PM Patrick Delaunay wrote: > > Define LOG_CATEGORY to allow filtering with log command. > > Signed-off-by: Patrick Delaunay > --- > > net/eth-uclass.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/net/eth-uclass.c b/net/eth-uclass.c > index 34ca731d1e..6

Re: [PATCH 1/3] net: eth-phy: define LOG_CATEGORY

2021-04-29 Thread Ramon Fried
On Mon, Apr 26, 2021 at 6:25 PM Patrick Delaunay wrote: > > Define LOG_CATEGORY to allow filtering with log command. > > Signed-off-by: Patrick Delaunay > --- > > drivers/net/eth-phy-uclass.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/net/eth-phy-uclass.c b/drivers/net/et

Re: [PATCH] net: use a more deterministic approach to get the active ethernet device

2021-04-29 Thread Ramon Fried
On Wed, Feb 24, 2021 at 6:30 PM Michael Walle wrote: > > If the environment variable "ethact" is not set, the first device in the > uclass is returned. This depends on the probing order of the ethernet > devices. Moreover it is not not configurable at all. > > Try to return the ethernet device wit

Re: [PATCH] efi_loader: check query_variable_info attributes

2021-04-29 Thread Heinrich Schuchardt
On 4/29/21 7:46 PM, Vincent Stehlé wrote: QueryVariableInfo() must return EFI_INVALID_PARAMETER when an invalid combination of attribute bits is supplied. This fixes three SCT QueryVariableInfo_Conf failures. Signed-off-by: Vincent Stehlé Reviewed-by: Grant Likely Cc: Heinrich Schuchardt Cc:

Re: [GIT PULL] xilinx patches for v2021.07-rc2

2021-04-29 Thread Tom Rini
On Thu, Apr 29, 2021 at 03:59:06PM +0200, Michal Simek wrote: > Hi Tom, > > please pull these changes to your tree. I can't see any issue from > gitlab CI and also buildman doesn't show any problem. > Key part of this series is to fix issue with gmii2rgmii bridge which > ends up in a while loop o

[PATCH] efi_loader: check query_variable_info attributes

2021-04-29 Thread Vincent Stehlé
QueryVariableInfo() must return EFI_INVALID_PARAMETER when an invalid combination of attribute bits is supplied. This fixes three SCT QueryVariableInfo_Conf failures. Signed-off-by: Vincent Stehlé Reviewed-by: Grant Likely Cc: Heinrich Schuchardt Cc: Alexander Graf Changes since v1: - Remove

Re: Kirkwood: Fix tv sec/usec normalization in kwboot

2021-04-29 Thread Dagan Martinez
>From c200095d4136a071dd9526a48be642ce58fae8c9 Mon Sep 17 00:00:00 2001 From: Dagan Martinez Date: Tue, 27 Apr 2021 15:48:31 -0400 Subject: [PATCH v2] Kirkwood: Fix tv sec/usec normalization in kwboot `kwboot.c` had an issue where it failed to normalize the `tv` struct in the case where the `tv_u

[Patch v2 2/2] board: Add Zynq Mxic picozed development board support

2021-04-29 Thread Zhengxun Li
Add the Zynq Mxic picozed development board support. Signed-off-by: Zhengxun Li --- arch/arm/dts/Makefile | 3 +- arch/arm/dts/zynq-mxic-picozed.dts | 72 ++ 2 files changed, 74 insertions(+), 1 deletion(-) create mode 100644 arch/arm/dts/zynq-m

[Patch v2 1/2] clk: zynq: Add clock wizard driver

2021-04-29 Thread Zhengxun Li
The Clocking Wizard IP supports clock circuits customized to your clocking requirements. The wizard support for dynamically reconfiguring the clocking primitives for Multiply, Divide, Phase Shift/Offset, or Duty Cycle. Limited by uboot clk uclass without set_phase API, this patch only provides set

[Patch v2 0/2] Add Xilinx clock wizard driver support

2021-04-29 Thread Zhengxun Li
This series add support to enable clock wizard for zynq platform. Changes in v2: - naming is aligned with linux - delete inappropriate description and code Zhengxun Li (2): clk: zynq: Add clock wizard driver board: Add Zynq Mxic picozed development board support arch/arm/dts/Makefile

Re: [PATCH 1/5] clk: ti: add custom API for memory access

2021-04-29 Thread Tero Kristo
On 28/04/2021 20:31, Dario Binacchi wrote: Hi Tero, Il 27/04/2021 09:01 Tero Kristo ha scritto: Hi Dario, One question below. On 25/04/2021 17:17, Dario Binacchi wrote: As pointed by [1] and [2], commit d64b9cdcd4 ("fdt: translate address if #size-cells = <0>") is wrong: - It makes eve

Re: [PATCH] fdtdec: fdtdec_get_aliases_highest_id: skip aliases to disabled nodes

2021-04-29 Thread Tim Harvey
On Thu, Apr 29, 2021 at 9:10 AM Simon Glass wrote: > > Hi Tim, > > On Fri, 16 Apr 2021 at 14:30, Tim Harvey wrote: > > > > When looking for an alias with the highest id skip aliases for nodes > > that are disabled. > > > > Signed-off-by: Tim Harvey > > --- > > lib/fdtdec.c | 2 ++ > > 1 file ch

[PATCH] include: configs: soc64: Use CONFIG_SPL_ATF to differentiate bootfile

2021-04-29 Thread Siew Chin Lim
Legacy boot flow (SPL->U-Boot Proper->OS) boot to OS via Kernel Image and dtb files using booti command. ATF boot flow (SPL->ATF->U-Boot Proper->OS) boot to OS via kernel.itb file using bootm command. Change to use CONFIG_SPL_ATF to differentiate the bootfile of default environment variable. We s

Re: [PATCH 1/1] doc: fatinfo man-page

2021-04-29 Thread Heinrich Schuchardt
Am 29. April 2021 18:10:01 MESZ schrieb Simon Glass : >On Sat, 17 Apr 2021 at 11:04, Heinrich Schuchardt >wrote: >> >> Provide a man-page for the fatinfo command. > >It seems to provide a link to it, not the actual page. A follow up patch with the missing file has already been merged. Thanks for

Re: [PATCH v3 2/7] lmb: add lmb_is_reserved_flags

2021-04-29 Thread Simon Glass
Hi Patrick, On Wed, 28 Apr 2021 at 03:23, Patrick Delaunay wrote: > > Add a new function lmb_is_reserved_flags to check is a > address is reserved with a specific flags. > > This function can be used to check if an address was > reserved with no-map flags with: > > lmb_is_reserved_flags(lmb, addr

Re: [PATCH v3 1/7] lmb: Add support of flags for no-map properties

2021-04-29 Thread Simon Glass
Hi Patrick, On Wed, 28 Apr 2021 at 03:23, Patrick Delaunay wrote: > > Add "flags" in lmb_property to save the "no-map" property of > reserved region and a new function lmb_reserve_flags() to check > this flag. > > The default allocation use flags = LMB_NONE. > > The adjacent reserved memory regio

Re: [PATCH v2] dm: core: Add address translation in fdt_get_resource

2021-04-29 Thread Simon Glass
Hi Patrick, On Wed, 28 Apr 2021 at 03:39, Patrick Delaunay wrote: > > Today of_address_to_resource() is called only in > ofnode_read_resource() for livetree support and > fdt_get_resource() is called when livetree is not supported. > > The fdt_get_resource() doesn't do the address translation > s

Re: [PATCH v3 4/7] test: lmb: add test for lmb_reserve_flags

2021-04-29 Thread Simon Glass
On Wed, 28 Apr 2021 at 03:23, Patrick Delaunay wrote: > > Add a test to check the management of reserved region with flags. > > Signed-off-by: Patrick Delaunay > --- > > (no changes since v1) > > test/lib/lmb.c | 89 ++ > 1 file changed, 89 inserti

Re: [PATCH v3 5/7] image-fdt: save no-map parameter of reserve-memory

2021-04-29 Thread Simon Glass
Hi Patrick, On Wed, 28 Apr 2021 at 03:23, Patrick Delaunay wrote: > > Save the no-map information present in 'reserved-memory' node to allow > correct handling when the MMU is configured in board to avoid > speculative access. > > Signed-off-by: Patrick Delaunay > --- > > (no changes since v1) >

Re: [PATCH v1 3/6] common: integrate crypt-based passwords

2021-04-29 Thread Simon Glass
Hi Steffen, On Mon, 26 Apr 2021 at 05:19, Steffen Jaeckel wrote: > > Hook into the autoboot flow as an alternative to the existing > mechanisms. > > Signed-off-by: Steffen Jaeckel > --- > > common/Kconfig.boot | 23 +--- > common/autoboot.c | 85 +++

Re: [PATCH 0/5] Revert "fdt: translate address if #size-cells = <0>"

2021-04-29 Thread Simon Glass
On Sun, 25 Apr 2021 at 07:17, Dario Binacchi wrote: > > > As pointed by [1] and [2] the d64b9cdcd4 ("fdt: translate address if > #size-cells = <0>") > commit was wrong. The series reverts the patch and fixes the issue with > platform code, adding custom routines to access the clocks registers. >

Re: [PATCH v3 3/7] lmb: add lmb_dump_region() function

2021-04-29 Thread Simon Glass
On Wed, 28 Apr 2021 at 03:23, Patrick Delaunay wrote: > > Add lmb_dump_region() function, to simplify lmb_dump_all_force(). > This patch is based on Linux memblock dump function. > > An example of bdinfo output is: > > . > fdt_size= 0x000146a0 > FB base = 0xfdd0 > lmb_dump_all: >

Re: [PATCH] dm: define LOG_CATEGORY for all uclass

2021-04-29 Thread Simon Glass
On Tue, 27 Apr 2021 at 02:02, Patrick Delaunay wrote: > > Define LOG_CATEGORY for all uclass to allow filtering with > log command. > > Signed-off-by: Patrick Delaunay > --- > > drivers/adc/adc-uclass.c| 2 ++ > drivers/ata/ahci-uclass.c | 2 ++ > drivers/ax

Re: [PATCH v4 2/6] lib: ecdsa: Add skeleton to implement ecdsa verification in u-boot

2021-04-29 Thread Simon Glass
Hi Alex, On Mon, 26 Apr 2021 at 07:21, Alex G. wrote: > > > > On 4/23/21 11:56 PM, Simon Glass wrote: > > Hi Tom, Alex, > > > > On Fri, 23 Apr 2021 at 12:47, Tom Rini wrote: > >> > >> On Fri, Apr 23, 2021 at 11:55:57AM +1200, Simon Glass wrote: > >>> Hi Alex, > >>> > >>> On Thu, 22 Apr 2021 at 0

Re: [PATCH v1 1/6] lib: add crypt subsystem

2021-04-29 Thread Simon Glass
Hi Steffen, On Mon, 26 Apr 2021 at 05:19, Steffen Jaeckel wrote: > > Add the basic functionality required to support the standard crypt > format. > The files crypt-sha256.c and crypt-sha512.c originate from libxcrypt and > their formatting is therefor retained. > The integration is done via a cry

Re: [PATCH] net: use the same alias stem for ethernet as linux

2021-04-29 Thread Simon Glass
On Thu, 25 Feb 2021 at 07:51, Michael Walle wrote: > > Linux uses the prefix "ethernet" whereas u-boot uses "eth". This is from > the linux tree: > > $ grep "eth[0-9].*=.*&" arch/**/*dts{,i}|wc -l > 0 > $ grep "ethernet[0-9].*=.*&" arch/**/*dts{,i}|wc -l > 633 > > In u-boot device trees both prefi

Re: [PATCH 03/17] x86: Allow coreboot serial driver to guess the UART

2021-04-29 Thread Simon Glass
Hi Bin, On Sun, 25 Apr 2021 at 18:21, Bin Meng wrote: > > Hi Simon, > > On Sun, Apr 25, 2021 at 10:10 AM Simon Glass wrote: > > > > Hi Bin, > > > > On Sun, 25 Apr 2021 at 13:49, Bin Meng wrote: > > > > > > Hi Simon, > > > > > > On Sat, Apr 24, 2021 at 12:56 PM Simon Glass wrote: > > > > > > >

Re: [PATCH 2/5] common: integrate crypt-based passwords

2021-04-29 Thread Simon Glass
Hi Steffen, On Wed, 21 Apr 2021 at 01:55, Steffen Jaeckel wrote: > > On 4/21/21 9:14 AM, Simon Glass wrote: > > Hi Steffen, > > > > On Tue, 13 Apr 2021 at 10:16, Steffen Jaeckel > > wrote: > >> > >> Hook into the autoboot flow as an alternative to the existing > >> mechanisms. > >> > >> Signed-o

Re: [Uboot-stm32] [PATCH 1/2] cmd: pinmux: update result of do_status

2021-04-29 Thread Simon Glass
Hi Patrick, On Tue, 20 Apr 2021 at 03:21, Patrice CHOTARD wrote: > > Hi Patrick > > -Original Message- > From: Uboot-stm32 On > Behalf Of Patrick DELAUNAY > Sent: mercredi 28 octobre 2020 11:07 > To: u-boot@lists.denx.de > Cc: U-Boot STM32 ; Simon Glass > ; Patrick DELAUNAY ; Sean > A

Re: [PATCH v2 5/5] test/py: Add usb gadget binding test

2021-04-29 Thread Simon Glass
On Mon, 19 Apr 2021 at 02:58, Patrice Chotard wrote: > > Add a specific usb gadget binding test which check that > binding a driver without compatible string is working as expected. > > the command "bind /usb@1 usb_ether" should give the following "dm tree" > command output: > > [...] > usb

Re: [PATCH 2/2] cmd: pinmux: support pin name in status command

2021-04-29 Thread Simon Glass
Hi Patrick, On Wed, 28 Oct 2020 at 03:06, Patrick Delaunay wrote: > > Allow pin name parameter for pimux staus command, > as gpio command to get status of one pin. > > The possible usage of the command is: > > > pinmux dev pinctrl > > pinmux status > > > pinmux status -a > > > pinmux status > >

Re: [PATCH 1/1] doc: fatinfo man-page

2021-04-29 Thread Simon Glass
On Sat, 17 Apr 2021 at 11:04, Heinrich Schuchardt wrote: > > Provide a man-page for the fatinfo command. It seems to provide a link to it, not the actual page. > > Signed-off-by: Heinrich Schuchardt > --- > doc/usage/index.rst | 1 + > 1 file changed, 1 insertion(+) Reviewed-by: Simon Glass

Re: [PATCH v2 4/5] configs: sandbox: add USB_ETHER and GADGET_DOWNLOAD gadget support

2021-04-29 Thread Simon Glass
On Mon, 19 Apr 2021 at 02:49, Patrice Chotard wrote: > > This is needed for new gadget binding test. > > Signed-off-by: Patrice Chotard > Cc: Marek Vasut > Cc: Herbert Poetzl > --- > > (no changes since v1) > > configs/sandbox_defconfig | 3 +++ > 1 file changed, 3 insertions(+) Reviewed-by:

Re: [PATCH v2 04/13] gpio: Introduce CONFIG_ONLY_GENERIC_GPIO to cleanup #ifdefs

2021-04-29 Thread Simon Glass
Hi Masami, On Fri, 16 Apr 2021 at 16:38, Masami Hiramatsu wrote: > > Many architecture do not have specific asm/arch/gpio.h, so instead > of adding !defined(CONFIG_ARCH_xxx), introduce CONFIG_ONLY_GENERIC_GPIO This seems OK, but I think GPIO_GENERIC_ONLY is a better name, since it uses the GPIO

Re: [RFC PATCH v1] lib: rsa: introduce RSA_SOFTWARE_EXP_TINY

2021-04-29 Thread Simon Glass
Hi Igor, On Fri, 16 Apr 2021 at 01:10, Igor Opaniuk wrote: > > From: Igor Opaniuk > > Introduce RSA_SOFTWARE_EXP_TINY Kconfig option, which does not require > DM to be enabled. This can be handy on devices, where SPL + signed > U-Boot FIT image setup is used, where it isn't possible to enable SP

Re: [PATCH] fdtdec: fdtdec_get_aliases_highest_id: skip aliases to disabled nodes

2021-04-29 Thread Simon Glass
Hi Tim, On Fri, 16 Apr 2021 at 14:30, Tim Harvey wrote: > > When looking for an alias with the highest id skip aliases for nodes > that are disabled. > > Signed-off-by: Tim Harvey > --- > lib/fdtdec.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/lib/fdtdec.c b/lib/fdtdec.c > index

Re: [PATCH v2 2/4] test: Allow simple glob pattern in the test name

2021-04-29 Thread Simon Glass
On Thu, 11 Feb 2021 at 07:40, Andy Shevchenko wrote: > > When run `ut dm [test name]` allow to use simple pattern to run all tests > started with given prefix. For example, to run all ACPI test cases: > ut dm acpi* > > Signed-off-by: Andy Shevchenko > --- > v2: rebased against dm/test-wor

Re: [PATCH 1/2] binman: Tweak implementation of fmap

2021-04-29 Thread Simon Glass
Use an interator in two of the fmap tests so it is easier to add new items. Also check the name first since that is the first indication that something is wrong. Use a variable for the expected size of the fmap to avoid repeating the code. Signed-off-by: Simon Glass --- tools/binman/ftest.py |

Re: [PATCH v2 3/4] test: Use positive conditional in test_matches()

2021-04-29 Thread Simon Glass
On Thu, 11 Feb 2021 at 07:40, Andy Shevchenko wrote: > > It is easier to read the positive conditional. > > While at it, convert hard coded length of "_test_" to strlen("_test_") > which will be converted to a constant bu optimizing compiler. > > Signed-off-by: Andy Shevchenko > --- > v2: new pat

Re: [PATCH 3/5] Azure/GitLab: Ensure we use requirements.txt for testsuites

2021-04-29 Thread Simon Glass
On Fri, Feb 26, 2021 at 10:31 PM Tom Rini wrote: > > On Fri, Feb 26, 2021 at 10:25:30PM +0800, Bin Meng wrote: > > On Fri, Feb 26, 2021 at 8:53 PM Tom Rini wrote: > > > > > > Given that test/py/requirements.txt has all required test modules, make > > > use of that rather than a manual pip install

Re: [PATCH 2/2] binman: Support adding sections to FMAPs

2021-04-29 Thread Simon Glass
When used with hierarchical images, use the Chromium OS convention of adding a section before all the subentries it contains. Signed-off-by: Simon Glass --- tools/binman/entries.rst | 13 +-- tools/binman/etype/fmap.py | 20 +++-- tools/binm

Re: [PATCH] dm: core: Fix uninitialized return value from dm_scan_fdt_node

2021-04-29 Thread Simon Glass
On Thu, 8 Apr 2021 at 22:15, Sean Anderson wrote: > > If there are no nodes or if all nodes are disabled, this function would > return err without setting it first. Fix this by initializing err to > zero. > > Fixes: 94f7afdf7e ("dm: core: Ignore disabled devices when binding") > > Signed-off-by: S

Re: [PATCH 1/4] buildman: Tidy up a few comments

2021-04-29 Thread Simon Glass
Add some function comments which are missing, or missing arguments. Signed-off-by: Simon Glass --- tools/buildman/builderthread.py | 10 +++--- tools/buildman/control.py | 2 ++ tools/buildman/func_test.py | 13 +++-- 3 files changed, 20 insertions(+), 5 deletions(-) App

Re: [PATCH 2/4] buildman: Use common code to send an result

2021-04-29 Thread Simon Glass
At present the code to report a build result is duplicated. Put it in a common function to avoid this. Signed-off-by: Simon Glass --- tools/buildman/builderthread.py | 21 + 1 file changed, 13 insertions(+), 8 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH] patman: Parse checkpatch by message instead of by line

2021-04-29 Thread Simon Glass
On Thu, 1 Apr 2021 at 15:50, Evan Benn wrote: > > Parse each empty-line-delimited message separately. This saves having to > deal with all the different line content styles, we only care about the > header ERROR | WARNING | NOTE... > > Also make checkpatch print line information for a uboot specif

Re: [PATCH 4/4] buildman: Use bytes for the environment

2021-04-29 Thread Simon Glass
At present we sometimes see problems in gitlab where the environment has 0x80 characters or sequences which are not valid UTF-8. Avoid this by using bytes for the environment, both internal to buildman and when writing out the 'env' file. Add a test to make sure this works as expected. Reported-b

Re: [PATCH 3/4] buildman: Handle exceptions in threads gracefully

2021-04-29 Thread Simon Glass
There have been at least a few cases where an exception has occurred in a thread and resulted in buildman hanging: running out of disk space and getting a unicode error. Handle these by collecting a list of exceptions, printing them out and reporting failure if any are found. Add a test for this.

Re: [PATCH v3 2/2] test: dm: add test item for ofnode_get_addr() and ofnode_get_size()

2021-04-29 Thread Simon Glass
On Mon, 12 Apr 2021 at 18:51, chenguanqiao wrote: > > From: Chen Guanqiao > > Add test item for getting address and size functions > > Test the following function: > - ofnode_get_addr() > - ofnode_get_size() > > Signed-off-by: Chen Guanqiao > --- > test/dm/ofnode.c | 31

Re: [PATCH v3 1/2] dm: core: Add size operations on device tree references

2021-04-29 Thread Simon Glass
On Mon, 12 Apr 2021 at 18:51, chenguanqiao wrote: > > From: Chen Guanqiao > > Add functions to add size of addresses in the device tree using ofnode > references. > > If the size is not set, return FDT_SIZE_T_NONE. > > Signed-off-by: Chen Guanqiao > --- > drivers/core/ofnode.c | 11 +++

Re: [PATCH] binman: Correct testSplNoDtb() and Tpl also

2021-04-29 Thread Simon Glass
On Sun, Apr 25, 2021 at 08:39:32AM +1200, Simon Glass wrote: > These two tests require an ELF image so that symbol information can be > written into the SPL/TPL binary. At present they rely on other tests > having set it up first, but every test must run independently. This can > cause occasional

Re: [PATCH 1/1] tpm: missing event types

2021-04-29 Thread Simon Glass
On Wed, Apr 21, 2021 at 12:24:29PM +0200, Heinrich Schuchardt wrote: > Add a reference for the TPM event types and provide missing constants. > > Signed-off-by: Heinrich Schuchardt > --- > include/tpm-v2.h | 24 > 1 file changed, 16 insertions(+), 8 deletions(-) > Applied

Re: [PATCH] dtoc: Correct dtoc output when testing

2021-04-29 Thread Simon Glass
At present each invocation of run_steps() updates OUTPUT_FILES_COMMON, since it does not make a copy of the dict. This is fine for a single invocation, but for tests, run_steps() is invoked many times. As a result it may include unwanted items from the previous run, if it happens that a test runs

[PATCH] configs: sama7g5ek: increase bootm len

2021-04-29 Thread Eugen Hristev
Increase the BOOTM_LEN to 32M . This would allow a bigger kernel image to be booted, for example the multi_v7_defconfig. Loading Kernel Image Image too large: increase CONFIG_SYS_BOOTM_LEN Must RESET board to recover resetting ... Signed-off-by: Eugen Hristev --- include/configs/sama7g5ek.h

Re: Please pull u-boot-cfi-flash/master

2021-04-29 Thread Tom Rini
On Thu, Apr 29, 2021 at 10:48:45AM +0200, Stefan Roese wrote: > Hi Tom, > > please pull the following cfi-flash related patches: > Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: Please pull u-boot-marvell/master

2021-04-29 Thread Tom Rini
On Thu, Apr 29, 2021 at 08:47:03AM +0200, Stefan Roese wrote: > Hi Tom, > > please pull the next batch of Marvell Armada related patches. Here the > summary log: > Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: [PULL] u-boot-usb/master

2021-04-29 Thread Tom Rini
On Thu, Apr 29, 2021 at 12:09:43AM +0200, Marek Vasut wrote: > The following changes since commit 79b0f08d6af498e6fda8cd257d62e2095764410c: > > configs: Resync with savedefconfig (2021-04-27 08:28:38 -0400) > > are available in the Git repository at: > > git://source.denx.de/u-boot-usb.git

Re: [PULL u-boot] Please pull u-boot-amlogic-20210428

2021-04-29 Thread Tom Rini
On Wed, Apr 28, 2021 at 05:50:41PM +0200, Neil Armstrong wrote: > Hi Tom, > > A single patch to fix boot on (at least) Odroid-C4, where the MDIO bus reset > used random > sleep values instead of the proper pdata values. > > The CI job is at > https://gitlab.denx.de/u-boot/custodians/u-boot-aml

Re: [PATCH 6/6] net: octeontx: smi: fix mii probe

2021-04-29 Thread Tim Harvey
On Wed, Apr 28, 2021 at 10:21 PM Stefan Roese wrote: > > Hi Tim, > > On 28.04.21 17:11, Tim Harvey wrote: > > On Mon, Apr 26, 2021 at 10:19 PM Stefan Roese wrote: > >> > >> Hi Tim, > >> > >> On 26.03.21 16:55, Tim Harvey wrote: > >>> On Thu, Mar 25, 2021 at 11:48 PM Stefan Roese wrote: > >

[GIT PULL] xilinx patches for v2021.07-rc2

2021-04-29 Thread Michal Simek
Hi Tom, please pull these changes to your tree. I can't see any issue from gitlab CI and also buildman doesn't show any problem. Key part of this series is to fix issue with gmii2rgmii bridge which ends up in a while loop on every ZynqMP board. And then usb dfu fixes with also enabling dfu functio

Re: [PATCH] spi: zynqmp: Remove gd reference

2021-04-29 Thread Michal Simek
po 26. 4. 2021 v 8:28 odesílatel Michal Simek napsal: > > gd is not used in this file that's why doesn't make sense to declare it. > > Signed-off-by: Michal Simek > --- > > drivers/spi/zynqmp_gqspi.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/spi/zynqmp_gqspi.c b/drivers/

Re: [PATCH v7 0/8] Add FU740 chip and HiFive Unmatched board support

2021-04-29 Thread Bin Meng
Hi Green, On Thu, Apr 29, 2021 at 7:11 PM Green Wan wrote: > > Hi Bin, > > How should this patch set be proceeded? > > To summary the major changes, > - I've rebased to mainstream and merged pcie refactoring code which > based on pcie_dw_common.c > - separate unmatched dts into separated patch. >

RE: [v1 06/17] drivers: clk: Add memory clock driver for Intel N5X device

2021-04-29 Thread Lim, Elly Siew Chin
> -Original Message- > From: Tan, Ley Foon > Sent: Thursday, April 8, 2021 6:32 PM > To: Lim, Elly Siew Chin ; u-boot@lists.denx.de > Cc: Marek Vasut ; See, Chin Liang > ; Simon Goldschmidt > ; Chee, Tien Fong > ; Westergreen, Dalon > ; Simon Glass ; Gan, > Yau Wai > Subject: RE: [v1 0

RE: [v1 04/17] drivers: clk: Add clock driver for Intel N5X device

2021-04-29 Thread Lim, Elly Siew Chin
> -Original Message- > From: Tan, Ley Foon > Sent: Monday, April 5, 2021 3:37 PM > To: Lim, Elly Siew Chin ; u-boot@lists.denx.de > Cc: Marek Vasut ; See, Chin Liang > ; Simon Goldschmidt > ; Chee, Tien Fong > ; Westergreen, Dalon > ; Simon Glass ; Gan, > Yau Wai > Subject: RE: [v1 04/

Re: [PATCH v7 0/8] Add FU740 chip and HiFive Unmatched board support

2021-04-29 Thread Green Wan
Hi Bin, How should this patch set be proceeded? To summary the major changes, - I've rebased to mainstream and merged pcie refactoring code which based on pcie_dw_common.c - separate unmatched dts into separated patch. Thanks, - Green On Thu, Apr 22, 2021 at 5:13 PM Green Wan wrote: > > This p

Re: [Patch v2 1/2] clk: zynq: Add clock wizard driver

2021-04-29 Thread Michal Simek
On 4/29/21 11:31 AM, Zhengxun Li wrote: > The Clocking Wizard IP supports clock circuits customized > to your clocking requirements. The wizard support for > dynamically reconfiguring the clocking primitives for > Multiply, Divide, Phase Shift/Offset, or Duty Cycle. > > Limited by uboot clk ucl

Re: [Patch v2 2/2] board: Add Zynq Mxic picozed development board support

2021-04-29 Thread Michal Simek
On 4/29/21 11:31 AM, Zhengxun Li wrote: > Add the Zynq Mxic picozed development board support. > > Signed-off-by: Zhengxun Li > --- > arch/arm/dts/Makefile | 3 +- > arch/arm/dts/zynq-mxic-picozed.dts | 72 > ++ > 2 files changed, 74 insertio

[PATCH 2/2] MAINTAINERS: Add an entry for VirtIO

2021-04-29 Thread Bin Meng
This was missed when VirtIO support was initially brought to U-Boot back in 2018. Add an entry for it and list myself as the maintainer. Signed-off-by: Bin Meng --- MAINTAINERS | 12 1 file changed, 12 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 2d267aeff2..20092cb3

[PATCH 1/2] doc: develop: Convert README.virtio to reST

2021-04-29 Thread Bin Meng
This convers the existing README.virtio to reST, and puts it under the develop/driver-model/ directory. Signed-off-by: Bin Meng --- doc/develop/driver-model/index.rst| 1 + .../driver-model/virtio.rst} | 90 +-- 2 files changed, 63 insertions(+), 28

Re: [PATCH v1 00/23] phy: marvell: Sync Armada 3k/7k/8k SERDES code with Marvell version

2021-04-29 Thread Stefan Roese
Hi Marek, (Added Tom and Simon to Cc) On 29.04.21 10:27, Marek Behun wrote: On Thu, 29 Apr 2021 08:46:36 +0200 Stefan Roese wrote: phy: marvell: add RX training command Applied to u-boot-marvell/master Thanks, Stefan Stefan, do you think it would make sense to at least create a spec

Re: [PATCH] arm64: zynqmp: Enable capsule update

2021-04-29 Thread Michal Simek
On 4/29/21 10:52 AM, Heinrich Schuchardt wrote: > On 26.04.21 13:08, Michal Simek wrote: >> Enable EFI capsule update features to be enabled by default also with all >> dfu valid options for ZynqMP. >> >> This feature was tested on Xilinx ZynqMP zcu104 board with defining >> dfu_alt_info="mmc 0:

Re: [PATCH] arm64: zynqmp: Enable capsule update

2021-04-29 Thread Heinrich Schuchardt
On 26.04.21 13:08, Michal Simek wrote: > Enable EFI capsule update features to be enabled by default also with all > dfu valid options for ZynqMP. > > This feature was tested on Xilinx ZynqMP zcu104 board with defining > dfu_alt_info="mmc 0:1=boot.bin fat 0 1;u-boot.itb fat 0 1" > and > dfu_alt_inf

Please pull u-boot-cfi-flash/master

2021-04-29 Thread Stefan Roese
Hi Tom, please pull the following cfi-flash related patches: - mtd: cfi: Fix PPB lock status readout (Marek) Here the Azure build, without any issues: https://dev.a

Re: [PATCH] mtd: cfi: Fix PPB lock status readout

2021-04-29 Thread Stefan Roese
On 11.04.21 20:47, Marek Vasut wrote: According to S26KL512S datasheet [1] and S29GL01GS datasheet [2], the procedure to read out PPB lock bits is to send the PPB Entry, PPB Read, Reset/ASO Exit. Currently, the code does send incorrect PPB Entry, PPB Read and Reset/ASO Exit is completely missing.

Re: [PATCH v1 00/23] phy: marvell: Sync Armada 3k/7k/8k SERDES code with Marvell version

2021-04-29 Thread Marek Behun
On Thu, 29 Apr 2021 08:46:36 +0200 Stefan Roese wrote: > >phy: marvell: add RX training command > > Applied to u-boot-marvell/master > > Thanks, > Stefan Stefan, do you think it would make sense to at least create a special mechanism for these platform commands? For example via a top-level

Re: [PATCH] net: phy: xilinx: Break while loop over ethernet phy

2021-04-29 Thread Michal Simek
Hi Bin, On 4/28/21 6:36 PM, Bin Meng wrote: > Hi Michal, > > On Thu, Apr 29, 2021 at 12:17 AM Michal Simek wrote: >> >> Hi Bin, >> >> On 4/28/21 5:57 PM, Bin Meng wrote: >>> Hi Michal, >>> >>> On Wed, Apr 28, 2021 at 11:03 PM Michal Simek >>> wrote: Hi Bin, On 4/28/21 4:37

  1   2   >