Re: [PATCH next] soc: qcom: smem: Fix signedness bug in smem_dram_parse()
From: Konrad Dybcio
Date: Mon Aug 17 2026 - 06:14:09 EST
On 8/13/26 9:09 AM, Dan Carpenter wrote:
> The "ver" variable is type "enum ddr_info_version". Enum types are
> dependent on the compiler but in practical terms here, "ver" is
> basically an unsigned int. It needs to be signed for the -EINVAL
> to be handled correctly.
>
> Fixes: 1d234eeafc56 ("soc: qcom: smem: Expose DDR data from SMEM")
> Signed-off-by: Dan Carpenter <error27@xxxxxxxxx>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxxxx>
Konrad