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

2021-05-09 Thread Bin Meng
On Thu, Apr 29, 2021 at 5:40 PM Bin Meng wrote: > > This was missed when VirtIO support was initially brought to U-Boot > back in 2018. Add an entry for it and list myself as the maintainer. > > Signed-off-by: Bin Meng > --- > > MAINTAINERS | 12 > 1 file changed, 12 insertions(+)

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

2021-05-09 Thread Bin Meng
On Thu, Apr 29, 2021 at 5:40 PM Bin Meng wrote: > > This convers the existing README.virtio to reST, and puts it under > the develop/driver-model/ directory. > > Signed-off-by: Bin Meng > --- > > doc/develop/driver-model/index.rst| 1 + > .../driver-model/virtio.rst}

Re: [PATCH v2 15/16] dtoc: Check that a parent is not missing

2021-05-09 Thread Bin Meng
Hi Simon, On Sat, Apr 24, 2021 at 1:00 PM Simon Glass wrote: > > With of-platdata-inst we want to set up a reference to each devices' > parent device, if there is one. If we find that the device has a parent > (i.e. is not a root node) but it is not in the list of devices being > written, then

Re: [PATCH v2 13/16] x86: coreboot: Document the memory map

2021-05-09 Thread Bin Meng
On Sat, Apr 24, 2021 at 12:57 PM Simon Glass wrote: > > Add information about memory usage when U-Boot is started from coreboot. > This is useful when debugging. Also, since coreboot takes a chunk of > memory in the middle of SDRAM for use by PCI devices, it can help avoid > overwriting this with

Re: [PATCH v2 12/16] x86: coreboot: Use vendor in the Kconfig

2021-05-09 Thread Bin Meng
On Sat, Apr 24, 2021 at 12:57 PM Simon Glass wrote: > > Use VENDOR_COREBOOT instead of TARGET_COREBOOT so we can have multiple > coreboot boards, sharing options. Only SYS_CONFIG_NAME needs to be > defined TARGET_COREBOOT. > > Signed-off-by: Simon Glass > --- > > (no changes since v1) > >

Re: [PATCH] doc: usage: modify mmc man-page

2021-05-09 Thread Bin Meng
On Mon, May 10, 2021 at 1:00 PM Jaehoon Chung wrote: > > Modify mmc man-page. > - Change a descritpion about mmc erase command. typo: description > - Add whitespace to distinguish. > > Signed-off-by: Jaehoon Chung > --- > doc/usage/mmc.rst | 3 ++- > 1 file changed, 2 insertions(+), 1

[PATCH] doc: usage: modify mmc man-page

2021-05-09 Thread Jaehoon Chung
Modify mmc man-page. - Change a descritpion about mmc erase command. - Add whitespace to distinguish. Signed-off-by: Jaehoon Chung --- doc/usage/mmc.rst | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/doc/usage/mmc.rst b/doc/usage/mmc.rst index 57284ed6741e..6457c934c084

RE: [PATCH v2] mmc: Update environment variable with active partition

2021-05-09 Thread Reuben Dowle
Hi Jaehoon, > > boot_ack / boot_partition / partition_access are optional variable, not > mandatory. > And it's confused that different variable is used with same command. > > I think that it's enough to use "mmc partconf ". Then it's showed which > partition is used. > > If you have to apply

Re: [PATCH v2] mmc: Update environment variable with active partition

2021-05-09 Thread Jaehoon Chung
Hi Reuben, On 5/10/21 9:14 AM, Reuben Dowle wrote: > This patch allows uboot scripts make choices about where to boot from based > on the active mmc boot partition. This allows having two copies of kernel, > filesystems etc, and choosing which to boot from based off of the active > bootloader

Re: [PATCH v2 09/16] x86: coreboot: Show the BIOS date

2021-05-09 Thread Bin Meng
On Sat, Apr 24, 2021 at 12:57 PM Simon Glass wrote: > > The BIOS version may not be present, e.g. on a Chrome OS build. Add the > BIOS date as well, so we get some sort of indication of coreboot's > vintage. > > Signed-off-by: Simon Glass > --- > > Changes in v2: > - Reorder declaration and use

Re: [PATCH] x86: Correct regwidth prompt in cbsysinfo

2021-05-09 Thread Bin Meng
On Mon, May 10, 2021 at 11:03 AM Bin Meng wrote: > > On Sat, Apr 24, 2021 at 6:05 AM Simon Glass wrote: > > > > This should be 'regwidth', not 'baud'. Fix it. > > > > Signed-off-by: Simon Glass > > --- > > > > cmd/x86/cbsysinfo.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > >

Re: [PATCH] x86: Correct regwidth prompt in cbsysinfo

2021-05-09 Thread Bin Meng
On Sat, Apr 24, 2021 at 6:05 AM Simon Glass wrote: > > This should be 'regwidth', not 'baud'. Fix it. > > Signed-off-by: Simon Glass > --- > > cmd/x86/cbsysinfo.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Reviewed-by: Bin Meng

Re: [PATCH v3 1/2] efi_loader: expose efi_image_parse() even if UEFI Secure Boot is disabled

2021-05-09 Thread Takahiro Akashi
On Mon, May 10, 2021 at 09:49:03AM +0900, Masahisa Kojima wrote: > Hi Heinrich, > > Sorry for the late reply. > > On Sat, 8 May 2021 at 23:08, Heinrich Schuchardt wrote: > > > > On 4/28/21 3:16 PM, Heinrich Schuchardt wrote: > > > On 28.04.21 14:19, Masahisa Kojima wrote: > > > > >> /** > >

Re: [PATCH] riscv: Fix arch_fixup_fdt always failing without /chosen

2021-05-09 Thread Rick Chen
> If /chosen was missing, chosen_offset would never get updated with the new > /chosen node. This would cause fdt_setprop_u32 to fail. This patch fixes > this by setting chosen_offset. In addition, log any errors from setting > boot-hartid as well. > > Fixes: 177c53fe6c6 ("riscv: Move all fdt

Re: [PATCH v3 1/2] efi_loader: expose efi_image_parse() even if UEFI Secure Boot is disabled

2021-05-09 Thread Masahisa Kojima
Hi Heinrich, Sorry for the late reply. On Sat, 8 May 2021 at 23:08, Heinrich Schuchardt wrote: > > On 4/28/21 3:16 PM, Heinrich Schuchardt wrote: > > On 28.04.21 14:19, Masahisa Kojima wrote: > > >> /** > >>* cmp_pe_section() - compare virtual addresses of two PE image sections > >>*

Re: [RFC] efi_loader: improve firmware capsule authentication

2021-05-09 Thread AKASHI Takahiro
On Fri, May 07, 2021 at 08:47:28PM +0200, Heinrich Schuchardt wrote: > On 5/7/21 6:29 AM, AKASHI Takahiro wrote: > > Heinrich, > > > > On Mon, Apr 26, 2021 at 11:44:59AM +0900, AKASHI Takahiro wrote: > > > Heinrich, > > > > > > Do you have any comments? > > > # not only on this issue, but also

[PATCH v2] mmc: Update environment variable with active partition

2021-05-09 Thread Reuben Dowle
This patch allows uboot scripts make choices about where to boot from based on the active mmc boot partition. This allows having two copies of kernel, filesystems etc, and choosing which to boot from based off of the active bootloader partition. Signed-off-by: Reuben Dowle --- cmd/mmc.c | 18

Re: [RFC PATCH u-boot 00/12] U-Boot LTO (Sandbox + ARM Nokia RX-51)

2021-05-09 Thread Marek Behun
On Sun, 9 May 2021 09:14:14 -0500 Adam Ford wrote: > On Sat, Mar 6, 2021 at 10:06 PM Marek Behun wrote: > > > > On Sat, 6 Mar 2021 21:45:02 -0600 > > Adam Ford wrote: > > > > > On Sat, Mar 6, 2021 at 3:49 PM Marek Behun wrote: > > > > > > > > On Sat, 6 Mar 2021 22:38:52 +0100 > > > > Pali

Re: [RFC PATCH u-boot 00/12] U-Boot LTO (Sandbox + ARM Nokia RX-51)

2021-05-09 Thread Adam Ford
On Sat, Mar 6, 2021 at 10:06 PM Marek Behun wrote: > > On Sat, 6 Mar 2021 21:45:02 -0600 > Adam Ford wrote: > > > On Sat, Mar 6, 2021 at 3:49 PM Marek Behun wrote: > > > > > > On Sat, 6 Mar 2021 22:38:52 +0100 > > > Pali Rohár wrote: > > > > > > > On Saturday 06 March 2021 22:19:22 Marek Behun

Re: [PATCH 01/18] soc: ti: k3-socinfo: Add entry for AM64X SoC family

2021-05-09 Thread Lokesh Vutla
On 08/05/21 3:36 pm, Vignesh Raghavendra wrote: > Hi, > > On 5/6/21 4:44 PM, Lokesh Vutla wrote: >> Add support for AM64 SoC identification. >> >> Signed-off-by: Lokesh Vutla >> --- >> drivers/soc/soc_ti_k3.c | 4 >> 1 file changed, 4 insertions(+) >> >> diff --git

[PATCH v3 13/13] riscv: Drop USE_SPL_FIT_GENERATOR

2021-05-09 Thread Bin Meng
Now that we have switched to binman to generate u-boot.itb for all RISC-V boards, USE_SPL_FIT_GENERATOR is no longer needed and can be dropped. Signed-off-by: Bin Meng --- Changes in v3: - remove USE_SPL_FIT_GENERATOR in ae350_ defconfigs Changes in v2: - new patch: "riscv: Drop

[PATCH v3 12/13] riscv: ae350: Switch to use binman to generate u-boot.itb

2021-05-09 Thread Bin Meng
Use the new BINMAN_STANDALONE_FDT option for AE350 based SPL defconfigs, so that binman is now used to generate u-boot.itb. Signed-off-by: Bin Meng --- Changes in v3: - new patch: "riscv: ae350: Switch to use binman to generate u-boot.itb" arch/riscv/dts/ae350_32.dts | 2 ++

[PATCH v3 11/13] riscv: qemu: Switch to use binman to generate u-boot.itb

2021-05-09 Thread Bin Meng
By utilizing the newly introduced BINMAN_STANDALONE_FDT option, along with a new dedicated device tree source file for the QEMU virt target used for binman only, we can now use binman to generate u-boot.itb. Signed-off-by: Bin Meng Reviewed-by: Simon Glass --- (no changes since v2) Changes

[PATCH v3 10/13] riscv: dts: Sort build targets in alphabetical order

2021-05-09 Thread Bin Meng
Sort the RISC-V DTS build targets by their Kconfig target names in alphabetical order. Signed-off-by: Bin Meng --- (no changes since v2) Changes in v2: - new patch: "riscv: dts: Sort build targets in alphabetical order" arch/riscv/dts/Makefile | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH v3 09/13] binman: Support packaging U-Boot for scenarios like OF_BOARD or OF_PRIOR_STAGE

2021-05-09 Thread Bin Meng
For scenarios like OF_BOARD or OF_PRIOR_STAGE, no device tree blob is provided in the U-Boot build phase hence the binman node information is not available. In order to support such use case, a new Kconfig option BINMAN_STANDALONE_FDT is introduced, to tell the build system that a device tree blob

[PATCH v3 08/13] lib: kconfig: Limit BINMAN_FDT for OF_SEPARATE or OF_EMBED

2021-05-09 Thread Bin Meng
Generally speaking BINMAN_FDT makes sense for OF_SEPARATE or OF_EMBED. For the other OF_CONTROL methods, it's quite possible binman node is not available as binman is invoked during the build phase instead of runtime. Let's only turn it on for OF_SEPARATE or OF_EMBED by default. Signed-off-by:

[PATCH v3 06/13] makefile: Pass OpenSBI blob to binman make rules

2021-05-09 Thread Bin Meng
This updates the make rules to pass OpenSBI blob to binman. Signed-off-by: Bin Meng Reviewed-by: Simon Glass --- (no changes since v1) Makefile | 1 + 1 file changed, 1 insertion(+) diff --git a/Makefile b/Makefile index 404977efa5..3d88559b67 100644 --- a/Makefile +++ b/Makefile @@ -1287,6

[PATCH v3 07/13] riscv: sifive: unleashed: Switch to use binman to generate u-boot.itb

2021-05-09 Thread Bin Meng
At present SiFive Unleashed board uses the Makefile to create the FIT, using USE_SPL_FIT_GENERATOR, which is deprecated as per the Makefile warning. Update to use binman instead. Signed-off-by: Bin Meng Reviewed-by: Simon Glass --- Changes in v3: - Fix the opensbi entry type in binman.dtsi

[PATCH v3 05/13] binman: Add support for RISC-V OpenSBI fw_dynamic blob

2021-05-09 Thread Bin Meng
Add an entry for RISC-V OpenSBI's 'fw_dynamic' firmware payload. Signed-off-by: Bin Meng Reviewed-by: Simon Glass --- (no changes since v2) Changes in v2: - drop "size = <16>" in the binman node tools/binman/entries.rst | 13 + tools/binman/etype/opensbi.py | 23

[PATCH v3 04/13] binman: test: Rename 172_fit_fdt.dts to 170_fit_fdt.dts

2021-05-09 Thread Bin Meng
Currently there are 2 binman test cases using the same 172 number. It seems that 172_fit_fdt.dts was originally named as 170_, but commit c0f1ebe9c1b9 ("binman: Allow selecting default FIT configuration") changed its name to 172_ for no reason. Let's change it back. Signed-off-by: Bin Meng

[PATCH v3 03/13] binman: Correct the comment for ATF entry type

2021-05-09 Thread Bin Meng
This is wrongly referring to Intel ME, which should be ATF. Signed-off-by: Bin Meng Reviewed-by: Simon Glass --- (no changes since v1) tools/binman/etype/atf_bl31.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/binman/etype/atf_bl31.py

[PATCH v3 02/13] binman: Correct '-a' description in the doc

2021-05-09 Thread Bin Meng
It needs a space around '-a'. Signed-off-by: Bin Meng Reviewed-by: Simon Glass --- (no changes since v1) tools/binman/binman.rst | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/binman/binman.rst b/tools/binman/binman.rst index 1aa2459d50..b3df3a6428 100644 ---

[PATCH v3 01/13] common: kconfig: Correct a typo in SPL_LOAD_FIT

2021-05-09 Thread Bin Meng
It should be FDT, not FTD. Signed-off-by: Bin Meng Reviewed-by: Simon Glass --- (no changes since v1) common/Kconfig.boot | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/common/Kconfig.boot b/common/Kconfig.boot index 5a18d62d78..94d82c27dd 100644 ---

[PATCH v3 00/13] riscv: Switch to use binman to generate u-boot.itb

2021-05-09 Thread Bin Meng
This series updates binman to handle creation of u-boot.itb image for RISC-V boards. Azure results: PASS https://dev.azure.com/bmeng/GitHub/_build/results?buildId=360=results The following tests were performed: * booting qemu-riscv{32|64}_spl_defconfig on QEMU virt * booting