From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 01/16] ARM: warnings in arch/arm/include/asm/uaccess.h
Date: Tue, 9 Oct 2012 12:08:54 +0000 [thread overview]
Message-ID: <201210091208.55090.arnd@arndb.de> (raw)
In-Reply-To: <50726966.5010503@snapgear.com>
On Monday 08 October 2012, Greg Ungerer wrote:
> On 06/10/12 00:55, Arnd Bergmann wrote:
> > On NOMMU ARM, the __addr_ok() and __range_ok() macros do not evaluate
> > their arguments, which may lead to harmless build warnings in some
> > code where the variables are not used otherwise. Adding a cast to void
> > gets rid of the warning and does not make any semantic changes.
> >
> > Without this patch, building at91x40_defconfig results in:
> >
> > fs/read_write.c: In function 'rw_copy_check_uvector':
> > fs/read_write.c:684:9: warning: unused variable 'buf' [-Wunused-variable]
> >
> > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> > Cc: Greg Ungerer <gerg@uclinux.org>
> > Cc: Russell King <rmk+kernel@arm.linux.org.uk>
>
> Acked-by: Greg Ungerer <gerg@uclinux.org>
Thanks!
> I can pick it up and push to the arm-soc tree.
Well, my idea was that I would prefer the patches go through some other tree
besides arm-soc since they are not really the main purpose for this tree.
The core ARM patches in particular should go through Russell's ARM tree.
I have a few more that come in during the merge window and plan to send
him a pull request for those.
Arnd
WARNING: multiple messages have this Message-ID (diff)
From: Arnd Bergmann <arnd@arndb.de>
To: Greg Ungerer <gerg@snapgear.com>
Cc: linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, arm@kernel.org,
Greg Ungerer <gerg@uclinux.org>,
Russell King <rmk+kernel@arm.linux.org.uk>
Subject: Re: [PATCH 01/16] ARM: warnings in arch/arm/include/asm/uaccess.h
Date: Tue, 9 Oct 2012 12:08:54 +0000 [thread overview]
Message-ID: <201210091208.55090.arnd@arndb.de> (raw)
In-Reply-To: <50726966.5010503@snapgear.com>
On Monday 08 October 2012, Greg Ungerer wrote:
> On 06/10/12 00:55, Arnd Bergmann wrote:
> > On NOMMU ARM, the __addr_ok() and __range_ok() macros do not evaluate
> > their arguments, which may lead to harmless build warnings in some
> > code where the variables are not used otherwise. Adding a cast to void
> > gets rid of the warning and does not make any semantic changes.
> >
> > Without this patch, building at91x40_defconfig results in:
> >
> > fs/read_write.c: In function 'rw_copy_check_uvector':
> > fs/read_write.c:684:9: warning: unused variable 'buf' [-Wunused-variable]
> >
> > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> > Cc: Greg Ungerer <gerg@uclinux.org>
> > Cc: Russell King <rmk+kernel@arm.linux.org.uk>
>
> Acked-by: Greg Ungerer <gerg@uclinux.org>
Thanks!
> I can pick it up and push to the arm-soc tree.
Well, my idea was that I would prefer the patches go through some other tree
besides arm-soc since they are not really the main purpose for this tree.
The core ARM patches in particular should go through Russell's ARM tree.
I have a few more that come in during the merge window and plan to send
him a pull request for those.
Arnd
next prev parent reply other threads:[~2012-10-09 12:08 UTC|newest]
Thread overview: 83+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-05 14:55 [PATCH 00/16] ARM: mostly harmless gcc warnings Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 01/16] ARM: warnings in arch/arm/include/asm/uaccess.h Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-08 5:49 ` Greg Ungerer
2012-10-08 5:49 ` Greg Ungerer
2012-10-09 12:08 ` Arnd Bergmann [this message]
2012-10-09 12:08 ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 02/16] ARM: binfmt_flat: unused variable 'persistent' Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-08 5:50 ` Greg Ungerer
2012-10-08 5:50 ` Greg Ungerer
2012-10-05 14:55 ` [PATCH 03/16] SCSI: ARM: ncr5380/oak uses no interrupts Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 04/16] SCSI: ARM: make fas216_dumpinfo function conditional Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 05/16] vfs: bogus warnings in fs/namei.c Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-08 11:51 ` Jan Kara
2012-10-08 11:51 ` Jan Kara
2012-10-09 12:27 ` Arnd Bergmann
2012-10-09 12:27 ` Arnd Bergmann
2012-10-09 13:07 ` Arnd Bergmann
2012-10-09 13:07 ` Arnd Bergmann
2012-10-09 13:43 ` Jan Kara
2012-10-09 13:43 ` Jan Kara
2012-10-11 4:37 ` Al Viro
2012-10-11 4:37 ` Al Viro
2012-10-11 13:20 ` [PATCH v2] " Arnd Bergmann
2012-10-11 13:20 ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 06/16] mm/slob: use min_t() to compare ARCH_SLAB_MINALIGN Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 16:46 ` Christoph Lameter
2012-10-05 16:46 ` Christoph Lameter
2012-10-31 7:25 ` Pekka Enberg
2012-10-31 7:25 ` Pekka Enberg
2012-10-05 14:55 ` [PATCH 07/16] cgroup: fix warning when building without any subsys Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 15:50 ` Ben Blum
2012-10-05 15:50 ` Ben Blum
2012-10-06 2:14 ` Tejun Heo
2012-10-06 2:14 ` Tejun Heo
2012-10-06 6:19 ` Arnd Bergmann
2012-10-06 6:19 ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 08/16] ipvs: fix ip_vs_set_timeout debug messages Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 20:39 ` Julian Anastasov
2012-10-05 20:39 ` Julian Anastasov
2012-10-06 6:45 ` Arnd Bergmann
2012-10-06 6:45 ` Arnd Bergmann
2012-10-06 8:09 ` Julian Anastasov
2012-10-06 8:09 ` Julian Anastasov
2012-10-06 9:54 ` Arnd Bergmann
2012-10-06 9:54 ` Arnd Bergmann
2012-10-09 1:48 ` Simon Horman
2012-10-09 1:48 ` Simon Horman
2012-10-05 14:55 ` [PATCH 09/16] USB: EHCI: mark ehci_orion_conf_mbus_windows __devinit Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 18:05 ` Alan Stern
2012-10-05 18:05 ` Alan Stern
2012-10-05 14:55 ` [PATCH 10/16] clk: don't mark clkdev_add_table as init Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 11/16] pcmcia: sharpsl: don't discard sharpsl_pcmcia_ops Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 12/16] video: mark nuc900fb_map_video_memory as __devinit Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 13/16] ARM: be really quiet when building with 'make -s' Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 16:36 ` Nicolas Pitre
2012-10-05 16:36 ` Nicolas Pitre
2012-10-05 14:55 ` [PATCH 14/16] ARM: pxa: armcore: fix PCI PIO warnings Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 15:32 ` Igor Grinberg
2012-10-05 15:32 ` Igor Grinberg
2012-10-05 14:55 ` [PATCH 15/16] spi/s3c64xx: use correct dma_transfer_direction type Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 14:55 ` [PATCH 16/16] ARM: pass -marm to gcc by default for both C and assembler Arnd Bergmann
2012-10-05 14:55 ` Arnd Bergmann
2012-10-05 18:42 ` Nicolas Pitre
2012-10-05 18:42 ` Nicolas Pitre
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=201210091208.55090.arnd@arndb.de \
--to=arnd@arndb.de \
--cc=linux-arm-kernel@lists.infradead.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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.