Re: [RFC PATCH 0/3] fastboot: sunxi: Determine MMC device at runtime

2021-05-24 Thread Sean Anderson
On 5/24/21 11:28 AM, Maxime Ripard wrote: > On Mon, May 24, 2021 at 10:37:31AM -0400, Sean Anderson wrote: >> >> >> On 5/23/21 8:36 PM, Andre Przywara wrote: >>> At the moment the fastboot code relies on the Kconfig variable >>> CONFIG_FASTBOOT_FLASH_MMC_DEV to point to the MMC device to use

Re: [PATCH u-boot v4 36/36] ARM: enable LTO for some boards

2021-05-24 Thread Tom Rini
On Fri, May 21, 2021 at 12:56:41PM -0400, Tom Rini wrote: > On Fri, May 21, 2021 at 06:00:31PM +0200, Marek Behún wrote: > > On Fri, 21 May 2021 10:11:47 -0400 > > Tom Rini wrote: > > > > > On Thu, May 20, 2021 at 01:56:29PM -0500, Adam Ford wrote: > > > > On Thu, May 20, 2021 at 6:25 AM Marek

Re: [RFC PATCH 0/3] fastboot: sunxi: Determine MMC device at runtime

2021-05-24 Thread Sean Anderson
On 5/24/21 11:15 AM, Andre Przywara wrote: > On Mon, 24 May 2021 10:37:31 -0400 > Sean Anderson wrote: > >> On 5/23/21 8:36 PM, Andre Przywara wrote: >> > At the moment the fastboot code relies on the Kconfig variable >> > CONFIG_FASTBOOT_FLASH_MMC_DEV to point to the MMC device to use

Re: [RFC PATCH 0/3] fastboot: sunxi: Determine MMC device at runtime

2021-05-24 Thread Maxime Ripard
On Mon, May 24, 2021 at 10:37:31AM -0400, Sean Anderson wrote: > > > On 5/23/21 8:36 PM, Andre Przywara wrote: > > At the moment the fastboot code relies on the Kconfig variable > > CONFIG_FASTBOOT_FLASH_MMC_DEV to point to the MMC device to use for the > > flash command. This value needs to be

Re: [RFC PATCH 0/3] fastboot: sunxi: Determine MMC device at runtime

2021-05-24 Thread Andre Przywara
On Mon, 24 May 2021 10:37:31 -0400 Sean Anderson wrote: > On 5/23/21 8:36 PM, Andre Przywara wrote: > > At the moment the fastboot code relies on the Kconfig variable > > CONFIG_FASTBOOT_FLASH_MMC_DEV to point to the MMC device to use for the > > flash command. This value needs to be the

Re: [RFC PATCH 2/3] sunxi: Implement fastboot_get_mmc_device()

2021-05-24 Thread Andre Przywara
On Mon, 24 May 2021 10:33:58 -0400 Sean Anderson wrote: Hi Sean, > On 5/23/21 8:36 PM, Andre Przywara wrote: > > The default MMC device to use for the fastboot flash command is hard to > > decide at runtime, since the numbering of the MMC devices depends on > > devicetree nodes. On the hardware

Re: [RFC PATCH 0/3] fastboot: sunxi: Determine MMC device at runtime

2021-05-24 Thread Sean Anderson
On 5/23/21 8:36 PM, Andre Przywara wrote: > At the moment the fastboot code relies on the Kconfig variable > CONFIG_FASTBOOT_FLASH_MMC_DEV to point to the MMC device to use for the > flash command. This value needs to be the *U-Boot device number*, which > is picked by the U-Boot device model

Re: [RFC PATCH 2/3] sunxi: Implement fastboot_get_mmc_device()

2021-05-24 Thread Sean Anderson
On 5/23/21 8:36 PM, Andre Przywara wrote: The default MMC device to use for the fastboot flash command is hard to decide at runtime, since the numbering of the MMC devices depends on devicetree nodes. On the hardware side, the eMMC is connected to device 2, but this might be U-Boot's device 1

Re: [PATCH 4/4] efi_loader: simplify accessing variables

2021-05-24 Thread Ilias Apalodimas
On Mon, May 24, 2021 at 11:56:56AM +0200, Heinrich Schuchardt wrote: > Use efi_get_variable_int() instead of EFI_CALL(RT->get_variable()). > Use efi_set_variable_int() instead of EFI_CALL(efi_set_variable()). > > Signed-off-by: Heinrich Schuchardt > --- > cmd/efidebug.c | 33

Re: [PATCH 3/4] efi_loader: use efi_create_indexed_name()

2021-05-24 Thread Ilias Apalodimas
On Mon, May 24, 2021 at 11:56:55AM +0200, Heinrich Schuchardt wrote: > Simplify the creation of indexed variable names like 'Boot' by using > function efi_create_indexed_name(). > > Signed-off-by: Heinrich Schuchardt > --- > cmd/efidebug.c | 37 ++--- > 1

Re: [PATCH 2/4] efi_loader: simplify show_efi_boot_order()

2021-05-24 Thread Ilias Apalodimas
On Mon, May 24, 2021 at 11:56:54AM +0200, Heinrich Schuchardt wrote: > To print a UTF-16 string use %ls instead of converting string to UTF-8. > > Signed-off-by: Heinrich Schuchardt > --- > cmd/efidebug.c | 15 +-- > 1 file changed, 1 insertion(+), 14 deletions(-) > > diff --git

Re: [PATCH 1/4] efi_loader: simplify show_efi_boot_opt_data()

2021-05-24 Thread Ilias Apalodimas
On Mon, May 24, 2021 at 11:56:53AM +0200, Heinrich Schuchardt wrote: > To print a UTF-16 string use %ls instead of converting string to UTF-8. > > Signed-off-by: Heinrich Schuchardt > --- > cmd/efidebug.c | 13 + > 1 file changed, 1 insertion(+), 12 deletions(-) > > diff --git

Re: [PATCH v8 1/3] lib: introduce HASH_CALCULATE option

2021-05-24 Thread Heinrich Schuchardt
On 5/14/21 2:53 AM, Masahisa Kojima wrote: Build error occurs when CONFIG_EFI_SECURE_BOOT or CONFIG_EFI_CAPSULE_AUTHENTICATE is enabled, because hash-checksum.c is not compiled. Since hash_calculate() implemented in hash-checksum.c can be commonly used aside from FIT image signature

Re: [PATCH v8 3/3] efi_loader: add PE/COFF image measurement

2021-05-24 Thread Ilias Apalodimas
new_efi); > + > bool efi_image_parse(void *efi, size_t len, struct efi_image_regions **regp, >WIN_CERTIFICATE **auth, size_t *auth_len); > > diff --git a/include/efi_tcg2.h b/include/efi_tcg2.h > index 40e241ce31..bcfb98168a 100644 > --- a/include/efi_tcg2.h > +++

Re: [PATCH] spi: stm32_qspi: Fix short data write operation

2021-05-24 Thread Daniil Stas
On Mon, 24 May 2021 09:40:05 +0200 Patrice CHOTARD wrote: > Hi Daniil > > On 5/24/21 12:24 AM, Daniil Stas wrote: > > TCF flag only means that all data was sent to FIFO. To check if the > > data was sent out of FIFO we should also wait for the BUSY flag to > > be cleared. Otherwise there is a

Re: [PATCH 11/18] arm: Remove nsa310s board

2021-05-24 Thread Tom Rini
On Sun, May 23, 2021 at 11:54:55PM -0700, Tony (bodhi) wrote: > Hi Tom, > > I'm still actively supporting this board, and will do the rebase and DM > conversion. > > I'm not sure about the process from here. Will I need to submit the patches > like for a new board again? Hi, So the deadline

Re: [PATCH v11 2/8] drivers: clk: add fu740 support

2021-05-24 Thread Leo Liang
On Thu, May 20, 2021 at 03:40:27PM +0800, Green Wan wrote: > Add fu740 support. One abstract layer is added for supporting > multiple chips such as fu540 and fu740. > > Signed-off-by: Green Wan > --- > arch/riscv/cpu/fu540/Kconfig | 2 +- > drivers/clk/sifive/Kconfig | 8 +- >

[PATCH 4/4] efi_loader: simplify accessing variables

2021-05-24 Thread Heinrich Schuchardt
Use efi_get_variable_int() instead of EFI_CALL(RT->get_variable()). Use efi_set_variable_int() instead of EFI_CALL(efi_set_variable()). Signed-off-by: Heinrich Schuchardt --- cmd/efidebug.c | 33 ++--- 1 file changed, 18 insertions(+), 15 deletions(-) diff --git

[PATCH 2/4] efi_loader: simplify show_efi_boot_order()

2021-05-24 Thread Heinrich Schuchardt
To print a UTF-16 string use %ls instead of converting string to UTF-8. Signed-off-by: Heinrich Schuchardt --- cmd/efidebug.c | 15 +-- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/cmd/efidebug.c b/cmd/efidebug.c index 9444eca49c..4c0facd26c 100644 ---

[PATCH 3/4] efi_loader: use efi_create_indexed_name()

2021-05-24 Thread Heinrich Schuchardt
Simplify the creation of indexed variable names like 'Boot' by using function efi_create_indexed_name(). Signed-off-by: Heinrich Schuchardt --- cmd/efidebug.c | 37 ++--- 1 file changed, 14 insertions(+), 23 deletions(-) diff --git a/cmd/efidebug.c

[PATCH 1/4] efi_loader: simplify show_efi_boot_opt_data()

2021-05-24 Thread Heinrich Schuchardt
To print a UTF-16 string use %ls instead of converting string to UTF-8. Signed-off-by: Heinrich Schuchardt --- cmd/efidebug.c | 13 + 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/cmd/efidebug.c b/cmd/efidebug.c index 0bf7b8856c..9444eca49c 100644 ---

[PATCH 0/4] efi_loader: simplify efidebug implementation

2021-05-24 Thread Heinrich Schuchardt
This series contains several patches reducing the code size of the efidebug command implementation. Heinrich Schuchardt (4): efi_loader: simplify show_efi_boot_opt_data() efi_loader: simplify show_efi_boot_order() efi_loader: use efi_create_indexed_name() efi_loader: simplify accessing

Re: [PATCH] GitLab: Remove tags for sandbox_noinst tests

2021-05-24 Thread Pratyush Yadav
On 21/05/21 09:59PM, Harm Berntsen wrote: > Commit 712cc962b70d ("GitLab: Remove "tags" stanzas") removed the 'all' > tag from all the jobs. The sandbox_noinst_test.py test was added in > between the author date and commit date (commit 6c914e4232cd > ("azure/gitlab: Add tests for sandbox_noinst"))

Re: [PATCH] net: dwc_eth_qos: Fix needless phy auto-negotiation restarts

2021-05-24 Thread Patrice CHOTARD
Hi Daniil +Christophe Thanks Patrice On 5/24/21 12:24 AM, Daniil Stas wrote: > Disabling clk_ck clock leads to link up status loss in phy, which > leads to auto-negotiation restart before each network command > execution. > > This issue is especially big for PXE boot protocol because of >

[PATCH] cmd:(cosmetic)Mentioned in 'chpart' command HELP text that it is for MTD devices

2021-05-24 Thread opensource . kab
From: Adarsh Babu Kalepalli Modified the help text of 'chpart' command ,mentioning that it is for MTD devices. Signed-off-by: Adarsh Babu Kalepalli --- cmd/mtdparts.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cmd/mtdparts.c b/cmd/mtdparts.c index

Re: [PATCH] spi: stm32_qspi: Fix short data write operation

2021-05-24 Thread Patrice CHOTARD
Hi Daniil On 5/24/21 12:24 AM, Daniil Stas wrote: > TCF flag only means that all data was sent to FIFO. To check if the > data was sent out of FIFO we should also wait for the BUSY flag to be > cleared. Otherwise there is a race condition which can lead to > inability to write short (one byte

Re: [PATCH 6/6] arm: a37xx: pci: Increase PCIe MEM size from 16 MiB to 128 MiB - 64 KiB

2021-05-24 Thread Pali Rohár
On Monday 17 May 2021 08:39:56 Pali Rohár wrote: > For some configurations with more PCIe cards and PCIe bridges 16 MiB of > PCIe MEM space is not enough. Since TF-A already allocates a 128 MiB CPU > window for PCIe and IO port space is only 64 KiB in total, use all the > remaining space for PCIe

Re: [PATCH v2] drivers: pci: pcie_dw_common: fix Werror compilation error

2021-05-24 Thread Leo Liang
On Wed, May 19, 2021 at 07:16:15PM +0800, Green Wan wrote: > Fix compilation error when Werror is turned on. The warning could > possible break some CI builds. > > Signed-off-by: Green Wan > --- > drivers/pci/pcie_dw_common.c | 54 +++- > 1 file changed, 29

Re: [PATCH 11/18] arm: Remove nsa310s board

2021-05-24 Thread Tony (bodhi)
Hi Tom, I'm still actively supporting this board, and will do the rebase and DM conversion. I'm not sure about the process from here. Will I need to submit the patches like for a new board again? Thanks, Tony Dinh On Sat, May 22, 2021 at 5:48 AM Tom Rini wrote: > These boards have not been

<    1   2