[U-Boot] [RFC PATCH 0/7] FPGA changes

[U-Boot] [RFC PATCH 0/7] FPGA changes

[Chris Packham]

Patches 1-3,5-6 are moving existing options to Kconfig. I haven't
touched all the boards that use these options so they are still present
in the whitelist. Hopefully someone more familiar with the zynq boards
can build on top of these.

Patches 4 and 7 are the functional changes I actually want to introduce.
We have some boards where the FPGA images require the reverse bitstream
mode. I think this may be some setting in the Xilinx tools but it seems
to be permanently set in the images coming out of our hardware team.


Chris Packham (7):
  fpga: CONFIG_SYS_FPGA_PROG_FEEDBACK -> CONFIG_FPGA_PROG_FEEDBACK
  fpga: add Kconfig support for Xilinx Spartan devices
  Move CONFIG_FPGA_SPARTAN options to defconfig
  fpga/spartan3: make output quieter
  fpga: add Kconfig for CONFIG_FPGA_PROG_FEEDBACK
  Move FPGA_PROG_FEEDBACK to defconfig
  fpga/spartan3: add reverse bitstream mode

 README                           |  2 +-
 board/astro/mcf5373l/fpga.c      |  4 ++--
 configs/PMC440_defconfig         |  3 +++
 configs/apf27_defconfig          |  3 +++
 configs/astro_mcf5373l_defconfig |  3 +++
 configs/mt_ventoux_defconfig     |  3 +++
 configs/x600_defconfig           |  2 ++
 drivers/fpga/ACEX1K.c            |  8 ++++----
 drivers/fpga/Kconfig             | 24 ++++++++++++++++++++++++
 drivers/fpga/cyclon2.c           | 10 +++++-----
 drivers/fpga/spartan2.c          | 22 +++++++++++-----------
 drivers/fpga/spartan3.c          | 32 ++++++++++++++++++++++----------
 drivers/fpga/virtex2.c           | 18 +++++++++---------
 include/configs/M54455EVB.h      |  2 +-
 include/configs/PMC440.h         |  4 ----
 include/configs/apf27.h          |  6 ------
 include/configs/astro_mcf5373l.h |  4 ----
 include/configs/mt_ventoux.h     |  5 -----
 include/configs/x600.h           |  3 ---
 scripts/config_whitelist.txt     |  2 +-
 20 files changed, 94 insertions(+), 66 deletions(-)

-- 
2.10.2

[U-Boot] [RFC PATCH 1/7] fpga: CONFIG_SYS_FPGA_PROG_FEEDBACK -> CONFIG_FPGA_PROG_FEEDBACK

[Chris Packham]

Prepare for move to Kconfig by removing "SYS" from the existing macros.

Signed-off-by: Chris Packham <judge.packham@gmail.com>
---

 README                           |  2 +-
 board/astro/mcf5373l/fpga.c      |  4 ++--
 drivers/fpga/ACEX1K.c            |  8 ++++----
 drivers/fpga/cyclon2.c           | 10 +++++-----
 drivers/fpga/spartan2.c          | 22 +++++++++++-----------
 drivers/fpga/spartan3.c          | 20 ++++++++++----------
 drivers/fpga/virtex2.c           | 18 +++++++++---------
 include/configs/M54455EVB.h      |  2 +-
 include/configs/apf27.h          |  2 +-
 include/configs/astro_mcf5373l.h |  2 +-
 include/configs/mt_ventoux.h     |  4 ++--
 scripts/config_whitelist.txt     |  2 +-
 12 files changed, 48 insertions(+), 48 deletions(-)

diff --git a/README b/README
index 47bc215fe9c5..fdc194ab9792 100644
--- a/README
+++ b/README
@@ -2536,7 +2536,7 @@ The following options need to be configured:
 		Enable support for fpga loadbp command - load partial bitstream
 		(Xilinx only)
 
-		CONFIG_SYS_FPGA_PROG_FEEDBACK
+		CONFIG_FPGA_PROG_FEEDBACK
 
 		Enable printing of hash marks during FPGA configuration.
 
diff --git a/board/astro/mcf5373l/fpga.c b/board/astro/mcf5373l/fpga.c
index 53e00728c3d8..1b53fd2521e3 100644
--- a/board/astro/mcf5373l/fpga.c
+++ b/board/astro/mcf5373l/fpga.c
@@ -128,7 +128,7 @@ int altera_write_fn(const void *buf, size_t len, int flush, int cookie)
 #if defined(CONFIG_HW_WATCHDOG) || defined(CONFIG_WATCHDOG)
 			WATCHDOG_RESET();
 #endif
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 			putc('.');	/* let them know we are alive */
 #endif
 #ifdef CONFIG_SYS_FPGA_CHECK_CTRLC
@@ -347,7 +347,7 @@ int xilinx_fastwr_config_fn(void *buf, size_t len, int flush, int cookie)
 #if defined(CONFIG_HW_WATCHDOG) || defined(CONFIG_WATCHDOG)
 			WATCHDOG_RESET();
 #endif
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 			putc('.');	/* let them know we are alive */
 #endif
 #ifdef CONFIG_SYS_FPGA_CHECK_CTRLC
diff --git a/drivers/fpga/ACEX1K.c b/drivers/fpga/ACEX1K.c
index 1627f0e6ffde..1fb60e5983c3 100644
--- a/drivers/fpga/ACEX1K.c
+++ b/drivers/fpga/ACEX1K.c
@@ -112,7 +112,7 @@ static int ACEX1K_ps_load(Altera_desc *desc, const void *buf, size_t bsize)
 				"done:\t0x%p\n\n",
 				__FUNCTION__, &fn, fn, fn->config, fn->status,
 				fn->clk, fn->data, fn->done);
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		printf ("Loading FPGA Device %d...", cookie);
 #endif
 
@@ -188,7 +188,7 @@ static int ACEX1K_ps_load(Altera_desc *desc, const void *buf, size_t bsize)
 				i --;
 			} while (i > 0);
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 			if (bytecount % (bsize / 40) == 0)
 				putc ('.');		/* let them know we are alive */
 #endif
@@ -196,7 +196,7 @@ static int ACEX1K_ps_load(Altera_desc *desc, const void *buf, size_t bsize)
 
 		CONFIG_FPGA_DELAY ();
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		putc (' ');			/* terminate the dotted line */
 #endif
 
@@ -223,7 +223,7 @@ static int ACEX1K_ps_load(Altera_desc *desc, const void *buf, size_t bsize)
 
 	ret_val = FPGA_SUCCESS;
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		if (ret_val == FPGA_SUCCESS) {
 			puts ("Done.\n");
 		}
diff --git a/drivers/fpga/cyclon2.c b/drivers/fpga/cyclon2.c
index 8ab7679b48f7..2be1c1a40a23 100644
--- a/drivers/fpga/cyclon2.c
+++ b/drivers/fpga/cyclon2.c
@@ -115,7 +115,7 @@ static int CYC2_ps_load(Altera_desc *desc, const void *buf, size_t bsize)
 				"done:\t0x%p\n\n",
 				__FUNCTION__, &fn, fn, fn->config, fn->status,
 				fn->write, fn->done);
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		printf ("Loading FPGA Device %d...", cookie);
 #endif
 
@@ -153,13 +153,13 @@ static int CYC2_ps_load(Altera_desc *desc, const void *buf, size_t bsize)
 			(*fn->abort) (cookie);
 			return FPGA_FAIL;
 		}
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		puts(" OK? ...");
 #endif
 
 		CONFIG_FPGA_DELAY ();
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		putc (' ');			/* terminate the dotted line */
 #endif
 
@@ -172,13 +172,13 @@ static int CYC2_ps_load(Altera_desc *desc, const void *buf, size_t bsize)
 		(*fn->abort) (cookie);
 		return (FPGA_FAIL);
 	}
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 	puts(" OK\n");
 #endif
 
 	ret_val = FPGA_SUCCESS;
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 	if (ret_val == FPGA_SUCCESS) {
 		puts ("Done.\n");
 	}
diff --git a/drivers/fpga/spartan2.c b/drivers/fpga/spartan2.c
index 859fb3c7787a..45e8534cdcb5 100644
--- a/drivers/fpga/spartan2.c
+++ b/drivers/fpga/spartan2.c
@@ -16,7 +16,7 @@
 #endif
 
 #undef CONFIG_SYS_FPGA_CHECK_BUSY
-#undef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#undef CONFIG_FPGA_PROG_FEEDBACK
 
 /* Note: The assumption is that we cannot possibly run fast enough to
  * overrun the device (the Slave Parallel mode can free run at 50MHz).
@@ -135,7 +135,7 @@ static int spartan2_sp_load(xilinx_desc *desc, const void *buf, size_t bsize)
 		 * Continuous Data Loading in Slave Parallel Mode for
 		 * the Spartan-II Family.
 		 */
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		printf ("Loading FPGA Device %d...\n", cookie);
 #endif
 		/*
@@ -197,7 +197,7 @@ static int spartan2_sp_load(xilinx_desc *desc, const void *buf, size_t bsize)
 			}
 #endif
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 			if (bytecount % (bsize / 40) == 0)
 				putc ('.');		/* let them know we are alive */
 #endif
@@ -207,7 +207,7 @@ static int spartan2_sp_load(xilinx_desc *desc, const void *buf, size_t bsize)
 		(*fn->cs) (false, true, cookie);	/* Deassert the chip select */
 		(*fn->wr) (false, true, cookie);	/* Deassert the write pin */
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		putc ('\n');			/* terminate the dotted line */
 #endif
 
@@ -235,7 +235,7 @@ static int spartan2_sp_load(xilinx_desc *desc, const void *buf, size_t bsize)
 		if (*fn->post)
 			(*fn->post) (cookie);
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		if (ret_val == FPGA_SUCCESS)
 			puts ("Done.\n");
 		else
@@ -271,7 +271,7 @@ static int spartan2_sp_dump(xilinx_desc *desc, const void *buf, size_t bsize)
 			(*fn->clk) (false, true, cookie);	/* Deassert the clock pin */
 			(*fn->clk) (true, true, cookie);	/* Assert the clock pin */
 			(*fn->rdata) (&(data[bytecount++]), cookie);	/* read the data */
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 			if (bytecount % (bsize / 40) == 0)
 				putc ('.');		/* let them know we are alive */
 #endif
@@ -281,7 +281,7 @@ static int spartan2_sp_dump(xilinx_desc *desc, const void *buf, size_t bsize)
 		(*fn->clk) (false, true, cookie);	/* Deassert the clock pin */
 		(*fn->clk) (true, true, cookie);	/* Assert the clock pin */
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		putc ('\n');			/* terminate the dotted line */
 #endif
 		puts ("Done.\n");
@@ -323,7 +323,7 @@ static int spartan2_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
 				"done:\t0x%p\n\n",
 				__FUNCTION__, &fn, fn, fn->pgm, fn->init,
 				fn->clk, fn->wr, fn->done);
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		printf ("Loading FPGA Device %d...\n", cookie);
 #endif
 
@@ -386,7 +386,7 @@ static int spartan2_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
 				i --;
 			} while (i > 0);
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 			if (bytecount % (bsize / 40) == 0)
 				putc ('.');		/* let them know we are alive */
 #endif
@@ -394,7 +394,7 @@ static int spartan2_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
 
 		CONFIG_FPGA_DELAY ();
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		putc ('\n');			/* terminate the dotted line */
 #endif
 
@@ -426,7 +426,7 @@ static int spartan2_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
 		if (*fn->post)
 			(*fn->post) (cookie);
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		if (ret_val == FPGA_SUCCESS)
 			puts ("Done.\n");
 		else
diff --git a/drivers/fpga/spartan3.c b/drivers/fpga/spartan3.c
index b0213e69992b..759ad95e2afc 100644
--- a/drivers/fpga/spartan3.c
+++ b/drivers/fpga/spartan3.c
@@ -139,7 +139,7 @@ static int spartan3_sp_load(xilinx_desc *desc, const void *buf, size_t bsize)
 		 * Continuous Data Loading in Slave Parallel Mode for
 		 * the Spartan-II Family.
 		 */
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		printf ("Loading FPGA Device %d...\n", cookie);
 #endif
 		/*
@@ -201,7 +201,7 @@ static int spartan3_sp_load(xilinx_desc *desc, const void *buf, size_t bsize)
 			}
 #endif
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 			if (bytecount % (bsize / 40) == 0)
 				putc ('.');		/* let them know we are alive */
 #endif
@@ -211,7 +211,7 @@ static int spartan3_sp_load(xilinx_desc *desc, const void *buf, size_t bsize)
 		(*fn->cs) (false, true, cookie);	/* Deassert the chip select */
 		(*fn->wr) (false, true, cookie);	/* Deassert the write pin */
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		putc ('\n');			/* terminate the dotted line */
 #endif
 
@@ -241,7 +241,7 @@ static int spartan3_sp_load(xilinx_desc *desc, const void *buf, size_t bsize)
 		if (*fn->post)
 			(*fn->post) (cookie);
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		if (ret_val == FPGA_SUCCESS)
 			puts ("Done.\n");
 		else
@@ -277,7 +277,7 @@ static int spartan3_sp_dump(xilinx_desc *desc, const void *buf, size_t bsize)
 			(*fn->clk) (false, true, cookie);	/* Deassert the clock pin */
 			(*fn->clk) (true, true, cookie);	/* Assert the clock pin */
 			(*fn->rdata) (&(data[bytecount++]), cookie);	/* read the data */
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 			if (bytecount % (bsize / 40) == 0)
 				putc ('.');		/* let them know we are alive */
 #endif
@@ -287,7 +287,7 @@ static int spartan3_sp_dump(xilinx_desc *desc, const void *buf, size_t bsize)
 		(*fn->clk) (false, true, cookie);	/* Deassert the clock pin */
 		(*fn->clk) (true, true, cookie);	/* Assert the clock pin */
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		putc ('\n');			/* terminate the dotted line */
 #endif
 		puts ("Done.\n");
@@ -329,7 +329,7 @@ static int spartan3_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
 				"done:\t0x%p\n\n",
 				__FUNCTION__, &fn, fn, fn->pgm, fn->init,
 				fn->clk, fn->wr, fn->done);
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		printf ("Loading FPGA Device %d...\n", cookie);
 #endif
 
@@ -401,7 +401,7 @@ static int spartan3_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
 					i --;
 				} while (i > 0);
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 				if (bytecount % (bsize / 40) == 0)
 					putc ('.');		/* let them know we are alive */
 #endif
@@ -410,7 +410,7 @@ static int spartan3_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
 
 		CONFIG_FPGA_DELAY ();
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		putc ('\n');			/* terminate the dotted line */
 #endif
 
@@ -444,7 +444,7 @@ static int spartan3_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
 		if (*fn->post)
 			(*fn->post) (cookie);
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		if (ret_val == FPGA_SUCCESS)
 			puts ("Done.\n");
 		else
diff --git a/drivers/fpga/virtex2.c b/drivers/fpga/virtex2.c
index f7cf02ab5b30..cf7723c46f64 100644
--- a/drivers/fpga/virtex2.c
+++ b/drivers/fpga/virtex2.c
@@ -39,8 +39,8 @@
 #define CONFIG_FPGA_DELAY()
 #endif
 
-#ifndef CONFIG_SYS_FPGA_PROG_FEEDBACK
-#define CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifndef CONFIG_FPGA_PROG_FEEDBACK
+#define CONFIG_FPGA_PROG_FEEDBACK
 #endif
 
 /*
@@ -190,7 +190,7 @@ static int virtex2_ssm_load(xilinx_desc *desc, const void *buf, size_t bsize)
 				fn->clk, fn->cs, fn->wr, fn->rdata, fn->wdata,
 				fn->busy, fn->abort, fn->post);
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		printf ("Initializing FPGA Device %d...\n", cookie);
 #endif
 		/*
@@ -294,7 +294,7 @@ static int virtex2_ssm_load(xilinx_desc *desc, const void *buf, size_t bsize)
 			}
 #endif
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 			if (bytecount % (bsize / 40) == 0)
 				putc ('.');
 #endif
@@ -307,7 +307,7 @@ static int virtex2_ssm_load(xilinx_desc *desc, const void *buf, size_t bsize)
 		(*fn->cs) (false, true, cookie);
 		(*fn->wr) (false, true, cookie);
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		putc ('\n');
 #endif
 
@@ -329,7 +329,7 @@ static int virtex2_ssm_load(xilinx_desc *desc, const void *buf, size_t bsize)
 		}
 
 		if (ret_val == FPGA_SUCCESS) {
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 			printf ("Initialization of FPGA device %d complete\n", cookie);
 #endif
 			/*
@@ -339,7 +339,7 @@ static int virtex2_ssm_load(xilinx_desc *desc, const void *buf, size_t bsize)
 				(*fn->post) (cookie);
 			}
 		} else {
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 			printf ("** Initialization of FPGA device %d FAILED\n",
 					cookie);
 #endif
@@ -382,7 +382,7 @@ static int virtex2_ssm_dump(xilinx_desc *desc, const void *buf, size_t bsize)
 			(*fn->clk) (false, true, cookie);
 			(*fn->clk) (true, true, cookie);
 			(*fn->rdata) (&(data[bytecount++]), cookie);
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 			if (bytecount % (bsize / 40) == 0)
 				putc ('.');
 #endif
@@ -395,7 +395,7 @@ static int virtex2_ssm_dump(xilinx_desc *desc, const void *buf, size_t bsize)
 		(*fn->clk) (false, true, cookie);
 		(*fn->clk) (true, true, cookie);
 
-#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		putc ('\n');
 #endif
 		puts ("Done.\n");
diff --git a/include/configs/M54455EVB.h b/include/configs/M54455EVB.h
index abec3737a498..d80521664611 100644
--- a/include/configs/M54455EVB.h
+++ b/include/configs/M54455EVB.h
@@ -193,7 +193,7 @@
 /* experiment
 #define CONFIG_FPGA
 #define CONFIG_FPGA_COUNT	1
-#define CONFIG_SYS_FPGA_PROG_FEEDBACK
+#define CONFIG_FPGA_PROG_FEEDBACK
 #define CONFIG_SYS_FPGA_CHECK_CTRLC
 */
 
diff --git a/include/configs/apf27.h b/include/configs/apf27.h
index b903ba53d1dc..a97ba9f39c50 100644
--- a/include/configs/apf27.h
+++ b/include/configs/apf27.h
@@ -267,7 +267,7 @@
 #define CONFIG_FPGA_XILINX
 #define CONFIG_FPGA_SPARTAN3
 #define CONFIG_SYS_FPGA_WAIT		250 /* 250 ms */
-#define CONFIG_SYS_FPGA_PROG_FEEDBACK
+#define CONFIG_FPGA_PROG_FEEDBACK
 #define CONFIG_SYS_FPGA_CHECK_CTRLC
 #define CONFIG_SYS_FPGA_CHECK_ERROR
 
diff --git a/include/configs/astro_mcf5373l.h b/include/configs/astro_mcf5373l.h
index e8dca0b2ccdc..0af356bcb49d 100644
--- a/include/configs/astro_mcf5373l.h
+++ b/include/configs/astro_mcf5373l.h
@@ -209,7 +209,7 @@
 #define	CONFIG_FPGA_SPARTAN3
 #define CONFIG_FPGA_ALTERA
 #define CONFIG_FPGA_CYCLON2
-#define CONFIG_SYS_FPGA_PROG_FEEDBACK
+#define CONFIG_FPGA_PROG_FEEDBACK
 #define CONFIG_SYS_FPGA_WAIT		1000
 
 /* End of user parameters to be customized */
diff --git a/include/configs/mt_ventoux.h b/include/configs/mt_ventoux.h
index 2420612800fb..1353b11b1250 100644
--- a/include/configs/mt_ventoux.h
+++ b/include/configs/mt_ventoux.h
@@ -45,11 +45,11 @@
 #define CONFIG_FPGA
 #define CONFIG_FPGA_XILINX
 #define CONFIG_FPGA_SPARTAN3
-#define CONFIG_SYS_FPGA_PROG_FEEDBACK
+#define CONFIG_FPGA_PROG_FEEDBACK
 #define CONFIG_SYS_FPGA_WAIT	10000
 #define CONFIG_MAX_FPGA_DEVICES	1
 #define CONFIG_FPGA_DELAY() udelay(1)
-#define CONFIG_SYS_FPGA_PROG_FEEDBACK
+#define CONFIG_FPGA_PROG_FEEDBACK
 
 #define CONFIG_SPLASH_SCREEN
 #define CONFIG_VIDEO_BMP_RLE8
diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt
index fc2a08a85661..4a2e894367a6 100644
--- a/scripts/config_whitelist.txt
+++ b/scripts/config_whitelist.txt
@@ -5314,7 +5314,7 @@ CONFIG_SYS_FPGA_PHY0_INT
 CONFIG_SYS_FPGA_PHY1_INT
 CONFIG_SYS_FPGA_PRG
 CONFIG_SYS_FPGA_PROG
-CONFIG_SYS_FPGA_PROG_FEEDBACK
+CONFIG_FPGA_PROG_FEEDBACK
 CONFIG_SYS_FPGA_PROG_TIME
 CONFIG_SYS_FPGA_PTR
 CONFIG_SYS_FPGA_REG_BASE
-- 
2.10.2

[U-Boot] [RFC PATCH 1/7] fpga: CONFIG_SYS_FPGA_PROG_FEEDBACK -> CONFIG_FPGA_PROG_FEEDBACK

[Michal Simek]

On 22.11.2016 09:48, Chris Packham wrote:
> Prepare for move to Kconfig by removing "SYS" from the existing macros.
> 
> Signed-off-by: Chris Packham <judge.packham@gmail.com>
> ---
> 
>  README                           |  2 +-
>  board/astro/mcf5373l/fpga.c      |  4 ++--
>  drivers/fpga/ACEX1K.c            |  8 ++++----
>  drivers/fpga/cyclon2.c           | 10 +++++-----
>  drivers/fpga/spartan2.c          | 22 +++++++++++-----------
>  drivers/fpga/spartan3.c          | 20 ++++++++++----------
>  drivers/fpga/virtex2.c           | 18 +++++++++---------
>  include/configs/M54455EVB.h      |  2 +-
>  include/configs/apf27.h          |  2 +-
>  include/configs/astro_mcf5373l.h |  2 +-
>  include/configs/mt_ventoux.h     |  4 ++--
>  scripts/config_whitelist.txt     |  2 +-
>  12 files changed, 48 insertions(+), 48 deletions(-)
> 
> diff --git a/README b/README
> index 47bc215fe9c5..fdc194ab9792 100644
> --- a/README
> +++ b/README
> @@ -2536,7 +2536,7 @@ The following options need to be configured:
>  		Enable support for fpga loadbp command - load partial bitstream
>  		(Xilinx only)
>  
> -		CONFIG_SYS_FPGA_PROG_FEEDBACK
> +		CONFIG_FPGA_PROG_FEEDBACK
>  
>  		Enable printing of hash marks during FPGA configuration.
>  
> diff --git a/board/astro/mcf5373l/fpga.c b/board/astro/mcf5373l/fpga.c
> index 53e00728c3d8..1b53fd2521e3 100644
> --- a/board/astro/mcf5373l/fpga.c
> +++ b/board/astro/mcf5373l/fpga.c
> @@ -128,7 +128,7 @@ int altera_write_fn(const void *buf, size_t len, int flush, int cookie)
>  #if defined(CONFIG_HW_WATCHDOG) || defined(CONFIG_WATCHDOG)
>  			WATCHDOG_RESET();
>  #endif
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  			putc('.');	/* let them know we are alive */
>  #endif
>  #ifdef CONFIG_SYS_FPGA_CHECK_CTRLC
> @@ -347,7 +347,7 @@ int xilinx_fastwr_config_fn(void *buf, size_t len, int flush, int cookie)
>  #if defined(CONFIG_HW_WATCHDOG) || defined(CONFIG_WATCHDOG)
>  			WATCHDOG_RESET();
>  #endif
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  			putc('.');	/* let them know we are alive */
>  #endif
>  #ifdef CONFIG_SYS_FPGA_CHECK_CTRLC
> diff --git a/drivers/fpga/ACEX1K.c b/drivers/fpga/ACEX1K.c
> index 1627f0e6ffde..1fb60e5983c3 100644
> --- a/drivers/fpga/ACEX1K.c
> +++ b/drivers/fpga/ACEX1K.c
> @@ -112,7 +112,7 @@ static int ACEX1K_ps_load(Altera_desc *desc, const void *buf, size_t bsize)
>  				"done:\t0x%p\n\n",
>  				__FUNCTION__, &fn, fn, fn->config, fn->status,
>  				fn->clk, fn->data, fn->done);
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		printf ("Loading FPGA Device %d...", cookie);
>  #endif
>  
> @@ -188,7 +188,7 @@ static int ACEX1K_ps_load(Altera_desc *desc, const void *buf, size_t bsize)
>  				i --;
>  			} while (i > 0);
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  			if (bytecount % (bsize / 40) == 0)
>  				putc ('.');		/* let them know we are alive */
>  #endif
> @@ -196,7 +196,7 @@ static int ACEX1K_ps_load(Altera_desc *desc, const void *buf, size_t bsize)
>  
>  		CONFIG_FPGA_DELAY ();
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		putc (' ');			/* terminate the dotted line */
>  #endif
>  
> @@ -223,7 +223,7 @@ static int ACEX1K_ps_load(Altera_desc *desc, const void *buf, size_t bsize)
>  
>  	ret_val = FPGA_SUCCESS;
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		if (ret_val == FPGA_SUCCESS) {
>  			puts ("Done.\n");
>  		}
> diff --git a/drivers/fpga/cyclon2.c b/drivers/fpga/cyclon2.c
> index 8ab7679b48f7..2be1c1a40a23 100644
> --- a/drivers/fpga/cyclon2.c
> +++ b/drivers/fpga/cyclon2.c
> @@ -115,7 +115,7 @@ static int CYC2_ps_load(Altera_desc *desc, const void *buf, size_t bsize)
>  				"done:\t0x%p\n\n",
>  				__FUNCTION__, &fn, fn, fn->config, fn->status,
>  				fn->write, fn->done);
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		printf ("Loading FPGA Device %d...", cookie);
>  #endif
>  
> @@ -153,13 +153,13 @@ static int CYC2_ps_load(Altera_desc *desc, const void *buf, size_t bsize)
>  			(*fn->abort) (cookie);
>  			return FPGA_FAIL;
>  		}
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		puts(" OK? ...");
>  #endif
>  
>  		CONFIG_FPGA_DELAY ();
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		putc (' ');			/* terminate the dotted line */
>  #endif
>  
> @@ -172,13 +172,13 @@ static int CYC2_ps_load(Altera_desc *desc, const void *buf, size_t bsize)
>  		(*fn->abort) (cookie);
>  		return (FPGA_FAIL);
>  	}
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  	puts(" OK\n");
>  #endif
>  
>  	ret_val = FPGA_SUCCESS;
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  	if (ret_val == FPGA_SUCCESS) {
>  		puts ("Done.\n");
>  	}
> diff --git a/drivers/fpga/spartan2.c b/drivers/fpga/spartan2.c
> index 859fb3c7787a..45e8534cdcb5 100644
> --- a/drivers/fpga/spartan2.c
> +++ b/drivers/fpga/spartan2.c
> @@ -16,7 +16,7 @@
>  #endif
>  
>  #undef CONFIG_SYS_FPGA_CHECK_BUSY
> -#undef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#undef CONFIG_FPGA_PROG_FEEDBACK
>  
>  /* Note: The assumption is that we cannot possibly run fast enough to
>   * overrun the device (the Slave Parallel mode can free run at 50MHz).
> @@ -135,7 +135,7 @@ static int spartan2_sp_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  		 * Continuous Data Loading in Slave Parallel Mode for
>  		 * the Spartan-II Family.
>  		 */
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		printf ("Loading FPGA Device %d...\n", cookie);
>  #endif
>  		/*
> @@ -197,7 +197,7 @@ static int spartan2_sp_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  			}
>  #endif
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  			if (bytecount % (bsize / 40) == 0)
>  				putc ('.');		/* let them know we are alive */
>  #endif
> @@ -207,7 +207,7 @@ static int spartan2_sp_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  		(*fn->cs) (false, true, cookie);	/* Deassert the chip select */
>  		(*fn->wr) (false, true, cookie);	/* Deassert the write pin */
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		putc ('\n');			/* terminate the dotted line */
>  #endif
>  
> @@ -235,7 +235,7 @@ static int spartan2_sp_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  		if (*fn->post)
>  			(*fn->post) (cookie);
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		if (ret_val == FPGA_SUCCESS)
>  			puts ("Done.\n");
>  		else
> @@ -271,7 +271,7 @@ static int spartan2_sp_dump(xilinx_desc *desc, const void *buf, size_t bsize)
>  			(*fn->clk) (false, true, cookie);	/* Deassert the clock pin */
>  			(*fn->clk) (true, true, cookie);	/* Assert the clock pin */
>  			(*fn->rdata) (&(data[bytecount++]), cookie);	/* read the data */
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  			if (bytecount % (bsize / 40) == 0)
>  				putc ('.');		/* let them know we are alive */
>  #endif
> @@ -281,7 +281,7 @@ static int spartan2_sp_dump(xilinx_desc *desc, const void *buf, size_t bsize)
>  		(*fn->clk) (false, true, cookie);	/* Deassert the clock pin */
>  		(*fn->clk) (true, true, cookie);	/* Assert the clock pin */
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		putc ('\n');			/* terminate the dotted line */
>  #endif
>  		puts ("Done.\n");
> @@ -323,7 +323,7 @@ static int spartan2_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  				"done:\t0x%p\n\n",
>  				__FUNCTION__, &fn, fn, fn->pgm, fn->init,
>  				fn->clk, fn->wr, fn->done);
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		printf ("Loading FPGA Device %d...\n", cookie);
>  #endif
>  
> @@ -386,7 +386,7 @@ static int spartan2_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  				i --;
>  			} while (i > 0);
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  			if (bytecount % (bsize / 40) == 0)
>  				putc ('.');		/* let them know we are alive */
>  #endif
> @@ -394,7 +394,7 @@ static int spartan2_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  
>  		CONFIG_FPGA_DELAY ();
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		putc ('\n');			/* terminate the dotted line */
>  #endif
>  
> @@ -426,7 +426,7 @@ static int spartan2_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  		if (*fn->post)
>  			(*fn->post) (cookie);
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		if (ret_val == FPGA_SUCCESS)
>  			puts ("Done.\n");
>  		else
> diff --git a/drivers/fpga/spartan3.c b/drivers/fpga/spartan3.c
> index b0213e69992b..759ad95e2afc 100644
> --- a/drivers/fpga/spartan3.c
> +++ b/drivers/fpga/spartan3.c
> @@ -139,7 +139,7 @@ static int spartan3_sp_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  		 * Continuous Data Loading in Slave Parallel Mode for
>  		 * the Spartan-II Family.
>  		 */
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		printf ("Loading FPGA Device %d...\n", cookie);
>  #endif
>  		/*
> @@ -201,7 +201,7 @@ static int spartan3_sp_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  			}
>  #endif
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  			if (bytecount % (bsize / 40) == 0)
>  				putc ('.');		/* let them know we are alive */
>  #endif
> @@ -211,7 +211,7 @@ static int spartan3_sp_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  		(*fn->cs) (false, true, cookie);	/* Deassert the chip select */
>  		(*fn->wr) (false, true, cookie);	/* Deassert the write pin */
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		putc ('\n');			/* terminate the dotted line */
>  #endif
>  
> @@ -241,7 +241,7 @@ static int spartan3_sp_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  		if (*fn->post)
>  			(*fn->post) (cookie);
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		if (ret_val == FPGA_SUCCESS)
>  			puts ("Done.\n");
>  		else
> @@ -277,7 +277,7 @@ static int spartan3_sp_dump(xilinx_desc *desc, const void *buf, size_t bsize)
>  			(*fn->clk) (false, true, cookie);	/* Deassert the clock pin */
>  			(*fn->clk) (true, true, cookie);	/* Assert the clock pin */
>  			(*fn->rdata) (&(data[bytecount++]), cookie);	/* read the data */
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  			if (bytecount % (bsize / 40) == 0)
>  				putc ('.');		/* let them know we are alive */
>  #endif
> @@ -287,7 +287,7 @@ static int spartan3_sp_dump(xilinx_desc *desc, const void *buf, size_t bsize)
>  		(*fn->clk) (false, true, cookie);	/* Deassert the clock pin */
>  		(*fn->clk) (true, true, cookie);	/* Assert the clock pin */
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		putc ('\n');			/* terminate the dotted line */
>  #endif
>  		puts ("Done.\n");
> @@ -329,7 +329,7 @@ static int spartan3_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  				"done:\t0x%p\n\n",
>  				__FUNCTION__, &fn, fn, fn->pgm, fn->init,
>  				fn->clk, fn->wr, fn->done);
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		printf ("Loading FPGA Device %d...\n", cookie);
>  #endif
>  
> @@ -401,7 +401,7 @@ static int spartan3_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  					i --;
>  				} while (i > 0);
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  				if (bytecount % (bsize / 40) == 0)
>  					putc ('.');		/* let them know we are alive */
>  #endif
> @@ -410,7 +410,7 @@ static int spartan3_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  
>  		CONFIG_FPGA_DELAY ();
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		putc ('\n');			/* terminate the dotted line */
>  #endif
>  
> @@ -444,7 +444,7 @@ static int spartan3_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  		if (*fn->post)
>  			(*fn->post) (cookie);
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		if (ret_val == FPGA_SUCCESS)
>  			puts ("Done.\n");
>  		else
> diff --git a/drivers/fpga/virtex2.c b/drivers/fpga/virtex2.c
> index f7cf02ab5b30..cf7723c46f64 100644
> --- a/drivers/fpga/virtex2.c
> +++ b/drivers/fpga/virtex2.c
> @@ -39,8 +39,8 @@
>  #define CONFIG_FPGA_DELAY()
>  #endif
>  
> -#ifndef CONFIG_SYS_FPGA_PROG_FEEDBACK
> -#define CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifndef CONFIG_FPGA_PROG_FEEDBACK
> +#define CONFIG_FPGA_PROG_FEEDBACK
>  #endif
>  
>  /*
> @@ -190,7 +190,7 @@ static int virtex2_ssm_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  				fn->clk, fn->cs, fn->wr, fn->rdata, fn->wdata,
>  				fn->busy, fn->abort, fn->post);
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		printf ("Initializing FPGA Device %d...\n", cookie);
>  #endif
>  		/*
> @@ -294,7 +294,7 @@ static int virtex2_ssm_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  			}
>  #endif
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  			if (bytecount % (bsize / 40) == 0)
>  				putc ('.');
>  #endif
> @@ -307,7 +307,7 @@ static int virtex2_ssm_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  		(*fn->cs) (false, true, cookie);
>  		(*fn->wr) (false, true, cookie);
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		putc ('\n');
>  #endif
>  
> @@ -329,7 +329,7 @@ static int virtex2_ssm_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  		}
>  
>  		if (ret_val == FPGA_SUCCESS) {
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  			printf ("Initialization of FPGA device %d complete\n", cookie);
>  #endif
>  			/*
> @@ -339,7 +339,7 @@ static int virtex2_ssm_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  				(*fn->post) (cookie);
>  			}
>  		} else {
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  			printf ("** Initialization of FPGA device %d FAILED\n",
>  					cookie);
>  #endif
> @@ -382,7 +382,7 @@ static int virtex2_ssm_dump(xilinx_desc *desc, const void *buf, size_t bsize)
>  			(*fn->clk) (false, true, cookie);
>  			(*fn->clk) (true, true, cookie);
>  			(*fn->rdata) (&(data[bytecount++]), cookie);
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  			if (bytecount % (bsize / 40) == 0)
>  				putc ('.');
>  #endif
> @@ -395,7 +395,7 @@ static int virtex2_ssm_dump(xilinx_desc *desc, const void *buf, size_t bsize)
>  		(*fn->clk) (false, true, cookie);
>  		(*fn->clk) (true, true, cookie);
>  
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		putc ('\n');
>  #endif
>  		puts ("Done.\n");
> diff --git a/include/configs/M54455EVB.h b/include/configs/M54455EVB.h
> index abec3737a498..d80521664611 100644
> --- a/include/configs/M54455EVB.h
> +++ b/include/configs/M54455EVB.h
> @@ -193,7 +193,7 @@
>  /* experiment
>  #define CONFIG_FPGA
>  #define CONFIG_FPGA_COUNT	1
> -#define CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#define CONFIG_FPGA_PROG_FEEDBACK
>  #define CONFIG_SYS_FPGA_CHECK_CTRLC
>  */
>  
> diff --git a/include/configs/apf27.h b/include/configs/apf27.h
> index b903ba53d1dc..a97ba9f39c50 100644
> --- a/include/configs/apf27.h
> +++ b/include/configs/apf27.h
> @@ -267,7 +267,7 @@
>  #define CONFIG_FPGA_XILINX
>  #define CONFIG_FPGA_SPARTAN3
>  #define CONFIG_SYS_FPGA_WAIT		250 /* 250 ms */
> -#define CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#define CONFIG_FPGA_PROG_FEEDBACK
>  #define CONFIG_SYS_FPGA_CHECK_CTRLC
>  #define CONFIG_SYS_FPGA_CHECK_ERROR
>  
> diff --git a/include/configs/astro_mcf5373l.h b/include/configs/astro_mcf5373l.h
> index e8dca0b2ccdc..0af356bcb49d 100644
> --- a/include/configs/astro_mcf5373l.h
> +++ b/include/configs/astro_mcf5373l.h
> @@ -209,7 +209,7 @@
>  #define	CONFIG_FPGA_SPARTAN3
>  #define CONFIG_FPGA_ALTERA
>  #define CONFIG_FPGA_CYCLON2
> -#define CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#define CONFIG_FPGA_PROG_FEEDBACK
>  #define CONFIG_SYS_FPGA_WAIT		1000
>  
>  /* End of user parameters to be customized */
> diff --git a/include/configs/mt_ventoux.h b/include/configs/mt_ventoux.h
> index 2420612800fb..1353b11b1250 100644
> --- a/include/configs/mt_ventoux.h
> +++ b/include/configs/mt_ventoux.h
> @@ -45,11 +45,11 @@
>  #define CONFIG_FPGA
>  #define CONFIG_FPGA_XILINX
>  #define CONFIG_FPGA_SPARTAN3
> -#define CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#define CONFIG_FPGA_PROG_FEEDBACK
>  #define CONFIG_SYS_FPGA_WAIT	10000
>  #define CONFIG_MAX_FPGA_DEVICES	1
>  #define CONFIG_FPGA_DELAY() udelay(1)
> -#define CONFIG_SYS_FPGA_PROG_FEEDBACK
> +#define CONFIG_FPGA_PROG_FEEDBACK
>  
>  #define CONFIG_SPLASH_SCREEN
>  #define CONFIG_VIDEO_BMP_RLE8
> diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt
> index fc2a08a85661..4a2e894367a6 100644
> --- a/scripts/config_whitelist.txt
> +++ b/scripts/config_whitelist.txt
> @@ -5314,7 +5314,7 @@ CONFIG_SYS_FPGA_PHY0_INT
>  CONFIG_SYS_FPGA_PHY1_INT
>  CONFIG_SYS_FPGA_PRG
>  CONFIG_SYS_FPGA_PROG
> -CONFIG_SYS_FPGA_PROG_FEEDBACK
> +CONFIG_FPGA_PROG_FEEDBACK
>  CONFIG_SYS_FPGA_PROG_TIME
>  CONFIG_SYS_FPGA_PTR
>  CONFIG_SYS_FPGA_REG_BASE
> 

This is breaking buildman. You should introduce  Kconfig option in first
patch and than this one.

[u-boot]$ ./tools/buildman/buildman  apf27 -c 2
boards.cfg is up to date. Nothing to do.
Building current source for 1 boards (1 thread, 8 jobs per thread)
       arm:  +   apf27           )
+comm: file 2 is not in sorted order
+Error: You must add new CONFIG options using Kconfig
+The following new ad-hoc CONFIG options were detected:
+CONFIG_FPGA_PROG_FEEDBACK
+
+Please add these via Kconfig instead. Find a suitable Kconfig
+file and add a 'config' or 'menuconfig' option.
+make[1]: *** [all] Error 1
+make: *** [sub-make] Error 2
    0    0    1 /1      apf27

Thanks,
Michal

[U-Boot] [RFC PATCH 1/7] fpga: CONFIG_SYS_FPGA_PROG_FEEDBACK -> CONFIG_FPGA_PROG_FEEDBACK

[Chris Packham]

(dropped addresses that we're bouncing from Cc, hope I got Jagans new
address right).

On Thu, Nov 24, 2016 at 1:09 AM, Michal Simek <michal.simek@xilinx.com> wrote:
> On 22.11.2016 09:48, Chris Packham wrote:
>> Prepare for move to Kconfig by removing "SYS" from the existing macros.
>>
>> Signed-off-by: Chris Packham <judge.packham@gmail.com>
>> ---
>
> This is breaking buildman. You should introduce  Kconfig option in first
> patch and than this one.
>
> [u-boot]$ ./tools/buildman/buildman  apf27 -c 2
> boards.cfg is up to date. Nothing to do.
> Building current source for 1 boards (1 thread, 8 jobs per thread)
>        arm:  +   apf27           )
> +comm: file 2 is not in sorted order
> +Error: You must add new CONFIG options using Kconfig
> +The following new ad-hoc CONFIG options were detected:
> +CONFIG_FPGA_PROG_FEEDBACK
> +
> +Please add these via Kconfig instead. Find a suitable Kconfig
> +file and add a 'config' or 'menuconfig' option.
> +make[1]: *** [all] Error 1
> +make: *** [sub-make] Error 2
>     0    0    1 /1      apf27

So I agree that this should probably be closer to the other two
patches. But won't I have the same problem either way. Unless I squash
this with "Move FPGA_PROG_FEEDBACK to defconfig" which I can do Is
that acceptable to everyone.

>
> Thanks,
> Michal
>

[U-Boot] [RFC PATCH 1/7] fpga: CONFIG_SYS_FPGA_PROG_FEEDBACK -> CONFIG_FPGA_PROG_FEEDBACK

[Michal Simek]

On 25.11.2016 00:57, Chris Packham wrote:
> (dropped addresses that we're bouncing from Cc, hope I got Jagans new
> address right).
> 
> On Thu, Nov 24, 2016 at 1:09 AM, Michal Simek <michal.simek@xilinx.com> wrote:
>> On 22.11.2016 09:48, Chris Packham wrote:
>>> Prepare for move to Kconfig by removing "SYS" from the existing macros.
>>>
>>> Signed-off-by: Chris Packham <judge.packham@gmail.com>
>>> ---
>>
>> This is breaking buildman. You should introduce  Kconfig option in first
>> patch and than this one.
>>
>> [u-boot]$ ./tools/buildman/buildman  apf27 -c 2
>> boards.cfg is up to date. Nothing to do.
>> Building current source for 1 boards (1 thread, 8 jobs per thread)
>>        arm:  +   apf27           )
>> +comm: file 2 is not in sorted order
>> +Error: You must add new CONFIG options using Kconfig
>> +The following new ad-hoc CONFIG options were detected:
>> +CONFIG_FPGA_PROG_FEEDBACK
>> +
>> +Please add these via Kconfig instead. Find a suitable Kconfig
>> +file and add a 'config' or 'menuconfig' option.
>> +make[1]: *** [all] Error 1
>> +make: *** [sub-make] Error 2
>>     0    0    1 /1      apf27
> 
> So I agree that this should probably be closer to the other two
> patches. But won't I have the same problem either way. Unless I squash
> this with "Move FPGA_PROG_FEEDBACK to defconfig" which I can do Is
> that acceptable to everyone.

you shouldn't break buildman.
You don't need to squash patches - there is no reason for that.
Add option to Kconfig in the first patch. Then convert macros in the
second patch and then move option from configs to defconfig in third patch.
This way will keep buildman happy.

Thanks,
Michal

[U-Boot] [RFC PATCH 2/7] fpga: add Kconfig support for Xilinx Spartan devices

[Chris Packham]

Signed-off-by: Chris Packham <judge.packham@gmail.com>
---

 drivers/fpga/Kconfig | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig
index f3f6bf7f6747..d137ab2df1d7 100644
--- a/drivers/fpga/Kconfig
+++ b/drivers/fpga/Kconfig
@@ -17,4 +17,15 @@ config FPGA_ZYNQMPPL
 	  Enable FPGA driver for loading bitstream in BIT and BIN format
 	  on Xilinx Zynq UltraScale+ (ZynqMP) device.
 
+config FPGA_SPARTAN3
+	bool "Enable Xilinx Spartan-III driver"
+	depends on FPGA_XILINX
+	help
+	  Enable FPGA driver for Xilinx Spartan-III devices.
+
+config FPGA_SPARTAN2
+	bool "Enable Xilinx Spartan-II driver"
+	depends on FPGA_XILINX
+	help
+	  Enable FPGA driver for Xilinx Spartan-II devices.
 endmenu
-- 
2.10.2

[U-Boot] [RFC PATCH 2/7] fpga: add Kconfig support for Xilinx Spartan devices

[Michal Simek]

On 22.11.2016 09:48, Chris Packham wrote:
> Signed-off-by: Chris Packham <judge.packham@gmail.com>
> ---
> 
>  drivers/fpga/Kconfig | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig
> index f3f6bf7f6747..d137ab2df1d7 100644
> --- a/drivers/fpga/Kconfig
> +++ b/drivers/fpga/Kconfig
> @@ -17,4 +17,15 @@ config FPGA_ZYNQMPPL
>  	  Enable FPGA driver for loading bitstream in BIT and BIN format
>  	  on Xilinx Zynq UltraScale+ (ZynqMP) device.
>  
> +config FPGA_SPARTAN3
> +	bool "Enable Xilinx Spartan-III driver"
> +	depends on FPGA_XILINX
> +	help
> +	  Enable FPGA driver for Xilinx Spartan-III devices.
> +
> +config FPGA_SPARTAN2
> +	bool "Enable Xilinx Spartan-II driver"
> +	depends on FPGA_XILINX
> +	help
> +	  Enable FPGA driver for Xilinx Spartan-II devices.
>  endmenu
> 

Acked-by: Michal Simek <michal.simek@xilinx.com>

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP SoCs


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161123/063b3e17/attachment.sig>

[U-Boot] [RFC PATCH 3/7] Move CONFIG_FPGA_SPARTAN options to defconfig

[Chris Packham]

Signed-off-by: Chris Packham <judge.packham@gmail.com>
---

 configs/PMC440_defconfig         | 3 +++
 configs/apf27_defconfig          | 2 ++
 configs/astro_mcf5373l_defconfig | 2 ++
 configs/mt_ventoux_defconfig     | 2 ++
 configs/x600_defconfig           | 2 ++
 include/configs/PMC440.h         | 4 ----
 include/configs/apf27.h          | 5 -----
 include/configs/astro_mcf5373l.h | 3 ---
 include/configs/mt_ventoux.h     | 3 ---
 include/configs/x600.h           | 3 ---
 10 files changed, 11 insertions(+), 18 deletions(-)

diff --git a/configs/PMC440_defconfig b/configs/PMC440_defconfig
index cd2f0d2a5b4b..8a2e9c964e14 100644
--- a/configs/PMC440_defconfig
+++ b/configs/PMC440_defconfig
@@ -16,6 +16,9 @@ CONFIG_CMD_DHCP=y
 CONFIG_CMD_MII=y
 CONFIG_CMD_PING=y
 CONFIG_CMD_FAT=y
+CONFIG_FPGA_XILINX=y
+CONFIG_FPGA_SPARTAN3=y
+CONFIG_FPGA_SPARTAN2=y
 CONFIG_SYS_NS16550=y
 CONFIG_USB=y
 CONFIG_USB_STORAGE=y
diff --git a/configs/apf27_defconfig b/configs/apf27_defconfig
index a02e180d2f2c..f8f177ad56ce 100644
--- a/configs/apf27_defconfig
+++ b/configs/apf27_defconfig
@@ -21,4 +21,6 @@ CONFIG_CMD_CACHE=y
 CONFIG_CMD_EXT2=y
 CONFIG_CMD_FAT=y
 CONFIG_CMD_UBI=y
+CONFIG_FPGA_XILINX=y
+CONFIG_FPGA_SPARTAN3=y
 CONFIG_OF_LIBFDT=y
diff --git a/configs/astro_mcf5373l_defconfig b/configs/astro_mcf5373l_defconfig
index 94e931eabadc..5ea1b66e5b2a 100644
--- a/configs/astro_mcf5373l_defconfig
+++ b/configs/astro_mcf5373l_defconfig
@@ -9,3 +9,5 @@ CONFIG_CMD_I2C=y
 # CONFIG_CMD_NET is not set
 # CONFIG_CMD_NFS is not set
 CONFIG_CMD_CACHE=y
+CONFIG_FPGA_XILINX=y
+CONFIG_FPGA_SPARTAN3=y
diff --git a/configs/mt_ventoux_defconfig b/configs/mt_ventoux_defconfig
index 9e330a7f44ca..d4e58395cf22 100644
--- a/configs/mt_ventoux_defconfig
+++ b/configs/mt_ventoux_defconfig
@@ -24,6 +24,8 @@ CONFIG_CMD_CACHE=y
 CONFIG_CMD_EXT2=y
 CONFIG_CMD_FAT=y
 CONFIG_CMD_UBI=y
+CONFIG_FPGA_XILINX=y
+CONFIG_FPGA_SPARTAN3=y
 CONFIG_SYS_NS16550=y
 CONFIG_USB=y
 CONFIG_USB_ULPI_VIEWPORT_OMAP=y
diff --git a/configs/x600_defconfig b/configs/x600_defconfig
index 65b7d9565aac..0db6b5c6ba20 100644
--- a/configs/x600_defconfig
+++ b/configs/x600_defconfig
@@ -26,6 +26,8 @@ CONFIG_CMD_CACHE=y
 CONFIG_CMD_FAT=y
 CONFIG_CMD_FS_GENERIC=y
 CONFIG_CMD_UBI=y
+CONFIG_FPGA_XILINX=y
+CONFIG_FPGA_SPARTAN3=y
 CONFIG_SYS_I2C_DW=y
 CONFIG_NETDEVICES=y
 CONFIG_ETH_DESIGNWARE=y
diff --git a/include/configs/PMC440.h b/include/configs/PMC440.h
index 2e7f6e480323..750f8447220c 100644
--- a/include/configs/PMC440.h
+++ b/include/configs/PMC440.h
@@ -343,10 +343,6 @@
 /*-----------------------------------------------------------------------
  * FPGA stuff
  *----------------------------------------------------------------------*/
-#define CONFIG_FPGA
-#define CONFIG_FPGA_XILINX
-#define CONFIG_FPGA_SPARTAN2
-#define CONFIG_FPGA_SPARTAN3
 
 #define CONFIG_FPGA_COUNT	2
 /*-----------------------------------------------------------------------
diff --git a/include/configs/apf27.h b/include/configs/apf27.h
index a97ba9f39c50..00798fa1027f 100644
--- a/include/configs/apf27.h
+++ b/include/configs/apf27.h
@@ -260,12 +260,7 @@
 /*
  * FPGA
  */
-#ifndef CONFIG_SPL_BUILD
-#define CONFIG_FPGA
-#endif
 #define CONFIG_FPGA_COUNT		1
-#define CONFIG_FPGA_XILINX
-#define CONFIG_FPGA_SPARTAN3
 #define CONFIG_SYS_FPGA_WAIT		250 /* 250 ms */
 #define CONFIG_FPGA_PROG_FEEDBACK
 #define CONFIG_SYS_FPGA_CHECK_CTRLC
diff --git a/include/configs/astro_mcf5373l.h b/include/configs/astro_mcf5373l.h
index 0af356bcb49d..d5d6540ec9b9 100644
--- a/include/configs/astro_mcf5373l.h
+++ b/include/configs/astro_mcf5373l.h
@@ -204,9 +204,6 @@
 #define CONFIG_SYS_BARGSIZE		CONFIG_SYS_CBSIZE
 
 #define CONFIG_FPGA_COUNT	1
-#define CONFIG_FPGA
-#define	CONFIG_FPGA_XILINX
-#define	CONFIG_FPGA_SPARTAN3
 #define CONFIG_FPGA_ALTERA
 #define CONFIG_FPGA_CYCLON2
 #define CONFIG_FPGA_PROG_FEEDBACK
diff --git a/include/configs/mt_ventoux.h b/include/configs/mt_ventoux.h
index 1353b11b1250..b6198dbdf6d3 100644
--- a/include/configs/mt_ventoux.h
+++ b/include/configs/mt_ventoux.h
@@ -42,9 +42,6 @@
  * FPGA
  */
 #define CONFIG_CMD_FPGA_LOADMK
-#define CONFIG_FPGA
-#define CONFIG_FPGA_XILINX
-#define CONFIG_FPGA_SPARTAN3
 #define CONFIG_FPGA_PROG_FEEDBACK
 #define CONFIG_SYS_FPGA_WAIT	10000
 #define CONFIG_MAX_FPGA_DEVICES	1
diff --git a/include/configs/x600.h b/include/configs/x600.h
index 5e51f6535624..a879c4c7c113 100644
--- a/include/configs/x600.h
+++ b/include/configs/x600.h
@@ -96,9 +96,6 @@
 #define CONFIG_SYS_I2C_RTC_ADDR	0x68
 
 /* FPGA config options */
-#define CONFIG_FPGA
-#define CONFIG_FPGA_XILINX
-#define CONFIG_FPGA_SPARTAN3
 #define CONFIG_FPGA_COUNT	1
 
 /* USB EHCI options */
-- 
2.10.2

[U-Boot] [RFC PATCH 3/7] Move CONFIG_FPGA_SPARTAN options to defconfig

[Michal Simek]

On 22.11.2016 09:48, Chris Packham wrote:
> Signed-off-by: Chris Packham <judge.packham@gmail.com>
> ---
> 
>  configs/PMC440_defconfig         | 3 +++
>  configs/apf27_defconfig          | 2 ++
>  configs/astro_mcf5373l_defconfig | 2 ++
>  configs/mt_ventoux_defconfig     | 2 ++
>  configs/x600_defconfig           | 2 ++
>  include/configs/PMC440.h         | 4 ----
>  include/configs/apf27.h          | 5 -----
>  include/configs/astro_mcf5373l.h | 3 ---
>  include/configs/mt_ventoux.h     | 3 ---
>  include/configs/x600.h           | 3 ---
>  10 files changed, 11 insertions(+), 18 deletions(-)
> 
> diff --git a/configs/PMC440_defconfig b/configs/PMC440_defconfig
> index cd2f0d2a5b4b..8a2e9c964e14 100644
> --- a/configs/PMC440_defconfig
> +++ b/configs/PMC440_defconfig
> @@ -16,6 +16,9 @@ CONFIG_CMD_DHCP=y
>  CONFIG_CMD_MII=y
>  CONFIG_CMD_PING=y
>  CONFIG_CMD_FAT=y
> +CONFIG_FPGA_XILINX=y
> +CONFIG_FPGA_SPARTAN3=y
> +CONFIG_FPGA_SPARTAN2=y
>  CONFIG_SYS_NS16550=y
>  CONFIG_USB=y
>  CONFIG_USB_STORAGE=y
> diff --git a/configs/apf27_defconfig b/configs/apf27_defconfig
> index a02e180d2f2c..f8f177ad56ce 100644
> --- a/configs/apf27_defconfig
> +++ b/configs/apf27_defconfig
> @@ -21,4 +21,6 @@ CONFIG_CMD_CACHE=y
>  CONFIG_CMD_EXT2=y
>  CONFIG_CMD_FAT=y
>  CONFIG_CMD_UBI=y
> +CONFIG_FPGA_XILINX=y
> +CONFIG_FPGA_SPARTAN3=y
>  CONFIG_OF_LIBFDT=y
> diff --git a/configs/astro_mcf5373l_defconfig b/configs/astro_mcf5373l_defconfig
> index 94e931eabadc..5ea1b66e5b2a 100644
> --- a/configs/astro_mcf5373l_defconfig
> +++ b/configs/astro_mcf5373l_defconfig
> @@ -9,3 +9,5 @@ CONFIG_CMD_I2C=y
>  # CONFIG_CMD_NET is not set
>  # CONFIG_CMD_NFS is not set
>  CONFIG_CMD_CACHE=y
> +CONFIG_FPGA_XILINX=y
> +CONFIG_FPGA_SPARTAN3=y
> diff --git a/configs/mt_ventoux_defconfig b/configs/mt_ventoux_defconfig
> index 9e330a7f44ca..d4e58395cf22 100644
> --- a/configs/mt_ventoux_defconfig
> +++ b/configs/mt_ventoux_defconfig
> @@ -24,6 +24,8 @@ CONFIG_CMD_CACHE=y
>  CONFIG_CMD_EXT2=y
>  CONFIG_CMD_FAT=y
>  CONFIG_CMD_UBI=y
> +CONFIG_FPGA_XILINX=y
> +CONFIG_FPGA_SPARTAN3=y
>  CONFIG_SYS_NS16550=y
>  CONFIG_USB=y
>  CONFIG_USB_ULPI_VIEWPORT_OMAP=y
> diff --git a/configs/x600_defconfig b/configs/x600_defconfig
> index 65b7d9565aac..0db6b5c6ba20 100644
> --- a/configs/x600_defconfig
> +++ b/configs/x600_defconfig
> @@ -26,6 +26,8 @@ CONFIG_CMD_CACHE=y
>  CONFIG_CMD_FAT=y
>  CONFIG_CMD_FS_GENERIC=y
>  CONFIG_CMD_UBI=y
> +CONFIG_FPGA_XILINX=y
> +CONFIG_FPGA_SPARTAN3=y
>  CONFIG_SYS_I2C_DW=y
>  CONFIG_NETDEVICES=y
>  CONFIG_ETH_DESIGNWARE=y
> diff --git a/include/configs/PMC440.h b/include/configs/PMC440.h
> index 2e7f6e480323..750f8447220c 100644
> --- a/include/configs/PMC440.h
> +++ b/include/configs/PMC440.h
> @@ -343,10 +343,6 @@
>  /*-----------------------------------------------------------------------
>   * FPGA stuff
>   *----------------------------------------------------------------------*/
> -#define CONFIG_FPGA
> -#define CONFIG_FPGA_XILINX
> -#define CONFIG_FPGA_SPARTAN2
> -#define CONFIG_FPGA_SPARTAN3
>  
>  #define CONFIG_FPGA_COUNT	2
>  /*-----------------------------------------------------------------------
> diff --git a/include/configs/apf27.h b/include/configs/apf27.h
> index a97ba9f39c50..00798fa1027f 100644
> --- a/include/configs/apf27.h
> +++ b/include/configs/apf27.h
> @@ -260,12 +260,7 @@
>  /*
>   * FPGA
>   */
> -#ifndef CONFIG_SPL_BUILD
> -#define CONFIG_FPGA
> -#endif

Isn't this causing some compilation issue for SPL case?

Thanks,
Michal


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161123/57a6b180/attachment.sig>

[U-Boot] [RFC PATCH 3/7] Move CONFIG_FPGA_SPARTAN options to defconfig

[Chris Packham]

On Thu, Nov 24, 2016 at 1:14 AM, Michal Simek <monstr@monstr.eu> wrote:
> On 22.11.2016 09:48, Chris Packham wrote:
>> Signed-off-by: Chris Packham <judge.packham@gmail.com>
>> ---
>
> Isn't this causing some compilation issue for SPL case?
>

I have compile tested apf27 and didn't see an issue. I'm guessing none
of the FPGA infrastructure is available in SPL.

[U-Boot] [RFC PATCH 3/7] Move CONFIG_FPGA_SPARTAN options to defconfig

[Chris Packham]

On Fri, Nov 25, 2016 at 1:53 PM, Chris Packham <judge.packham@gmail.com> wrote:
> On Thu, Nov 24, 2016 at 1:14 AM, Michal Simek <monstr@monstr.eu> wrote:
>> On 22.11.2016 09:48, Chris Packham wrote:
>>> Signed-off-by: Chris Packham <judge.packham@gmail.com>
>>> ---
>>
>> Isn't this causing some compilation issue for SPL case?
>>
>
> I have compile tested apf27 and didn't see an issue. I'm guessing none
> of the FPGA infrastructure is available in SPL.

A better non-guess answer is that yes indeed all of drivers/fgpa is
not included in SPL. But CONFIG_FPGA now ends up defined and nm tells
me I've probably broken that board

 nm spl/u-boot-spl | grep fpga_init
         U fpga_init

So how is this usually handled? I can surround the code in
board/armadeus/apf27 with #ifndef CONFIG_SPL_BUILD but that seems kind
of hacky.

[U-Boot] [RFC PATCH 4/7] fpga/spartan3: make output quieter

[Chris Packham]

Suppress some putc calls with CONFIG_SYS_FPGA_PROG_FEEDBACK.

Signed-off-by: Chris Packham <judge.packham@gmail.com>
---

 drivers/fpga/spartan3.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/fpga/spartan3.c b/drivers/fpga/spartan3.c
index 759ad95e2afc..34aa097ab6c2 100644
--- a/drivers/fpga/spartan3.c
+++ b/drivers/fpga/spartan3.c
@@ -428,7 +428,9 @@ static int spartan3_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
 			CONFIG_FPGA_DELAY ();
 			(*fn->clk) (true, true, cookie);	/* Assert the clock pin */
 
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 			putc ('*');
+#endif
 
 			if (get_timer (ts) > CONFIG_SYS_FPGA_WAIT) {	/* check the time */
 				puts ("** Timeout waiting for DONE to clear.\n");
@@ -436,7 +438,9 @@ static int spartan3_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
 				break;
 			}
 		}
+#ifdef CONFIG_FPGA_PROG_FEEDBACK
 		putc ('\n');			/* terminate the dotted line */
+#endif
 
 		/*
 		 * Run the post configuration function if there is one.
-- 
2.10.2

[U-Boot] [RFC PATCH 4/7] fpga/spartan3: make output quieter

[Michal Simek]

On 22.11.2016 09:48, Chris Packham wrote:
> Suppress some putc calls with CONFIG_SYS_FPGA_PROG_FEEDBACK.
> 
> Signed-off-by: Chris Packham <judge.packham@gmail.com>
> ---
> 
>  drivers/fpga/spartan3.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/fpga/spartan3.c b/drivers/fpga/spartan3.c
> index 759ad95e2afc..34aa097ab6c2 100644
> --- a/drivers/fpga/spartan3.c
> +++ b/drivers/fpga/spartan3.c
> @@ -428,7 +428,9 @@ static int spartan3_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  			CONFIG_FPGA_DELAY ();
>  			(*fn->clk) (true, true, cookie);	/* Assert the clock pin */
>  
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  			putc ('*');
> +#endif
>  
>  			if (get_timer (ts) > CONFIG_SYS_FPGA_WAIT) {	/* check the time */
>  				puts ("** Timeout waiting for DONE to clear.\n");
> @@ -436,7 +438,9 @@ static int spartan3_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  				break;
>  			}
>  		}
> +#ifdef CONFIG_FPGA_PROG_FEEDBACK
>  		putc ('\n');			/* terminate the dotted line */
> +#endif
>  
>  		/*
>  		 * Run the post configuration function if there is one.
> 

How does the output look like?

Thanks,
Michal


-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP SoCs


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161123/897aee69/attachment.sig>

[U-Boot] [RFC PATCH 5/7] fpga: add Kconfig for CONFIG_FPGA_PROG_FEEDBACK

[Chris Packham]

Signed-off-by: Chris Packham <judge.packham@gmail.com>
---

 drivers/fpga/Kconfig | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig
index d137ab2df1d7..1b3b03c8f938 100644
--- a/drivers/fpga/Kconfig
+++ b/drivers/fpga/Kconfig
@@ -28,4 +28,10 @@ config FPGA_SPARTAN2
 	depends on FPGA_XILINX
 	help
 	  Enable FPGA driver for Xilinx Spartan-II devices.
+
+config FPGA_PROG_FEEDBACK
+	bool "FPGA programming feedback"
+	help
+	  Enable printing of hash marks during FPGA configuration.
+
 endmenu
-- 
2.10.2

[U-Boot] [RFC PATCH 5/7] fpga: add Kconfig for CONFIG_FPGA_PROG_FEEDBACK

[Michal Simek]

On 22.11.2016 09:49, Chris Packham wrote:
> Signed-off-by: Chris Packham <judge.packham@gmail.com>
> ---
> 
>  drivers/fpga/Kconfig | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig
> index d137ab2df1d7..1b3b03c8f938 100644
> --- a/drivers/fpga/Kconfig
> +++ b/drivers/fpga/Kconfig
> @@ -28,4 +28,10 @@ config FPGA_SPARTAN2
>  	depends on FPGA_XILINX
>  	help
>  	  Enable FPGA driver for Xilinx Spartan-II devices.
> +
> +config FPGA_PROG_FEEDBACK
> +	bool "FPGA programming feedback"
> +	help
> +	  Enable printing of hash marks during FPGA configuration.
> +
>  endmenu
> 

I would prefer if you can add depends on here because this feature is
not available on newer drivers.

It means you have to add some Kconfig things for cyclone.

Thanks,
Michal


-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP SoCs


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161123/2ee8cf90/attachment.sig>

[U-Boot] [RFC PATCH 5/7] fpga: add Kconfig for CONFIG_FPGA_PROG_FEEDBACK

[Chris Packham]

On Thu, Nov 24, 2016 at 1:13 AM, Michal Simek <monstr@monstr.eu> wrote:
> On 22.11.2016 09:49, Chris Packham wrote:
>> Signed-off-by: Chris Packham <judge.packham@gmail.com>
>> ---
>>
>>  drivers/fpga/Kconfig | 6 ++++++
>>  1 file changed, 6 insertions(+)
>>
>> diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig
>> index d137ab2df1d7..1b3b03c8f938 100644
>> --- a/drivers/fpga/Kconfig
>> +++ b/drivers/fpga/Kconfig
>> @@ -28,4 +28,10 @@ config FPGA_SPARTAN2
>>       depends on FPGA_XILINX
>>       help
>>         Enable FPGA driver for Xilinx Spartan-II devices.
>> +
>> +config FPGA_PROG_FEEDBACK
>> +     bool "FPGA programming feedback"
>> +     help
>> +       Enable printing of hash marks during FPGA configuration.
>> +
>>  endmenu
>>
>
> I would prefer if you can add depends on here because this feature is
> not available on newer drivers.
>
> It means you have to add some Kconfig things for cyclone.

Cyclone, virtex and ACEX1K. I'll see if can do it. Guess I need to
start playing with buildman (and/or travis).

>
> Thanks,
> Michal
>
>
> --
> Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
> w: www.monstr.eu p: +42-0-721842854
> Maintainer of Linux kernel - Xilinx Microblaze
> Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
> U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP SoCs
>
>

[U-Boot] [RFC PATCH 5/7] fpga: add Kconfig for CONFIG_FPGA_PROG_FEEDBACK

[Michal Simek]

On 25.11.2016 01:10, Chris Packham wrote:
> On Thu, Nov 24, 2016 at 1:13 AM, Michal Simek <monstr@monstr.eu> wrote:
>> On 22.11.2016 09:49, Chris Packham wrote:
>>> Signed-off-by: Chris Packham <judge.packham@gmail.com>
>>> ---
>>>
>>>  drivers/fpga/Kconfig | 6 ++++++
>>>  1 file changed, 6 insertions(+)
>>>
>>> diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig
>>> index d137ab2df1d7..1b3b03c8f938 100644
>>> --- a/drivers/fpga/Kconfig
>>> +++ b/drivers/fpga/Kconfig
>>> @@ -28,4 +28,10 @@ config FPGA_SPARTAN2
>>>       depends on FPGA_XILINX
>>>       help
>>>         Enable FPGA driver for Xilinx Spartan-II devices.
>>> +
>>> +config FPGA_PROG_FEEDBACK
>>> +     bool "FPGA programming feedback"
>>> +     help
>>> +       Enable printing of hash marks during FPGA configuration.
>>> +
>>>  endmenu
>>>
>>
>> I would prefer if you can add depends on here because this feature is
>> not available on newer drivers.
>>
>> It means you have to add some Kconfig things for cyclone.
> 
> Cyclone, virtex and ACEX1K. I'll see if can do it. Guess I need to
> start playing with buildman (and/or travis).

I was playing with travis recently and it is easy to use too. You can
use it but for start buildman running on your PC should be fine.

Thanks,
Michal

[U-Boot] [RFC PATCH 6/7] Move FPGA_PROG_FEEDBACK to defconfig

[Chris Packham]

Signed-off-by: Chris Packham <judge.packham@gmail.com>
---

 configs/apf27_defconfig          | 1 +
 configs/astro_mcf5373l_defconfig | 1 +
 configs/mt_ventoux_defconfig     | 1 +
 include/configs/apf27.h          | 1 -
 include/configs/astro_mcf5373l.h | 1 -
 include/configs/mt_ventoux.h     | 2 --
 6 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/configs/apf27_defconfig b/configs/apf27_defconfig
index f8f177ad56ce..4e0810a2064f 100644
--- a/configs/apf27_defconfig
+++ b/configs/apf27_defconfig
@@ -23,4 +23,5 @@ CONFIG_CMD_FAT=y
 CONFIG_CMD_UBI=y
 CONFIG_FPGA_XILINX=y
 CONFIG_FPGA_SPARTAN3=y
+CONFIG_FPGA_PROG_FEEDBACK=y
 CONFIG_OF_LIBFDT=y
diff --git a/configs/astro_mcf5373l_defconfig b/configs/astro_mcf5373l_defconfig
index 5ea1b66e5b2a..188a2ae6d0b7 100644
--- a/configs/astro_mcf5373l_defconfig
+++ b/configs/astro_mcf5373l_defconfig
@@ -11,3 +11,4 @@ CONFIG_CMD_I2C=y
 CONFIG_CMD_CACHE=y
 CONFIG_FPGA_XILINX=y
 CONFIG_FPGA_SPARTAN3=y
+CONFIG_FPGA_PROG_FEEDBACK=y
diff --git a/configs/mt_ventoux_defconfig b/configs/mt_ventoux_defconfig
index d4e58395cf22..10e5ebc20945 100644
--- a/configs/mt_ventoux_defconfig
+++ b/configs/mt_ventoux_defconfig
@@ -26,6 +26,7 @@ CONFIG_CMD_FAT=y
 CONFIG_CMD_UBI=y
 CONFIG_FPGA_XILINX=y
 CONFIG_FPGA_SPARTAN3=y
+CONFIG_FPGA_PROG_FEEDBACK=y
 CONFIG_SYS_NS16550=y
 CONFIG_USB=y
 CONFIG_USB_ULPI_VIEWPORT_OMAP=y
diff --git a/include/configs/apf27.h b/include/configs/apf27.h
index 00798fa1027f..df0dde0cdcd2 100644
--- a/include/configs/apf27.h
+++ b/include/configs/apf27.h
@@ -262,7 +262,6 @@
  */
 #define CONFIG_FPGA_COUNT		1
 #define CONFIG_SYS_FPGA_WAIT		250 /* 250 ms */
-#define CONFIG_FPGA_PROG_FEEDBACK
 #define CONFIG_SYS_FPGA_CHECK_CTRLC
 #define CONFIG_SYS_FPGA_CHECK_ERROR
 
diff --git a/include/configs/astro_mcf5373l.h b/include/configs/astro_mcf5373l.h
index d5d6540ec9b9..e5eaec238af7 100644
--- a/include/configs/astro_mcf5373l.h
+++ b/include/configs/astro_mcf5373l.h
@@ -206,7 +206,6 @@
 #define CONFIG_FPGA_COUNT	1
 #define CONFIG_FPGA_ALTERA
 #define CONFIG_FPGA_CYCLON2
-#define CONFIG_FPGA_PROG_FEEDBACK
 #define CONFIG_SYS_FPGA_WAIT		1000
 
 /* End of user parameters to be customized */
diff --git a/include/configs/mt_ventoux.h b/include/configs/mt_ventoux.h
index b6198dbdf6d3..97fe785ef957 100644
--- a/include/configs/mt_ventoux.h
+++ b/include/configs/mt_ventoux.h
@@ -42,11 +42,9 @@
  * FPGA
  */
 #define CONFIG_CMD_FPGA_LOADMK
-#define CONFIG_FPGA_PROG_FEEDBACK
 #define CONFIG_SYS_FPGA_WAIT	10000
 #define CONFIG_MAX_FPGA_DEVICES	1
 #define CONFIG_FPGA_DELAY() udelay(1)
-#define CONFIG_FPGA_PROG_FEEDBACK
 
 #define CONFIG_SPLASH_SCREEN
 #define CONFIG_VIDEO_BMP_RLE8
-- 
2.10.2

[U-Boot] [RFC PATCH 6/7] Move FPGA_PROG_FEEDBACK to defconfig

[Michal Simek]

Please put some description here.

Thanks,
Michal

On 22.11.2016 09:49, Chris Packham wrote:
> Signed-off-by: Chris Packham <judge.packham@gmail.com>
> ---
> 
>  configs/apf27_defconfig          | 1 +
>  configs/astro_mcf5373l_defconfig | 1 +
>  configs/mt_ventoux_defconfig     | 1 +
>  include/configs/apf27.h          | 1 -
>  include/configs/astro_mcf5373l.h | 1 -
>  include/configs/mt_ventoux.h     | 2 --
>  6 files changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/configs/apf27_defconfig b/configs/apf27_defconfig
> index f8f177ad56ce..4e0810a2064f 100644
> --- a/configs/apf27_defconfig
> +++ b/configs/apf27_defconfig
> @@ -23,4 +23,5 @@ CONFIG_CMD_FAT=y
>  CONFIG_CMD_UBI=y
>  CONFIG_FPGA_XILINX=y
>  CONFIG_FPGA_SPARTAN3=y
> +CONFIG_FPGA_PROG_FEEDBACK=y
>  CONFIG_OF_LIBFDT=y
> diff --git a/configs/astro_mcf5373l_defconfig b/configs/astro_mcf5373l_defconfig
> index 5ea1b66e5b2a..188a2ae6d0b7 100644
> --- a/configs/astro_mcf5373l_defconfig
> +++ b/configs/astro_mcf5373l_defconfig
> @@ -11,3 +11,4 @@ CONFIG_CMD_I2C=y
>  CONFIG_CMD_CACHE=y
>  CONFIG_FPGA_XILINX=y
>  CONFIG_FPGA_SPARTAN3=y
> +CONFIG_FPGA_PROG_FEEDBACK=y
> diff --git a/configs/mt_ventoux_defconfig b/configs/mt_ventoux_defconfig
> index d4e58395cf22..10e5ebc20945 100644
> --- a/configs/mt_ventoux_defconfig
> +++ b/configs/mt_ventoux_defconfig
> @@ -26,6 +26,7 @@ CONFIG_CMD_FAT=y
>  CONFIG_CMD_UBI=y
>  CONFIG_FPGA_XILINX=y
>  CONFIG_FPGA_SPARTAN3=y
> +CONFIG_FPGA_PROG_FEEDBACK=y
>  CONFIG_SYS_NS16550=y
>  CONFIG_USB=y
>  CONFIG_USB_ULPI_VIEWPORT_OMAP=y
> diff --git a/include/configs/apf27.h b/include/configs/apf27.h
> index 00798fa1027f..df0dde0cdcd2 100644
> --- a/include/configs/apf27.h
> +++ b/include/configs/apf27.h
> @@ -262,7 +262,6 @@
>   */
>  #define CONFIG_FPGA_COUNT		1
>  #define CONFIG_SYS_FPGA_WAIT		250 /* 250 ms */
> -#define CONFIG_FPGA_PROG_FEEDBACK
>  #define CONFIG_SYS_FPGA_CHECK_CTRLC
>  #define CONFIG_SYS_FPGA_CHECK_ERROR
>  
> diff --git a/include/configs/astro_mcf5373l.h b/include/configs/astro_mcf5373l.h
> index d5d6540ec9b9..e5eaec238af7 100644
> --- a/include/configs/astro_mcf5373l.h
> +++ b/include/configs/astro_mcf5373l.h
> @@ -206,7 +206,6 @@
>  #define CONFIG_FPGA_COUNT	1
>  #define CONFIG_FPGA_ALTERA
>  #define CONFIG_FPGA_CYCLON2
> -#define CONFIG_FPGA_PROG_FEEDBACK
>  #define CONFIG_SYS_FPGA_WAIT		1000
>  
>  /* End of user parameters to be customized */
> diff --git a/include/configs/mt_ventoux.h b/include/configs/mt_ventoux.h
> index b6198dbdf6d3..97fe785ef957 100644
> --- a/include/configs/mt_ventoux.h
> +++ b/include/configs/mt_ventoux.h
> @@ -42,11 +42,9 @@
>   * FPGA
>   */
>  #define CONFIG_CMD_FPGA_LOADMK
> -#define CONFIG_FPGA_PROG_FEEDBACK
>  #define CONFIG_SYS_FPGA_WAIT	10000
>  #define CONFIG_MAX_FPGA_DEVICES	1
>  #define CONFIG_FPGA_DELAY() udelay(1)
> -#define CONFIG_FPGA_PROG_FEEDBACK
>  
>  #define CONFIG_SPLASH_SCREEN
>  #define CONFIG_VIDEO_BMP_RLE8
> 


-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP SoCs


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161123/b1ef637f/attachment.sig>

[U-Boot] [RFC PATCH 7/7] fpga/spartan3: add reverse bitstream mode

[Chris Packham]

The Xilinx tools we use appear to give us a bitstream with the bits
reversed (compared to what the existing code expects) add a new
CONFIG_FPGA_REVERSE_BITSTREAM option which tells the spartan3 driver
to output the serial stream with the leftmost bit first.

Signed-off-by: Chris Packham <judge.packham@gmail.com>
---

 drivers/fpga/Kconfig    | 7 +++++++
 drivers/fpga/spartan3.c | 8 ++++++++
 2 files changed, 15 insertions(+)

diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig
index 1b3b03c8f938..26b7c864d275 100644
--- a/drivers/fpga/Kconfig
+++ b/drivers/fpga/Kconfig
@@ -29,6 +29,13 @@ config FPGA_SPARTAN2
 	help
 	  Enable FPGA driver for Xilinx Spartan-II devices.
 
+config FPGA_REVERSE_BITSTREAM
+	bool "Reverse bitstream when programming"
+	depends on FPGA_SPARTAN3
+	help
+	  Some Xilinx tools generate programs that need to be uploaded LSb
+	  first. Set this option if your fpga binary requires this.
+
 config FPGA_PROG_FEEDBACK
 	bool "FPGA programming feedback"
 	help
diff --git a/drivers/fpga/spartan3.c b/drivers/fpga/spartan3.c
index 34aa097ab6c2..2a57c8ab77b5 100644
--- a/drivers/fpga/spartan3.c
+++ b/drivers/fpga/spartan3.c
@@ -392,12 +392,20 @@ static int spartan3_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
 					(*fn->clk) (false, true, cookie);
 					CONFIG_FPGA_DELAY ();
 					/* Write data */
+#ifdef CONFIG_FPGA_REVERSE_BITSTREAM
+					(*fn->wr) ((val & 0x01), true, cookie);
+#else
 					(*fn->wr) ((val & 0x80), true, cookie);
+#endif
 					CONFIG_FPGA_DELAY ();
 					/* Assert the clock */
 					(*fn->clk) (true, true, cookie);
 					CONFIG_FPGA_DELAY ();
+#ifdef CONFIG_FPGA_REVERSE_BITSTREAM
+					val >>= 1;
+#else
 					val <<= 1;
+#endif
 					i --;
 				} while (i > 0);
 
-- 
2.10.2

[U-Boot] [RFC PATCH 7/7] fpga/spartan3: add reverse bitstream mode

[Michal Simek]

On 22.11.2016 09:49, Chris Packham wrote:
> The Xilinx tools we use appear to give us a bitstream with the bits
> reversed (compared to what the existing code expects) add a new
> CONFIG_FPGA_REVERSE_BITSTREAM option which tells the spartan3 driver
> to output the serial stream with the leftmost bit first.
> 
> Signed-off-by: Chris Packham <judge.packham@gmail.com>
> ---
> 
>  drivers/fpga/Kconfig    | 7 +++++++
>  drivers/fpga/spartan3.c | 8 ++++++++
>  2 files changed, 15 insertions(+)
> 
> diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig
> index 1b3b03c8f938..26b7c864d275 100644
> --- a/drivers/fpga/Kconfig
> +++ b/drivers/fpga/Kconfig
> @@ -29,6 +29,13 @@ config FPGA_SPARTAN2
>  	help
>  	  Enable FPGA driver for Xilinx Spartan-II devices.
>  
> +config FPGA_REVERSE_BITSTREAM
> +	bool "Reverse bitstream when programming"
> +	depends on FPGA_SPARTAN3
> +	help
> +	  Some Xilinx tools generate programs that need to be uploaded LSb
> +	  first. Set this option if your fpga binary requires this.

I am not aware about that option because this is pretty ancient.
Isn't there just a flag somewhere?

> +
>  config FPGA_PROG_FEEDBACK
>  	bool "FPGA programming feedback"
>  	help
> diff --git a/drivers/fpga/spartan3.c b/drivers/fpga/spartan3.c
> index 34aa097ab6c2..2a57c8ab77b5 100644
> --- a/drivers/fpga/spartan3.c
> +++ b/drivers/fpga/spartan3.c
> @@ -392,12 +392,20 @@ static int spartan3_ss_load(xilinx_desc *desc, const void *buf, size_t bsize)
>  					(*fn->clk) (false, true, cookie);
>  					CONFIG_FPGA_DELAY ();
>  					/* Write data */
> +#ifdef CONFIG_FPGA_REVERSE_BITSTREAM
> +					(*fn->wr) ((val & 0x01), true, cookie);
> +#else
>  					(*fn->wr) ((val & 0x80), true, cookie);
> +#endif
>  					CONFIG_FPGA_DELAY ();
>  					/* Assert the clock */
>  					(*fn->clk) (true, true, cookie);
>  					CONFIG_FPGA_DELAY ();
> +#ifdef CONFIG_FPGA_REVERSE_BITSTREAM
> +					val >>= 1;
> +#else
>  					val <<= 1;
> +#endif
>  					i --;
>  				} while (i > 0);
>  
> 

Acked-by: Michal Simek <michal.simek@xilinx.com>

Thanks,
Michal


-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP SoCs


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161123/21e24f42/attachment.sig>