[PATCH 2/3] fpga: Remove ancient ACEX1K support
Alexander Dahl
ada at thorsis.com
Fri Jul 25 14:10:23 CEST 2025
Hello Michal,
Am Fri, Jul 25, 2025 at 01:25:53PM +0200 schrieb Michal Simek:
> Coverity (CID 583149) reports issue on code which is not enabled by any
> real platform that's why remove it completely.
>
> Signed-off-by: Michal Simek <michal.simek at amd.com>
> ---
>
> configs/sandbox_defconfig | 1 -
> drivers/fpga/ACEX1K.c | 245 --------------------------------------
> drivers/fpga/Kconfig | 6 -
> drivers/fpga/Makefile | 1 -
> drivers/fpga/altera.c | 5 +-
> include/ACEX1K.h | 49 --------
> 6 files changed, 1 insertion(+), 306 deletions(-)
> delete mode 100644 drivers/fpga/ACEX1K.c
>
> diff --git a/configs/sandbox_defconfig b/configs/sandbox_defconfig
> index 0c030f4a7925..96546cc0bcfe 100644
> --- a/configs/sandbox_defconfig
> +++ b/configs/sandbox_defconfig
> @@ -206,7 +206,6 @@ CONFIG_ARM_FFA_TRANSPORT=y
> CONFIG_FPGA_ALTERA=y
> CONFIG_FPGA_STRATIX_II=y
> CONFIG_FPGA_STRATIX_V=y
> -CONFIG_FPGA_ACEX1K=y
> CONFIG_FPGA_CYCLON2=y
> CONFIG_FPGA_LATTICE=y
> CONFIG_FPGA_XILINX=y
> diff --git a/drivers/fpga/ACEX1K.c b/drivers/fpga/ACEX1K.c
> deleted file mode 100644
> index e1514fc56d04..000000000000
> --- a/drivers/fpga/ACEX1K.c
> +++ /dev/null
> @@ -1,245 +0,0 @@
> -// SPDX-License-Identifier: GPL-2.0+
> -/*
> - * (C) Copyright 2003
> - * Steven Scholz, imc Measurement & Control, steven.scholz at imc-berlin.de
> - *
> - * (C) Copyright 2002
> - * Rich Ireland, Enterasys Networks, rireland at enterasys.com.
> - */
> -
> -#define LOG_CATEGORY UCLASS_FPGA
> -
> -#include <config.h> /* core U-Boot definitions */
> -#include <console.h>
> -#include <log.h>
> -#include <ACEX1K.h> /* ACEX device family */
> -#include <linux/delay.h>
> -#include <time.h>
> -
> -/* Note: The assumption is that we cannot possibly run fast enough to
> - * overrun the device (the Slave Parallel mode can free run at 50MHz).
> - * If there is a need to operate slower, define CFG_FPGA_DELAY in
> - * the board config file to slow things down.
> - */
> -#ifndef CFG_FPGA_DELAY
> -#define CFG_FPGA_DELAY()
> -#endif
> -
> -#ifndef CFG_SYS_FPGA_WAIT
> -#define CFG_SYS_FPGA_WAIT CONFIG_SYS_HZ/10 /* 100 ms */
> -#endif
> -
> -static int ACEX1K_ps_load(Altera_desc *desc, const void *buf, size_t bsize);
> -static int ACEX1K_ps_dump(Altera_desc *desc, const void *buf, size_t bsize);
> -/* static int ACEX1K_ps_info(Altera_desc *desc); */
> -
> -/* ------------------------------------------------------------------------- */
> -/* ACEX1K Generic Implementation */
> -int ACEX1K_load(Altera_desc *desc, const void *buf, size_t bsize)
> -{
> - int ret_val = FPGA_FAIL;
> -
> - switch (desc->iface) {
> - case passive_serial:
> - log_debug("Launching Passive Serial Loader\n");
> - ret_val = ACEX1K_ps_load (desc, buf, bsize);
> - break;
> -
> - /* Add new interface types here */
> -
> - default:
> - printf ("%s: Unsupported interface type, %d\n",
> - __FUNCTION__, desc->iface);
> - }
> -
> - return ret_val;
> -}
> -
> -int ACEX1K_dump(Altera_desc *desc, const void *buf, size_t bsize)
> -{
> - int ret_val = FPGA_FAIL;
> -
> - switch (desc->iface) {
> - case passive_serial:
> - log_debug("Launching Passive Serial Dump\n");
> - ret_val = ACEX1K_ps_dump (desc, buf, bsize);
> - break;
> -
> - /* Add new interface types here */
> -
> - default:
> - printf ("%s: Unsupported interface type, %d\n",
> - __FUNCTION__, desc->iface);
> - }
> -
> - return ret_val;
> -}
> -
> -int ACEX1K_info( Altera_desc *desc )
> -{
> - return FPGA_SUCCESS;
> -}
> -
> -/* ------------------------------------------------------------------------- */
> -/* ACEX1K Passive Serial Generic Implementation */
> -
> -static int ACEX1K_ps_load(Altera_desc *desc, const void *buf, size_t bsize)
> -{
> - int ret_val = FPGA_FAIL; /* assume the worst */
> - Altera_ACEX1K_Passive_Serial_fns *fn = desc->iface_fns;
> - int i;
> -
> - log_debug("start with interface functions @ 0x%p\n", fn);
> -
> - if (fn) {
> - size_t bytecount = 0;
> - unsigned char *data = (unsigned char *) buf;
> - int cookie = desc->cookie; /* make a local copy */
> - unsigned long ts; /* timestamp */
> -
> - log_debug("Function Table:\n"
> - "ptr:\t0x%p\n"
> - "struct: 0x%p\n"
> - "config:\t0x%p\n"
> - "status:\t0x%p\n"
> - "clk:\t0x%p\n"
> - "data:\t0x%p\n"
> - "done:\t0x%p\n\n",
> - &fn, fn, fn->config, fn->status,
> - fn->clk, fn->data, fn->done);
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> - printf ("Loading FPGA Device %d...", cookie);
> -#endif
> -
> - /*
> - * Run the pre configuration function if there is one.
> - */
> - if (*fn->pre) {
> - (*fn->pre) (cookie);
> - }
> -
> - /* Establish the initial state */
> - (*fn->config) (true, true, cookie); /* Assert nCONFIG */
> -
> - udelay(2); /* T_cfg > 2us */
> -
> - /* nSTATUS should be asserted now */
> - (*fn->done) (cookie);
> - if ( !(*fn->status) (cookie) ) {
> - puts ("** nSTATUS is not asserted.\n");
> - (*fn->abort) (cookie);
> - return FPGA_FAIL;
> - }
> -
> - (*fn->config) (false, true, cookie); /* Deassert nCONFIG */
> - udelay(2); /* T_cf2st1 < 4us */
> -
> - /* Wait for nSTATUS to be released (i.e. deasserted) */
> - ts = get_timer (0); /* get current time */
> - do {
> - CFG_FPGA_DELAY ();
> - if (get_timer (ts) > CFG_SYS_FPGA_WAIT) { /* check the time */
> - puts ("** Timeout waiting for STATUS to go high.\n");
> - (*fn->abort) (cookie);
> - return FPGA_FAIL;
> - }
> - (*fn->done) (cookie);
> - } while ((*fn->status) (cookie));
> -
> - /* Get ready for the burn */
> - CFG_FPGA_DELAY ();
> -
> - /* Load the data */
> - while (bytecount < bsize) {
> - unsigned char val=0;
> -#ifdef CONFIG_SYS_FPGA_CHECK_CTRLC
> - if (ctrlc ()) {
> - (*fn->abort) (cookie);
> - return FPGA_FAIL;
> - }
> -#endif
> - /* Altera detects an error if INIT goes low (active)
> - while DONE is low (inactive) */
> -#if 0 /* not yet implemented */
> - if ((*fn->done) (cookie) == 0 && (*fn->init) (cookie)) {
> - puts ("** CRC error during FPGA load.\n");
> - (*fn->abort) (cookie);
> - return (FPGA_FAIL);
> - }
> -#endif
> - val = data [bytecount ++ ];
> - i = 8;
> - do {
> - /* Deassert the clock */
> - (*fn->clk) (false, true, cookie);
> - CFG_FPGA_DELAY ();
> - /* Write data */
> - (*fn->data) ((val & 0x01), true, cookie);
> - CFG_FPGA_DELAY ();
> - /* Assert the clock */
> - (*fn->clk) (true, true, cookie);
> - CFG_FPGA_DELAY ();
> - val >>= 1;
> - i --;
> - } while (i > 0);
> -
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> - if (bytecount % (bsize / 40) == 0)
> - putc ('.'); /* let them know we are alive */
> -#endif
> - }
> -
> - CFG_FPGA_DELAY ();
> -
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> - putc (' '); /* terminate the dotted line */
> -#endif
> -
> - /*
> - * Checking FPGA's CONF_DONE signal - correctly booted ?
> - */
> -
> - if ( ! (*fn->done) (cookie) ) {
> - puts ("** Booting failed! CONF_DONE is still deasserted.\n");
> - (*fn->abort) (cookie);
> - return (FPGA_FAIL);
> - }
> -
> - /*
> - * "DCLK must be clocked an additional 10 times fpr ACEX 1K..."
> - */
> -
> - for (i = 0; i < 12; i++) {
> - CFG_FPGA_DELAY ();
> - (*fn->clk) (true, true, cookie); /* Assert the clock pin */
> - CFG_FPGA_DELAY ();
> - (*fn->clk) (false, true, cookie); /* Deassert the clock pin */
> - }
> -
> - ret_val = FPGA_SUCCESS;
> -
> -#ifdef CONFIG_SYS_FPGA_PROG_FEEDBACK
> - if (ret_val == FPGA_SUCCESS) {
> - puts ("Done.\n");
> - }
> - else {
> - puts ("Fail.\n");
> - }
> -#endif
> - (*fn->post) (cookie);
> -
> - } else {
> - printf ("%s: NULL Interface function table!\n", __FUNCTION__);
> - }
> -
> - return ret_val;
> -}
> -
> -static int ACEX1K_ps_dump(Altera_desc *desc, const void *buf, size_t bsize)
> -{
> - /* Readback is only available through the Slave Parallel and */
> - /* boundary-scan interfaces. */
> - printf ("%s: Passive Serial Dumping is unavailable\n",
> - __FUNCTION__);
> - return FPGA_FAIL;
> -}
> diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig
> index 9456ca3149ae..18e7d7c99a9a 100644
> --- a/drivers/fpga/Kconfig
> +++ b/drivers/fpga/Kconfig
> @@ -34,12 +34,6 @@ config FPGA_STRATIX_V
> help
> Say Y here to enable the Altera Stratix V FPGA specific driver.
>
> -config FPGA_ACEX1K
> - bool "Enable Altera ACEX 1K driver"
> - depends on FPGA_ALTERA
> - help
> - Say Y here to enable the Altera ACEX 1K FPGA specific driver.
> -
> config FPGA_CYCLON2
> bool "Enable Altera FPGA driver for Cyclone II"
> depends on FPGA_ALTERA
> diff --git a/drivers/fpga/Makefile b/drivers/fpga/Makefile
> index 610c168fc35c..6c779d5b5df6 100644
> --- a/drivers/fpga/Makefile
> +++ b/drivers/fpga/Makefile
> @@ -17,7 +17,6 @@ obj-$(CONFIG_FPGA_XILINX) += xilinx.o
> obj-$(CONFIG_FPGA_LATTICE) += ivm_core.o lattice.o
> ifdef CONFIG_FPGA_ALTERA
> obj-y += altera.o
> -obj-$(CONFIG_FPGA_ACEX1K) += ACEX1K.o
> obj-$(CONFIG_FPGA_CYCLON2) += cyclon2.o
> obj-$(CONFIG_FPGA_INTEL_SDM_MAILBOX) += intel_sdm_mb.o
> obj-$(CONFIG_FPGA_STRATIX_II) += stratixII.o
> diff --git a/drivers/fpga/altera.c b/drivers/fpga/altera.c
> index 64fda3a307c7..4a9aa74357ec 100644
> --- a/drivers/fpga/altera.c
> +++ b/drivers/fpga/altera.c
> @@ -28,10 +28,7 @@ static const struct altera_fpga {
> int (*dump)(Altera_desc *, const void *, size_t);
> int (*info)(Altera_desc *);
> } altera_fpga[] = {
> -#if defined(CONFIG_FPGA_ACEX1K)
> - { Altera_ACEX1K, "ACEX1K", ACEX1K_load, ACEX1K_dump, ACEX1K_info },
> - { Altera_CYC2, "ACEX1K", ACEX1K_load, ACEX1K_dump, ACEX1K_info },
> -#elif defined(CONFIG_FPGA_CYCLON2)
> +#if defined(CONFIG_FPGA_CYCLON2)
> { Altera_ACEX1K, "CycloneII", CYC2_load, CYC2_dump, CYC2_info },
> { Altera_CYC2, "CycloneII", CYC2_load, CYC2_dump, CYC2_info },
> #endif
> diff --git a/include/ACEX1K.h b/include/ACEX1K.h
> index 7c5253c66cca..44d1b9765657 100644
> --- a/include/ACEX1K.h
> +++ b/include/ACEX1K.h
> @@ -12,26 +12,10 @@
>
> #include <altera.h>
>
> -extern int ACEX1K_load(Altera_desc *desc, const void *image, size_t size);
> -extern int ACEX1K_dump(Altera_desc *desc, const void *buf, size_t bsize);
> -extern int ACEX1K_info(Altera_desc *desc);
> -
> extern int CYC2_load(Altera_desc *desc, const void *image, size_t size);
> extern int CYC2_dump(Altera_desc *desc, const void *buf, size_t bsize);
> extern int CYC2_info(Altera_desc *desc);
Maybe CYC2_load() etc. can be moved to a different header file like
altera.h for example?
>
> -/* Slave Serial Implementation function table */
> -typedef struct {
> - Altera_pre_fn pre;
> - Altera_config_fn config;
> - Altera_clk_fn clk;
> - Altera_status_fn status;
> - Altera_done_fn done;
> - Altera_data_fn data;
> - Altera_abort_fn abort;
> - Altera_post_fn post;
> -} Altera_ACEX1K_Passive_Serial_fns;
> -
> /* Slave Serial Implementation function table */
> typedef struct {
> Altera_pre_fn pre;
> @@ -43,37 +27,4 @@ typedef struct {
> Altera_post_fn post;
> } Altera_CYC2_Passive_Serial_fns;
>
> -/* Device Image Sizes
> - *********************************************************************/
> -/* ACEX1K */
> -/* FIXME: Which size do we mean?
> - * Datasheet says 1337000/8=167125Bytes,
> - * Filesize of an *.rbf file is 166965 Bytes
> - */
> -#if 0
> -#define Altera_EP1K100_SIZE 1337000/8 /* 167125 Bytes */
> -#endif
> -#define Altera_EP1K100_SIZE (166965*8)
> -
> -#define Altera_EP2C8_SIZE 247942
> -#define Altera_EP2C20_SIZE 586562
> -#define Altera_EP2C35_SIZE 883905
> -#define Altera_EP3C5_SIZE 368011 /* .rbf size in bytes */
> -
> -#define ALTERA_EP4CE6_SIZE 368011 /* 2944088 Bits */
> -#define ALTERA_EP4CE10_SIZE 368011 /* 2944088 Bits */
> -#define ALTERA_EP4CE15_SIZE 510856 /* 4086848 Bits */
> -#define ALTERA_EP4CE22_SIZE 718569 /* 5748552 Bits */
> -#define ALTERA_EP4CE30_SIZE 1191788 /* 9534304 Bits */
> -#define ALTERA_EP4CE40_SIZE 1191788 /* 9534304 Bits */
> -#define ALTERA_EP4CE55_SIZE 1861195 /* 14889560 Bits */
> -#define ALTERA_EP4CE75_SIZE 2495719 /* 19965752 Bits */
> -#define ALTERA_EP4CE115_SIZE 3571462 /* 28571696 Bits */
Can we move these definitions for device image sizes to altera.h
instead removing them? I added some of those with commit 3b2a595fc659
("fpga: altera: Add some more device sizes") back in 2019, and I'm
using some of them in downstream code/dts.
Greets
Alex
> -
> -/* Descriptor Macros
> - *********************************************************************/
> -/* ACEX1K devices */
> -#define Altera_EP1K100_DESC(iface, fn_table, cookie) \
> -{ Altera_ACEX1K, iface, Altera_EP1K100_SIZE, fn_table, cookie }
> -
> #endif /* _ACEX1K_H_ */
> --
> 2.43.0
>
More information about the U-Boot
mailing list