diff options
| author | Martin Kaiser <[email protected]> | 2023-02-13 19:54:08 +0000 |
|---|---|---|
| committer | Greg Kroah-Hartman <[email protected]> | 2023-02-16 12:55:03 +0000 |
| commit | b5929325f06300d28696b9a030539a4009154788 (patch) | |
| tree | 26419298c19cc423248a2771389efaab7c5c804d /drivers/fpga/microchip-spi.c | |
| parent | staging: r8188eu: replace hand coded loop with list_for_each_entry (diff) | |
| download | kernel-b5929325f06300d28696b9a030539a4009154788.tar.gz kernel-b5929325f06300d28696b9a030539a4009154788.zip | |
staging: r8188eu: Revert "staging: r8188eu: simplify rtw_get_ff_hwaddr"
This reverts commit fd48124e0982 ("staging: r8188eu: simplify
rtw_get_ff_hwaddr").
The cleanup in this commit removes the qsel to addr mappings in
rtw_get_ff_hwaddr. The underlying assumption is that rtw_write_port
uses its addr parameter only for the high_queue check.
This is obviously incorrect as rtw_write_port calls
ffaddr2pipehdl(pdvobj, addr);
where addr is mapped to a usb bulk endpoint.
Unfortunately, testing did not show any problems. The Edimax V2 on which I
tested has two bulk out endpoints. I guess that with the incorrect patch,
addr could only be 0 (no high queue) or 6 (high queue), both of which were
mapped to the first bulk out endpoint. Data transfers did still work...
Fixes: fd48124e0982 ("staging: r8188eu: simplify rtw_get_ff_hwaddr")
Signed-off-by: Martin Kaiser <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Greg Kroah-Hartman <[email protected]>
Diffstat (limited to 'drivers/fpga/microchip-spi.c')
0 files changed, 0 insertions, 0 deletions