On 7/18/2022 7:17 PM, Danny van Heumen wrote:
------- Original Message -------
On Thursday, July 14th, 2022 at 12:04, Ulf Hansson <ulf.hansson@xxxxxxxxxx> wrote:
[..]
That said, I think the $subject patch looks reasonable to me. So feel
free to add:
Reviewed-by: Ulf Hansson ulf.hansson@xxxxxxxxxx
I am a first-time contributor. Is this your way of saying that I should submit
the patch somewhere other than 'linux-wireless@...'? I suppose this fix is not
urgent enough for 'stable@...', or is it?
I would appreciate information on who will and/or how to follow-up.
Hi Danny,
The above means that Uffe is okay with the patch being included. Kalle
Valo is the linux-wireless maintainer and he will apply the patch to the
linux-wireless repo. The status of your patch can be monitored through
patchwork [1].
By the way, you can add the Fixes: tag referring to the commit that
introduced the issue. Not sure which one should be considered here, but
it is either this one ...
Fixes: 9982464379e8 ("brcmfmac: make sdio suspend wait for threads to
freeze")
... or ...
Fixes: 7836102a750a ("brcmfmac: reset SDIO bus on a firmware crash")
How to create the tag is described here [2]. With git configured
properly you can simply do:
$ git log --pretty=fixes -1 <commit_sha1>
Stable patches have their additional rules [3]. The bus reset was added
in kernel v5.9. Hope this helps.
Regards,
Arend
[1]
https://patchwork.kernel.org/project/linux-wireless/patch/id1HN6qCMAirApBzTA6fT7ZFWBBGCJhULpflxQ7NT6cgCboVnn3RHpiOFjA9SbRqzBRFLk9ES0C4FNvO6fUQsNg7pqF6ZSNAYUo99nHy8PY=@dannyvanheumen.nl/
[2]
https://www.kernel.org/doc/html/v4.10/process/submitting-patches.html#describe-your-changes
[3]
https://www.kernel.org/doc/html/v4.10/process/stable-kernel-rules.html#procedure-for-submitting-patches-to-the-stable-tree
