diff options
| author | Fengguang Wu <fengguang.wu@intel.com> | 2013-01-12 02:31:40 -0500 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2013-01-12 03:15:11 -0500 |
| commit | 25bac122b8d874d625982aead1974be819eca841 (patch) | |
| tree | 2b7e98e440009d863c9330147f215bc27bc1e99a /drivers/block/mtip32xx/mtip32xx.c | |
| parent | 3d6a87430e764cf4132710538139c10970929189 (diff) | |
drivers/block/mtip32xx/mtip32xx.c:4029:1: sparse: symbol 'mtip_workq_sdbf0' was not declared. Should it be static?
Hi Asai,
FYI, there are new sparse warnings show up in
tree: git://git.kernel.dk/linux-block.git for-3.9/drivers
head: 3d6a87430e764cf4132710538139c10970929189
commit: 16c906e51c6f08a15763a85b5686e1ded35e77ab [1/3] mtip32xx: Add workqueue and NUMA support
drivers/block/mtip32xx/mtip32xx.c:3267:17: sparse: cast to restricted __le32
>> drivers/block/mtip32xx/mtip32xx.c:4029:1: sparse: symbol 'mtip_workq_sdbf0' was not declared. Should it be static?
>> drivers/block/mtip32xx/mtip32xx.c:4030:1: sparse: symbol 'mtip_workq_sdbf1' was not declared. Should it be static?
>> drivers/block/mtip32xx/mtip32xx.c:4031:1: sparse: symbol 'mtip_workq_sdbf2' was not declared. Should it be static?
>> drivers/block/mtip32xx/mtip32xx.c:4032:1: sparse: symbol 'mtip_workq_sdbf3' was not declared. Should it be static?
>> drivers/block/mtip32xx/mtip32xx.c:4033:1: sparse: symbol 'mtip_workq_sdbf4' was not declared. Should it be static?
>> drivers/block/mtip32xx/mtip32xx.c:4034:1: sparse: symbol 'mtip_workq_sdbf5' was not declared. Should it be static?
>> drivers/block/mtip32xx/mtip32xx.c:4035:1: sparse: symbol 'mtip_workq_sdbf6' was not declared. Should it be static?
>> drivers/block/mtip32xx/mtip32xx.c:4036:1: sparse: symbol 'mtip_workq_sdbf7' was not declared. Should it be static?
drivers/block/mtip32xx/mtip32xx.c: In function 'mtip_hw_read_flags':
drivers/block/mtip32xx/mtip32xx.c:2723:1: warning: the frame size of 1036 bytes is larger than 1024 bytes [-Wframe-larger-than=]
drivers/block/mtip32xx/mtip32xx.c: In function 'mtip_hw_read_registers':
drivers/block/mtip32xx/mtip32xx.c:2700:1: warning: the frame size of 1044 bytes is larger than 1024 bytes [-Wframe-larger-than=]
Please consider folding the below diff :-)
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'drivers/block/mtip32xx/mtip32xx.c')
| -rw-r--r-- | drivers/block/mtip32xx/mtip32xx.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/drivers/block/mtip32xx/mtip32xx.c b/drivers/block/mtip32xx/mtip32xx.c index b7579b76b4a1..f07dbcba7ba6 100644 --- a/drivers/block/mtip32xx/mtip32xx.c +++ b/drivers/block/mtip32xx/mtip32xx.c | |||
| @@ -4122,14 +4122,14 @@ static inline int mtip_get_next_rr_node(void) | |||
| 4122 | return next_node; | 4122 | return next_node; |
| 4123 | } | 4123 | } |
| 4124 | 4124 | ||
| 4125 | DEFINE_HANDLER(0); | 4125 | static DEFINE_HANDLER(0); |
| 4126 | DEFINE_HANDLER(1); | 4126 | static DEFINE_HANDLER(1); |
| 4127 | DEFINE_HANDLER(2); | 4127 | static DEFINE_HANDLER(2); |
| 4128 | DEFINE_HANDLER(3); | 4128 | static DEFINE_HANDLER(3); |
| 4129 | DEFINE_HANDLER(4); | 4129 | static DEFINE_HANDLER(4); |
| 4130 | DEFINE_HANDLER(5); | 4130 | static DEFINE_HANDLER(5); |
| 4131 | DEFINE_HANDLER(6); | 4131 | static DEFINE_HANDLER(6); |
| 4132 | DEFINE_HANDLER(7); | 4132 | static DEFINE_HANDLER(7); |
| 4133 | 4133 | ||
| 4134 | /* | 4134 | /* |
| 4135 | * Called for each supported PCI device detected. | 4135 | * Called for each supported PCI device detected. |