* [U-Boot] [PATCH v6 0/4] Stratix10 FPGA reconfiguration support
@ 2018-12-18 8:54 chee.hong.ang at intel.com
2018-12-18 8:54 ` [U-Boot] [PATCH v6 1/4] arm: socfpga: stratix10: Add macros for mailbox's arguments chee.hong.ang at intel.com
` (4 more replies)
0 siblings, 5 replies; 13+ messages in thread
From: chee.hong.ang at intel.com @ 2018-12-18 8:54 UTC (permalink / raw)
To: u-boot
From: "Ang, Chee Hong" <chee.hong.ang@intel.com>
Summary of v6 changes:
- Patch 1/4 and 4/4 are unchanged
- Patch 2/4:
- fixed compilation warnings in drivers/fpga/stratix10.c
- Patch 3/4:
- socfpga_fpga_add() in misc.c
- define fpga descriptor structure in misc_arria10.c, misc_gen5.c &
misc_s10.c respectively
- removed for-loop in socfpga_fpga_add() (only 1 FPGA device added)
v5 patchsets:
https://lists.denx.de/pipermail/u-boot/2018-November/349670.html
Ang, Chee Hong (4):
arm: socfpga: stratix10: Add macros for mailbox's arguments
arm: socfpga: stratix10: Add Stratix 10 FPGA Reconfiguration Driver
arm: socfpga: stratix10: Add Stratix10 FPGA into FPGA device table
arm: socfpga: stratix10: Enable Stratix10 FPGA Reconfiguration
arch/arm/mach-socfpga/include/mach/mailbox_s10.h | 6 +
arch/arm/mach-socfpga/include/mach/misc.h | 4 +-
arch/arm/mach-socfpga/misc.c | 26 +-
arch/arm/mach-socfpga/misc_arria10.c | 22 +-
arch/arm/mach-socfpga/misc_gen5.c | 22 +-
arch/arm/mach-socfpga/misc_s10.c | 22 ++
configs/socfpga_stratix10_defconfig | 1 +
drivers/fpga/Kconfig | 11 +
drivers/fpga/Makefile | 1 +
drivers/fpga/altera.c | 6 +
drivers/fpga/stratix10.c | 288 +++++++++++++++++++++++
include/altera.h | 8 +
12 files changed, 389 insertions(+), 28 deletions(-)
create mode 100644 drivers/fpga/stratix10.c
--
2.7.4
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] [PATCH v6 1/4] arm: socfpga: stratix10: Add macros for mailbox's arguments
2018-12-18 8:54 [U-Boot] [PATCH v6 0/4] Stratix10 FPGA reconfiguration support chee.hong.ang at intel.com
@ 2018-12-18 8:54 ` chee.hong.ang at intel.com
2018-12-18 8:54 ` [U-Boot] [PATCH v6 2/4] arm: socfpga: stratix10: Add Stratix 10 FPGA Reconfiguration Driver chee.hong.ang at intel.com
` (3 subsequent siblings)
4 siblings, 0 replies; 13+ messages in thread
From: chee.hong.ang at intel.com @ 2018-12-18 8:54 UTC (permalink / raw)
To: u-boot
From: "Ang, Chee Hong" <chee.hong.ang@intel.com>
Add macros for specifying number of arguments in mailbox command.
Signed-off-by: Ang, Chee Hong <chee.hong.ang@intel.com>
---
arch/arm/mach-socfpga/include/mach/mailbox_s10.h | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/arch/arm/mach-socfpga/include/mach/mailbox_s10.h b/arch/arm/mach-socfpga/include/mach/mailbox_s10.h
index 660df35..ae728a5 100644
--- a/arch/arm/mach-socfpga/include/mach/mailbox_s10.h
+++ b/arch/arm/mach-socfpga/include/mach/mailbox_s10.h
@@ -107,6 +107,12 @@ enum ALT_SDM_MBOX_RESP_CODE {
#define RECONFIG_STATUS_PIN_STATUS 2
#define RECONFIG_STATUS_SOFTFUNC_STATUS 3
+/* Macros for specifying number of arguments in mailbox command */
+#define MBOX_NUM_ARGS(n, b) (((n) & 0xFF) << (b))
+#define MBOX_DIRECT_COUNT(n) MBOX_NUM_ARGS((n), 0)
+#define MBOX_ARG_DESC_COUNT(n) MBOX_NUM_ARGS((n), 8)
+#define MBOX_RESP_DESC_COUNT(n) MBOX_NUM_ARGS((n), 16)
+
#define MBOX_CFGSTAT_STATE_IDLE 0x00000000
#define MBOX_CFGSTAT_STATE_CONFIG 0x10000000
#define MBOX_CFGSTAT_STATE_FAILACK 0x08000000
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [U-Boot] [PATCH v6 2/4] arm: socfpga: stratix10: Add Stratix 10 FPGA Reconfiguration Driver
2018-12-18 8:54 [U-Boot] [PATCH v6 0/4] Stratix10 FPGA reconfiguration support chee.hong.ang at intel.com
2018-12-18 8:54 ` [U-Boot] [PATCH v6 1/4] arm: socfpga: stratix10: Add macros for mailbox's arguments chee.hong.ang at intel.com
@ 2018-12-18 8:54 ` chee.hong.ang at intel.com
2018-12-18 8:54 ` [U-Boot] [PATCH v6 3/4] arm: socfpga: stratix10: Add Stratix10 FPGA into FPGA device table chee.hong.ang at intel.com
` (2 subsequent siblings)
4 siblings, 0 replies; 13+ messages in thread
From: chee.hong.ang at intel.com @ 2018-12-18 8:54 UTC (permalink / raw)
To: u-boot
From: "Ang, Chee Hong" <chee.hong.ang@intel.com>
Enable FPGA reconfiguration support for Stratix 10 SoC.
Signed-off-by: Ang, Chee Hong <chee.hong.ang@intel.com>
---
drivers/fpga/Kconfig | 11 ++
drivers/fpga/Makefile | 1 +
drivers/fpga/stratix10.c | 288 +++++++++++++++++++++++++++++++++++++++++++++++
include/altera.h | 4 +
4 files changed, 304 insertions(+)
create mode 100644 drivers/fpga/stratix10.c
diff --git a/drivers/fpga/Kconfig b/drivers/fpga/Kconfig
index 50e9019..8f59193 100644
--- a/drivers/fpga/Kconfig
+++ b/drivers/fpga/Kconfig
@@ -31,6 +31,17 @@ config FPGA_CYCLON2
Enable FPGA driver for loading bitstream in BIT and BIN format
on Altera Cyclone II device.
+config FPGA_STRATIX10
+ bool "Enable Altera FPGA driver for Stratix 10"
+ depends on TARGET_SOCFPGA_STRATIX10
+ select FPGA_ALTERA
+ help
+ Say Y here to enable the Altera Stratix 10 FPGA specific driver
+
+ This provides common functionality for Altera Stratix 10 devices.
+ Enable FPGA driver for writing bitstream into Altera Stratix10
+ device.
+
config FPGA_XILINX
bool "Enable Xilinx FPGA drivers"
select FPGA
diff --git a/drivers/fpga/Makefile b/drivers/fpga/Makefile
index 97d7d5d..5a778c1 100644
--- a/drivers/fpga/Makefile
+++ b/drivers/fpga/Makefile
@@ -17,6 +17,7 @@ obj-$(CONFIG_FPGA_ACEX1K) += ACEX1K.o
obj-$(CONFIG_FPGA_CYCLON2) += cyclon2.o
obj-$(CONFIG_FPGA_STRATIX_II) += stratixII.o
obj-$(CONFIG_FPGA_STRATIX_V) += stratixv.o
+obj-$(CONFIG_FPGA_STRATIX10) += stratix10.o
obj-$(CONFIG_FPGA_SOCFPGA) += socfpga.o
obj-$(CONFIG_TARGET_SOCFPGA_GEN5) += socfpga_gen5.o
obj-$(CONFIG_TARGET_SOCFPGA_ARRIA10) += socfpga_arria10.o
diff --git a/drivers/fpga/stratix10.c b/drivers/fpga/stratix10.c
new file mode 100644
index 0000000..aae0521
--- /dev/null
+++ b/drivers/fpga/stratix10.c
@@ -0,0 +1,288 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Copyright (C) 2018 Intel Corporation <www.intel.com>
+ */
+
+#include <common.h>
+#include <altera.h>
+#include <asm/arch/mailbox_s10.h>
+
+#define RECONFIG_STATUS_POLL_RESP_TIMEOUT_MS 60000
+#define RECONFIG_STATUS_INTERVAL_DELAY_US 1000000
+
+static const struct mbox_cfgstat_state {
+ int err_no;
+ const char *error_name;
+} mbox_cfgstat_state[] = {
+ {MBOX_CFGSTAT_STATE_IDLE, "FPGA in idle mode."},
+ {MBOX_CFGSTAT_STATE_CONFIG, "FPGA in config mode."},
+ {MBOX_CFGSTAT_STATE_FAILACK, "Acknowledgment failed!"},
+ {MBOX_CFGSTAT_STATE_ERROR_INVALID, "Invalid bitstream!"},
+ {MBOX_CFGSTAT_STATE_ERROR_CORRUPT, "Corrupted bitstream!"},
+ {MBOX_CFGSTAT_STATE_ERROR_AUTH, "Authentication failed!"},
+ {MBOX_CFGSTAT_STATE_ERROR_CORE_IO, "I/O error!"},
+ {MBOX_CFGSTAT_STATE_ERROR_HARDWARE, "Hardware error!"},
+ {MBOX_CFGSTAT_STATE_ERROR_FAKE, "Fake error!"},
+ {MBOX_CFGSTAT_STATE_ERROR_BOOT_INFO, "Error in boot info!"},
+ {MBOX_CFGSTAT_STATE_ERROR_QSPI_ERROR, "Error in QSPI!"},
+ {MBOX_RESP_ERROR, "Mailbox general error!"},
+ {-ETIMEDOUT, "I/O timeout error"},
+ {-1, "Unknown error!"}
+};
+
+#define MBOX_CFGSTAT_MAX ARRAY_SIZE(mbox_cfgstat_state)
+
+static const char *mbox_cfgstat_to_str(int err)
+{
+ int i;
+
+ for (i = 0; i < MBOX_CFGSTAT_MAX - 1; i++) {
+ if (mbox_cfgstat_state[i].err_no == err)
+ return mbox_cfgstat_state[i].error_name;
+ }
+
+ return mbox_cfgstat_state[MBOX_CFGSTAT_MAX - 1].error_name;
+}
+
+/*
+ * Add the ongoing transaction's command ID into pending list and return
+ * the command ID for next transfer.
+ */
+static u8 add_transfer(u32 *xfer_pending_list, size_t list_size, u8 id)
+{
+ int i;
+
+ for (i = 0; i < list_size; i++) {
+ if (xfer_pending_list[i])
+ continue;
+ xfer_pending_list[i] = id;
+ debug("ID(%d) added to transaction pending list\n", id);
+ /*
+ * Increment command ID for next transaction.
+ * Valid command ID (4 bits) is from 1 to 15.
+ */
+ id = (id % 15) + 1;
+ break;
+ }
+
+ return id;
+}
+
+/*
+ * Check whether response ID match the command ID in the transfer
+ * pending list. If a match is found in the transfer pending list,
+ * it clears the transfer pending list and return the matched
+ * command ID.
+ */
+static int get_and_clr_transfer(u32 *xfer_pending_list, size_t list_size,
+ u8 id)
+{
+ int i;
+
+ for (i = 0; i < list_size; i++) {
+ if (id != xfer_pending_list[i])
+ continue;
+ xfer_pending_list[i] = 0;
+ return id;
+ }
+
+ return 0;
+}
+
+/*
+ * Polling the FPGA configuration status.
+ * Return 0 for success, non-zero for error.
+ */
+static int reconfig_status_polling_resp(void)
+{
+ int ret;
+ unsigned long start = get_timer(0);
+
+ while (1) {
+ ret = mbox_get_fpga_config_status(MBOX_RECONFIG_STATUS);
+ if (!ret)
+ return 0; /* configuration success */
+
+ if (ret != MBOX_CFGSTAT_STATE_CONFIG)
+ return ret;
+
+ if (get_timer(start) > RECONFIG_STATUS_POLL_RESP_TIMEOUT_MS)
+ break; /* time out */
+
+ puts(".");
+ udelay(RECONFIG_STATUS_INTERVAL_DELAY_US);
+ }
+
+ return -ETIMEDOUT;
+}
+
+static u32 get_resp_hdr(u32 *r_index, u32 *w_index, u32 *resp_count,
+ u32 *resp_buf, u32 buf_size, u32 client_id)
+{
+ u32 buf[MBOX_RESP_BUFFER_SIZE];
+ u32 mbox_hdr;
+ u32 resp_len;
+ u32 hdr_len;
+ u32 i;
+
+ if (*resp_count < buf_size) {
+ u32 rcv_len_max = buf_size - *resp_count;
+
+ if (rcv_len_max > MBOX_RESP_BUFFER_SIZE)
+ rcv_len_max = MBOX_RESP_BUFFER_SIZE;
+ resp_len = mbox_rcv_resp(buf, rcv_len_max);
+
+ for (i = 0; i < resp_len; i++) {
+ resp_buf[(*w_index)++] = buf[i];
+ *w_index %= buf_size;
+ (*resp_count)++;
+ }
+ }
+
+ /* No response in buffer */
+ if (*resp_count == 0)
+ return 0;
+
+ mbox_hdr = resp_buf[*r_index];
+
+ hdr_len = MBOX_RESP_LEN_GET(mbox_hdr);
+
+ /* Insufficient header length to return a mailbox header */
+ if ((*resp_count - 1) < hdr_len)
+ return 0;
+
+ *r_index += (hdr_len + 1);
+ *r_index %= buf_size;
+ *resp_count -= (hdr_len + 1);
+
+ /* Make sure response belongs to us */
+ if (MBOX_RESP_CLIENT_GET(mbox_hdr) != client_id)
+ return 0;
+
+ return mbox_hdr;
+}
+
+/* Send bit stream data to SDM via RECONFIG_DATA mailbox command */
+static int send_reconfig_data(const void *rbf_data, size_t rbf_size,
+ u32 xfer_max, u32 buf_size_max)
+{
+ u32 response_buffer[MBOX_RESP_BUFFER_SIZE];
+ u32 xfer_pending[MBOX_RESP_BUFFER_SIZE];
+ u32 resp_rindex = 0;
+ u32 resp_windex = 0;
+ u32 resp_count = 0;
+ u32 xfer_count = 0;
+ u8 resp_err = 0;
+ u8 cmd_id = 1;
+ u32 args[3];
+ int ret;
+
+ debug("SDM xfer_max = %d\n", xfer_max);
+ debug("SDM buf_size_max = %x\n\n", buf_size_max);
+
+ memset(xfer_pending, 0, sizeof(xfer_pending));
+
+ while (rbf_size || xfer_count) {
+ if (!resp_err && rbf_size && xfer_count < xfer_max) {
+ args[0] = MBOX_ARG_DESC_COUNT(1);
+ args[1] = (u64)rbf_data;
+ if (rbf_size >= buf_size_max) {
+ args[2] = buf_size_max;
+ rbf_size -= buf_size_max;
+ rbf_data += buf_size_max;
+ } else {
+ args[2] = (u64)rbf_size;
+ rbf_size = 0;
+ }
+
+ ret = mbox_send_cmd_only(cmd_id, MBOX_RECONFIG_DATA,
+ MBOX_CMD_INDIRECT, 3, args);
+ if (ret) {
+ resp_err = 1;
+ } else {
+ xfer_count++;
+ cmd_id = add_transfer(xfer_pending,
+ MBOX_RESP_BUFFER_SIZE,
+ cmd_id);
+ }
+ puts(".");
+ } else {
+ u32 resp_hdr = get_resp_hdr(&resp_rindex, &resp_windex,
+ &resp_count,
+ response_buffer,
+ MBOX_RESP_BUFFER_SIZE,
+ MBOX_CLIENT_ID_UBOOT);
+
+ /*
+ * If no valid response header found or
+ * non-zero length from RECONFIG_DATA
+ */
+ if (!resp_hdr || MBOX_RESP_LEN_GET(resp_hdr))
+ continue;
+
+ /* Check for response's status */
+ if (!resp_err) {
+ ret = MBOX_RESP_ERR_GET(resp_hdr);
+ debug("Response error code: %08x\n", ret);
+ /* Error in response */
+ if (ret)
+ resp_err = 1;
+ }
+
+ ret = get_and_clr_transfer(xfer_pending,
+ MBOX_RESP_BUFFER_SIZE,
+ MBOX_RESP_ID_GET(resp_hdr));
+ if (ret) {
+ /* Claim and reuse the ID */
+ cmd_id = (u8)ret;
+ xfer_count--;
+ }
+
+ if (resp_err && !xfer_count)
+ return ret;
+ }
+ }
+
+ return 0;
+}
+
+/*
+ * This is the interface used by FPGA driver.
+ * Return 0 for success, non-zero for error.
+ */
+int stratix10_load(Altera_desc *desc, const void *rbf_data, size_t rbf_size)
+{
+ int ret;
+ u32 resp_len = 2;
+ u32 resp_buf[2];
+
+ debug("Sending MBOX_RECONFIG...\n");
+ ret = mbox_send_cmd(MBOX_ID_UBOOT, MBOX_RECONFIG, MBOX_CMD_DIRECT, 0,
+ NULL, 0, &resp_len, resp_buf);
+ if (ret) {
+ puts("Failure in RECONFIG mailbox command!\n");
+ return ret;
+ }
+
+ ret = send_reconfig_data(rbf_data, rbf_size, resp_buf[0], resp_buf[1]);
+ if (ret) {
+ printf("RECONFIG_DATA error: %08x, %s\n", ret,
+ mbox_cfgstat_to_str(ret));
+ return ret;
+ }
+
+ /* Make sure we don't send MBOX_RECONFIG_STATUS too fast */
+ udelay(RECONFIG_STATUS_INTERVAL_DELAY_US);
+
+ debug("Polling with MBOX_RECONFIG_STATUS...\n");
+ ret = reconfig_status_polling_resp();
+ if (ret) {
+ printf("RECONFIG_STATUS Error: %08x, %s\n", ret,
+ mbox_cfgstat_to_str(ret));
+ return ret;
+ }
+
+ puts("FPGA reconfiguration OK!\n");
+
+ return ret;
+}
diff --git a/include/altera.h b/include/altera.h
index ead5d3d..233b467 100644
--- a/include/altera.h
+++ b/include/altera.h
@@ -116,4 +116,8 @@ int socfpga_load(Altera_desc *desc, const void *rbf_data, size_t rbf_size);
int stratixv_load(Altera_desc *desc, const void *rbf_data, size_t rbf_size);
#endif
+#ifdef CONFIG_FPGA_STRATIX10
+int stratix10_load(Altera_desc *desc, const void *rbf_data, size_t rbf_size);
+#endif
+
#endif /* _ALTERA_H_ */
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [U-Boot] [PATCH v6 3/4] arm: socfpga: stratix10: Add Stratix10 FPGA into FPGA device table
2018-12-18 8:54 [U-Boot] [PATCH v6 0/4] Stratix10 FPGA reconfiguration support chee.hong.ang at intel.com
2018-12-18 8:54 ` [U-Boot] [PATCH v6 1/4] arm: socfpga: stratix10: Add macros for mailbox's arguments chee.hong.ang at intel.com
2018-12-18 8:54 ` [U-Boot] [PATCH v6 2/4] arm: socfpga: stratix10: Add Stratix 10 FPGA Reconfiguration Driver chee.hong.ang at intel.com
@ 2018-12-18 8:54 ` chee.hong.ang at intel.com
2018-12-18 8:54 ` [U-Boot] [PATCH v6 4/4] arm: socfpga: stratix10: Enable Stratix10 FPGA Reconfiguration chee.hong.ang at intel.com
2018-12-18 17:47 ` [U-Boot] [PATCH v6 0/4] Stratix10 FPGA reconfiguration support Marek Vasut
4 siblings, 0 replies; 13+ messages in thread
From: chee.hong.ang at intel.com @ 2018-12-18 8:54 UTC (permalink / raw)
To: u-boot
From: "Ang, Chee Hong" <chee.hong.ang@intel.com>
Enable 'fpga' command in u-boot. User will be able to use the FPGA
command to program the FPGA on Stratix10 SoC.
Signed-off-by: Ang, Chee Hong <chee.hong.ang@intel.com>
---
arch/arm/mach-socfpga/include/mach/misc.h | 4 ++--
arch/arm/mach-socfpga/misc.c | 26 ++------------------------
arch/arm/mach-socfpga/misc_arria10.c | 22 +++++++++++++++++++++-
arch/arm/mach-socfpga/misc_gen5.c | 22 +++++++++++++++++++++-
arch/arm/mach-socfpga/misc_s10.c | 22 ++++++++++++++++++++++
drivers/fpga/altera.c | 6 ++++++
include/altera.h | 4 ++++
7 files changed, 78 insertions(+), 28 deletions(-)
diff --git a/arch/arm/mach-socfpga/include/mach/misc.h b/arch/arm/mach-socfpga/include/mach/misc.h
index 4fc9570..3964831 100644
--- a/arch/arm/mach-socfpga/include/mach/misc.h
+++ b/arch/arm/mach-socfpga/include/mach/misc.h
@@ -16,9 +16,9 @@ struct bsel {
extern struct bsel bsel_str[];
#ifdef CONFIG_FPGA
-void socfpga_fpga_add(void);
+void socfpga_fpga_add(void *fpga_desc);
#else
-static inline void socfpga_fpga_add(void) {}
+inline void socfpga_fpga_add(void *fpga_desc) {}
#endif
#ifdef CONFIG_TARGET_SOCFPGA_GEN5
diff --git a/arch/arm/mach-socfpga/misc.c b/arch/arm/mach-socfpga/misc.c
index a4f6d5c..78fbe28 100644
--- a/arch/arm/mach-socfpga/misc.c
+++ b/arch/arm/mach-socfpga/misc.c
@@ -88,33 +88,11 @@ int overwrite_console(void)
#endif
#ifdef CONFIG_FPGA
-/*
- * FPGA programming support for SoC FPGA Cyclone V
- */
-static Altera_desc altera_fpga[] = {
- {
- /* Family */
- Altera_SoCFPGA,
- /* Interface type */
- fast_passive_parallel,
- /* No limitation as additional data will be ignored */
- -1,
- /* No device function table */
- NULL,
- /* Base interface address specified in driver */
- NULL,
- /* No cookie implementation */
- 0
- },
-};
-
/* add device descriptor to FPGA device table */
-void socfpga_fpga_add(void)
+void socfpga_fpga_add(void *fpga_desc)
{
- int i;
fpga_init();
- for (i = 0; i < ARRAY_SIZE(altera_fpga); i++)
- fpga_add(fpga_altera, &altera_fpga[i]);
+ fpga_add(fpga_altera, fpga_desc);
}
#endif
diff --git a/arch/arm/mach-socfpga/misc_arria10.c b/arch/arm/mach-socfpga/misc_arria10.c
index f347ae8..430e2b4 100644
--- a/arch/arm/mach-socfpga/misc_arria10.c
+++ b/arch/arm/mach-socfpga/misc_arria10.c
@@ -37,6 +37,26 @@ static const struct socfpga_noc_fw_ocram *noc_fw_ocram_base =
(void *)SOCFPGA_SDR_FIREWALL_OCRAM_ADDRESS;
/*
+ * FPGA programming support for SoC FPGA Arria 10
+ */
+static Altera_desc altera_fpga[] = {
+ {
+ /* Family */
+ Altera_SoCFPGA,
+ /* Interface type */
+ fast_passive_parallel,
+ /* No limitation as additional data will be ignored */
+ -1,
+ /* No device function table */
+ NULL,
+ /* Base interface address specified in driver */
+ NULL,
+ /* No cookie implementation */
+ 0
+ },
+};
+
+/*
+ * This function initializes security policies to be consistent across
+ * all logic units in the Arria 10.
+ *
@@ -73,7 +93,7 @@ void socfpga_sdram_remap_zero(void)
int arch_early_init_r(void)
{
/* Add device descriptor to FPGA device table */
- socfpga_fpga_add();
+ socfpga_fpga_add(&altera_fpga[0]);
return 0;
}
diff --git a/arch/arm/mach-socfpga/misc_gen5.c b/arch/arm/mach-socfpga/misc_gen5.c
index 429c3d6..21f5ac3 100644
--- a/arch/arm/mach-socfpga/misc_gen5.c
+++ b/arch/arm/mach-socfpga/misc_gen5.c
@@ -35,6 +35,26 @@ static struct scu_registers *scu_regs =
(struct scu_registers *)SOCFPGA_MPUSCU_ADDRESS;
/*
+ * FPGA programming support for SoC FPGA Cyclone V
+ */
+static Altera_desc altera_fpga[] = {
+ {
+ /* Family */
+ Altera_SoCFPGA,
+ /* Interface type */
+ fast_passive_parallel,
+ /* No limitation as additional data will be ignored */
+ -1,
+ /* No device function table */
+ NULL,
+ /* Base interface address specified in driver */
+ NULL,
+ /* No cookie implementation */
+ 0
+ },
+};
+
+/*
* DesignWare Ethernet initialization
*/
#ifdef CONFIG_ETH_DESIGNWARE
@@ -214,7 +234,7 @@ int arch_early_init_r(void)
socfpga_sdram_remap_zero();
/* Add device descriptor to FPGA device table */
- socfpga_fpga_add();
+ socfpga_fpga_add(&altera_fpga[0]);
#ifdef CONFIG_DESIGNWARE_SPI
/* Get Designware SPI controller out of reset */
diff --git a/arch/arm/mach-socfpga/misc_s10.c b/arch/arm/mach-socfpga/misc_s10.c
index e599362..113eace 100644
--- a/arch/arm/mach-socfpga/misc_s10.c
+++ b/arch/arm/mach-socfpga/misc_s10.c
@@ -25,6 +25,26 @@ static struct socfpga_system_manager *sysmgr_regs =
(struct socfpga_system_manager *)SOCFPGA_SYSMGR_ADDRESS;
/*
+ * FPGA programming support for SoC FPGA Stratix 10
+ */
+static Altera_desc altera_fpga[] = {
+ {
+ /* Family */
+ Intel_FPGA_Stratix10,
+ /* Interface type */
+ secure_device_manager_mailbox,
+ /* No limitation as additional data will be ignored */
+ -1,
+ /* No device function table */
+ NULL,
+ /* Base interface address specified in driver */
+ NULL,
+ /* No cookie implementation */
+ 0
+ },
+};
+
+/*
* DesignWare Ethernet initialization
*/
#ifdef CONFIG_ETH_DESIGNWARE
@@ -125,6 +145,8 @@ int arch_misc_init(void)
int arch_early_init_r(void)
{
+ socfpga_fpga_add(&altera_fpga[0]);
+
return 0;
}
diff --git a/drivers/fpga/altera.c b/drivers/fpga/altera.c
index 9605554..7c8f518 100644
--- a/drivers/fpga/altera.c
+++ b/drivers/fpga/altera.c
@@ -39,6 +39,9 @@ static const struct altera_fpga {
#if defined(CONFIG_FPGA_STRATIX_V)
{ Altera_StratixV, "StratixV", stratixv_load, NULL, NULL },
#endif
+#if defined(CONFIG_FPGA_STRATIX10)
+ { Intel_FPGA_Stratix10, "Stratix10", stratix10_load, NULL, NULL },
+#endif
#if defined(CONFIG_FPGA_SOCFPGA)
{ Altera_SoCFPGA, "SoC FPGA", socfpga_load, NULL, NULL },
#endif
@@ -154,6 +157,9 @@ int altera_info(Altera_desc *desc)
case fast_passive_parallel_security:
printf("Fast Passive Parallel with Security (FPPS)\n");
break;
+ case secure_device_manager_mailbox:
+ puts("Secure Device Manager (SDM) Mailbox\n");
+ break;
/* Add new interface types here */
default:
printf("Unsupported interface type, %d\n", desc->iface);
diff --git a/include/altera.h b/include/altera.h
index 233b467..22d55cf 100644
--- a/include/altera.h
+++ b/include/altera.h
@@ -39,6 +39,8 @@ enum altera_iface {
fast_passive_parallel,
/* fast passive parallel with security (FPPS) */
fast_passive_parallel_security,
+ /* secure device manager (SDM) mailbox */
+ secure_device_manager_mailbox,
/* insert all new types before this */
max_altera_iface_type,
};
@@ -54,6 +56,8 @@ enum altera_family {
Altera_StratixII,
/* StratixV Family */
Altera_StratixV,
+ /* Stratix10 Family */
+ Intel_FPGA_Stratix10,
/* SoCFPGA Family */
Altera_SoCFPGA,
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [U-Boot] [PATCH v6 4/4] arm: socfpga: stratix10: Enable Stratix10 FPGA Reconfiguration
2018-12-18 8:54 [U-Boot] [PATCH v6 0/4] Stratix10 FPGA reconfiguration support chee.hong.ang at intel.com
` (2 preceding siblings ...)
2018-12-18 8:54 ` [U-Boot] [PATCH v6 3/4] arm: socfpga: stratix10: Add Stratix10 FPGA into FPGA device table chee.hong.ang at intel.com
@ 2018-12-18 8:54 ` chee.hong.ang at intel.com
2018-12-18 17:47 ` Marek Vasut
2018-12-18 17:47 ` [U-Boot] [PATCH v6 0/4] Stratix10 FPGA reconfiguration support Marek Vasut
4 siblings, 1 reply; 13+ messages in thread
From: chee.hong.ang at intel.com @ 2018-12-18 8:54 UTC (permalink / raw)
To: u-boot
From: "Ang, Chee Hong" <chee.hong.ang@intel.com>
Enable Stratix10 FPGA reconfiguration support in defconfig.
Signed-off-by: Ang, Chee Hong <chee.hong.ang@intel.com>
---
configs/socfpga_stratix10_defconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/configs/socfpga_stratix10_defconfig b/configs/socfpga_stratix10_defconfig
index 5f3d733..155c406 100644
--- a/configs/socfpga_stratix10_defconfig
+++ b/configs/socfpga_stratix10_defconfig
@@ -10,6 +10,7 @@ CONFIG_NR_DRAM_BANKS=1
CONFIG_BOOTDELAY=5
CONFIG_SPL_SPI_LOAD=y
CONFIG_HUSH_PARSER=y
+CONFIG_FPGA_STRATIX10=y
CONFIG_SYS_PROMPT="SOCFPGA_STRATIX10 # "
CONFIG_CMD_MEMTEST=y
# CONFIG_CMD_FLASH is not set
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [U-Boot] [PATCH v6 4/4] arm: socfpga: stratix10: Enable Stratix10 FPGA Reconfiguration
2018-12-18 8:54 ` [U-Boot] [PATCH v6 4/4] arm: socfpga: stratix10: Enable Stratix10 FPGA Reconfiguration chee.hong.ang at intel.com
@ 2018-12-18 17:47 ` Marek Vasut
2018-12-19 4:55 ` Ang, Chee Hong
0 siblings, 1 reply; 13+ messages in thread
From: Marek Vasut @ 2018-12-18 17:47 UTC (permalink / raw)
To: u-boot
On 12/18/2018 09:54 AM, chee.hong.ang at intel.com wrote:
> From: "Ang, Chee Hong" <chee.hong.ang@intel.com>
>
> Enable Stratix10 FPGA reconfiguration support in defconfig.
>
> Signed-off-by: Ang, Chee Hong <chee.hong.ang@intel.com>
> ---
> configs/socfpga_stratix10_defconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/configs/socfpga_stratix10_defconfig b/configs/socfpga_stratix10_defconfig
> index 5f3d733..155c406 100644
> --- a/configs/socfpga_stratix10_defconfig
> +++ b/configs/socfpga_stratix10_defconfig
> @@ -10,6 +10,7 @@ CONFIG_NR_DRAM_BANKS=1
> CONFIG_BOOTDELAY=5
> CONFIG_SPL_SPI_LOAD=y
> CONFIG_HUSH_PARSER=y
> +CONFIG_FPGA_STRATIX10=y
> CONFIG_SYS_PROMPT="SOCFPGA_STRATIX10 # "
> CONFIG_CMD_MEMTEST=y
> # CONFIG_CMD_FLASH is not set
>
Can you send a subsequent patch which uses Kconfig imply to select
FPGA_STRATIX10 on S10 instead of adding it in defconfig ?
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] [PATCH v6 0/4] Stratix10 FPGA reconfiguration support
2018-12-18 8:54 [U-Boot] [PATCH v6 0/4] Stratix10 FPGA reconfiguration support chee.hong.ang at intel.com
` (3 preceding siblings ...)
2018-12-18 8:54 ` [U-Boot] [PATCH v6 4/4] arm: socfpga: stratix10: Enable Stratix10 FPGA Reconfiguration chee.hong.ang at intel.com
@ 2018-12-18 17:47 ` Marek Vasut
2018-12-19 4:53 ` Ang, Chee Hong
4 siblings, 1 reply; 13+ messages in thread
From: Marek Vasut @ 2018-12-18 17:47 UTC (permalink / raw)
To: u-boot
On 12/18/2018 09:54 AM, chee.hong.ang at intel.com wrote:
> From: "Ang, Chee Hong" <chee.hong.ang@intel.com>
>
> Summary of v6 changes:
> - Patch 1/4 and 4/4 are unchanged
> - Patch 2/4:
> - fixed compilation warnings in drivers/fpga/stratix10.c
> - Patch 3/4:
> - socfpga_fpga_add() in misc.c
> - define fpga descriptor structure in misc_arria10.c, misc_gen5.c &
> misc_s10.c respectively
> - removed for-loop in socfpga_fpga_add() (only 1 FPGA device added)
>
> v5 patchsets:
> https://lists.denx.de/pipermail/u-boot/2018-November/349670.html
>
> Ang, Chee Hong (4):
> arm: socfpga: stratix10: Add macros for mailbox's arguments
> arm: socfpga: stratix10: Add Stratix 10 FPGA Reconfiguration Driver
> arm: socfpga: stratix10: Add Stratix10 FPGA into FPGA device table
> arm: socfpga: stratix10: Enable Stratix10 FPGA Reconfiguration
>
> arch/arm/mach-socfpga/include/mach/mailbox_s10.h | 6 +
> arch/arm/mach-socfpga/include/mach/misc.h | 4 +-
> arch/arm/mach-socfpga/misc.c | 26 +-
> arch/arm/mach-socfpga/misc_arria10.c | 22 +-
> arch/arm/mach-socfpga/misc_gen5.c | 22 +-
> arch/arm/mach-socfpga/misc_s10.c | 22 ++
> configs/socfpga_stratix10_defconfig | 1 +
> drivers/fpga/Kconfig | 11 +
> drivers/fpga/Makefile | 1 +
> drivers/fpga/altera.c | 6 +
> drivers/fpga/stratix10.c | 288 +++++++++++++++++++++++
> include/altera.h | 8 +
> 12 files changed, 389 insertions(+), 28 deletions(-)
> create mode 100644 drivers/fpga/stratix10.c
I take it this fixes the previous stratix 10 build breakage, right ?
let's see what travis says.
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] [PATCH v6 0/4] Stratix10 FPGA reconfiguration support
2018-12-18 17:47 ` [U-Boot] [PATCH v6 0/4] Stratix10 FPGA reconfiguration support Marek Vasut
@ 2018-12-19 4:53 ` Ang, Chee Hong
2018-12-19 8:18 ` Marek Vasut
0 siblings, 1 reply; 13+ messages in thread
From: Ang, Chee Hong @ 2018-12-19 4:53 UTC (permalink / raw)
To: u-boot
On Tue, 2018-12-18 at 18:47 +0100, Marek Vasut wrote:
> On 12/18/2018 09:54 AM, chee.hong.ang at intel.com wrote:
> >
> > From: "Ang, Chee Hong" <chee.hong.ang@intel.com>
> >
> > Summary of v6 changes:
> > - Patch 1/4 and 4/4 are unchanged
> > - Patch 2/4:
> > - fixed compilation warnings in drivers/fpga/stratix10.c
> > - Patch 3/4:
> > - socfpga_fpga_add() in misc.c
> > - define fpga descriptor structure in misc_arria10.c, misc_gen5.c
> > &
> > misc_s10.c respectively
> > - removed for-loop in socfpga_fpga_add() (only 1 FPGA device
> > added)
> >
> > v5 patchsets:
> > https://lists.denx.de/pipermail/u-boot/2018-November/349670.html
> >
> > Ang, Chee Hong (4):
> > arm: socfpga: stratix10: Add macros for mailbox's arguments
> > arm: socfpga: stratix10: Add Stratix 10 FPGA Reconfiguration
> > Driver
> > arm: socfpga: stratix10: Add Stratix10 FPGA into FPGA device
> > table
> > arm: socfpga: stratix10: Enable Stratix10 FPGA Reconfiguration
> >
> > arch/arm/mach-socfpga/include/mach/mailbox_s10.h | 6 +
> > arch/arm/mach-socfpga/include/mach/misc.h | 4 +-
> > arch/arm/mach-socfpga/misc.c | 26 +-
> > arch/arm/mach-socfpga/misc_arria10.c | 22 +-
> > arch/arm/mach-socfpga/misc_gen5.c | 22 +-
> > arch/arm/mach-socfpga/misc_s10.c | 22 ++
> > configs/socfpga_stratix10_defconfig | 1 +
> > drivers/fpga/Kconfig | 11 +
> > drivers/fpga/Makefile | 1 +
> > drivers/fpga/altera.c | 6 +
> > drivers/fpga/stratix10.c | 288
> > +++++++++++++++++++++++
> > include/altera.h | 8 +
> > 12 files changed, 389 insertions(+), 28 deletions(-)
> > create mode 100644 drivers/fpga/stratix10.c
> I take it this fixes the previous stratix 10 build breakage, right ?
> let's see what travis says.
Yes. Actually they are just compilation warnings to me. Your travis's
build settings treat all warning as error. Please let me know about the
travis report. Thanks.
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] [PATCH v6 4/4] arm: socfpga: stratix10: Enable Stratix10 FPGA Reconfiguration
2018-12-18 17:47 ` Marek Vasut
@ 2018-12-19 4:55 ` Ang, Chee Hong
2018-12-19 8:41 ` Marek Vasut
0 siblings, 1 reply; 13+ messages in thread
From: Ang, Chee Hong @ 2018-12-19 4:55 UTC (permalink / raw)
To: u-boot
On Tue, 2018-12-18 at 18:47 +0100, Marek Vasut wrote:
> On 12/18/2018 09:54 AM, chee.hong.ang at intel.com wrote:
> >
> > From: "Ang, Chee Hong" <chee.hong.ang@intel.com>
> >
> > Enable Stratix10 FPGA reconfiguration support in defconfig.
> >
> > Signed-off-by: Ang, Chee Hong <chee.hong.ang@intel.com>
> > ---
> > configs/socfpga_stratix10_defconfig | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/configs/socfpga_stratix10_defconfig
> > b/configs/socfpga_stratix10_defconfig
> > index 5f3d733..155c406 100644
> > --- a/configs/socfpga_stratix10_defconfig
> > +++ b/configs/socfpga_stratix10_defconfig
> > @@ -10,6 +10,7 @@ CONFIG_NR_DRAM_BANKS=1
> > CONFIG_BOOTDELAY=5
> > CONFIG_SPL_SPI_LOAD=y
> > CONFIG_HUSH_PARSER=y
> > +CONFIG_FPGA_STRATIX10=y
> > CONFIG_SYS_PROMPT="SOCFPGA_STRATIX10 # "
> > CONFIG_CMD_MEMTEST=y
> > # CONFIG_CMD_FLASH is not set
> >
> Can you send a subsequent patch which uses Kconfig imply to select
> FPGA_STRATIX10 on S10 instead of adding it in defconfig ?
Noted. Will address this in next patchsets.
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] [PATCH v6 0/4] Stratix10 FPGA reconfiguration support
2018-12-19 4:53 ` Ang, Chee Hong
@ 2018-12-19 8:18 ` Marek Vasut
0 siblings, 0 replies; 13+ messages in thread
From: Marek Vasut @ 2018-12-19 8:18 UTC (permalink / raw)
To: u-boot
On 12/19/2018 05:53 AM, Ang, Chee Hong wrote:
> On Tue, 2018-12-18 at 18:47 +0100, Marek Vasut wrote:
>> On 12/18/2018 09:54 AM, chee.hong.ang at intel.com wrote:
>>>
>>> From: "Ang, Chee Hong" <chee.hong.ang@intel.com>
>>>
>>> Summary of v6 changes:
>>> - Patch 1/4 and 4/4 are unchanged
>>> - Patch 2/4:
>>> - fixed compilation warnings in drivers/fpga/stratix10.c
>>> - Patch 3/4:
>>> - socfpga_fpga_add() in misc.c
>>> - define fpga descriptor structure in misc_arria10.c, misc_gen5.c
>>> &
>>> misc_s10.c respectively
>>> - removed for-loop in socfpga_fpga_add() (only 1 FPGA device
>>> added)
>>>
>>> v5 patchsets:
>>> https://lists.denx.de/pipermail/u-boot/2018-November/349670.html
>>>
>>> Ang, Chee Hong (4):
>>> arm: socfpga: stratix10: Add macros for mailbox's arguments
>>> arm: socfpga: stratix10: Add Stratix 10 FPGA Reconfiguration
>>> Driver
>>> arm: socfpga: stratix10: Add Stratix10 FPGA into FPGA device
>>> table
>>> arm: socfpga: stratix10: Enable Stratix10 FPGA Reconfiguration
>>>
>>> arch/arm/mach-socfpga/include/mach/mailbox_s10.h | 6 +
>>> arch/arm/mach-socfpga/include/mach/misc.h | 4 +-
>>> arch/arm/mach-socfpga/misc.c | 26 +-
>>> arch/arm/mach-socfpga/misc_arria10.c | 22 +-
>>> arch/arm/mach-socfpga/misc_gen5.c | 22 +-
>>> arch/arm/mach-socfpga/misc_s10.c | 22 ++
>>> configs/socfpga_stratix10_defconfig | 1 +
>>> drivers/fpga/Kconfig | 11 +
>>> drivers/fpga/Makefile | 1 +
>>> drivers/fpga/altera.c | 6 +
>>> drivers/fpga/stratix10.c | 288
>>> +++++++++++++++++++++++
>>> include/altera.h | 8 +
>>> 12 files changed, 389 insertions(+), 28 deletions(-)
>>> create mode 100644 drivers/fpga/stratix10.c
>> I take it this fixes the previous stratix 10 build breakage, right ?
>> let's see what travis says.
> Yes. Actually they are just compilation warnings to me. Your travis's
> build settings treat all warning as error. Please let me know about the
> travis report. Thanks.
+drivers/fpga/stratix10.c: In function 'reconfig_status_polling_resp':
+drivers/fpga/stratix10.c:102:9: error: implicit declaration of function
'mbox_get_fpga_config_status' [-Werror=implicit-function-declaration]
+ ret = mbox_get_fpga_config_status(MBOX_RECONFIG_STATUS);
+ ^~~~~~~~~~~~~~~~~~~~~~~~~~~
+cc1: all warnings being treated as errors
+make[3]: *** [drivers/fpga/stratix10.o] Error 1
+make[2]: *** [drivers/fpga] Error 2
+make[1]: *** [drivers] Error 2
+make: *** [sub-make] Error 2
Seems like a compile error to me. Please at least build the patches
before posting them. See my other email.
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] [PATCH v6 4/4] arm: socfpga: stratix10: Enable Stratix10 FPGA Reconfiguration
2018-12-19 4:55 ` Ang, Chee Hong
@ 2018-12-19 8:41 ` Marek Vasut
2018-12-19 9:48 ` Ang, Chee Hong
0 siblings, 1 reply; 13+ messages in thread
From: Marek Vasut @ 2018-12-19 8:41 UTC (permalink / raw)
To: u-boot
On 12/19/2018 05:55 AM, Ang, Chee Hong wrote:
> On Tue, 2018-12-18 at 18:47 +0100, Marek Vasut wrote:
>> On 12/18/2018 09:54 AM, chee.hong.ang at intel.com wrote:
>>>
>>> From: "Ang, Chee Hong" <chee.hong.ang@intel.com>
>>>
>>> Enable Stratix10 FPGA reconfiguration support in defconfig.
>>>
>>> Signed-off-by: Ang, Chee Hong <chee.hong.ang@intel.com>
>>> ---
>>> configs/socfpga_stratix10_defconfig | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/configs/socfpga_stratix10_defconfig
>>> b/configs/socfpga_stratix10_defconfig
>>> index 5f3d733..155c406 100644
>>> --- a/configs/socfpga_stratix10_defconfig
>>> +++ b/configs/socfpga_stratix10_defconfig
>>> @@ -10,6 +10,7 @@ CONFIG_NR_DRAM_BANKS=1
>>> CONFIG_BOOTDELAY=5
>>> CONFIG_SPL_SPI_LOAD=y
>>> CONFIG_HUSH_PARSER=y
>>> +CONFIG_FPGA_STRATIX10=y
>>> CONFIG_SYS_PROMPT="SOCFPGA_STRATIX10 # "
>>> CONFIG_CMD_MEMTEST=y
>>> # CONFIG_CMD_FLASH is not set
>>>
>> Can you send a subsequent patch which uses Kconfig imply to select
>> FPGA_STRATIX10 on S10 instead of adding it in defconfig ?
> Noted. Will address this in next patchsets.
I asked you for an incremental patch, not for reposting the whole
patchset (also, please track changelog with new versions of patches).
But travis would seem to indicate, again, that the patches break some
other target [1]. Can you please at least build test the next version of
patches on all systems before posting them ?
You can very well just set up the travis CI with your github repo/fork
of u-boot repo, push a branch there and wait for travis to build that
branch on all supported systems and tell you what the situation is. It
saves me time, which I otherwise spend on applying, rejecting and
dropping of your patches after they fail to build.
[1] https://travis-ci.org/marex/u-boot-socfpga/jobs/469629378#L1142
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] [PATCH v6 4/4] arm: socfpga: stratix10: Enable Stratix10 FPGA Reconfiguration
2018-12-19 8:41 ` Marek Vasut
@ 2018-12-19 9:48 ` Ang, Chee Hong
2018-12-19 10:07 ` Marek Vasut
0 siblings, 1 reply; 13+ messages in thread
From: Ang, Chee Hong @ 2018-12-19 9:48 UTC (permalink / raw)
To: u-boot
On Wed, 2018-12-19 at 09:41 +0100, Marek Vasut wrote:
> On 12/19/2018 05:55 AM, Ang, Chee Hong wrote:
> >
> > On Tue, 2018-12-18 at 18:47 +0100, Marek Vasut wrote:
> > >
> > > On 12/18/2018 09:54 AM, chee.hong.ang at intel.com wrote:
> > > >
> > > >
> > > > From: "Ang, Chee Hong" <chee.hong.ang@intel.com>
> > > >
> > > > Enable Stratix10 FPGA reconfiguration support in defconfig.
> > > >
> > > > Signed-off-by: Ang, Chee Hong <chee.hong.ang@intel.com>
> > > > ---
> > > > configs/socfpga_stratix10_defconfig | 1 +
> > > > 1 file changed, 1 insertion(+)
> > > >
> > > > diff --git a/configs/socfpga_stratix10_defconfig
> > > > b/configs/socfpga_stratix10_defconfig
> > > > index 5f3d733..155c406 100644
> > > > --- a/configs/socfpga_stratix10_defconfig
> > > > +++ b/configs/socfpga_stratix10_defconfig
> > > > @@ -10,6 +10,7 @@ CONFIG_NR_DRAM_BANKS=1
> > > > CONFIG_BOOTDELAY=5
> > > > CONFIG_SPL_SPI_LOAD=y
> > > > CONFIG_HUSH_PARSER=y
> > > > +CONFIG_FPGA_STRATIX10=y
> > > > CONFIG_SYS_PROMPT="SOCFPGA_STRATIX10 # "
> > > > CONFIG_CMD_MEMTEST=y
> > > > # CONFIG_CMD_FLASH is not set
> > > >
> > > Can you send a subsequent patch which uses Kconfig imply to
> > > select
> > > FPGA_STRATIX10 on S10 instead of adding it in defconfig ?
> > Noted. Will address this in next patchsets.
> I asked you for an incremental patch, not for reposting the whole
> patchset (also, please track changelog with new versions of patches).
>
> But travis would seem to indicate, again, that the patches break some
> other target [1]. Can you please at least build test the next version
> of
> patches on all systems before posting them ?
>
> You can very well just set up the travis CI with your github
> repo/fork
> of u-boot repo, push a branch there and wait for travis to build that
> branch on all supported systems and tell you what the situation is.
> It
> saves me time, which I otherwise spend on applying, rejecting and
> dropping of your patches after they fail to build.
>
> [1] https://travis-ci.org/marex/u-boot-socfpga/jobs/469629378#L1142
Ok. I only test the build for current (S10) target. Some of the code
are not tested with other build target. I will setup the travis CI and
test it on my side before submitting new patchsets.
But 1 of the build error is due to your code base missing the following
patch:
http://u-boot.10912.n7.nabble.com/PATCH-v2-Add-generic-FPGA-reconfig-ma
ilbox-API-for-S10-td343559.html#a343560
All current patch has dependency on the patch mentioned above.
You will get build errors without this patch applied before current
patchsets.
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* [U-Boot] [PATCH v6 4/4] arm: socfpga: stratix10: Enable Stratix10 FPGA Reconfiguration
2018-12-19 9:48 ` Ang, Chee Hong
@ 2018-12-19 10:07 ` Marek Vasut
0 siblings, 0 replies; 13+ messages in thread
From: Marek Vasut @ 2018-12-19 10:07 UTC (permalink / raw)
To: u-boot
On 12/19/2018 10:48 AM, Ang, Chee Hong wrote:
> On Wed, 2018-12-19 at 09:41 +0100, Marek Vasut wrote:
>> On 12/19/2018 05:55 AM, Ang, Chee Hong wrote:
>>>
>>> On Tue, 2018-12-18 at 18:47 +0100, Marek Vasut wrote:
>>>>
>>>> On 12/18/2018 09:54 AM, chee.hong.ang at intel.com wrote:
>>>>>
>>>>>
>>>>> From: "Ang, Chee Hong" <chee.hong.ang@intel.com>
>>>>>
>>>>> Enable Stratix10 FPGA reconfiguration support in defconfig.
>>>>>
>>>>> Signed-off-by: Ang, Chee Hong <chee.hong.ang@intel.com>
>>>>> ---
>>>>> configs/socfpga_stratix10_defconfig | 1 +
>>>>> 1 file changed, 1 insertion(+)
>>>>>
>>>>> diff --git a/configs/socfpga_stratix10_defconfig
>>>>> b/configs/socfpga_stratix10_defconfig
>>>>> index 5f3d733..155c406 100644
>>>>> --- a/configs/socfpga_stratix10_defconfig
>>>>> +++ b/configs/socfpga_stratix10_defconfig
>>>>> @@ -10,6 +10,7 @@ CONFIG_NR_DRAM_BANKS=1
>>>>> CONFIG_BOOTDELAY=5
>>>>> CONFIG_SPL_SPI_LOAD=y
>>>>> CONFIG_HUSH_PARSER=y
>>>>> +CONFIG_FPGA_STRATIX10=y
>>>>> CONFIG_SYS_PROMPT="SOCFPGA_STRATIX10 # "
>>>>> CONFIG_CMD_MEMTEST=y
>>>>> # CONFIG_CMD_FLASH is not set
>>>>>
>>>> Can you send a subsequent patch which uses Kconfig imply to
>>>> select
>>>> FPGA_STRATIX10 on S10 instead of adding it in defconfig ?
>>> Noted. Will address this in next patchsets.
>> I asked you for an incremental patch, not for reposting the whole
>> patchset (also, please track changelog with new versions of patches).
>>
>> But travis would seem to indicate, again, that the patches break some
>> other target [1]. Can you please at least build test the next version
>> of
>> patches on all systems before posting them ?
>>
>> You can very well just set up the travis CI with your github
>> repo/fork
>> of u-boot repo, push a branch there and wait for travis to build that
>> branch on all supported systems and tell you what the situation is.
>> It
>> saves me time, which I otherwise spend on applying, rejecting and
>> dropping of your patches after they fail to build.
>>
>> [1] https://travis-ci.org/marex/u-boot-socfpga/jobs/469629378#L1142
>
> Ok. I only test the build for current (S10) target. Some of the code
> are not tested with other build target. I will setup the travis CI and
> test it on my side before submitting new patchsets.
> But 1 of the build error is due to your code base missing the following
> patch:
> http://u-boot.10912.n7.nabble.com/PATCH-v2-Add-generic-FPGA-reconfig-ma
> ilbox-API-for-S10-td343559.html#a343560
> All current patch has dependency on the patch mentioned above.
> You will get build errors without this patch applied before current
> patchsets.
Then please submit the complete series, the dependency is not mentioned
anywhere. And yes, please do keep testing.
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2018-12-19 10:07 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-12-18 8:54 [U-Boot] [PATCH v6 0/4] Stratix10 FPGA reconfiguration support chee.hong.ang at intel.com
2018-12-18 8:54 ` [U-Boot] [PATCH v6 1/4] arm: socfpga: stratix10: Add macros for mailbox's arguments chee.hong.ang at intel.com
2018-12-18 8:54 ` [U-Boot] [PATCH v6 2/4] arm: socfpga: stratix10: Add Stratix 10 FPGA Reconfiguration Driver chee.hong.ang at intel.com
2018-12-18 8:54 ` [U-Boot] [PATCH v6 3/4] arm: socfpga: stratix10: Add Stratix10 FPGA into FPGA device table chee.hong.ang at intel.com
2018-12-18 8:54 ` [U-Boot] [PATCH v6 4/4] arm: socfpga: stratix10: Enable Stratix10 FPGA Reconfiguration chee.hong.ang at intel.com
2018-12-18 17:47 ` Marek Vasut
2018-12-19 4:55 ` Ang, Chee Hong
2018-12-19 8:41 ` Marek Vasut
2018-12-19 9:48 ` Ang, Chee Hong
2018-12-19 10:07 ` Marek Vasut
2018-12-18 17:47 ` [U-Boot] [PATCH v6 0/4] Stratix10 FPGA reconfiguration support Marek Vasut
2018-12-19 4:53 ` Ang, Chee Hong
2018-12-19 8:18 ` Marek Vasut
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox