From f696bf2b3ca9ec77a69ad0e18f53e8cf2497ecc2 Mon Sep 17 00:00:00 2001 From: Lior David Date: Wed, 9 Mar 2016 10:30:37 +0200 Subject: [PATCH] wil6210: clean ioctl debug message Fix a debug message related to IOCTL that was incorrectly logged with the MISC category, and move it inside wil_ioctl so it will always be logged even if we call wil_ioctl from other places. Change-Id: I3cf69b18f195e5db01a1f5832013342c9f481286 Signed-off-by: Lior David Signed-off-by: Maya Erez Signed-off-by: Kalle Valo Git-commit: 6777e71ca91ea488488362a919900488e0ade3f2 Git-repo: https://github.com/kvalo/ath.git CRs-Fixed: 982931 [merez@codeaurora.org: fixed conflict due to android private ioctl] Signed-off-by: Maya Erez --- drivers/net/wireless/ath/wil6210/ioctl.c | 14 +++++++++++--- drivers/net/wireless/ath/wil6210/netdev.c | 6 +----- 2 files changed, 12 insertions(+), 8 deletions(-) diff --git a/drivers/net/wireless/ath/wil6210/ioctl.c b/drivers/net/wireless/ath/wil6210/ioctl.c index ce7040fdbf6a..47058ccccb5b 100644 --- a/drivers/net/wireless/ath/wil6210/ioctl.c +++ b/drivers/net/wireless/ath/wil6210/ioctl.c @@ -215,15 +215,23 @@ out_free: } int wil_ioctl(struct wil6210_priv *wil, void __user *data, int cmd) { + int ret; + switch (cmd) { case WIL_IOCTL_MEMIO: - return wil_ioc_memio_dword(wil, data); + ret = wil_ioc_memio_dword(wil, data); + break; case WIL_IOCTL_MEMIO_BLOCK: - return wil_ioc_memio_block(wil, data); + ret = wil_ioc_memio_block(wil, data); + break; case (SIOCDEVPRIVATE + 1): - return wil_ioc_android(wil, data); + ret = wil_ioc_android(wil, data); + break; default: wil_dbg_ioctl(wil, "Unsupported IOCTL 0x%04x\n", cmd); return -ENOIOCTLCMD; } + + wil_dbg_ioctl(wil, "ioctl(0x%04x) -> %d\n", cmd, ret); + return ret; } diff --git a/drivers/net/wireless/ath/wil6210/netdev.c b/drivers/net/wireless/ath/wil6210/netdev.c index c7beb11c61b5..2a05031ba631 100644 --- a/drivers/net/wireless/ath/wil6210/netdev.c +++ b/drivers/net/wireless/ath/wil6210/netdev.c @@ -60,11 +60,7 @@ static int wil_do_ioctl(struct net_device *ndev, struct ifreq *ifr, int cmd) { struct wil6210_priv *wil = ndev_to_wil(ndev); - int ret = wil_ioctl(wil, ifr->ifr_data, cmd); - - wil_dbg_misc(wil, "ioctl(0x%04x) -> %d\n", cmd, ret); - - return ret; + return wil_ioctl(wil, ifr->ifr_data, cmd); } static const struct net_device_ops wil_netdev_ops = {