> -----Original Message-----
> From: Ravulapalli, Naresh Kumar <naresh.kumar.ravulapa...@altera.com>
> Sent: Tuesday, May 6, 2025 10:13 PM
> To: u-boot@lists.denx.de
> Cc: Marek Vasut <ma...@denx.de>; Simon Goldschmidt
> <simon.k.r.goldschm...@gmail.com>; Chee, Tien Fong
> <tien.fong.c...@altera.com>; Meng, Tingting <tingting.m...@altera.com>;
> Tom Rini <tr...@konsulko.com>; Ravulapalli, Naresh Kumar
> <naresh.kumar.ravulapa...@altera.com>
> Subject: [PATCH 3/5] drivers: ddr: altera: arria10: Fix incorrect address for
> mpu1
> 
> SOCFPGA_SDR_FIREWALL_MPU_FPGA_ADDRESS is added in
> SOCFPGA_SDR_FIREWALL_MPU_FPGA_ADDRESS_OFFSET() macro already.
> Remove extra SOCFPGA_SDR_FIREWALL_MPU_FPGA_ADDRESS in mpu1
> address computation.
> 
> Signed-off-by: Naresh Kumar Ravulapalli
> <nareshkumar.ravulapa...@altera.com>
> ---
>  drivers/ddr/altera/sdram_arria10.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/ddr/altera/sdram_arria10.c
> b/drivers/ddr/altera/sdram_arria10.c
> index 4a82af0b711..d8ad94f31cd 100644
> --- a/drivers/ddr/altera/sdram_arria10.c
> +++ b/drivers/ddr/altera/sdram_arria10.c
> @@ -477,7 +477,6 @@ const struct firewall_entry firewall_table[] = {
>       },
>       {
>               "mpu1",
> -             SOCFPGA_SDR_FIREWALL_MPU_FPGA_ADDRESS +
> 
>       SOCFPGA_SDR_FIREWALL_MPU_FPGA_ADDRESS_OFFSET(mpuregio
> n1addr),
> 
>       SOCFPGA_SDR_FIREWALL_MPU_FPGA_ADDRESS_OFFSET(enable),
>               ALT_NOC_FW_DDR_SCR_EN_MPUREG1EN_SET_MSK
> --
> 2.35.3

Reviewed-by: Tien Fong Chee <tien.fong.c...@altera.com>

Best regards,
Tien Fong

Reply via email to