Hi William,

On 23/07/2026 12:30, William Bright wrote:
> Introduce a defconfig for SM8550 based boards.

Thanks for the respin, this is looking good.
> 
> Signed-off-by: William Bright <[email protected]>
> ---
>  configs/qcom_sm8550_defconfig | 20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)
> 
> diff --git a/configs/qcom_sm8550_defconfig b/configs/qcom_sm8550_defconfig
> new file mode 100644
> index 00000000000..5d2fe13df68
> --- /dev/null
> +++ b/configs/qcom_sm8550_defconfig
> @@ -0,0 +1,20 @@
> +# Configuration for building U-Boot to be flashed
> +# to the uefi partition of SM8550 dev boards with
> +# the "Linux Embedded" partition layout (which have
> +# a dedicated "uefi_a" partition for edk2/U-Boot)
> +
> +#include "qcom_defconfig"
> +
> +# Otherwise buildman thinks this isn't an ARM platform
> +CONFIG_ARM=y
> +CONFIG_DEBUG_UART=y
> +CONFIG_DEBUG_UART_ANNOUNCE=y
> +CONFIG_DEBUG_UART_BASE=0xa9c000
> +CONFIG_DEBUG_UART_MSM_GENI=y
> +CONFIG_DEBUG_UART_CLOCK=14745600
> +
> +# Address where U-Boot will be loaded
> +CONFIG_TEXT_BASE=0xaf000000
> +CONFIG_QCOM_GENERATE_MBN=y
> +CONFIG_DEFAULT_DEVICE_TREE="qcom/sm8550-hdk"
> +CONFIG_OF_EMBED=y

Do we need OF_EMBED here? By default the .mbn is built from
u-boot-dtb.bin so the DTB is appended is already.

Assuming things work ok with that dropped

Reviewed-by: Casey Connolly <[email protected]>

> 

-- 
// Casey (she/her)

Reply via email to