From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Simon Horman <simon@horms.net>
Cc: kernel test robot <lkp@intel.com>,
devel@driverdev.osuosl.org, llvm@lists.linux.dev,
oe-kbuild-all@lists.linux.dev
Subject: Re: [staging:staging-testing 36/53] drivers/staging/rtl8192e/rtllib_crypt_tkip.c:640:7: warning: variable 'iv16' set but not used
Date: Sat, 24 Aug 2024 08:07:33 +0200 [thread overview]
Message-ID: <2024082424-crescent-delta-7191@gregkh> (raw)
In-Reply-To: <20240823172749.GA2164@vergenet.net>
On Fri, Aug 23, 2024 at 06:27:49PM +0100, Simon Horman wrote:
> On Fri, Aug 23, 2024 at 06:16:49PM +0100, Simon Horman wrote:
> > On Fri, Aug 23, 2024 at 08:35:15PM +0800, kernel test robot wrote:
> > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing
> > > head: 5315266844ea3b0b8b6be9842d5901e439fa838a
> > > commit: 5f1a6826ea4900f8540d5eeb29f97796860f2d08 [36/53] staging: rtl8192e: remove set but otherwise unused local variable iv32
> > > config: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20240823/202408232049.UJef268y-lkp@intel.com/config)
> > > compiler: clang version 18.1.5 (https://github.com/llvm/llvm-project 617a15a9eac96088ae5e9134248d8236e34b91b1)
> > > reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240823/202408232049.UJef268y-lkp@intel.com/reproduce)
> > >
> > > If you fix the issue in a separate patch/commit (i.e. not just a new version of
> > > the same patch/commit), kindly add following tags
> > > | Reported-by: kernel test robot <lkp@intel.com>
> > > | Closes: https://lore.kernel.org/oe-kbuild-all/202408232049.UJef268y-lkp@intel.com/
> > >
> > > All warnings (new ones prefixed by >>):
> > >
> > > >> drivers/staging/rtl8192e/rtllib_crypt_tkip.c:640:7: warning: variable 'iv16' set but not used [-Wunused-but-set-variable]
> > > 640 | u16 iv16 = tkey->tx_iv16;
> > > | ^
> > > 1 warning generated.
> >
> > Sorry about this.
> >
> > It seems that my patch, cited above, which removed a set but otherwise
> > unused variable results in iv16 being set but otherwise unused.
> >
> > I'll prepare a follow-up patch to address this.
>
> Patch is here:
>
> - [PATCH] staging: rtl8192e: remove set but otherwise unused local variable iv16
> https://lore.kernel.org/linux-staging/20240823-rtl8192e-iv16-v1-1-000702673065@kernel.org/T/#u
Now applied, thanks for the quick response!
prev parent reply other threads:[~2024-08-24 6:07 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-23 12:35 [staging:staging-testing 36/53] drivers/staging/rtl8192e/rtllib_crypt_tkip.c:640:7: warning: variable 'iv16' set but not used kernel test robot
2024-08-23 17:16 ` Simon Horman
2024-08-23 17:27 ` Simon Horman
2024-08-24 6:07 ` Greg Kroah-Hartman [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2024082424-crescent-delta-7191@gregkh \
--to=gregkh@linuxfoundation.org \
--cc=devel@driverdev.osuosl.org \
--cc=lkp@intel.com \
--cc=llvm@lists.linux.dev \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=simon@horms.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.