From: Paul Gortmaker <paul.gortmaker@windriver.com>
To: Mike Frysinger <vapier.adi@gmail.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
mingo@elte.hu, linux-kernel@vger.kernel.org,
akpm@linux-foundation.org, linux-arch@vger.kernel.org,
sfr@canb.auug.org.au
Subject: Re: [RFC/PULL 00/11] introduce export.h; reduce module.h usage
Date: Wed, 10 Aug 2011 15:09:29 -0400 [thread overview]
Message-ID: <4E42D769.8030704@windriver.com> (raw)
In-Reply-To: <CAMjpGUcOagn-o_mETLDr-=pa-3sxq_iDjF=A8OGhdJg5+9ewpA@mail.gmail.com>
On 11-08-07 10:42 PM, Mike Frysinger wrote:
[...]
>
> Blackfin is broken because of this. a simple grep of the tree
I've run all the in-tree blackfin configs, so it is all good.
Have you thought about getting at least one blackfin defconfig in
the linux-next build coverage? Gives you a better chance of
having people see when they've broken it.
Thanks,
Paul.
> probably could pick this up. maybe something like:
> grep EXPORT_SYMBOL * -rIl | xargs grep -L linux/export.h
>
> from my logs, i see at least:
> arch/blackfin/kernel/dma-mapping.c:103: warning: type defaults to
> 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/kernel/dma-mapping.c:111: warning: type defaults to
> 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/kernel/dma-mapping.c:121: warning: type defaults to
> 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/kernel/dma-mapping.c:136: warning: type defaults to
> 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/kernel/dma-mapping.c:148: warning: type defaults to
> 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/kernel/sys_bfin.c:50: warning: type defaults to 'int' in
> declaration of 'EXPORT_SYMBOL'
> arch/blackfin/mach-bf518/boards/ezbrd.c:825: warning: type defaults to
> 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/mach-bf527/boards/ad7160eval.c:852: warning: type
> defaults to 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/mach-bf527/boards/ezbrd.c:876: warning: type defaults to
> 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/mach-bf527/boards/ezkit.c:1317: warning: type defaults
> to 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/mach-bf527/boards/tll6527m.c:938: warning: type defaults
> to 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/mach-bf537/boards/dnp5370.c:396: warning: type defaults
> to 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/mm/init.c:22: warning: type defaults to 'int' in
> declaration of 'EXPORT_SYMBOL'
> arch/blackfin/mm/init.c:32: warning: type defaults to 'int' in
> declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:1002: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:107: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:1089: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:124: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:271: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:293: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:353: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:373: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:451: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:515: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:591: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:599: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:610: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:621: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:775: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:786: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:797: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:90: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:968: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> -mike
WARNING: multiple messages have this Message-ID (diff)
From: Paul Gortmaker <paul.gortmaker@windriver.com>
To: Mike Frysinger <vapier.adi@gmail.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>, <mingo@elte.hu>,
<linux-kernel@vger.kernel.org>, <akpm@linux-foundation.org>,
<linux-arch@vger.kernel.org>, <sfr@canb.auug.org.au>
Subject: Re: [RFC/PULL 00/11] introduce export.h; reduce module.h usage
Date: Wed, 10 Aug 2011 15:09:29 -0400 [thread overview]
Message-ID: <4E42D769.8030704@windriver.com> (raw)
In-Reply-To: <CAMjpGUcOagn-o_mETLDr-=pa-3sxq_iDjF=A8OGhdJg5+9ewpA@mail.gmail.com>
On 11-08-07 10:42 PM, Mike Frysinger wrote:
[...]
>
> Blackfin is broken because of this. a simple grep of the tree
I've run all the in-tree blackfin configs, so it is all good.
Have you thought about getting at least one blackfin defconfig in
the linux-next build coverage? Gives you a better chance of
having people see when they've broken it.
Thanks,
Paul.
> probably could pick this up. maybe something like:
> grep EXPORT_SYMBOL * -rIl | xargs grep -L linux/export.h
>
> from my logs, i see at least:
> arch/blackfin/kernel/dma-mapping.c:103: warning: type defaults to
> 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/kernel/dma-mapping.c:111: warning: type defaults to
> 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/kernel/dma-mapping.c:121: warning: type defaults to
> 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/kernel/dma-mapping.c:136: warning: type defaults to
> 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/kernel/dma-mapping.c:148: warning: type defaults to
> 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/kernel/sys_bfin.c:50: warning: type defaults to 'int' in
> declaration of 'EXPORT_SYMBOL'
> arch/blackfin/mach-bf518/boards/ezbrd.c:825: warning: type defaults to
> 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/mach-bf527/boards/ad7160eval.c:852: warning: type
> defaults to 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/mach-bf527/boards/ezbrd.c:876: warning: type defaults to
> 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/mach-bf527/boards/ezkit.c:1317: warning: type defaults
> to 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/mach-bf527/boards/tll6527m.c:938: warning: type defaults
> to 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/mach-bf537/boards/dnp5370.c:396: warning: type defaults
> to 'int' in declaration of 'EXPORT_SYMBOL'
> arch/blackfin/mm/init.c:22: warning: type defaults to 'int' in
> declaration of 'EXPORT_SYMBOL'
> arch/blackfin/mm/init.c:32: warning: type defaults to 'int' in
> declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:1002: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:107: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:1089: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:124: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:271: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:293: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:353: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:373: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:451: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:515: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:591: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:599: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:610: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:621: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:775: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:786: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:797: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:90: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> sound/soc/blackfin/bf5xx-sport.c:968: warning: type defaults to 'int'
> in declaration of 'EXPORT_SYMBOL'
> -mike
next prev parent reply other threads:[~2011-08-10 19:10 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-28 5:16 [RFC/PULL 00/11] introduce export.h; reduce module.h usage Paul Gortmaker
2011-07-28 5:16 ` [PATCH 01/11] module.h: split out the EXPORT_SYMBOL for faster compiles Paul Gortmaker
2011-07-28 5:16 ` [PATCH 02/11] sysdev.h: dont include <linux/module.h> for no reason Paul Gortmaker
2011-07-28 5:16 ` [PATCH 03/11] net: inet_timewait_sock doesnt need <linux/module.h> Paul Gortmaker
2011-07-28 5:16 ` [PATCH 04/11] device_cgroup.h: delete needless include <linux/module.h> Paul Gortmaker
2011-07-28 5:16 ` [PATCH 05/11] crypto.h: remove unused crypto_tfm_alg_modname() inline Paul Gortmaker
2011-07-28 5:16 ` [PATCH 06/11] linux/stop_machine.h: fix implicit use of smp.h for smp_processor_id Paul Gortmaker
2011-07-28 5:16 ` [PATCH 07/11] pm_runtime.h: explicitly requires notifier.h Paul Gortmaker
2011-07-28 5:16 ` [PATCH 08/11] uwb.h: fix implicit use of asm/page.h for PAGE_SIZE Paul Gortmaker
2011-07-28 5:16 ` [PATCH 09/11] net: sch_generic remove redundant use of <linux/module.h> Paul Gortmaker
2011-07-28 5:16 ` [PATCH 10/11] include: convert various register fcns to macros to avoid include chaining Paul Gortmaker
2011-07-28 5:16 ` [PATCH 11/11] include: replace linux/module.h with "struct module" wherever possible Paul Gortmaker
2011-07-28 7:15 ` [RFC/PULL 00/11] introduce export.h; reduce module.h usage Ingo Molnar
2011-07-28 15:42 ` Paul Gortmaker
2011-07-28 15:42 ` Paul Gortmaker
2011-07-29 1:20 ` Stephen Rothwell
2011-07-29 1:20 ` Stephen Rothwell
2011-08-01 11:49 ` Geert Uytterhoeven
2011-08-02 5:02 ` Paul Gortmaker
2011-08-02 5:02 ` Paul Gortmaker
2011-08-02 6:54 ` Geert Uytterhoeven
2011-07-28 9:40 ` Anca Emanuel
2011-07-28 13:19 ` Linus Torvalds
2011-07-28 15:27 ` Paul Gortmaker
2011-07-28 15:27 ` Paul Gortmaker
2011-07-28 19:18 ` Ingo Molnar
2011-08-02 18:30 ` Paul Gortmaker
2011-08-02 18:30 ` Paul Gortmaker
2011-08-08 2:42 ` Mike Frysinger
2011-08-10 19:09 ` Paul Gortmaker [this message]
2011-08-10 19:09 ` Paul Gortmaker
2011-08-11 4:34 ` Mike Frysinger
2011-08-11 13:10 ` Paul Gortmaker
2011-08-11 13:10 ` Paul Gortmaker
2011-08-11 17:15 ` Mike Frysinger
2011-08-11 17:20 ` Paul Gortmaker
2011-08-11 17:20 ` Paul Gortmaker
2011-10-13 2:07 ` Rusty Russell
2011-10-13 21:51 ` Paul Gortmaker
2011-10-13 21:51 ` Paul Gortmaker
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=4E42D769.8030704@windriver.com \
--to=paul.gortmaker@windriver.com \
--cc=akpm@linux-foundation.org \
--cc=linux-arch@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=sfr@canb.auug.org.au \
--cc=torvalds@linux-foundation.org \
--cc=vapier.adi@gmail.com \
/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.