[PATCH v4 0/5] Improve user feedback in case of FPGA bitstream load failure

vtpieter at gmail.com vtpieter at gmail.com
Sat Jul 5 22:57:01 CEST 2025


From: Pieter Van Trappen <pieter.van.trappen at cern.ch>

The main motivation for this patch series is to make the `fpga *`
commands' return message more explicit because simply returning the
command usage without indicating the root cause is not helpful.

In addition, the sandbox_defconfig was amended to facilitate compiling
fpga-based sources and various style, compiler errors and warnings
have been fixed.

v4:
 * make better use of prefixes in commit subjects
 * change patches' order to ensure no commit introduces compiler
 warnings or errors i.e. defconfig is now patch 5/5
 * add new patch 4/5 to remove compiler Werror=unused-function
 * add an additional Kconfig dependency to patch 2/5

v3: https://lore.kernel.org/all/20250626093519.115655-1-vtpieter@gmail.com/
 * enable FPGA configs in sandbox_defconfig instead of new one

v2: https://lore.kernel.org/all/20250626093519.115655-1-vtpieter@gmail.com/
 * use log_* instead of puts() which is now preferred
 * harmonize user feedback format; remove useless strings
 * add a sandbox_fpga_defconfig; amend fpga Kconfig
 * fix various compiler errors and warnings

v1: https://lore.kernel.org/all/20250626093519.115655-1-vtpieter@gmail.com/

Pieter Van Trappen (5):
  cmd: fpga: improve user feedback in case of bitstream load failure
  drivers: fpga/firmware: add Kconfig dependencies
  drivers: fpga: correct compiler errors and warnings
  drivers: fpga: cleanup printf usage
  configs: enable FPGA commands and drivers at sandbox_defconfig

 cmd/fpga.c                | 101 ++++++++++++++---------------
 configs/sandbox_defconfig |  20 ++++++
 drivers/firmware/Kconfig  |   1 +
 drivers/fpga/ACEX1K.c     |   1 +
 drivers/fpga/Kconfig      |   1 +
 drivers/fpga/fpga.c       | 103 ++++++++++++------------------
 drivers/fpga/ivm_core.c   |   1 +
 drivers/fpga/lattice.c    |   4 +-
 drivers/fpga/spartan2.c   |   1 +
 drivers/fpga/stratixII.c  | 131 +++++++++++++++++---------------------
 drivers/fpga/stratixv.c   |   2 +-
 drivers/fpga/versalpl.c   |   1 -
 drivers/fpga/virtex2.c    |   2 +
 drivers/fpga/xilinx.c     |   2 +-
 drivers/fpga/zynqmppl.c   |   4 +-
 drivers/fpga/zynqpl.c     |   4 +-
 include/fpga.h            |   2 +-
 include/stratixII.h       |   6 +-
 18 files changed, 192 insertions(+), 195 deletions(-)


base-commit: 7598b469c16d97128d9c22839b06d94c5c331a7e
-- 
2.48.1



More information about the U-Boot mailing list