On Tue, 17 Apr 2018 10:41:29 +0200
Xavi Drudis Ferran <[email protected]> wrote:

The patch below seem sensible to me:
> request_firmware_nowait(module, uevent, NONFREE_FIRMWARE,
> -                                    device, gfp, NULL, cont);
> +                                    device, gfp, context, cont);

However merging the patch without finding why the NULL was put there
doesn't seem a good idea as it could break many drivers along the way.

Would it be possible to try to find this information by:
- Finding when it was first introduced
- By looking at the ChangeLogs of the version introducing it

Note that I'm not a Linux-libre maintainer, here I'm just helping to
review the patch.

Denis.

Attachment: pgpNJoUsY_vu6.pgp
Description: OpenPGP digital signature

_______________________________________________
linux-libre mailing list
[email protected]
http://www.fsfla.org/cgi-bin/mailman/listinfo/linux-libre

Reply via email to