uhid_snes: Remove USB_ST_TRANSFERRED handling for the status request.

[jhb]

Summary

The result of the request computed in new_status was never returned to
the caller leaving new_status as a set-but-unused variable. Removing
new_status leaves sc->previous_status as a write-only variable.
Removing sc->previous_status leaves current_status as a write-only
variable, so it collapses down to removing the entire
USB_ST_TRANSFERRED case.

Arguably, all of the support for UHID_SNES_STATUS_DT_RD should be
removed as it doesn't return anything to the caller. If the request
should be fixed instead then this commit should be reverted and
new_status should be returned to whoever submitted the request.

[• hselasky]

I suspect you can just remove the new_status . I think it is some leftover.

[wulf]

In D34840#789553, @hselasky wrote:

I suspect you can just remove the new_status . I think it is some leftover.

+1

[jhb]

sys/dev/usb/input/uhid_snes.c

521

So if I remove new_status I need to remove all of this block I think? Once I remove new_status, then sc->sc_previous_status becomes a write-only variable so I need to remove that. Once I do that, then current_status becomes a write-only variable so then I have to remove that and the rest of the code in this block. I'm fine with doing all that if it is all dead code. However, it begs the question of if this entire function is dead code since the logical request it is associated with seems pointless as it doesn't return the status obtained from the device anywhere?