Re: [PATCH 2/3] drm/imagination: Update Rogue heap comments

Alessio Belle <[email protected]>
Newsgroups gmane.linux.kernel,gmane.comp.video.dri.devel
Message-ID <[email protected]>
On Mon, 2026-08-03 at 11:22 +0100, Luigi Santivetti wrote:
> From: Alexandru Dadu <[email protected]>
> 
> Update Rogue heap memory comments to fix typos.
> 
> Signed-off-by: Alexandru Dadu <[email protected]>
> Signed-off-by: Luigi Santivetti <[email protected]>

Reviewed-by: Alessio Belle <[email protected]>

Thanks,
Alessio

> ---
>  drivers/gpu/drm/imagination/pvr_rogue_heap_config.h | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/gpu/drm/imagination/pvr_rogue_heap_config.h b/drivers/gpu/drm/imagination/pvr_rogue_heap_config.h
> index 684766006703..5c33b342dda5 100644
> --- a/drivers/gpu/drm/imagination/pvr_rogue_heap_config.h
> +++ b/drivers/gpu/drm/imagination/pvr_rogue_heap_config.h
> @@ -33,12 +33,12 @@
>  /* 0 MiB to 4 MiB, size of 4 MiB : RESERVED */
>  
>  /* 0x00_0040_0000 - 0x7F_FFC0_0000 **/
> -/* 4 MiB to 512 GiB, size of 512 GiB less 4 MiB : RESERVED **/
> +/* 4 MiB to 512 GiB, size of 512 GiB less 4 MiB : RESERVED */
>  
>  /* 0x80_0000_0000 ************************************************************/
>  
>  /* 0x80_0000_0000 - 0x9F_FFFF_FFFF **/
> -/* 512 GiB to 640 GiB, size of 128 GiB : GENERAL_HEAP **/
> +/* 512 GiB to 640 GiB, size of 128 GiB : GENERAL_HEAP */
>  #define ROGUE_GENERAL_HEAP_BASE 0x8000000000ull
>  #define ROGUE_GENERAL_HEAP_SIZE SZ_128G
>  
> @@ -62,7 +62,7 @@
>  #define ROGUE_PDSCODEDATA_HEAP_SIZE SZ_4G
>  
>  /* 0xDB_0000_0000 - 0xDB_FFFF_FFFF */
> -/* 876 GiB to 880 GiB, size of 256 MiB (reserved 4GiB) : BRN **/
> +/* 876 GiB to 880 GiB, size of 256 MiB (reserved 4GiB) : BRN */
>  /*
>   * The BRN63142 quirk workaround requires Region Header memory to be at the top
>   * of a 16GiB aligned range. This is so when masked with 0x03FFFFFFFF the
> @@ -82,7 +82,7 @@
>  /* 0xE1_0000_0000 - 0xE1_BFFF_FFFF */
>  /* 900 GiB to 903 GiB, size of 3 GiB : RESERVED */
>  
> -/* 0xE1_C000_000 - 0xE1_FFFF_FFFF */
> +/* 0xE1_C000_0000 - 0xE1_FFFF_FFFF */
>  /* 903 GiB to 904 GiB, reserved 1 GiB, : FIRMWARE_HEAP */
>  #define ROGUE_FW_HEAP_BASE 0xE1C0000000ull
>  
> @@ -90,7 +90,7 @@
>  /* 904 GiB to 912 GiB, size of 8 GiB : FREE */
>  
>  /* 0xE4_0000_0000 - 0xE7_FFFF_FFFF */
> -/* 912 GiB to 968 GiB, size of 16 GiB : TRANSFER_FRAG */
> +/* 912 GiB to 928 GiB, size of 16 GiB : TRANSFER_FRAG */
>  #define ROGUE_TRANSFER_FRAG_HEAP_BASE 0xE400000000ull
>  #define ROGUE_TRANSFER_FRAG_HEAP_SIZE SZ_16G
>  
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.