smd: add do while loops to init and prepare macros

Now callers are forced to use semicolon in the end.

Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
smd: add HAL debug messages

Easier to follow what's wcn36xx is doing.

Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
debug: use printk() instead pr_debug()

If CONFIG_DYNAMIC_DEBUG is enabled user would need
to use both debug_mask and /sys/kernel/debug/dynamic_debug/
to get debug logs. That's just confusing, it should be
enough to use debug_mask module parameter.

Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
smd: print what hal response had an error

Also introduce wcn36xx_warn() and use that to print
the warning.

Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
Cleanup TX debugging

Also simplify wcn36xx_tx() in the process.

Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
Improve rx debugging

Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
dxe: move some debug logs from RX to DXE level

And make the log messages more compact as well.

Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
Revert "debug: use printk() instead pr_debug()"

This reverts commit 113f8ef1a26f9033fbfc13de83886ae68dea20cb.

There's not concensus how to handle dynamic debugging, so
revert this change until we find a good solution.