Re: [PATCH 2/7] firmware: add offset to request_firmware_into_buf

From: Luis Chamberlain
Date: Thu Aug 22 2019 - 15:47:17 EST


On Thu, Aug 22, 2019 at 12:24:46PM -0700, Scott Branden wrote:
> @@ -923,16 +936,22 @@ EXPORT_SYMBOL_GPL(firmware_request_cache);
> */
> int
> request_firmware_into_buf(const struct firmware **firmware_p, const char *name,
> - struct device *device, void *buf, size_t size)
> + struct device *device, void *buf, size_t size,
> + size_t offset, unsigned int pread_flags)

This implies you having to change the other callers, and while currently
our list of drivers is small, following the history of the firmware API
and the long history of debate of *how* we should evolve its API, its
preferred we add yet another new caller for this functionality. So
please add a new caller, and use EXPORT_SYMBOL_GPL().

And while at it, pleaase use firmware_request_*() as the prefix, as we
have want to use that as the instilled prefix. We have yet to complete
the rename of the others older callers but its just a matter of time.

So something like: firmware_request_into_buf_offset()

And thanks for adding a test case!

Luis