On 11/6/2026 1:59 AM, Rob Clark wrote:
> Don't let the length bleed into b31.
>
> Tested-by: Alexander Koskovich <[email protected]> # Milos (Nothing Phone 3a)
> Reviewed-by: Konrad Dybcio <[email protected]>
> Signed-off-by: Rob Clark <[email protected]>
Reviewed-by: Akhil P Oommen <[email protected]>
-Akhil
> ---
> drivers/gpu/drm/msm/adreno/a6xx_gpu.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gpu.h
> b/drivers/gpu/drm/msm/adreno/a6xx_gpu.h
> index 4a3898e58aee..cfe7a3ed44c4 100644
> --- a/drivers/gpu/drm/msm/adreno/a6xx_gpu.h
> +++ b/drivers/gpu/drm/msm/adreno/a6xx_gpu.h
> @@ -244,7 +244,7 @@ struct a7xx_cp_smmu_info {
> */
> #define A6XX_PROTECT_NORDWR(_reg, _len) \
> ((1 << 31) | \
> - (((_len) & 0x3FFF) << 18) | ((_reg) & 0x3FFFF))
> + (((_len) & 0x1FFF) << 18) | ((_reg) & 0x3FFFF))
>
> /*
> * Same as above, but allow reads over the range. For areas of mixed use
> (such
> @@ -252,7 +252,7 @@ struct a7xx_cp_smmu_info {
> * single register
> */
> #define A6XX_PROTECT_RDONLY(_reg, _len) \
> - ((((_len) & 0x3FFF) << 18) | ((_reg) & 0x3FFFF))
> + ((((_len) & 0x1FFF) << 18) | ((_reg) & 0x3FFFF))
>
> extern const struct adreno_gpu_funcs a6xx_gpu_funcs;
> extern const struct adreno_gpu_funcs a6xx_gmuwrapper_funcs;
