diff mbox series

arm: mvebu: AC5/AC5X: Disable SMBIOS

Message ID 20231013030016.3307423-1-judge.packham@gmail.com
State Accepted
Commit d2387c6227715eee8fd07466ed4eeffdcc9f86d2
Delegated to: Stefan Roese
Headers show
Series arm: mvebu: AC5/AC5X: Disable SMBIOS | expand

Commit Message

Chris Packham Oct. 13, 2023, 3 a.m. UTC
The RD-AC5X doesn't make use of EFI or SMBIOS. Recently we started seeing
boot failures such as

    WARNING: SMBIOS table_address overflow 27f60f020
    Failed to write SMBIOS table
    initcall failed at event 10/(unknown) (err=-22)
    ### ERROR ### Please RESET the board ###

The error is because the physical address of the RAM on the AC5X SoC is
above the 32GiB boundary. As we don't need SMBIOS or EFI this can be
safely disabled.

Signed-off-by: Chris Packham <judge.packham@gmail.com>
---
This probably should have been part of the series I sent as
https://lore.kernel.org/u-boot/20231003035800.2626121-1-judge.packham@gmail.com/
but I didn't have the RD-AC5X board at the time. There is another issue
that's stopping the RD-AC5X board from booting with current master but
with the suspect commit reverted you can see the same SMBIOS issue I was
seeing on the x240.

 configs/mvebu_ac5_rd_defconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Stefan Roese Oct. 16, 2023, 9:02 a.m. UTC | #1
On 10/13/23 05:00, Chris Packham wrote:
> The RD-AC5X doesn't make use of EFI or SMBIOS. Recently we started seeing
> boot failures such as
> 
>      WARNING: SMBIOS table_address overflow 27f60f020
>      Failed to write SMBIOS table
>      initcall failed at event 10/(unknown) (err=-22)
>      ### ERROR ### Please RESET the board ###
> 
> The error is because the physical address of the RAM on the AC5X SoC is
> above the 32GiB boundary. As we don't need SMBIOS or EFI this can be
> safely disabled.
> 
> Signed-off-by: Chris Packham <judge.packham@gmail.com>

Reviewed-by: Stefan Roese <sr@denx.de>

Thanks,
Stefan

> ---
> This probably should have been part of the series I sent as
> https://lore.kernel.org/u-boot/20231003035800.2626121-1-judge.packham@gmail.com/
> but I didn't have the RD-AC5X board at the time. There is another issue
> that's stopping the RD-AC5X board from booting with current master but
> with the suspect commit reverted you can see the same SMBIOS issue I was
> seeing on the x240.
> 
>   configs/mvebu_ac5_rd_defconfig | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/configs/mvebu_ac5_rd_defconfig b/configs/mvebu_ac5_rd_defconfig
> index dbf1e3136cdf..e8fa22b648be 100644
> --- a/configs/mvebu_ac5_rd_defconfig
> +++ b/configs/mvebu_ac5_rd_defconfig
> @@ -85,3 +85,4 @@ CONFIG_USB_ETHER_ASIX88179=y
>   CONFIG_USB_ETHER_MCS7830=y
>   CONFIG_USB_ETHER_RTL8152=y
>   CONFIG_USB_ETHER_SMSC95XX=y
> +# CONFIG_SMBIOS is not set

Viele Grüße,
Stefan Roese
Stefan Roese Oct. 16, 2023, 2:45 p.m. UTC | #2
On 10/16/23 11:02, Stefan Roese wrote:
> On 10/13/23 05:00, Chris Packham wrote:
>> The RD-AC5X doesn't make use of EFI or SMBIOS. Recently we started seeing
>> boot failures such as
>>
>>      WARNING: SMBIOS table_address overflow 27f60f020
>>      Failed to write SMBIOS table
>>      initcall failed at event 10/(unknown) (err=-22)
>>      ### ERROR ### Please RESET the board ###
>>
>> The error is because the physical address of the RAM on the AC5X SoC is
>> above the 32GiB boundary. As we don't need SMBIOS or EFI this can be
>> safely disabled.
>>
>> Signed-off-by: Chris Packham <judge.packham@gmail.com>
> 
> Reviewed-by: Stefan Roese <sr@denx.de>

Applied to u-boot-marvell/master

Thanks,
Stefan


> Thanks,
> Stefan
> 
>> ---
>> This probably should have been part of the series I sent as
>> https://lore.kernel.org/u-boot/20231003035800.2626121-1-judge.packham@gmail.com/
>> but I didn't have the RD-AC5X board at the time. There is another issue
>> that's stopping the RD-AC5X board from booting with current master but
>> with the suspect commit reverted you can see the same SMBIOS issue I was
>> seeing on the x240.
>>
>>   configs/mvebu_ac5_rd_defconfig | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/configs/mvebu_ac5_rd_defconfig 
>> b/configs/mvebu_ac5_rd_defconfig
>> index dbf1e3136cdf..e8fa22b648be 100644
>> --- a/configs/mvebu_ac5_rd_defconfig
>> +++ b/configs/mvebu_ac5_rd_defconfig
>> @@ -85,3 +85,4 @@ CONFIG_USB_ETHER_ASIX88179=y
>>   CONFIG_USB_ETHER_MCS7830=y
>>   CONFIG_USB_ETHER_RTL8152=y
>>   CONFIG_USB_ETHER_SMSC95XX=y
>> +# CONFIG_SMBIOS is not set
> 
> Viele Grüße,
> Stefan Roese
> 

Viele Grüße,
Stefan Roese
diff mbox series

Patch

diff --git a/configs/mvebu_ac5_rd_defconfig b/configs/mvebu_ac5_rd_defconfig
index dbf1e3136cdf..e8fa22b648be 100644
--- a/configs/mvebu_ac5_rd_defconfig
+++ b/configs/mvebu_ac5_rd_defconfig
@@ -85,3 +85,4 @@  CONFIG_USB_ETHER_ASIX88179=y
 CONFIG_USB_ETHER_MCS7830=y
 CONFIG_USB_ETHER_RTL8152=y
 CONFIG_USB_ETHER_SMSC95XX=y
+# CONFIG_SMBIOS is not set