linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] x86: mrst: quiet sparse noise about plain integer as NULL pointer
@ 2012-04-24 22:00 H Hartley Sweeten
  2012-04-25 10:28 ` Alan Cox
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: H Hartley Sweeten @ 2012-04-24 22:00 UTC (permalink / raw)
  To: Linux Kernel; +Cc: tglx, mingo, hpa, x86

The second parameter to intel_scu_notifier_post is a void * not an integer.

This quiets the sparse noise:

arch/x86/platform/mrst/mrst.c:808:48: warning: Using plain integer as NULL pointer
arch/x86/platform/mrst/mrst.c:817:43: warning: Using plain integer as NULL pointer

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: "H. Peter Anvin" <hpa@zytor.com>

---

diff --git a/arch/x86/platform/mrst/mrst.c b/arch/x86/platform/mrst/mrst.c
index e0a3723..e31bcd8 100644
--- a/arch/x86/platform/mrst/mrst.c
+++ b/arch/x86/platform/mrst/mrst.c
@@ -805,7 +805,7 @@ void intel_scu_devices_create(void)
 		} else
 			i2c_register_board_info(i2c_bus[i], i2c_devs[i], 1);
 	}
-	intel_scu_notifier_post(SCU_AVAILABLE, 0L);
+	intel_scu_notifier_post(SCU_AVAILABLE, NULL);
 }
 EXPORT_SYMBOL_GPL(intel_scu_devices_create);
 
@@ -814,7 +814,7 @@ void intel_scu_devices_destroy(void)
 {
 	int i;
 
-	intel_scu_notifier_post(SCU_DOWN, 0L);
+	intel_scu_notifier_post(SCU_DOWN, NULL);
 
 	for (i = 0; i < ipc_next_dev; i++)
 		platform_device_del(ipc_devs[i]);

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] x86: mrst: quiet sparse noise about plain integer as NULL pointer
  2012-04-24 22:00 [PATCH] x86: mrst: quiet sparse noise about plain integer as NULL pointer H Hartley Sweeten
@ 2012-04-25 10:28 ` Alan Cox
  2012-04-25 10:57 ` Kirill A. Shutemov
  2012-04-25 13:56 ` [tip:x86/urgent] x86/mrst: Quiet " tip-bot for H Hartley Sweeten
  2 siblings, 0 replies; 4+ messages in thread
From: Alan Cox @ 2012-04-25 10:28 UTC (permalink / raw)
  To: H Hartley Sweeten; +Cc: Linux Kernel, tglx, mingo, hpa, x86

> This quiets the sparse noise:
> 
> arch/x86/platform/mrst/mrst.c:808:48: warning: Using plain integer as NULL pointer
> arch/x86/platform/mrst/mrst.c:817:43: warning: Using plain integer as NULL pointer
> 
> Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>

Acked-by: Alan Cox <alan@linux.intel.com>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] x86: mrst: quiet sparse noise about plain integer as NULL pointer
  2012-04-24 22:00 [PATCH] x86: mrst: quiet sparse noise about plain integer as NULL pointer H Hartley Sweeten
  2012-04-25 10:28 ` Alan Cox
@ 2012-04-25 10:57 ` Kirill A. Shutemov
  2012-04-25 13:56 ` [tip:x86/urgent] x86/mrst: Quiet " tip-bot for H Hartley Sweeten
  2 siblings, 0 replies; 4+ messages in thread
From: Kirill A. Shutemov @ 2012-04-25 10:57 UTC (permalink / raw)
  To: H Hartley Sweeten; +Cc: Linux Kernel, tglx, mingo, hpa, x86

On Tue, Apr 24, 2012 at 03:00:53PM -0700, H Hartley Sweeten wrote:
> The second parameter to intel_scu_notifier_post is a void * not an integer.
> 
> This quiets the sparse noise:
> 
> arch/x86/platform/mrst/mrst.c:808:48: warning: Using plain integer as NULL pointer
> arch/x86/platform/mrst/mrst.c:817:43: warning: Using plain integer as NULL pointer
> 
> Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Cc: Ingo Molnar <mingo@redhat.com>
> Cc: "H. Peter Anvin" <hpa@zytor.com>

Acked-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>

-- 
 Kirill A. Shutemov

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [tip:x86/urgent] x86/mrst: Quiet sparse noise about plain integer as NULL pointer
  2012-04-24 22:00 [PATCH] x86: mrst: quiet sparse noise about plain integer as NULL pointer H Hartley Sweeten
  2012-04-25 10:28 ` Alan Cox
  2012-04-25 10:57 ` Kirill A. Shutemov
@ 2012-04-25 13:56 ` tip-bot for H Hartley Sweeten
  2 siblings, 0 replies; 4+ messages in thread
From: tip-bot for H Hartley Sweeten @ 2012-04-25 13:56 UTC (permalink / raw)
  To: linux-tip-commits
  Cc: linux-kernel, alan, hpa, mingo, hartleys, tglx, hsweeten

Commit-ID:  d0d3bc65afcdd69bdd3b5bebdf8b3ee3680efa0e
Gitweb:     http://git.kernel.org/tip/d0d3bc65afcdd69bdd3b5bebdf8b3ee3680efa0e
Author:     H Hartley Sweeten <hartleys@visionengravers.com>
AuthorDate: Tue, 24 Apr 2012 15:00:53 -0700
Committer:  Ingo Molnar <mingo@kernel.org>
CommitDate: Wed, 25 Apr 2012 12:47:08 +0200

x86/mrst: Quiet sparse noise about plain integer as NULL pointer

The second parameter to intel_scu_notifier_post is a void *, not
an integer.

This quiets the sparse noise:

 arch/x86/platform/mrst/mrst.c:808:48: warning: Using plain integer as NULL pointer
 arch/x86/platform/mrst/mrst.c:817:43: warning: Using plain integer as NULL pointer

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Acked-by: Alan Cox <alan@linux.intel.com>
Link: http://lkml.kernel.org/r/201204241500.53685.hartleys@visionengravers.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
---
 arch/x86/platform/mrst/mrst.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/x86/platform/mrst/mrst.c b/arch/x86/platform/mrst/mrst.c
index e0a3723..e31bcd8 100644
--- a/arch/x86/platform/mrst/mrst.c
+++ b/arch/x86/platform/mrst/mrst.c
@@ -805,7 +805,7 @@ void intel_scu_devices_create(void)
 		} else
 			i2c_register_board_info(i2c_bus[i], i2c_devs[i], 1);
 	}
-	intel_scu_notifier_post(SCU_AVAILABLE, 0L);
+	intel_scu_notifier_post(SCU_AVAILABLE, NULL);
 }
 EXPORT_SYMBOL_GPL(intel_scu_devices_create);
 
@@ -814,7 +814,7 @@ void intel_scu_devices_destroy(void)
 {
 	int i;
 
-	intel_scu_notifier_post(SCU_DOWN, 0L);
+	intel_scu_notifier_post(SCU_DOWN, NULL);
 
 	for (i = 0; i < ipc_next_dev; i++)
 		platform_device_del(ipc_devs[i]);

^ permalink raw reply related	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2012-04-25 13:56 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-04-24 22:00 [PATCH] x86: mrst: quiet sparse noise about plain integer as NULL pointer H Hartley Sweeten
2012-04-25 10:28 ` Alan Cox
2012-04-25 10:57 ` Kirill A. Shutemov
2012-04-25 13:56 ` [tip:x86/urgent] x86/mrst: Quiet " tip-bot for H Hartley Sweeten

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).