[PATCH v2 36/37] baseband/acc100: make HARQ layout memory 4GB
Maxime Coquelin
maxime.coquelin at redhat.com
Thu Sep 15 13:33:56 CEST 2022
On 8/20/22 04:31, Hernan Vargas wrote:
> HARQ layout memory should be 4GB instead of 2GB.
>
> Fixes: 4cf90079797 ("baseband/acc100: add HW register definitions")
> Cc: stable at dpdk.org
>
> Signed-off-by: Hernan Vargas <hernan.vargas at intel.com>
> ---
> drivers/baseband/acc100/acc100_pmd.h | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/baseband/acc100/acc100_pmd.h b/drivers/baseband/acc100/acc100_pmd.h
> index 27801767b7..f9ccb1ea8e 100644
> --- a/drivers/baseband/acc100/acc100_pmd.h
> +++ b/drivers/baseband/acc100/acc100_pmd.h
> @@ -61,8 +61,10 @@
> #define ACC100_SIZE_64MBYTE (64*1024*1024)
> /* Number of elements in an Info Ring */
> #define ACC100_INFO_RING_NUM_ENTRIES 1024
> -/* Number of elements in HARQ layout memory */
> -#define ACC100_HARQ_LAYOUT (64*1024*1024)
> +/* Number of elements in HARQ layout memory
> + * 128M x 32kB = 4GB addressable memory
> + */
> +#define ACC100_HARQ_LAYOUT (128*1024*1024)
> /* Assume offset for HARQ in memory */
> #define ACC100_HARQ_OFFSET (32*1024)
> #define ACC100_HARQ_OFFSET_SHIFT 15
Please move it to the top of the series for next revision.
Reviewed-by: Maxime Coquelin <maxime.coquelin at redhat.com>
Thanks,
Maxime
More information about the dev
mailing list