Re: [PATCH] drivers: staging: Mark functions as static and remove unused function in bpctl_mod.c

2014-01-25 Thread Josh Triplett
On Fri, Jan 24, 2014 at 04:14:00AM +0530, Rashika Kheria wrote:
 Mark functions as static in bpctl_mod.c because they are not used
 outside this file. Remove unused function from bpctl_mod.c.
 
[...]
 
 Signed-off-by: Rashika Kheria rashika.khe...@gmail.com

Reviewed-by: Josh Triplett j...@joshtriplett.org
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH] drivers: staging: Mark functions as static and remove unused function in bpctl_mod.c

2014-01-24 Thread Dan Carpenter
On Fri, Jan 24, 2014 at 04:14:00AM +0530, Rashika Kheria wrote:
 Mark functions as static in bpctl_mod.c because they are not used
 outside this file. Remove unused function from bpctl_mod.c.
 
 This also eliminates the following warnings from bpctl_mod.c:
 drivers/staging/silicom/bpctl_mod.c:1507:6: warning: no previous prototype 
 for ‘send_bypass_clear_pulse’ [-Wmissing-prototypes]
 drivers/staging/silicom/bpctl_mod.c:1762:5: warning: no previous prototype 
 for ‘cmnd_on’ [-Wmissing-prototypes]

Don't do these massive text blocks.  Just put one or two of the errors
so we get the idea.

regards,
dan carpenter

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


Re: [PATCH] drivers: staging: Mark functions as static and remove unused function in bpctl_mod.c

2014-01-24 Thread Rashika Kheria
On Fri, Jan 24, 2014 at 1:34 PM, Dan Carpenter dan.carpen...@oracle.com wrote:
 On Fri, Jan 24, 2014 at 04:14:00AM +0530, Rashika Kheria wrote:
 Mark functions as static in bpctl_mod.c because they are not used
 outside this file. Remove unused function from bpctl_mod.c.

 This also eliminates the following warnings from bpctl_mod.c:
 drivers/staging/silicom/bpctl_mod.c:1507:6: warning: no previous prototype 
 for ‘send_bypass_clear_pulse’ [-Wmissing-prototypes]
 drivers/staging/silicom/bpctl_mod.c:1762:5: warning: no previous prototype 
 for ‘cmnd_on’ [-Wmissing-prototypes]

 Don't do these massive text blocks.  Just put one or two of the errors
 so we get the idea.


Sure.

Thanks,
-- 
Rashika Kheria
B.Tech CSE
IIIT Hyderabad
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel