[PATCH v3 9/9] sunxi: binman: Add support for including SCP firmware

2020-09-05 Thread Samuel Holland
Allwinner sun50i SoCs contain an OpenRISC 1000 CPU that functions as a System Control Processor, or SCP. ARM Trusted Firmware (ATF) communicates with the SCP over SCPI to implement the PSCI system suspend, shutdown and reset functionality. Currently, SCP firmware is optional; the system will boot

[PATCH v3 6/9] sunxi: binman: Use a macro for the BL31 load address

2020-09-05 Thread Samuel Holland
This consolidates the SoC-specific part at the top of the file to avoid cluttering it up with preprocessor conditions. Signed-off-by: Samuel Holland --- arch/arm/dts/sunxi-u-boot.dtsi | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git

[PATCH v3 8/9] binman: Add support for SCP firmware

2020-09-05 Thread Samuel Holland
Add an entry type for a firmware blob for a Sytem Control Processor, given by an entry arg. This firmware is a raw binary blob. Signed-off-by: Samuel Holland --- Makefile | 2 +- tools/binman/etype/scp.py | 19 +++ 2 files changed, 20 insertions(+), 1

[PATCH v3 1/9] Makefile: Only define u-boot.itb rule when applicable

2020-09-05 Thread Samuel Holland
If neither CONFIG_SPL_FIT_SOURCE nor CONFIG_USE_SPL_FIT_GENERATOR is enabled, U_BOOT_ITS will be undefined, and attempting to make u-boot.itb will pass invalid arguments to mkimage, causing it to print its help message. Remove the rule in that case, so it is more obvious that u-boot.itb is not

[PATCH v3 4/9] sunxi: binman: Fix spacing between nodes

2020-09-05 Thread Samuel Holland
Nodes should have a blank line separating them from sibling nodes and properties. Add the necessary lines. Signed-off-by: Samuel Holland --- arch/arm/dts/sunxi-u-boot.dtsi | 5 + 1 file changed, 5 insertions(+) diff --git a/arch/arm/dts/sunxi-u-boot.dtsi b/arch/arm/dts/sunxi-u-boot.dtsi

[PATCH v3 2/9] binman: Only write FDT once per node

2020-09-05 Thread Samuel Holland
Due to an extra level of indentation, the "data" property containing the FDT was being written repeatedly after every other property in the node. Move the block up one level, so the property is added exactly once. Signed-off-by: Samuel Holland --- tools/binman/etype/fit.py | 8 1 file

[PATCH v3 7/9] sunxi: binman: Update FIT component descriptions

2020-09-05 Thread Samuel Holland
Since commit d879616e9e64 ("spl: fit: simplify logic for FDT loading for non-OS boots"), the SPL looks at the "os" properties of FIT images to determine where to append the FDT. The "os" property of the "firmware" image also determines how to execute the next stage of the boot process, as in

[PATCH v3 0/9] sunxi: binman fixes and SCP firmware support

2020-09-05 Thread Samuel Holland
This is a rebase of my previous patch series adding SCP firmware support for system suspend, onto the binman rewrite of mksunxi_fit_atf.sh. It is based on u-boot-dm/next. - The first two patches apply to binman FIT support generally. - The third patch is unchanged from previous versions. -

[PATCH v3 5/9] sunxi: binman: Provide a default BL31 filename

2020-09-05 Thread Samuel Holland
Prior to commit 7f7f8aca8257 ("sunxi: Convert 64-bit boards to use binman"), if the BL31 environment variable was not defined, the firmware would be loaded from a file "bl31.bin" in the current directory. Restore that behavior by providing that as the default filename in case no entry arg is

[PATCH v3 3/9] spl: fit: Minimally parse OS properties with FIT_IMAGE_TINY

2020-09-05 Thread Samuel Holland
Some boards, specifically 64-bit Allwinner boards (sun50i), are extremely limited on SPL size. One strategy that was used to make space was to remove the FIT "os" property parsing code, because it uses a rather large lookup table. However, this forces the legacy FIT parsing code path, which

Re: [PATCH v3 08/12] sunxi: Convert 64-bit boards to use binman

2020-09-05 Thread Simon Glass
Hi Samuel, On Sat, 5 Sep 2020 at 19:49, Samuel Holland wrote: > > Simon, > > On 9/5/20 7:18 PM, Simon Glass wrote: > > On Sat, 5 Sep 2020 at 17:10, Samuel Holland wrote: > >> On 9/1/20 6:14 AM, Simon Glass wrote: > >>> At present 64-bit sunxi boards use the Makefile to create a FIT, using > >>>

Re: [PATCH v3 08/12] sunxi: Convert 64-bit boards to use binman

2020-09-05 Thread Samuel Holland
Simon, On 9/5/20 7:18 PM, Simon Glass wrote: > On Sat, 5 Sep 2020 at 17:10, Samuel Holland wrote: >> On 9/1/20 6:14 AM, Simon Glass wrote: >>> At present 64-bit sunxi boards use the Makefile to create a FIT, using >>> USE_SPL_FIT_GENERATOR. This is deprecated. >>> >>> Update sunxi to use binman

Re: [PATCH v3 06/12] binman: Support generating FITs with multiple dtbs

2020-09-05 Thread Simon Glass
Hi Samuel, On Sat, 5 Sep 2020 at 16:41, Samuel Holland wrote: > > On 9/1/20 6:13 AM, Simon Glass wrote: > > In some cases it is useful to generate a FIT which has a number of DTB > > images, selectable by configuration. Add support for this in binman, > > using a simple iterator and string

Re: [PATCH v3 05/12] binman: Add support for ATF BL31

2020-09-05 Thread Simon Glass
Hi Samuel, On Sat, 5 Sep 2020 at 16:57, Samuel Holland wrote: > > On 9/1/20 6:13 AM, Simon Glass wrote: > > Add an entry for ARM Trusted Firmware's 'BL31' payload, which is the > > device's main firmware. Typically this is U-Boot. > > > > Signed-off-by: Simon Glass > > --- > > > > Changes in

Re: [PATCH v3 08/12] sunxi: Convert 64-bit boards to use binman

2020-09-05 Thread Simon Glass
Hi Samuel, On Sat, 5 Sep 2020 at 17:10, Samuel Holland wrote: > > On 9/1/20 6:14 AM, Simon Glass wrote: > > At present 64-bit sunxi boards use the Makefile to create a FIT, using > > USE_SPL_FIT_GENERATOR. This is deprecated. > > > > Update sunxi to use binman instead. > > > > Signed-off-by:

Re: [PATCH 1/3] binman: Support cross-compiling test files to x86

2020-09-05 Thread Simon Glass
Hi Alper, On Sat, 5 Sep 2020 at 16:51, Alper Nebi Yasak wrote: > > On 05/09/2020 19:36, Simon Glass wrote: > > For me this fails on x86_64, complaining for example: > > > > Exception: Error 2 running 'make -C /tmp/binmant.d17vfu3j/elftest -f > >

Re: [PATCH v3 08/12] sunxi: Convert 64-bit boards to use binman

2020-09-05 Thread Simon Glass
Hi Samuel, On Sat, 5 Sep 2020 at 17:42, Samuel Holland wrote: > > On 9/5/20 6:10 PM, Samuel Holland wrote: > > On 9/1/20 6:14 AM, Simon Glass wrote: > >> At present 64-bit sunxi boards use the Makefile to create a FIT, using > >> USE_SPL_FIT_GENERATOR. This is deprecated. > >> > >> Update sunxi

Re: [PATCH v3 06/12] binman: Support generating FITs with multiple dtbs

2020-09-05 Thread Simon Glass
Hi Samuel, On Sat, 5 Sep 2020 at 17:19, Samuel Holland wrote: > > On 9/5/20 5:41 PM, Samuel Holland wrote: > > On 9/1/20 6:13 AM, Simon Glass wrote: > >> In some cases it is useful to generate a FIT which has a number of DTB > >> images, selectable by configuration. Add support for this in

Re: [PATCH 2/3] binman: Use target-specific tools when cross-compiling

2020-09-05 Thread Simon Glass
Hi Alper, On Sat, 5 Sep 2020 at 17:04, Alper Nebi Yasak wrote: > > On 05/09/2020 19:37, Simon Glass wrote: > > This looks good, but it drops the use of DTC to specify the > > device-tree compiler. Can you add it back? > > I think you're referring to this hunk: > > > # If we don't have a

Re: [PATCH v3 08/12] sunxi: Convert 64-bit boards to use binman

2020-09-05 Thread Samuel Holland
On 9/5/20 6:10 PM, Samuel Holland wrote: > On 9/1/20 6:14 AM, Simon Glass wrote: >> At present 64-bit sunxi boards use the Makefile to create a FIT, using >> USE_SPL_FIT_GENERATOR. This is deprecated. >> >> Update sunxi to use binman instead. >> >> Signed-off-by: Simon Glass >> --- >> >> (no

Re: [PATCH v3 06/12] binman: Support generating FITs with multiple dtbs

2020-09-05 Thread Samuel Holland
On 9/5/20 5:41 PM, Samuel Holland wrote: > On 9/1/20 6:13 AM, Simon Glass wrote: >> In some cases it is useful to generate a FIT which has a number of DTB >> images, selectable by configuration. Add support for this in binman, >> using a simple iterator and string substitution. >> >>

Re: [PATCH 3/3] binman: Allow resolving host-specific tools from env vars

2020-09-05 Thread Alper Nebi Yasak
On 05/09/2020 19:37, Simon Glass wrote: > Please see below. > > Also please add a mention of the CROSS_COMPILE thing in binman's README. > >> +def GetHostCompileTool(name): >> +"""Get the host-specific version for a compile tool >> + >> +This checks the environment variables that specify

Re: [PATCH v3 08/12] sunxi: Convert 64-bit boards to use binman

2020-09-05 Thread Samuel Holland
On 9/1/20 6:14 AM, Simon Glass wrote: > At present 64-bit sunxi boards use the Makefile to create a FIT, using > USE_SPL_FIT_GENERATOR. This is deprecated. > > Update sunxi to use binman instead. > > Signed-off-by: Simon Glass > --- > > (no changes since v2) > > Changes in v2: > - Add a

Re: [PATCH 2/3] binman: Use target-specific tools when cross-compiling

2020-09-05 Thread Alper Nebi Yasak
On 05/09/2020 19:37, Simon Glass wrote: > This looks good, but it drops the use of DTC to specify the > device-tree compiler. Can you add it back? I think you're referring to this hunk: > # If we don't have a directory, put it in the tools tempdir > search_list = [] > for path in

Re: [PATCH v3 05/12] binman: Add support for ATF BL31

2020-09-05 Thread Samuel Holland
On 9/1/20 6:13 AM, Simon Glass wrote: > Add an entry for ARM Trusted Firmware's 'BL31' payload, which is the > device's main firmware. Typically this is U-Boot. > > Signed-off-by: Simon Glass > --- > > Changes in v3: > - Rebase on top of earlier binman series > > Changes in v2: > - Add the URL

Re: [PATCH 1/3] binman: Support cross-compiling test files to x86

2020-09-05 Thread Alper Nebi Yasak
On 05/09/2020 19:36, Simon Glass wrote: > For me this fails on x86_64, complaining for example: > > Exception: Error 2 running 'make -C /tmp/binmant.d17vfu3j/elftest -f > /scratch/sglass/cosarm/src/third_party/u-boot/files/tools/binman/test/Makefile >

Re: [PATCH v3 06/12] binman: Support generating FITs with multiple dtbs

2020-09-05 Thread Samuel Holland
On 9/1/20 6:13 AM, Simon Glass wrote: > In some cases it is useful to generate a FIT which has a number of DTB > images, selectable by configuration. Add support for this in binman, > using a simple iterator and string substitution. > > Signed-off-by: Simon Glass > --- > > Changes in v3: > -

Re: [PATCH] spl: Add SPL_SERIAL as requirement for SDP_USB_SDP

2020-09-05 Thread Fabio Estevam
Hi Otavio, On Sat, Sep 5, 2020 at 6:37 PM Otavio Salvador wrote: > > Signed-off-by: Otavio Salvador Please always provide a commit log for the patches.

[PATCH] spl: Add SPL_SERIAL as requirement for SDP_USB_SDP

2020-09-05 Thread Otavio Salvador
Signed-off-by: Otavio Salvador --- common/spl/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/common/spl/Kconfig b/common/spl/Kconfig index af8255a8d6..20c5c4af3c 100644 --- a/common/spl/Kconfig +++ b/common/spl/Kconfig @@ -1237,6 +1237,7 @@ endchoice config SPL_USB_SDP_SUPPORT

Re: [PATCH v3 07/12] Makefile: Support missing external blobs always

2020-09-05 Thread Simon Glass
At present binman warns about missing external blobs only when the BUILD_ROM is defined. Enable this behaviour always, since many boards are starting to use these (e.g. ARM Trusted Firmware's BL31). Signed-off-by: Simon Glass --- (no changes since v2) Changes in v2: - Add new patch to support

Re: [PATCH v3 06/12] binman: Support generating FITs with multiple dtbs

2020-09-05 Thread Simon Glass
In some cases it is useful to generate a FIT which has a number of DTB images, selectable by configuration. Add support for this in binman, using a simple iterator and string substitution. Signed-off-by: Simon Glass --- Changes in v3: - Rebase on top of earlier binman series Changes in v2: -

Re: [PATCH v3 01/12] binman: Allow entry args to be required

2020-09-05 Thread Simon Glass
If an entry argument is needed by an entry but the entry argument is not present, then a strange error can occur when trying to read the file. Fix this by allowing arguments to be required. Select this option for the cros-ec-rw entry. If a filename is provided in the node, allow that to be used.

Re: [PATCH 1/2] core: improve of_match_ptr with OF_PLATDATA

2020-09-05 Thread Simon Glass
On Wed, 29 Jul 2020 at 10:17, Walter Lozano wrote: > > Currently of_match_ptr is used to avoid referencing compatible strings > when OF_CONTROL is not enabled. This behaviour could be improved by > taking into account also OF_PLATDATA, as when this configuration is > enabled the compatible

Re: [PATCH v3 02/12] binman: Fix up a few missing comments

2020-09-05 Thread Simon Glass
Tidy up a few test functions which lack argument comments. Rename one that has the same name as a different test. Also fix up the comment for PrepareImagesAndDtbs(). Signed-off-by: Simon Glass --- (no changes since v1) tools/binman/control.py | 5 + tools/binman/ftest.py | 22

Re: [PATCH v3 03/12] libfdt: Detected out-of-space with fdt_finish()

2020-09-05 Thread Simon Glass
At present the Python sequential-write interface can produce an error when it calls fdt_finish(), since this needs to add a terminating tag to the end of the struct section. Fix this by automatically expanding the buffer if needed. Signed-off-by: Simon Glass --- (no changes since v1)

Re: [PATCH v2] buildman: Use git worktrees instead of git clones when possible

2020-09-05 Thread Simon Glass
On Sat, 5 Sep 2020 at 10:36, Simon Glass wrote: > > On Thu, 3 Sep 2020 at 06:51, Alper Nebi Yasak > wrote: > > > > This patch makes buildman create linked working trees instead of clones > > of the source repository, but keeps updating the older clones of the > > repository that might already

Re: [PATCH v3 05/12] binman: Add support for ATF BL31

2020-09-05 Thread Simon Glass
Add an entry for ARM Trusted Firmware's 'BL31' payload, which is the device's main firmware. Typically this is U-Boot. Signed-off-by: Simon Glass --- Changes in v3: - Rebase on top of earlier binman series Changes in v2: - Add the URL of ARM Trusted Firmware and mention of U-Boot docs - Fix

Re: [PATCH v3 04/12] binman: Move 'external' support into base class

2020-09-05 Thread Simon Glass
At present we have an Entry_blob_ext which implement a blob which holds an external binary. We need to support other entry types that hold external binaries, e.g. Entry_blob_named_by_arg. Move the support into the base Entry class to allow this. Signed-off-by: Simon Glass --- (no changes since

Re: [PATCH] Revert "env: Warn on force access if ENV_ACCESS_IGNORE_FORCE set"

2020-09-05 Thread Marek Vasut
On 8/27/20 3:03 PM, Tom Rini wrote: > On Thu, Aug 27, 2020 at 11:49:25AM +0200, Marek Vasut wrote: >> On 8/27/20 11:43 AM, Alex Kiernan wrote: >>> This reverts commit 0f036bf4b87e6416f5c4d23865a62a62d9073c20. >>> >>> With this change applied and CONFIG_ENV_ACCESS_IGNORE_FORCE disabled, >>> the

[PATCH v3 17/17] cros: Add information about booting Chrome OS on x86

2020-09-05 Thread Simon Glass
Recent versions of Chrome OS do not have a kernel in the root disk, to save space. With the improvements to the 'zboot' command it is fairly easy to load the kernel from the raw partition. Add instructions on how to do this. Signed-off-by: Simon Glass Reviewed-by: Wolfgang Wallner Reviewed-by:

[PATCH v3 15/17] x86: zboot: Allow overriding the command line

2020-09-05 Thread Simon Glass
When booting Chrome OS images the command line is stored separately from the kernel. Add a way to specify this address so that images boot correctly. Also add comments to the zimage.h header. Signed-off-by: Simon Glass Reviewed-by: Wolfgang Wallner Reviewed-by: Bin Meng --- Changes in v3: -

[PATCH v3 16/17] cros: Update chromium documentation

2020-09-05 Thread Simon Glass
A few things have changed since this was written about 18 months ago. Update the README. Signed-off-by: Simon Glass Reviewed-by: Wolfgang Wallner Reviewed-by: Bin Meng --- (no changes since v1) doc/README.chromium | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git

[PATCH v3 13/17] x86: zboot: Allow setting a separate setup base address

2020-09-05 Thread Simon Glass
At present the setup block is always obtained from the image automatically. In some cases it can be useful to use a setup block obtained elsewhere, e.g. if the image has already been unpacked. Add an argument to support this and update the logic to use it if provided. Signed-off-by: Simon Glass

[PATCH v3 14/17] x86: zboot: Add an option to dump the setup information

2020-09-05 Thread Simon Glass
There is a lot of information in the setup block and it is quite hard to decode manually. Add a 'zboot dump' command to decode it into a human-readable format. Signed-off-by: Simon Glass Reviewed-by: Wolfgang Wallner --- Changes in v3: - Update two local arrays to be static - Use 'Commandline'

Re: [PATCH v2 07/16] x86: zboot: Set up a sub-command structure

2020-09-05 Thread Simon Glass
Hi Bin, On Tue, 1 Sep 2020 at 03:30, Bin Meng wrote: > > Hi Simon, > > On Sun, Aug 30, 2020 at 5:42 AM Simon Glass wrote: > > > > Add subcommands to zboot. At present there is only one called 'start' > > which does the whole boot. It is the default command so is optional. > > > > Change the 's'

[PATCH v3 10/17] x86: zboot: Add an 'load' subcommand

2020-09-05 Thread Simon Glass
Add a subcommand that loads the kernel into the right places in memory. Signed-off-by: Simon Glass --- Changes in v3: - Move command help into the patch for each command - Split the 'load' command into its own patch arch/x86/lib/zimage.c | 21 +++-- 1 file changed, 15

[PATCH v3 09/17] x86: zboot: Add an 'info' subcommand

2020-09-05 Thread Simon Glass
Add a little subcommand that prints out where the kernel was loaded and its setup pointer. Run it by default in the normal boot. Signed-off-by: Simon Glass Reviewed-by: Wolfgang Wallner Reviewed-by: Bin Meng --- Changes in v3: - Move command help into the patch for each command

Re: [PATCH v2 3/3] sunxi: Add support for including SCP firmware

2020-09-05 Thread Simon Glass
Hi Samuel, On Thu, 3 Sep 2020 at 00:08, Samuel Holland wrote: > > Allwinner sun50i SoCs contain an OpenRISC 1000 CPU that functions as a > System Control Processor, or SCP. ARM Trusted Firmware (ATF) > communicates with the SCP over SCPI to implement the PSCI system suspend > and shutdown

[PATCH v3 11/17] x86: zboot: Add an 'setup' subcommand

2020-09-05 Thread Simon Glass
Add a subcommand that sets up the kernel ready for execution. Signed-off-by: Simon Glass Reviewed-by: Wolfgang Wallner --- Changes in v3: - Fix 'summary' typo - Move command help into the patch for each command arch/x86/lib/zimage.c | 34 ++ 1 file changed, 26

[PATCH v3 12/17] x86: zboot: Set environment variables for image locations

2020-09-05 Thread Simon Glass
At present it is not possible to tell from a script where the setup block is, or where the image was loaded to. Add environment variables for this. Signed-off-by: Simon Glass Reviewed-by: Wolfgang Wallner Reviewed-by: Bin Meng --- Changes in v3: - Mention that the environment variables are

[PATCH v3 08/17] x86: zboot: Add a 'go' subcommand

2020-09-05 Thread Simon Glass
Split out the code that actually boots linux into a separate sub-command. Add base_ptr to the state to support this. Show an error if the boot fails, since this should not happen. Signed-off-by: Simon Glass Reviewed-by: Wolfgang Wallner Reviewed-by: Bin Meng --- Changes in v3: - Move command

[PATCH v3 07/17] x86: zboot: Set up a sub-command structure

2020-09-05 Thread Simon Glass
Add subcommands to zboot. At present there is only one called 'start' which does the whole boot. It is the default command so is optional. Change the 's' string variable to const while we are here. Signed-off-by: Simon Glass Reviewed-by: Wolfgang Wallner --- Changes in v3: - Add a comment

[PATCH v3 05/17] x86: zboot: Correct image type

2020-09-05 Thread Simon Glass
At present U-Boot sets a loader type of 8 which means LILO version 8, according to the spec. Update it to 0x80, which means U-Boot with no particular version. Signed-off-by: Simon Glass Reviewed-by: Wolfgang Wallner Reviewed-by: Bin Meng --- (no changes since v1) arch/x86/lib/zimage.c | 3

[PATCH v3 06/17] x86: zimage: Disable interrupts just before booting

2020-09-05 Thread Simon Glass
At present if an error occurs while setting up the boot, interrupts are left disabled. Move this call later in the sequence to avoid this problem. Signed-off-by: Simon Glass Reviewed-by: Wolfgang Wallner Reviewed-by: Bin Meng --- (no changes since v1) arch/x86/lib/zimage.c | 2 +- 1 file

Re: [PATCH v2 2/3] mksunxi_fit_atf.sh: Update FIT component descriptions

2020-09-05 Thread Simon Glass
Hi Samuel, On Thu, 3 Sep 2020 at 00:08, Samuel Holland wrote: > > Since commit d879616e9e64 ("spl: fit: simplify logic for FDT loading for > non-OS boots"), the SPL looks at the "os" properties of FIT images to > determine where to append the FDT. > > The "os" property of the "firmware" image

[PATCH v3 02/17] x86: zimage: Use a state struct to hold the state

2020-09-05 Thread Simon Glass
At present the 'zboot' command does everything in one go. It would be better if it supported sub-commands like bootm, so it is possible to examine what will be booted before actually booting it. In preparation for this, move the 'state' of the command into a struct. This will allow it to be

[PATCH v3 01/17] x86: Update the bootparam header

2020-09-05 Thread Simon Glass
This header is missing a few of the newer features from the specification. Add these as well as a link to the spec. Also use the BIT() macros where appropriate. Signed-off-by: Simon Glass Reviewed-by: Wolfgang Wallner --- (no changes since v1) arch/x86/include/asm/bootparam.h | 25

[PATCH v3 04/17] x86: zboot: Move kernel-version code into a function

2020-09-05 Thread Simon Glass
To help reduce the size and complexity of load_zimage(), move the code that reads the kernel version into a separate function. Update get_boot_protocol() to allow printing the 'Magic signature' message only once, under control of its callers. Signed-off-by: Simon Glass Reviewed-by: Wolfgang

[PATCH v3 03/17] x86: zimage: Avoid using #ifdef

2020-09-05 Thread Simon Glass
Use IS_ENABLED() instead of #ifdef in this file. Signed-off-by: Simon Glass Reviewed-by: Wolfgang Wallner Reviewed-by: Bin Meng --- (no changes since v1) arch/x86/lib/zimage.c | 14 +- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/arch/x86/lib/zimage.c

[PATCH v3 00/17] x86: zboot: Enhance the 'zboot' command

2020-09-05 Thread Simon Glass
This command is currently monolithic and does not support scripts which want to adjust the boot process. This series updates it to be more like 'bootm', in that it has sub-commands for each stage of the boot. This allows some stages to be adjusted or skipped. It also adds a way to dump out the

Re: [PATCH v2 2/2] sandbox: enable support of generic udp protocol

2020-09-05 Thread Simon Glass
On Mon, 31 Aug 2020 at 11:27, Philippe Reynes wrote: > > This commit enable the support of the generic udp protocol. > > Signed-off-by: Philippe Reynes > --- > > Changelog: > v2: > - new patch in the serie > > configs/sandbox_defconfig | 1 + > 1 file changed, 1 insertion(+) > Reviewed-by:

Re: [PATCH v2 1/2] net: add a generic udp protocol

2020-09-05 Thread Simon Glass
On Mon, 31 Aug 2020 at 11:27, Philippe Reynes wrote: > > This commit adds a generic udp protocol framework in the > network loop. So protocol based on udp may be implemented > without modifying the network loop (for example custom > wait magic packet). > > Signed-off-by: Philippe Reynes > --- >

[PATCH] x86: Drop duplicate declaration of emulator state

2020-09-05 Thread Simon Glass
With x86 we can execute an option ROM either natively or using the x86 emulator (if enabled with CONFIG_BIOSEMU). Both of these share the _X86EMU_env variable, with the native code using it to hold register state during interrupt processing. At present, in 32-bit U-Boot, the variable is declared

Re: x86: multiple definition of `_X86EMU_env'

2020-09-05 Thread Simon Glass
Hi Heinrich, On Tue, 1 Sep 2020 at 00:41, Heinrich Schuchardt wrote: > > Dear Simon, > > make chromebook_coral_defconfig > make > > using gcc (Debian 10.2.0-5) 10.2.0 and GNU ld (GNU Binutils for Debian) > 2.35 results in: > > ld.bfd: drivers/built-in.o:drivers/bios_emulator/x86emu/sys.c:47: >

Re: [PATCH v2] buildman: Use git worktrees instead of git clones when possible

2020-09-05 Thread Simon Glass
On Sat, 5 Sep 2020 at 10:36, Simon Glass wrote: > > On Thu, 3 Sep 2020 at 06:51, Alper Nebi Yasak > wrote: > > > > This patch makes buildman create linked working trees instead of clones > > of the source repository, but keeps updating the older clones of the > > repository that might already

Re: [PATCH 2/3] binman: Use target-specific tools when cross-compiling

2020-09-05 Thread Simon Glass
Hi Alper, On Sat, 5 Sep 2020 at 08:44, Alper Nebi Yasak wrote: > > Currently, binman always runs the compile tools like cc, objcopy, strip, > etc. using their literal name. Instead, this patch makes it use the > target-specific versions by default, derived from the tool-specific > environment

Re: [PATCH 3/3] binman: Allow resolving host-specific tools from env vars

2020-09-05 Thread Simon Glass
On Sat, 5 Sep 2020 at 08:44, Alper Nebi Yasak wrote: > > This patch lets tools.Run() use host-specific versions with the > for_host keyword argument, based on the host-specific environment > variables (HOSTCC, HOSTOBJCOPY, HOSTSTRIP, etc.). > > Signed-off-by: Alper Nebi Yasak > --- > Not sure if

Re: [PATCH 1/3] binman: Support cross-compiling test files to x86

2020-09-05 Thread Simon Glass
Hi Alper, On Sat, 5 Sep 2020 at 08:44, Alper Nebi Yasak wrote: > > These test files are currently "intended for use on x86 hosts", but most > of the tests using them can still pass when cross-compiled to x86 on an > arm64 host. > > This patch enables non-x86 hosts to run the tests by specifying

Re: [PATCH v2] buildman: Use git worktrees instead of git clones when possible

2020-09-05 Thread Simon Glass
On Thu, 3 Sep 2020 at 06:51, Alper Nebi Yasak wrote: > > This patch makes buildman create linked working trees instead of clones > of the source repository, but keeps updating the older clones of the > repository that might already exist. These worktrees share "everything > except working

[PATCH 3/3] binman: Allow resolving host-specific tools from env vars

2020-09-05 Thread Alper Nebi Yasak
This patch lets tools.Run() use host-specific versions with the for_host keyword argument, based on the host-specific environment variables (HOSTCC, HOSTOBJCOPY, HOSTSTRIP, etc.). Signed-off-by: Alper Nebi Yasak --- Not sure if this patch will ever be useful, but it complements the previous

[PATCH 0/3] binman: Make tests work on non-x86 architectures via cross-compilation

2020-09-05 Thread Alper Nebi Yasak
Right now the 'binman test' command fails spectacularly on arm64 since it cannot even setup the test environments properly due to errors during setUpClass(). I can get a 100% coverage result with all tests passing if I cross-compile things to x86 and run the cross-compiling versions of some tools.

[PATCH 2/3] binman: Use target-specific tools when cross-compiling

2020-09-05 Thread Alper Nebi Yasak
Currently, binman always runs the compile tools like cc, objcopy, strip, etc. using their literal name. Instead, this patch makes it use the target-specific versions by default, derived from the tool-specific environment variables (CC, OBJCOPY, STRIP, etc.) or from the CROSS_COMPILE environment

[PATCH 1/3] binman: Support cross-compiling test files to x86

2020-09-05 Thread Alper Nebi Yasak
These test files are currently "intended for use on x86 hosts", but most of the tests using them can still pass when cross-compiled to x86 on an arm64 host. This patch enables non-x86 hosts to run the tests by specifying a cross-compiler via CROSS_COMPILE. The list of variables it sets is taken

Re: [PATCH v5 11/11] riscv: Add FPIOA and GPIO support for Kendryte K210

2020-09-05 Thread Sean Anderson
On 9/2/20 11:59 AM, Sean Anderson wrote: > On 9/2/20 8:26 AM, Heinrich Schuchardt wrote: >> After adding some debug functions the error appeared and disappeared >> when changing the code in function panic(). So my guess is that there is >> some alignment problem in the static data section. > > I

[PATCH] riscv: Only enable OF_BOARD_FIXUP for S-Mode

2020-09-05 Thread Sean Anderson
It is unsafe to enable OF_BOARD_FIXUP only based on OF_SEPARATE. OF_SEPARATE may indicate that the user wishes U-Boot to use a different device tree than one obtained via OF_PRIOR_STAGE. However, OF_SEPARATE may also indicate that the device tree which would be obtained via OF_PRIOR_STAGE is

Re: [PATCH 2/2] doc: describe building with GCC

2020-09-05 Thread Tom Rini
On Sat, Sep 05, 2020 at 10:58:53AM +0200, Heinrich Schuchardt wrote: > Provide a description of the U-Boot build process with GCC in the HTML > documentation. > > Signed-off-by: Heinrich Schuchardt > --- > doc/build/gcc.rst | 119 >

Re: [PATCH] ARM: dts: stm32: Pull UART4 RX high on AV96

2020-09-05 Thread Marek Vasut
On 8/22/20 10:45 PM, Marek Vasut wrote: > There is no dedicated pull resistor on the AV96 UART4 (console UART) > pin. In case there is no UART adapter installed on the AV96, the line > is floating and can trigger reception of garbage characters, which in > turn can abort U-Boot autoboot. Add

[PATCH] arm: mvebu: Espressobin: Set environment variable fdtfile

2020-09-05 Thread Andre Heider
Required for the generic distro mechanism. Linux ships with 4 variants: marvell/armada-3720-espressobin-v7-emmc.dtb marvell/armada-3720-espressobin-v7.dtb marvell/armada-3720-espressobin-emmc.dtb marvell/armada-3720-espressobin.dtb Use available information to determine the appropriate filename.

Fwd: [PATCH 1/1] espresso7420: remove unused symbol CONFIG_DEFAULT_CONSOLE

2020-09-05 Thread Heinrich Schuchardt
Hello Thomas, I tried to send an U-Boot patch to Thomas Abraham . But the email address thomas...@samsung.com does not exist anymore. I assume that you are the same person. What shall happen with board/samsung/espresso7420/MAINTAINERS? Best regards Heinrich Forwarded Message

[PATCH] doc/sipeed/maix: describe RESET and BOOT button

2020-09-05 Thread Heinrich Schuchardt
In the boot flow description add the RESET and BOOT button as well as the function of the DTR and RTS lines of the serial interface. Signed-off-by: Heinrich Schuchardt --- doc/board/sipeed/maix.rst | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git

Re: [PATCH 18/31] misc: am33xx: add control module driver

2020-09-05 Thread Dario Binacchi
Hi Lokesh, > Il 03/09/2020 08:31 Lokesh Vutla ha scritto: > > > On 25/08/20 2:51 pm, Dario Binacchi wrote: > > The implementation of this driver was needed to bind the device tree > > sub-nodes of the 'clocks' node. In fact, the lack of the compatible > > property in the 'clocks' node does

[PATCH 1/1] doc: correct kflash settings for Maix One Dock

2020-09-05 Thread Heinrich Schuchardt
The correct kflash parameter value for the Maix One Dock is "dan". See: https://github.com/sipeed/platform-kendryte210/blob/master/boards/sipeed-maix-one-dock.json#L22 Fixes: 137dc153fda9 ("doc: riscv: Update documentation for Sipeed MAIX boards") Signed-off-by: Heinrich Schuchardt ---

[PATCH 1/1] espresso7420: remove unused symbol CONFIG_DEFAULT_CONSOLE

2020-09-05 Thread Heinrich Schuchardt
Symbol CONFIG_DEFAULT_CONSOLE in include/configs/espresso7420.h is not used for espresso7420_defconfig. Signed-off-by: Heinrich Schuchardt --- include/configs/espresso7420.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/include/configs/espresso7420.h b/include/configs/espresso7420.h

[PATCH] Makefile: mrproper shall delete doc/output/

2020-09-05 Thread Heinrich Schuchardt
HTML documentation is generated in doc/output/. This directory shall be deleted by 'make mrproper' Signed-off-by: Heinrich Schuchardt --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 5dd4c6bd40..eb6c47ef00 100644 --- a/Makefile +++

[PATCH 0/2] doc: describe building with GCC

2020-09-05 Thread Heinrich Schuchardt
Provide a newcomer friendly documentation how to obtain the U-Boot source and how to build U-Boot with GCC. Heinrich Schuchardt (2): doc: describe source repository doc: describe building with GCC doc/build/gcc.rst| 119 +++ doc/build/index.rst |

[PATCH 1/2] doc: describe source repository

2020-09-05 Thread Heinrich Schuchardt
Add a chapter to the HTML documentation describing how to retrieve the U-Boot sources. Signed-off-by: Heinrich Schuchardt --- doc/build/index.rst | 1 + doc/build/source.rst | 30 ++ 2 files changed, 31 insertions(+) create mode 100644 doc/build/source.rst diff

[PATCH 2/2] doc: describe building with GCC

2020-09-05 Thread Heinrich Schuchardt
Provide a description of the U-Boot build process with GCC in the HTML documentation. Signed-off-by: Heinrich Schuchardt --- doc/build/gcc.rst | 119 doc/build/index.rst | 1 + 2 files changed, 120 insertions(+) create mode 100644