[3.2.y] staging: r8712u: Fix regression caused by commit 8c213fa

[Larry Finger]

On 05/08/2012 10:18 PM, Ben Hutchings wrote:
> On Mon, 2012-05-07 at 21:04 -0500, Larry Finger wrote:
>> On 05/07/2012 08:18 PM, Jonathan Nieder wrote:
>>> Hi,
>>>
>>> Larry Finger wrote:
>>>
>>>> I missed a Cc for stable in this patch:
>>>>
>>>> ================================================
>>>> commit 2080913e017ab9f88379d93fd09546ad95faf87b
>>>> Author: Larry Finger<Larry.Finger at lwfinger.net>
>>>> Date:   Sat Feb 25 18:10:21 2012 -0600
>>>>
>>>>       staging: r8712u: Fix regression caused by commit 8c213fa
>>>>
>>>>       In commit 8c213fa "staging: r8712u: Use asynchronous firmware loading",
>>>>       the command to release the firmware was placed in the wrong routine.
>>>>
>>>>       In combination with the bug introduced in commit a5ee652 "staging: r8712u:
>>>>       Interface-state not fully tracked", the driver attempts to upload firmware
>>>>       that had already been released. This bug is the source of one of the
>>>>       problems in https://bugs.archlinux.org/task/27996#comment89833.
>>>>
>>>>       Tested-by: Alberto Lago Ballesteros<saniukeokusainaya at gmail.com>
>>>>       Tested-by: Adrian<agib at gmx.de>
>>>>       Signed-off-by: Larry Finger<Larry.Finger at lwfinger.net>
>>>>       Signed-off-by: Greg Kroah-Hartman<gregkh at linuxfoundation.org>
>>>
>>> Accordingly, the fix described above was included in 3.3.4.
>>>
>>> a5ee652 was merged during the 3.2 merge window and 8c213fa was
>>> backported as part of 3.2.6, so I imagine the 3.2-stable tree needs
>>> this fix, too.  (Luckily neither patch is in 3.0.y.)
>>
>> Yes, that is correct.
>
> I've queued this for 3.2.y, thanks.

Thank you,

Larry