On 05/31/2017 10:49 AM, Alexey Brodkin wrote:
This initial port add support of ARC HS Development Kit board with some
basic features such as SMP and serial port, USB, SD/MMC and Ethernet.
Note as opposed to other ARC boards we link Linux kernel to
0x9000_0000 intentionally because cores 1 and 3 configured with DCCM
situated at our more usual link base 0x8000_0000.

Also there's a prerequisite for this change required to get
.dts compiled, see
http://lists.infradead.org/pipermail/linux-snps-arc/2017-May/002462.html
Per Rob's comment yesterday - this is not needed ?


Signed-off-by: Eugeniy Paltsev <palt...@synopsys.com>
Signed-off-by: Alexey Brodkin <abrod...@synopsys.com>
---

Changes v1 -> v2:
  * Update copyright year from 2016 to more up to date 2017
  * Merge early UART clock with AXS10x as in both cases that's 33.3 MHz
  * Bump memory to 1Gb, we don't use more for now because it requires
    trickier IOC setup and usage
  * Update early platform init code:
     - Added missing fixup_pae_regs() to per-cpu init
     - Mark most of functions as "static __init"
     - Use writel_relaxed() for setting CREG_PAE, CREG_PAE_UPDATE is still
       written with stronger writel() since we don't want reordering to happen,
       otherwise value written to CREG_PAE won't be applied

  Documentation/devicetree/bindings/arc/hsdk.txt |   7 ++
  arch/arc/Kconfig                               |   1 +
  arch/arc/Makefile                              |   1 +
  arch/arc/boot/dts/hsdk.dts                     | 151 +++++++++++++++++++++++++
  arch/arc/configs/hsdk_defconfig                |  72 ++++++++++++
  arch/arc/kernel/devtree.c                      |   5 +-
  arch/arc/plat-hsdk/Kconfig                     |  12 ++
  arch/arc/plat-hsdk/Makefile                    |   9 ++
  arch/arc/plat-hsdk/platform.c                  |  74 ++++++++++++
  9 files changed, 330 insertions(+), 2 deletions(-)
  create mode 100644 Documentation/devicetree/bindings/arc/hsdk.txt
  create mode 100644 arch/arc/boot/dts/hsdk.dts
  create mode 100644 arch/arc/configs/hsdk_defconfig
  create mode 100644 arch/arc/plat-hsdk/Kconfig
  create mode 100644 arch/arc/plat-hsdk/Makefile
  create mode 100644 arch/arc/plat-hsdk/platform.c

diff --git a/Documentation/devicetree/bindings/arc/hsdk.txt 
b/Documentation/devicetree/bindings/arc/hsdk.txt
new file mode 100644
index 000000000000..be50654bbf61
--- /dev/null
+++ b/Documentation/devicetree/bindings/arc/hsdk.txt
@@ -0,0 +1,7 @@
+Synopsys DesignWare ARC HS Development Kit Device Tree Bindings
+---------------------------------------------------------------------------
+
+ARC HSDK Board with quad-core ARC HS38x4 in silicon.
+
+Required root node properties:
+    - compatible = "snps,hsdk";
Does this require an approval from RobH or DT folks - care to CC them in next 
version ?
+ * Default configuration of PAE regs doesn't work for us causeing
spello !

+ * problems with DMA to/from peripherals even if PAE40 is not used.
+ */
+static void __init fixup_pae_regs(void)
+{
+#define ARC_PERIPHERAL_BASE    0xf0000000
+#define        CREG_BASE               (ARC_PERIPHERAL_BASE + 0x1000)
+#define        CREG_PAE                (CREG_BASE + 0x180)
+#define        CREG_PAE_UPDATE         (CREG_BASE + 0x194)
+
+       /* Default is 1, which means "PAE offset = 4GByte" */
+       writel_relaxed(0, (void __iomem *) CREG_PAE);
+
+       /* Really apply settings made above */
+       writel(1, (void __iomem *) CREG_PAE_UPDATE);
+}
+
+static void __init hsdk_early_init(void)
+{
+       relocate_iccm();
+       fixup_pae_regs();
+}
+
+static void __init hsdk_init_per_cpu(unsigned int cpu)
+{
+       relocate_iccm();
+       fixup_pae_regs();
+}
Instead of duplicating can u just call hsdk_early_init() here.
oh wait, init_per_cpu is called on each cpu, including the master, so due to init_early hook, u end up calling above twice on master !

+
+/*----------------------- Machine Descriptions ------------------------------
+ *
+ * Machine description is simply a set of platform/board specific callbacks
+ * This is not directly related to DeviceTree based dynamic device creation,
+ * however as part of early device tree scan, we also select the right
+ * callback set, by matching the DT compatible name.
+ */
No Cargo-culting please ! Take this comment out.

+
+static const char *hsdk_compat[] __initconst = {
+       "snps,hsdk",
+       NULL,
+};
+
+MACHINE_START(SIMULATION, "hsdk")
+       .dt_compat      = hsdk_compat,
+       .init_early     = hsdk_early_init,
+       .init_per_cpu   = hsdk_init_per_cpu,
+MACHINE_END


_______________________________________________
linux-snps-arc mailing list
linux-snps-arc@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-snps-arc

Reply via email to