RE: [EXT] Re: [PATCH v2 00/15] Add CAAM driver model support

2021-09-27 Thread Gaurav Jain
Hi Tim > -Original Message- > From: Tim Harvey > Sent: Friday, September 24, 2021 4:32 AM > To: Gaurav Jain ; Peng Fan > Cc: u-boot ; Stefano Babic ; Fabio > Estevam ; Simon Glass ; Priyanka > Jain ; Ye Li ; Horia Geanta > ; Ji Luo ; Franck Lenormand > ; Silvano Di Ninno ; > Sahil

RE: [EXT] Re: [PATCH v2 03/15] i.MX8M: crypto: updated device tree for supporting DM in SPL

2021-09-27 Thread Gaurav Jain
Hello Tim > -Original Message- > From: Tim Harvey > Sent: Friday, September 24, 2021 4:10 AM > To: Gaurav Jain > Cc: u-boot ; Stefano Babic ; Fabio > Estevam ; Peng Fan ; Simon Glass > ; Priyanka Jain ; Ye Li > ; Horia Geanta ; Ji Luo > ; Franck Lenormand ; Silvano Di > Ninno ; Sahil

Re: [PATCH 0/5] Apple M1 Support

2021-09-27 Thread Simon Glass
Hi Mark, On Sun, 26 Sept 2021 at 09:53, Simon Glass wrote: > > Hi Mark, > > On Sat, 25 Sept 2021 at 10:46, Mark Kettenis wrote: > > > > > From: Simon Glass > > > Date: Sat, 25 Sep 2021 08:42:30 -0600 > > > > > > Hi Mark, > > > > > > On Sat, 25 Sept 2021 at 07:52, Mark Kettenis > > > wrote: >

Re: [PATCH v2 2/2] nvme: Fix error in nvme_setup_prps

2021-09-27 Thread Shawn Lin
在 2021/9/27 21:22, Jon Lin 写道: Consulting to "NVM Express® Base Specification, revision 2.0". If more PRP List pages are required, then the last entry of the PRP List contains the Page Base Address of the next PRP List page. The next PRP List page shall be memory page aligned. Yep, this is

Re: [PATCH v2 1/2] nvme: Enable FUA

2021-09-27 Thread Shawn Lin
在 2021/9/27 21:22, Jon Lin 写道: Most NVME devcies maintain data in internal cache for an uncertain times, and u-boot has no method to force NVME to flush cache. So this patch adds FUA to avoid data loss caused by power off after data programming. Signed-off-by: Jon Lin Reviewed-by: Stefan Agner

[PATCH 6/9] net: add driver for NXP SJA1105 DSA L2 switch

2021-09-27 Thread Vladimir Oltean
The SJA1105 driver is largely reused from Linux. Its programming model is that it is blank out of reset, and it waits for a static configuration stream over SPI, which contains all runtime parameters (it has no notion of "default values"). Keeping a binary array for the configuration stream would

[PATCH v2] rtc: ds1307: Handle oscillator-stop bit correctly

2021-09-27 Thread Mark Tomlinson
The DS1307 driver was originally based on the DS1337 driver. However, the functionality of the clock set/get functions has diverged. In the original DS1337 driver, the set/get functions did the following: 1) Setting the clock ensured the oscillator was enabled. 2) Getting the clock checked and

[PATCH 1/2] riscv: Support booting SiFive Unmatched from SPI.

2021-09-27 Thread Thomas Skibo
Configure SPI flash devices into SPL. Add SPI boot option to spl.c. Document how to format flash for booting. Signed-off-by: Thomas Skibo --- .../dts/hifive-unmatched-a00-u-boot.dtsi | 11 +++ board/sifive/unmatched/spl.c | 3 ++ configs/sifive_unmatched_defconfig

[PATCH 2/2] riscv: Enable SPI flash env for SiFive Unmatched.

2021-09-27 Thread Thomas Skibo
Enable saving environment to SPI flash memory on SiFive Unmatched. Signed-off-by: Thomas Skibo --- arch/riscv/cpu/fu740/Kconfig | 13 + board/sifive/unmatched/Kconfig | 1 + 2 files changed, 14 insertions(+) diff --git a/arch/riscv/cpu/fu740/Kconfig

[PATCH 0/2] riscv: Support booting SiFive Unmatched from SPI flash.

2021-09-27 Thread Thomas Skibo
This patch set adds support for booting the SiFive Unmatched board from SPI flash memory and saving the environment to flash. Thomas Skibo (2): riscv: Support booting SiFive Unmatched from SPI. riscv: Enable SPI flash env for SiFive Unmatched. arch/riscv/cpu/fu740/Kconfig |

imx6 DM_VIDEObroken

2021-09-27 Thread Tim Harvey
Anatolij, Since commit d37618d18d49 ("imx: convert gwventana board to DM_VIDEO") video support for IMX6 based Ventana boards has been broken. I find that while the bind function for fsl_imx6q_ipu is called the probe never is (ipuv3_video_probe). Do you know why this is? I see that with commit

[PATCH 8/9] configs: ls1021a-tsn: enable sja1105 switch driver

2021-09-27 Thread Vladimir Oltean
The sja1105 is a 5-port switch that uses a DM_DSA driver. Its 5th (CPU) port is connected internally to the eth2 port of the LS1021A SoC. Signed-off-by: Vladimir Oltean --- configs/ls1021atsn_qspi_defconfig | 2 ++ configs/ls1021atsn_sdcard_defconfig | 2 ++ 2 files changed, 4 insertions(+)

[PATCH 5/9] net: dsa: felix: configure the in-band autoneg property based on OF node info

2021-09-27 Thread Vladimir Oltean
Instead of trying to guess which operating modes need in-band negotiation to be active and which ones don't, parse the available information from the device tree. That will be correct in the cases we can already guess, and more. Signed-off-by: Vladimir Oltean ---

[PATCH 9/9] configs: ls1021a-tsn: enable the generation of random Ethernet MAC addresses

2021-09-27 Thread Vladimir Oltean
Don't fail when booting a board with an empty EEPROM for MAC addresses. Signed-off-by: Vladimir Oltean --- configs/ls1021atsn_qspi_defconfig | 1 + configs/ls1021atsn_sdcard_defconfig | 1 + 2 files changed, 2 insertions(+) diff --git a/configs/ls1021atsn_qspi_defconfig

[PATCH 4/9] net: introduce a helper to determine whether to use in-band autoneg

2021-09-27 Thread Vladimir Oltean
Certain serial SERDES protocols like 1000base-x, 2500base-x, SGMII, USXGMII can operate either in a mode where the PHY (be it on-board or inside an SFP module) passes the link parameters (speed, duplex, pause) to the MAC through in-band through control words standardized by IEEE 802.3 clause 37,

[PATCH 3/9] net: dsa: allow drivers to get the port OF node

2021-09-27 Thread Vladimir Oltean
In the current DSA switch driver API, only the udevice of the switch (belonging to UCLASS_DSA) is exposed, as well as an "int port" argument. So drivers do not have access to the udevice of individual ports (belonging to UCLASS_ETH), one of the reasons being that not all ports have an associated

[PATCH 7/9] arm: dts: ls1021a-tsn: add sja1105 and eth2 bindings

2021-09-27 Thread Vladimir Oltean
The eth aliases are for correct probing order, so that each Ethernet port will get a predictable MAC address from the environment. Signed-off-by: Vladimir Oltean --- arch/arm/dts/ls1021a-tsn.dts | 103 +++ 1 file changed, 103 insertions(+) diff --git

[PATCH 2/9] include: import if_vlan.h from Linux

2021-09-27 Thread Vladimir Oltean
This is needed for the VLAN header structure. Signed-off-by: Vladimir Oltean --- include/linux/if_vlan.h | 26 ++ 1 file changed, 26 insertions(+) create mode 100644 include/linux/if_vlan.h diff --git a/include/linux/if_vlan.h b/include/linux/if_vlan.h new file mode

[PATCH 1/9] net: tsec: add support for promiscuous mode

2021-09-27 Thread Vladimir Oltean
The Freescale TSEC can be a DSA master, and the ports of the attached DSA switch can have different MAC addresses compared to the TSEC. Nonetheless, the TSEC must receive the packets on behalf of those switch ports. Therefore, implement the promiscuous mode method to allow DSA to set this. Note

[PATCH 0/9] Support the SJA1105 DSA switch on the NXP LS1021A-TSN board

2021-09-27 Thread Vladimir Oltean
The NXP LS1021A-TSN board (and many others) contains an automotive Ethernet switch. Some boards only have Ethernet ports routed through the switch, and it is therefore desirable to have TFTP functional. These patches add support for the SJA1105 and SJA1110 switch family (tested on first-gen

[PATCH] configs: ls1028a: ensure Ethernet is enabled

2021-09-27 Thread Vladimir Oltean
CONFIG_FSL_ENETC is not explicitly enabled in the NXP LS1028A config files, instead it is selected by CONFIG_MSCC_FELIX_SWITCH, a state of matters which is fragile. CONFIG_MSCC_FELIX_SWITCH depends on CONFIG_DM_DSA, which depends on CONFIG_PHY_FIXED. Not all LS1028A boards did enable

[PATCH] net: phy: mscc: add support for VSC8502 in dual RGMII mode

2021-09-27 Thread Vladimir Oltean
The VSC8502 is a Microchip (formerly Microsemi, formerly Vitesse) dual port, gigabit Ethernet copper PHY which supports the MII, GMII and RGMII MAC-side interfaces. Of these, I could only test RGMII, and my board needed RGMII delays to be applied by software, so I am able to confirm that this

Re: GSuit limit for recipients

2021-09-27 Thread Tom Rini
On Mon, Sep 27, 2021 at 05:58:08PM +0300, Oleksandr Suvorov wrote: > Dear community, > > This weekend I tried to post (using patman) a patch set that should > have been sent to 120 recipients (a lot of files were changed). > > I've got an error from the Gmail SMTP server: > 4.5.3 Your message

[PATCH] arm: dts: armada8040: Fix CP0 eMMC/SDIO support

2021-09-27 Thread Robert Marko
During the migration to a single DTSI for the CP110-s specific pinctrl compatibles were moved to the SoC DTSI as CP0 and CP1 have some specifics. Namely, CP0 eMMC/SDIO support depends on the mvebu-pinctrl driver setting the BIT(0) in eMMC PHY IO Control 0 Register to 0 in order for the connect the

Re: Please pull u-boot-dm/next

2021-09-27 Thread Tom Rini
On Mon, Sep 27, 2021 at 08:58:17AM -0600, Simon Glass wrote: > Hi Tom, > > This is for the -next branch. > > https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/9260 > > > > The following changes since commit bb38d77ca779cc8bdad3d4ceb6cecc687f4987c2: > > vexpress_ca9x4: Add

Contributor call tomorrow

2021-09-27 Thread Simon Glass
Hi, This is tomorrow. If you plan to join and want to add a topic, please do so at [1]. Regards, Simon [1] https://bit.ly/3bFvwA1 or https://docs.google.com/document/d/1YBOMsbM19uSFyoJWnt7-PsOLBaevzQUgV-hiR88a5-o/edit#heading=h.flytinyefvov

Re: [PATCH v2 2/3] efi_loader: add UEFI GPT measurement

2021-09-27 Thread Ilias Apalodimas
On Tue, Sep 21, 2021 at 04:19:30PM +0900, Masahisa Kojima wrote: > This commit adds the UEFI GPT disk partition topology > measurement required in TCG PC Client PFP Spec. > > Signed-off-by: Masahisa Kojima > --- > > (no changes since v1) > > include/blk.h | 3 + >

Re: [PATCH v5 02/29] kconfig: Add tools support to CONFIG_IS_ENABLED()

2021-09-27 Thread Simon Glass
Hi Masahiro, On Mon, 27 Sept 2021 at 10:51, Masahiro Yamada wrote: > > On Tue, Sep 28, 2021 at 1:11 AM Alex G. wrote: > > > > > > > > On 9/25/21 8:43 PM, Simon Glass wrote: > > > At present we must separately test for the host build for many options, > > > since we force them to be enabled. For

Re: [PATCH v5 02/29] kconfig: Add tools support to CONFIG_IS_ENABLED()

2021-09-27 Thread Simon Glass
Hi Alex, On Mon, 27 Sept 2021 at 10:11, Alex G. wrote: > > > > On 9/25/21 8:43 PM, Simon Glass wrote: > > At present we must separately test for the host build for many options, > > since we force them to be enabled. For example, CONFIG_FIT is always > > enabled in the host tools, even if

Re: [PATCH 2/4] dm: hash: Add new UCLASS_HASH support

2021-09-27 Thread Simon Glass
Hi Alex, On Mon, 27 Sept 2021 at 09:37, Alex G. wrote: > > On 9/23/21 9:49 PM, Simon Glass wrote:> On Thu, 16 Sept 2021 at 09:43, > Alex G. wrote: > >> On 7/29/21 8:08 PM, Chia-Wei Wang wrote: > > >>> + > >>> +enum HASH_ALGO hash_algo_lookup_by_name(const char *name) > >> > >> string ->

Re: [PATCH v2 1/3] efi_loader: add SMBIOS table measurement

2021-09-27 Thread Simon Glass
Hi Ilias, On Mon, 27 Sept 2021 at 02:52, Ilias Apalodimas wrote: > > Hi Simon, > > [...] > > > > > > - remove unnecessary const qualifier from smbios_string() > > > > > - create non-const version of next_header() > > > > > > > > > > include/efi_loader.h | 2 + > > > > >

Re: [PATCH 3/3] treewide: Remove OF_PRIOR_STAGE

2021-09-27 Thread Simon Glass
On Mon, 27 Sept 2021 at 00:48, Ilias Apalodimas wrote: > > The previous patches removed OF_PRIOR_STAGE from the last consumers of the > Kconfig option. Cleanup any references to it in documentation, code and > configuration options. > > Signed-off-by: Ilias Apalodimas > --- > dts/Kconfig

Re: [PATCH 2/3] board: arm: Remove OF_PRIOR_STAGE

2021-09-27 Thread Simon Glass
On Mon, 27 Sept 2021 at 00:48, Ilias Apalodimas wrote: > > At some point back in 2018 prior_stage_fdt_address and OF_PRIOR_STAGE got > introduced, in order to support a DTB handed over by an earlier stage boo > loader. However we have another option in the Kconfig (OF_BOARD) which has >

Re: [PATCH 3/4] WS cleanup: remove trailing white space

2021-09-27 Thread Simon Glass
On Mon, 27 Sept 2021 at 09:43, Wolfgang Denk wrote: > nit: commit message > Signed-off-by: Wolfgang Denk > --- > Licenses/lgpl-2.0.txt | 2 +- > Makefile | 4 ++-- > arch/arm/include/asm/arch-am33xx/cpu.h| 2 +- >

Re: [PATCH 3/5] misc: Add Apple DART driver

2021-09-27 Thread Simon Glass
Hi Mark, On Sun, 26 Sept 2021 at 14:53, Mark Kettenis wrote: > > > From: Simon Glass > > Date: Mon, 20 Sep 2021 19:11:25 -0600 > > Hi Simon, > > > [..] > > > > > > > +static int apple_dart_bind(struct udevice *dev) > > > > > +{ > > > > > + void *base; > > > > > + int sid, i; > > > >

Re: [PATCH 1/3] treewide: Remove OF_PRIOR_STAGE from RISC-V boards

2021-09-27 Thread Simon Glass
On Mon, 27 Sept 2021 at 00:48, Ilias Apalodimas wrote: > > At some point back in 2018 prior_stage_fdt_address and OF_PRIOR_STAGE got > introduced, in order to support a DTB handed over by an earlier stage boo > loader. However we have another option in the Kconfig (OF_BOARD) which has >

Re: [PATCH 3/3] cmd: add acmconsole command

2021-09-27 Thread Pali Rohár
On Thursday 19 August 2021 13:13:06 Loic Poulain wrote: > This command allows to start CDC ACM function and redirect console > (stdin, stdout, stderr) to USB (acmconsole start). The console can > then be accessed through the USB host for debugging purpose. The > host can stop the session

Re: [PATCH 2/3] usb: gadget: Add CDC ACM function

2021-09-27 Thread Pali Rohár
On Thursday 19 August 2021 13:13:05 Loic Poulain wrote: > +static void acm_notify_complete(struct usb_ep *ep, struct usb_request *req) > +{ > + if (req->status) > + printf("CDC ACM notify failed (%d)\n", req->status); > + > + /* nothing to do */ > +} > + > +static void

[PATCH] ARM: omap3_logic: Cleanup usage of MUX_VAL

2021-09-27 Thread Adam Ford
The macro called MUX_VAL generates a writel instruction with semicolon at the end. This table was written to use semicolons, however one was missed: MUX_VAL(CP(SYS_BOOT2), (IEN | PTD | DIS | M4))/* GPIO_4 */ Since the extra semicolon is unnecessary with the use of the macro, remove

Re: [PATCH v5 02/29] kconfig: Add tools support to CONFIG_IS_ENABLED()

2021-09-27 Thread Masahiro Yamada
On Tue, Sep 28, 2021 at 1:11 AM Alex G. wrote: > > > > On 9/25/21 8:43 PM, Simon Glass wrote: > > At present we must separately test for the host build for many options, > > since we force them to be enabled. For example, CONFIG_FIT is always > > enabled in the host tools, even if CONFIG_FIT is

[PATCH] drivers: ddr: lc_common_dimm_params.c : Fix Divison by zero issue

2021-09-27 Thread Maninder Singh
Adds check for mclk_ps variable before calculating caslat_actual. Verified changes on lx2160ardb board. Signed-off-by: Maninder Singh --- drivers/ddr/fsl/lc_common_dimm_params.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/ddr/fsl/lc_common_dimm_params.c

Re: [PATCH v5 05/29] hash: Use Kconfig to enable hashing in host tools and SPL

2021-09-27 Thread Alex G.
On 9/27/21 11:06 AM, Simon Glass wrote: Hi Alex, On Mon, 27 Sept 2021 at 09:53, Alex G. wrote: Hi Simon On 9/25/21 8:43 PM, Simon Glass wrote: At present when building host tools, we force CONFIG_SHAxxx to be enabled regardless of the board Kconfig setting. This is done in the image.h

Re: Status of the various RISC-V specification and policy

2021-09-27 Thread Mark Himelstein
the words in this document : https://wiki.riscv.org/plugins/servlet/mobile?contentId=13098230#content/view/13098230 make it very clear when changes are allowed or not and likely or not. if you think the verbiage is somehow ambiguous please help us make it better. Mark sent from a

Re: [PATCH v5 02/29] kconfig: Add tools support to CONFIG_IS_ENABLED()

2021-09-27 Thread Alex G.
On 9/25/21 8:43 PM, Simon Glass wrote: At present we must separately test for the host build for many options, since we force them to be enabled. For example, CONFIG_FIT is always enabled in the host tools, even if CONFIG_FIT is not enabled by the board itself. It would be more convenient if

Re: [PATCH v5 05/29] hash: Use Kconfig to enable hashing in host tools and SPL

2021-09-27 Thread Simon Glass
Hi Alex, On Mon, 27 Sept 2021 at 09:53, Alex G. wrote: > > Hi Simon > > On 9/25/21 8:43 PM, Simon Glass wrote: > > At present when building host tools, we force CONFIG_SHAxxx to be enabled > > regardless of the board Kconfig setting. This is done in the image.h > > header file. > > > > For SPL

Re: [PATCH v5 05/29] hash: Use Kconfig to enable hashing in host tools and SPL

2021-09-27 Thread Alex G.
Hi Simon On 9/25/21 8:43 PM, Simon Glass wrote: At present when building host tools, we force CONFIG_SHAxxx to be enabled regardless of the board Kconfig setting. This is done in the image.h header file. For SPL we currently just assume the algorithm is desired if U-Boot proper enables it.

Re: Bug in board/logicpd/omap3som/omap3logic.h ???

2021-09-27 Thread Wolfgang Denk
Dear Adam, In message you wrote: > > > > > > commit 25e4ff45b17 "ARM: omap3_logic: Enable OMAP EHCI support for > > > SOM-LV Boards" added (among other things) these lines: > > > > > > ... > > > 243 MUX_VAL(CP(MCSPI2_SOMI),(IEN | PTD | DIS | M0)); > > > /*HSUSB2_DATA5*/

Re: Status of the various RISC-V specification and policy

2021-09-27 Thread Palmer Dabbelt
On Tue, 21 Sep 2021 17:20:17 PDT (-0700), ati...@atishpatra.org wrote: Hi All, Please find the below email from Stephano about the freeze announcement for various RISC-V specifications that will be part of privilege specification v1.12. All the review discussions are happening in the isa-dev

[PATCH 1/4] WS cleanup: remove trailing empty lines

2021-09-27 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk --- arch/arm/cpu/arm926ejs/cache.c| 1 - arch/arm/cpu/armv7/psci-common.c | 1 - arch/arm/cpu/armv8/hisilicon/pinmux.c | 2 -- arch/arm/cpu/armv8/xen/hypercall.S

[PATCH 3/4] WS cleanup: remove trailing white space

2021-09-27 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk --- Licenses/lgpl-2.0.txt | 2 +- Makefile | 4 ++-- arch/arm/include/asm/arch-am33xx/cpu.h| 2 +- arch/arm/lib/lib1funcs.S | 6 +++---

[PATCH 2/4] WS cleanup: remove excessive empty lines

2021-09-27 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk --- README| 1 - arch/arm/mach-mvebu/serdes/a38x/high_speed_env_spec.c | 2 -- arch/xtensa/include/asm/arch-dc232b/core.h| 2 -- arch/xtensa/include/asm/arch-dc232b/tie-asm.h | 4

[PATCH 0/4] White Space cleanup

2021-09-27 Thread Wolfgang Denk
The following is an attempt of a minimal-invasive white space cleaup. It performs the following actions: - Remove empty lines at the end of a file - Remove white space at the end of lines - Remove excessive empty lines: there is almost never a good reason to have more than 2 consecutive empty

Re: [PATCH 2/4] dm: hash: Add new UCLASS_HASH support

2021-09-27 Thread Alex G.
On 9/23/21 9:49 PM, Simon Glass wrote:> On Thu, 16 Sept 2021 at 09:43, Alex G. wrote: On 7/29/21 8:08 PM, Chia-Wei Wang wrote: + +enum HASH_ALGO hash_algo_lookup_by_name(const char *name) string -> hash_lookup_algo() -> ops struct Is the current way to do things.

[PATCH v2 17/18] doc: Move distro boot doc to rST

2021-09-27 Thread Simon Glass
Move this over to the new rST format. Signed-off-by: Simon Glass --- (no changes since v1) doc/{README.distro => develop/distro.rst} | 177 ++ doc/develop/index.rst | 1 + 2 files changed, 80 insertions(+), 98 deletions(-) rename doc/{README.distro

[PATCH v2 15/18] pxe: Return the file size from the getfile() function

2021-09-27 Thread Simon Glass
It is pretty strange that the pxe code uses the 'filesize' environment variable find the size of a file it has just read. Partly this is because it uses the command-line interpreter to parse its request to load the file. As a first step towards unwinding this, return it directly from the

[PATCH v2 16/18] pxe: Refactor sysboot to have one helper

2021-09-27 Thread Simon Glass
The only difference between the three helpers is the filesystem type. Factor this out and call the filesystem functions directly, instead of through the command-line interpreter. This allows the file size to be obtained directly, instead of via an environment variable. We cannot do the same thing

[PATCH v2 14/18] lib: Add a function to convert a string to a hex value

2021-09-27 Thread Simon Glass
Add an xtoa() function, similar to itoa() but for hex instead. Signed-off-by: Simon Glass --- (no changes since v1) include/vsprintf.h | 16 ++-- lib/vsprintf.c | 20 ++-- test/print_ut.c| 24 3 files changed, 56 insertions(+), 4

[PATCH v2 13/18] lib: Add tests for simple_itoa()

2021-09-27 Thread Simon Glass
Add test and a comment for this function. Signed-off-by: Simon Glass --- (no changes since v1) include/vsprintf.h | 13 - test/print_ut.c| 17 + 2 files changed, 29 insertions(+), 1 deletion(-) diff --git a/include/vsprintf.h b/include/vsprintf.h index

[PATCH v2 10/18] pxe: Move common parsing coding into pxe_util

2021-09-27 Thread Simon Glass
Both the syslinux and pxe commands use essentially the same code to parse and run extlinux.conf files. Move this into a common function. Signed-off-by: Simon Glass --- (no changes since v1) boot/pxe_utils.c| 20 cmd/pxe.c | 15 --- cmd/sysboot.c

[PATCH v2 11/18] pxe: Clean up the use of bootfile

2021-09-27 Thread Simon Glass
The 'bootfile' environment variable is read in the bowels of pxe_util to provide a directory to which all loaded files are relative. This is not obvious from the API to PXE and it is strange to make the caller set an environment variable rather than pass this as a parameter. The code is also

[PATCH v2 12/18] pxe: Drop get_bootfile_path()

2021-09-27 Thread Simon Glass
This function no longer makes sense, since it is pretty easy to prepend the boot directory to the filename. Drop it and update its only caller. Signed-off-by: Simon Glass --- (no changes since v1) boot/pxe_utils.c| 53 + include/pxe_utils.h | 3

[PATCH v2 09/18] pxe: Tidy up code style a little in pxe_utils

2021-09-27 Thread Simon Glass
There are a few more blank lines than makes sense for readability. Also free() handles a NULL pointer so drop the pointless checks. Signed-off-by: Simon Glass --- (no changes since v1) boot/pxe_utils.c | 66 ++-- 1 file changed, 13 insertions(+), 53

[PATCH v2 18/18] pxe: Allow calling the pxe_get logic directly

2021-09-27 Thread Simon Glass
Refactor this code so that we can call the 'pxe get' command without going through the command-line interpreter. This makes it easier to get the information we need, without going through environment variables. Signed-off-by: Simon Glass --- Changes in v2: - Rebase to -next - Split out from the

[PATCH v2 07/18] pxe: Move pxe_utils files

2021-09-27 Thread Simon Glass
Move the header file into the main include/ directory so we can use it from the bootmethod code. Move the C file into boot/ since it relates to booting. Signed-off-by: Simon Glass --- (no changes since v1) boot/Makefile| 3 +++ {cmd => boot}/pxe_utils.c| 0 cmd/Makefile

[PATCH v2 03/18] pxe: Use a context pointer

2021-09-27 Thread Simon Glass
At present the PXE functions pass around a pointer to command-table entry which is very strange. It is only needed in a few places and it is odd to pass around a data structure from another module in this way. For bootmethod we will need to provide some context information when reading files.

[PATCH v2 05/18] pxe: Add a userdata field to the context

2021-09-27 Thread Simon Glass
Allow the caller to provide some info which is passed back to the readfile() method. Signed-off-by: Simon Glass --- (no changes since v1) cmd/pxe.c | 4 ++-- cmd/pxe_utils.c | 3 ++- cmd/pxe_utils.h | 6 +- cmd/sysboot.c | 2 +- 4 files changed, 10 insertions(+), 5 deletions(-)

[PATCH v2 08/18] pxe: Tidy up some comments in pxe_utils

2021-09-27 Thread Simon Glass
Some of these functions are a big vague in the comments. Tidy them up a bit. Signed-off-by: Simon Glass --- (no changes since v1) boot/pxe_utils.c | 189 ++- 1 file changed, 138 insertions(+), 51 deletions(-) diff --git a/boot/pxe_utils.c

[PATCH v2 02/18] pxe: Move API comments to the header files

2021-09-27 Thread Simon Glass
Put the function comments in the header file so that the full API can we examined in one place. Expand the comments to cover parameters and return values. Signed-off-by: Simon Glass --- (no changes since v1) cmd/pxe_utils.c | 45 - cmd/pxe_utils.h | 77

[PATCH v2 04/18] pxe: Move do_getfile() into the context

2021-09-27 Thread Simon Glass
Rather than having a global variable, pass the function as part of the context. Signed-off-by: Simon Glass --- (no changes since v1) cmd/pxe.c | 10 -- cmd/pxe_utils.c | 9 - cmd/pxe_utils.h | 20 +--- cmd/sysboot.c | 20 ++-- 4 files

[PATCH v2 06/18] pxe: Tidy up the is_pxe global

2021-09-27 Thread Simon Glass
Move this into the context to avoid a global variable. Also rename it since the current name does not explain what it actually affects. Signed-off-by: Simon Glass --- (no changes since v1) cmd/pxe.c | 6 ++ cmd/pxe_utils.c | 13 +++-- cmd/pxe_utils.h | 8 +---

[PATCH v2 01/18] Create a new boot/ directory

2021-09-27 Thread Simon Glass
Quite a lot of the code in common/relates to booting and images. Before adding more it seems like a good time to move the code into its own directory. Most files with 'boot' or 'image' in them are moved, except: - autoboot.c which relates to U-Boot automatically running a script - bootstage.c

[PATCH v2 00/18] pxe: Refactoring to tidy up and prepare for bootflow

2021-09-27 Thread Simon Glass
This collects together the patches previously sent relating to PXE. Firstly, it moves the boot code out of common/ and into a new boot/ directory. This helps to collect these related files in one place, as common/ is quite large. Secondly, it provides patache so clean up the PXE code and

Re: [PATCH] usb: xhci: reset endpoint on USB stall

2021-09-27 Thread Marek Vasut
On 9/27/21 2:42 PM, Stefan Agner wrote: There are devices which cause a USB stall when trying to read strings. Specifically Arduino Mega R3 stalls when trying to read the product string. The stall currently remains unhandled, and subsequent retries submit new transfers on a stopped endpoint

Re: Bug in board/logicpd/omap3som/omap3logic.h ???

2021-09-27 Thread Adam Ford
On Mon, Sep 27, 2021 at 9:49 AM Tom Rini wrote: > > On Mon, Sep 27, 2021 at 04:45:01PM +0200, Wolfgang Denk wrote: > > > Dear Adam, > > > > commit 25e4ff45b17 "ARM: omap3_logic: Enable OMAP EHCI support for > > SOM-LV Boards" added (among other things) these lines: > > > > ... > > 243

Please pull u-boot-dm/next

2021-09-27 Thread Simon Glass
Hi Tom, This is for the -next branch. https://source.denx.de/u-boot/custodians/u-boot-dm/-/pipelines/9260 The following changes since commit bb38d77ca779cc8bdad3d4ceb6cecc687f4987c2: vexpress_ca9x4: Add missing SYS_LOAD_ADDR (2021-09-24 22:12:35 -0400) are available in the Git repository

GSuit limit for recipients

2021-09-27 Thread Oleksandr Suvorov
Dear community, This weekend I tried to post (using patman) a patch set that should have been sent to 120 recipients (a lot of files were changed). I've got an error from the Gmail SMTP server: 4.5.3 Your message has too many recipients. For more information regarding 4.5.3 Google's sending

Re: [PATCH 0/2] ARM: DRA7: Enable OPP_HIGH for GPU, DSPEVE and IVA voltage domains

2021-09-27 Thread Tom Rini
On Mon, Sep 27, 2021 at 09:43:01AM -0500, Suman Anna wrote: > Hi Amjad, > > On 9/27/21 7:34 AM, Amjad Ouled-Ameur wrote: > > This patchset enables the OPP_HIGH configuration for GPU, DSPEVE and IVA > > voltage domains by default for various TI DRA7xx and AM57xx boards. This > > is being done to

Re: Bug in board/logicpd/omap3som/omap3logic.h ???

2021-09-27 Thread Tom Rini
On Mon, Sep 27, 2021 at 04:45:01PM +0200, Wolfgang Denk wrote: > Dear Adam, > > commit 25e4ff45b17 "ARM: omap3_logic: Enable OMAP EHCI support for > SOM-LV Boards" added (among other things) these lines: > > ... > 243 MUX_VAL(CP(MCSPI2_SOMI),(IEN | PTD | DIS | M0)); >

Bug in board/logicpd/omap3som/omap3logic.h ???

2021-09-27 Thread Wolfgang Denk
Dear Adam, commit 25e4ff45b17 "ARM: omap3_logic: Enable OMAP EHCI support for SOM-LV Boards" added (among other things) these lines: ... 243 MUX_VAL(CP(MCSPI2_SOMI),(IEN | PTD | DIS | M0)); /*HSUSB2_DATA5*/ 244 MUX_VAL(CP(MCSPI2_CS0), (IEN | PTD | EN |

Re: [PATCH 0/2] ARM: DRA7: Enable OPP_HIGH for GPU, DSPEVE and IVA voltage domains

2021-09-27 Thread Suman Anna
Hi Amjad, On 9/27/21 7:34 AM, Amjad Ouled-Ameur wrote: > This patchset enables the OPP_HIGH configuration for GPU, DSPEVE and IVA > voltage domains by default for various TI DRA7xx and AM57xx boards. This > is being done to meet the performance needs of 1080p MultiMedia usecases > > Amjad

Re: [PATCH v2 3/3] efi_loader: add DeployedMode and AuditMode variable measurement

2021-09-27 Thread Ilias Apalodimas
On Tue, 21 Sept 2021 at 10:17, Masahisa Kojima wrote: > > This commit adds the DeployedMode and AuditMode variable > measurement required in TCG PC Client PFP Spec. > > Signed-off-by: Masahisa Kojima > --- > > (no changes since v1) > > lib/efi_loader/efi_tcg2.c | 47

[ANN] U-Boot v2021.10-rc5 released

2021-09-27 Thread Tom Rini
Hey all, It is release day and here is v2021.10-rc5. At this point we're a week out from the release. I know of the series here: https://patchwork.ozlabs.org/project/uboot/list/?series=263618=* to restore mtd naming of nor%d to fix a regression within "mtd" for NOR flashes. I think everyone is

Re: Two jobs at once on denx-vulcan?

2021-09-27 Thread Tom Rini
On Fri, Sep 24, 2021 at 05:36:31PM -0600, Simon Glass wrote: > Hi Tom, > > On Fri, 24 Sept 2021 at 08:55, Tom Rini wrote: > > > > On Fri, Sep 24, 2021 at 08:38:49AM -0600, Simon Glass wrote: > > > Hi Tom, > > > > > > On Fri, 24 Sept 2021 at 08:20, Tom Rini wrote: > > > > > > > > On Fri, Sep 24,

Re: [PATCH] nvme: Enable FUA

2021-09-27 Thread Jon Lin
On 2021/9/27 20:28, Stefan Agner wrote: On 2021-09-26 11:12, Jon Lin wrote: Most NVME devcies maintain data in internal cache for an uncertain times, and u-boot has no method to force NVME to flush cache. So this patch adds FUA to avoid data loss caused by power off after data programming.

[PATCH v2 1/2] nvme: Enable FUA

2021-09-27 Thread Jon Lin
Most NVME devcies maintain data in internal cache for an uncertain times, and u-boot has no method to force NVME to flush cache. So this patch adds FUA to avoid data loss caused by power off after data programming. Signed-off-by: Jon Lin Reviewed-by: Stefan Agner --- (no changes since v1)

[PATCH v2 2/2] nvme: Fix error in nvme_setup_prps

2021-09-27 Thread Jon Lin
Consulting to "NVM Express® Base Specification, revision 2.0". If more PRP List pages are required, then the last entry of the PRP List contains the Page Base Address of the next PRP List page. The next PRP List page shall be memory page aligned. Signed-off-by: Jon Lin --- (no changes since

Re: Please pull u-boot-marvell/master

2021-09-27 Thread Tom Rini
On Mon, Sep 27, 2021 at 12:58:43PM +0200, Stefan Roese wrote: > Hi Tom, > > please pull these 2 Marvell MVEBU related fixes: > Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

Re: Pull request for efi-2021-10-rc5

2021-09-27 Thread Tom Rini
On Sun, Sep 26, 2021 at 07:55:27AM +0200, Heinrich Schuchardt wrote: > The following changes since commit 8284d6f838684cfa6e3303a447deb38027c5d7f0: > > Merge branch '2021-09-24-assorted-minor-updates' (2021-09-24 14:28:54 > -0400) > > are available in the Git repository at: > >

[PATCH] usb: xhci: reset endpoint on USB stall

2021-09-27 Thread Stefan Agner
There are devices which cause a USB stall when trying to read strings. Specifically Arduino Mega R3 stalls when trying to read the product string. The stall currently remains unhandled, and subsequent retries submit new transfers on a stopped endpoint which ultimately cause a crash in abort_td():

[PATCH 2/2] ARM: DRA7: Enable OPP_HIGH for GPU voltage domain

2021-09-27 Thread Amjad Ouled-Ameur
Enable the OPP_HIGH configuration for GPU voltage domain by default for various TI DRA7xx and AM57xx boards. This is being done to meet the performance needs of 1080p GFX/MultiMedia usecases. This domain does not support DVFS and the kernel will continue to run at the boot OPP chosen here. Based

[PATCH 1/2] ARM: DRA7: Enable OPP_HIGH for DSPEVE and IVA voltage domains

2021-09-27 Thread Amjad Ouled-Ameur
Enable the OPP_HIGH configuration for DSPEVE and IVA voltage domains by default for various TI DRA7xx and AM57xx boards. This is being done to meet the performance needs of 1080p MultiMedia usecases and other DSP usecases. These domains do not support DVFS and the kernel will continue to run at

[PATCH 0/2] ARM: DRA7: Enable OPP_HIGH for GPU, DSPEVE and IVA voltage domains

2021-09-27 Thread Amjad Ouled-Ameur
This patchset enables the OPP_HIGH configuration for GPU, DSPEVE and IVA voltage domains by default for various TI DRA7xx and AM57xx boards. This is being done to meet the performance needs of 1080p MultiMedia usecases Amjad Ouled-Ameur (2): ARM: DRA7: Enable OPP_HIGH for DSPEVE and IVA voltage

Re: [PATCH] nvme: Enable FUA

2021-09-27 Thread Stefan Agner
On 2021-09-26 11:12, Jon Lin wrote: > Most NVME devcies maintain data in internal cache for an uncertain > times, and u-boot has no method to force NVME to flush cache. > So this patch adds FUA to avoid data loss caused by power off after data > programming. Maybe worth mentioning that FUA stands

Re: [PATCH] board: rpi: always set fdt_addr if provided by firmware

2021-09-27 Thread Oleksandr Suvorov
Hello Matthias, On Mon, Sep 27, 2021 at 3:01 PM Matthias Brugger wrote: > > Hi Oleksandr, > > Thanks for your patch. > > There was the very same patch posted already, please follow-up in that thread > for any comments from your side: >

Re: [PATCH] board: rpi: always set fdt_addr if provided by firmware

2021-09-27 Thread Matthias Brugger
Hi Oleksandr, Thanks for your patch. There was the very same patch posted already, please follow-up in that thread for any comments from your side: https://patchwork.ozlabs.org/project/uboot/patch/20210512123945.25649-1-m.salv...@koansoftware.com/ Regards, Matthias On 25/09/2021 12:22,

[PATCH 18/25] net: davinci_emac: ensure mdiodev->name is NULL terminated after MDIO_NAME_LEN truncation

2021-09-27 Thread Vladimir Oltean
strncpy() simply bails out when copying a source string whose size exceeds the destination string size, potentially leaving the destination string unterminated. One possible way to address is to pass MDIO_NAME_LEN - 1 and a previously zero-initialized destination string, but this is more

[PATCH 23/25] net: dsa: ensure port names are NULL-terminated after DSA_PORT_NAME_LENGTH truncation

2021-09-27 Thread Vladimir Oltean
strncpy() simply bails out when copying a source string whose size exceeds the destination string size, potentially leaving the destination string unterminated. One possible way to address is to pass DSA_PORT_NAME_LENGTH - 1 and a previously zero-initialized destination string, but this is more

[PATCH 24/25] arch: powerpc: mpc85xx: free MDIO bus if mdio_register fails

2021-09-27 Thread Vladimir Oltean
If mdio_register fails, it is nice to not leave behind dangling allocated memory. Signed-off-by: Vladimir Oltean --- arch/powerpc/cpu/mpc85xx/ether_fcc.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/cpu/mpc85xx/ether_fcc.c

[PATCH 25/25] scripts: ensure the cocci script for miiphy_register does not leak the MDIO bus

2021-09-27 Thread Vladimir Oltean
When mdio_register fails, mdio_free should be called on the mdiodev that was previously allocated with mdio_alloc. Signed-off-by: Vladimir Oltean --- scripts/coccinelle/net/mdio_register.cocci | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 22/25] net: dsa: felix: check return code of mdio_alloc and mdio_register

2021-09-27 Thread Vladimir Oltean
These functions can return errors, it's best to catch them and trigger the driver unwind code path. Signed-off-by: Vladimir Oltean --- drivers/net/mscc_eswitch/felix_switch.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/drivers/net/mscc_eswitch/felix_switch.c

  1   2   >