[PATCH 00/16] fix sparse warnings: Using plain integer as NULL pointer

[PATCH 00/16] fix sparse warnings: Using plain integer as NULL pointer

[Hannes Eder]

The following series fix the sparse warning 'Using plain integer as
NULL pointer' all over the kernel tree.  This series is based on
linux-next/next-20090220.

---

Hannes Eder (16):
      ALSA: msnd: fix sparse warning: Using plain integer as NULL pointer
      ipv6: fix sparse warning: Using plain integer as NULL pointer
      memcg: fix sparse warning: Using plain integer as NULL pointer
      kprobes: fix sparse warnings: Using plain integer as NULL pointer
      reiserfs: fix sparse warning: Using plain integer as NULL pointer
      fs/super.c: fix sparse warning: Using plain integer as NULL pointer
      viafb: fix sparse warning: Using plain integer as NULL pointer
      usb host: fix sparse warning: Using plain integer as NULL pointer
      [SCSI] FlashPoint: fix sparse warnings: Using plain integer as NULL pointer
      panasonic-laptop: fix sparse warning: Using plain integer as NULL pointer
      MTD: tests: fix sparse warnings: Using plain integer as NULL pointer
      dvb: fix sparse warnings: Using plain integer as NULL pointer
      ide: fix sparse warnings: Using plain integer as NULL pointer
      drm: radeon: fix sparse warnings: Using plain integer as NULL pointer
      x86, numa_32.c: fix sparse warning: Using plain integer as NULL pointer
      kexec/i386: fix sparse warnings: Using plain integer as NULL pointer


 arch/x86/kernel/machine_kexec_32.c      |    2 +-
 arch/x86/kernel/traps.c                 |    2 +-
 arch/x86/mm/numa_32.c                   |    2 +-
 drivers/gpu/drm/radeon/radeon_cp.c      |    2 +-
 drivers/gpu/drm/radeon/radeon_state.c   |    2 +-
 drivers/ide/ide-disk_proc.c             |    2 +-
 drivers/ide/ide-floppy_proc.c           |    2 +-
 drivers/ide/ide-proc.c                  |    2 +-
 drivers/ide/ide-tape.c                  |    2 +-
 drivers/media/dvb/frontends/drx397xD.c  |    2 +-
 drivers/mtd/tests/mtd_oobtest.c         |   24 ++++++++++++------------
 drivers/mtd/tests/mtd_readtest.c        |    2 +-
 drivers/platform/x86/panasonic-laptop.c |    2 +-
 drivers/scsi/FlashPoint.c               |    7 +++----
 drivers/usb/host/oxu210hp-hcd.c         |    2 +-
 drivers/video/via/hw.c                  |    2 +-
 fs/reiserfs/inode.c                     |    2 +-
 fs/super.c                              |    2 +-
 kernel/test_kprobes.c                   |    6 +++---
 mm/memcontrol.c                         |    2 +-
 net/ipv6/xfrm6_state.c                  |    2 +-
 sound/isa/msnd/msnd_pinnacle.c          |    2 +-
 22 files changed, 37 insertions(+), 38 deletions(-)

[PATCH 00/16] fix sparse warnings: Using plain integer as NULL pointer

[Hannes Eder]

The following series fix the sparse warning 'Using plain integer as
NULL pointer' all over the kernel tree.  This series is based on
linux-next/next-20090220.

---

Hannes Eder (16):
      ALSA: msnd: fix sparse warning: Using plain integer as NULL pointer
      ipv6: fix sparse warning: Using plain integer as NULL pointer
      memcg: fix sparse warning: Using plain integer as NULL pointer
      kprobes: fix sparse warnings: Using plain integer as NULL pointer
      reiserfs: fix sparse warning: Using plain integer as NULL pointer
      fs/super.c: fix sparse warning: Using plain integer as NULL pointer
      viafb: fix sparse warning: Using plain integer as NULL pointer
      usb host: fix sparse warning: Using plain integer as NULL pointer
      [SCSI] FlashPoint: fix sparse warnings: Using plain integer as NULL pointer
      panasonic-laptop: fix sparse warning: Using plain integer as NULL pointer
      MTD: tests: fix sparse warnings: Using plain integer as NULL pointer
      dvb: fix sparse warnings: Using plain integer as NULL pointer
      ide: fix sparse warnings: Using plain integer as NULL pointer
      drm: radeon: fix sparse warnings: Using plain integer as NULL pointer
      x86, numa_32.c: fix sparse warning: Using plain integer as NULL pointer
      kexec/i386: fix sparse warnings: Using plain integer as NULL pointer


 arch/x86/kernel/machine_kexec_32.c      |    2 +-
 arch/x86/kernel/traps.c                 |    2 +-
 arch/x86/mm/numa_32.c                   |    2 +-
 drivers/gpu/drm/radeon/radeon_cp.c      |    2 +-
 drivers/gpu/drm/radeon/radeon_state.c   |    2 +-
 drivers/ide/ide-disk_proc.c             |    2 +-
 drivers/ide/ide-floppy_proc.c           |    2 +-
 drivers/ide/ide-proc.c                  |    2 +-
 drivers/ide/ide-tape.c                  |    2 +-
 drivers/media/dvb/frontends/drx397xD.c  |    2 +-
 drivers/mtd/tests/mtd_oobtest.c         |   24 ++++++++++++------------
 drivers/mtd/tests/mtd_readtest.c        |    2 +-
 drivers/platform/x86/panasonic-laptop.c |    2 +-
 drivers/scsi/FlashPoint.c               |    7 +++----
 drivers/usb/host/oxu210hp-hcd.c         |    2 +-
 drivers/video/via/hw.c                  |    2 +-
 fs/reiserfs/inode.c                     |    2 +-
 fs/super.c                              |    2 +-
 kernel/test_kprobes.c                   |    6 +++---
 mm/memcontrol.c                         |    2 +-
 net/ipv6/xfrm6_state.c                  |    2 +-
 sound/isa/msnd/msnd_pinnacle.c          |    2 +-
 22 files changed, 37 insertions(+), 38 deletions(-)

[PATCH 01/16] kexec/i386: fix sparse warnings: Using plain integer as

[Hannes Eder]

Fix this sparse warnings:
  arch/x86/kernel/machine_kexec_32.c:124:22: warning: Using plain integer as NULL pointer
  arch/x86/kernel/traps.c:950:24: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 arch/x86/kernel/machine_kexec_32.c |    2 +-
 arch/x86/kernel/traps.c            |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/x86/kernel/machine_kexec_32.c b/arch/x86/kernel/machine_kexec_32.c
index 37f4200..f5fc8c7 100644
--- a/arch/x86/kernel/machine_kexec_32.c
+++ b/arch/x86/kernel/machine_kexec_32.c
@@ -121,7 +121,7 @@ static void machine_kexec_page_table_set_one(
 static void machine_kexec_prepare_page_tables(struct kimage *image)
 {
 	void *control_page;
-	pmd_t *pmd = 0;
+	pmd_t *pmd = NULL;
 
 	control_page = page_address(image->control_code_page);
 #ifdef CONFIG_X86_PAE
diff --git a/arch/x86/kernel/traps.c b/arch/x86/kernel/traps.c
index ba90406..6882089 100644
--- a/arch/x86/kernel/traps.c
+++ b/arch/x86/kernel/traps.c
@@ -947,7 +947,7 @@ dotraplinkage void do_iret_error(struct pt_regs *regs, long error_code)
 	info.si_signo = SIGILL;
 	info.si_errno = 0;
 	info.si_code = ILL_BADSTK;
-	info.si_addr = 0;
+	info.si_addr = NULL;
 	if (notify_die(DIE_TRAP, "iret exception",
 			regs, error_code, 32, SIGILL) = NOTIFY_STOP)
 		return;

[PATCH 01/16] kexec/i386: fix sparse warnings: Using plain integer as NULL pointer

[Hannes Eder]

Fix this sparse warnings:
  arch/x86/kernel/machine_kexec_32.c:124:22: warning: Using plain integer as NULL pointer
  arch/x86/kernel/traps.c:950:24: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 arch/x86/kernel/machine_kexec_32.c |    2 +-
 arch/x86/kernel/traps.c            |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/x86/kernel/machine_kexec_32.c b/arch/x86/kernel/machine_kexec_32.c
index 37f4200..f5fc8c7 100644
--- a/arch/x86/kernel/machine_kexec_32.c
+++ b/arch/x86/kernel/machine_kexec_32.c
@@ -121,7 +121,7 @@ static void machine_kexec_page_table_set_one(
 static void machine_kexec_prepare_page_tables(struct kimage *image)
 {
 	void *control_page;
-	pmd_t *pmd = 0;
+	pmd_t *pmd = NULL;
 
 	control_page = page_address(image->control_code_page);
 #ifdef CONFIG_X86_PAE
diff --git a/arch/x86/kernel/traps.c b/arch/x86/kernel/traps.c
index ba90406..6882089 100644
--- a/arch/x86/kernel/traps.c
+++ b/arch/x86/kernel/traps.c
@@ -947,7 +947,7 @@ dotraplinkage void do_iret_error(struct pt_regs *regs, long error_code)
 	info.si_signo = SIGILL;
 	info.si_errno = 0;
 	info.si_code = ILL_BADSTK;
-	info.si_addr = 0;
+	info.si_addr = NULL;
 	if (notify_die(DIE_TRAP, "iret exception",
 			regs, error_code, 32, SIGILL) == NOTIFY_STOP)
 		return;

Re: [PATCH 01/16] kexec/i386: fix sparse warnings: Using plain

[Hannes Eder]

Added To: Ingo Molnar <mingo@elte.hu>

On Sun, Feb 22, 2009 at 1:00 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warnings:
>  arch/x86/kernel/machine_kexec_32.c:124:22: warning: Using plain integer as NULL pointer
>  arch/x86/kernel/traps.c:950:24: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  arch/x86/kernel/machine_kexec_32.c |    2 +-
>  arch/x86/kernel/traps.c            |    2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/x86/kernel/machine_kexec_32.c b/arch/x86/kernel/machine_kexec_32.c
> index 37f4200..f5fc8c7 100644
> --- a/arch/x86/kernel/machine_kexec_32.c
> +++ b/arch/x86/kernel/machine_kexec_32.c
> @@ -121,7 +121,7 @@ static void machine_kexec_page_table_set_one(
>  static void machine_kexec_prepare_page_tables(struct kimage *image)
>  {
>        void *control_page;
> -       pmd_t *pmd = 0;
> +       pmd_t *pmd = NULL;
>
>        control_page = page_address(image->control_code_page);
>  #ifdef CONFIG_X86_PAE
> diff --git a/arch/x86/kernel/traps.c b/arch/x86/kernel/traps.c
> index ba90406..6882089 100644
> --- a/arch/x86/kernel/traps.c
> +++ b/arch/x86/kernel/traps.c
> @@ -947,7 +947,7 @@ dotraplinkage void do_iret_error(struct pt_regs *regs, long error_code)
>        info.si_signo = SIGILL;
>        info.si_errno = 0;
>        info.si_code = ILL_BADSTK;
> -       info.si_addr = 0;
> +       info.si_addr = NULL;
>        if (notify_die(DIE_TRAP, "iret exception",
>                        regs, error_code, 32, SIGILL) = NOTIFY_STOP)
>                return;
>
>
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 01/16] kexec/i386: fix sparse warnings: Using plain integer as NULL pointer

[Hannes Eder]

Added To: Ingo Molnar <mingo@elte.hu>

On Sun, Feb 22, 2009 at 1:00 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warnings:
>  arch/x86/kernel/machine_kexec_32.c:124:22: warning: Using plain integer as NULL pointer
>  arch/x86/kernel/traps.c:950:24: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  arch/x86/kernel/machine_kexec_32.c |    2 +-
>  arch/x86/kernel/traps.c            |    2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/x86/kernel/machine_kexec_32.c b/arch/x86/kernel/machine_kexec_32.c
> index 37f4200..f5fc8c7 100644
> --- a/arch/x86/kernel/machine_kexec_32.c
> +++ b/arch/x86/kernel/machine_kexec_32.c
> @@ -121,7 +121,7 @@ static void machine_kexec_page_table_set_one(
>  static void machine_kexec_prepare_page_tables(struct kimage *image)
>  {
>        void *control_page;
> -       pmd_t *pmd = 0;
> +       pmd_t *pmd = NULL;
>
>        control_page = page_address(image->control_code_page);
>  #ifdef CONFIG_X86_PAE
> diff --git a/arch/x86/kernel/traps.c b/arch/x86/kernel/traps.c
> index ba90406..6882089 100644
> --- a/arch/x86/kernel/traps.c
> +++ b/arch/x86/kernel/traps.c
> @@ -947,7 +947,7 @@ dotraplinkage void do_iret_error(struct pt_regs *regs, long error_code)
>        info.si_signo = SIGILL;
>        info.si_errno = 0;
>        info.si_code = ILL_BADSTK;
> -       info.si_addr = 0;
> +       info.si_addr = NULL;
>        if (notify_die(DIE_TRAP, "iret exception",
>                        regs, error_code, 32, SIGILL) == NOTIFY_STOP)
>                return;
>
>

[PATCH 02/16] x86,

[Hannes Eder]

Fix this sparse warning:
  arch/x86/mm/numa_32.c:197:24: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 arch/x86/mm/numa_32.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/x86/mm/numa_32.c b/arch/x86/mm/numa_32.c
index d1f7439..3957cd6 100644
--- a/arch/x86/mm/numa_32.c
+++ b/arch/x86/mm/numa_32.c
@@ -194,7 +194,7 @@ void *alloc_remap(int nid, unsigned long size)
 	size = ALIGN(size, L1_CACHE_BYTES);
 
 	if (!allocation || (allocation + size) >= node_remap_end_vaddr[nid])
-		return 0;
+		return NULL;
 
 	node_remap_alloc_vaddr[nid] += size;
 	memset(allocation, 0, size);

[PATCH 02/16] x86, numa_32.c: fix sparse warning: Using plain integer as NULL pointer

[Hannes Eder]

Fix this sparse warning:
  arch/x86/mm/numa_32.c:197:24: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 arch/x86/mm/numa_32.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/x86/mm/numa_32.c b/arch/x86/mm/numa_32.c
index d1f7439..3957cd6 100644
--- a/arch/x86/mm/numa_32.c
+++ b/arch/x86/mm/numa_32.c
@@ -194,7 +194,7 @@ void *alloc_remap(int nid, unsigned long size)
 	size = ALIGN(size, L1_CACHE_BYTES);
 
 	if (!allocation || (allocation + size) >= node_remap_end_vaddr[nid])
-		return 0;
+		return NULL;
 
 	node_remap_alloc_vaddr[nid] += size;
 	memset(allocation, 0, size);

Re: [PATCH 02/16] x86, numa_32.c: fix sparse warning: Using plain

[Hannes Eder]

Added To: Ingo Molnar <mingo@elte.hu>

On Sun, Feb 22, 2009 at 1:01 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warning:
>  arch/x86/mm/numa_32.c:197:24: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  arch/x86/mm/numa_32.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/arch/x86/mm/numa_32.c b/arch/x86/mm/numa_32.c
> index d1f7439..3957cd6 100644
> --- a/arch/x86/mm/numa_32.c
> +++ b/arch/x86/mm/numa_32.c
> @@ -194,7 +194,7 @@ void *alloc_remap(int nid, unsigned long size)
>        size = ALIGN(size, L1_CACHE_BYTES);
>
>        if (!allocation || (allocation + size) >= node_remap_end_vaddr[nid])
> -               return 0;
> +               return NULL;
>
>        node_remap_alloc_vaddr[nid] += size;
>        memset(allocation, 0, size);
>
>
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 02/16] x86, numa_32.c: fix sparse warning: Using plain integer as NULL pointer

[Hannes Eder]

Added To: Ingo Molnar <mingo@elte.hu>

On Sun, Feb 22, 2009 at 1:01 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warning:
>  arch/x86/mm/numa_32.c:197:24: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  arch/x86/mm/numa_32.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/arch/x86/mm/numa_32.c b/arch/x86/mm/numa_32.c
> index d1f7439..3957cd6 100644
> --- a/arch/x86/mm/numa_32.c
> +++ b/arch/x86/mm/numa_32.c
> @@ -194,7 +194,7 @@ void *alloc_remap(int nid, unsigned long size)
>        size = ALIGN(size, L1_CACHE_BYTES);
>
>        if (!allocation || (allocation + size) >= node_remap_end_vaddr[nid])
> -               return 0;
> +               return NULL;
>
>        node_remap_alloc_vaddr[nid] += size;
>        memset(allocation, 0, size);
>
>

[PATCH 03/16] drm: radeon: fix sparse warnings: Using plain integer

[Hannes Eder]

Fix this sparse warning:
  drivers/gpu/drm/radeon/radeon_cp.c:1363:52: warning: Using plain integer as NULL pointer
  drivers/gpu/drm/radeon/radeon_state.c:1983:61: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 drivers/gpu/drm/radeon/radeon_cp.c    |    2 +-
 drivers/gpu/drm/radeon/radeon_state.c |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/radeon/radeon_cp.c b/drivers/gpu/drm/radeon/radeon_cp.c
index 119e642..f5f4e96 100644
--- a/drivers/gpu/drm/radeon/radeon_cp.c
+++ b/drivers/gpu/drm/radeon/radeon_cp.c
@@ -1360,7 +1360,7 @@ static int radeon_do_cleanup_cp(struct drm_device * dev)
 		if (dev_priv->gart_info.gart_table_location = DRM_ATI_GART_FB)
 		{
 			drm_core_ioremapfree(&dev_priv->gart_info.mapping, dev);
-			dev_priv->gart_info.addr = 0;
+			dev_priv->gart_info.addr = NULL;
 		}
 	}
 	/* only clear to the start of flags */
diff --git a/drivers/gpu/drm/radeon/radeon_state.c b/drivers/gpu/drm/radeon/radeon_state.c
index 043293a..61e75bc 100644
--- a/drivers/gpu/drm/radeon/radeon_state.c
+++ b/drivers/gpu/drm/radeon/radeon_state.c
@@ -1980,7 +1980,7 @@ static int alloc_surface(drm_radeon_surface_alloc_t *new,
 
 	/* find a virtual surface */
 	for (i = 0; i < 2 * RADEON_MAX_SURFACES; i++)
-		if (dev_priv->virt_surfaces[i].file_priv = 0)
+		if (dev_priv->virt_surfaces[i].file_priv = NULL)
 			break;
 	if (i = 2 * RADEON_MAX_SURFACES) {
 		return -1;

[PATCH 03/16] drm: radeon: fix sparse warnings: Using plain integer as NULL pointer

[Hannes Eder]

Fix this sparse warning:
  drivers/gpu/drm/radeon/radeon_cp.c:1363:52: warning: Using plain integer as NULL pointer
  drivers/gpu/drm/radeon/radeon_state.c:1983:61: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 drivers/gpu/drm/radeon/radeon_cp.c    |    2 +-
 drivers/gpu/drm/radeon/radeon_state.c |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/radeon/radeon_cp.c b/drivers/gpu/drm/radeon/radeon_cp.c
index 119e642..f5f4e96 100644
--- a/drivers/gpu/drm/radeon/radeon_cp.c
+++ b/drivers/gpu/drm/radeon/radeon_cp.c
@@ -1360,7 +1360,7 @@ static int radeon_do_cleanup_cp(struct drm_device * dev)
 		if (dev_priv->gart_info.gart_table_location == DRM_ATI_GART_FB)
 		{
 			drm_core_ioremapfree(&dev_priv->gart_info.mapping, dev);
-			dev_priv->gart_info.addr = 0;
+			dev_priv->gart_info.addr = NULL;
 		}
 	}
 	/* only clear to the start of flags */
diff --git a/drivers/gpu/drm/radeon/radeon_state.c b/drivers/gpu/drm/radeon/radeon_state.c
index 043293a..61e75bc 100644
--- a/drivers/gpu/drm/radeon/radeon_state.c
+++ b/drivers/gpu/drm/radeon/radeon_state.c
@@ -1980,7 +1980,7 @@ static int alloc_surface(drm_radeon_surface_alloc_t *new,
 
 	/* find a virtual surface */
 	for (i = 0; i < 2 * RADEON_MAX_SURFACES; i++)
-		if (dev_priv->virt_surfaces[i].file_priv == 0)
+		if (dev_priv->virt_surfaces[i].file_priv == NULL)
 			break;
 	if (i == 2 * RADEON_MAX_SURFACES) {
 		return -1;

Re: [PATCH 03/16] drm: radeon: fix sparse warnings: Using plain

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:01 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warning:
>  drivers/gpu/drm/radeon/radeon_cp.c:1363:52: warning: Using plain integer as NULL pointer
>  drivers/gpu/drm/radeon/radeon_state.c:1983:61: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  drivers/gpu/drm/radeon/radeon_cp.c    |    2 +-
>  drivers/gpu/drm/radeon/radeon_state.c |    2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_cp.c b/drivers/gpu/drm/radeon/radeon_cp.c
> index 119e642..f5f4e96 100644
> --- a/drivers/gpu/drm/radeon/radeon_cp.c
> +++ b/drivers/gpu/drm/radeon/radeon_cp.c
> @@ -1360,7 +1360,7 @@ static int radeon_do_cleanup_cp(struct drm_device * dev)
>                if (dev_priv->gart_info.gart_table_location = DRM_ATI_GART_FB)
>                {
>                        drm_core_ioremapfree(&dev_priv->gart_info.mapping, dev);
> -                       dev_priv->gart_info.addr = 0;
> +                       dev_priv->gart_info.addr = NULL;
>                }
>        }
>        /* only clear to the start of flags */
> diff --git a/drivers/gpu/drm/radeon/radeon_state.c b/drivers/gpu/drm/radeon/radeon_state.c
> index 043293a..61e75bc 100644
> --- a/drivers/gpu/drm/radeon/radeon_state.c
> +++ b/drivers/gpu/drm/radeon/radeon_state.c
> @@ -1980,7 +1980,7 @@ static int alloc_surface(drm_radeon_surface_alloc_t *new,
>
>        /* find a virtual surface */
>        for (i = 0; i < 2 * RADEON_MAX_SURFACES; i++)
> -               if (dev_priv->virt_surfaces[i].file_priv = 0)
> +               if (dev_priv->virt_surfaces[i].file_priv = NULL)
>                        break;
>        if (i = 2 * RADEON_MAX_SURFACES) {
>                return -1;
>
>
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 03/16] drm: radeon: fix sparse warnings: Using plain integer as NULL pointer

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:01 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warning:
>  drivers/gpu/drm/radeon/radeon_cp.c:1363:52: warning: Using plain integer as NULL pointer
>  drivers/gpu/drm/radeon/radeon_state.c:1983:61: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  drivers/gpu/drm/radeon/radeon_cp.c    |    2 +-
>  drivers/gpu/drm/radeon/radeon_state.c |    2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_cp.c b/drivers/gpu/drm/radeon/radeon_cp.c
> index 119e642..f5f4e96 100644
> --- a/drivers/gpu/drm/radeon/radeon_cp.c
> +++ b/drivers/gpu/drm/radeon/radeon_cp.c
> @@ -1360,7 +1360,7 @@ static int radeon_do_cleanup_cp(struct drm_device * dev)
>                if (dev_priv->gart_info.gart_table_location == DRM_ATI_GART_FB)
>                {
>                        drm_core_ioremapfree(&dev_priv->gart_info.mapping, dev);
> -                       dev_priv->gart_info.addr = 0;
> +                       dev_priv->gart_info.addr = NULL;
>                }
>        }
>        /* only clear to the start of flags */
> diff --git a/drivers/gpu/drm/radeon/radeon_state.c b/drivers/gpu/drm/radeon/radeon_state.c
> index 043293a..61e75bc 100644
> --- a/drivers/gpu/drm/radeon/radeon_state.c
> +++ b/drivers/gpu/drm/radeon/radeon_state.c
> @@ -1980,7 +1980,7 @@ static int alloc_surface(drm_radeon_surface_alloc_t *new,
>
>        /* find a virtual surface */
>        for (i = 0; i < 2 * RADEON_MAX_SURFACES; i++)
> -               if (dev_priv->virt_surfaces[i].file_priv == 0)
> +               if (dev_priv->virt_surfaces[i].file_priv == NULL)
>                        break;
>        if (i == 2 * RADEON_MAX_SURFACES) {
>                return -1;
>
>

[PATCH 04/16] ide: fix sparse warnings: Using plain integer as NULL

[Hannes Eder]

Fix this sparse warnings:
  drivers/ide/ide-disk_proc.c:130:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-floppy_proc.c:32:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-proc.c:234:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-tape.c:2141:11: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 drivers/ide/ide-disk_proc.c   |    2 +-
 drivers/ide/ide-floppy_proc.c |    2 +-
 drivers/ide/ide-proc.c        |    2 +-
 drivers/ide/ide-tape.c        |    2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/ide/ide-disk_proc.c b/drivers/ide/ide-disk_proc.c
index 3f2a078..eaea3be 100644
--- a/drivers/ide/ide-disk_proc.c
+++ b/drivers/ide/ide-disk_proc.c
@@ -127,5 +127,5 @@ const struct ide_proc_devset ide_disk_settings[] = {
 	IDE_PROC_DEVSET(multcount,	0,    16),
 	IDE_PROC_DEVSET(nowerr,		0,     1),
 	IDE_PROC_DEVSET(wcache,		0,     1),
-	{ 0 },
+	{ NULL },
 };
diff --git a/drivers/ide/ide-floppy_proc.c b/drivers/ide/ide-floppy_proc.c
index 3ec762c..fcd4d81 100644
--- a/drivers/ide/ide-floppy_proc.c
+++ b/drivers/ide/ide-floppy_proc.c
@@ -29,5 +29,5 @@ const struct ide_proc_devset ide_floppy_settings[] = {
 	IDE_PROC_DEVSET(bios_head, 0,  255),
 	IDE_PROC_DEVSET(bios_sect, 0,   63),
 	IDE_PROC_DEVSET(ticks,	   0,  255),
-	{ 0 },
+	{ NULL },
 };
diff --git a/drivers/ide/ide-proc.c b/drivers/ide/ide-proc.c
index 0ee8887..10a88bf 100644
--- a/drivers/ide/ide-proc.c
+++ b/drivers/ide/ide-proc.c
@@ -231,7 +231,7 @@ static const struct ide_proc_devset ide_generic_settings[] = {
 	IDE_PROC_DEVSET(pio_mode, 0, 255),
 	IDE_PROC_DEVSET(unmaskirq, 0, 1),
 	IDE_PROC_DEVSET(using_dma, 0, 1),
-	{ 0 },
+	{ NULL },
 };
 
 static void proc_ide_settings_warn(void)
diff --git a/drivers/ide/ide-tape.c b/drivers/ide/ide-tape.c
index 1b97d7a..e9e570f 100644
--- a/drivers/ide/ide-tape.c
+++ b/drivers/ide/ide-tape.c
@@ -2138,7 +2138,7 @@ static const struct ide_proc_devset idetape_settings[] = {
 	__IDE_PROC_DEVSET(speed,	0, 0xffff, NULL, NULL),
 	__IDE_PROC_DEVSET(tdsc,		IDETAPE_DSC_RW_MIN, IDETAPE_DSC_RW_MAX,
 					mulf_tdsc, divf_tdsc),
-	{ 0 },
+	{ NULL },
 };
 #endif
 

[PATCH 04/16] ide: fix sparse warnings: Using plain integer as NULL pointer

[Hannes Eder]

Fix this sparse warnings:
  drivers/ide/ide-disk_proc.c:130:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-floppy_proc.c:32:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-proc.c:234:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-tape.c:2141:11: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 drivers/ide/ide-disk_proc.c   |    2 +-
 drivers/ide/ide-floppy_proc.c |    2 +-
 drivers/ide/ide-proc.c        |    2 +-
 drivers/ide/ide-tape.c        |    2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/ide/ide-disk_proc.c b/drivers/ide/ide-disk_proc.c
index 3f2a078..eaea3be 100644
--- a/drivers/ide/ide-disk_proc.c
+++ b/drivers/ide/ide-disk_proc.c
@@ -127,5 +127,5 @@ const struct ide_proc_devset ide_disk_settings[] = {
 	IDE_PROC_DEVSET(multcount,	0,    16),
 	IDE_PROC_DEVSET(nowerr,		0,     1),
 	IDE_PROC_DEVSET(wcache,		0,     1),
-	{ 0 },
+	{ NULL },
 };
diff --git a/drivers/ide/ide-floppy_proc.c b/drivers/ide/ide-floppy_proc.c
index 3ec762c..fcd4d81 100644
--- a/drivers/ide/ide-floppy_proc.c
+++ b/drivers/ide/ide-floppy_proc.c
@@ -29,5 +29,5 @@ const struct ide_proc_devset ide_floppy_settings[] = {
 	IDE_PROC_DEVSET(bios_head, 0,  255),
 	IDE_PROC_DEVSET(bios_sect, 0,   63),
 	IDE_PROC_DEVSET(ticks,	   0,  255),
-	{ 0 },
+	{ NULL },
 };
diff --git a/drivers/ide/ide-proc.c b/drivers/ide/ide-proc.c
index 0ee8887..10a88bf 100644
--- a/drivers/ide/ide-proc.c
+++ b/drivers/ide/ide-proc.c
@@ -231,7 +231,7 @@ static const struct ide_proc_devset ide_generic_settings[] = {
 	IDE_PROC_DEVSET(pio_mode, 0, 255),
 	IDE_PROC_DEVSET(unmaskirq, 0, 1),
 	IDE_PROC_DEVSET(using_dma, 0, 1),
-	{ 0 },
+	{ NULL },
 };
 
 static void proc_ide_settings_warn(void)
diff --git a/drivers/ide/ide-tape.c b/drivers/ide/ide-tape.c
index 1b97d7a..e9e570f 100644
--- a/drivers/ide/ide-tape.c
+++ b/drivers/ide/ide-tape.c
@@ -2138,7 +2138,7 @@ static const struct ide_proc_devset idetape_settings[] = {
 	__IDE_PROC_DEVSET(speed,	0, 0xffff, NULL, NULL),
 	__IDE_PROC_DEVSET(tdsc,		IDETAPE_DSC_RW_MIN, IDETAPE_DSC_RW_MAX,
 					mulf_tdsc, divf_tdsc),
-	{ 0 },
+	{ NULL },
 };
 #endif
 

Re: [PATCH 04/16] ide: fix sparse warnings: Using plain integer as

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:01 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warnings:
>  drivers/ide/ide-disk_proc.c:130:11: warning: Using plain integer as NULL pointer
>  drivers/ide/ide-floppy_proc.c:32:11: warning: Using plain integer as NULL pointer
>  drivers/ide/ide-proc.c:234:11: warning: Using plain integer as NULL pointer
>  drivers/ide/ide-tape.c:2141:11: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  drivers/ide/ide-disk_proc.c   |    2 +-
>  drivers/ide/ide-floppy_proc.c |    2 +-
>  drivers/ide/ide-proc.c        |    2 +-
>  drivers/ide/ide-tape.c        |    2 +-
>  4 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/ide/ide-disk_proc.c b/drivers/ide/ide-disk_proc.c
> index 3f2a078..eaea3be 100644
> --- a/drivers/ide/ide-disk_proc.c
> +++ b/drivers/ide/ide-disk_proc.c
> @@ -127,5 +127,5 @@ const struct ide_proc_devset ide_disk_settings[] = {
>        IDE_PROC_DEVSET(multcount,      0,    16),
>        IDE_PROC_DEVSET(nowerr,         0,     1),
>        IDE_PROC_DEVSET(wcache,         0,     1),
> -       { 0 },
> +       { NULL },
>  };
> diff --git a/drivers/ide/ide-floppy_proc.c b/drivers/ide/ide-floppy_proc.c
> index 3ec762c..fcd4d81 100644
> --- a/drivers/ide/ide-floppy_proc.c
> +++ b/drivers/ide/ide-floppy_proc.c
> @@ -29,5 +29,5 @@ const struct ide_proc_devset ide_floppy_settings[] = {
>        IDE_PROC_DEVSET(bios_head, 0,  255),
>        IDE_PROC_DEVSET(bios_sect, 0,   63),
>        IDE_PROC_DEVSET(ticks,     0,  255),
> -       { 0 },
> +       { NULL },
>  };
> diff --git a/drivers/ide/ide-proc.c b/drivers/ide/ide-proc.c
> index 0ee8887..10a88bf 100644
> --- a/drivers/ide/ide-proc.c
> +++ b/drivers/ide/ide-proc.c
> @@ -231,7 +231,7 @@ static const struct ide_proc_devset ide_generic_settings[] = {
>        IDE_PROC_DEVSET(pio_mode, 0, 255),
>        IDE_PROC_DEVSET(unmaskirq, 0, 1),
>        IDE_PROC_DEVSET(using_dma, 0, 1),
> -       { 0 },
> +       { NULL },
>  };
>
>  static void proc_ide_settings_warn(void)
> diff --git a/drivers/ide/ide-tape.c b/drivers/ide/ide-tape.c
> index 1b97d7a..e9e570f 100644
> --- a/drivers/ide/ide-tape.c
> +++ b/drivers/ide/ide-tape.c
> @@ -2138,7 +2138,7 @@ static const struct ide_proc_devset idetape_settings[] = {
>        __IDE_PROC_DEVSET(speed,        0, 0xffff, NULL, NULL),
>        __IDE_PROC_DEVSET(tdsc,         IDETAPE_DSC_RW_MIN, IDETAPE_DSC_RW_MAX,
>                                        mulf_tdsc, divf_tdsc),
> -       { 0 },
> +       { NULL },
>  };
>  #endif
>
>
>
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 04/16] ide: fix sparse warnings: Using plain integer as NULL pointer

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:01 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warnings:
>  drivers/ide/ide-disk_proc.c:130:11: warning: Using plain integer as NULL pointer
>  drivers/ide/ide-floppy_proc.c:32:11: warning: Using plain integer as NULL pointer
>  drivers/ide/ide-proc.c:234:11: warning: Using plain integer as NULL pointer
>  drivers/ide/ide-tape.c:2141:11: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  drivers/ide/ide-disk_proc.c   |    2 +-
>  drivers/ide/ide-floppy_proc.c |    2 +-
>  drivers/ide/ide-proc.c        |    2 +-
>  drivers/ide/ide-tape.c        |    2 +-
>  4 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/ide/ide-disk_proc.c b/drivers/ide/ide-disk_proc.c
> index 3f2a078..eaea3be 100644
> --- a/drivers/ide/ide-disk_proc.c
> +++ b/drivers/ide/ide-disk_proc.c
> @@ -127,5 +127,5 @@ const struct ide_proc_devset ide_disk_settings[] = {
>        IDE_PROC_DEVSET(multcount,      0,    16),
>        IDE_PROC_DEVSET(nowerr,         0,     1),
>        IDE_PROC_DEVSET(wcache,         0,     1),
> -       { 0 },
> +       { NULL },
>  };
> diff --git a/drivers/ide/ide-floppy_proc.c b/drivers/ide/ide-floppy_proc.c
> index 3ec762c..fcd4d81 100644
> --- a/drivers/ide/ide-floppy_proc.c
> +++ b/drivers/ide/ide-floppy_proc.c
> @@ -29,5 +29,5 @@ const struct ide_proc_devset ide_floppy_settings[] = {
>        IDE_PROC_DEVSET(bios_head, 0,  255),
>        IDE_PROC_DEVSET(bios_sect, 0,   63),
>        IDE_PROC_DEVSET(ticks,     0,  255),
> -       { 0 },
> +       { NULL },
>  };
> diff --git a/drivers/ide/ide-proc.c b/drivers/ide/ide-proc.c
> index 0ee8887..10a88bf 100644
> --- a/drivers/ide/ide-proc.c
> +++ b/drivers/ide/ide-proc.c
> @@ -231,7 +231,7 @@ static const struct ide_proc_devset ide_generic_settings[] = {
>        IDE_PROC_DEVSET(pio_mode, 0, 255),
>        IDE_PROC_DEVSET(unmaskirq, 0, 1),
>        IDE_PROC_DEVSET(using_dma, 0, 1),
> -       { 0 },
> +       { NULL },
>  };
>
>  static void proc_ide_settings_warn(void)
> diff --git a/drivers/ide/ide-tape.c b/drivers/ide/ide-tape.c
> index 1b97d7a..e9e570f 100644
> --- a/drivers/ide/ide-tape.c
> +++ b/drivers/ide/ide-tape.c
> @@ -2138,7 +2138,7 @@ static const struct ide_proc_devset idetape_settings[] = {
>        __IDE_PROC_DEVSET(speed,        0, 0xffff, NULL, NULL),
>        __IDE_PROC_DEVSET(tdsc,         IDETAPE_DSC_RW_MIN, IDETAPE_DSC_RW_MAX,
>                                        mulf_tdsc, divf_tdsc),
> -       { 0 },
> +       { NULL },
>  };
>  #endif
>
>
>

[PATCH 05/16] dvb: fix sparse warnings: Using plain integer as NULL

[Hannes Eder]

Fix this sparse warnings:
  drivers/media/dvb/frontends/drx397xD_fw.h:21:2: warning: Using plain integer as NULL pointer
  drivers/media/dvb/frontends/drx397xD_fw.h:22:2: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 drivers/media/dvb/frontends/drx397xD.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/media/dvb/frontends/drx397xD.c b/drivers/media/dvb/frontends/drx397xD.c
index 1e81e71..172f1f9 100644
--- a/drivers/media/dvb/frontends/drx397xD.c
+++ b/drivers/media/dvb/frontends/drx397xD.c
@@ -74,7 +74,7 @@ static struct {
 } fw[] = {
 #define _FW_ENTRY(a, b, c)	{					\
 			.name	= a,					\
-			.file	= 0,					\
+			.file	= NULL,					\
 			.lock	= __RW_LOCK_UNLOCKED(fw[c].lock),	\
 			.refcnt = 0,					\
 			.data	= { }		}

[PATCH 05/16] dvb: fix sparse warnings: Using plain integer as NULL pointer

[Hannes Eder]

Fix this sparse warnings:
  drivers/media/dvb/frontends/drx397xD_fw.h:21:2: warning: Using plain integer as NULL pointer
  drivers/media/dvb/frontends/drx397xD_fw.h:22:2: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 drivers/media/dvb/frontends/drx397xD.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/media/dvb/frontends/drx397xD.c b/drivers/media/dvb/frontends/drx397xD.c
index 1e81e71..172f1f9 100644
--- a/drivers/media/dvb/frontends/drx397xD.c
+++ b/drivers/media/dvb/frontends/drx397xD.c
@@ -74,7 +74,7 @@ static struct {
 } fw[] = {
 #define _FW_ENTRY(a, b, c)	{					\
 			.name	= a,					\
-			.file	= 0,					\
+			.file	= NULL,					\
 			.lock	= __RW_LOCK_UNLOCKED(fw[c].lock),	\
 			.refcnt = 0,					\
 			.data	= { }		}

Re: [PATCH 05/16] dvb: fix sparse warnings: Using plain integer as

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:02 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warnings:
>  drivers/media/dvb/frontends/drx397xD_fw.h:21:2: warning: Using plain integer as NULL pointer
>  drivers/media/dvb/frontends/drx397xD_fw.h:22:2: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  drivers/media/dvb/frontends/drx397xD.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/media/dvb/frontends/drx397xD.c b/drivers/media/dvb/frontends/drx397xD.c
> index 1e81e71..172f1f9 100644
> --- a/drivers/media/dvb/frontends/drx397xD.c
> +++ b/drivers/media/dvb/frontends/drx397xD.c
> @@ -74,7 +74,7 @@ static struct {
>  } fw[] = {
>  #define _FW_ENTRY(a, b, c)     {                                       \
>                        .name   = a,                                    \
> -                       .file   = 0,                                    \
> +                       .file   = NULL,                                 \
>                        .lock   = __RW_LOCK_UNLOCKED(fw[c].lock),       \
>                        .refcnt = 0,                                    \
>                        .data   = { }           }
>
>
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 05/16] dvb: fix sparse warnings: Using plain integer as NULL pointer

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:02 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warnings:
>  drivers/media/dvb/frontends/drx397xD_fw.h:21:2: warning: Using plain integer as NULL pointer
>  drivers/media/dvb/frontends/drx397xD_fw.h:22:2: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  drivers/media/dvb/frontends/drx397xD.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/media/dvb/frontends/drx397xD.c b/drivers/media/dvb/frontends/drx397xD.c
> index 1e81e71..172f1f9 100644
> --- a/drivers/media/dvb/frontends/drx397xD.c
> +++ b/drivers/media/dvb/frontends/drx397xD.c
> @@ -74,7 +74,7 @@ static struct {
>  } fw[] = {
>  #define _FW_ENTRY(a, b, c)     {                                       \
>                        .name   = a,                                    \
> -                       .file   = 0,                                    \
> +                       .file   = NULL,                                 \
>                        .lock   = __RW_LOCK_UNLOCKED(fw[c].lock),       \
>                        .refcnt = 0,                                    \
>                        .data   = { }           }
>
>

[PATCH 06/16] MTD: tests: fix sparse warnings: Using plain integer as

[Hannes Eder]

Fix this sparse warnings:
  drivers/mtd/tests/mtd_oobtest.c:139:33: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:192:33: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:219:41: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:284:25: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:525:25: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:545:25: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:569:33: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:589:33: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:613:33: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:633:33: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:673:41: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:701:33: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_readtest.c:74:41: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 drivers/mtd/tests/mtd_oobtest.c  |   24 ++++++++++++------------
 drivers/mtd/tests/mtd_readtest.c |    2 +-
 2 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/drivers/mtd/tests/mtd_oobtest.c b/drivers/mtd/tests/mtd_oobtest.c
index afbc3f8..a18e8d2 100644
--- a/drivers/mtd/tests/mtd_oobtest.c
+++ b/drivers/mtd/tests/mtd_oobtest.c
@@ -136,7 +136,7 @@ static int write_eraseblock(int ebnum)
 		ops.ooblen    = use_len;
 		ops.oobretlen = 0;
 		ops.ooboffs   = use_offset;
-		ops.datbuf    = 0;
+		ops.datbuf    = NULL;
 		ops.oobbuf    = writebuf;
 		err = mtd->write_oob(mtd, addr, &ops);
 		if (err || ops.oobretlen != use_len) {
@@ -189,7 +189,7 @@ static int verify_eraseblock(int ebnum)
 		ops.ooblen    = use_len;
 		ops.oobretlen = 0;
 		ops.ooboffs   = use_offset;
-		ops.datbuf    = 0;
+		ops.datbuf    = NULL;
 		ops.oobbuf    = readbuf;
 		err = mtd->read_oob(mtd, addr, &ops);
 		if (err || ops.oobretlen != use_len) {
@@ -216,7 +216,7 @@ static int verify_eraseblock(int ebnum)
 			ops.ooblen    = mtd->ecclayout->oobavail;
 			ops.oobretlen = 0;
 			ops.ooboffs   = 0;
-			ops.datbuf    = 0;
+			ops.datbuf    = NULL;
 			ops.oobbuf    = readbuf;
 			err = mtd->read_oob(mtd, addr, &ops);
 			if (err || ops.oobretlen != mtd->ecclayout->oobavail) {
@@ -281,7 +281,7 @@ static int verify_eraseblock_in_one_go(int ebnum)
 	ops.ooblen    = len;
 	ops.oobretlen = 0;
 	ops.ooboffs   = 0;
-	ops.datbuf    = 0;
+	ops.datbuf    = NULL;
 	ops.oobbuf    = readbuf;
 	err = mtd->read_oob(mtd, addr, &ops);
 	if (err || ops.oobretlen != len) {
@@ -522,7 +522,7 @@ static int __init mtd_oobtest_init(void)
 	ops.ooblen    = 1;
 	ops.oobretlen = 0;
 	ops.ooboffs   = mtd->ecclayout->oobavail;
-	ops.datbuf    = 0;
+	ops.datbuf    = NULL;
 	ops.oobbuf    = writebuf;
 	printk(PRINT_PREF "attempting to start write past end of OOB\n");
 	printk(PRINT_PREF "an error is expected...\n");
@@ -542,7 +542,7 @@ static int __init mtd_oobtest_init(void)
 	ops.ooblen    = 1;
 	ops.oobretlen = 0;
 	ops.ooboffs   = mtd->ecclayout->oobavail;
-	ops.datbuf    = 0;
+	ops.datbuf    = NULL;
 	ops.oobbuf    = readbuf;
 	printk(PRINT_PREF "attempting to start read past end of OOB\n");
 	printk(PRINT_PREF "an error is expected...\n");
@@ -566,7 +566,7 @@ static int __init mtd_oobtest_init(void)
 		ops.ooblen    = mtd->ecclayout->oobavail + 1;
 		ops.oobretlen = 0;
 		ops.ooboffs   = 0;
-		ops.datbuf    = 0;
+		ops.datbuf    = NULL;
 		ops.oobbuf    = writebuf;
 		printk(PRINT_PREF "attempting to write past end of device\n");
 		printk(PRINT_PREF "an error is expected...\n");
@@ -586,7 +586,7 @@ static int __init mtd_oobtest_init(void)
 		ops.ooblen    = mtd->ecclayout->oobavail + 1;
 		ops.oobretlen = 0;
 		ops.ooboffs   = 0;
-		ops.datbuf    = 0;
+		ops.datbuf    = NULL;
 		ops.oobbuf    = readbuf;
 		printk(PRINT_PREF "attempting to read past end of device\n");
 		printk(PRINT_PREF "an error is expected...\n");
@@ -610,7 +610,7 @@ static int __init mtd_oobtest_init(void)
 		ops.ooblen    = mtd->ecclayout->oobavail;
 		ops.oobretlen = 0;
 		ops.ooboffs   = 1;
-		ops.datbuf    = 0;
+		ops.datbuf    = NULL;
 		ops.oobbuf    = writebuf;
 		printk(PRINT_PREF "attempting to write past end of device\n");
 		printk(PRINT_PREF "an error is expected...\n");
@@ -630,7 +630,7 @@ static int __init mtd_oobtest_init(void)
 		ops.ooblen    = mtd->ecclayout->oobavail;
 		ops.oobretlen = 0;
 		ops.ooboffs   = 1;
-		ops.datbuf    = 0;
+		ops.datbuf    = NULL;
 		ops.oobbuf    = readbuf;
 		printk(PRINT_PREF "attempting to read past end of device\n");
 		printk(PRINT_PREF "an error is expected...\n");
@@ -670,7 +670,7 @@ static int __init mtd_oobtest_init(void)
 			ops.ooblen    = sz;
 			ops.oobretlen = 0;
 			ops.ooboffs   = 0;
-			ops.datbuf    = 0;
+			ops.datbuf    = NULL;
 			ops.oobbuf    = writebuf;
 			err = mtd->write_oob(mtd, addr, &ops);
 			if (err)
@@ -698,7 +698,7 @@ static int __init mtd_oobtest_init(void)
 		ops.ooblen    = mtd->ecclayout->oobavail * 2;
 		ops.oobretlen = 0;
 		ops.ooboffs   = 0;
-		ops.datbuf    = 0;
+		ops.datbuf    = NULL;
 		ops.oobbuf    = readbuf;
 		err = mtd->read_oob(mtd, addr, &ops);
 		if (err)
diff --git a/drivers/mtd/tests/mtd_readtest.c b/drivers/mtd/tests/mtd_readtest.c
index 645e77f..79fc453 100644
--- a/drivers/mtd/tests/mtd_readtest.c
+++ b/drivers/mtd/tests/mtd_readtest.c
@@ -71,7 +71,7 @@ static int read_eraseblock_by_page(int ebnum)
 			ops.ooblen    = mtd->oobsize;
 			ops.oobretlen = 0;
 			ops.ooboffs   = 0;
-			ops.datbuf    = 0;
+			ops.datbuf    = NULL;
 			ops.oobbuf    = oobbuf;
 			ret = mtd->read_oob(mtd, addr, &ops);
 			if (ret || ops.oobretlen != mtd->oobsize) {

[PATCH 06/16] MTD: tests: fix sparse warnings: Using plain integer as NULL pointer

[Hannes Eder]

Fix this sparse warnings:
  drivers/mtd/tests/mtd_oobtest.c:139:33: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:192:33: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:219:41: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:284:25: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:525:25: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:545:25: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:569:33: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:589:33: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:613:33: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:633:33: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:673:41: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_oobtest.c:701:33: warning: Using plain integer as NULL pointer
  drivers/mtd/tests/mtd_readtest.c:74:41: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 drivers/mtd/tests/mtd_oobtest.c  |   24 ++++++++++++------------
 drivers/mtd/tests/mtd_readtest.c |    2 +-
 2 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/drivers/mtd/tests/mtd_oobtest.c b/drivers/mtd/tests/mtd_oobtest.c
index afbc3f8..a18e8d2 100644
--- a/drivers/mtd/tests/mtd_oobtest.c
+++ b/drivers/mtd/tests/mtd_oobtest.c
@@ -136,7 +136,7 @@ static int write_eraseblock(int ebnum)
 		ops.ooblen    = use_len;
 		ops.oobretlen = 0;
 		ops.ooboffs   = use_offset;
-		ops.datbuf    = 0;
+		ops.datbuf    = NULL;
 		ops.oobbuf    = writebuf;
 		err = mtd->write_oob(mtd, addr, &ops);
 		if (err || ops.oobretlen != use_len) {
@@ -189,7 +189,7 @@ static int verify_eraseblock(int ebnum)
 		ops.ooblen    = use_len;
 		ops.oobretlen = 0;
 		ops.ooboffs   = use_offset;
-		ops.datbuf    = 0;
+		ops.datbuf    = NULL;
 		ops.oobbuf    = readbuf;
 		err = mtd->read_oob(mtd, addr, &ops);
 		if (err || ops.oobretlen != use_len) {
@@ -216,7 +216,7 @@ static int verify_eraseblock(int ebnum)
 			ops.ooblen    = mtd->ecclayout->oobavail;
 			ops.oobretlen = 0;
 			ops.ooboffs   = 0;
-			ops.datbuf    = 0;
+			ops.datbuf    = NULL;
 			ops.oobbuf    = readbuf;
 			err = mtd->read_oob(mtd, addr, &ops);
 			if (err || ops.oobretlen != mtd->ecclayout->oobavail) {
@@ -281,7 +281,7 @@ static int verify_eraseblock_in_one_go(int ebnum)
 	ops.ooblen    = len;
 	ops.oobretlen = 0;
 	ops.ooboffs   = 0;
-	ops.datbuf    = 0;
+	ops.datbuf    = NULL;
 	ops.oobbuf    = readbuf;
 	err = mtd->read_oob(mtd, addr, &ops);
 	if (err || ops.oobretlen != len) {
@@ -522,7 +522,7 @@ static int __init mtd_oobtest_init(void)
 	ops.ooblen    = 1;
 	ops.oobretlen = 0;
 	ops.ooboffs   = mtd->ecclayout->oobavail;
-	ops.datbuf    = 0;
+	ops.datbuf    = NULL;
 	ops.oobbuf    = writebuf;
 	printk(PRINT_PREF "attempting to start write past end of OOB\n");
 	printk(PRINT_PREF "an error is expected...\n");
@@ -542,7 +542,7 @@ static int __init mtd_oobtest_init(void)
 	ops.ooblen    = 1;
 	ops.oobretlen = 0;
 	ops.ooboffs   = mtd->ecclayout->oobavail;
-	ops.datbuf    = 0;
+	ops.datbuf    = NULL;
 	ops.oobbuf    = readbuf;
 	printk(PRINT_PREF "attempting to start read past end of OOB\n");
 	printk(PRINT_PREF "an error is expected...\n");
@@ -566,7 +566,7 @@ static int __init mtd_oobtest_init(void)
 		ops.ooblen    = mtd->ecclayout->oobavail + 1;
 		ops.oobretlen = 0;
 		ops.ooboffs   = 0;
-		ops.datbuf    = 0;
+		ops.datbuf    = NULL;
 		ops.oobbuf    = writebuf;
 		printk(PRINT_PREF "attempting to write past end of device\n");
 		printk(PRINT_PREF "an error is expected...\n");
@@ -586,7 +586,7 @@ static int __init mtd_oobtest_init(void)
 		ops.ooblen    = mtd->ecclayout->oobavail + 1;
 		ops.oobretlen = 0;
 		ops.ooboffs   = 0;
-		ops.datbuf    = 0;
+		ops.datbuf    = NULL;
 		ops.oobbuf    = readbuf;
 		printk(PRINT_PREF "attempting to read past end of device\n");
 		printk(PRINT_PREF "an error is expected...\n");
@@ -610,7 +610,7 @@ static int __init mtd_oobtest_init(void)
 		ops.ooblen    = mtd->ecclayout->oobavail;
 		ops.oobretlen = 0;
 		ops.ooboffs   = 1;
-		ops.datbuf    = 0;
+		ops.datbuf    = NULL;
 		ops.oobbuf    = writebuf;
 		printk(PRINT_PREF "attempting to write past end of device\n");
 		printk(PRINT_PREF "an error is expected...\n");
@@ -630,7 +630,7 @@ static int __init mtd_oobtest_init(void)
 		ops.ooblen    = mtd->ecclayout->oobavail;
 		ops.oobretlen = 0;
 		ops.ooboffs   = 1;
-		ops.datbuf    = 0;
+		ops.datbuf    = NULL;
 		ops.oobbuf    = readbuf;
 		printk(PRINT_PREF "attempting to read past end of device\n");
 		printk(PRINT_PREF "an error is expected...\n");
@@ -670,7 +670,7 @@ static int __init mtd_oobtest_init(void)
 			ops.ooblen    = sz;
 			ops.oobretlen = 0;
 			ops.ooboffs   = 0;
-			ops.datbuf    = 0;
+			ops.datbuf    = NULL;
 			ops.oobbuf    = writebuf;
 			err = mtd->write_oob(mtd, addr, &ops);
 			if (err)
@@ -698,7 +698,7 @@ static int __init mtd_oobtest_init(void)
 		ops.ooblen    = mtd->ecclayout->oobavail * 2;
 		ops.oobretlen = 0;
 		ops.ooboffs   = 0;
-		ops.datbuf    = 0;
+		ops.datbuf    = NULL;
 		ops.oobbuf    = readbuf;
 		err = mtd->read_oob(mtd, addr, &ops);
 		if (err)
diff --git a/drivers/mtd/tests/mtd_readtest.c b/drivers/mtd/tests/mtd_readtest.c
index 645e77f..79fc453 100644
--- a/drivers/mtd/tests/mtd_readtest.c
+++ b/drivers/mtd/tests/mtd_readtest.c
@@ -71,7 +71,7 @@ static int read_eraseblock_by_page(int ebnum)
 			ops.ooblen    = mtd->oobsize;
 			ops.oobretlen = 0;
 			ops.ooboffs   = 0;
-			ops.datbuf    = 0;
+			ops.datbuf    = NULL;
 			ops.oobbuf    = oobbuf;
 			ret = mtd->read_oob(mtd, addr, &ops);
 			if (ret || ops.oobretlen != mtd->oobsize) {

Re: [PATCH 06/16] MTD: tests: fix sparse warnings: Using plain

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:02 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warnings:
>  drivers/mtd/tests/mtd_oobtest.c:139:33: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:192:33: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:219:41: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:284:25: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:525:25: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:545:25: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:569:33: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:589:33: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:613:33: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:633:33: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:673:41: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:701:33: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_readtest.c:74:41: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  drivers/mtd/tests/mtd_oobtest.c  |   24 ++++++++++++------------
>  drivers/mtd/tests/mtd_readtest.c |    2 +-
>  2 files changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/mtd/tests/mtd_oobtest.c b/drivers/mtd/tests/mtd_oobtest.c
> index afbc3f8..a18e8d2 100644
> --- a/drivers/mtd/tests/mtd_oobtest.c
> +++ b/drivers/mtd/tests/mtd_oobtest.c
> @@ -136,7 +136,7 @@ static int write_eraseblock(int ebnum)
>                ops.ooblen    = use_len;
>                ops.oobretlen = 0;
>                ops.ooboffs   = use_offset;
> -               ops.datbuf    = 0;
> +               ops.datbuf    = NULL;
>                ops.oobbuf    = writebuf;
>                err = mtd->write_oob(mtd, addr, &ops);
>                if (err || ops.oobretlen != use_len) {
> @@ -189,7 +189,7 @@ static int verify_eraseblock(int ebnum)
>                ops.ooblen    = use_len;
>                ops.oobretlen = 0;
>                ops.ooboffs   = use_offset;
> -               ops.datbuf    = 0;
> +               ops.datbuf    = NULL;
>                ops.oobbuf    = readbuf;
>                err = mtd->read_oob(mtd, addr, &ops);
>                if (err || ops.oobretlen != use_len) {
> @@ -216,7 +216,7 @@ static int verify_eraseblock(int ebnum)
>                        ops.ooblen    = mtd->ecclayout->oobavail;
>                        ops.oobretlen = 0;
>                        ops.ooboffs   = 0;
> -                       ops.datbuf    = 0;
> +                       ops.datbuf    = NULL;
>                        ops.oobbuf    = readbuf;
>                        err = mtd->read_oob(mtd, addr, &ops);
>                        if (err || ops.oobretlen != mtd->ecclayout->oobavail) {
> @@ -281,7 +281,7 @@ static int verify_eraseblock_in_one_go(int ebnum)
>        ops.ooblen    = len;
>        ops.oobretlen = 0;
>        ops.ooboffs   = 0;
> -       ops.datbuf    = 0;
> +       ops.datbuf    = NULL;
>        ops.oobbuf    = readbuf;
>        err = mtd->read_oob(mtd, addr, &ops);
>        if (err || ops.oobretlen != len) {
> @@ -522,7 +522,7 @@ static int __init mtd_oobtest_init(void)
>        ops.ooblen    = 1;
>        ops.oobretlen = 0;
>        ops.ooboffs   = mtd->ecclayout->oobavail;
> -       ops.datbuf    = 0;
> +       ops.datbuf    = NULL;
>        ops.oobbuf    = writebuf;
>        printk(PRINT_PREF "attempting to start write past end of OOB\n");
>        printk(PRINT_PREF "an error is expected...\n");
> @@ -542,7 +542,7 @@ static int __init mtd_oobtest_init(void)
>        ops.ooblen    = 1;
>        ops.oobretlen = 0;
>        ops.ooboffs   = mtd->ecclayout->oobavail;
> -       ops.datbuf    = 0;
> +       ops.datbuf    = NULL;
>        ops.oobbuf    = readbuf;
>        printk(PRINT_PREF "attempting to start read past end of OOB\n");
>        printk(PRINT_PREF "an error is expected...\n");
> @@ -566,7 +566,7 @@ static int __init mtd_oobtest_init(void)
>                ops.ooblen    = mtd->ecclayout->oobavail + 1;
>                ops.oobretlen = 0;
>                ops.ooboffs   = 0;
> -               ops.datbuf    = 0;
> +               ops.datbuf    = NULL;
>                ops.oobbuf    = writebuf;
>                printk(PRINT_PREF "attempting to write past end of device\n");
>                printk(PRINT_PREF "an error is expected...\n");
> @@ -586,7 +586,7 @@ static int __init mtd_oobtest_init(void)
>                ops.ooblen    = mtd->ecclayout->oobavail + 1;
>                ops.oobretlen = 0;
>                ops.ooboffs   = 0;
> -               ops.datbuf    = 0;
> +               ops.datbuf    = NULL;
>                ops.oobbuf    = readbuf;
>                printk(PRINT_PREF "attempting to read past end of device\n");
>                printk(PRINT_PREF "an error is expected...\n");
> @@ -610,7 +610,7 @@ static int __init mtd_oobtest_init(void)
>                ops.ooblen    = mtd->ecclayout->oobavail;
>                ops.oobretlen = 0;
>                ops.ooboffs   = 1;
> -               ops.datbuf    = 0;
> +               ops.datbuf    = NULL;
>                ops.oobbuf    = writebuf;
>                printk(PRINT_PREF "attempting to write past end of device\n");
>                printk(PRINT_PREF "an error is expected...\n");
> @@ -630,7 +630,7 @@ static int __init mtd_oobtest_init(void)
>                ops.ooblen    = mtd->ecclayout->oobavail;
>                ops.oobretlen = 0;
>                ops.ooboffs   = 1;
> -               ops.datbuf    = 0;
> +               ops.datbuf    = NULL;
>                ops.oobbuf    = readbuf;
>                printk(PRINT_PREF "attempting to read past end of device\n");
>                printk(PRINT_PREF "an error is expected...\n");
> @@ -670,7 +670,7 @@ static int __init mtd_oobtest_init(void)
>                        ops.ooblen    = sz;
>                        ops.oobretlen = 0;
>                        ops.ooboffs   = 0;
> -                       ops.datbuf    = 0;
> +                       ops.datbuf    = NULL;
>                        ops.oobbuf    = writebuf;
>                        err = mtd->write_oob(mtd, addr, &ops);
>                        if (err)
> @@ -698,7 +698,7 @@ static int __init mtd_oobtest_init(void)
>                ops.ooblen    = mtd->ecclayout->oobavail * 2;
>                ops.oobretlen = 0;
>                ops.ooboffs   = 0;
> -               ops.datbuf    = 0;
> +               ops.datbuf    = NULL;
>                ops.oobbuf    = readbuf;
>                err = mtd->read_oob(mtd, addr, &ops);
>                if (err)
> diff --git a/drivers/mtd/tests/mtd_readtest.c b/drivers/mtd/tests/mtd_readtest.c
> index 645e77f..79fc453 100644
> --- a/drivers/mtd/tests/mtd_readtest.c
> +++ b/drivers/mtd/tests/mtd_readtest.c
> @@ -71,7 +71,7 @@ static int read_eraseblock_by_page(int ebnum)
>                        ops.ooblen    = mtd->oobsize;
>                        ops.oobretlen = 0;
>                        ops.ooboffs   = 0;
> -                       ops.datbuf    = 0;
> +                       ops.datbuf    = NULL;
>                        ops.oobbuf    = oobbuf;
>                        ret = mtd->read_oob(mtd, addr, &ops);
>                        if (ret || ops.oobretlen != mtd->oobsize) {
>
>
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 06/16] MTD: tests: fix sparse warnings: Using plain integer as NULL pointer

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:02 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warnings:
>  drivers/mtd/tests/mtd_oobtest.c:139:33: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:192:33: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:219:41: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:284:25: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:525:25: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:545:25: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:569:33: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:589:33: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:613:33: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:633:33: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:673:41: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_oobtest.c:701:33: warning: Using plain integer as NULL pointer
>  drivers/mtd/tests/mtd_readtest.c:74:41: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  drivers/mtd/tests/mtd_oobtest.c  |   24 ++++++++++++------------
>  drivers/mtd/tests/mtd_readtest.c |    2 +-
>  2 files changed, 13 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/mtd/tests/mtd_oobtest.c b/drivers/mtd/tests/mtd_oobtest.c
> index afbc3f8..a18e8d2 100644
> --- a/drivers/mtd/tests/mtd_oobtest.c
> +++ b/drivers/mtd/tests/mtd_oobtest.c
> @@ -136,7 +136,7 @@ static int write_eraseblock(int ebnum)
>                ops.ooblen    = use_len;
>                ops.oobretlen = 0;
>                ops.ooboffs   = use_offset;
> -               ops.datbuf    = 0;
> +               ops.datbuf    = NULL;
>                ops.oobbuf    = writebuf;
>                err = mtd->write_oob(mtd, addr, &ops);
>                if (err || ops.oobretlen != use_len) {
> @@ -189,7 +189,7 @@ static int verify_eraseblock(int ebnum)
>                ops.ooblen    = use_len;
>                ops.oobretlen = 0;
>                ops.ooboffs   = use_offset;
> -               ops.datbuf    = 0;
> +               ops.datbuf    = NULL;
>                ops.oobbuf    = readbuf;
>                err = mtd->read_oob(mtd, addr, &ops);
>                if (err || ops.oobretlen != use_len) {
> @@ -216,7 +216,7 @@ static int verify_eraseblock(int ebnum)
>                        ops.ooblen    = mtd->ecclayout->oobavail;
>                        ops.oobretlen = 0;
>                        ops.ooboffs   = 0;
> -                       ops.datbuf    = 0;
> +                       ops.datbuf    = NULL;
>                        ops.oobbuf    = readbuf;
>                        err = mtd->read_oob(mtd, addr, &ops);
>                        if (err || ops.oobretlen != mtd->ecclayout->oobavail) {
> @@ -281,7 +281,7 @@ static int verify_eraseblock_in_one_go(int ebnum)
>        ops.ooblen    = len;
>        ops.oobretlen = 0;
>        ops.ooboffs   = 0;
> -       ops.datbuf    = 0;
> +       ops.datbuf    = NULL;
>        ops.oobbuf    = readbuf;
>        err = mtd->read_oob(mtd, addr, &ops);
>        if (err || ops.oobretlen != len) {
> @@ -522,7 +522,7 @@ static int __init mtd_oobtest_init(void)
>        ops.ooblen    = 1;
>        ops.oobretlen = 0;
>        ops.ooboffs   = mtd->ecclayout->oobavail;
> -       ops.datbuf    = 0;
> +       ops.datbuf    = NULL;
>        ops.oobbuf    = writebuf;
>        printk(PRINT_PREF "attempting to start write past end of OOB\n");
>        printk(PRINT_PREF "an error is expected...\n");
> @@ -542,7 +542,7 @@ static int __init mtd_oobtest_init(void)
>        ops.ooblen    = 1;
>        ops.oobretlen = 0;
>        ops.ooboffs   = mtd->ecclayout->oobavail;
> -       ops.datbuf    = 0;
> +       ops.datbuf    = NULL;
>        ops.oobbuf    = readbuf;
>        printk(PRINT_PREF "attempting to start read past end of OOB\n");
>        printk(PRINT_PREF "an error is expected...\n");
> @@ -566,7 +566,7 @@ static int __init mtd_oobtest_init(void)
>                ops.ooblen    = mtd->ecclayout->oobavail + 1;
>                ops.oobretlen = 0;
>                ops.ooboffs   = 0;
> -               ops.datbuf    = 0;
> +               ops.datbuf    = NULL;
>                ops.oobbuf    = writebuf;
>                printk(PRINT_PREF "attempting to write past end of device\n");
>                printk(PRINT_PREF "an error is expected...\n");
> @@ -586,7 +586,7 @@ static int __init mtd_oobtest_init(void)
>                ops.ooblen    = mtd->ecclayout->oobavail + 1;
>                ops.oobretlen = 0;
>                ops.ooboffs   = 0;
> -               ops.datbuf    = 0;
> +               ops.datbuf    = NULL;
>                ops.oobbuf    = readbuf;
>                printk(PRINT_PREF "attempting to read past end of device\n");
>                printk(PRINT_PREF "an error is expected...\n");
> @@ -610,7 +610,7 @@ static int __init mtd_oobtest_init(void)
>                ops.ooblen    = mtd->ecclayout->oobavail;
>                ops.oobretlen = 0;
>                ops.ooboffs   = 1;
> -               ops.datbuf    = 0;
> +               ops.datbuf    = NULL;
>                ops.oobbuf    = writebuf;
>                printk(PRINT_PREF "attempting to write past end of device\n");
>                printk(PRINT_PREF "an error is expected...\n");
> @@ -630,7 +630,7 @@ static int __init mtd_oobtest_init(void)
>                ops.ooblen    = mtd->ecclayout->oobavail;
>                ops.oobretlen = 0;
>                ops.ooboffs   = 1;
> -               ops.datbuf    = 0;
> +               ops.datbuf    = NULL;
>                ops.oobbuf    = readbuf;
>                printk(PRINT_PREF "attempting to read past end of device\n");
>                printk(PRINT_PREF "an error is expected...\n");
> @@ -670,7 +670,7 @@ static int __init mtd_oobtest_init(void)
>                        ops.ooblen    = sz;
>                        ops.oobretlen = 0;
>                        ops.ooboffs   = 0;
> -                       ops.datbuf    = 0;
> +                       ops.datbuf    = NULL;
>                        ops.oobbuf    = writebuf;
>                        err = mtd->write_oob(mtd, addr, &ops);
>                        if (err)
> @@ -698,7 +698,7 @@ static int __init mtd_oobtest_init(void)
>                ops.ooblen    = mtd->ecclayout->oobavail * 2;
>                ops.oobretlen = 0;
>                ops.ooboffs   = 0;
> -               ops.datbuf    = 0;
> +               ops.datbuf    = NULL;
>                ops.oobbuf    = readbuf;
>                err = mtd->read_oob(mtd, addr, &ops);
>                if (err)
> diff --git a/drivers/mtd/tests/mtd_readtest.c b/drivers/mtd/tests/mtd_readtest.c
> index 645e77f..79fc453 100644
> --- a/drivers/mtd/tests/mtd_readtest.c
> +++ b/drivers/mtd/tests/mtd_readtest.c
> @@ -71,7 +71,7 @@ static int read_eraseblock_by_page(int ebnum)
>                        ops.ooblen    = mtd->oobsize;
>                        ops.oobretlen = 0;
>                        ops.ooboffs   = 0;
> -                       ops.datbuf    = 0;
> +                       ops.datbuf    = NULL;
>                        ops.oobbuf    = oobbuf;
>                        ret = mtd->read_oob(mtd, addr, &ops);
>                        if (ret || ops.oobretlen != mtd->oobsize) {
>
>

Re: [PATCH 06/16] MTD: tests: fix sparse warnings: Using plain integer

[Artem Bityutskiy]

Hannes Eder wrote:
> [Added To]
> 
> On Sun, Feb 22, 2009 at 1:02 AM, Hannes Eder <hannes@hanneseder.net> wrote:
>> Fix this sparse warnings:
>>  drivers/mtd/tests/mtd_oobtest.c:139:33: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:192:33: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:219:41: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:284:25: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:525:25: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:545:25: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:569:33: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:589:33: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:613:33: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:633:33: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:673:41: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:701:33: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_readtest.c:74:41: warning: Using plain integer as NULL pointer
>>
>> Signed-off-by: Hannes Eder <hannes@hanneseder.net>

Acked-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

-- 
Best Regards,
Artem Bityutskiy (Артём Битюцкий)

Re: [PATCH 06/16] MTD: tests: fix sparse warnings: Using plain integer as NULL pointer

[Artem Bityutskiy]

Hannes Eder wrote:
> [Added To]
> 
> On Sun, Feb 22, 2009 at 1:02 AM, Hannes Eder <hannes@hanneseder.net> wrote:
>> Fix this sparse warnings:
>>  drivers/mtd/tests/mtd_oobtest.c:139:33: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:192:33: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:219:41: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:284:25: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:525:25: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:545:25: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:569:33: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:589:33: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:613:33: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:633:33: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:673:41: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_oobtest.c:701:33: warning: Using plain integer as NULL pointer
>>  drivers/mtd/tests/mtd_readtest.c:74:41: warning: Using plain integer as NULL pointer
>>
>> Signed-off-by: Hannes Eder <hannes@hanneseder.net>

Acked-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

-- 
Best Regards,
Artem Bityutskiy (Артём Битюцкий)

[PATCH 07/16] panasonic-laptop: fix sparse warning: Using plain

[Hannes Eder]

Fix this sparse warning:
  drivers/platform/x86/panasonic-laptop.c:273:70: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 drivers/platform/x86/panasonic-laptop.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/platform/x86/panasonic-laptop.c b/drivers/platform/x86/panasonic-laptop.c
index c47a44d..e178cde 100644
--- a/drivers/platform/x86/panasonic-laptop.c
+++ b/drivers/platform/x86/panasonic-laptop.c
@@ -270,7 +270,7 @@ static int acpi_pcc_retrieve_biosdata(struct pcc_acpi *pcc, u32 *sinf)
 	union acpi_object *hkey = NULL;
 	int i;
 
-	status = acpi_evaluate_object(pcc->handle, METHOD_HKEY_SINF, 0,
+	status = acpi_evaluate_object(pcc->handle, METHOD_HKEY_SINF, NULL,
 				      &buffer);
 	if (ACPI_FAILURE(status)) {
 		ACPI_DEBUG_PRINT((ACPI_DB_ERROR,

[PATCH 07/16] panasonic-laptop: fix sparse warning: Using plain integer as NULL pointer

[Hannes Eder]

Fix this sparse warning:
  drivers/platform/x86/panasonic-laptop.c:273:70: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 drivers/platform/x86/panasonic-laptop.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/platform/x86/panasonic-laptop.c b/drivers/platform/x86/panasonic-laptop.c
index c47a44d..e178cde 100644
--- a/drivers/platform/x86/panasonic-laptop.c
+++ b/drivers/platform/x86/panasonic-laptop.c
@@ -270,7 +270,7 @@ static int acpi_pcc_retrieve_biosdata(struct pcc_acpi *pcc, u32 *sinf)
 	union acpi_object *hkey = NULL;
 	int i;
 
-	status = acpi_evaluate_object(pcc->handle, METHOD_HKEY_SINF, 0,
+	status = acpi_evaluate_object(pcc->handle, METHOD_HKEY_SINF, NULL,
 				      &buffer);
 	if (ACPI_FAILURE(status)) {
 		ACPI_DEBUG_PRINT((ACPI_DB_ERROR,

Re: [PATCH 07/16] panasonic-laptop: fix sparse warning: Using plain

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:02 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warning:
>  drivers/platform/x86/panasonic-laptop.c:273:70: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  drivers/platform/x86/panasonic-laptop.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/platform/x86/panasonic-laptop.c b/drivers/platform/x86/panasonic-laptop.c
> index c47a44d..e178cde 100644
> --- a/drivers/platform/x86/panasonic-laptop.c
> +++ b/drivers/platform/x86/panasonic-laptop.c
> @@ -270,7 +270,7 @@ static int acpi_pcc_retrieve_biosdata(struct pcc_acpi *pcc, u32 *sinf)
>        union acpi_object *hkey = NULL;
>        int i;
>
> -       status = acpi_evaluate_object(pcc->handle, METHOD_HKEY_SINF, 0,
> +       status = acpi_evaluate_object(pcc->handle, METHOD_HKEY_SINF, NULL,
>                                      &buffer);
>        if (ACPI_FAILURE(status)) {
>                ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
>
>
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 07/16] panasonic-laptop: fix sparse warning: Using plain integer as NULL pointer

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:02 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warning:
>  drivers/platform/x86/panasonic-laptop.c:273:70: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  drivers/platform/x86/panasonic-laptop.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/platform/x86/panasonic-laptop.c b/drivers/platform/x86/panasonic-laptop.c
> index c47a44d..e178cde 100644
> --- a/drivers/platform/x86/panasonic-laptop.c
> +++ b/drivers/platform/x86/panasonic-laptop.c
> @@ -270,7 +270,7 @@ static int acpi_pcc_retrieve_biosdata(struct pcc_acpi *pcc, u32 *sinf)
>        union acpi_object *hkey = NULL;
>        int i;
>
> -       status = acpi_evaluate_object(pcc->handle, METHOD_HKEY_SINF, 0,
> +       status = acpi_evaluate_object(pcc->handle, METHOD_HKEY_SINF, NULL,
>                                      &buffer);
>        if (ACPI_FAILURE(status)) {
>                ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
>
>

[PATCH 08/16] [SCSI] FlashPoint: fix sparse warnings: Using plain

[Hannes Eder]

Fix this sparse warnings:
  drivers/scsi/FlashPoint.c:906:9: warning: Using plain integer as NULL pointer
  drivers/scsi/FlashPoint.c:907:53: warning: Using plain integer as NULL pointer
  drivers/scsi/FlashPoint.c:922:1: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 drivers/scsi/FlashPoint.c |    7 +++----
 1 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/scsi/FlashPoint.c b/drivers/scsi/FlashPoint.c
index cb601cf..9b2ea80 100644
--- a/drivers/scsi/FlashPoint.c
+++ b/drivers/scsi/FlashPoint.c
@@ -903,8 +903,8 @@ static void FPT_autoCmdCmplt(unsigned long p_port, unsigned char p_card);
 static void FPT_autoLoadDefaultMap(unsigned long p_port);
 
 static struct sccb_mgr_tar_info FPT_sccbMgrTbl[MAX_CARDS][MAX_SCSI_TAR] -    { {{0}} };
-static struct sccb_card FPT_BL_Card[MAX_CARDS] = { {0} };
+    { {{NULL}} };
+static struct sccb_card FPT_BL_Card[MAX_CARDS] = { {NULL} };
 static SCCBSCAM_INFO FPT_scamInfo[MAX_SCSI_TAR] = { {{0}} };
 static struct nvram_info FPT_nvRamInfo[MAX_MB_CARDS] = { {0} };
 
@@ -918,8 +918,7 @@ static unsigned char FPT_scamHAString[]  
 static unsigned short FPT_default_intena = 0;
 
-static void (*FPT_s_PhaseTbl[8]) (unsigned long, unsigned char) = {
-0};
+static void (*FPT_s_PhaseTbl[8]) (unsigned long, unsigned char) = {NULL};
 
 /*---------------------------------------------------------------------
  *

[PATCH 08/16] [SCSI] FlashPoint: fix sparse warnings: Using plain integer as NULL pointer

[Hannes Eder]

Fix this sparse warnings:
  drivers/scsi/FlashPoint.c:906:9: warning: Using plain integer as NULL pointer
  drivers/scsi/FlashPoint.c:907:53: warning: Using plain integer as NULL pointer
  drivers/scsi/FlashPoint.c:922:1: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 drivers/scsi/FlashPoint.c |    7 +++----
 1 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/scsi/FlashPoint.c b/drivers/scsi/FlashPoint.c
index cb601cf..9b2ea80 100644
--- a/drivers/scsi/FlashPoint.c
+++ b/drivers/scsi/FlashPoint.c
@@ -903,8 +903,8 @@ static void FPT_autoCmdCmplt(unsigned long p_port, unsigned char p_card);
 static void FPT_autoLoadDefaultMap(unsigned long p_port);
 
 static struct sccb_mgr_tar_info FPT_sccbMgrTbl[MAX_CARDS][MAX_SCSI_TAR] =
-    { {{0}} };
-static struct sccb_card FPT_BL_Card[MAX_CARDS] = { {0} };
+    { {{NULL}} };
+static struct sccb_card FPT_BL_Card[MAX_CARDS] = { {NULL} };
 static SCCBSCAM_INFO FPT_scamInfo[MAX_SCSI_TAR] = { {{0}} };
 static struct nvram_info FPT_nvRamInfo[MAX_MB_CARDS] = { {0} };
 
@@ -918,8 +918,7 @@ static unsigned char FPT_scamHAString[] =
 
 static unsigned short FPT_default_intena = 0;
 
-static void (*FPT_s_PhaseTbl[8]) (unsigned long, unsigned char) = {
-0};
+static void (*FPT_s_PhaseTbl[8]) (unsigned long, unsigned char) = {NULL};
 
 /*---------------------------------------------------------------------
  *

[PATCH 08/16 v2] [SCSI] FlashPoint: fix sparse warnings: Using plain

[Hannes Eder]

Fix this sparse warnings:
  drivers/scsi/FlashPoint.c:906:9: warning: Using plain integer as NULL pointer
  drivers/scsi/FlashPoint.c:907:53: warning: Using plain integer as NULL pointer
  drivers/scsi/FlashPoint.c:922:1: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
v2: fix checkpatch.pl issue.

 drivers/scsi/FlashPoint.c |    7 +++----
 1 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/scsi/FlashPoint.c b/drivers/scsi/FlashPoint.c
index cb601cf..61e268e 100644
--- a/drivers/scsi/FlashPoint.c
+++ b/drivers/scsi/FlashPoint.c
@@ -903,8 +903,8 @@ static void FPT_autoCmdCmplt(unsigned long p_port, unsigned char p_card);
 static void FPT_autoLoadDefaultMap(unsigned long p_port);
 
 static struct sccb_mgr_tar_info FPT_sccbMgrTbl[MAX_CARDS][MAX_SCSI_TAR] -    { {{0}} };
-static struct sccb_card FPT_BL_Card[MAX_CARDS] = { {0} };
+    { { {NULL} } };
+static struct sccb_card FPT_BL_Card[MAX_CARDS] = { {NULL} };
 static SCCBSCAM_INFO FPT_scamInfo[MAX_SCSI_TAR] = { {{0}} };
 static struct nvram_info FPT_nvRamInfo[MAX_MB_CARDS] = { {0} };
 
@@ -918,8 +918,7 @@ static unsigned char FPT_scamHAString[]  
 static unsigned short FPT_default_intena = 0;
 
-static void (*FPT_s_PhaseTbl[8]) (unsigned long, unsigned char) = {
-0};
+static void (*FPT_s_PhaseTbl[8]) (unsigned long, unsigned char) = {NULL};
 
 /*---------------------------------------------------------------------
  *

[PATCH 08/16 v2] [SCSI] FlashPoint: fix sparse warnings: Using plain integer as NULL pointer

[Hannes Eder]

Fix this sparse warnings:
  drivers/scsi/FlashPoint.c:906:9: warning: Using plain integer as NULL pointer
  drivers/scsi/FlashPoint.c:907:53: warning: Using plain integer as NULL pointer
  drivers/scsi/FlashPoint.c:922:1: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
v2: fix checkpatch.pl issue.

 drivers/scsi/FlashPoint.c |    7 +++----
 1 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/scsi/FlashPoint.c b/drivers/scsi/FlashPoint.c
index cb601cf..61e268e 100644
--- a/drivers/scsi/FlashPoint.c
+++ b/drivers/scsi/FlashPoint.c
@@ -903,8 +903,8 @@ static void FPT_autoCmdCmplt(unsigned long p_port, unsigned char p_card);
 static void FPT_autoLoadDefaultMap(unsigned long p_port);
 
 static struct sccb_mgr_tar_info FPT_sccbMgrTbl[MAX_CARDS][MAX_SCSI_TAR] =
-    { {{0}} };
-static struct sccb_card FPT_BL_Card[MAX_CARDS] = { {0} };
+    { { {NULL} } };
+static struct sccb_card FPT_BL_Card[MAX_CARDS] = { {NULL} };
 static SCCBSCAM_INFO FPT_scamInfo[MAX_SCSI_TAR] = { {{0}} };
 static struct nvram_info FPT_nvRamInfo[MAX_MB_CARDS] = { {0} };
 
@@ -918,8 +918,7 @@ static unsigned char FPT_scamHAString[] =
 
 static unsigned short FPT_default_intena = 0;
 
-static void (*FPT_s_PhaseTbl[8]) (unsigned long, unsigned char) = {
-0};
+static void (*FPT_s_PhaseTbl[8]) (unsigned long, unsigned char) = {NULL};
 
 /*---------------------------------------------------------------------
  *

[PATCH 09/16] usb host: fix sparse warning: Using plain integer as

[Hannes Eder]

Fix this sparse warning:
  drivers/usb/host/oxu210hp-hcd.c:2687:42: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 drivers/usb/host/oxu210hp-hcd.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/usb/host/oxu210hp-hcd.c b/drivers/usb/host/oxu210hp-hcd.c
index 2947c69..5ac489e 100644
--- a/drivers/usb/host/oxu210hp-hcd.c
+++ b/drivers/usb/host/oxu210hp-hcd.c
@@ -2684,7 +2684,7 @@ static int oxu_reset(struct usb_hcd *hcd)
 	oxu->urb_len = 0;
 
 	/* FIMXE */
-	hcd->self.controller->dma_mask = 0UL;
+	hcd->self.controller->dma_mask = NULL;
 
 	if (oxu->is_otg) {
 		oxu->caps = hcd->regs + OXU_OTG_CAP_OFFSET;

[PATCH 09/16] usb host: fix sparse warning: Using plain integer as NULL pointer

[Hannes Eder]

Fix this sparse warning:
  drivers/usb/host/oxu210hp-hcd.c:2687:42: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 drivers/usb/host/oxu210hp-hcd.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/usb/host/oxu210hp-hcd.c b/drivers/usb/host/oxu210hp-hcd.c
index 2947c69..5ac489e 100644
--- a/drivers/usb/host/oxu210hp-hcd.c
+++ b/drivers/usb/host/oxu210hp-hcd.c
@@ -2684,7 +2684,7 @@ static int oxu_reset(struct usb_hcd *hcd)
 	oxu->urb_len = 0;
 
 	/* FIMXE */
-	hcd->self.controller->dma_mask = 0UL;
+	hcd->self.controller->dma_mask = NULL;
 
 	if (oxu->is_otg) {
 		oxu->caps = hcd->regs + OXU_OTG_CAP_OFFSET;

Re: [PATCH 09/16] usb host: fix sparse warning: Using plain integer

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:04 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warning:
>  drivers/usb/host/oxu210hp-hcd.c:2687:42: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  drivers/usb/host/oxu210hp-hcd.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/usb/host/oxu210hp-hcd.c b/drivers/usb/host/oxu210hp-hcd.c
> index 2947c69..5ac489e 100644
> --- a/drivers/usb/host/oxu210hp-hcd.c
> +++ b/drivers/usb/host/oxu210hp-hcd.c
> @@ -2684,7 +2684,7 @@ static int oxu_reset(struct usb_hcd *hcd)
>        oxu->urb_len = 0;
>
>        /* FIMXE */
> -       hcd->self.controller->dma_mask = 0UL;
> +       hcd->self.controller->dma_mask = NULL;
>
>        if (oxu->is_otg) {
>                oxu->caps = hcd->regs + OXU_OTG_CAP_OFFSET;
>
>
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 09/16] usb host: fix sparse warning: Using plain integer as NULL pointer

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:04 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warning:
>  drivers/usb/host/oxu210hp-hcd.c:2687:42: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  drivers/usb/host/oxu210hp-hcd.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/usb/host/oxu210hp-hcd.c b/drivers/usb/host/oxu210hp-hcd.c
> index 2947c69..5ac489e 100644
> --- a/drivers/usb/host/oxu210hp-hcd.c
> +++ b/drivers/usb/host/oxu210hp-hcd.c
> @@ -2684,7 +2684,7 @@ static int oxu_reset(struct usb_hcd *hcd)
>        oxu->urb_len = 0;
>
>        /* FIMXE */
> -       hcd->self.controller->dma_mask = 0UL;
> +       hcd->self.controller->dma_mask = NULL;
>
>        if (oxu->is_otg) {
>                oxu->caps = hcd->regs + OXU_OTG_CAP_OFFSET;
>
>

[PATCH 10/16] viafb: fix sparse warning: Using plain integer as NULL

[Hannes Eder]

Fix this sparse warning:
  drivers/video/via/hw.c:2020:46: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 drivers/video/via/hw.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/video/via/hw.c b/drivers/video/via/hw.c
index fcd53ce..70d6b6c 100644
--- a/drivers/video/via/hw.c
+++ b/drivers/video/via/hw.c
@@ -2017,7 +2017,7 @@ static void init_gfx_chip_info(void)
 	/* Indentify GFX Chip Name */
 	for (i = 0; pciidlist[i].vendor != 0; i++) {
 		pdev = pci_get_device(pciidlist[i].vendor,
-			pciidlist[i].device, 0);
+			pciidlist[i].device, NULL);
 		if (pdev)
 			break;
 	}

[PATCH 10/16] viafb: fix sparse warning: Using plain integer as NULL pointer

[Hannes Eder]

Fix this sparse warning:
  drivers/video/via/hw.c:2020:46: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 drivers/video/via/hw.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/video/via/hw.c b/drivers/video/via/hw.c
index fcd53ce..70d6b6c 100644
--- a/drivers/video/via/hw.c
+++ b/drivers/video/via/hw.c
@@ -2017,7 +2017,7 @@ static void init_gfx_chip_info(void)
 	/* Indentify GFX Chip Name */
 	for (i = 0; pciidlist[i].vendor != 0; i++) {
 		pdev = pci_get_device(pciidlist[i].vendor,
-			pciidlist[i].device, 0);
+			pciidlist[i].device, NULL);
 		if (pdev)
 			break;
 	}

Re: [PATCH 10/16] viafb: fix sparse warning: Using plain integer as

[Hannes Eder]

[Added To, Cc]

On Sun, Feb 22, 2009 at 1:04 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warning:
>  drivers/video/via/hw.c:2020:46: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  drivers/video/via/hw.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/video/via/hw.c b/drivers/video/via/hw.c
> index fcd53ce..70d6b6c 100644
> --- a/drivers/video/via/hw.c
> +++ b/drivers/video/via/hw.c
> @@ -2017,7 +2017,7 @@ static void init_gfx_chip_info(void)
>        /* Indentify GFX Chip Name */
>        for (i = 0; pciidlist[i].vendor != 0; i++) {
>                pdev = pci_get_device(pciidlist[i].vendor,
> -                       pciidlist[i].device, 0);
> +                       pciidlist[i].device, NULL);
>                if (pdev)
>                        break;
>        }
>
>
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 10/16] viafb: fix sparse warning: Using plain integer as NULL pointer

[Hannes Eder]

[Added To, Cc]

On Sun, Feb 22, 2009 at 1:04 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warning:
>  drivers/video/via/hw.c:2020:46: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  drivers/video/via/hw.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/video/via/hw.c b/drivers/video/via/hw.c
> index fcd53ce..70d6b6c 100644
> --- a/drivers/video/via/hw.c
> +++ b/drivers/video/via/hw.c
> @@ -2017,7 +2017,7 @@ static void init_gfx_chip_info(void)
>        /* Indentify GFX Chip Name */
>        for (i = 0; pciidlist[i].vendor != 0; i++) {
>                pdev = pci_get_device(pciidlist[i].vendor,
> -                       pciidlist[i].device, 0);
> +                       pciidlist[i].device, NULL);
>                if (pdev)
>                        break;
>        }
>
>

[PATCH 11/16] fs/super.c: fix sparse warning: Using plain integer as

[Hannes Eder]

Fix this sparse warning:
  fs/super.c:849:26: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 fs/super.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/fs/super.c b/fs/super.c
index c38eef4..c6ad02d 100644
--- a/fs/super.c
+++ b/fs/super.c
@@ -846,7 +846,7 @@ void kill_block_super(struct super_block *sb)
 	struct block_device *bdev = sb->s_bdev;
 	fmode_t mode = sb->s_mode;
 
-	bdev->bd_super = 0;
+	bdev->bd_super = NULL;
 	generic_shutdown_super(sb);
 	sync_blockdev(bdev);
 	close_bdev_exclusive(bdev, mode);

[PATCH 11/16] fs/super.c: fix sparse warning: Using plain integer as NULL pointer

[Hannes Eder]

Fix this sparse warning:
  fs/super.c:849:26: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 fs/super.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/fs/super.c b/fs/super.c
index c38eef4..c6ad02d 100644
--- a/fs/super.c
+++ b/fs/super.c
@@ -846,7 +846,7 @@ void kill_block_super(struct super_block *sb)
 	struct block_device *bdev = sb->s_bdev;
 	fmode_t mode = sb->s_mode;
 
-	bdev->bd_super = 0;
+	bdev->bd_super = NULL;
 	generic_shutdown_super(sb);
 	sync_blockdev(bdev);
 	close_bdev_exclusive(bdev, mode);

Re: [PATCH 11/16] fs/super.c: fix sparse warning: Using plain integer

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:05 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warning:
>  fs/super.c:849:26: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  fs/super.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/fs/super.c b/fs/super.c
> index c38eef4..c6ad02d 100644
> --- a/fs/super.c
> +++ b/fs/super.c
> @@ -846,7 +846,7 @@ void kill_block_super(struct super_block *sb)
>        struct block_device *bdev = sb->s_bdev;
>        fmode_t mode = sb->s_mode;
>
> -       bdev->bd_super = 0;
> +       bdev->bd_super = NULL;
>        generic_shutdown_super(sb);
>        sync_blockdev(bdev);
>        close_bdev_exclusive(bdev, mode);
>
>
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 11/16] fs/super.c: fix sparse warning: Using plain integer as NULL pointer

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:05 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warning:
>  fs/super.c:849:26: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  fs/super.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/fs/super.c b/fs/super.c
> index c38eef4..c6ad02d 100644
> --- a/fs/super.c
> +++ b/fs/super.c
> @@ -846,7 +846,7 @@ void kill_block_super(struct super_block *sb)
>        struct block_device *bdev = sb->s_bdev;
>        fmode_t mode = sb->s_mode;
>
> -       bdev->bd_super = 0;
> +       bdev->bd_super = NULL;
>        generic_shutdown_super(sb);
>        sync_blockdev(bdev);
>        close_bdev_exclusive(bdev, mode);
>
>

Re: [PATCH 11/16] fs/super.c: fix sparse warning: Using plain

[Al Viro]

On Fri, Feb 27, 2009 at 02:10:14AM +0100, Hannes Eder wrote:
> [Added To]
> 
> On Sun, Feb 22, 2009 at 1:05 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> > Fix this sparse warning:
> > ?fs/super.c:849:26: warning: Using plain integer as NULL pointer

May I suggest a better subject for those?  Namely,

Subject: NULL noise: <filename or glob>

Nobody _cares_ which tool generates that and boilerplates in subjects should
be as short as possible.

Re: [PATCH 11/16] fs/super.c: fix sparse warning: Using plain integer as NULL pointer

[Al Viro]

On Fri, Feb 27, 2009 at 02:10:14AM +0100, Hannes Eder wrote:
> [Added To]
> 
> On Sun, Feb 22, 2009 at 1:05 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> > Fix this sparse warning:
> > ?fs/super.c:849:26: warning: Using plain integer as NULL pointer

May I suggest a better subject for those?  Namely,

Subject: NULL noise: <filename or glob>

Nobody _cares_ which tool generates that and boilerplates in subjects should
be as short as possible.

[PATCH 11/16 v2] NULL noise: drivers/ide/ide-*.c

[Hannes Eder]

Fix this sparse warnings:
  drivers/ide/ide-disk_proc.c:130:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-floppy_proc.c:32:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-proc.c:234:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-tape.c:2141:11: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
v2: other subject, as suggested by Al Viro

 drivers/ide/ide-disk_proc.c   |    2 +-
 drivers/ide/ide-floppy_proc.c |    2 +-
 drivers/ide/ide-proc.c        |    2 +-
 drivers/ide/ide-tape.c        |    2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/ide/ide-disk_proc.c b/drivers/ide/ide-disk_proc.c
index 3f2a078..eaea3be 100644
--- a/drivers/ide/ide-disk_proc.c
+++ b/drivers/ide/ide-disk_proc.c
@@ -127,5 +127,5 @@ const struct ide_proc_devset ide_disk_settings[] = {
 	IDE_PROC_DEVSET(multcount,	0,    16),
 	IDE_PROC_DEVSET(nowerr,		0,     1),
 	IDE_PROC_DEVSET(wcache,		0,     1),
-	{ 0 },
+	{ NULL },
 };
diff --git a/drivers/ide/ide-floppy_proc.c b/drivers/ide/ide-floppy_proc.c
index 3ec762c..fcd4d81 100644
--- a/drivers/ide/ide-floppy_proc.c
+++ b/drivers/ide/ide-floppy_proc.c
@@ -29,5 +29,5 @@ const struct ide_proc_devset ide_floppy_settings[] = {
 	IDE_PROC_DEVSET(bios_head, 0,  255),
 	IDE_PROC_DEVSET(bios_sect, 0,   63),
 	IDE_PROC_DEVSET(ticks,	   0,  255),
-	{ 0 },
+	{ NULL },
 };
diff --git a/drivers/ide/ide-proc.c b/drivers/ide/ide-proc.c
index 0ee8887..10a88bf 100644
--- a/drivers/ide/ide-proc.c
+++ b/drivers/ide/ide-proc.c
@@ -231,7 +231,7 @@ static const struct ide_proc_devset ide_generic_settings[] = {
 	IDE_PROC_DEVSET(pio_mode, 0, 255),
 	IDE_PROC_DEVSET(unmaskirq, 0, 1),
 	IDE_PROC_DEVSET(using_dma, 0, 1),
-	{ 0 },
+	{ NULL },
 };
 
 static void proc_ide_settings_warn(void)
diff --git a/drivers/ide/ide-tape.c b/drivers/ide/ide-tape.c
index 733cb51..cafc67d 100644
--- a/drivers/ide/ide-tape.c
+++ b/drivers/ide/ide-tape.c
@@ -2142,7 +2142,7 @@ static const struct ide_proc_devset idetape_settings[] = {
 	__IDE_PROC_DEVSET(speed,	0, 0xffff, NULL, NULL),
 	__IDE_PROC_DEVSET(tdsc,		IDETAPE_DSC_RW_MIN, IDETAPE_DSC_RW_MAX,
 					mulf_tdsc, divf_tdsc),
-	{ 0 },
+	{ NULL },
 };
 #endif
 

[PATCH 11/16 v2] NULL noise: drivers/ide/ide-*.c

[Hannes Eder]

Fix this sparse warnings:
  drivers/ide/ide-disk_proc.c:130:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-floppy_proc.c:32:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-proc.c:234:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-tape.c:2141:11: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
v2: other subject, as suggested by Al Viro

 drivers/ide/ide-disk_proc.c   |    2 +-
 drivers/ide/ide-floppy_proc.c |    2 +-
 drivers/ide/ide-proc.c        |    2 +-
 drivers/ide/ide-tape.c        |    2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/ide/ide-disk_proc.c b/drivers/ide/ide-disk_proc.c
index 3f2a078..eaea3be 100644
--- a/drivers/ide/ide-disk_proc.c
+++ b/drivers/ide/ide-disk_proc.c
@@ -127,5 +127,5 @@ const struct ide_proc_devset ide_disk_settings[] = {
 	IDE_PROC_DEVSET(multcount,	0,    16),
 	IDE_PROC_DEVSET(nowerr,		0,     1),
 	IDE_PROC_DEVSET(wcache,		0,     1),
-	{ 0 },
+	{ NULL },
 };
diff --git a/drivers/ide/ide-floppy_proc.c b/drivers/ide/ide-floppy_proc.c
index 3ec762c..fcd4d81 100644
--- a/drivers/ide/ide-floppy_proc.c
+++ b/drivers/ide/ide-floppy_proc.c
@@ -29,5 +29,5 @@ const struct ide_proc_devset ide_floppy_settings[] = {
 	IDE_PROC_DEVSET(bios_head, 0,  255),
 	IDE_PROC_DEVSET(bios_sect, 0,   63),
 	IDE_PROC_DEVSET(ticks,	   0,  255),
-	{ 0 },
+	{ NULL },
 };
diff --git a/drivers/ide/ide-proc.c b/drivers/ide/ide-proc.c
index 0ee8887..10a88bf 100644
--- a/drivers/ide/ide-proc.c
+++ b/drivers/ide/ide-proc.c
@@ -231,7 +231,7 @@ static const struct ide_proc_devset ide_generic_settings[] = {
 	IDE_PROC_DEVSET(pio_mode, 0, 255),
 	IDE_PROC_DEVSET(unmaskirq, 0, 1),
 	IDE_PROC_DEVSET(using_dma, 0, 1),
-	{ 0 },
+	{ NULL },
 };
 
 static void proc_ide_settings_warn(void)
diff --git a/drivers/ide/ide-tape.c b/drivers/ide/ide-tape.c
index 733cb51..cafc67d 100644
--- a/drivers/ide/ide-tape.c
+++ b/drivers/ide/ide-tape.c
@@ -2142,7 +2142,7 @@ static const struct ide_proc_devset idetape_settings[] = {
 	__IDE_PROC_DEVSET(speed,	0, 0xffff, NULL, NULL),
 	__IDE_PROC_DEVSET(tdsc,		IDETAPE_DSC_RW_MIN, IDETAPE_DSC_RW_MAX,
 					mulf_tdsc, divf_tdsc),
-	{ 0 },
+	{ NULL },
 };
 #endif
 

[PATCH 04/16 v3] NULL noise: drivers/ide/ide-*.c

[Hannes Eder]

Fix this sparse warnings:
  drivers/ide/ide-disk_proc.c:130:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-floppy_proc.c:32:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-proc.c:234:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-tape.c:2141:11: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
v2: other subject, as suggested by Al Viro
v3: ups. ... send it to the right recipient, fix patch number
---
 drivers/ide/ide-disk_proc.c   |    2 +-
 drivers/ide/ide-floppy_proc.c |    2 +-
 drivers/ide/ide-proc.c        |    2 +-
 drivers/ide/ide-tape.c        |    2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/ide/ide-disk_proc.c b/drivers/ide/ide-disk_proc.c
index 3f2a078..eaea3be 100644
--- a/drivers/ide/ide-disk_proc.c
+++ b/drivers/ide/ide-disk_proc.c
@@ -127,5 +127,5 @@ const struct ide_proc_devset ide_disk_settings[] = {
 	IDE_PROC_DEVSET(multcount,	0,    16),
 	IDE_PROC_DEVSET(nowerr,		0,     1),
 	IDE_PROC_DEVSET(wcache,		0,     1),
-	{ 0 },
+	{ NULL },
 };
diff --git a/drivers/ide/ide-floppy_proc.c b/drivers/ide/ide-floppy_proc.c
index 3ec762c..fcd4d81 100644
--- a/drivers/ide/ide-floppy_proc.c
+++ b/drivers/ide/ide-floppy_proc.c
@@ -29,5 +29,5 @@ const struct ide_proc_devset ide_floppy_settings[] = {
 	IDE_PROC_DEVSET(bios_head, 0,  255),
 	IDE_PROC_DEVSET(bios_sect, 0,   63),
 	IDE_PROC_DEVSET(ticks,	   0,  255),
-	{ 0 },
+	{ NULL },
 };
diff --git a/drivers/ide/ide-proc.c b/drivers/ide/ide-proc.c
index 0ee8887..10a88bf 100644
--- a/drivers/ide/ide-proc.c
+++ b/drivers/ide/ide-proc.c
@@ -231,7 +231,7 @@ static const struct ide_proc_devset ide_generic_settings[] = {
 	IDE_PROC_DEVSET(pio_mode, 0, 255),
 	IDE_PROC_DEVSET(unmaskirq, 0, 1),
 	IDE_PROC_DEVSET(using_dma, 0, 1),
-	{ 0 },
+	{ NULL },
 };
 
 static void proc_ide_settings_warn(void)
diff --git a/drivers/ide/ide-tape.c b/drivers/ide/ide-tape.c
index 733cb51..cafc67d 100644
--- a/drivers/ide/ide-tape.c
+++ b/drivers/ide/ide-tape.c
@@ -2142,7 +2142,7 @@ static const struct ide_proc_devset idetape_settings[] = {
 	__IDE_PROC_DEVSET(speed,	0, 0xffff, NULL, NULL),
 	__IDE_PROC_DEVSET(tdsc,		IDETAPE_DSC_RW_MIN, IDETAPE_DSC_RW_MAX,
 					mulf_tdsc, divf_tdsc),
-	{ 0 },
+	{ NULL },
 };
 #endif
 

[PATCH 04/16 v3] NULL noise: drivers/ide/ide-*.c

[Hannes Eder]

Fix this sparse warnings:
  drivers/ide/ide-disk_proc.c:130:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-floppy_proc.c:32:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-proc.c:234:11: warning: Using plain integer as NULL pointer
  drivers/ide/ide-tape.c:2141:11: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
v2: other subject, as suggested by Al Viro
v3: ups. ... send it to the right recipient, fix patch number
---
 drivers/ide/ide-disk_proc.c   |    2 +-
 drivers/ide/ide-floppy_proc.c |    2 +-
 drivers/ide/ide-proc.c        |    2 +-
 drivers/ide/ide-tape.c        |    2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/ide/ide-disk_proc.c b/drivers/ide/ide-disk_proc.c
index 3f2a078..eaea3be 100644
--- a/drivers/ide/ide-disk_proc.c
+++ b/drivers/ide/ide-disk_proc.c
@@ -127,5 +127,5 @@ const struct ide_proc_devset ide_disk_settings[] = {
 	IDE_PROC_DEVSET(multcount,	0,    16),
 	IDE_PROC_DEVSET(nowerr,		0,     1),
 	IDE_PROC_DEVSET(wcache,		0,     1),
-	{ 0 },
+	{ NULL },
 };
diff --git a/drivers/ide/ide-floppy_proc.c b/drivers/ide/ide-floppy_proc.c
index 3ec762c..fcd4d81 100644
--- a/drivers/ide/ide-floppy_proc.c
+++ b/drivers/ide/ide-floppy_proc.c
@@ -29,5 +29,5 @@ const struct ide_proc_devset ide_floppy_settings[] = {
 	IDE_PROC_DEVSET(bios_head, 0,  255),
 	IDE_PROC_DEVSET(bios_sect, 0,   63),
 	IDE_PROC_DEVSET(ticks,	   0,  255),
-	{ 0 },
+	{ NULL },
 };
diff --git a/drivers/ide/ide-proc.c b/drivers/ide/ide-proc.c
index 0ee8887..10a88bf 100644
--- a/drivers/ide/ide-proc.c
+++ b/drivers/ide/ide-proc.c
@@ -231,7 +231,7 @@ static const struct ide_proc_devset ide_generic_settings[] = {
 	IDE_PROC_DEVSET(pio_mode, 0, 255),
 	IDE_PROC_DEVSET(unmaskirq, 0, 1),
 	IDE_PROC_DEVSET(using_dma, 0, 1),
-	{ 0 },
+	{ NULL },
 };
 
 static void proc_ide_settings_warn(void)
diff --git a/drivers/ide/ide-tape.c b/drivers/ide/ide-tape.c
index 733cb51..cafc67d 100644
--- a/drivers/ide/ide-tape.c
+++ b/drivers/ide/ide-tape.c
@@ -2142,7 +2142,7 @@ static const struct ide_proc_devset idetape_settings[] = {
 	__IDE_PROC_DEVSET(speed,	0, 0xffff, NULL, NULL),
 	__IDE_PROC_DEVSET(tdsc,		IDETAPE_DSC_RW_MIN, IDETAPE_DSC_RW_MAX,
 					mulf_tdsc, divf_tdsc),
-	{ 0 },
+	{ NULL },
 };
 #endif
 

Re: [PATCH 04/16 v3] NULL noise: drivers/ide/ide-*.c

[Bartlomiej Zolnierkiewicz]

On Tuesday 03 March 2009, Hannes Eder wrote:
> Fix this sparse warnings:
>   drivers/ide/ide-disk_proc.c:130:11: warning: Using plain integer as NULL pointer
>   drivers/ide/ide-floppy_proc.c:32:11: warning: Using plain integer as NULL pointer
>   drivers/ide/ide-proc.c:234:11: warning: Using plain integer as NULL pointer
>   drivers/ide/ide-tape.c:2141:11: warning: Using plain integer as NULL pointer
> 
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>

applied

Re: [PATCH 04/16 v3] NULL noise: drivers/ide/ide-*.c

[Bartlomiej Zolnierkiewicz]

On Tuesday 03 March 2009, Hannes Eder wrote:
> Fix this sparse warnings:
>   drivers/ide/ide-disk_proc.c:130:11: warning: Using plain integer as NULL pointer
>   drivers/ide/ide-floppy_proc.c:32:11: warning: Using plain integer as NULL pointer
>   drivers/ide/ide-proc.c:234:11: warning: Using plain integer as NULL pointer
>   drivers/ide/ide-tape.c:2141:11: warning: Using plain integer as NULL pointer
> 
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>

applied

[PATCH 12/16] reiserfs: fix sparse warning: Using plain integer as

[Hannes Eder]

Fix this sparse warning:
  fs/reiserfs/inode.c:2555:19: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 fs/reiserfs/inode.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/fs/reiserfs/inode.c b/fs/reiserfs/inode.c
index 823227a..e69d63a 100644
--- a/fs/reiserfs/inode.c
+++ b/fs/reiserfs/inode.c
@@ -2552,7 +2552,7 @@ static int reiserfs_write_begin(struct file *file,
 	int old_ref = 0;
 
  	inode = mapping->host;
-	*fsdata = 0;
+	*fsdata = NULL;
  	if (flags & AOP_FLAG_CONT_EXPAND &&
  	    (pos & (inode->i_sb->s_blocksize - 1)) = 0) {
  		pos ++;

[PATCH 12/16] reiserfs: fix sparse warning: Using plain integer as NULL pointer

[Hannes Eder]

Fix this sparse warning:
  fs/reiserfs/inode.c:2555:19: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 fs/reiserfs/inode.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/fs/reiserfs/inode.c b/fs/reiserfs/inode.c
index 823227a..e69d63a 100644
--- a/fs/reiserfs/inode.c
+++ b/fs/reiserfs/inode.c
@@ -2552,7 +2552,7 @@ static int reiserfs_write_begin(struct file *file,
 	int old_ref = 0;
 
  	inode = mapping->host;
-	*fsdata = 0;
+	*fsdata = NULL;
  	if (flags & AOP_FLAG_CONT_EXPAND &&
  	    (pos & (inode->i_sb->s_blocksize - 1)) == 0) {
  		pos ++;

Re: [PATCH 12/16] reiserfs: fix sparse warning: Using plain integer

[Hannes Eder]

[Added To, Cc]

On Sun, Feb 22, 2009 at 1:05 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warning:
>  fs/reiserfs/inode.c:2555:19: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  fs/reiserfs/inode.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/fs/reiserfs/inode.c b/fs/reiserfs/inode.c
> index 823227a..e69d63a 100644
> --- a/fs/reiserfs/inode.c
> +++ b/fs/reiserfs/inode.c
> @@ -2552,7 +2552,7 @@ static int reiserfs_write_begin(struct file *file,
>        int old_ref = 0;
>
>        inode = mapping->host;
> -       *fsdata = 0;
> +       *fsdata = NULL;
>        if (flags & AOP_FLAG_CONT_EXPAND &&
>            (pos & (inode->i_sb->s_blocksize - 1)) = 0) {
>                pos ++;
>
>
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 12/16] reiserfs: fix sparse warning: Using plain integer as NULL pointer

[Hannes Eder]

[Added To, Cc]

On Sun, Feb 22, 2009 at 1:05 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warning:
>  fs/reiserfs/inode.c:2555:19: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  fs/reiserfs/inode.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/fs/reiserfs/inode.c b/fs/reiserfs/inode.c
> index 823227a..e69d63a 100644
> --- a/fs/reiserfs/inode.c
> +++ b/fs/reiserfs/inode.c
> @@ -2552,7 +2552,7 @@ static int reiserfs_write_begin(struct file *file,
>        int old_ref = 0;
>
>        inode = mapping->host;
> -       *fsdata = 0;
> +       *fsdata = NULL;
>        if (flags & AOP_FLAG_CONT_EXPAND &&
>            (pos & (inode->i_sb->s_blocksize - 1)) == 0) {
>                pos ++;
>
>

Re: [PATCH 12/16] reiserfs: fix sparse warning: Using plain integer as NULL pointer

[Hannes Eder]

[Added To, Cc]

On Sun, Feb 22, 2009 at 1:05 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warning:
>  fs/reiserfs/inode.c:2555:19: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  fs/reiserfs/inode.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/fs/reiserfs/inode.c b/fs/reiserfs/inode.c
> index 823227a..e69d63a 100644
> --- a/fs/reiserfs/inode.c
> +++ b/fs/reiserfs/inode.c
> @@ -2552,7 +2552,7 @@ static int reiserfs_write_begin(struct file *file,
>        int old_ref = 0;
>
>        inode = mapping->host;
> -       *fsdata = 0;
> +       *fsdata = NULL;
>        if (flags & AOP_FLAG_CONT_EXPAND &&
>            (pos & (inode->i_sb->s_blocksize - 1)) == 0) {
>                pos ++;
>
>
--
To unsubscribe from this list: send the line "unsubscribe reiserfs-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 13/16] kprobes: fix sparse warnings: Using plain integer as

[Hannes Eder]

Fix this sparse warnings:
  kernel/test_kprobes.c:118:19: warning: Using plain integer as NULL pointer
  kernel/test_kprobes.c:213:22: warning: Using plain integer as NULL pointer
  kernel/test_kprobes.c:326:22: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 kernel/test_kprobes.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/kernel/test_kprobes.c b/kernel/test_kprobes.c
index 4f10451..9a43351 100644
--- a/kernel/test_kprobes.c
+++ b/kernel/test_kprobes.c
@@ -115,7 +115,7 @@ static int test_kprobes(void)
 	int ret;
 	struct kprobe *kps[2] = {&kp, &kp2};
 
-	kp.addr = 0; /* addr should be cleard for reusing kprobe. */
+	kp.addr = NULL; /* addr should be cleard for reusing kprobe. */
 	ret = register_kprobes(kps, 2);
 	if (ret < 0) {
 		printk(KERN_ERR "Kprobe smoke test failed: "
@@ -210,7 +210,7 @@ static int test_jprobes(void)
 	int ret;
 	struct jprobe *jps[2] = {&jp, &jp2};
 
-	jp.kp.addr = 0; /* addr should be cleard for reusing kprobe. */
+	jp.kp.addr = NULL; /* addr should be cleard for reusing kprobe. */
 	ret = register_jprobes(jps, 2);
 	if (ret < 0) {
 		printk(KERN_ERR "Kprobe smoke test failed: "
@@ -323,7 +323,7 @@ static int test_kretprobes(void)
 	int ret;
 	struct kretprobe *rps[2] = {&rp, &rp2};
 
-	rp.kp.addr = 0; /* addr should be cleard for reusing kprobe. */
+	rp.kp.addr = NULL; /* addr should be cleard for reusing kprobe. */
 	ret = register_kretprobes(rps, 2);
 	if (ret < 0) {
 		printk(KERN_ERR "Kprobe smoke test failed: "

[PATCH 13/16] kprobes: fix sparse warnings: Using plain integer as NULL pointer

[Hannes Eder]

Fix this sparse warnings:
  kernel/test_kprobes.c:118:19: warning: Using plain integer as NULL pointer
  kernel/test_kprobes.c:213:22: warning: Using plain integer as NULL pointer
  kernel/test_kprobes.c:326:22: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 kernel/test_kprobes.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/kernel/test_kprobes.c b/kernel/test_kprobes.c
index 4f10451..9a43351 100644
--- a/kernel/test_kprobes.c
+++ b/kernel/test_kprobes.c
@@ -115,7 +115,7 @@ static int test_kprobes(void)
 	int ret;
 	struct kprobe *kps[2] = {&kp, &kp2};
 
-	kp.addr = 0; /* addr should be cleard for reusing kprobe. */
+	kp.addr = NULL; /* addr should be cleard for reusing kprobe. */
 	ret = register_kprobes(kps, 2);
 	if (ret < 0) {
 		printk(KERN_ERR "Kprobe smoke test failed: "
@@ -210,7 +210,7 @@ static int test_jprobes(void)
 	int ret;
 	struct jprobe *jps[2] = {&jp, &jp2};
 
-	jp.kp.addr = 0; /* addr should be cleard for reusing kprobe. */
+	jp.kp.addr = NULL; /* addr should be cleard for reusing kprobe. */
 	ret = register_jprobes(jps, 2);
 	if (ret < 0) {
 		printk(KERN_ERR "Kprobe smoke test failed: "
@@ -323,7 +323,7 @@ static int test_kretprobes(void)
 	int ret;
 	struct kretprobe *rps[2] = {&rp, &rp2};
 
-	rp.kp.addr = 0; /* addr should be cleard for reusing kprobe. */
+	rp.kp.addr = NULL; /* addr should be cleard for reusing kprobe. */
 	ret = register_kretprobes(rps, 2);
 	if (ret < 0) {
 		printk(KERN_ERR "Kprobe smoke test failed: "

Re: [PATCH 13/16] kprobes: fix sparse warnings: Using plain integer

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:05 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warnings:
>  kernel/test_kprobes.c:118:19: warning: Using plain integer as NULL pointer
>  kernel/test_kprobes.c:213:22: warning: Using plain integer as NULL pointer
>  kernel/test_kprobes.c:326:22: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  kernel/test_kprobes.c |    6 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/kernel/test_kprobes.c b/kernel/test_kprobes.c
> index 4f10451..9a43351 100644
> --- a/kernel/test_kprobes.c
> +++ b/kernel/test_kprobes.c
> @@ -115,7 +115,7 @@ static int test_kprobes(void)
>        int ret;
>        struct kprobe *kps[2] = {&kp, &kp2};
>
> -       kp.addr = 0; /* addr should be cleard for reusing kprobe. */
> +       kp.addr = NULL; /* addr should be cleard for reusing kprobe. */
>        ret = register_kprobes(kps, 2);
>        if (ret < 0) {
>                printk(KERN_ERR "Kprobe smoke test failed: "
> @@ -210,7 +210,7 @@ static int test_jprobes(void)
>        int ret;
>        struct jprobe *jps[2] = {&jp, &jp2};
>
> -       jp.kp.addr = 0; /* addr should be cleard for reusing kprobe. */
> +       jp.kp.addr = NULL; /* addr should be cleard for reusing kprobe. */
>        ret = register_jprobes(jps, 2);
>        if (ret < 0) {
>                printk(KERN_ERR "Kprobe smoke test failed: "
> @@ -323,7 +323,7 @@ static int test_kretprobes(void)
>        int ret;
>        struct kretprobe *rps[2] = {&rp, &rp2};
>
> -       rp.kp.addr = 0; /* addr should be cleard for reusing kprobe. */
> +       rp.kp.addr = NULL; /* addr should be cleard for reusing kprobe. */
>        ret = register_kretprobes(rps, 2);
>        if (ret < 0) {
>                printk(KERN_ERR "Kprobe smoke test failed: "
>
>
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 13/16] kprobes: fix sparse warnings: Using plain integer as NULL pointer

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:05 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warnings:
>  kernel/test_kprobes.c:118:19: warning: Using plain integer as NULL pointer
>  kernel/test_kprobes.c:213:22: warning: Using plain integer as NULL pointer
>  kernel/test_kprobes.c:326:22: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  kernel/test_kprobes.c |    6 +++---
>  1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/kernel/test_kprobes.c b/kernel/test_kprobes.c
> index 4f10451..9a43351 100644
> --- a/kernel/test_kprobes.c
> +++ b/kernel/test_kprobes.c
> @@ -115,7 +115,7 @@ static int test_kprobes(void)
>        int ret;
>        struct kprobe *kps[2] = {&kp, &kp2};
>
> -       kp.addr = 0; /* addr should be cleard for reusing kprobe. */
> +       kp.addr = NULL; /* addr should be cleard for reusing kprobe. */
>        ret = register_kprobes(kps, 2);
>        if (ret < 0) {
>                printk(KERN_ERR "Kprobe smoke test failed: "
> @@ -210,7 +210,7 @@ static int test_jprobes(void)
>        int ret;
>        struct jprobe *jps[2] = {&jp, &jp2};
>
> -       jp.kp.addr = 0; /* addr should be cleard for reusing kprobe. */
> +       jp.kp.addr = NULL; /* addr should be cleard for reusing kprobe. */
>        ret = register_jprobes(jps, 2);
>        if (ret < 0) {
>                printk(KERN_ERR "Kprobe smoke test failed: "
> @@ -323,7 +323,7 @@ static int test_kretprobes(void)
>        int ret;
>        struct kretprobe *rps[2] = {&rp, &rp2};
>
> -       rp.kp.addr = 0; /* addr should be cleard for reusing kprobe. */
> +       rp.kp.addr = NULL; /* addr should be cleard for reusing kprobe. */
>        ret = register_kretprobes(rps, 2);
>        if (ret < 0) {
>                printk(KERN_ERR "Kprobe smoke test failed: "
>
>

[PATCH 14/16] memcg: fix sparse warning: Using plain integer as NULL

[Hannes Eder]

Fix this sparse warning:
  mm/memcontrol.c:1637:32: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 mm/memcontrol.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/mm/memcontrol.c b/mm/memcontrol.c
index 8e4be9c..09d6650 100644
--- a/mm/memcontrol.c
+++ b/mm/memcontrol.c
@@ -1634,7 +1634,7 @@ static int mem_cgroup_force_empty_list(struct mem_cgroup *mem,
 		pc = list_entry(list->prev, struct page_cgroup, lru);
 		if (busy = pc) {
 			list_move(&pc->lru, list);
-			busy = 0;
+			busy = NULL;
 			spin_unlock_irqrestore(&zone->lru_lock, flags);
 			continue;
 		}

[PATCH 14/16] memcg: fix sparse warning: Using plain integer as NULL pointer

[Hannes Eder]

Fix this sparse warning:
  mm/memcontrol.c:1637:32: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 mm/memcontrol.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/mm/memcontrol.c b/mm/memcontrol.c
index 8e4be9c..09d6650 100644
--- a/mm/memcontrol.c
+++ b/mm/memcontrol.c
@@ -1634,7 +1634,7 @@ static int mem_cgroup_force_empty_list(struct mem_cgroup *mem,
 		pc = list_entry(list->prev, struct page_cgroup, lru);
 		if (busy == pc) {
 			list_move(&pc->lru, list);
-			busy = 0;
+			busy = NULL;
 			spin_unlock_irqrestore(&zone->lru_lock, flags);
 			continue;
 		}

Re: [PATCH 14/16] memcg: fix sparse warning: Using plain integer as

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:05 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warning:
>  mm/memcontrol.c:1637:32: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  mm/memcontrol.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/mm/memcontrol.c b/mm/memcontrol.c
> index 8e4be9c..09d6650 100644
> --- a/mm/memcontrol.c
> +++ b/mm/memcontrol.c
> @@ -1634,7 +1634,7 @@ static int mem_cgroup_force_empty_list(struct mem_cgroup *mem,
>                pc = list_entry(list->prev, struct page_cgroup, lru);
>                if (busy = pc) {
>                        list_move(&pc->lru, list);
> -                       busy = 0;
> +                       busy = NULL;
>                        spin_unlock_irqrestore(&zone->lru_lock, flags);
>                        continue;
>                }
>
>
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 14/16] memcg: fix sparse warning: Using plain integer as NULL pointer

[Hannes Eder]

[Added To]

On Sun, Feb 22, 2009 at 1:05 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> Fix this sparse warning:
>  mm/memcontrol.c:1637:32: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> ---
>  mm/memcontrol.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/mm/memcontrol.c b/mm/memcontrol.c
> index 8e4be9c..09d6650 100644
> --- a/mm/memcontrol.c
> +++ b/mm/memcontrol.c
> @@ -1634,7 +1634,7 @@ static int mem_cgroup_force_empty_list(struct mem_cgroup *mem,
>                pc = list_entry(list->prev, struct page_cgroup, lru);
>                if (busy == pc) {
>                        list_move(&pc->lru, list);
> -                       busy = 0;
> +                       busy = NULL;
>                        spin_unlock_irqrestore(&zone->lru_lock, flags);
>                        continue;
>                }
>
>

Re: [PATCH 14/16] memcg: fix sparse warning: Using plain integer as

[KAMEZAWA Hiroyuki]

On Fri, 27 Feb 2009 02:15:46 +0100
Hannes Eder <hannes@hanneseder.net> wrote:

> [Added To]
> 
Thanks,

Acked-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>

> On Sun, Feb 22, 2009 at 1:05 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> > Fix this sparse warning:
> >  mm/memcontrol.c:1637:32: warning: Using plain integer as NULL pointer
> >
> > Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> > ---
> >  mm/memcontrol.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/mm/memcontrol.c b/mm/memcontrol.c
> > index 8e4be9c..09d6650 100644
> > --- a/mm/memcontrol.c
> > +++ b/mm/memcontrol.c
> > @@ -1634,7 +1634,7 @@ static int mem_cgroup_force_empty_list(struct mem_cgroup *mem,
> >                pc = list_entry(list->prev, struct page_cgroup, lru);
> >                if (busy = pc) {
> >                        list_move(&pc->lru, list);
> > -                       busy = 0;
> > +                       busy = NULL;
> >                        spin_unlock_irqrestore(&zone->lru_lock, flags);
> >                        continue;
> >                }
> >
> >
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
> 

Re: [PATCH 14/16] memcg: fix sparse warning: Using plain integer as NULL pointer

[KAMEZAWA Hiroyuki]

On Fri, 27 Feb 2009 02:15:46 +0100
Hannes Eder <hannes@hanneseder.net> wrote:

> [Added To]
> 
Thanks,

Acked-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>

> On Sun, Feb 22, 2009 at 1:05 AM, Hannes Eder <hannes@hanneseder.net> wrote:
> > Fix this sparse warning:
> >  mm/memcontrol.c:1637:32: warning: Using plain integer as NULL pointer
> >
> > Signed-off-by: Hannes Eder <hannes@hanneseder.net>
> > ---
> >  mm/memcontrol.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/mm/memcontrol.c b/mm/memcontrol.c
> > index 8e4be9c..09d6650 100644
> > --- a/mm/memcontrol.c
> > +++ b/mm/memcontrol.c
> > @@ -1634,7 +1634,7 @@ static int mem_cgroup_force_empty_list(struct mem_cgroup *mem,
> >                pc = list_entry(list->prev, struct page_cgroup, lru);
> >                if (busy == pc) {
> >                        list_move(&pc->lru, list);
> > -                       busy = 0;
> > +                       busy = NULL;
> >                        spin_unlock_irqrestore(&zone->lru_lock, flags);
> >                        continue;
> >                }
> >
> >
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
> 

[PATCH 15/16] ipv6: fix sparse warning: Using plain integer as NULL

[Hannes Eder]

Fix this sparse warning:
  net/ipv6/xfrm6_state.c:72:26: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 net/ipv6/xfrm6_state.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/net/ipv6/xfrm6_state.c b/net/ipv6/xfrm6_state.c
index 0e685b0..f417b77 100644
--- a/net/ipv6/xfrm6_state.c
+++ b/net/ipv6/xfrm6_state.c
@@ -69,7 +69,7 @@ __xfrm6_sort(void **dst, void **src, int n, int (*cmp)(void *p), int maxclass)
 
 	for (i = 0; i < n; i++) {
 		dst[count[class[i] - 1]++] = src[i];
-		src[i] = 0;
+		src[i] = NULL;
 	}
 
 	return 0;

[PATCH 15/16] ipv6: fix sparse warning: Using plain integer as NULL pointer

[Hannes Eder]

Fix this sparse warning:
  net/ipv6/xfrm6_state.c:72:26: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 net/ipv6/xfrm6_state.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/net/ipv6/xfrm6_state.c b/net/ipv6/xfrm6_state.c
index 0e685b0..f417b77 100644
--- a/net/ipv6/xfrm6_state.c
+++ b/net/ipv6/xfrm6_state.c
@@ -69,7 +69,7 @@ __xfrm6_sort(void **dst, void **src, int n, int (*cmp)(void *p), int maxclass)
 
 	for (i = 0; i < n; i++) {
 		dst[count[class[i] - 1]++] = src[i];
-		src[i] = 0;
+		src[i] = NULL;
 	}
 
 	return 0;

Re: [PATCH 15/16] ipv6: fix sparse warning: Using plain integer as

[David Miller]

From: Hannes Eder <hannes@hanneseder.net>
Date: Sun, 22 Feb 2009 01:05:44 +0100

> Fix this sparse warning:
>   net/ipv6/xfrm6_state.c:72:26: warning: Using plain integer as NULL pointer
> 
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>

Applied, thanks.

Re: [PATCH 15/16] ipv6: fix sparse warning: Using plain integer as NULL pointer

[David Miller]

From: Hannes Eder <hannes@hanneseder.net>
Date: Sun, 22 Feb 2009 01:05:44 +0100

> Fix this sparse warning:
>   net/ipv6/xfrm6_state.c:72:26: warning: Using plain integer as NULL pointer
> 
> Signed-off-by: Hannes Eder <hannes@hanneseder.net>

Applied, thanks.

[PATCH 16/16] ALSA: msnd: fix sparse warning: Using plain integer as

[Hannes Eder]

Fix this sparse warning:
  sound/isa/msnd/msnd_pinnacle.c:808:1: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 sound/isa/msnd/msnd_pinnacle.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/sound/isa/msnd/msnd_pinnacle.c b/sound/isa/msnd/msnd_pinnacle.c
index 60b6abd..add6c0a 100644
--- a/sound/isa/msnd/msnd_pinnacle.c
+++ b/sound/isa/msnd/msnd_pinnacle.c
@@ -805,7 +805,7 @@ module_param(calibrate_signal, int, S_IRUGO);
 #ifndef MSND_CLASSIC
 module_param_array(digital, int, NULL, S_IRUGO);
 module_param_array(cfg, long, NULL, S_IRUGO);
-module_param_array(reset, int, 0, S_IRUGO);
+module_param_array(reset, int, NULL, S_IRUGO);
 module_param_array(mpu_io, long, NULL, S_IRUGO);
 module_param_array(mpu_irq, int, NULL, S_IRUGO);
 module_param_array(ide_io0, long, NULL, S_IRUGO);

[PATCH 16/16] ALSA: msnd: fix sparse warning: Using plain integer as NULL pointer

[Hannes Eder]

Fix this sparse warning:
  sound/isa/msnd/msnd_pinnacle.c:808:1: warning: Using plain integer as NULL pointer

Signed-off-by: Hannes Eder <hannes@hanneseder.net>
---
 sound/isa/msnd/msnd_pinnacle.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/sound/isa/msnd/msnd_pinnacle.c b/sound/isa/msnd/msnd_pinnacle.c
index 60b6abd..add6c0a 100644
--- a/sound/isa/msnd/msnd_pinnacle.c
+++ b/sound/isa/msnd/msnd_pinnacle.c
@@ -805,7 +805,7 @@ module_param(calibrate_signal, int, S_IRUGO);
 #ifndef MSND_CLASSIC
 module_param_array(digital, int, NULL, S_IRUGO);
 module_param_array(cfg, long, NULL, S_IRUGO);
-module_param_array(reset, int, 0, S_IRUGO);
+module_param_array(reset, int, NULL, S_IRUGO);
 module_param_array(mpu_io, long, NULL, S_IRUGO);
 module_param_array(mpu_irq, int, NULL, S_IRUGO);
 module_param_array(ide_io0, long, NULL, S_IRUGO);