From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 40E337C for ; Sun, 28 May 2023 16:01:48 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 800BBC433D2; Sun, 28 May 2023 16:01:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1685289707; bh=ZSx3cN26RRXAFdFdF6AibUO0wV80iZxnw1MftFAxYoE=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=QWmKjD0ImUXOQOJZndc2nsL9yflp+kIBQScghbG00CjSP1sd+jis59pMnjl9ZfGxS ozkQeFLtPEz59OVAa75zMevyBb1Ghy/bRcnKEj9OeoOEstcZD6wFiuIoCb05I8cslc oqzTaEzgAKuWKU3h5wWRY+SgIdA41k8LLidlqPtw= Date: Sun, 28 May 2023 17:01:45 +0100 From: Greg Kroah-Hartman To: Yogesh Hegde Cc: linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/4] staging: rtl8192e: Rename variable SetWirelessMode Message-ID: <2023052837-half-pretended-e024@gregkh> References: <5a1ee0e9ef53ebe6085c14dba08c438862ec992e.1685284727.git.yogi.kernel@gmail.com> Precedence: bulk X-Mailing-List: linux-staging@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5a1ee0e9ef53ebe6085c14dba08c438862ec992e.1685284727.git.yogi.kernel@gmail.com> On Sun, May 28, 2023 at 08:21:33PM +0530, Yogesh Hegde wrote: > Rename variable SetWirelessMode to set_wireless_mode to avoid > CamelCase which is not accepted by checkpatch. > > Signed-off-by: Yogesh Hegde > --- > drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 2 +- > drivers/staging/rtl8192e/rtllib.h | 2 +- > drivers/staging/rtl8192e/rtllib_softmac.c | 12 ++++++------ > 3 files changed, 8 insertions(+), 8 deletions(-) > > diff --git a/drivers/staging/rtl8192e/rtl8192e/rtl_core.c b/drivers/staging/rtl8192e/rtl8192e/rtl_core.c > index 17b70dde7eeb..699c4accf2db 100644 > --- a/drivers/staging/rtl8192e/rtl8192e/rtl_core.c > +++ b/drivers/staging/rtl8192e/rtl8192e/rtl_core.c > @@ -716,7 +716,7 @@ static void _rtl92e_init_priv_handler(struct net_device *dev) > priv->rtllib->check_nic_enough_desc = _rtl92e_check_nic_enough_desc; > priv->rtllib->handle_assoc_response = _rtl92e_handle_assoc_response; > priv->rtllib->handle_beacon = _rtl92e_handle_beacon; > - priv->rtllib->SetWirelessMode = rtl92e_set_wireless_mode; > + priv->rtllib->set_wireless_mode = rtl92e_set_wireless_mode; As this is only set in one place, why is it needed at all? Why not just call the real function instead? Same for the other patches in this series, why not turn this into a set of cleanup patches removing the unnecessary indirection and removing the CamelCase variables entirely? thanks, greg k-h