From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ezequiel Garcia Subject: Re: [PATCH 2/6] arm: fix compiler warning by properly inlining flat_set_persistent() Date: Tue, 3 Jul 2012 15:53:17 -0300 Message-ID: References: <1339229611-2326-1-git-send-email-elezegarcia@gmail.com> <1339229611-2326-2-git-send-email-elezegarcia@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from mail-yx0-f174.google.com ([209.85.213.174]:59250 "EHLO mail-yx0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756489Ab2GCSxR convert rfc822-to-8bit (ORCPT ); Tue, 3 Jul 2012 14:53:17 -0400 In-Reply-To: <1339229611-2326-2-git-send-email-elezegarcia@gmail.com> Sender: linux-arch-owner@vger.kernel.org List-ID: To: linux-kernel@vger.kernel.org Cc: linux-arch@vger.kernel.org, Ezequiel Garcia , Russell King , linux-arm-kernel@lists.infradead.org Hi, On Sat, Jun 9, 2012 at 5:13 AM, Ezequiel Garcia = wrote: > This patch removes the following warning: > fs/binfmt_flat.c:752: warning: unused variable =91persistent=92 > There is neither change in functionality, nor extra code generated. > > Cc: Russell King > Cc: > Signed-off-by: Ezequiel Garcia > --- > arch/arm/include/asm/flat.h | 7 ++++++- > 1 files changed, 6 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/include/asm/flat.h b/arch/arm/include/asm/flat.= h > index 59426a4..6ef6cc5 100644 > --- a/arch/arm/include/asm/flat.h > +++ b/arch/arm/include/asm/flat.h > @@ -11,6 +11,11 @@ > #define flat_get_addr_from_rp(rp, relval, flags, persistent) = get_unaligned(rp) > #define flat_put_addr_at_rp(rp, val, relval) put_unaligned= (val,rp) > #define flat_get_relocate_addr(rel) (rel) > -#define flat_set_persistent(relval, p) 0 > + > +static inline int flat_set_persistent(unsigned long relval, > + unsigned long *persistent) > +{ > + return 0; > +} > > #endif /* __ARM_FLAT_H__ */ > -- > 1.7.3.4 > Is there something wrong with this patch preventing it from being appli= ed? Or did I send it to wrong list? Thanks, Ezequiel.