On Fri, Jul 28, 2023 at 09:48:08PM +0200, Mirsad Todorovac wrote:
> v3 -> v4
> - fix additional memory leaks of the allocated firmware buffers
> - fix noticed racing conditions in conformance with the existing code
> - make it a single patch
This is not quite right.
Your patch commit 48e156023059 ("test_firmware: fix the memory leak of
the allocated firmware buffer" is already upstream and now you're taking
that same patch and modifying it?
If you have something else you want to fix you can use the latest
lib/firmware.c refelected on linux-next and send a patch against that
to augment with more fixes.
If your goal however, is to make sure these patches end up in v5.4
(as I think you are trying based on your last email) you first send
a patch matching exactly what is in the upstream commit for inclusion
in v5.4. Do not modify the commit unless you are making changes need
to be made due to backporting, and if you do you specify that at the
bottommon of the commit after singed offs of before in brackets
[like this].
Furthermore, I see you have other fixes other than this one merged
already on upstream so if you need those for v5.4 you need to send those
too.
Luis
