Re: [PATCH v2 5/6] firmware: Add request_firmware_prefer_user() function

From: Luis R. Rodriguez
Date: Fri Nov 10 2017 - 15:28:52 EST


On Fri, Nov 10, 2017 at 12:26 PM, Luis R. Rodriguez <mcgrof@xxxxxxxxxx> wrote:
> even if you *leave* the flag in place and a driver required
> this, but the kernel was compiled without CONFIG_FW_LOADER_USER_HELPER then
> calling fw_load_from_user_helper would just already return -ENOENT, as such it
> would in turn fallback to direct fs loading so the #ifef'ery seems to be not
> needed.

> The commit log in no way documents any of the justifications for further
> extending use of the usermode helper.

Also, any new functionality introduced should have a respective test
case in tools/testing/selftests/firmware/ and lib/test_firmware.c, as
well as extending existing documentation on
Documentation/driver-api/firmware/.

Luis