Hi Simon,
Thanks for looping me in
On Sat, 18 Nov 2023 at 19:11, Simon Glass wrote:
>
> +Ilias too as this involves a design decision
>
> Hi Shantur,
>
> On Fri, 17 Nov 2023 at 14:22, Shantur Rathore wrote:
> >
> > efi_set_bootdev is already called as part of tftp while doing dhcp_run()
>
> Is
On 11/19/23 02:04, Simon Holesch wrote:
The ChipIdea device controller wasn't properly cleaned up when disabled.
So enabling it again left it in a broken state. The problem occurred for
example when the host unbinds the driver and binds it again.
During the first setup, when the out request is q
On 11/19/23 02:04, Simon Holesch wrote:
Please also add missing commit message (can be just the subject copied
into the body)
Signed-off-by: Simon Holesch
---
drivers/usb/gadget/f_fastboot.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/gadget/f_fastboot.
On Sat, Nov 18, 2023 at 10:10:04AM -0700, Simon Glass wrote:
> Hi Chris,
>
> On Fri, 17 Nov 2023 at 14:15, Chris Morgan wrote:
> >
> > From: Chris Morgan
> >
> > Make the inability to detect a panel using the auto detection code not
> > fail the entire boot process. This means that if the panel
Signed-off-by: Simon Holesch
---
drivers/usb/gadget/f_fastboot.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/gadget/f_fastboot.c b/drivers/usb/gadget/f_fastboot.c
index 741775a7bc..9f322c9550 100644
--- a/drivers/usb/gadget/f_fastboot.c
+++ b/drivers/usb/gadget
The ChipIdea device controller wasn't properly cleaned up when disabled.
So enabling it again left it in a broken state. The problem occurred for
example when the host unbinds the driver and binds it again.
During the first setup, when the out request is queued, the endpoint is
primed (`epprime`).
On 11/19/23 01:43, Simon Glass wrote:
Hi Heinrich,
On Sat, 18 Nov 2023 at 15:52, Heinrich Schuchardt
wrote:
On 11/18/23 22:28, Shantur Rathore wrote:
Hi Heinrich,
On Fri, Nov 17, 2023 at 3:12 PM Heinrich Schuchardt
wrote:
On 11/16/23 19:45, Shantur Rathore wrote:
On Thu, Nov 16, 2023 at
Hi Heinrich,
On Sat, 18 Nov 2023 at 16:06, Heinrich Schuchardt
wrote:
>
> On 11/18/23 18:58, Simon Glass wrote:
> > Hi Heinrich,
> >
> > On Sat, 18 Nov 2023 at 10:34, Heinrich Schuchardt
> > wrote:
> >>
> >> On 11/18/23 18:10, Simon Glass wrote:
> >>> Hi Heinrich,
> >>>
> >>> On Thu, 16 Nov 2023
Hi Shantur,
On Sat, 18 Nov 2023 at 14:17, Shantur Rathore wrote:
>
> Hi Simon,
>
> On Sat, Nov 18, 2023 at 5:11 PM Simon Glass wrote:
> >
> > +Ilias too as this involves a design decision
> >
> > Hi Shantur,
> >
> > On Fri, 17 Nov 2023 at 14:22, Shantur Rathore wrote:
> > >
> > > efi_set_bootde
On Sat, 18 Nov 2023 at 15:07, Shantur Rathore wrote:
>
> We need to set boot->fname before calling efi_set_bootdev
> otherwise this crashes as bflow->fname is null.
>
> Signed-off-by: Shantur Rathore
> ---
> boot/bootmeth_efi.c | 8 +++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
>
Re
On Sat, 18 Nov 2023 at 15:54, Heinrich Schuchardt
wrote:
>
> ACPI tables may comprise either RSDT, XSDT, or both. The current code fails
> to check the presence of the RSDT table before accessing it. This leads to
> an exception if the RSDT table is not provided.
>
> The XSDT table takes precedenc
Hi Tom,
On Sat, 18 Nov 2023 at 12:15, Tom Rini wrote:
>
> Splitting things in two again..
>
> On Sat, Nov 18, 2023 at 10:10:17AM -0700, Simon Glass wrote:
>
> [snip]
> > This is the bug report of trying to boot into U-Boot on arm64 with
> > more than 3GB of memory.
>
> OK, so can we just please f
On Sat, 18 Nov 2023 at 15:07, Shantur Rathore wrote:
>
> While booting with efi, if fdt isn't available externally,
> just use the built-in one.
>
> Signed-off-by: Shantur Rathore
> ---
Change log needed when you send a new version
> boot/bootmeth_efi.c | 19 +--
> include/boot
On Sat, 18 Nov 2023 at 14:57, Heinrich Schuchardt
wrote:
>
> Provide a unit test for acpi_find_table()
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v2:
> new patch
> ---
> test/dm/acpi.c | 96 ++
> 1 file changed, 96 insertions(+)
>
Revi
Hi Shantur,
On Sat, 18 Nov 2023 at 15:07, Shantur Rathore wrote:
>
> bootmeth_efi doesn't allocate any buffer to load efi in any case.
> enable static buffer flag for all cases.
>
> Signed-off-by: Shantur Rathore
> ---
> boot/bootmeth_efi.c | 7 ++-
> 1 file changed, 6 insertions(+), 1 dele
Hi Heinrich,
On Sat, 18 Nov 2023 at 15:52, Heinrich Schuchardt
wrote:
>
> On 11/18/23 22:28, Shantur Rathore wrote:
> > Hi Heinrich,
> >
> > On Fri, Nov 17, 2023 at 3:12 PM Heinrich Schuchardt
> > wrote:
> >>
> >> On 11/16/23 19:45, Shantur Rathore wrote:
> >>> On Thu, Nov 16, 2023 at 6:15 PM He
On Sat, 18 Nov 2023 at 14:57, Heinrich Schuchardt
wrote:
>
> The RSDT table is deprecated and does not exist on all systems.
>
> By preference scan XSDT for the table to find. If no XSDT table exists, try
> to use the RSDT table.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> v2:
> consid
Hi Shantur,
On Sat, 18 Nov 2023 at 15:04, Shantur Rathore wrote:
>
> bootmeth_efi sets up bootp_arch which is read later in bootp.c
> Currently bootp_arch is being set as integer string and being
> read in bootp.c as hex, this sends incorrect arch value to dhcp server
> which in return sends wron
On Tue, 14 Nov 2023 at 13:34, Shantur Rathore wrote:
>
> Signed-off-by: Shantur Rathore
> ---
> arch/arm/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
Reviewed-by: Simon Glass
But it needs a commit message
>
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index d812685c98..fca6ef6d7
On Sat, 18 Nov 2023 at 14:52, Heinrich Schuchardt
wrote:
>
> * Converting to void * is superfluous when calling memset().
> * acpi_fill_header() already fills oem_table_id.
>
> Fixes: d953137526cc ("x86: Move SSDT table to a writer function")
> Signed-off-by: Heinrich Schuchardt
> ---
> lib/acpi
On 11/18/23 18:58, Simon Glass wrote:
Hi Heinrich,
On Sat, 18 Nov 2023 at 10:34, Heinrich Schuchardt
wrote:
On 11/18/23 18:10, Simon Glass wrote:
Hi Heinrich,
On Thu, 16 Nov 2023 at 02:29, Heinrich Schuchardt
wrote:
ACPI tables cannot convey memory reservations for least ARM and RISC-V.
The size of the ACPI table header is not a multiple of 8. We have to mark
struct acpi_xsdt as packed to correctly access field Entry.
Add a unit test for the offsets of field Entry in the RSDT and XSDT tables.
Signed-off-by: Heinrich Schuchardt
Reviewed-by: Simon Glass
---
v3:
no change
The size of the ACPI table header is not a multiple of 8. We have to mark
struct acpi_xsdt as packed to correctly access field Entry.
Add a unit test for the offsets of field Entry in the RSDT and XSDT tables.
ACPI tables may comprise either RSDT, XSDT, or both. The current code fails
to check th
ACPI tables may comprise either RSDT, XSDT, or both. The current code fails
to check the presence of the RSDT table before accessing it. This leads to
an exception if the RSDT table is not provided.
The XSDT table takes precedence over the RSDT table.
The return values of list_rsdt() and list_rsd
On 11/18/23 22:28, Shantur Rathore wrote:
Hi Heinrich,
On Fri, Nov 17, 2023 at 3:12 PM Heinrich Schuchardt
wrote:
On 11/16/23 19:45, Shantur Rathore wrote:
On Thu, Nov 16, 2023 at 6:15 PM Heinrich Schuchardt
wrote:
On 11/16/23 17:52, Shantur Rathore wrote:
Hi Simon,
Currently bootstd -
From: Yang Xiwen
assign clk_dev_ops(clkp->dev) to ops to ensure correct clk operations
are called on clocks.
This fixes the incorrect enable_count issue as described in [1].
[1]:
https://lore.kernel.org/all/sezpr06mb695927a6deeef8489a06897396...@sezpr06mb6959.apcprd06.prod.outlook.com/
Signed
From: Yang Xiwen
It can now act as an clk provider on which ccf_clk_ops can be tested.
Also add "#clock-cells=<1>" to test.dts.
Signed-off-by: Yang Xiwen
---
arch/sandbox/dts/test.dts | 1 +
drivers/clk/clk_sandbox_ccf.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/arch/sandbox/dt
From: Yang Xiwen
This clock is added to dts. Get it in the devm group in the driver or
the testcases will fail.
Signed-off-by: Yang Xiwen
---
arch/sandbox/dts/test.dts | 5 +++--
arch/sandbox/include/asm/clk.h | 1 +
drivers/clk/clk_sandbox_test.c | 5 +
3 files changed, 9 insertions(
As described in [1], enable_count is incremented by 2 when
ccf_clk_enable() is called. This series of patch fixed this issue and
added a testcase for that.
[1]:
https://lore.kernel.org/all/sezpr06mb695927a6deeef8489a06897396...@sezpr06mb6959.apcprd06.prod.outlook.com/
Signed-off-by: Yang Xiwen
From: Yang Xiwen
get i2c_root clock from device tree. In this way we get an CCF clock and
also test ccf_clk_ops.
Signed-off-by: Yang Xiwen
---
test/dm/clk_ccf.c | 14 +++---
1 file changed, 11 insertions(+), 3 deletions(-)
diff --git a/test/dm/clk_ccf.c b/test/dm/clk_ccf.c
index e4ebb
bootmeth_efi doesn't allocate any buffer to load efi in any case.
enable static buffer flag for all cases.
Signed-off-by: Shantur Rathore
---
boot/bootmeth_efi.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/boot/bootmeth_efi.c b/boot/bootmeth_efi.c
index 370d6fba9f..
While booting with efi, if fdt isn't available externally,
just use the built-in one.
Signed-off-by: Shantur Rathore
---
boot/bootmeth_efi.c | 19 +--
include/bootflow.h | 2 ++
2 files changed, 15 insertions(+), 6 deletions(-)
diff --git a/boot/bootmeth_efi.c b/boot/bootmeth_
We need to set boot->fname before calling efi_set_bootdev
otherwise this crashes as bflow->fname is null.
Signed-off-by: Shantur Rathore
---
boot/bootmeth_efi.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/boot/bootmeth_efi.c b/boot/bootmeth_efi.c
index 682cf5b23b..
From: Yang Xiwen
It can now act as an clk provider on which ccf_clk_ops can be tested.
Also add "#clock-cells=<1>" to test.dts.
---
arch/sandbox/dts/test.dts | 1 +
drivers/clk/clk_sandbox_ccf.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/arch/sandbox/dts/test.dts b/arch/sandbox/dt
From: Yang Xiwen
get i2c_root clock from device tree. In this way we get an CCF clock and
also test ccf_clk_ops.
Signed-off-by: Yang Xiwen
---
test/dm/clk_ccf.c | 14 +++---
1 file changed, 11 insertions(+), 3 deletions(-)
diff --git a/test/dm/clk_ccf.c b/test/dm/clk_ccf.c
index e4ebb
As described in [1], enable_count is incremented by 2 when
ccf_clk_enable() is called. This series of patch fixed this issue and
added a testcase for that.
[1]:
https://lore.kernel.org/all/sezpr06mb695927a6deeef8489a06897396...@sezpr06mb6959.apcprd06.prod.outlook.com/
Signed-off-by: Yang Xiwen
From: Yang Xiwen
assign clk_dev_ops(clkp->dev) to ops to ensure correct clk operations
are called on clocks.
This fixes the incorrect enable_count issue as described in [1].
[1]:
https://lore.kernel.org/all/sezpr06mb695927a6deeef8489a06897396...@sezpr06mb6959.apcprd06.prod.outlook.com/
Signed
From: Yang Xiwen
This clock is added to dts. Get it in the devm group in the driver or
the testcases will fail.
Signed-off-by: Yang Xiwen
---
arch/sandbox/dts/test.dts | 5 +++--
arch/sandbox/include/asm/clk.h | 1 +
drivers/clk/clk_sandbox_test.c | 5 +
3 files changed, 9 insertions(
bootmeth_efi sets up bootp_arch which is read later in bootp.c
Currently bootp_arch is being set as integer string and being
read in bootp.c as hex, this sends incorrect arch value to dhcp server
which in return sends wrong file for network boot.
For ARM64 UEFI Arch value is 0xb (11), here we set
Currently bootmeth_efi crashes while doing a network (dhcp) boot.
This patch series fixes issues and both network and disk boot works.
Shantur Rathore (4):
bootflow: bootmeth_efi: Set bootp_arch as hex
bootflow: bootmeth_efi: set bflow->fname from bootfile name
bootflow: bootmeth_efi: Handle
The RSDT table is deprecated and does not exist on all systems.
By preference scan XSDT for the table to find. If no XSDT table exists, try
to use the RSDT table.
Signed-off-by: Heinrich Schuchardt
---
v2:
consider that map_sysmem(0, 0) != NULL
---
lib/acpi/acpi.c | 20 -
Provide a unit test for acpi_find_table()
Signed-off-by: Heinrich Schuchardt
---
v2:
new patch
---
test/dm/acpi.c | 96 ++
1 file changed, 96 insertions(+)
diff --git a/test/dm/acpi.c b/test/dm/acpi.c
index 5997bda649..58cc6c3474 100644
--
The RSDT table is deprecated and does not exist on all systems.
By preference scan XSDT for the table to find. If no XSDT table exists, try
to use the RSDT table.
Provide a unit test for acpi_find_table().
v2:
Consider that map_sysmem(0, 0) != NULL.
Add unit test.
*** BLURB HERE
* Converting to void * is superfluous when calling memset().
* acpi_fill_header() already fills oem_table_id.
Fixes: d953137526cc ("x86: Move SSDT table to a writer function")
Signed-off-by: Heinrich Schuchardt
---
lib/acpi/ssdt.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --g
Hi Heinrich,
On Fri, Nov 17, 2023 at 3:12 PM Heinrich Schuchardt
wrote:
>
> On 11/16/23 19:45, Shantur Rathore wrote:
> > On Thu, Nov 16, 2023 at 6:15 PM Heinrich Schuchardt
> > wrote:
> >>
> >> On 11/16/23 17:52, Shantur Rathore wrote:
> >>> Hi Simon,
> >>>
> >>> Currently bootstd - bootmethod_
Hi Simon,
On Sat, Nov 18, 2023 at 5:11 PM Simon Glass wrote:
>
> +Ilias too as this involves a design decision
>
> Hi Shantur,
>
> On Fri, 17 Nov 2023 at 14:22, Shantur Rathore wrote:
> >
> > efi_set_bootdev is already called as part of tftp while doing dhcp_run()
>
> Is that in tftp_complete()
It would be useful to be able to boot an OS when CONFIG_CMDLINE is
disabled. This could allow reduced code size.
Standard boot provides a way to handle programmatic boot, without
scripts, so such a feature is possible. The main impediment is the
inability to use the booting features of U-Boot with
At present bootstd requires CONFIG_CMDLINE to operate. Add a new
'programmatic' boot which can be used when no command line is available.
For now it does almost nothing, since most bootmeths require the
command line.
Signed-off-by: Simon Glass
---
Changes in v3:
- Add a panic if programmatic boo
This function only uses two arguments. The 'arch' always has a constant
value, so drop it. This simplifies the function call.
Tidy up the function comment while we are here.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
---
(no changes since v1)
boot/bootm.c | 3 +--
boot/image-boa
This function only uses one argument from bootm (argv[2]) so pass it in
directly.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
---
(no changes since v1)
boot/bootm.c | 5 +++--
boot/image-fdt.c | 9 +++--
include/image.h | 5 ++---
3 files changed, 8 insertions(+), 11 deletions(-
This function only uses two arguments. The 'arch' always has a constant
value, so drop it. This simplifies the function call.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
---
(no changes since v1)
boot/bootm.c | 3 +--
boot/image-board.c | 5 ++---
include/image.h| 9 +++--
The normal bootm flow calls bootm_find_other() can call the
BOOTM_STATE_FINDOTHER state as part of its processing. Fix the
condition there so that this hack can be removed.
Also drop the confusing check for the OS type, since do_bootm_tee() is
only called if the condition is met - see bootm_os_get
The Android mechanism uses the loadaddr envrionment-variable to get the
load address, if none is provided. This is equivalent to
image_load_addr so use that instead, converting it to a string as
needed.
This change will permit passing img_addr to this function, in a future
change.
Signed-off-by:
Add some functions which provide an argument to a command, or NULL if
the argument does not exist.
Use the same numbering as argv[] since it seems less confusing than the
previous idea.
Signed-off-by: Simon Glass
Suggested-by: Tom Rini
---
Changes in v3:
- Drop RFC tag
Changes in v2:
- Add ne
The normal bootm flow calls bootm_find_other() can call the
BOOTM_STATE_FINDOTHER state as part of its processing. Fix the
condition there so that this hack can be removed.
Signed-off-by: Simon Glass
---
(no changes since v1)
boot/bootm.c| 10 +-
boot/bootm_os.c | 6 --
2 file
Rather than passing it all the command-line args, pass in the pieces
that it needs. These are the image address, the ramdisk address/name
and the FDT address/name.
Ultimately this will allow usage of this function without being called
from the command line.
Move the function comment to the header
Move this code into a function to reduce code size and make it easier
to understand. Drop the unnecessary 0x to help a little with code size.
Use this in bootm_find_images()
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
---
(no changes since v1)
boot/bootm.c | 51 +
Rather than passing the full list of command arguments, pass only those
which are needed.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
---
(no changes since v1)
boot/bootm.c | 20 +++-
1 file changed, 11 insertions(+), 9 deletions(-)
diff --git a/boot/bootm.c b/boot/bootm
The OF_LIBFDT #ifdef makes it harder to use a local variable for that
code block. Convert it to if() instead.
Signed-off-by: Simon Glass
---
(no changes since v1)
boot/bootm.c | 42 +-
1 file changed, 21 insertions(+), 21 deletions(-)
diff --git a/boot/
These unmaps should happen regardless of the return value. Move them
before the 'return' statement.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
---
(no changes since v2)
Changes in v2:
- Add new patch to adjust position of unmap_sysmem() in boot_get_kernel()
boot/bootm.c | 4 ++--
1 fil
Rather than having boot_get_fdt() calculate this, move the calculation
into the caller. This removes the access to argv[0] in this function,
so we can later refactor it to just accept argv[2] instead of the whole
argv[].
Move the function comment to the header file and fix the u8 argument,
while w
Most of the fdt_support.h header file is included only if OF_LIBFDT or
OF_CONTROL are enabled. This means that calling functions defined in
that file must happen inside an #ifdef
This is unnecessary, so reduce the condition to just !USE_HOSTCC
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
--
This function normally only uses one argument so pass it in directly.
Move comments to the header file so could one day include these
functions in API docs. Fix up the u8 argument while here, since it
avoids the compiler having to mask the value on some machines.
The Android case here is bit stran
This function only uses one argument so pass it in directly.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
---
(no changes since v2)
Changes in v2:
- Fix 'boot_find_os' typo
- Pass in the command name
boot/bootm.c | 17 +++--
1 file changed, 11 insertions(+), 6 deletions(-)
d
The same error message is printed in two places. Move it out to the
caller so we can avoid passing in the command name. Leave the bootstage
handling where it is.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
---
(no changes since v2)
Changes in v2:
- Use the command table to provide the com
The U_BOOT_CMD_COMPLETE() macro has a semicolon at the end, perhaps
inadvertently. Some code has taken advantage of this.
Tidy this up by dropping the semicolon from the macro and adding it to
macro invocations as required.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
Reviewed-by: Ilias Apa
This option is defined in both SPL and TPL builds, so correct the docs
related to this. Also point to spl_phase() which is normally a better
option. Mention VPL as well.
Signed-off-by: Simon Glass
Reported-by: Heinrich Schuchardt
---
(no changes since v1)
README | 26 ++---
This is a misnomer since we don't necessarily know that the image is a
FIT. Use the existing BOOTSTAGE_ID_CHECK_IMAGETYPE instead.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
---
(no changes since v1)
boot/bootm.c| 2 +-
include/bootstage.h | 1 -
2 files changed, 1 insertion(+),
Rather than looking for a zero-sized image, use the error code returned
to determine if things are OK.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
---
(no changes since v2)
Changes in v2:
- Use the command table to provide the command name, instead of "bootm"
boot/bootm.c | 6 +++---
1
Move the argument decoding to the caller, to avoid needing to pass the
command-line arguments.
Add a function comment while we are here.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
---
(no changes since v1)
boot/bootm.c | 23 ---
1 file changed, 16 insertions(+), 7 d
Put a list of these in the function documentation so it is easier to
decode what went wrong.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
---
(no changes since v1)
include/image.h | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/include/image.h b/include/image.h
This function obtains lots of error codes and then throws them away.
Update it to return the error, moving the image pointer to an
argument.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
---
(no changes since v2)
Changes in v2:
- Use the command table to provide the command name, instead of
This function only uses one argument and just needs to know the name of
the command which called it. Adjust the function to use only what it
needs. This will make it easier to call from a non-command context.
Tidy up the function comment while we are here.
Signed-off-by: Simon Glass
Reviewed-by:
This function does not modify its first argument, so mark it const. Also
move the comments to the header file and expand them to provide more
useful information.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
---
(no changes since v1)
boot/bootm.c | 3 +--
boot/image-board.c | 17 +--
This function does not use its arguments. Drop them.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
Reviewed-by: Ilias Apalodimas
---
(no changes since v1)
boot/bootm.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/boot/bootm.c b/boot/bootm.c
index cb61485c226c..
Add a dm/uclass-id.h to the bootdev header file, since it uses
enum uclass_id
Signed-off-by: Simon Glass
Reviewed-by: Ilias Apalodimas
---
(no changes since v1)
include/bootdev.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/bootdev.h b/include/bootdev.h
index b079a91b5b7f..c136
Move this code and image_get_kernel() higher in the file to avoid the
need for a forward declaration.
No attempt is made to remove #ifdefs or adjust the code in any other
way.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
---
(no changes since v1)
boot/bootm.c | 376 ++
The declaration of U_BOOT_ENV_LOCATION() uses a different #ifdef
condition from the code it calls. Use the same condition to avoid a
build warning if CONFIG_CMD_SAVEENV is disabled.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
---
(no changes since v1)
env/mmc.c | 2 +-
1 file changed, 1
Since we can now cleanly disable CMDLINE when needed, drop the rules
which discard the command code. It will not be built in the first
place.
Signed-off-by: Simon Glass
Reviewed-by: Tom Rini
Reviewed-by: Ilias Apalodimas
---
(no changes since v1)
arch/arm/cpu/u-boot.lds | 3 ---
arch/x8
Hi Tom,
The following changes since commit ae7ec8b0be41b59ef323f7531c0fe6745e8fef45:
Merge branch '2023-11-16-assorted-updates' into next (2023-11-17 08:38:05
-0500)
are available in the Git repository at:
https://source.denx.de/u-boot/custodians/u-boot-tpm/ tags/efi-next-18112023
for you
Splitting things in two again..
On Sat, Nov 18, 2023 at 10:10:17AM -0700, Simon Glass wrote:
[snip]
> This is the bug report of trying to boot into U-Boot on arm64 with
> more than 3GB of memory.
OK, so can we just please fix this, for v2024.01, in a way that won't
make things harder? Fixed memo
On 11/12/23 21:01, Simon Glass wrote:
On Sun, 12 Nov 2023 at 00:03, Heinrich Schuchardt
wrote:
ACPI tables may comprise either RSDT, XSDT, or both. The current code fails
to check the presence of the RSDT table before accessing it. This leads to
an exception if the RSDT table is not provided.
When using DFU alt info via EFI capsule updates on an Ultra96 V2 board,
`dfu_alt_info` is automatically set to write a `boot.bin` file to the SD
card when in SD boot mode and not using multiboot. After the update is
applied, the board reboots and does not come back up. When writing an
upper-case `B
The zynqmp bootrom will not boot from an SD card with a fat32 fileystem
with a lowercase boot.bin file on it. This means that using DFU to
update the boot.bin file will render the device unusable since it is not
written in upper-case.
Signed-off-by: Jared Baur
---
board/xilinx/zynqmp/zynqmp.c |
On Sat, Nov 18, 2023 at 10:58:50AM -0700, Simon Glass wrote:
[snip]
> I found these:
>
> https://patchwork.ozlabs.org/project/uboot/list/?series=262148&state=*
> http://patchwork.ozlabs.org/project/uboot/patch/20210925121958.26001-1-p...@kernel.org/
>
> IMO the second one from Pali makes sense,
Splitting this in to two different threads..
On Sat, Nov 18, 2023 at 10:58:50AM -0700, Simon Glass wrote:
> Hi Tom,
>
> On Sat, 18 Nov 2023 at 10:45, Tom Rini wrote:
> >
> > On Sat, Nov 18, 2023 at 10:09:55AM -0700, Simon Glass wrote:
> > > Hi Tom,
> > >
> > > On Fri, 17 Nov 2023 at 15:54, Tom R
Hi,
On Sun, 1 Oct 2023 at 19:16, Simon Glass wrote:
>
> Move the code from the 'cls' command into the console file, so it can
> be called from elsewhere.
>
> Signed-off-by: Simon Glass
> ---
>
> cmd/cls.c | 25 +++--
> common/console.c | 31 +
Hi Heinrich,
On Sat, 18 Nov 2023 at 10:34, Heinrich Schuchardt
wrote:
>
> On 11/18/23 18:10, Simon Glass wrote:
> > Hi Heinrich,
> >
> > On Thu, 16 Nov 2023 at 02:29, Heinrich Schuchardt
> > wrote:
> >>
> >> ACPI tables cannot convey memory reservations for least ARM and RISC-V.
> >> x86 uses th
Hi Tom,
On Sat, 18 Nov 2023 at 10:45, Tom Rini wrote:
>
> On Sat, Nov 18, 2023 at 10:09:55AM -0700, Simon Glass wrote:
> > Hi Tom,
> >
> > On Fri, 17 Nov 2023 at 15:54, Tom Rini wrote:
> > >
> > > Remove and replace common.h and config.h in sandbox when it's not needed
> > > and add some explici
On Sat, Nov 18, 2023 at 10:09:55AM -0700, Simon Glass wrote:
> Hi Tom,
>
> On Fri, 17 Nov 2023 at 15:54, Tom Rini wrote:
> >
> > Remove and replace common.h and config.h in sandbox when it's not needed
> > and add some explicit includes where needed.
> >
> > Signed-off-by: Tom Rini
> > ---
> >
On 11/18/23 18:10, Simon Glass wrote:
Hi Heinrich,
On Thu, 16 Nov 2023 at 02:29, Heinrich Schuchardt
wrote:
ACPI tables cannot convey memory reservations for least ARM and RISC-V.
x86 uses the BIOS E820 table for this purpose. We cannot simply ignore the
device-tree when booting via ACPI.
W
On Fri, 17 Nov 2023 at 15:53, Tom Rini wrote:
>
> Given how we define QE_MURAM_SIZE today, this header needs to have
> added to it.
>
> Signed-off-by: Tom Rini
> ---
> include/linux/immap_qe.h | 2 ++
> 1 file changed, 2 insertions(+)
>
Reviewed-by: Simon Glass
Hi Tom,
On Fri, 3 Nov 2023 at 14:13, Tom Rini wrote:
>
> On Fri, Nov 03, 2023 at 01:38:59PM -0600, Simon Glass wrote:
> > Hi Tom,
> >
> > On Fri, 3 Nov 2023 at 13:26, Tom Rini wrote:
> > >
> > > On Fri, Nov 03, 2023 at 12:14:46PM -0600, Simon Glass wrote:
> > > > Hi Heinrich,
> > > >
> > > > On
Hi,
On Fri, 4 Aug 2023 at 05:05, Macpaul Lin wrote:
>
> From: Fabien Parent
>
> The MediaTek MT8195 is a ARM64-based SoC with a quad-core Cortex-A73 and
> a quad-core Cortex-A53. It is including UART, SPI, USB3.0 device and hosts,
> SD and MMC cards, UFS, PWM, I2C, I2S, S/PDIF, and several LPDDR
Hi Tom,
On Tue, 14 Nov 2023 at 11:04, Tom Rini wrote:
>
> On Tue, Nov 14, 2023 at 11:38:46AM -0500, Tom Rini wrote:
> > On Sat, Nov 11, 2023 at 08:49:55PM -0700, Simon Glass wrote:
> >
> > > The kernel_noload image type indicates that no loading is to be done by
> > > U-Boot. This works well when
Hi Heinrich,
On Tue, 14 Nov 2023 at 09:39, Heinrich Schuchardt
wrote:
>
> The RSDT table is deprecated and does not exist on all systems.
>
> By preference scan XSDT for the table to find. If no XSDT table exists, try
> to use the RSDT table.
>
> Signed-off-by: Heinrich Schuchardt
> ---
> lib/a
Hi Heinrich,
On Thu, 16 Nov 2023 at 02:29, Heinrich Schuchardt
wrote:
>
> ACPI tables cannot convey memory reservations for least ARM and RISC-V.
> x86 uses the BIOS E820 table for this purpose. We cannot simply ignore the
> device-tree when booting via ACPI.
Why is that? I had thought that we h
Hi Heinrich,
On Thu, 16 Nov 2023 at 12:43, Heinrich Schuchardt
wrote:
>
> On 11/16/23 19:33, Tom Rini wrote:
> > On Wed, Nov 15, 2023 at 03:23:50PM +0100, Heinrich Schuchardt wrote:
> >
> >> RISC-V QEMU provides the ACPI tables. We do not need to generate them
> >> ourselves.
> >>
> >> Signed-off
Hi Chris,
On Fri, 17 Nov 2023 at 14:15, Chris Morgan wrote:
>
> From: Chris Morgan
>
> Make the inability to detect a panel using the auto detection code not
> fail the entire boot process. This means that if the panel ID cannot
> be read we don't set an environment variable for the panel, and i
On Fri, 17 Nov 2023 at 15:54, Tom Rini wrote:
>
> In order to make it easier to move on to dropping common.h from code
> directly, remove common.h inclusion from the rest of the header file
> which had been including it.
>
> Signed-off-by: Tom Rini
> ---
> board/BuR/common/br_resetc.h
1 - 100 of 118 matches
Mail list logo
