From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff Garzik Subject: Re: [PATCH 06/11] ata: sparse fixes for pata_amd.c Date: Fri, 15 Feb 2008 13:52:36 -0500 Message-ID: <47B5DF74.3020801@garzik.org> References: <3ad7c41048d88a3f8aad098570b9fac9ff719aff.1202965399.git.harvey.harrison@gmail.com> <1202966060.2748.20.camel@brick> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from srv5.dvmed.net ([207.36.208.214]:54673 "EHLO mail.dvmed.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1761068AbYBOSwi (ORCPT ); Fri, 15 Feb 2008 13:52:38 -0500 In-Reply-To: <1202966060.2748.20.camel@brick> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Harvey Harrison Cc: Tejun Heo , LKML , linux-ide Harvey Harrison wrote: > drop return statement. > drivers/ata/pata_amd.c:149:2: warning: returning void-valued expression > > Commit ce54d1616302117fa98513ae916bb3333e1c02ea pata_amd: update mode selection for NV PATAs > > added the initializer for nv_mode_filter but missed deleting the previously > set mode_filter > > drivers/ata/pata_amd.c:509:3: warning: Initializer entry defined twice > drivers/ata/pata_amd.c:521:3: also defined here > drivers/ata/pata_amd.c:544:3: warning: Initializer entry defined twice > drivers/ata/pata_amd.c:556:3: also defined here > > Signed-off-by: Harvey Harrison > --- > drivers/ata/pata_amd.c | 7 ++----- > 1 files changed, 2 insertions(+), 5 deletions(-) applied 6-10