Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Feedback-ID: i56a14606:Fastmail
User-Agent: Cyrus-JMAP/3.7.0-alpha0-935-ge4ccd4c47b-fm-20220914.001-ge4ccd4c4
Mime-Version: 1.0
Message-Id: <ae8041cd-0f20-47e9-a4be-92159785e8a8@www.fastmail.com>
In-Reply-To: <YyxzrjA7hW6gdV5e@owl.dominikbrodowski.net>
References: <20220725012955.4928-1-jrdr.linux@gmail.com>
 <YyxzrjA7hW6gdV5e@owl.dominikbrodowski.net>
Date:   Fri, 23 Sep 2022 08:58:47 +0200
From:   "Arnd Bergmann" <arnd@arndb.de>
To:     "Dominik Brodowski" <linux@dominikbrodowski.net>,
        "Souptick Joarder" <jrdr.linux@gmail.com>
Cc:     "Tony Lindgren" <tony@atomide.com>, linux-kernel@vger.kernel.org,
        "kernel test robot" <lkp@intel.com>
Subject: Re: [PATCH] pcmcia: Removed unused variable control.
Content-Type: text/plain
Precedence: bulk

On Thu, Sep 22, 2022, at 4:39 PM, Dominik Brodowski wrote:
> Am Mon, Jul 25, 2022 at 06:59:55AM +0530 schrieb Souptick Joarder:
>> From: "Souptick Joarder (HPE)" <jrdr.linux@gmail.com>
>> 
>> Kernel test robot throws below warning ->
>> drivers/pcmcia/omap_cf.c:127:7: warning: variable 'control'
>> set but not used [-Wunused-but-set-variable]
>> 
>> Removed unused variable control.
>
> From a code-generation standpoint, this is obiously correct, and probably
> the compiler removes that omap_readw() already.

omap_readw() is an 'extern' function, so it does not get removed.

Even if it was inline, it's still "and volatile" read that gets
left in because it may (and probably does) have side-effects.

> But, to be honest, I'm not
> perfectly sure on what the device expects -- and whether it's required to
> write back the control register (or parts of it). Does anyone still have the
> hardware (or specs)? If not, I'm tempted to apply this patch on the basis
> that the compiler might remove that omap_readw() anyway.

I think assigning to an unused variable was at some point needed to
avoid a compiler warning because otherwise omap_readw() was just a
pointer dereference. The safe way to transform the code should be to
drop the variable but leave the function call.

Note that the driver is only used on the 'osk' reference board,
not on any devices that were in mass-production.

     Arnd