2010-12-13 14:08:48

by Mark Brown

[permalink] [raw]
Subject: [PATCH] mfd: Use NULL to initialise NULL pointers in ab8500-debugfs

Partly for coding style reasons, but mostly because sparse warns on it.

Signed-off-by: Mark Brown <[email protected]>
---
drivers/mfd/ab8500-debugfs.c | 14 +++++++-------
1 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/drivers/mfd/ab8500-debugfs.c b/drivers/mfd/ab8500-debugfs.c
index dfdc76e..3c1541a 100644
--- a/drivers/mfd/ab8500-debugfs.c
+++ b/drivers/mfd/ab8500-debugfs.c
@@ -189,7 +189,7 @@ static struct ab8500_i2c_ranges debug_ranges[AB8500_NUM_BANKS] = {
},
[AB8500_DBI] = {
.num_ranges = 0,
- .range = 0,
+ .range = NULL,
},
[AB8500_ECI_AV_ACC] = {
.num_ranges = 1,
@@ -202,7 +202,7 @@ static struct ab8500_i2c_ranges debug_ranges[AB8500_NUM_BANKS] = {
},
[0x9] = {
.num_ranges = 0,
- .range = 0,
+ .range = NULL,
},
[AB8500_GPADC] = {
.num_ranges = 1,
@@ -278,7 +278,7 @@ static struct ab8500_i2c_ranges debug_ranges[AB8500_NUM_BANKS] = {
},
[AB8500_INTERRUPT] = {
.num_ranges = 0,
- .range = 0,
+ .range = NULL,
},
[AB8500_RTC] = {
.num_ranges = 1,
@@ -328,19 +328,19 @@ static struct ab8500_i2c_ranges debug_ranges[AB8500_NUM_BANKS] = {
},
[0x11] = {
.num_ranges = 0,
- .range = 0,
+ .range = NULL,
},
[0x12] = {
.num_ranges = 0,
- .range = 0,
+ .range = NULL,
},
[0x13] = {
.num_ranges = 0,
- .range = 0,
+ .range = NULL,
},
[0x14] = {
.num_ranges = 0,
- .range = 0,
+ .range = NULL,
},
[AB8500_OTP_EMUL] = {
.num_ranges = 1,
--
1.7.1


2010-12-14 11:12:52

by Mattias Wallin

[permalink] [raw]
Subject: Re: [PATCH] mfd: Use NULL to initialise NULL pointers in ab8500-debugfs

On 12/13/2010 03:08 PM, Mark Brown wrote:
> Partly for coding style reasons, but mostly because sparse warns on it.
>
> Signed-off-by: Mark Brown <[email protected]>
> ---
Acked-by: Mattias Wallin <[email protected]>

Yours,
Mattias Wallin