Hi,
We are using this framework for a while and by adding more and more features it requires small redesigned how commands are handled. This is something what I have put together to improve readability of this code and also remove code which is bogus and completely untested.
The series depends on these patches. https://lists.denx.de/pipermail/u-boot/2018-June/332487.html https://lists.denx.de/pipermail/u-boot/2018-June/332488.html https://lists.denx.de/pipermail/u-boot/2018-June/330651.html
I am sending this as RFC to get a feedback on these changes. I have rebased it on the latest tree and need to run another testing round again.
Thanks, Michal
Michal Simek (15): test/py: Extend fpga command to test all fpga load types cmd: fpga: Move error handling to do_fpga() cmd: fpga: Move fpga_get_op to avoid local function declaration cmd: fpga: Cleanup error handling in connection to FPGA_NONE cmd: fpga: Move parameter checking for loadfs/loads cmd: fpga: Remove parameter checking from fpga loadfs command cmd: fpga: Clean wrong_parms handling cmd: fpga: Create new do_fpga_wrapper for using u-boot subcommands cmd: fpga: Extract fpga info command to separate function cmd: fpga: Fix dump and all direct fpga load commands cmd: fpga: Fix loadfs command cmd: fpga: Fix loadmk command cmd: fpga: Use CMD_RET_FAILURE instead of simple 1 cmd: fpga: Fix loads command MAINTAINERS: Add myself as the FPGA maintainer
MAINTAINERS | 8 + cmd/fpga.c | 606 ++++++++++++++++++++++++--------------------- test/py/tests/test_fpga.py | 516 ++++++++++++++++++++++++++++++++++++++ 3 files changed, 843 insertions(+), 287 deletions(-) create mode 100644 test/py/tests/test_fpga.py