[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 at 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 at 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>