linux-kernel - 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?

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20150108183800.GC29629@katana>
Date:	Thu, 8 Jan 2015 19:38:00 +0100
From:	Wolfram Sang <wsa@...-dreams.de>
To:	kbuild test robot <fengguang.wu@...el.com>
Cc:	kbuild-all@...org, Lars Poeschel <poeschel@...onage.de>,
	linux-i2c@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: 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?

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.


Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)