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 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 al

[PATCH 1/2] drivers: bcm: Mark functions as static in Qos.c

2013-12-21 Thread Rashika Kheria
] drivers/staging/bcm/Qos.c:61:6: warning: no previous prototype for ‘MatchDestIpAddress’ [-Wmissing-prototypes] drivers/staging/bcm/Qos.c:94:6: warning: no previous prototype for ‘MatchTos’ [-Wmissing-prototypes] Signed-off-by: Rashika Kheria --- drivers/staging/bcm/Qos.c |6 +++--- 1 file

[PATCH 2/2] drivers: bcm: Remove unused function in nvm.c

2013-12-21 Thread Rashika Kheria
Remove unused function PropagateCalParamsFromEEPROMToMemory() in nvm.c. This eliminates the following warning in nvm.c: drivers/staging/bcm/nvm.c:1369:5: warning: no previous prototype for ‘PropagateCalParamsFromEEPROMToMemory’ [-Wmissing-prototypes] Signed-off-by: Rashika Kheria --- drivers

[PATCH 2/2] drivers: dgap: Include appropriate header file in dgap_trace.c

2013-12-21 Thread Rashika Kheria
‘dgap_tracer_free’ [-Wmissing-prototypes] Signed-off-by: Rashika Kheria --- drivers/staging/dgap/dgap_trace.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/dgap/dgap_trace.c b/drivers/staging/dgap/dgap_trace.c index 0f9a956..6c8b510 100644 --- a/drivers/staging/dgap/dgap_trace.c

[PATCH 1/2] drivers: dgap: Include appropriate header file in dgap_parse.c

2013-12-21 Thread Rashika Kheria
previous prototype for ‘dgap_get_config_letters’ [-Wmissing-prototypes] Signed-off-by: Rashika Kheria --- drivers/staging/dgap/dgap_parse.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/dgap/dgap_parse.c b/drivers/staging/dgap/dgap_parse.c index ff9d194..36fd93d 100644 --- a

[PATCH] drivers: dgnc: Include appropriate header file in dgnc_trace.c

2013-12-21 Thread Rashika Kheria
‘dgnc_tracer_free’ [-Wmissing-prototypes] Signed-off-by: Rashika Kheria --- drivers/staging/dgnc/dgnc_trace.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/dgnc/dgnc_trace.c b/drivers/staging/dgnc/dgnc_trace.c index a98b7d4..e2164e4 100644 --- a/drivers/staging/dgnc/dgnc_trace.c

[PATCH] drivers: dwc2: Mark function as static in core.c

2013-12-21 Thread Rashika Kheria
: Rashika Kheria --- drivers/staging/dwc2/core.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/dwc2/core.c b/drivers/staging/dwc2/core.c index 6d001b5..c029cb6 100644 --- a/drivers/staging/dwc2/core.c +++ b/drivers/staging/dwc2/core.c @@ -2736,7 +2736,7 @@ int

[PATCH] drivers: sbe-2t3e3: Mark functions as static in ctrl.c

2013-12-21 Thread Rashika Kheria
previous prototype for ‘t3e3_port_get_stats’ [-Wmissing-prototypes] drivers/staging/sbe-2t3e3/ctrl.c:285:6: warning: no previous prototype for ‘t3e3_port_del_stats’ [-Wmissing-prototypes] Signed-off-by: Rashika Kheria --- drivers/staging/sbe-2t3e3/ctrl.c | 14 +++--- 1 file changed

[PATCH] drivers: sm7xxfb: Mark function as static in sm7xxfb.c

2013-12-21 Thread Rashika Kheria
Mark function smtcfb_setmode() as static in sm7xxfb.c because it is not used outside this file. This eliminates the following warning in sm7xxfb.c: drivers/staging/sm7xxfb/sm7xxfb.c:588:6: warning: no previous prototype for ‘smtcfb_setmode’ [-Wmissing-prototypes] Signed-off-by: Rashika Kheria

[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

Re: [Suggestion] drivers: staging: dgrp: dgrp_net_ops.c: 'buf' is used without initialized.

2013-12-10 Thread Rashika Kheria
> The first conditition is always true and the second condition is always >> false. >> >> + memcpy(buf, b, remain); >> I will check for this part of the code. > > Yeah, for me, the code in handle_data_in_packet() needs re-construction &g

[PATCH v4 2/2] Staging: zram: Fix decrement of variable by calling bdput()

2013-10-22 Thread Rashika Kheria
variable after usage. Signed-off-by: Rashika Kheria --- drivers/staging/zram/zram_drv.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/zram/zram_drv.c b/drivers/staging/zram/zram_drv.c index 8679a06..2cb33ab 100644 --- a/drivers/staging/zram/zram_drv.c +++ b/drivers/staging/

[PATCH v4 1/2] Staging: zram: Fix variable dereferenced before check

2013-10-22 Thread Rashika Kheria
sk' (see line 896) Signed-off-by: Rashika Kheria --- This revision fixes the following issues of the previous revision: Remove unnecessary checks drivers/staging/zram/zram_drv.c | 16 +++- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/drivers/staging/zram/zram_

[PATCH v3] Staging: zram: Fix variable dereferenced before check

2013-10-21 Thread Rashika Kheria
sk' (see line 896) Signed-off-by: Rashika Kheria --- This revision fixes the following issues of the previous revision- Not included null check drivers/staging/zram/zram_drv.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/staging/zram/zram_drv.c