Re: [PATCH] arm: dts: imx8mq-pinfunc: add a define for the SION bit

2022-02-01 Thread Angus Ainslie
On 2022-01-30 05:12, Fabio Estevam wrote: Hi Angus, On Sun, Jan 30, 2022 at 10:10 AM Angus Ainslie wrote: I added this because I wanted to use it in the SPL. I can put it in the header file for my board but I think it would be better in an imx8m header file. If not the pinfunc header is

Re: [PATCH v2 1/2] efi_loader: Avoid using efi_update_capsule() from update capsule on disk

2022-02-01 Thread AKASHI Takahiro
On Tue, Feb 01, 2022 at 10:33:20PM +0530, Sughosh Ganu wrote: > On Tue, 1 Feb 2022 at 22:14, Heinrich Schuchardt wrote: > > > > > > > > Am 1. Februar 2022 16:42:43 MEZ schrieb Sughosh Ganu > > : > > >hi Masami, > > > > > >On Tue, 1 Feb 2022 at 14:03, Masami Hiramatsu > > > wrote: > > >> > > >>

Re: [PATCH v2 1/2] efi_loader: Avoid using efi_update_capsule() from update capsule on disk

2022-02-01 Thread Masami Hiramatsu
Hi Sughosh, Could you tell me why do you need to do the FWU code in the efi_update_capsule? If you need to add some logic to both of the efi_update_capsule API and capsule-on-disk, it is better to be implemented in the efi_capsule_update_firmware() as a common part. Or, make an independent

Re: [PATCH] arm: dts: imx8mq-pinfunc: add a define for the SION bit

2022-02-01 Thread Fabio Estevam
On Tue, Feb 1, 2022 at 4:05 PM Angus Ainslie wrote: > So it turns out it's already defined here > > drivers/pinctrl/nxp/pinctrl-imx.h > > Could that definition be moved here ? > > arch/arm/include/asm/mach-imx/iomux-v3.h Yes, that would work too.

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

2022-02-01 Thread Tom Rini
On Tue, Feb 01, 2022 at 12:54:17PM +0100, Stefan Roese wrote: > Hi Tom, > > please pull this CFI flash related patch: > Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature

[PATCH 08/19] dm: add event notification

2022-02-01 Thread AKASHI Takahiro
From: Simon Glass This is a draft implementation of event notification mechanism from Simon. Under this scheme, any U-Boot subsystem can register some kind of callback function to a particular event (more event types will be added later) and that function will be invoked once the event is fired.

[PATCH 10/19] dm: tag: add some document

2022-02-01 Thread AKASHI Takahiro
Some basic stuff about tag support is explained under doc/devlop/driver-model. Signed-off-by: AKASHI Takahiro --- doc/develop/driver-model/design.rst | 20 1 file changed, 20 insertions(+) diff --git a/doc/develop/driver-model/design.rst

[PATCH 12/19] dm: disk: add UCLASS_PARTITION

2022-02-01 Thread AKASHI Takahiro
With this new function, UCLASS_PARTITION devices will be created as child nodes of UCLASS_BLK device. Signed-off-by: AKASHI Takahiro --- disk/Makefile | 3 + disk/disk-uclass.c | 153 + include/dm/uclass-id.h | 1 + include/part.h

[PATCH 09/19] dm: add tag support

2022-02-01 Thread AKASHI Takahiro
With dm-tag feature, any U-Boot subsystem is allowed to associate arbitrary number of data with a particular udevice. This can been see as expanding "struct udevice" without modifying the definition. As a first user, UEFI subsystem makes use of tags to associate an efi_disk object with a block

[PATCH 11/19] test: dm: add tests for tag support

2022-02-01 Thread AKASHI Takahiro
The new test covers all tag-related interfaces. Signed-off-by: AKASHI Takahiro --- test/dm/Makefile | 1 + test/dm/tag.c| 80 2 files changed, 81 insertions(+) create mode 100644 test/dm/tag.c diff --git a/test/dm/Makefile

Re: [PATCH v2 1/2] efi_loader: Avoid using efi_update_capsule() from update capsule on disk

2022-02-01 Thread Sughosh Ganu
On Wed, 2 Feb 2022 at 05:17, AKASHI Takahiro wrote: > > On Tue, Feb 01, 2022 at 10:33:20PM +0530, Sughosh Ganu wrote: > > On Tue, 1 Feb 2022 at 22:14, Heinrich Schuchardt wrote: > > > > > > > > > > > > Am 1. Februar 2022 16:42:43 MEZ schrieb Sughosh Ganu > > > : > > > >hi Masami, > > > > > > >

Re: [PATCH v2 2/2] efi_loader: Reset system after CapsuleUpdate on disk

2022-02-01 Thread Masami Hiramatsu
Hi Takahiro, 2022年2月2日(水) 13:15 AKASHI Takahiro : > > On Wed, Feb 02, 2022 at 10:53:05AM +0900, Masami Hiramatsu wrote: > > Hi Takahiro, > > > > 2022年2月1日(火) 20:38 AKASHI Takahiro : > > > > > > > > On Tue, Feb 01, 2022 at 05:33:09PM +0900, Masami Hiramatsu wrote: > > > > Add a config option to

Re: [PATCH 1/7] clk: Make rfree return void

2022-02-01 Thread Simon Glass
Hi Sean, On Tue, 1 Feb 2022 at 07:49, Sean Anderson wrote: > > On 1/27/22 4:35 PM, Simon Glass wrote: > > Hi Sean, > > > > On Thu, 27 Jan 2022 at 08:43, Sean Anderson wrote: > >> > >> On 1/27/22 10:05 AM, Simon Glass wrote: > >>> Hi Sean, > >>> > >>> On Sat, 15 Jan 2022 at 15:25, Sean Anderson

pull request of u-boot-fsl-qoriq for v2022.04

2022-02-01 Thread Priyanka Jain
Dear Tom, Please find my pull-request for u-boot-fsl-qoriq/master https://github.com/u-boot/u-boot/pull/107 Summary Update and fixes for ls1088a, FMAN, ls1046ardb, ls1043ardb sync ls1028ardb u-boot and Linux device-tree Regards Priyanka The following changes since commit

[PATCH 01/19] scsi: call device_probe() after scanning

2022-02-01 Thread AKASHI Takahiro
Every time a scsi bus/port is scanned and a new block device is detected, we want to call device_probe() as it will give us a chance to run additional post-processings for some purposes. In particular, support for creating partitions on a device will be added. Signed-off-by: AKASHI Takahiro

[PATCH 07/19] virtio: call device_probe() in scanning

2022-02-01 Thread AKASHI Takahiro
virtio_init() enumerates all the peripherals that are to be materialised with udevices(UCLASS_VIRIO) and creates particular device instances (UCLASS_BlK or whatever else) as children. On the other hand, device_probe() won't be invoked against those resultant udevices unlike other ordinary device

[PATCH 05/19] sata: call device_probe() after scanning

2022-02-01 Thread AKASHI Takahiro
Every time a sata bus/port is scanned and a new device is detected, we want to call device_probe() as it will give us a chance to run additional post-processings for some purposes. In particular, support for creating partitions on a device will be added. Signed-off-by: AKASHI Takahiro

[PATCH 02/19] usb: storage: call device_probe() after scanning

2022-02-01 Thread AKASHI Takahiro
Every time a usb bus/port is scanned and a new device is detected, we want to call device_probe() as it will give us a chance to run additional post-processings for some purposes. In particular, support for creating partitions on a device will be added. Signed-off-by: AKASHI Takahiro

[PATCH 03/19] mmc: call device_probe() after scanning

2022-02-01 Thread AKASHI Takahiro
Every time a mmc bus/port is scanned and a new device is detected, we want to call device_probe() as it will give us a chance to run additional post-processings for some purposes. In particular, support for creating partitions on a device will be added. Signed-off-by: AKASHI Takahiro ---

[PATCH 04/19] nvme: call device_probe() after scanning

2022-02-01 Thread AKASHI Takahiro
Every time a nvme bus/port is scanned and a new device is detected, we want to call device_probe() as it will give us a chance to run additional post-processings for some purposes. In particular, support for creating partitions on a device will be added. Signed-off-by: AKASHI Takahiro

[PATCH 06/19] block: ide: call device_probe() after scanning

2022-02-01 Thread AKASHI Takahiro
Every time an ide bus/port is scanned and a new device is detected, we want to call device_probe() as it will give us a chance to run additional post-processings for some purposes. In particular, support for creating partitions on a device will be added. Signed-off-by: AKASHI Takahiro

[PATCH 15/19] efi_loader: disk: a helper function to delete efi_disk objects

2022-02-01 Thread AKASHI Takahiro
This function is expected to be called, in particular from dm's pre_remove hook, when associated block devices no longer exist. Add efi_disk_remove() function. This function removes an efi_disk object for a raw disk device (UCLASS_BLK) and related objects for its partitions (UCLASS_PARTITION).

[PATCH 14/19] efi_loader: disk: a helper function to create efi_disk objects from udevice

2022-02-01 Thread AKASHI Takahiro
Add efi_disk_probe() function. This function creates an efi_disk object for a raw disk device (UCLASS_BLK) and additional objects for related partitions (UCLASS_PARTITION). So this function is expected to be called through driver model's "probe" interface every time one raw disk device is

[PATCH 16/19] dm: disk: add read/write interfaces with udevice

2022-02-01 Thread AKASHI Takahiro
In include/blk.h, Simon suggested: ===> /* * These functions should take struct udevice instead of struct blk_desc, * but this is convenient for migration to driver model. Add a 'd' prefix * to the function operations, so that blk_read(), etc. can be reserved for * functions with the correct

[PATCH 17/19] efi_loader: disk: use udevice instead of blk_desc

2022-02-01 Thread AKASHI Takahiro
In most of all cases, we can avoid using blk_desc which is expected to be private to udevice(UCLASS_BLK), that is, the data should not be manupulated outside the device driver unless really needed. Now efi_disk's internally use dev_read/write() interfaces. Signed-off-by: AKASHI Takahiro

[PATCH 13/19] dm: blk: add a device-probe hook for scanning disk partitions

2022-02-01 Thread AKASHI Takahiro
Now that all the block device drivers have enable a probe hook, we will call part_create_block_devices() to enumerate all the partitions and create associated udevices when a block device is detected. Signed-off-by: AKASHI Takahiro Reviewed-by: Simon Glass --- drivers/block/blk-uclass.c | 4

[PATCH 19/19] efi_driver: align with efi_disk-dm integration

2022-02-01 Thread AKASHI Takahiro
With DM-efi_disk integration, we don't need to explicitly call efi_disk_create_partitions(). The only thing to do is to associate an efi_disk object to the corresponding udevice as we skip most of processing in efi_disk_probe() by the previous commit ("efi_loader: disk: not create BLK device for

[PATCH 18/19] efi_loader: disk: not create BLK device for BLK(IF_TYPE_EFI_LOADER) devices

2022-02-01 Thread AKASHI Takahiro
When we create an efi_disk device with an UEFI application using driver binding protocol, the 'efi_driver' framework tries to create a corresponding block device(UCLASS_BLK/IF_TYPE_EFI). This will lead to calling a PROBE callback, efi_disk_probe(). In this case, however, we don't need to create

Re: [PATCH v2 2/2] efi_loader: Reset system after CapsuleUpdate on disk

2022-02-01 Thread Masami Hiramatsu
Hi Takahiro, 2022年2月1日(火) 20:38 AKASHI Takahiro : > > On Tue, Feb 01, 2022 at 05:33:09PM +0900, Masami Hiramatsu wrote: > > Add a config option to reset system soon after processing capsule update > > on disk. This is required in UEFI specification 2.9 Section 8.5.5 > > "Delivery of Capsules

Re: [PATCH 1/7] clk: Make rfree return void

2022-02-01 Thread Sean Anderson
On 2/1/22 10:59 PM, Simon Glass wrote: Hi Sean, On Tue, 1 Feb 2022 at 07:49, Sean Anderson wrote: On 1/27/22 4:35 PM, Simon Glass wrote: Hi Sean, On Thu, 27 Jan 2022 at 08:43, Sean Anderson wrote: On 1/27/22 10:05 AM, Simon Glass wrote: Hi Sean, On Sat, 15 Jan 2022 at 15:25, Sean

[PATCH 00/19] efi_loader: more tightly integrate UEFI disks to driver model

2022-02-01 Thread AKASHI Takahiro
Background: === The purpose of this patch is to reignite the discussion about how UEFI subystem would best be integrated into U-Boot driver model. In the past, I poposed a couple of patch series, the latest one[1], while Heinrich revealed his idea[2], and the approach taken here is

RE: [PATCH 00/10] board: sl28: add sl28cpld support and board cleanups

2022-02-01 Thread Priyanka Jain
>-Original Message- >From: Tom Rini >Sent: Monday, January 31, 2022 8:56 PM >To: Michael Walle ; Priyanka Jain >Cc: u-boot@lists.denx.de >Subject: Re: [PATCH 00/10] board: sl28: add sl28cpld support and board >cleanups > >On Mon, Jan 31, 2022 at 08:51:39AM +0100, Michael Walle wrote:

Re: [PATCH v2 1/2] efi_loader: Avoid using efi_update_capsule() from update capsule on disk

2022-02-01 Thread Masami Hiramatsu
Hi Sughosh, 2022年2月2日(水) 14:35 Sughosh Ganu : > > hi Masami, > > On Wed, 2 Feb 2022 at 05:39, Masami Hiramatsu > wrote: > > > > Hi Sughosh, > > > > Could you tell me why do you need to do the FWU code in the > > efi_update_capsule? > > I thought I explained this in my previous email. Putting

Re: [PATCH v2 1/2] efi_loader: Avoid using efi_update_capsule() from update capsule on disk

2022-02-01 Thread Sughosh Ganu
hi Masami, On Wed, 2 Feb 2022 at 05:39, Masami Hiramatsu wrote: > > Hi Sughosh, > > Could you tell me why do you need to do the FWU code in the > efi_update_capsule? I thought I explained this in my previous email. Putting the FWU checks in efi_update_capsule caters to the scenario where FWU

[PATCH] arm: kirkwood: iConnect : Add Ethernet support

2022-02-01 Thread Tony Dinh
- Currently, CONFIG_RESET_PHY_R symbol is used in arch/arm/mach-kirkwood/include/mach/config.h for all Kirkwood boards with mv8831116 PHY, with each board defines the function reset_phy(). Undefine it for this board. - Add board_eth_init(), CONFIG_DM_ETH, and CONFIG_PHY_MARVELL to bring up

[PATCH v4] sf: Query write-protection status before operating the flash

2022-02-01 Thread Jan Kiszka
From: Jan Kiszka Do not suggest successful operation if a flash area to be changed is actually locked, thus will not execute the request. Rather report an error and bail out. That's way more user-friendly than asking them to manually check for this case. Derived from original patch by Chao

Re: [PATCH v2 2/2] efi_loader: Reset system after CapsuleUpdate on disk

2022-02-01 Thread AKASHI Takahiro
On Wed, Feb 02, 2022 at 10:53:05AM +0900, Masami Hiramatsu wrote: > Hi Takahiro, > > 2022年2月1日(火) 20:38 AKASHI Takahiro : > > > > > On Tue, Feb 01, 2022 at 05:33:09PM +0900, Masami Hiramatsu wrote: > > > Add a config option to reset system soon after processing capsule update > > > on disk. This

Re: [PATCH] arm: dts: imx8mq kernel dts updates

2022-02-01 Thread Angus Ainslie
Hi Fabio, On 2022-01-31 17:55, Fabio Estevam wrote: Hi Angus, On Mon, Jan 17, 2022 at 5:43 PM Angus Ainslie wrote: --- /dev/null +++ b/include/dt-bindings/rfkill/rfkill.h @@ -0,0 +1,21 @@ +/* SPDX-License-Identifier: GPL-2.0-only or MIT */ +/* + * This header provides constants for rfkill

[PATCH] mtd: nand: Add support to dedicated function to set timings

2022-02-01 Thread Kory Maincent
With the current code if the board have a NAND ONFI compliant without support to the get and set features, U-boot return an ENOTSUP error and we can not talk to the memory. Indeed onfi_set_features return ENOTSUP error if set/get features is not supported. In the case of timings we should not

Re: [PATCH] mtd: nand: Add support to dedicated function to set timings

2022-02-01 Thread Miquel Raynal
Hi Köry, kory.mainc...@bootlin.com wrote on Tue, 1 Feb 2022 18:45:14 +0100: Perhaps the title prefix should be "mtd: rawnand:". I am making a number of reworking proposals below, take what you like. > With the current code if the board have a NAND ONFI compliant without

Re: [PATCH v2 2/2] Makefile: Don't allow new boards with SPL_FIT_GENERATOR

2022-02-01 Thread Mark Kettenis
> From: Simon Glass > Date: Tue, 1 Feb 2022 08:42:35 -0700 > > It seems that rk3399 uses bl31.elf and splits out the sections into > pieces. What a mess! I wonder if that is necessary for ATF to work? It > seems to do the same for TEE. That's because bl31.elf really consists of three binary

Re: [PATCH v2 2/2] Makefile: Don't allow new boards with SPL_FIT_GENERATOR

2022-02-01 Thread Simon Glass
Hi Mark, On Tue, 1 Feb 2022 at 09:08, Mark Kettenis wrote: > > > From: Simon Glass > > Date: Tue, 1 Feb 2022 08:42:35 -0700 > > > > It seems that rk3399 uses bl31.elf and splits out the sections into > > pieces. What a mess! I wonder if that is necessary for ATF to work? It > > seems to do the

Re: [PATCH v2 1/2] efi_loader: Avoid using efi_update_capsule() from update capsule on disk

2022-02-01 Thread Sughosh Ganu
On Tue, 1 Feb 2022 at 22:14, Heinrich Schuchardt wrote: > > > > Am 1. Februar 2022 16:42:43 MEZ schrieb Sughosh Ganu > : > >hi Masami, > > > >On Tue, 1 Feb 2022 at 14:03, Masami Hiramatsu > > wrote: > >> > >> The efi_update_capsule() may have to handle the capsule flags as an UEFI > >> runtime

Re: Early debug UART not working on AM33XX SoC

2022-02-01 Thread Felix Brack
Hello Simon On 01.02.22 15:05, Simon Glass wrote: > Hi Felix, > > On Tue, 1 Feb 2022 at 03:48, Felix Brack wrote: >> >> Hello Simon, >> >> On 31.01.22 17:12, Simon Glass wrote: >>> Hi Felix, >>> >>> On Mon, 31 Jan 2022 at 02:43, Felix Brack wrote: Hello Simon On 27.01.22

Re: [PATCH v2 1/2] efi_loader: Avoid using efi_update_capsule() from update capsule on disk

2022-02-01 Thread Heinrich Schuchardt
Am 1. Februar 2022 16:42:43 MEZ schrieb Sughosh Ganu : >hi Masami, > >On Tue, 1 Feb 2022 at 14:03, Masami Hiramatsu > wrote: >> >> The efi_update_capsule() may have to handle the capsule flags as an UEFI >> runtime and boottime service, but the capsule-on-disk process doesn't. >> Thus, the

Re: Early debug UART not working on AM33XX SoC

2022-02-01 Thread Felix Brack
Hello Simon, On 01.02.22 15:05, Simon Glass wrote: > Hi Felix, > > On Tue, 1 Feb 2022 at 03:48, Felix Brack wrote: >> >> Hello Simon, >> >> On 31.01.22 17:12, Simon Glass wrote: >>> Hi Felix, >>> >>> On Mon, 31 Jan 2022 at 02:43, Felix Brack wrote: Hello Simon On 27.01.22

[PATCH v2 2/2] efi_loader: Reset system after CapsuleUpdate on disk

2022-02-01 Thread Masami Hiramatsu
Add a config option to reset system soon after processing capsule update on disk. This is required in UEFI specification 2.9 Section 8.5.5 "Delivery of Capsules via file on Mass Storage device" as; In all cases that a capsule is identified for processing the system is restarted after

[PATCH v2 1/2] efi_loader: Avoid using efi_update_capsule() from update capsule on disk

2022-02-01 Thread Masami Hiramatsu
The efi_update_capsule() may have to handle the capsule flags as an UEFI runtime and boottime service, but the capsule-on-disk process doesn't. Thus, the capsule-on-disk should use the efi_capsule_update_firmware() directly instead of efi_update_capsule(). Suggested-by: AKASHI Takahiro

[PATCH v2 0/2] EFI: Reset system after capsule-on-disk

2022-02-01 Thread Masami Hiramatsu
Hi, Here is the patch to reset after capsule-on-disk. This version fixes some bugs and remove kconfig for the reset (which uses panic). The reset after completing the capsule-on-disk is stated in the UEFI specification 2.9, section 8.5.5 "Delivery of Capsules via file on Mass Storage device" as

Re: Early debug UART not working on AM33XX SoC

2022-02-01 Thread Felix Brack
Hello Simon, On 31.01.22 17:12, Simon Glass wrote: > Hi Felix, > > On Mon, 31 Jan 2022 at 02:43, Felix Brack wrote: >> >> Hello Simon >> >> On 27.01.22 18:33, Simon Glass wrote: >>> Hi Felix, >>> >>> On Thu, 27 Jan 2022 at 09:27, Felix Brack wrote: Hello Simon, On 27.01.22

Re: [PATCH v4 4/4] phytec: phycore_imx8mp: override env_get_location

2022-02-01 Thread Tommaso Merciai
On Tue, Feb 01, 2022 at 04:16:52AM +0100, Marek Vasut wrote: > On 1/31/22 23:15, Tommaso Merciai wrote: > > On Mon, Jan 31, 2022 at 06:03:58PM +0100, Marek Vasut wrote: > > > On 1/31/22 17:58, Tommaso Merciai wrote: > > > > Override env_get_location function at board level, previously dropped > >

Re: Question on running uboot_testpy with tbot

2022-02-01 Thread Harald Seiler
Hi Simon, On Thu, 2022-01-27 at 09:05 -0700, Simon Glass wrote: > Hi Harald, > [...] > > Thanks for all the info. I tried master and it mostly works, although > Iget the same error: > [...] > │ File >

Re: [PATCH] mtd: cfi_mtd: populate mtd->dev with flash_info->dev

2022-02-01 Thread Stefan Roese
On 12/3/21 10:27, Patrice Chotard wrote: Populate mtd->dev with flash_info->dev which allows to get full mtd information using the "mtd list" command. Before, "mtd list" command returns : List of MTD devices: * nor0 - type: NOR flash - block size: 0x4 bytes - min I/O: 0x1 bytes

Re: [PATCH v4 4/4] phytec: phycore_imx8mp: override env_get_location

2022-02-01 Thread Adam Ford
On Tue, Feb 1, 2022 at 5:22 AM Marek Vasut wrote: > > On 2/1/22 12:16, Adam Ford wrote: > > On Mon, Jan 31, 2022 at 9:18 PM Marek Vasut wrote: > >> > >> On 2/1/22 01:20, Adam Ford wrote: > >>> On Mon, Jan 31, 2022 at 4:16 PM Tommaso Merciai > >>> wrote: > > On Mon, Jan 31, 2022 at

Re: [PATCH v4 4/4] phytec: phycore_imx8mp: override env_get_location

2022-02-01 Thread Marek Vasut
On 2/1/22 12:23, Adam Ford wrote: On Tue, Feb 1, 2022 at 5:22 AM Marek Vasut wrote: On 2/1/22 12:16, Adam Ford wrote: On Mon, Jan 31, 2022 at 9:18 PM Marek Vasut wrote: On 2/1/22 01:20, Adam Ford wrote: On Mon, Jan 31, 2022 at 4:16 PM Tommaso Merciai wrote: On Mon, Jan 31, 2022 at

Re: [PATCH v4 4/4] phytec: phycore_imx8mp: override env_get_location

2022-02-01 Thread Adam Ford
On Tue, Feb 1, 2022 at 5:46 AM Marek Vasut wrote: > > On 2/1/22 12:22, Adam Ford wrote: > > On Tue, Feb 1, 2022 at 3:09 AM Tommaso Merciai > > wrote: > >> > >> On Tue, Feb 01, 2022 at 04:16:52AM +0100, Marek Vasut wrote: > >>> On 1/31/22 23:15, Tommaso Merciai wrote: > On Mon, Jan 31, 2022

Issue with spi-uclass.c and mxc_spi.c when DM_SPI_FLASH is enabled

2022-02-01 Thread Abder
Hi everyone, Recently I've been working on the upgrade of an old u-boot to u-boot2020, and I encountered an issue when I enabled DM_SPI and DM_SPI_FLASH ! My board (based on IMX6) uses a NOR flash SPI to store the u-boot environment and the SPI interface is initialized early in boot. Using the

[PATCH] video: stm32: stm32_ltdc: fix the check of return value of clk_set_rate()

2022-02-01 Thread Patrick Delaunay
From: Gabriel Fernandez The clk_set_rate() function returns rate as an 'ulong' not an 'int' and rate > 0 by default. This patch avoids to display the associated warning when the set rate function returns the new frequency. Fixes: aeaf330649e8 ("video: stm32: stm32_ltdc: add bridge to display

Re: [PATCH v4 4/4] phytec: phycore_imx8mp: override env_get_location

2022-02-01 Thread Marek Vasut
On 2/1/22 10:09, Tommaso Merciai wrote: On Tue, Feb 01, 2022 at 04:16:52AM +0100, Marek Vasut wrote: On 1/31/22 23:15, Tommaso Merciai wrote: On Mon, Jan 31, 2022 at 06:03:58PM +0100, Marek Vasut wrote: On 1/31/22 17:58, Tommaso Merciai wrote: Override env_get_location function at board

Re: [PATCH v4 4/4] phytec: phycore_imx8mp: override env_get_location

2022-02-01 Thread Adam Ford
On Tue, Feb 1, 2022 at 3:09 AM Tommaso Merciai wrote: > > On Tue, Feb 01, 2022 at 04:16:52AM +0100, Marek Vasut wrote: > > On 1/31/22 23:15, Tommaso Merciai wrote: > > > On Mon, Jan 31, 2022 at 06:03:58PM +0100, Marek Vasut wrote: > > > > On 1/31/22 17:58, Tommaso Merciai wrote: > > > > >

Re: [PATCH v4 4/4] phytec: phycore_imx8mp: override env_get_location

2022-02-01 Thread Marek Vasut
On 2/1/22 12:16, Adam Ford wrote: On Mon, Jan 31, 2022 at 9:18 PM Marek Vasut wrote: On 2/1/22 01:20, Adam Ford wrote: On Mon, Jan 31, 2022 at 4:16 PM Tommaso Merciai wrote: On Mon, Jan 31, 2022 at 06:03:58PM +0100, Marek Vasut wrote: On 1/31/22 17:58, Tommaso Merciai wrote: Override

Re: [PATCH v4 4/4] phytec: phycore_imx8mp: override env_get_location

2022-02-01 Thread Marek Vasut
On 2/1/22 12:51, Adam Ford wrote: On Tue, Feb 1, 2022 at 5:46 AM Marek Vasut wrote: On 2/1/22 12:22, Adam Ford wrote: On Tue, Feb 1, 2022 at 3:09 AM Tommaso Merciai wrote: On Tue, Feb 01, 2022 at 04:16:52AM +0100, Marek Vasut wrote: On 1/31/22 23:15, Tommaso Merciai wrote: On Mon, Jan

Please pull u-boot-cfi-flash/master

2022-02-01 Thread Stefan Roese
Hi Tom, please pull this CFI flash related patch: - cfi_mtd: populate mtd->dev with flash_info->dev (Patrice) Here the Azure build, without any issues:

Re: Issue with spi-uclass.c and mxc_spi.c when DM_SPI_FLASH is enabled

2022-02-01 Thread Fabio Estevam
Hi Abder, On Tue, Feb 1, 2022 at 9:31 AM Abder wrote: > > Hi everyone, > > Recently I've been working on the upgrade of an old u-boot to u-boot2020, U-Boot 2020.x is old too :-) Please try the latest U-Boot and report if you have issues.

Re: [PATCH v4 4/4] phytec: phycore_imx8mp: override env_get_location

2022-02-01 Thread Adam Ford
On Mon, Jan 31, 2022 at 9:18 PM Marek Vasut wrote: > > On 2/1/22 01:20, Adam Ford wrote: > > On Mon, Jan 31, 2022 at 4:16 PM Tommaso Merciai > > wrote: > >> > >> On Mon, Jan 31, 2022 at 06:03:58PM +0100, Marek Vasut wrote: > >>> On 1/31/22 17:58, Tommaso Merciai wrote: > Override

Re: [PATCH v2 2/2] efi_loader: Reset system after CapsuleUpdate on disk

2022-02-01 Thread AKASHI Takahiro
On Tue, Feb 01, 2022 at 05:33:09PM +0900, Masami Hiramatsu wrote: > Add a config option to reset system soon after processing capsule update > on disk. This is required in UEFI specification 2.9 Section 8.5.5 > "Delivery of Capsules via file on Mass Storage device" as; > > In all cases that

Re: [PATCH v4 4/4] phytec: phycore_imx8mp: override env_get_location

2022-02-01 Thread Marek Vasut
On 2/1/22 12:22, Adam Ford wrote: On Tue, Feb 1, 2022 at 3:09 AM Tommaso Merciai wrote: On Tue, Feb 01, 2022 at 04:16:52AM +0100, Marek Vasut wrote: On 1/31/22 23:15, Tommaso Merciai wrote: On Mon, Jan 31, 2022 at 06:03:58PM +0100, Marek Vasut wrote: On 1/31/22 17:58, Tommaso Merciai

Re: [PATCH] mtd: cfi_mtd: populate mtd->dev with flash_info->dev

2022-02-01 Thread Stefan Roese
On 12/3/21 10:27, Patrice Chotard wrote: Populate mtd->dev with flash_info->dev which allows to get full mtd information using the "mtd list" command. Before, "mtd list" command returns : List of MTD devices: * nor0 - type: NOR flash - block size: 0x4 bytes - min I/O: 0x1 bytes

Re: [PATCH v4 4/4] phytec: phycore_imx8mp: override env_get_location

2022-02-01 Thread Marek Vasut
On 2/1/22 12:56, Tommaso Merciai wrote: On Tue, Feb 01, 2022 at 05:22:06AM -0600, Adam Ford wrote: On Tue, Feb 1, 2022 at 3:09 AM Tommaso Merciai wrote: On Tue, Feb 01, 2022 at 04:16:52AM +0100, Marek Vasut wrote: On 1/31/22 23:15, Tommaso Merciai wrote: On Mon, Jan 31, 2022 at 06:03:58PM

[PATCH] board: st: common: update test on misc_read result in command stboard

2022-02-01 Thread Patrick Delaunay
Update management of misc_read/misc_write, which now returns length of data after the commit 8729b1ae2cbd ("misc: Update read() and write() methods to return bytes xfered"): raise a error when the result is not the expected length. Fixes: 658fde8a36ff ("board: stm32mp1: stboard: lock the OTP

Re: [PATCH v4 4/4] phytec: phycore_imx8mp: override env_get_location

2022-02-01 Thread Tommaso Merciai
On Tue, Feb 01, 2022 at 05:22:06AM -0600, Adam Ford wrote: > On Tue, Feb 1, 2022 at 3:09 AM Tommaso Merciai > wrote: > > > > On Tue, Feb 01, 2022 at 04:16:52AM +0100, Marek Vasut wrote: > > > On 1/31/22 23:15, Tommaso Merciai wrote: > > > > On Mon, Jan 31, 2022 at 06:03:58PM +0100, Marek Vasut

Re: [PATCH v4 4/4] phytec: phycore_imx8mp: override env_get_location

2022-02-01 Thread Adam Ford
On Tue, Feb 1, 2022 at 5:58 AM Marek Vasut wrote: > > On 2/1/22 12:56, Tommaso Merciai wrote: > > On Tue, Feb 01, 2022 at 05:22:06AM -0600, Adam Ford wrote: > >> On Tue, Feb 1, 2022 at 3:09 AM Tommaso Merciai > >> wrote: > >>> > >>> On Tue, Feb 01, 2022 at 04:16:52AM +0100, Marek Vasut wrote: >

Re: [PATCH v4 4/4] phytec: phycore_imx8mp: override env_get_location

2022-02-01 Thread Marek Vasut
On 2/1/22 13:26, Adam Ford wrote: On Tue, Feb 1, 2022 at 5:58 AM Marek Vasut wrote: On 2/1/22 12:56, Tommaso Merciai wrote: On Tue, Feb 01, 2022 at 05:22:06AM -0600, Adam Ford wrote: On Tue, Feb 1, 2022 at 3:09 AM Tommaso Merciai wrote: On Tue, Feb 01, 2022 at 04:16:52AM +0100, Marek

Re: [PATCH 4/4] cmd: clk: fix long help message

2022-02-01 Thread Sean Anderson
On 1/31/22 11:21 AM, Patrick Delaunay wrote: Fix the long help message for "clk setfreq" command Fixes: 7ab418fbe612 ("clk: add support for setting clk rate from cmdline") Signed-off-by: Patrick Delaunay --- cmd/clk.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH] clk: ccf: correct the test on the parent uclass in clk_enable/clk_disable

2022-02-01 Thread Sean Anderson
On 1/24/22 8:17 AM, Patrick Delaunay wrote: It is safe to check if the uclass id on the device is UCLASS_CLK before to call the clk_ functions, but today this comparison is not done on the device used in API: clkp->dev->parent but on the device himself: clkp->dev. This patch corrects this

Re: [PATCH] clk: update clk_clean_rate_cache to use private clk struct

2022-02-01 Thread Sean Anderson
On 1/31/22 10:18 AM, Patrick Delaunay wrote: In clk_clean_rate_cache, clk->rate should update the private clock struct, in particular when CCF is activated, to save the cached rate value. When clk_get_parent_rate is called, the cached information is read from pclk->rate, with pclk =

Re: [PATCH v2 2/2] Makefile: Don't allow new boards with SPL_FIT_GENERATOR

2022-02-01 Thread Simon Glass
-Philipp Hi Tom, On Tue, 1 Feb 2022 at 07:05, Simon Glass wrote: > > Hi Tom, > > On Mon, 31 Jan 2022 at 16:32, Tom Rini wrote: > > > > On Mon, Jan 31, 2022 at 06:25:33PM -0500, Tom Rini wrote: > > > On Mon, Jan 31, 2022 at 03:59:08PM -0700, Simon Glass wrote: > > > > Hi Tom, > > > > > > > >

Re: [PATCH v2 1/2] efi_loader: Avoid using efi_update_capsule() from update capsule on disk

2022-02-01 Thread Sughosh Ganu
hi Masami, On Tue, 1 Feb 2022 at 14:03, Masami Hiramatsu wrote: > > The efi_update_capsule() may have to handle the capsule flags as an UEFI > runtime and boottime service, but the capsule-on-disk process doesn't. > Thus, the capsule-on-disk should use the efi_capsule_update_firmware() >

Re: Early debug UART not working on AM33XX SoC

2022-02-01 Thread Simon Glass
Hi Felix, On Tue, 1 Feb 2022 at 03:48, Felix Brack wrote: > > Hello Simon, > > On 31.01.22 17:12, Simon Glass wrote: > > Hi Felix, > > > > On Mon, 31 Jan 2022 at 02:43, Felix Brack wrote: > >> > >> Hello Simon > >> > >> On 27.01.22 18:33, Simon Glass wrote: > >>> Hi Felix, > >>> > >>> On Thu,

Re: [PATCH v2 2/2] Makefile: Don't allow new boards with SPL_FIT_GENERATOR

2022-02-01 Thread Simon Glass
Hi Tom, On Mon, 31 Jan 2022 at 16:32, Tom Rini wrote: > > On Mon, Jan 31, 2022 at 06:25:33PM -0500, Tom Rini wrote: > > On Mon, Jan 31, 2022 at 03:59:08PM -0700, Simon Glass wrote: > > > Hi Tom, > > > > > > (yes Mark I am trying to stop further boards going in that use the > > > shell scripts) >

Re: [PATCH 3/4] cmd: clk: update result of do_clk_setfreq

2022-02-01 Thread Sean Anderson
On 1/31/22 11:21 AM, Patrick Delaunay wrote: Update the result of do_clk_setfreq and always returns a CMD_RET_ value (-EINVAL was a possible result). This patch avoid the CLI output "exit not allowed from main input shell." Fixes: 7ab418fbe612 ("clk: add support for setting clk rate from

Re: [PATCH 2/4] cmd: clk: replace clk_lookup by uclass_get_device_by_name

2022-02-01 Thread Sean Anderson
On 1/31/22 11:21 AM, Patrick Delaunay wrote: The function clk_lookup can be replaced by a direct call to uclass_get_device_by_name for UCLASS_CLK. This patch removes duplicated codes by the generic DM API and avoids issue in clk_lookup because result of uclass_get_device wasn't tested; when ret

Re: [PATCH 1/7] clk: Make rfree return void

2022-02-01 Thread Sean Anderson
On 1/27/22 4:35 PM, Simon Glass wrote: Hi Sean, On Thu, 27 Jan 2022 at 08:43, Sean Anderson wrote: On 1/27/22 10:05 AM, Simon Glass wrote: Hi Sean, On Sat, 15 Jan 2022 at 15:25, Sean Anderson wrote: When freeing a clock there is not much we can do if there is an error, and most callers