Am 30.05.2019 um 11:03 schrieb Ley Foon Tan:
Add base address for Intel Agilex SoC.

Reuse base_addr_s10.h for Agilex, only one base address is
different from S10.

Signed-off-by: Ley Foon Tan <ley.foon....@intel.com>
---

Wait, this is v2, right? What hss changed since v1? I notice v2 has 15 patches while v1 had 14.

Have you ever considered using patman and its helper tags? It would greatly reduce the effort for reviewers to keep things consistent and including a list of changes in each patch.

I mean, when reading v2, I want to rely on you saying "patches 1, 3, and 5 of 14 have changed, the rest have not" to speed up my reviewing. Patman really helps you with that, just try it! And if you don't want to, well, look at how other developers send their multi-version patches...

Regards,
Simon

  arch/arm/mach-socfpga/include/mach/base_addr_s10.h | 4 ++++
  1 file changed, 4 insertions(+)

diff --git a/arch/arm/mach-socfpga/include/mach/base_addr_s10.h 
b/arch/arm/mach-socfpga/include/mach/base_addr_s10.h
index 1f549d7e70..d3eca65e97 100644
--- a/arch/arm/mach-socfpga/include/mach/base_addr_s10.h
+++ b/arch/arm/mach-socfpga/include/mach/base_addr_s10.h
@@ -10,7 +10,11 @@
  #define SOCFPGA_SDR_SCHEDULER_ADDRESS         0xf8000400
  #define SOCFPGA_HMC_MMR_IO48_ADDRESS          0xf8010000
  #define SOCFPGA_SDR_ADDRESS                   0xf8011000
+#ifdef CONFIG_TARGET_SOCFPGA_AGILEX
+#define SOCFPGA_FW_MPU_DDR_SCR_ADDRESS         0xf8020200
+#else
  #define SOCFPGA_FW_MPU_DDR_SCR_ADDRESS                0xf8020100
+#endif
  #define SOCFPGA_SMMU_ADDRESS                  0xfa000000
  #define SOCFPGA_MAILBOX_ADDRESS                       0xffa30000
  #define SOCFPGA_UART0_ADDRESS                 0xffc02000


_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to