Skip to content

firmware: development for v4.17

Luis R. Rodriguez linux-fsdevel linux-wireless lkml 3 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest20 patches 18 repliesIn review
  2. v2 posting 2/211 patches 21 repliesSuperseded
  3. v2 posting 1/211 patches 1 replySuperseded

Patches in v3 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/20test_firmware: add simple firmware firmware test librarytools/testing/selftests/firmware/fw_fallback.sh tools/testing/selftests/firmware/fw_filesystem.sh tools/testing/selftests/firmware/fw_lib.sh+51 -200 replies
2/20test_firmware: enable custom fallback testing on limited kernel configstools/testing/selftests/firmware/config tools/testing/selftests/firmware/fw_fallback.sh tools/testing/selftests/firmware/fw_lib.sh+33 -10 replies
3/20test_firmware: replace syfs fallback check with kconfig_has helpertools/testing/selftests/firmware/fw_fallback.sh+1 -40 replies
4/20firmware: enable to split firmware_class into separate target filesdrivers/base/Makefile drivers/base/firmware_loader.c+1 -00 replies
5/20firmware: simplify CONFIG_FW_LOADER_USER_HELPER_FALLBACK furtherdrivers/base/firmware_loader.c+18 -72 replies
6/20firmware: use helpers for setting up a temporary cache timeoutdrivers/base/firmware_loader.c+30 -191 reply
7/20firmware: move loading timeout under struct firmware_fallback_configdrivers/base/firmware_loader.c+33 -132 replies
8/20firmware: split firmware fallback functionality into its own filedrivers/base/Makefile drivers/base/firmware_fallback.c drivers/base/firmware_fallback.h +3 more+874 -7990 replies
9/20firmware: move firmware loader into its own directoryMAINTAINERS drivers/base/Makefile drivers/base/firmware_loader/Makefile +5 more+18 -140 replies
10/20firmware: enable run time change of forcing fallback loaderdrivers/base/firmware_loader/fallback.c drivers/base/firmware_loader/fallback.h drivers/base/firmware_loader/fallback_table.c +1 more+32 -10 replies
11/20firmware: enable to force disable the fallback mechanism at run timedrivers/base/firmware_loader/fallback.c drivers/base/firmware_loader/fallback.h drivers/base/firmware_loader/fallback_table.c+18 -02 replies
12/20test_firmware: expand on library with shared helperstools/testing/selftests/firmware/fw_fallback.sh tools/testing/selftests/firmware/fw_filesystem.sh tools/testing/selftests/firmware/fw_lib.sh+63 -610 replies
13/20test_firmware: test three firmware kernel configs using a proc knobtools/testing/selftests/firmware/Makefile tools/testing/selftests/firmware/fw_lib.sh tools/testing/selftests/firmware/fw_run_tests.sh+122 -10 replies
14/20rename: _request_firmware_load() fw_load_sysfs_fallback()Documentation/driver-api/firmware/fallback-mechanisms.rst drivers/base/firmware_loader/fallback.c+11 -40 replies
15/20firmware: fix checking for return values for fw_add_devm_name()drivers/base/firmware_loader/main.c+9 -30 replies
16/20firmware: add helper to check to see if fw cache is setupdrivers/base/firmware_loader/main.c+12 -20 replies
17/20test_firmware: modify custom fallback tests to use unique filestools/testing/selftests/firmware/fw_fallback.sh tools/testing/selftests/firmware/fw_filesystem.sh tools/testing/selftests/firmware/fw_lib.sh+46 -80 replies
18/20firmware: ensure the firmware cache is not used on incompatible callsdrivers/base/firmware_loader/main.c+14 -00 replies
19/20firmware: add request_firmware_cache() to help with cache on rebootDocumentation/driver-api/firmware/request_firmware.rst drivers/base/firmware_loader/main.c include/linux/firmware.h+41 -07 replies
20/20mt7601u: use request_firmware_cache() to address cache on rebootdrivers/net/wireless/mediatek/mt7601u/mcu.c+1 -11 reply

Reviews trailers per patch

9 of 20 patches have a Reviewed-by or Acked-by.

PatchKees CookCantabileJakub Kicinski
1/20
2/20A-b
3/20A-b
4/20
5/20A-b
6/20A-b
7/20A-b
8/20
9/20
10/20A-b
11/20A-b
12/20
13/20A-b
14/20
15/20
16/20
17/20
18/20
19/20
20/20T-b A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v3 thread

Cover letter [PATCH v3 00/20] firmware: development for v4.17

Greg,

Here's a respin of what I have queued up for v4.17 for the firmware API. It combines the cleanup I've been working on and the addition of the new API call request_firmware_cache() for fixing a corner case suspend issue on some type of cards with an optimization in place where the firmware is *not* needed on reboot.

The cleanup work allows us to test the firmware API with one kernel configuration. I've addressed Kees' feedback on this respin and combined the code into drivers/base/firmware_class/.

I've made one new test_firmware change in consideration for one firmware change, the patch "firmware: ensure the firmware cache is not used on incompatible calls" requires us to modify our tests scripts to use the APIs sanely as well.

I've put up these changes on my git tree, refer to the branch "20180307-firmware-dev-for-v4.17" based on linux-next [0] and the same name based on Linus' tree [1].

Questions, feedback, and specially rants are always welcomed.

[0] https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git/log/?h=20180307-firmware-dev-for-v4.17
[1] https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux.git/log/?h=20180307-firmware-dev-for-v4.17

Luis R. Rodriguez (20):
test_firmware: add simple firmware firmware test library
test_firmware: enable custom fallback testing on limited kernel
configs
test_firmware: replace syfs fallback check with kconfig_has helper
firmware: enable to split firmware_class into separate target files
firmware: simplify CONFIG_FW_LOADER_USER_HELPER_FALLBACK further
firmware: use helpers for setting up a temporary cache timeout
firmware: move loading timeout under struct firmware_fallback_config
firmware: split firmware fallback functionality into its own file
firmware: move firmware loader into its own directory
firmware: enable run time change of forcing fallback loader
firmware: enable to force disable the fallback mechanism at run time
test_firmware: expand on library with shared helpers
test_firmware: test three firmware kernel configs using a proc knob
rename: _request_firmware_load() fw_load_sysfs_fallback()
firmware: fix checking for return values for fw_add_devm_name()
firmware: add helper to check to see if fw cache is setup
test_firmware: modify custom fallback tests to use unique files
firmware: ensure the firmware cache is not used on incompatible calls
firmware: add request_firmware_cache() to help with cache on reboot
mt7601u: use request_firmware_cache() to address cache on reboot

 .../driver-api/firmware/fallback-mechanisms.rst    |   2 +-
 .../driver-api/firmware/request_firmware.rst       |  14 +
 MAINTAINERS                                        |   2 +-
 drivers/base/Makefile                              |   2 +-
 drivers/base/firmware_loader/Makefile              |   7 +
 drivers/base/firmware_loader/fallback.c            | 674 +++++++++++++++++
 drivers/base/firmware_loader/fallback.h            |  67 ++
 drivers/base/firmware_loader/fallback_table.c      |  55 ++
 drivers/base/firmware_loader/firmware.h            | 115 +++
 .../{firmware_class.c => firmware_loader/main.c}   | 833 ++-------------------
 drivers/net/wireless/mediatek/mt7601u/mcu.c        |   2 +-
 include/linux/firmware.h                           |   3 +
 kernel/sysctl.c                                    |  11 +
 tools/testing/selftests/firmware/Makefile          |   2 +-
 tools/testing/selftests/firmware/config            |   4 +
 tools/testing/selftests/firmware/fw_fallback.sh    |  65 +-
 tools/testing/selftests/firmware/fw_filesystem.sh  |  72 +-
 tools/testing/selftests/firmware/fw_lib.sh         | 194 +++++
 tools/testing/selftests/firmware/fw_run_tests.sh   |  70 ++
 19 files changed, 1332 insertions(+), 862 deletions(-)
 create mode 100644 drivers/base/firmware_loader/Makefile
 create mode 100644 drivers/base/firmware_loader/fallback.c
 create mode 100644 drivers/base/firmware_loader/fallback.h
 create mode 100644 drivers/base/firmware_loader/fallback_table.c
 create mode 100644 drivers/base/firmware_loader/firmware.h
 rename drivers/base/{firmware_class.c => firmware_loader/main.c} (60%)
 create mode 100755 tools/testing/selftests/firmware/fw_lib.sh
 create mode 100755 tools/testing/selftests/firmware/fw_run_tests.sh
Signature · 1 line
2.16.2