arm: zynq: Move common ps7_init* initialization to arch code
authorMichal Simek <michal.simek@xilinx.com>
Fri, 10 Nov 2017 08:47:28 +0000 (09:47 +0100)
committerMichal Simek <michal.simek@xilinx.com>
Wed, 29 Nov 2017 07:02:39 +0000 (08:02 +0100)
This patch is based on work done in topic board where the first address
word also storing operation which should be done. This is reducing size
of configuration data.
This patch is not breaking an option to copy default ps7_init_gpl* files
from hdf file but it is doing preparation for ps7_init* consolidation.

The patch is also marking ps7_config as weak function.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
arch/arm/mach-zynq/include/mach/ps7_init_gpl.h
arch/arm/mach-zynq/ps7_spl_init.c
board/topic/zynq/Makefile
board/topic/zynq/ps7_init_common.c [deleted file]
board/topic/zynq/ps7_init_gpl.h [deleted file]
board/topic/zynq/zynq-topic-miami/ps7_init_gpl.c
board/topic/zynq/zynq-topic-miamilite/ps7_init_gpl.c
board/topic/zynq/zynq-topic-miamiplus/ps7_init_gpl.c

index 6e30108b23f4463dd0b241cbdd90d07deea848f4..c7c716e7f2c408524f8c49634a3abf40ef3af63d 100644 (file)
@@ -1,5 +1,6 @@
 /*
- * (c) Copyright 2010-2017 Xilinx, Inc. All rights reserved.
+ * (c) Copyright 2010-2014 Xilinx, Inc. All rights reserved.
+ * (c) Copyright 2016 Topic Embedded Products.
  *
  * SPDX-License-Identifier:    GPL-2.0+
  */
@@ -7,8 +8,33 @@
 #ifndef _ASM_ARCH_PS7_INIT_GPL_H
 #define _ASM_ARCH_PS7_INIT_GPL_H
 
+/* Opcode exit is 0 all the time */
+#define OPCODE_EXIT            0U
+#define OPCODE_MASKWRITE       0U
+#define OPCODE_MASKPOLL                1U
+#define OPCODE_MASKDELAY       2U
+#define OPCODE_ADDRESS_MASK    (~3U)
+
+/* Sentinel */
+#define EMIT_EXIT()                    OPCODE_EXIT
+/* Opcode is in lower 2 bits of address, address is always 4-byte aligned */
+#define EMIT_MASKWRITE(addr, mask, val)        OPCODE_MASKWRITE | addr, mask, val
+#define EMIT_MASKPOLL(addr, mask)      OPCODE_MASKPOLL | addr, mask
+#define EMIT_MASKDELAY(addr, mask)     OPCODE_MASKDELAY | addr, mask
+
+/* Returns codes of ps7_init* */
+#define PS7_INIT_SUCCESS               (0)
+#define PS7_INIT_CORRUPT               (1)
+#define PS7_INIT_TIMEOUT               (2)
+#define PS7_POLL_FAILED_DDR_INIT       (3)
+#define PS7_POLL_FAILED_DMA            (4)
+#define PS7_POLL_FAILED_PLL            (5)
+
 /* Called by spl.c */
 int ps7_init(void);
 int ps7_post_config(void);
 
+/* Defined in ps7_init_common.c */
+int ps7_config(unsigned long *ps7_config_init);
+
 #endif /* _ASM_ARCH_PS7_INIT_GPL_H */
index 6adf852578a6777c9ca867271e2c22939519c87b..180099577b04337bf970fc3683febcf444b575c6 100644 (file)
@@ -1,5 +1,6 @@
 /*
  * (c) Copyright 2010-2017 Xilinx, Inc. All rights reserved.
+ * (c) Copyright 2016 Topic Embedded Products.
  *
  * SPDX-License-Identifier:    GPL-2.0+
  */
@@ -25,3 +26,111 @@ __weak int ps7_post_config(void)
         */
        return 0;
 }
+
+/* For delay calculation using global registers*/
+#define SCU_GLOBAL_TIMER_COUNT_L32     0xF8F00200
+#define SCU_GLOBAL_TIMER_COUNT_U32     0xF8F00204
+#define SCU_GLOBAL_TIMER_CONTROL       0xF8F00208
+#define SCU_GLOBAL_TIMER_AUTO_INC      0xF8F00218
+#define APU_FREQ  666666666
+
+#define PS7_MASK_POLL_TIME 100000000
+
+/* IO accessors. No memory barriers desired. */
+static inline void iowrite(unsigned long val, unsigned long addr)
+{
+       __raw_writel(val, addr);
+}
+
+static inline unsigned long ioread(unsigned long addr)
+{
+       return __raw_readl(addr);
+}
+
+/* start timer */
+static void perf_start_clock(void)
+{
+       iowrite((1 << 0) | /* Timer Enable */
+               (1 << 3) | /* Auto-increment */
+               (0 << 8), /* Pre-scale */
+               SCU_GLOBAL_TIMER_CONTROL);
+}
+
+/* Compute mask for given delay in miliseconds*/
+static int get_number_of_cycles_for_delay(unsigned int delay)
+{
+       return (APU_FREQ / (2 * 1000)) * delay;
+}
+
+/* stop timer */
+static void perf_disable_clock(void)
+{
+       iowrite(0, SCU_GLOBAL_TIMER_CONTROL);
+}
+
+/* stop timer and reset timer count regs */
+static void perf_reset_clock(void)
+{
+       perf_disable_clock();
+       iowrite(0, SCU_GLOBAL_TIMER_COUNT_L32);
+       iowrite(0, SCU_GLOBAL_TIMER_COUNT_U32);
+}
+
+static void perf_reset_and_start_timer(void)
+{
+       perf_reset_clock();
+       perf_start_clock();
+}
+
+int __weak ps7_config(unsigned long *ps7_config_init)
+{
+       unsigned long *ptr = ps7_config_init;
+       unsigned long opcode;
+       unsigned long addr;
+       unsigned long val;
+       unsigned long mask;
+       unsigned int numargs;
+       int i;
+       int delay;
+
+       for (;;) {
+               opcode = ptr[0];
+               if (opcode == OPCODE_EXIT)
+                       return PS7_INIT_SUCCESS;
+               addr = (opcode & OPCODE_ADDRESS_MASK);
+
+               switch (opcode & ~OPCODE_ADDRESS_MASK) {
+               case OPCODE_MASKWRITE:
+                       numargs = 3;
+                       mask = ptr[1];
+                       val = ptr[2];
+                       iowrite((ioread(addr) & ~mask) | (val & mask), addr);
+                       break;
+
+               case OPCODE_MASKPOLL:
+                       numargs = 2;
+                       mask = ptr[1];
+                       i = 0;
+                       while (!(ioread(addr) & mask)) {
+                               if (i == PS7_MASK_POLL_TIME)
+                                       return PS7_INIT_TIMEOUT;
+                               i++;
+                       }
+                       break;
+
+               case OPCODE_MASKDELAY:
+                       numargs = 2;
+                       mask = ptr[1];
+                       delay = get_number_of_cycles_for_delay(mask);
+                       perf_reset_and_start_timer();
+                       while (ioread(addr) < delay)
+                               ;
+                       break;
+
+               default:
+                       return PS7_INIT_CORRUPT;
+               }
+
+               ptr += numargs;
+       }
+}
index eaf59cd55c6c4e9e57309b0e89235d83985cf476..789348207535cb4f52e0694693a3e393812095bc 100644 (file)
@@ -7,4 +7,4 @@ obj-y   := board.o
 # Remove quotes
 hw-platform-y :=$(shell echo $(CONFIG_DEFAULT_DEVICE_TREE))
 
-obj-$(CONFIG_SPL_BUILD) += $(hw-platform-y)/ps7_init_gpl.o ps7_init_common.o
+obj-$(CONFIG_SPL_BUILD) += $(hw-platform-y)/ps7_init_gpl.o
diff --git a/board/topic/zynq/ps7_init_common.c b/board/topic/zynq/ps7_init_common.c
deleted file mode 100644 (file)
index b1d45c2..0000000
+++ /dev/null
@@ -1,117 +0,0 @@
-/*
- * (c) Copyright 2010-2014 Xilinx, Inc. All rights reserved.
- * (c) Copyright 2016 Topic Embedded Products.
- *
- * SPDX-License-Identifier:    GPL-2.0+
- */
-
-#include "ps7_init_gpl.h"
-#include <asm/io.h>
-
-/* For delay calculation using global registers*/
-#define SCU_GLOBAL_TIMER_COUNT_L32     0xF8F00200
-#define SCU_GLOBAL_TIMER_COUNT_U32     0xF8F00204
-#define SCU_GLOBAL_TIMER_CONTROL       0xF8F00208
-#define SCU_GLOBAL_TIMER_AUTO_INC      0xF8F00218
-#define APU_FREQ  666666666
-
-#define PS7_MASK_POLL_TIME 100000000
-
-/* IO accessors. No memory barriers desired. */
-static inline void iowrite(unsigned long val, unsigned long addr)
-{
-       __raw_writel(val, addr);
-}
-
-static inline unsigned long ioread(unsigned long addr)
-{
-       return __raw_readl(addr);
-}
-
-/* start timer */
-static void perf_start_clock(void)
-{
-       iowrite((1 << 0) | /* Timer Enable */
-               (1 << 3) | /* Auto-increment */
-               (0 << 8), /* Pre-scale */
-               SCU_GLOBAL_TIMER_CONTROL);
-}
-
-/* Compute mask for given delay in miliseconds*/
-static int get_number_of_cycles_for_delay(unsigned int delay)
-{
-       return (APU_FREQ / (2 * 1000)) * delay;
-}
-
-/* stop timer */
-static void perf_disable_clock(void)
-{
-       iowrite(0, SCU_GLOBAL_TIMER_CONTROL);
-}
-
-/* stop timer and reset timer count regs */
-static void perf_reset_clock(void)
-{
-       perf_disable_clock();
-       iowrite(0, SCU_GLOBAL_TIMER_COUNT_L32);
-       iowrite(0, SCU_GLOBAL_TIMER_COUNT_U32);
-}
-
-static void perf_reset_and_start_timer(void)
-{
-       perf_reset_clock();
-       perf_start_clock();
-}
-
-int ps7_config(unsigned long *ps7_config_init)
-{
-       unsigned long *ptr = ps7_config_init;
-       unsigned long opcode;
-       unsigned long addr;
-       unsigned long val;
-       unsigned long mask;
-       unsigned int numargs;
-       int i;
-       int delay;
-
-       for (;;) {
-               opcode = ptr[0];
-               if (opcode == OPCODE_EXIT)
-                       return PS7_INIT_SUCCESS;
-               addr = (opcode & OPCODE_ADDRESS_MASK);
-
-               switch (opcode & ~OPCODE_ADDRESS_MASK) {
-               case OPCODE_MASKWRITE:
-                       numargs = 3;
-                       mask = ptr[1];
-                       val = ptr[2];
-                       iowrite((ioread(addr) & ~mask) | (val & mask), addr);
-                       break;
-
-               case OPCODE_MASKPOLL:
-                       numargs = 2;
-                       mask = ptr[1];
-                       i = 0;
-                       while (!(ioread(addr) & mask)) {
-                               if (i == PS7_MASK_POLL_TIME)
-                                       return PS7_INIT_TIMEOUT;
-                               i++;
-                       }
-                       break;
-
-               case OPCODE_MASKDELAY:
-                       numargs = 2;
-                       mask = ptr[1];
-                       delay = get_number_of_cycles_for_delay(mask);
-                       perf_reset_and_start_timer();
-                       while (ioread(addr) < delay)
-                               ;
-                       break;
-
-               default:
-                       return PS7_INIT_CORRUPT;
-               }
-
-               ptr += numargs;
-       }
-}
diff --git a/board/topic/zynq/ps7_init_gpl.h b/board/topic/zynq/ps7_init_gpl.h
deleted file mode 100644 (file)
index ef719ac..0000000
+++ /dev/null
@@ -1,34 +0,0 @@
-/*
- * (c) Copyright 2010-2014 Xilinx, Inc. All rights reserved.
- * (c) Copyright 2016 Topic Embedded Products.
- *
- * SPDX-License-Identifier:    GPL-2.0+
- */
-
-#define OPCODE_EXIT       0U
-#define OPCODE_MASKWRITE  0U
-#define OPCODE_MASKPOLL   1U
-#define OPCODE_MASKDELAY  2U
-#define OPCODE_ADDRESS_MASK (~3U)
-
-/* Sentinel */
-#define EMIT_EXIT()                     OPCODE_EXIT
-/* Opcode is in lower 2 bits of address, address is always 4-byte aligned */
-#define EMIT_MASKWRITE(addr, mask, val) OPCODE_MASKWRITE | addr, mask, val
-#define EMIT_MASKPOLL(addr, mask)       OPCODE_MASKPOLL | addr, mask
-#define EMIT_MASKDELAY(addr, mask)      OPCODE_MASKDELAY | addr, mask
-
-/* Returns codes of ps7_init* */
-#define PS7_INIT_SUCCESS   (0)
-#define PS7_INIT_CORRUPT   (1)
-#define PS7_INIT_TIMEOUT   (2)
-#define PS7_POLL_FAILED_DDR_INIT (3)
-#define PS7_POLL_FAILED_DMA      (4)
-#define PS7_POLL_FAILED_PLL      (5)
-
-/* Called by spl.c */
-int ps7_init(void);
-int ps7_post_config(void);
-
-/* Defined in ps7_init_common.c */
-int ps7_config(unsigned long *ps7_config_init);
index b195d7a25bf4c67b6cba02f2f3a8436372bd0fea..ceed04383f5f60323279b36e1d6652adfaea9407 100644 (file)
@@ -5,7 +5,7 @@
  * SPDX-License-Identifier:    GPL-2.0+
  */
 
-#include "../ps7_init_gpl.h"
+#include <asm/arch/ps7_init_gpl.h>
 
 static unsigned long ps7_pll_init_data_3_0[] = {
        EMIT_MASKWRITE(0XF8000008, 0x0000FFFFU, 0x0000DF0DU),
index ec0cc7d19d0c14332e078c8f4b723a2a88aaf5fd..1205d19d9a35b96226d240e27b9e440e75abed33 100644 (file)
@@ -5,7 +5,7 @@
  * SPDX-License-Identifier:    GPL-2.0+
  */
 
-#include "../ps7_init_gpl.h"
+#include <asm/arch/ps7_init_gpl.h>
 
 static unsigned long ps7_pll_init_data_3_0[] = {
        EMIT_MASKWRITE(0xF8000008, 0x0000FFFFU, 0x0000DF0DU),
index 5a923366eb781743ca0617d11486f1e05b338de4..f42632b7fab58fa8b35ce56af81eb2d3d148f36e 100644 (file)
@@ -5,7 +5,7 @@
  * SPDX-License-Identifier:    GPL-2.0+
  */
 
-#include "../ps7_init_gpl.h"
+#include <asm/arch/ps7_init_gpl.h>
 
 static unsigned long ps7_pll_init_data_3_0[] = {
        EMIT_MASKWRITE(0XF8000008, 0x0000FFFFU, 0x0000DF0DU),