* [wsa:i2c/quirks 10/11] drivers/i2c/busses/i2c-viperboard.c:357:27: sparse: symbol 'vprbrd_quirks' was not declared. Should it be static?
@ 2015-01-08 18:07 kbuild test robot
[not found] ` <201501090239.gPJgrJIE%fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2015-01-08 18:38 ` [wsa:i2c/quirks 10/11] drivers/i2c/busses/i2c-viperboard.c:357:27: sparse: symbol 'vprbrd_quirks' was not declared. Should it be static? Wolfram Sang
0 siblings, 2 replies; 3+ messages in thread
From: kbuild test robot @ 2015-01-08 18:07 UTC (permalink / raw)
To: Wolfram Sang
Cc: kbuild-all-JC7UmRfGjtg, Lars Poeschel,
linux-i2c-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA
tree: git://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux i2c/quirks
head: b1d39af8ae21f4930854b0656aa83a83fd38049e
commit: 5f7099010eda61b777ba84cf9b64dc8428846d25 [10/11] i2c: viperboard: make use of the new infrastructure for quirks
reproduce:
# apt-get install sparse
git checkout 5f7099010eda61b777ba84cf9b64dc8428846d25
make ARCH=x86_64 allmodconfig
make C=1 CF=-D__CHECK_ENDIAN__
sparse warnings: (new ones prefixed by >>)
drivers/i2c/busses/i2c-viperboard.c:128:35: sparse: incorrect type in assignment (different base types)
drivers/i2c/busses/i2c-viperboard.c:128:35: expected unsigned short [unsigned] [usertype] addr
drivers/i2c/busses/i2c-viperboard.c:128:35: got restricted __le16 [usertype] <noident>
drivers/i2c/busses/i2c-viperboard.c:201:34: sparse: incorrect type in assignment (different base types)
drivers/i2c/busses/i2c-viperboard.c:201:34: expected unsigned short [unsigned] [usertype] tf1
drivers/i2c/busses/i2c-viperboard.c:201:34: got restricted __le16 [usertype] <noident>
drivers/i2c/busses/i2c-viperboard.c:202:34: sparse: incorrect type in assignment (different base types)
drivers/i2c/busses/i2c-viperboard.c:202:34: expected unsigned short [unsigned] [usertype] tf2
drivers/i2c/busses/i2c-viperboard.c:202:34: got restricted __le16 [usertype] <noident>
drivers/i2c/busses/i2c-viperboard.c:237:35: sparse: incorrect type in assignment (different base types)
drivers/i2c/busses/i2c-viperboard.c:237:35: expected unsigned short [unsigned] [usertype] addr
drivers/i2c/busses/i2c-viperboard.c:237:35: got restricted __le16 [usertype] <noident>
drivers/i2c/busses/i2c-viperboard.c:300:35: sparse: incorrect type in assignment (different base types)
drivers/i2c/busses/i2c-viperboard.c:300:35: expected unsigned short [unsigned] [usertype] len
drivers/i2c/busses/i2c-viperboard.c:300:35: got restricted __le16 [usertype] <noident>
drivers/i2c/busses/i2c-viperboard.c:325:35: sparse: incorrect type in assignment (different base types)
drivers/i2c/busses/i2c-viperboard.c:325:35: expected unsigned short [unsigned] [usertype] len
drivers/i2c/busses/i2c-viperboard.c:325:35: got restricted __le16 [usertype] <noident>
>> drivers/i2c/busses/i2c-viperboard.c:357:27: sparse: symbol 'vprbrd_quirks' was not declared. Should it be static?
Please review and possibly fold the followup patch.
---
0-DAY kernel test infrastructure Open Source Technology Center
http://lists.01.org/mailman/listinfo/kbuild Intel Corporation
^ permalink raw reply [flat|nested] 3+ messages in thread[parent not found: <201501090239.gPJgrJIE%fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>]
* [PATCH wsa] i2c: viperboard: vprbrd_quirks can be static
[not found] ` <201501090239.gPJgrJIE%fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
@ 2015-01-08 18:07 ` kbuild test robot
0 siblings, 0 replies; 3+ messages in thread
From: kbuild test robot @ 2015-01-08 18:07 UTC (permalink / raw)
To: Wolfram Sang
Cc: kbuild-all-JC7UmRfGjtg, Lars Poeschel,
linux-i2c-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA
drivers/i2c/busses/i2c-viperboard.c:357:27: sparse: symbol 'vprbrd_quirks' was not declared. Should it be static?
Signed-off-by: Fengguang Wu <fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
---
i2c-viperboard.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/i2c/busses/i2c-viperboard.c b/drivers/i2c/busses/i2c-viperboard.c
index f880f08..47e88ad 100644
--- a/drivers/i2c/busses/i2c-viperboard.c
+++ b/drivers/i2c/busses/i2c-viperboard.c
@@ -354,7 +354,7 @@ static const struct i2c_algorithm vprbrd_algorithm = {
.functionality = vprbrd_i2c_func,
};
-struct i2c_adapter_quirks vprbrd_quirks = {
+static struct i2c_adapter_quirks vprbrd_quirks = {
.max_read_len = 2048,
.max_write_len = 2048,
};
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [wsa:i2c/quirks 10/11] drivers/i2c/busses/i2c-viperboard.c:357:27: sparse: symbol 'vprbrd_quirks' was not declared. Should it be static?
2015-01-08 18:07 [wsa:i2c/quirks 10/11] drivers/i2c/busses/i2c-viperboard.c:357:27: sparse: symbol 'vprbrd_quirks' was not declared. Should it be static? kbuild test robot
[not found] ` <201501090239.gPJgrJIE%fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
@ 2015-01-08 18:38 ` Wolfram Sang
1 sibling, 0 replies; 3+ messages in thread
From: Wolfram Sang @ 2015-01-08 18:38 UTC (permalink / raw)
To: kbuild test robot; +Cc: kbuild-all, Lars Poeschel, linux-i2c, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 2634 bytes --]
On Fri, Jan 09, 2015 at 02:07:53AM +0800, kbuild test robot wrote:
> tree: git://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux i2c/quirks
> head: b1d39af8ae21f4930854b0656aa83a83fd38049e
> commit: 5f7099010eda61b777ba84cf9b64dc8428846d25 [10/11] i2c: viperboard: make use of the new infrastructure for quirks
> reproduce:
> # apt-get install sparse
> git checkout 5f7099010eda61b777ba84cf9b64dc8428846d25
> make ARCH=x86_64 allmodconfig
> make C=1 CF=-D__CHECK_ENDIAN__
>
>
> sparse warnings: (new ones prefixed by >>)
>
> drivers/i2c/busses/i2c-viperboard.c:128:35: sparse: incorrect type in assignment (different base types)
> drivers/i2c/busses/i2c-viperboard.c:128:35: expected unsigned short [unsigned] [usertype] addr
> drivers/i2c/busses/i2c-viperboard.c:128:35: got restricted __le16 [usertype] <noident>
> drivers/i2c/busses/i2c-viperboard.c:201:34: sparse: incorrect type in assignment (different base types)
> drivers/i2c/busses/i2c-viperboard.c:201:34: expected unsigned short [unsigned] [usertype] tf1
> drivers/i2c/busses/i2c-viperboard.c:201:34: got restricted __le16 [usertype] <noident>
> drivers/i2c/busses/i2c-viperboard.c:202:34: sparse: incorrect type in assignment (different base types)
> drivers/i2c/busses/i2c-viperboard.c:202:34: expected unsigned short [unsigned] [usertype] tf2
> drivers/i2c/busses/i2c-viperboard.c:202:34: got restricted __le16 [usertype] <noident>
> drivers/i2c/busses/i2c-viperboard.c:237:35: sparse: incorrect type in assignment (different base types)
> drivers/i2c/busses/i2c-viperboard.c:237:35: expected unsigned short [unsigned] [usertype] addr
> drivers/i2c/busses/i2c-viperboard.c:237:35: got restricted __le16 [usertype] <noident>
> drivers/i2c/busses/i2c-viperboard.c:300:35: sparse: incorrect type in assignment (different base types)
> drivers/i2c/busses/i2c-viperboard.c:300:35: expected unsigned short [unsigned] [usertype] len
> drivers/i2c/busses/i2c-viperboard.c:300:35: got restricted __le16 [usertype] <noident>
> drivers/i2c/busses/i2c-viperboard.c:325:35: sparse: incorrect type in assignment (different base types)
> drivers/i2c/busses/i2c-viperboard.c:325:35: expected unsigned short [unsigned] [usertype] len
> drivers/i2c/busses/i2c-viperboard.c:325:35: got restricted __le16 [usertype] <noident>
> >> drivers/i2c/busses/i2c-viperboard.c:357:27: sparse: symbol 'vprbrd_quirks' was not declared. Should it be static?
>
> Please review and possibly fold the followup patch.
Development branch noise, please ignore.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-01-08 18:38 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-01-08 18:07 [wsa:i2c/quirks 10/11] drivers/i2c/busses/i2c-viperboard.c:357:27: sparse: symbol 'vprbrd_quirks' was not declared. Should it be static? kbuild test robot
[not found] ` <201501090239.gPJgrJIE%fengguang.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2015-01-08 18:07 ` [PATCH wsa] i2c: viperboard: vprbrd_quirks can be static kbuild test robot
2015-01-08 18:38 ` [wsa:i2c/quirks 10/11] drivers/i2c/busses/i2c-viperboard.c:357:27: sparse: symbol 'vprbrd_quirks' was not declared. Should it be static? Wolfram Sang
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).