LKML Archive on lore.kernel.org help / color / mirror / Atom feed
From: David Rientjes <rientjes@google.com> To: Andrew Morton <akpm@linux-foundation.org> Cc: Paul Mundt <lethal@linux-sh.org>, linux-kernel@vger.kernel.org Subject: [patch 03/10] sh: dma: use __attribute_unused__ Date: Tue, 1 May 2007 21:28:56 -0700 (PDT) [thread overview] Message-ID: <Pine.LNX.4.64.0705012120130.19277@chino.kir.corp.google.com> (raw) In-Reply-To: <Pine.LNX.4.64.0705012119370.19277@chino.kir.corp.google.com> There is no such thing as labeling a variable as __attribute__((used)). Since ts_shift is not referenced in inline assembly, we assume that we're simply suppressing a warning here if the variable is declared but unreferenced. Cc: Paul Mundt <lethal@linux-sh.org> Signed-off-by: David Rientjes <rientjes@google.com> --- include/asm-sh/cpu-sh3/dma.h | 2 +- include/asm-sh/cpu-sh4/dma-sh7780.h | 2 +- include/asm-sh/cpu-sh4/dma.h | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/include/asm-sh/cpu-sh3/dma.h b/include/asm-sh/cpu-sh3/dma.h --- a/include/asm-sh/cpu-sh3/dma.h +++ b/include/asm-sh/cpu-sh3/dma.h @@ -26,7 +26,7 @@ enum { XMIT_SZ_128BIT, }; -static unsigned int ts_shift[] __attribute__ ((used)) = { +static unsigned int ts_shift[] __attribute_unused__ = { [XMIT_SZ_8BIT] = 0, [XMIT_SZ_16BIT] = 1, [XMIT_SZ_32BIT] = 2, diff --git a/include/asm-sh/cpu-sh4/dma-sh7780.h b/include/asm-sh/cpu-sh4/dma-sh7780.h --- a/include/asm-sh/cpu-sh4/dma-sh7780.h +++ b/include/asm-sh/cpu-sh4/dma-sh7780.h @@ -28,7 +28,7 @@ enum { /* * The DMA count is defined as the number of bytes to transfer. */ -static unsigned int __attribute__ ((used)) ts_shift[] = { +static unsigned int ts_shift[] __attribute_unused__ = { [XMIT_SZ_8BIT] = 0, [XMIT_SZ_16BIT] = 1, [XMIT_SZ_32BIT] = 2, diff --git a/include/asm-sh/cpu-sh4/dma.h b/include/asm-sh/cpu-sh4/dma.h --- a/include/asm-sh/cpu-sh4/dma.h +++ b/include/asm-sh/cpu-sh4/dma.h @@ -53,7 +53,7 @@ enum { /* * The DMA count is defined as the number of bytes to transfer. */ -static unsigned int ts_shift[] __attribute__ ((used)) = { +static unsigned int ts_shift[] __attribute_unused__ = { [XMIT_SZ_64BIT] = 3, [XMIT_SZ_8BIT] = 0, [XMIT_SZ_16BIT] = 1,
next prev parent reply other threads:[~2007-05-02 4:30 UTC|newest] Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top 2007-05-02 4:28 [patch 01/10] compiler: define __attribute_unused__ David Rientjes 2007-05-02 4:28 ` [patch 02/10] i386 pci: type may be unused David Rientjes 2007-05-02 4:28 ` David Rientjes [this message] 2007-05-02 4:28 ` [patch 04/10] scsi: fix ambiguous gdthtable definition David Rientjes 2007-05-02 4:28 ` [patch 05/10] frv: gdb: use __attribute_unused__ David Rientjes 2007-05-02 4:28 ` [patch 06/10] i386: voyager: " David Rientjes 2007-05-02 4:28 ` [patch 07/10] mips: excite: " David Rientjes 2007-05-02 4:28 ` [patch 08/10] mips: tlbex: " David Rientjes 2007-05-02 4:28 ` [patch 09/10] powerpc: ps3: " David Rientjes 2007-05-02 4:28 ` [patch 10/10] i386 mmzone: " David Rientjes 2007-05-02 15:42 ` [patch 09/10] powerpc: ps3: " Geoff Levand 2007-05-03 17:23 ` [patch 08/10] mips: tlbex: " Ralf Baechle 2007-05-03 17:23 ` [patch 07/10] mips: excite: " Ralf Baechle 2007-05-07 1:28 ` [patch 03/10] sh: dma: " Paul Mundt 2007-05-02 10:35 ` [patch 02/10] i386 pci: type may be unused Andi Kleen 2007-05-02 17:05 ` David Rientjes 2007-05-02 5:17 ` [patch 01/10] compiler: define __attribute_unused__ Alexey Dobriyan 2007-05-02 5:53 ` David Rientjes 2007-05-02 6:08 ` Andrew Morton 2007-05-02 6:46 ` Nick Piggin 2007-05-02 6:33 ` WANG Cong 2007-05-02 7:02 ` Robert P. J. Day 2007-05-02 7:17 ` David Rientjes 2007-05-02 7:47 ` Robert P. J. Day 2007-05-02 5:40 ` Rusty Russell 2007-05-02 6:06 ` David Rientjes 2007-05-02 6:25 ` David Rientjes 2007-05-02 6:29 ` Rusty Russell 2007-05-02 6:41 ` David Rientjes 2007-05-02 6:52 ` Andrew Morton 2007-05-02 7:04 ` Rusty Russell 2007-05-02 7:22 ` David Rientjes 2007-05-02 7:51 ` Rusty Russell 2007-05-02 14:55 ` Adrian Bunk 2007-05-02 15:05 ` Adrian Bunk 2007-05-02 17:16 ` David Rientjes 2007-05-03 17:51 ` Adrian Bunk 2007-05-03 18:56 ` David Rientjes 2007-05-03 19:05 ` Adrian Bunk 2007-05-03 19:24 ` David Rientjes
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=Pine.LNX.4.64.0705012120130.19277@chino.kir.corp.google.com \ --to=rientjes@google.com \ --cc=akpm@linux-foundation.org \ --cc=lethal@linux-sh.org \ --cc=linux-kernel@vger.kernel.org \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).