From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tejun Heo Subject: Re: [PATCH] ata: pata_artop: remove redundant initialization of pio Date: Mon, 18 Sep 2017 20:24:52 -0700 Message-ID: <20170919032452.GN378890@devbig577.frc2.facebook.com> References: <20170914185418.13790-1-colin.king@canonical.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20170914185418.13790-1-colin.king@canonical.com> Sender: linux-kernel-owner@vger.kernel.org To: Colin King Cc: Bartlomiej Zolnierkiewicz , linux-ide@vger.kernel.org, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: linux-ide@vger.kernel.org On Thu, Sep 14, 2017 at 07:54:18PM +0100, Colin King wrote: > From: Colin Ian King > > pio is initialized and the data is never read, instead it is almost > immediately being updated to a new value. Fix this by removing the > initialization. > > Detected by scan-build: > "warning: Value stored to 'pio' during its initialization is never read" > > Fixes: 669a5db411d8 ("[libata] Add a bunch of PATA drivers") > Signed-off-by: Colin Ian King Applied to libata/for-4.15. Thanks. -- tejun