Messages in this thread Patch in this message | | | From | Nick Meier <> | Subject | RE: [PATCH char-misc] mei: bus: () can be static | Date | Mon, 2 Mar 2015 21:57:46 +0000 |
| |
Yes - the function should have been static. This change correctly corrects the oversight.
thanks,
-Nick
-----Original Message----- From: kbuild test robot [mailto:fengguang.wu@intel.com] Sent: Sunday, March 1, 2015 8:49 PM To: Tomas Winkler Cc: kbuild-all@01.org; Greg Kroah-Hartman; Nick Meier; KY Srinivasan; Haiyang Zhang; devel@linuxdriverproject.org; linux-kernel@vger.kernel.org Subject: [PATCH char-misc] mei: bus: () can be static
drivers/hv/vmbus_drv.c:51:5: sparse: symbol 'hyperv_panic_event' was not declared. Should it be static? drivers/hv/vmbus_drv.c:51:5: sparse: symbol 'hyperv_panic_event' was not declared. Should it be static? drivers/hv/vmbus_drv.c:51:5: sparse: symbol 'hyperv_panic_event' was not declared. Should it be static? drivers/hv/vmbus_drv.c:51:5: sparse: symbol 'hyperv_panic_event' was not declared. Should it be static?
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com> --- vmbus_drv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/hv/vmbus_drv.c b/drivers/hv/vmbus_drv.c index 526fa8b..8313e25 100644 --- a/drivers/hv/vmbus_drv.c +++ b/drivers/hv/vmbus_drv.c @@ -48,7 +48,7 @@ static struct completion probe_event; static int irq; -int hyperv_panic_event(struct notifier_block *nb, +static int hyperv_panic_event(struct notifier_block *nb, unsigned long event, void *ptr) { struct pt_regs *regs;
| |