Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751900AbcJIPMM (ORCPT ); Sun, 9 Oct 2016 11:12:12 -0400 Received: from mail-lf0-f67.google.com ([209.85.215.67]:36441 "EHLO mail-lf0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751098AbcJIPMH (ORCPT ); Sun, 9 Oct 2016 11:12:07 -0400 From: Sergio Paracuellos To: gregkh@linuxfoundation.org Cc: devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: [PATCH 06/15] staging: wlan-ng: do not return -ENOSYS Date: Sun, 9 Oct 2016 17:10:23 +0200 Message-Id: <1476025832-15174-7-git-send-email-sergio.paracuellos@gmail.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1476025832-15174-1-git-send-email-sergio.paracuellos@gmail.com> References: <20161009143102.GA4231@kroah.com> <1476025832-15174-1-git-send-email-sergio.paracuellos@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1464 Lines: 43 Fixed coding style issue by replacing ENOSYS with EINVAL. This patch fix the following checkpatch.pl warnings in p80211netdev.c: WARNING: ENOSYS means 'invalid syscall nr' and nothing else Signed-off-by: Sergio Paracuellos --- drivers/staging/wlan-ng/p80211netdev.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wlan-ng/p80211netdev.c b/drivers/staging/wlan-ng/p80211netdev.c index fb28d8f..656033a 100644 --- a/drivers/staging/wlan-ng/p80211netdev.c +++ b/drivers/staging/wlan-ng/p80211netdev.c @@ -532,7 +532,7 @@ static int p80211netdev_ethtool(struct wlandevice *wlandev, * -ETIME p80211 cmd timed out (MSD may have its own timers) * -EFAULT memory fault copying msg from user buffer * -ENOMEM unable to allocate kernel msg buffer - * -ENOSYS bad magic, it the cmd really for us? + * -EINVAL bad magic, it the cmd really for us? * -EintR sleeping on cmd, awakened by signal, cmd cancelled. * * Call Context: @@ -560,7 +560,7 @@ static int p80211knetdev_do_ioctl(struct net_device *dev, /* Test the magic, assume ifr is good if it's there */ if (req->magic != P80211_IOCTL_MAGIC) { - result = -ENOSYS; + result = -EINVAL; goto bail; } @@ -568,7 +568,7 @@ static int p80211knetdev_do_ioctl(struct net_device *dev, result = 0; goto bail; } else if (cmd != P80211_IFREQ) { - result = -ENOSYS; + result = -EINVAL; goto bail; } -- 1.9.1