From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 331FBC4167B for ; Sun, 26 Nov 2023 10:29:23 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229644AbjKZK3O (ORCPT ); Sun, 26 Nov 2023 05:29:14 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47756 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229437AbjKZK3M (ORCPT ); Sun, 26 Nov 2023 05:29:12 -0500 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 67F5FFD for ; Sun, 26 Nov 2023 02:29:19 -0800 (PST) Received: by smtp.kernel.org (Postfix) with ESMTPSA id B637FC433C8; Sun, 26 Nov 2023 10:29:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1700994559; bh=f+nbJIhiGBaPFaiV+IIHo2OURbKlwrFmj3SbvRtEevY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=CE+FzQ2TQ/1pNSPO/OWVj8T5lKg0890qO7mvXIZoMLNwjFMKmxLs5W/Y0x3OZvyve FiQuY8DYfp60px5dTcn6Ecrg9mdBHMLPEuEgN0SeUnLC0ZIg7IeAjp9Z/jPzv17rV8 rXlA3FIyUKom5vwYJqa2gn7bcmldRJ47uXsJx414= Date: Sun, 26 Nov 2023 10:29:16 +0000 From: Greg KH To: Gary Rookard Cc: philipp.g.hortmann@gmail.com, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH 4/5] staging: rtl8192e: renamed variable nDataRate Message-ID: <2023112622-jigsaw-scrambler-d385@gregkh> References: <20231125223432.13780-1-garyrookard@fastmail.org> <20231125223432.13780-5-garyrookard@fastmail.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20231125223432.13780-5-garyrookard@fastmail.org> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Nov 25, 2023 at 05:34:31PM -0500, Gary Rookard wrote: > -u16 tx_count_to_data_rate(struct rtllib_device *ieee, u8 nDataRate) > +u16 tx_count_to_data_rate(struct rtllib_device *ieee, u8 n_data_rate) Why are you keeping the "n"? It's not needed, right? Remember, these were written in "Hungarian notation" which uses the variable name to denote the type of the variable, so "n" means "number" perhaps? So it can be dropped. thanks, greg k-h