From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from pb-smtp20.pobox.com (pb-smtp20.pobox.com [173.228.157.52]) (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 3A13A15B6 for ; Fri, 7 Jul 2023 02:57:03 +0000 (UTC) Received: from pb-smtp20.pobox.com (unknown [127.0.0.1]) by pb-smtp20.pobox.com (Postfix) with ESMTP id EE0991EE98; Thu, 6 Jul 2023 22:56:56 -0400 (EDT) (envelope-from tdavies@darkphysics.net) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed; d=pobox.com; h=date:from :to:cc:subject:message-id:references:mime-version:content-type :in-reply-to; s=sasl; bh=cgjU9i8w9xVOWqY/GY+HGWDhllAo0BsBJwnJhyP v1nI=; b=wYeK/Yfh4JlKq4ehry//EuT4y4hLygrRxGaB7H/r9YdXA34Q44wev+d Ja4//Hr/Zqr0V5G+HqQ2ox4b3ABoBPd0Y5nPElXmEdaWunQus/pBFDW+2SbeZwAM MLCjB93130CZNPxg7McVjIsfv0aek/Qi2tdSIEYtcyb48MhDh8rw= Received: from pb-smtp20.sea.icgroup.com (unknown [127.0.0.1]) by pb-smtp20.pobox.com (Postfix) with ESMTP id DA8A91EE97; Thu, 6 Jul 2023 22:56:56 -0400 (EDT) (envelope-from tdavies@darkphysics.net) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed; d=darkphysics.net; h=date:from:to:cc:subject:message-id:references:mime-version:content-type:in-reply-to; s=2019-09.pbsmtp; bh=cgjU9i8w9xVOWqY/GY+HGWDhllAo0BsBJwnJhyPv1nI=; b=i8MzwwVW3+e16fyVvyC0ABmrwg/pwVjbwD2wr17o57NBnoiKEJ3FPwWegSMfXCKe0xS6JfZkqDfheQf4otBnjkAxkhHrXrMDGCCnW/a2nQW06Ol/ljfWGnII51qtJacvOxYohNu2xpGYUFNoAeFSe0zoe1pdHj8FJsGC8utJtww= Received: from kimchi.darkphysics (unknown [76.146.178.2]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by pb-smtp20.pobox.com (Postfix) with ESMTPSA id 568161EE96; Thu, 6 Jul 2023 22:56:53 -0400 (EDT) (envelope-from tdavies@darkphysics.net) Date: Thu, 6 Jul 2023 19:56:47 -0700 From: Tree Davies To: Philipp Hortmann Cc: gregkh@linuxfoundation.org, dan.carpenter@linaro.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH] staging: rtl8192e: Rename variable bCurrentHTSupport Message-ID: References: <6dee04ae-4129-cd7c-0d31-70d29b86c8ff@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: <6dee04ae-4129-cd7c-0d31-70d29b86c8ff@gmail.com> X-Pobox-Relay-ID: ED5C24BE-1C71-11EE-ABD9-C2DA088D43B2-45285927!pb-smtp20.pobox.com On Thu, Jul 06, 2023 at 10:48:56PM +0200, Philipp Hortmann wrote: > On 7/6/23 03:59, Tree Davies wrote: > > This patch renames variable bCurrentHTSupport to bcurrent_ht_support > > to fix checkpatch warning Avoid CamelCase. > > > > Signed-off-by: Tree Davies > > --- > > drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 2 +- > > drivers/staging/rtl8192e/rtl8192e/rtl_wx.c | 2 +- > > drivers/staging/rtl8192e/rtl819x_BAProc.c | 12 ++++++------ > > drivers/staging/rtl8192e/rtl819x_HT.h | 2 +- > > drivers/staging/rtl8192e/rtl819x_HTProc.c | 16 ++++++++-------- > > drivers/staging/rtl8192e/rtllib_softmac.c | 16 ++++++++-------- > > drivers/staging/rtl8192e/rtllib_softmac_wx.c | 2 +- > > drivers/staging/rtl8192e/rtllib_tx.c | 8 ++++---- > > 8 files changed, 30 insertions(+), 30 deletions(-) > > > Hi Tree, > > when I try to apply your patch on the top of all the other patches I get the > following error message: > > kernel@matrix-ESPRIMO-P710:~/Documents/git/kernels/staging$ git apply -v > ~/Downloads/20230706-\[PATCH\]\ staging_\ rtl8192e_\ Rename\ variable\ > bCurrentHTSupport-10002.txt > Checking patch drivers/staging/rtl8192e/rtl8192e/rtl_core.c... > Checking patch drivers/staging/rtl8192e/rtl8192e/rtl_wx.c... > Checking patch drivers/staging/rtl8192e/rtl819x_BAProc.c... > Checking patch drivers/staging/rtl8192e/rtl819x_HT.h... > Checking patch drivers/staging/rtl8192e/rtl819x_HTProc.c... > Checking patch drivers/staging/rtl8192e/rtllib_softmac.c... > error: while searching for: > crypt = ieee->crypt_info.crypt[ieee->crypt_info.tx_keyidx]; > encrypt = ieee->host_encrypt && crypt && crypt->ops && > ((strcmp(crypt->ops->name, "R-WEP") == 0 || wpa_ie_len)); > if (ieee->ht_info->bCurrentHTSupport) { > tmp_ht_cap_buf = (u8 *)&(ieee->ht_info->SelfHTCap); > tmp_ht_cap_len = sizeof(ieee->ht_info->SelfHTCap); > tmp_ht_info_buf = (u8 *)&(ieee->ht_info->SelfHTInfo); > > error: patch failed: drivers/staging/rtl8192e/rtllib_softmac.c:816 > error: drivers/staging/rtl8192e/rtllib_softmac.c: patch does not apply > Checking patch drivers/staging/rtl8192e/rtllib_softmac_wx.c... > Checking patch drivers/staging/rtl8192e/rtllib_tx.c... > > > My be the previous send patches are not taken then your chance is better > that this one will fit. Will see what happens. > > > You need to put your patches into a patch series. > This reduces email traffic as the reviewer can just send one email and does > not need to send four. > > A possible command for a patch series is looking like this: > git format-patch -o ~/Documents/kernel/patches/ --cover-letter -n > --thread=shallow --to="Greg Kroah-Hartman ,linux-staging@lists.linux.dev,linux-kernel@vger.kernel.org" > 4bbbd60d84e15fdb7bffde98a687ed168a4dfbbd^..HEAD > > Use the git commit id of your first patch. > > Bye Philipp > Thanks Philipp, I am happy to send a v2 if needed. Let me know... Cheers! Tree