[PATCH v1 2/2] configs: qcom_qcs9100: Fix fastboot buffer address for QCS9100 board

[Casey Connolly]

On 30/06/2025 09:00, Balaji Selvanathan wrote:
> The default value of CONFIG_FASTBOOT_BUF_ADDR is 0, which causes
> NULL pointer dereference during fastboot commands.
> 
> Set it to 0xdb300000, a safe and sufficiently large region in RAM
> of the QCS9100 board, to prevent crashes and ensure reliable
> fastboot functionality.
> 
> Signed-off-by: Balaji Selvanathan <balaji.selvanathan at oss.qualcomm.com>

fwiw since you're using the default environment
(board/qualcomm/default.env) it defines an alias for the fastboot
command to use a region allocated at runtime (which is tracked by U-Boot
and guarunteed not to overlap with anything else). At the U-Boot prompt
just do "run fastboot" to execute the alias (which expands to "fastboot
-l $fastboot_addr_r usb 0")

Ideally we would get rid of this workaround and instead teach fastboot
to prioritise the runtime address from that variable if it's set.

Hard-coding the address like this could cause issues if it happens to
overlap with some LMB allocated region.

If you have the cycles free to fix it properly I would be very greatful,
but I'm happy to take this in the meantime.

Reviewed-by: Casey Connolly <casey.connolly at linaro.org>

Kind regards,

> ---
>  configs/qcom_qcs9100_defconfig | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/configs/qcom_qcs9100_defconfig b/configs/qcom_qcs9100_defconfig
> index 10ff4d25398..5b6475dcfe0 100644
> --- a/configs/qcom_qcs9100_defconfig
> +++ b/configs/qcom_qcs9100_defconfig
> @@ -15,4 +15,6 @@ CONFIG_DEBUG_UART_CLOCK=14745600
>  CONFIG_TEXT_BASE=0xaf000000
>  CONFIG_REMAKE_ELF=y
>  
> +CONFIG_FASTBOOT_BUF_ADDR=0xdb300000
> +
>  CONFIG_DEFAULT_DEVICE_TREE="qcom/qcs9100-ride-r3"

-- 
// Casey (she/her)