[PATCH v5 0/6] Improve user feedback in case of FPGA bitstream load failure

[Michal Simek]

On 7/8/25 17:24, vtpieter at gmail.com wrote:
> 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.
> 
> v5:
>   * add new patch 5/6 to fix Wstrict-prototypes error, reported by clang
>   * revert sprintf format to original in patch 1/6 zynqpl.c
>   * remove Kconfig new dependency in patch 2/6, fixed upstream already
> 
> v4: https://lore.kernel.org/u-boot/20250705205710.792370-1-vtpieter@gmail.com/
>   * 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 (6):
>    cmd: fpga: improve user feedback in case of bitstream load failure
>    drivers: fpga: add Kconfig dependency
>    drivers: fpga: correct compiler errors and warnings
>    drivers: fpga: cleanup printf usage
>    drivers: fpga: fix function declaration without a prototype
>    configs: enable FPGA commands and drivers at sandbox_defconfig
> 
>   cmd/fpga.c                | 101 ++++++++++++++---------------
>   configs/sandbox_defconfig |  20 ++++++
>   drivers/fpga/ACEX1K.c     |   1 +
>   drivers/fpga/Kconfig      |   1 +
>   drivers/fpga/fpga.c       | 103 ++++++++++++-----------------
>   drivers/fpga/ivm_core.c   |  13 ++--
>   drivers/fpga/lattice.c    |   4 +-
>   drivers/fpga/spartan2.c   |   1 +
>   drivers/fpga/stratixII.c  | 132 ++++++++++++++++++--------------------
>   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 +-
>   17 files changed, 198 insertions(+), 201 deletions(-)
> 
> 
> base-commit: 1323b480a6fc053475901a90bdaece2ddcc47310
> prerequisite-patch-id: d13ffa9f16b79939ca26d5334d15c077c11427af
> prerequisite-patch-id: cf4d9931473e7f673dad6a4e7c23fc3fe4678408
> prerequisite-patch-id: e6885654262d1a9ddbb8d33e988be9c7c04a95cf

Applied.
M