mwl8787: remove reset command

Drop this for now since it's unused; we can always add it back
later.  It's the only user of the async send_cmd.

Signed-off-by: Bob Copeland <me@bobcopeland.com>
mwl8787: drop '_sync' annotation to send_cmd

The hardware only supports one command in flight at a time anyway
and sdio bus xfers must sleep, so I don't see a whole lot of
value in an async version.  Make the default synchronous.

Signed-off-by: Bob Copeland <me@bobcopeland.com>
mwl8787: fix skb leak for failed testmode commands

We never process the response for failed responses,
so don't bother setting it.

Signed-off-by: Bob Copeland <me@bobcopeland.com>
mwl8787: replace mwl8787_send_cmd_tm with send_cmd_reply

I plan to move some of the response processing into the
corresponding send_cmd function so this paves the way.

Signed-off-by: Bob Copeland <me@bobcopeland.com>
mwl8787: fix prototype for mwl8787_get_tsf

Fixes:
drivers/net/wireless/mwl8787/main.c:466:2: warning: initialization from
incompatible pointer type [enabled by default]
drivers/net/wireless/mwl8787/main.c:466:2: warning: (near initialization
for ‘mwl8787_ops.get_tsf’) [enabled by default]

Signed-off-by: Bob Copeland <me@bobcopeland.com>
mwl8787: check that reply matches request cmd

Just in case fw starts doing something weird, we want to
be sure not to access random data by looking at the wrong
response.  In the future should probably also check that
skb size matches the size of the expected structure.

Signed-off-by: Bob Copeland <me@bobcopeland.com>
mwl8787: return tsf directly to caller

Signed-off-by: Bob Copeland <me@bobcopeland.com>