Re: [PATCH 30/31] tools: mtk_image: add support for nand headers used by newer chips

2022-08-07 Thread Weijie Gao
On Thu, 2022-08-04 at 07:57 -0600, Simon Glass wrote: > Hi Weijie, > > On Wed, 3 Aug 2022 at 21:41, Weijie Gao > wrote: > > > > This patch adds more nand headers in two new types: > > 1. HSM header, used for spi-nand thru SNFI interface > > 2. SPIM header, used for spi-nand thru spi-mem

Re: [PATCH 29/31] tools: mtk_image: split the code of generating NAND header into a new file

2022-08-07 Thread Weijie Gao
On Fri, 2022-08-05 at 20:26 +0200, Daniel Golle wrote: > Hi Weijie, > > please see one comment in-line below: > > On Thu, Aug 04, 2022 at 11:36:50AM +0800, Weijie Gao wrote: > > The predefined NAND headers take too much spaces in the > > mtk_image.c. > > Moving them into a new file can

Re: [PATCH 29/31] tools: mtk_image: split the code of generating NAND header into a new file

2022-08-07 Thread Weijie Gao
On Thu, 2022-08-04 at 07:57 -0600, Simon Glass wrote: > Hi Weijie, > > On Wed, 3 Aug 2022 at 21:37, Weijie Gao > wrote: > > > > The predefined NAND headers take too much spaces in the > > mtk_image.c. > > Moving them into a new file can significantly improve the > > readability of > > both

Re: [PATCH 27/31] clk: mediatek: add clock driver support for MediaTek MT7981 SoC

2022-08-07 Thread Weijie Gao
On Thu, 2022-08-04 at 07:57 -0600, Simon Glass wrote: > Hi Weijie, > > On Wed, 3 Aug 2022 at 21:40, Weijie Gao > wrote: > > > > This patch adds clock driver support for MediaTek MT7981 SoC > > > > Signed-off-by: Weijie Gao > > --- > > drivers/clk/mediatek/Makefile | 1 + > >

Re: [PATCH 26/31] clk: mediatek: add clock driver support for MediaTek MT7986 SoC

2022-08-07 Thread Weijie Gao
On Thu, 2022-08-04 at 07:57 -0600, Simon Glass wrote: > Hi Weijie, > > On Wed, 3 Aug 2022 at 21:40, Weijie Gao > wrote: > > > > This patch adds clock driver support for MediaTek MT7986 SoC > > > > Signed-off-by: Weijie Gao > > --- > > drivers/clk/mediatek/Makefile | 1 + > >

Re: [PATCH 25/31] clk: mediatek: add CLK_XTAL support for clock driver

2022-08-07 Thread Weijie Gao
On Thu, 2022-08-04 at 07:57 -0600, Simon Glass wrote: > Hi Weijie, > > On Wed, 3 Aug 2022 at 21:40, Weijie Gao > wrote: > > > > This add CLK_XTAL macro and flag to mediatek clock driver common > > part, > > to make thi SoC that has clock directlly connect to XTAL working. > > > >

Re: [PATCH 21/23] blk: Switch over to using uclass IDs

2022-08-07 Thread AKASHI Takahiro
My comment below is not directly related to the change, but On Sun, Aug 07, 2022 at 09:47:06AM -0600, Simon Glass wrote: > We currently have an if_type (interface type) and a uclass id. These are > closely related and we don't need to have both. > > Drop the if_type values and use the uclass

Re: [PATCH 22/31] clk: mediatek: add CLK_BYPASS_XTAL flag to allow bypassing searching clock parent of xtal clock

2022-08-07 Thread Weijie Gao
On Thu, 2022-08-04 at 07:57 -0600, Simon Glass wrote: > Hi Weijie, > > On Wed, 3 Aug 2022 at 21:39, Weijie Gao > wrote: > > > > The mtk clock framework in u-boot uses array index for searching > > clock > > parent (kernel uses strings for search), so we need to specify a > > special > > clock

Re: [PATCH 17/31] i2c: add support for MediaTek I2C interface

2022-08-07 Thread Weijie Gao
On Thu, 2022-08-04 at 07:57 -0600, Simon Glass wrote: > Hi Weijie, > > On Wed, 3 Aug 2022 at 21:38, Weijie Gao > wrote: > > > > This patch adds support for MediaTek I2C interface > > > > Signed-off-by: Weijie Gao > > --- > > drivers/i2c/Kconfig | 9 + > > drivers/i2c/Makefile | 1 + >

Re: [PATCH 10/31] serial: mtk: add support for using dynamic baud clock souce

2022-08-07 Thread Weijie Gao
On Thu, 2022-08-04 at 07:56 -0600, Simon Glass wrote: > Hi Weijie, > > On Wed, 3 Aug 2022 at 21:36, Weijie Gao > wrote: > > > > The baud clock on some platform may change due to assigned-clock- > > parent > > set in DT. In current flow the baud clock is only retrieved during > > probe > >

Re: [PATCH 07/31] net: mediatek: stop using bitfileds for DMA descriptors

2022-08-07 Thread Weijie Gao
On Sat, 2022-08-06 at 20:50 +0300, Ramon Fried wrote: > On Thu, Aug 4, 2022 at 6:35 AM Weijie Gao > wrote: > > > > This patch is a preparation for adding a new version of PDMA of > > which the > > DMA descriptor fields has changed. Using bitfields will result in a > > complex > > modification.

Re: [PATCH 06/31] net: mediatek: use a struct to cover variations of all SoCs

2022-08-07 Thread Weijie Gao
On Thu, 2022-08-04 at 07:56 -0600, Simon Glass wrote: > Hi Weijie, > > On Wed, 3 Aug 2022 at 21:36, Weijie Gao > wrote: > > > > Using a single soc id to control different initialization and TX/RX > > flow > > for all SoCs is not extensible if more hardware variations are > > added in > > the

Re: [PATCH 05/31] mmc: mediatek: add support for MediaTek MT7891/MT7986 SoCs

2022-08-07 Thread Weijie Gao
On Thu, 2022-08-04 at 07:57 -0600, Simon Glass wrote: > Hi Weijie, > > On Wed, 3 Aug 2022 at 21:35, Weijie Gao > wrote: > > > > This patch adds eMMC and SDXC support for MediaTek MT7981/MT7986 > > SoCs > > Add eMMC and SDXC support for MediaTek MT7981/MT7986 SoCs. > > (describe your changes

Re: [PATCH 1/1] usb: storage: stop probe on "Invalid device"

2022-08-07 Thread AKASHI Takahiro
Hi Simon, On Sun, Aug 07, 2022 at 09:47:56AM -0600, Simon Glass wrote: > Hi Marek, > > On Thu, 21 Jul 2022 at 11:29, Janne Grunau wrote: > > > > Fixes a crash during probing of sd card readers without medium present. > > > > Link: https://github.com/AsahiLinux/linux/issues/44 > > Signed-off-by:

Re: [PATCH 02/31] arm: mediatek: add support for MediaTek MT7981 SoC

2022-08-07 Thread Weijie Gao
On Thu, 2022-08-04 at 07:57 -0600, Simon Glass wrote: > Hi Weijie, > > On Wed, 3 Aug 2022 at 21:35, Weijie Gao > wrote: > > > > This patch adds basic support for MediaTek MT7981 SoC. > > This include the file that will initialize the SoC after boot and > > its > > device tree. > > > >

Re: [PATCH 23/23] blk: Rename if_type to uclass_id

2022-08-07 Thread AKASHI Takahiro
On Mon, Aug 08, 2022 at 10:32:16AM +0900, AKASHI Takahiro wrote: > My comment below is not directly related to the change, but Another comment, > On Sun, Aug 07, 2022 at 09:47:08AM -0600, Simon Glass wrote: > > Use the word 'uclass' instead of 'if_type' to complete the conversion. > > > >

Re: [PATCH 01/31] arm: mediatek: add support for MediaTek MT7986 SoC

2022-08-07 Thread Weijie Gao
Hi Daniel, There is a design change between MT7986(and later chips) and MT7622: The bootrom of MT7622 requires the device header offset be set to the absolute address of the BL2 in the flash device. For SPI-NOR/SPI-NAND and eMMC, the offset is always zero. But for SD devices, BL2 is stored in

Re: [PATCH 23/23] blk: Rename if_type to uclass_id

2022-08-07 Thread AKASHI Takahiro
My comment below is not directly related to the change, but On Sun, Aug 07, 2022 at 09:47:08AM -0600, Simon Glass wrote: > Use the word 'uclass' instead of 'if_type' to complete the conversion. > > Signed-off-by: Simon Glass > --- > > board/st/common/stm32mp_dfu.c| 2 +- >

Re: [PATCH 10/23] blk: Rename HAVE_BLOCK_DEVICE

2022-08-07 Thread AKASHI Takahiro
On Sun, Aug 07, 2022 at 09:46:55AM -0600, Simon Glass wrote: > This option is fact really related to SPL. For U-Boot proper we always use > driver model for block devices, so CONFIG_BLK is enabled if block devices > are in use. > > It is only for SPL that we have two cases: > > - SPL_BLK is

Re: [PATCH] ubifs: Allow to silence debug dumps

2022-08-07 Thread Tony Dinh
Hi Pali, On Sun, Aug 7, 2022 at 3:54 PM Pali Rohár wrote: > > On Sunday 07 August 2022 15:30:33 Tony Dinh wrote: > > Hi Pali, > > > > On Sun, Aug 7, 2022 at 12:27 PM Pali Rohár wrote: > > > > > > Debug dump logs are not always required. Add a new config option > > > UBIFS_SILENCE_DEBUG_DUMP to

Re: [PATCH] ubifs: Allow to silence debug dumps

2022-08-07 Thread Pali Rohár
On Sunday 07 August 2022 15:30:33 Tony Dinh wrote: > Hi Pali, > > On Sun, Aug 7, 2022 at 12:27 PM Pali Rohár wrote: > > > > Debug dump logs are not always required. Add a new config option > > UBIFS_SILENCE_DEBUG_DUMP to silence all debug dumps. On powerpc/mpc85xx > > when enabled this will

[PATCH 2/2] binman: Add more documentation about binman usage

2022-08-07 Thread Simon Glass
This is an attempt to answer the comments provided by Xavier [1]. [1] https://lore.kernel.org/all/Yulcol7HpTHtjXTX@begut/ Signed-off-by: Simon Glass --- doc/Makefile| 1 + tools/binman/binman.rst | 166 2 files changed, 167 insertions(+)

[PATCH 1/2] binman: Add rST references for binman entry types

2022-08-07 Thread Simon Glass
Add references in the documentation for each entry type, so we can refer to them from other documentation. Signed-off-by: Simon Glass --- tools/binman/entries.rst | 146 ++- tools/binman/entry.py| 5 ++ 2 files changed, 150 insertions(+), 1 deletion(-)

Re: [PATCH] ubifs: Allow to silence debug dumps

2022-08-07 Thread Tony Dinh
Hi Pali, On Sun, Aug 7, 2022 at 12:27 PM Pali Rohár wrote: > > Debug dump logs are not always required. Add a new config option > UBIFS_SILENCE_DEBUG_DUMP to silence all debug dumps. On powerpc/mpc85xx > when enabled this will decrease size of U-Boot binary by 11 kB. Great patch! I had a hunch

[PATCH] cmd: ubifs: Do not show usage when command fails

2022-08-07 Thread Pali Rohár
Return value -1 cause U-Boot to print usage message. Return value 1 (CMD_RET_FAILURE) indicates failure. So fix return value when ubifs command starts it execution and fails. Signed-off-by: Pali Rohár --- cmd/ubifs.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

Re: [PATCH] misc: atsha204a: Don't check for error when waking up the device

2022-08-07 Thread Pali Rohár
Paweł, could you please test this change if it works on your board? I was that you were fixing another wakeup issue in patch "misc: atsha204a: Increase wake delay by tWHI". On Thursday 04 August 2022 13:03:44 Pali Rohár wrote: > The device ignores any levels or transitions on the SCL pin when the

[PATCH] ubifs: Allow to silence debug dumps

2022-08-07 Thread Pali Rohár
Debug dump logs are not always required. Add a new config option UBIFS_SILENCE_DEBUG_DUMP to silence all debug dumps. On powerpc/mpc85xx when enabled this will decrease size of U-Boot binary by 11 kB. Signed-off-by: Pali Rohár --- fs/ubifs/Kconfig | 8 fs/ubifs/debug.c | 28

[PATCH] hwconfig: Allow to use restricted env

2022-08-07 Thread Pali Rohár
During early boot phase GD_FLG_ENV_READY is not set but env_get() may work when env is ready in restricted mode. Do not fail with error message "WARNING: Calling __hwconfig without a buffer and before environment is ready" when env is already working by checking for ENV_VALID flag. Signed-off-by:

[PATCH] distroboot: ubifs: Add support for specifying UBI header offset

2022-08-07 Thread Pali Rohár
Some UBI partitions may use non-standard UBI header offset. For attaching these UBI partitions it is required to pass second argument with offset to "ubi part" command. Therefore extend distroboot to allow specifying additional optional 6th argument with UBI header offset. This offset is set in

Re: [SPAM] Re: Replace make-fit-atf.py with binman. Was: migrate u-boot-rockchip.bin to binman and generate an image for SPI

2022-08-07 Thread Simon Glass
Hi Xavier, On Tue, 2 Aug 2022 at 11:19, Xavier Drudis Ferran wrote: > > El Tue, Aug 02, 2022 at 06:41:40AM -0600, Simon Glass deia: > > > > It seems we need a lot more guidance here. I can help write more into > > the packaging docs, perhaps: > > > >

Re: [PATCH] arm: k3: Correct an awk warning

2022-08-07 Thread Simon Glass
Hi Tom, On Sun, 7 Aug 2022 at 07:29, Tom Rini wrote: > > On Sun, Aug 07, 2022 at 07:12:19AM -0600, Simon Glass wrote: > > > The k3_gen_x509_cert.sh script produced this warning on gitlab and also > > on my machine, e.g. with j7200_evm_r5: > > > > awk: cmd. line:1: warning: regexp escape sequence

Re: [PATCH 21/21] net: Make DM_ETH be selected by NETDEVICE

2022-08-07 Thread Simon Glass
Hi Tom, On Sun, 7 Aug 2022 at 10:39, Tom Rini wrote: > > On Sun, Aug 07, 2022 at 09:47:54AM -0600, Simon Glass wrote: > > Hi Tom, > > > > On Sat, 6 Aug 2022 at 14:02, Tom Rini wrote: > > > > > > On Sat, Aug 06, 2022 at 12:21:21PM -0600, Simon Glass wrote: > > > > Hi Tom, > > > > > > > > On Tue,

Re: [PATCH 21/21] net: Make DM_ETH be selected by NETDEVICE

2022-08-07 Thread Tom Rini
On Sun, Aug 07, 2022 at 09:47:54AM -0600, Simon Glass wrote: > Hi Tom, > > On Sat, 6 Aug 2022 at 14:02, Tom Rini wrote: > > > > On Sat, Aug 06, 2022 at 12:21:21PM -0600, Simon Glass wrote: > > > Hi Tom, > > > > > > On Tue, 2 Aug 2022 at 05:35, Tom Rini wrote: > > > > > > > > The deadline for

[PATCH 21/23] blk: Switch over to using uclass IDs

2022-08-07 Thread Simon Glass
We currently have an if_type (interface type) and a uclass id. These are closely related and we don't need to have both. Drop the if_type values and use the uclass ones instead. Maintain the existing, subtle, one-way conversion between UCLASS_USB and UCLASS_MASS_STORAGE for now, and add a

[PATCH 00/23] blk: Rationalise the block interface

2022-08-07 Thread Simon Glass
The block interface has two separate implementations, one using driver model and one not. The latter is really only needed for SPL, where size constraints allegedly don't allow use of driver model. Of course we still need space for filesystems and other code, so it isn't clear that driver model is

[PATCH v7] board: purism: add the Purism Librem5 phone

2022-08-07 Thread Angus Ainslie
Initial commit of Librem5 u-boot and SPL All of the pre-requisite patches for this board are now upstream. Changes since v6: Move migrated symbols Rebase onto latest u-boot-imx Changes since v5: Proper handling of the Display Port firmware for CI builds Update the DP section of the docs

Re: [PATCH v3 2/2] ci: Add a test for a non-LTO build

2022-08-07 Thread Simon Glass
Hi Tom, On Wed, 3 Aug 2022 at 12:13, Simon Glass wrote: > > Check that sandbox builds and runs tests OK with LTO disabled. > > Signed-off-by: Simon Glass > --- > > (no changes since v1) > > .azure-pipelines.yml | 4 > .gitlab-ci.yml | 7 +++ > 2 files changed, 11 insertions(+) >

[PATCH 23/23] blk: Rename if_type to uclass_id

2022-08-07 Thread Simon Glass
Use the word 'uclass' instead of 'if_type' to complete the conversion. Signed-off-by: Simon Glass --- board/st/common/stm32mp_dfu.c| 2 +- cmd/blk_common.c | 18 +-- cmd/lsblk.c | 2 +- cmd/mmc.c

Re: [PATCH 1/1] block: fix blk_get_devnum_by_typename()

2022-08-07 Thread Simon Glass
Hi Heinrich, On Thu, 4 Aug 2022 at 00:06, Heinrich Schuchardt wrote: > > > > On 8/3/22 20:14, Simon Glass wrote: > > Hi Heinrich, > > > > On Tue, 2 Aug 2022 at 10:22, Heinrich Schuchardt > > wrote: > >> > >> > >> > >> On 8/2/22 14:41, Simon Glass wrote: > >>> Hi Heinrich, > >>> > >>> On Tue, 2

[PATCH 22/23] blk: Drop if_type

2022-08-07 Thread Simon Glass
Use the uclass ID instead. Signed-off-by: Simon Glass --- cmd/blk_common.c | 2 +- drivers/block/blk-uclass.c | 34 +++ drivers/block/blk_legacy.c | 20 +-- include/blk.h | 41 ++

Re: [PATCH v2] boot: allow bootmeth-distro without CONFIG_NET

2022-08-07 Thread Simon Glass
Hi, On Sun, 7 Aug 2022 at 07:10, Simon Glass wrote: > > On Thu, 28 Jul 2022 at 04:19, John Keeping wrote: > > > > Remove the dependency on CMD_PXE from BOOTMETH_DISTRO by introducing a > > new hidden kconfig symbol to control whether pxe_utils is compiled, > > allowing bootstd's distro method

Re: [PATCH 1/1] usb: storage: stop probe on "Invalid device"

2022-08-07 Thread Simon Glass
Hi Marek, On Thu, 21 Jul 2022 at 11:29, Janne Grunau wrote: > > Fixes a crash during probing of sd card readers without medium present. > > Link: https://github.com/AsahiLinux/linux/issues/44 > Signed-off-by: Janne Grunau > --- > common/usb_storage.c | 4 +--- > 1 file changed, 1 insertion(+),

Re: [PATCH 21/21] net: Make DM_ETH be selected by NETDEVICE

2022-08-07 Thread Simon Glass
Hi Tom, On Sat, 6 Aug 2022 at 14:02, Tom Rini wrote: > > On Sat, Aug 06, 2022 at 12:21:21PM -0600, Simon Glass wrote: > > Hi Tom, > > > > On Tue, 2 Aug 2022 at 05:35, Tom Rini wrote: > > > > > > The deadline for DM_ETH migration passed 2 years ago. Now that > > > platforms which cannot be

[PATCH 20/23] efi: Correct assumption about if_type

2022-08-07 Thread Simon Glass
efi_set_blk_dev_to_system_partition() assumes that 0 is an invalid if_type. This is true now but is about to be false. Fix this bug to avoid a test failure. Signed-off-by: Simon Glass --- lib/efi_loader/efi_var_file.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 17/23] blk: Drop IF_TYPE_SD

2022-08-07 Thread Simon Glass
This is not really needed since it does the same things as MMC. Drop it. Signed-off-by: Simon Glass --- disk/part.c| 2 -- drivers/block/blk-uclass.c | 2 -- include/blk.h | 1 - 3 files changed, 5 deletions(-) diff --git a/disk/part.c b/disk/part.c index

[PATCH 16/23] blk: Drop IF_TYPE_ATAPI

2022-08-07 Thread Simon Glass
This is not really needed since it does the same things as IDE. Drop it. Signed-off-by: Simon Glass --- disk/part.c| 5 - drivers/block/blk-uclass.c | 2 -- include/blk.h | 1 - 3 files changed, 8 deletions(-) diff --git a/disk/part.c b/disk/part.c index

[PATCH 19/23] blk: Rewrite if_type to name functions

2022-08-07 Thread Simon Glass
These are currently using a simple array lookup in one direction, and relying on if_type being sequential. With the move to uclass IDs this needs to change. Update the code to prepare for the new way. This patch is intended to introduce no functional change. The returning of "(none)" from

[PATCH 15/23] ide: Use a flag for an ATAPI device

2022-08-07 Thread Simon Glass
Rather than setting a different interface type, use a flag to indicate that a device is ATAPI. Signed-off-by: Simon Glass --- drivers/block/ide.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/block/ide.c b/drivers/block/ide.c index 3270a9f032f..6bc48e3402e

[PATCH 18/23] blk: Rename var in blk_get_devnum_by_typename()

2022-08-07 Thread Simon Glass
At present we use a variable with the same name as the enum. Change this since we plan to #define the enum to uclass_id. Signed-off-by: Simon Glass --- drivers/block/blk-uclass.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/block/blk-uclass.c

[PATCH 14/23] blk: Drop IF_TYPE_DOC

2022-08-07 Thread Simon Glass
This doesn't seem to be used for anything and it isn't clear what it is. It dates from the first U-Boot commit. Drop it. Signed-off-by: Simon Glass --- disk/part.c| 9 - drivers/block/blk-uclass.c | 2 -- include/blk.h | 1 - 3 files changed, 12

[PATCH 12/23] blk: Drop unnecessary CONFIG_SPL_LEGACY_BLOCK in defconfigs

2022-08-07 Thread Simon Glass
This is defined automatically when needed, so drop it from the few defconfig files that define it manually. Signed-off-by: Simon Glass --- configs/axm_defconfig | 1 - configs/bcm968380gerg_ram_defconfig| 1 - configs/gardena-smart-gateway-mt7688_defconfig

[PATCH 11/23] blk: Select SPL_LEGACY_BLOCK automatically

2022-08-07 Thread Simon Glass
Selecting this option can be handled in the Kconfig option itself, as it is with BLK. Update this an drop the various 'select' clauses. Signed-off-by: Simon Glass --- common/spl/Kconfig | 2 -- drivers/ata/Kconfig| 1 - drivers/block/Kconfig | 11 +++ drivers/mmc/Kconfig

[PATCH 13/23] blk: Hide the BLK and SPL_LEGACY_BLOCK options

2022-08-07 Thread Simon Glass
We don't want boards to be able to change these. They can be handled as dependencies of options that need them, such as SPL_MMC. There is no point in enabling the block interface without any storage devices to create a block device. Hide both options from the 'menuconfig' display and deny their

[PATCH 10/23] blk: Rename HAVE_BLOCK_DEVICE

2022-08-07 Thread Simon Glass
This option is fact really related to SPL. For U-Boot proper we always use driver model for block devices, so CONFIG_BLK is enabled if block devices are in use. It is only for SPL that we have two cases: - SPL_BLK is enabled, in which case we use driver model and blk-uclass.c - SPL_BLK is not

[PATCH 09/23] blk: Drop unnecessary #ifdef in in blk_legacy

2022-08-07 Thread Simon Glass
We can rely on the compiler to eliminate any dead code. Signed-off-by: Simon Glass --- drivers/block/blk_legacy.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/block/blk_legacy.c b/drivers/block/blk_legacy.c index df9a3597545..bd8a17df6a9 100644 --- a/drivers/block/blk_legacy.c

[PATCH 08/23] cmd: Drop use of HAVE_BLOCK_DEVICE

2022-08-07 Thread Simon Glass
This condition is not needed for these commands, since BLK is enabled for all boards which use block devices and commands are not available in SPL, so even if SPL_BLK is not enabled, it doesn't affect commands. Signed-off-by: Simon Glass --- cmd/Kconfig | 5 - cmd/Makefile | 2 +- 2 files

[PATCH 07/23] blk: Use a function for whether block devices are available

2022-08-07 Thread Simon Glass
At present we use HAVE_BLOCK_DEVICE to indicate when block devices are available. This is a very strange option, since it partially duplicates the BLK option used by driver model. It also covers both U-Boot proper and SPL, even though one might have block devices and another not. As a first step

[PATCH 06/23] disk: Use Makefile to omit partition drivers

2022-08-07 Thread Simon Glass
At present these files have an #ifdef covering the whole file. Move the condition to the Makefile instead. Add BLK to the condition since future patches will adjust things so that HAVE_BLOCK_DEVICE is only for SPL, but the partition drivers are needed in U-Boot proper too. Signed-off-by: Simon

[PATCH 05/23] sandbox: Avoid defining HAVE_BLOCK_DEVICE in Konfig

2022-08-07 Thread Simon Glass
This is not needed as it is implied or selected by other options anyway. Signed-off-by: Simon Glass --- arch/Kconfig | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/Kconfig b/arch/Kconfig index 6495e780fec..d3c5d86fb5b 100644 --- a/arch/Kconfig +++ b/arch/Kconfig @@ -144,7 +144,6 @@

[PATCH 03/23] blk: Enable CONFIG_BLK for all media

2022-08-07 Thread Simon Glass
Enable this option on all boards which support block devices. Drop the related depencies on BLK since these are not needed anymore. Disable BLOCK_CACHE on M5253DEMO as this causes a build error. Signed-off-by: Simon Glass --- arch/arm/Kconfig| 1 - configs/M5253DEMO_defconfig | 1

[PATCH 04/23] ata: Fix an instance of SPL_SATA_SUPPORT

2022-08-07 Thread Simon Glass
The _SUPPORT prefix should be dropped. This happened because the rename was applied around the same time as this new option, so did not include renaming the new option. The relevant commits are: f7560376ae sata: Rename SATA_SUPPORT to SATA 73059529b2 ata: ahci-pci: Add new option

[PATCH 02/23] disk: Correct help for TPL_PARTITIONS

2022-08-07 Thread Simon Glass
Fix a few typos in this help text. Signed-off-by: Simon Glass --- disk/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/disk/Kconfig b/disk/Kconfig index 81d8867ed7f..bc1c263613e 100644 --- a/disk/Kconfig +++ b/disk/Kconfig @@ -32,9 +32,9 @@ config TPL_PARTITIONS

[PATCH 01/23] binman: Put fake files in a subdirectory

2022-08-07 Thread Simon Glass
At present fake files from a previous build appear to be real files for a subsequent build, since they sit in the output directory. This can cause problems, since binman may need to parse the file, e.g. with the Intel description.bin files. Fix this by putting them in a 'binman-fake'

Re: rk3399 TPL memory setup code triggers clock frequency limit assertion

2022-08-07 Thread Michal Suchánek
On Sun, Aug 07, 2022 at 08:31:56PM +0530, Jagan Teki wrote: > On Sun, Aug 7, 2022 at 8:14 PM Michal Suchánek wrote: > > > > Hello, > > > > when compiled with clock debug rk3399 cannot be booted because memory > > setup code triggers clock assertion: > > > > U-Boot TPL

Re: rk3399 TPL memory setup code triggers clock frequency limit assertion

2022-08-07 Thread Jagan Teki
On Sun, Aug 7, 2022 at 8:14 PM Michal Suchánek wrote: > > Hello, > > when compiled with clock debug rk3399 cannot be booted because memory > setup code triggers clock assertion: > > U-Boot TPL 2022.07-00038-g61e11a8e9f-dirty (Aug 07 2022 - 16:13:17) > TPL PLL at ff76: fbdiv=50, refdiv=1,

rk3399 TPL memory setup code triggers clock frequency limit assertion

2022-08-07 Thread Michal Suchánek
Hello, when compiled with clock debug rk3399 cannot be booted because memory setup code triggers clock assertion: U-Boot TPL 2022.07-00038-g61e11a8e9f-dirty (Aug 07 2022 - 16:13:17) TPL PLL at ff76: fbdiv=50, refdiv=1, postdiv1=2, postdiv2=1, vco=120 khz, output=60 khz TPL PLL at

Re: [RESEND PATCH v3 1/1] mx7ulp: add base SPL support for mx7ulp

2022-08-07 Thread Oleksandr Suvorov
Hi Stefano, This patchset is superseded by that one https://patchwork.ozlabs.org/project/uboot/list/?series=312792 On Mon, Jul 11, 2022 at 5:05 PM Oleksandr Suvorov wrote: > > From: Ricardo Salveti > > Add a base implementation of mx7ulp SPL config header and soc, > and changes in makefiles in

Re: Have fsl_elbc_nand controllers subpage write support?

2022-08-07 Thread Pali Rohár
On Sunday 07 August 2022 14:00:27 Pali Rohár wrote: > Hello Scott! > > In past you disabled nand subpage write support for freescale eLBC and > IFC U-Boot nand drivers in following patch: > https://lore.kernel.org/u-boot/20121102234432.ga18...@buserror.net/ > > But in Linux kernel you disabled

Re: [PATCH] arm: k3: Correct an awk warning

2022-08-07 Thread Tom Rini
On Sun, Aug 07, 2022 at 07:12:19AM -0600, Simon Glass wrote: > The k3_gen_x509_cert.sh script produced this warning on gitlab and also > on my machine, e.g. with j7200_evm_r5: > > awk: cmd. line:1: warning: regexp escape sequence `\ ' is not a known >regexp operator > > There is no need to

[PATCH 2/2] patman: Add documentation to doc/

2022-08-07 Thread Simon Glass
Link to patman's documentation from the doc/ directory so that it appears in the 'make htmldocs' output. Signed-off-by: Simon Glass --- doc/develop/index.rst | 1 + doc/develop/patman.rst | 1 + doc/develop/sending_patches.rst | 16 + tools/patman/README.rst

[PATCH 1/2] doc: Drop a reference to travis

2022-08-07 Thread Simon Glass
This was widely used by U-Boot for a long time, but is not used anymore, with Gitlab and Azure taking over. Drop the text. Signed-off-by: Simon Glass --- doc/develop/codingstyle.rst | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/doc/develop/codingstyle.rst

[PATCH] arm: k3: Correct an awk warning

2022-08-07 Thread Simon Glass
The k3_gen_x509_cert.sh script produced this warning on gitlab and also on my machine, e.g. with j7200_evm_r5: awk: cmd. line:1: warning: regexp escape sequence `\ ' is not a known regexp operator There is no need to escape spaces, so drop the backslashes. Also split the line so it is a more

Re: [PATCH 2/7] dtoc: Tidy up fdt_tests RunTests()

2022-08-07 Thread Simon Glass
Pass the options args in rather than using the global variables. Use snake case, fix up comments and use a ternary operator to make pylint happy. Signed-off-by: Simon Glass --- tools/dtoc/test_fdt.py | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) Applied to u-boot-dm,

Re: [PATCH v2] boot: allow bootmeth-distro without CONFIG_NET

2022-08-07 Thread Simon Glass
On Thu, 28 Jul 2022 at 04:19, John Keeping wrote: > > Remove the dependency on CMD_PXE from BOOTMETH_DISTRO by introducing a > new hidden kconfig symbol to control whether pxe_utils is compiled, > allowing bootstd's distro method to be compiled without needing > networking support enabled. > >

Re: [PATCH 1/7] dtoc: Tidy up fdt_tests RunTestCoverage() args

2022-08-07 Thread Simon Glass
Pass the options args in rather than using the global various. Use snake case and fix up comments to make pylint happy. Signed-off-by: Simon Glass --- tools/dtoc/test_fdt.py | 12 1 file changed, 8 insertions(+), 4 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 3/7] dtoc: Fix fdt test coverage

2022-08-07 Thread Simon Glass
Fix a bug that the --processes option was ignored, thus resulting in no test coverage information being generated. Signed-off-by: Simon Glass Fixes: 42ae363ddd9 ("dtoc: Update fdt tests to use test_util") --- tools/dtoc/test_fdt.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Applied

Re: [PATCH 4/7] dtoc: Move main program into its own function

2022-08-07 Thread Simon Glass
Use a function for the main program so everything there doesn't look like a global variable to pylint. Signed-off-by: Simon Glass --- tools/dtoc/test_fdt.py | 44 +++--- 1 file changed, 24 insertions(+), 20 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 5/7] test_fdt: Convert to use argparse

2022-08-07 Thread Simon Glass
Drop the deprecated OptionParser. Signed-off-by: Simon Glass --- tools/dtoc/test_fdt.py | 39 --- 1 file changed, 20 insertions(+), 19 deletions(-) Applied to u-boot-dm, thanks!

Re: [PATCH 6/7] dtoc: Correct remaining pylint problems in test_fdt

2022-08-07 Thread Simon Glass
Fix various camel-case and other naming problems. Update the pylint base file to avoid regressions. Signed-off-by: Simon Glass --- scripts/pylint.base| 2 +- tools/dtoc/test_fdt.py | 254 +++-- 2 files changed, 142 insertions(+), 114 deletions(-)

Have fsl_elbc_nand controllers subpage write support?

2022-08-07 Thread Pali Rohár
Hello Scott! In past you disabled nand subpage write support for freescale eLBC and IFC U-Boot nand drivers in following patch: https://lore.kernel.org/u-boot/20121102234432.ga18...@buserror.net/ But in Linux kernel you disabled it only in freescale IFC nand driver:

Re: [BISECTED] BeagleBone Black doesn't boot after a58147c2dbbf

2022-08-07 Thread Matwey V. Kornilov
пт, 5 авг. 2022 г. в 18:01, Robert Nelson : > > On Fri, Jul 29, 2022 at 12:07 PM Matwey V. Kornilov > wrote: > > > > пт, 29 июл. 2022 г. в 19:46, Tom Rini : > > > > > > On Fri, Jul 29, 2022 at 07:38:28PM +0300, Matwey V. Kornilov wrote: > > > > пт, 29 июл. 2022 г. в 19:32, Tom Rini : > > > > > >