Re: [PATCH v1] soc: qcom: geni-se: Increase MAX_GENI_CFG_RAMn_CNT to 1020

[Konrad Dybcio]

On 5/22/26 9:21 AM, Viken Dadhaniya wrote:
> The current MAX_GENI_CFG_RAMn_CNT value of 455 is insufficient for
> certain Qualcomm SoCs whose GENI firmware requires up to 1020 RAM
> entries, causing incorrect bounds checking and preventing firmware
> loading on those targets.
> 
> Increase MAX_GENI_CFG_RAMn_CNT to 1020 to match the hardware-supported
> limit and ensure correct handling of configuration RAM regions across
> GENI-based SE blocks.
> 
> Signed-off-by: Viken Dadhaniya <viken.dadhaniya@xxxxxxxxxxxxxxxx>
> ---
>  drivers/soc/qcom/qcom-geni-se.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/soc/qcom/qcom-geni-se.c b/drivers/soc/qcom/qcom-geni-se.c
> index cd1779b6a91a..d0810960f3da 100644
> --- a/drivers/soc/qcom/qcom-geni-se.c
> +++ b/drivers/soc/qcom/qcom-geni-se.c
> @@ -152,7 +152,7 @@ struct se_fw_hdr {
>  /*Magic numbers*/
>  #define SE_MAGIC_NUM			0x57464553
>  
> -#define MAX_GENI_CFG_RAMn_CNT		455
> +#define MAX_GENI_CFG_RAMn_CNT		1020

Is there any SoC where this could overflow the register space
(you say "certain Qualcomm SoCs")?

Konrad