On Tue, Oct 27, 2015 at 1:42 PM, Kalle Valo wrote:
> punit vara writes:
>
>> Will my other patches which are already correct be added to wireless
>> tree ? or I have to resend everything ?
>
> Yes, please resend the whole patchset. I don't apply patches
> individ
Remove uncessary repeated arguments COMP_EFUSE, COMP_REGD, COMP_CHAN
with OR(|)
This is patch to the debug.c file that removes following warning
reported by coccicheck:
-duplicated argument to & or |
Signed-off-by: Punit Vara
---
drivers/net/wireless/rtlwifi/debug.c | 6 +++---
1
This patch is to the htt_rx.c that removes unneeded semicolon which is
reported by coccicheck.
Here semicolon just create empty statement so please remote it.
Signed-off-by: Punit Vara
---
drivers/net/wireless/ath/ath10k/htt_rx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
Remove uncessary repeated arguments with OR(|)
This is patch to the brcmsmac/channel.c file that removes following
warning reported by coccicheck:
-duplicated argument to & or |
Signed-off-by: Punit Vara
---
drivers/net/wireless/brcm80211/brcmsmac/channel.c | 1 -
1 file changed, 1 dele
This patch is to the carl9170/phy.c file that fixes warning reported by
coccicheck :
WARNING: end returns can be simplified
I have removed unneccessary variable declaration and simply return flow
for carl9170_regwrite_result()
Signed-off-by: Punit Vara
---
drivers/net/wireless/ath/carl9170
This patch is to the zd1201.c file that fixes up warning
reported by coccicheck:
WARNING: end returns can be simplified and declaration on line 1658 can
be dropped
Prefer direct return value instead of writing 2-3 more sentence.
Signed-off-by: Punit Vara
---
drivers/net/wireless/zd1201.c | 6
This is patch to the brcmsmac/main.c that removes unnecessary variable
which was declared to return zero.
This patch fixes up warning reported by coccicheck:
-Unneeded variable: "err". Return "0" on line 3788
Signed-off-by: Punit Vara
---
drivers/net/wireless/brcm80211
This patch is to the ath5k/eeprom.c that fixes up warning caught by
coccicheck:
-Unneeded variable: "ret". Return "0" on line 1733
Remove unneccesary variable ret created to return zero.
Signed-off-by: Punit Vara
---
drivers/net/wireless/ath/ath5k/eeprom.c | 4 ++-
This patch is to the brcmsmac/stf.c that fixes up warning caught by
coccicheck:
-Unneeded variable: "ret_code". Return "0" on line 328
Remove unneccesary variable ret_code created to return zero.
Signed-off-by: Punit Vara
---
drivers/net/wireless/brcm80211/brcmsmac/stf
This patch is to the 3945-mac.c file that fixes up following warning
by coccicheck:
drivers/net/wireless/iwlegacy/3945-mac.c:247:5-8: Unneeded variable:
"ret". Return "- EOPNOTSUPP" on line 249
Return -EOPNOTSUPP directly instead of return using ret
Signed-off-by: Punit Va
This patch is to the ath10k/wmi.h that removes unneeded semicolon which
is reported by coccicheck.
Here semicolon just create empty statement so please remote it.
Signed-off-by: Punit Vara
---
drivers/net/wireless/ath/ath10k/wmi.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
This patch is to the ath5k/eeprom.c that fixes up warning caught by
coccicheck:
Unneeded variable: "ret". Return "0" on line 980
Remove unneeded variable ret created to return zero.
Signed-off-by: Punit Vara
---
Remove empty line suggested by Sergei
drivers/net/wireles
On Fri, Oct 23, 2015 at 12:26 AM, Sergei Shtylyov
wrote:
> On 10/22/2015 09:47 PM, Punit Vara wrote:
>
>> Remove black line suggested by Sergei
>
>
>Such kind of comments should be under the --- tear line.
>
>>
>> This patch is to the ath5k/eepr
This warning is reported by coccicheck:
Signed-off-by: Punit Vara
---
drivers/net/wireless/ath/wcn36xx/main.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/net/wireless/ath/wcn36xx/main.c
b/drivers/net/wireless/ath/wcn36xx/main.c
index 900e72a..94bcc08 100644
On Fri, Oct 23, 2015 at 12:05 AM, Sergei Shtylyov
wrote:
> Hello.
>
> On 10/22/2015 09:26 PM, Punit Vara wrote:
>
>> This patch is to the ath5k/eeprom.c that fixes up warning caught by
>> coccicheck:
>>
>> -Unneeded variable: "ret". Return "0&
Remove empty line suggested by Sergei
This patch is to the ath5k/eeprom.c that fixes up warning caught by
coccicheck:
Unneeded variable: "ret". Return "0" on line 980
Remove unneeded variable ret created to return zero.
Signed-off-by: Punit Vara
---
drivers/net/wireles
Remove black line suggested by Sergei
This patch is to the ath5k/eeprom.c that fixes up warning caught by
coccicheck:
Unneeded variable: "ret". Return "0" on line 980
Remove unneeded variable ret created to return zero.
Signed-off-by: Punit Vara
---
drivers/net/wireles
This patch is to the ath5k/eeprom.c that fixes up warning caught by
coccicheck:
-Unneeded variable: "ret". Return "0" on line 1733
Remove unneccesary variable ret created to return zero.
Also removed empty line suggested by Sergei
Signed-off-by: Punit Vara
---
drivers/ne
On Thu, Oct 22, 2015 at 3:16 AM, Sergei Shtylyov
wrote:
> On 10/21/2015 05:55 PM, Punit Vara wrote:
>
>> This patch is to the ath5k/eeprom.c that fixes up warning caught by
>> coccicheck:
>>
>> Unneeded variable: "ret". Return "0" on line 980
&g
On Thu, Oct 22, 2015 at 3:13 AM, Sergei Shtylyov
wrote:
> Hello.
>
> On 10/21/2015 05:55 PM, Punit Vara wrote:
>
>> This patch is to the ath10k/pci.h file that fixes following warning
>
>
>pci.c, you mean?
>
>
>> reported by coccicheck:
>>
&
This patch is to the ath5k/eeprom.c that fixes up warning caught by
coccicheck:
-Unneeded variable: "ret". Return "0" on line 1733
Remove unneccesary variable ret created to return zero.
Signed-off-by: Punit Vara
---
drivers/net/wireless/ath/ath5k/eeprom.c | 5 ++--
remove int ret suggested by Jiri Slaby
This patch is to the at76c50x-usb.c file that fixes up warning
reported by coccicheck:
WARNING: end returns can be simplified if negative or 0 value
Prefer direct return value instead of writing 2-3 more sentence.
Signed-off-by: Punit Vara
---
drivers
Remove int ret suggested by kbuild test robot
This patch is to the wlcore/acx.c file that fixes up warning
reported by coccicheck:
WARNING: end returns can be simplified if negative or 0 value
Prefer direct return value instead of writing 2-3 more sentence.
Signed-off-by: Punit Vara
On Wed, Oct 21, 2015 at 8:30 PM, Jiri Slaby wrote:
> On 10/21/2015, 04:55 PM, Punit Vara wrote:
>> @@ -544,13 +544,10 @@ static void at76_ledtrig_tx_activity(void)
>> static int at76_remap(struct usb_device *udev)
>> {
>> int ret;
>> - ret = usb_contr
This patch is to the ath10k/wmi.h that removes unneeded semicolon which
is reported by coccicheck.
Here semicolon just create empty statement so please remote it.
Signed-off-by: Punit Vara
---
drivers/net/wireless/ath/ath10k/wmi.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
This patch is to the htt_rx.c that removes unneeded semicolon which is
reported by coccicheck.
Here semicolon just create empty statement so please remote it.
Signed-off-by: Punit Vara
---
drivers/net/wireless/ath/ath10k/htt_rx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
This patch is to the libipw_rx.c file that fixes following warning
reported by coccicheck:
WARNING: sum of probable bitmasks, consider |
I have replaced + with OR operator | for summing bitmasks
Signed-off-by: Punit Vara
---
drivers/net/wireless/ipw2x00/libipw_rx.c | 4 ++--
1 file changed, 2
This patch is to the ath10k/pci.h file that fixes following warning
reported by coccicheck:
WARNING: sum of probable bitmasks, consider |
I have replaced + with OR operator | for summing bitmasks
Signed-off-by: Punit Vara
---
drivers/net/wireless/ath/ath10k/pci.c | 10 +-
1 file
This patch is to the zd1201.c file that fixes up warning
reported by coccicheck:
WARNING: end returns can be simplified and declaration on line 1658 can
be dropped
Prefer direct return value instead of writing 2-3 more sentence.
Signed-off-by: Punit Vara
---
drivers/net/wireless/zd1201.c | 6
This patch is to the at76c50x-usb.c file that fixes up warning
reported by coccicheck:
WARNING: end returns can be simplified if negative or 0 value
Prefer direct return value instead of writing 2-3 more sentence.
Signed-off-by: Punit Vara
---
drivers/net/wireless/at76c50x-usb.c | 5 +
1
This patch is to the 3945-mac.c file that fixes up following warning
by coccicheck:
drivers/net/wireless/iwlegacy/3945-mac.c:247:5-8: Unneeded variable:
"ret". Return "- EOPNOTSUPP" on line 249
Return -EOPNOTSUPP directly instead of return using ret
Signed-off-by: Punit Va
Remove uncessary repeated arguments COMP_EFUSE, COMP_REGD, COMP_CHAN
with OR(|)
This is patch to the debug.c file that removes following warning
reported by coccicheck:
-duplicated argument to & or |
Signed-off-by: Punit Vara
---
drivers/net/wireless/rtlwifi/debug.c | 6 +++---
1
This is patch to the brcmsmac/main.c that removes unnecessary variable
which was declared to return zero.
This patch fixes up warning reported by coccicheck:
-Unneeded variable: "err". Return "0" on line 3788
Signed-off-by: Punit Vara
---
drivers/net/wireless/brcm80211
This patch is to the carl9170/phy.c file that fixes warning reported by
coccicheck :
WARNING: end returns can be simplified
I have removed unneccessary variable declaration and simply return flow
for carl9170_regwrite_result()
Signed-off-by: Punit Vara
---
drivers/net/wireless/ath/carl9170
This patch is to the brcmsmac/stf.c that fixes up warning caught by
coccicheck:
-Unneeded variable: "ret_code". Return "0" on line 328
Remove unneccesary variable ret_code created to return zero.
Siged-off-by: Punit Vara
---
drivers/net/wireless/brcm80211/brcmsmac/stf
This patch is to the ath5k/eeprom.c that fixes up warning caught by
coccicheck:
-Unneeded variable: "ret". Return "0" on line 1733
Remove unneccesary variable ret created to return zero.
Signed-off-by: Punit Vara
---
drivers/net/wireless/ath/ath5k/eeprom.c | 4 ++-
This patch is to the ath5k/eeprom.c that fixes up warning caught by
coccicheck:
Unneeded variable: "ret". Return "0" on line 980
Remove unneeded variable ret created to return zero.
Signed-off-by: Punit Vara
---
drivers/net/wireless/ath/wcn36xx/main.c | 3 +--
1 file
Fix various warning reported by coccicheck:
make coccicheck M=drivers/net/wireless
Punit Vara (15):
net: wireless: ath: use | instead of + for summing bitmasks
net: wireless: ath: Remove unnecessary semicolon
net: wireless: ath: Remove unnecessary semicolon
net: wireless: ipw2x00: use
Remove uncessary repeated arguments with OR(|)
This is patch to the brcmsmac/channel.c file that removes following
warning reported by coccicheck:
-duplicated argument to & or |
Signed-off-by: Punit Vara
---
drivers/net/wireless/brcm80211/brcmsmac/channel.c | 1 -
1 file changed, 1 dele
This patch is to the wlcore/acx.c file that fixes up warning
reported by coccicheck:
WARNING: end returns can be simplified if negative or 0 value
Prefer direct return value instead of writing 2-3 more sentence.
Signed-off-by: Punit Vara
---
drivers/net/wireless/ti/wlcore/acx.c | 6 +-
1
: Punit Vara
---
man2/adjtimex.2 | 24 ++--
1 file changed, 22 insertions(+), 2 deletions(-)
diff --git a/man2/adjtimex.2 b/man2/adjtimex.2
index 04b53b1..77b4e30 100644
--- a/man2/adjtimex.2
+++ b/man2/adjtimex.2
@@ -276,8 +276,28 @@ Mode (0 = Phase Locked Loop, 1 = Frequency
41 matches
Mail list logo