[PATCH] drivers: hv: Mark the function hv_synic_free_cpu() as static in hv.c

2013-12-14 Thread Rashika Kheria 
This patch marks the function hv_synic_free_cpu() as static in hv.c
because it is not used outside this file.

Thus, it also eliminates the following warning in hv.c:
drivers/hv/hv.c:304:6: warning: no previous prototype for ‘hv_synic_free_cpu’ 
[-Wmissing-prototypes]

Signed-off-by: Rashika Kheria rashika.khe...@gmail.com
---
 drivers/hv/hv.c |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/hv/hv.c b/drivers/hv/hv.c
index f0c5e07..bcb4950 100644
--- a/drivers/hv/hv.c
+++ b/drivers/hv/hv.c
@@ -301,7 +301,7 @@ err:
return -ENOMEM;
 }
 
-void hv_synic_free_cpu(int cpu)
+static void hv_synic_free_cpu(int cpu)
 {
kfree(hv_context.event_dpc[cpu]);
if (hv_context.synic_event_page[cpu])
-- 
1.7.9.5

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH] drivers: hv: Mark the function hv_synic_free_cpu() as static in hv.c

2013-12-14 Thread Josh Triplett 
On Sat, Dec 14, 2013 at 07:00:06PM +0530, Rashika Kheria wrote:
 This patch marks the function hv_synic_free_cpu() as static in hv.c
 because it is not used outside this file.
 
 Thus, it also eliminates the following warning in hv.c:
 drivers/hv/hv.c:304:6: warning: no previous prototype for ‘hv_synic_free_cpu’ 
 [-Wmissing-prototypes]
 
 Signed-off-by: Rashika Kheria rashika.khe...@gmail.com

Reviewed-by: Josh Triplett j...@joshtriplett.org

  drivers/hv/hv.c |2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/drivers/hv/hv.c b/drivers/hv/hv.c
 index f0c5e07..bcb4950 100644
 --- a/drivers/hv/hv.c
 +++ b/drivers/hv/hv.c
 @@ -301,7 +301,7 @@ err:
   return -ENOMEM;
  }
  
 -void hv_synic_free_cpu(int cpu)
 +static void hv_synic_free_cpu(int cpu)
  {
   kfree(hv_context.event_dpc[cpu]);
   if (hv_context.synic_event_page[cpu])
 -- 
 1.7.9.5
 
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

RE: [PATCH] drivers: hv: Mark the function hv_synic_free_cpu() as static in hv.c

2013-12-14 Thread KY Srinivasan 


 -Original Message-
 From: Josh Triplett [mailto:j...@joshtriplett.org]
 Sent: Saturday, December 14, 2013 12:57 PM
 To: Rashika Kheria
 Cc: linux-ker...@vger.kernel.org; KY Srinivasan; Haiyang Zhang;
 de...@linuxdriverproject.org
 Subject: Re: [PATCH] drivers: hv: Mark the function hv_synic_free_cpu() as 
 static
 in hv.c
 
 On Sat, Dec 14, 2013 at 07:00:06PM +0530, Rashika Kheria wrote:
  This patch marks the function hv_synic_free_cpu() as static in hv.c
  because it is not used outside this file.
 
  Thus, it also eliminates the following warning in hv.c:
  drivers/hv/hv.c:304:6: warning: no previous prototype for 
  ‘hv_synic_free_cpu’
 [-Wmissing-prototypes]
 
  Signed-off-by: Rashika Kheria rashika.khe...@gmail.com
 
 Reviewed-by: Josh Triplett j...@joshtriplett.org
Thank you.

Signed-off-by: K. Y. Srinivasan k...@microsoft.com
 
   drivers/hv/hv.c |2 +-
   1 file changed, 1 insertion(+), 1 deletion(-)
 
  diff --git a/drivers/hv/hv.c b/drivers/hv/hv.c
  index f0c5e07..bcb4950 100644
  --- a/drivers/hv/hv.c
  +++ b/drivers/hv/hv.c
  @@ -301,7 +301,7 @@ err:
  return -ENOMEM;
   }
 
  -void hv_synic_free_cpu(int cpu)
  +static void hv_synic_free_cpu(int cpu)
   {
  kfree(hv_context.event_dpc[cpu]);
  if (hv_context.synic_event_page[cpu])
  --
  1.7.9.5
 
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel