[PATCH] staging: rtl8188eu: coding style fixup
checkpatch.pl tiny fix get rid of 2 warnings and 2 errors for hal/fw.c: WARNING: braces {} are not necessary for single statement blocks WARNING: line over 80 characters ERROR: space required before the open brace '{' ERROR: space required after that ',' (ctx:VxV) Signed-off-by: Igor Bogomazov yg...@ygrex.ru Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: navin patidar navin.pati...@gmail.com Cc: Stephen Rothwell s...@canb.auug.org.au --- for linux-next 3.17.0 this is an updated version of the patch I've sent yesterday, that one did not provide a clear description diff --git a/drivers/staging/rtl8188eu/hal/fw.c b/drivers/staging/rtl8188eu/hal/fw.c --- a/drivers/staging/rtl8188eu/hal/fw.c +++ b/drivers/staging/rtl8188eu/hal/fw.c @@ -154,9 +154,8 @@ static int _rtl88e_fw_free_to_go(struct adapter *adapt) break; } while (counter++ POLLING_READY_TIMEOUT_COUNT); - if (counter = POLLING_READY_TIMEOUT_COUNT) { + if (counter = POLLING_READY_TIMEOUT_COUNT) goto exit; - } value32 = usb_read32(adapt, REG_MCUFWDL); value32 |= MCUFWDL_RDY; @@ -193,13 +192,13 @@ int rtl88eu_download_fw(struct adapter *adapt) u32 fwsize; int err; - if (request_firmware(fw, fw_name, device)){ + if (request_firmware(fw, fw_name, device)) { dev_err(device, Firmware %s not available\n, fw_name); return -ENOENT; } if (fw-size FW_8188E_SIZE) { - dev_err(device,Firmware size exceed 0x%X. Check it.\n, + dev_err(device, Firmware size exceed 0x%X. Check it.\n, FW_8188E_SIZE); return -1; } @@ -226,7 +225,8 @@ int rtl88eu_download_fw(struct adapter *adapt) rtl88e_firmware_selfreset(adapt); } _rtl88e_enable_fw_download(adapt, true); - usb_write8(adapt, REG_MCUFWDL, usb_read8(adapt, REG_MCUFWDL) | FWDL_ChkSum_rpt); + usb_write8(adapt, REG_MCUFWDL, + usb_read8(adapt, REG_MCUFWDL) | FWDL_ChkSum_rpt); _rtl88e_write_fw(adapt, pfwdata, fwsize); _rtl88e_enable_fw_download(adapt, false); ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH] staging: rtl8188eu: coding style fixup
On Fri, Oct 10, 2014 at 11:09:20AM +0400, Igor Bogomazov wrote: checkpatch.pl tiny fix get rid of 2 warnings and 2 errors for hal/fw.c: WARNING: braces {} are not necessary for single statement blocks WARNING: line over 80 characters ERROR: space required before the open brace '{' ERROR: space required after that ',' (ctx:VxV) you should usually do only one type of fix in a patch. Signed-off-by: Igor Bogomazov yg...@ygrex.ru Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: navin patidar navin.pati...@gmail.com Cc: Stephen Rothwell s...@canb.auug.org.au --- for linux-next 3.17.0 this is an updated version of the patch I've sent yesterday, that one did not provide a clear description diff --git a/drivers/staging/rtl8188eu/hal/fw.c b/drivers/staging/rtl8188eu/hal/fw.c --- a/drivers/staging/rtl8188eu/hal/fw.c +++ b/drivers/staging/rtl8188eu/hal/fw.c @@ -154,9 +154,8 @@ static int _rtl88e_fw_free_to_go(struct adapter *adapt) break; } while (counter++ POLLING_READY_TIMEOUT_COUNT); - if (counter = POLLING_READY_TIMEOUT_COUNT) { + if (counter = POLLING_READY_TIMEOUT_COUNT) goto exit; - } value32 = usb_read32(adapt, REG_MCUFWDL); value32 |= MCUFWDL_RDY; @@ -193,13 +192,13 @@ int rtl88eu_download_fw(struct adapter *adapt) u32 fwsize; int err; - if (request_firmware(fw, fw_name, device)){ + if (request_firmware(fw, fw_name, device)) { dev_err(device, Firmware %s not available\n, fw_name); return -ENOENT; } if (fw-size FW_8188E_SIZE) { - dev_err(device,Firmware size exceed 0x%X. Check it.\n, + dev_err(device, Firmware size exceed 0x%X. Check it.\n, FW_8188E_SIZE); align this with the open parenthesis return -1; } @@ -226,7 +225,8 @@ int rtl88eu_download_fw(struct adapter *adapt) rtl88e_firmware_selfreset(adapt); } _rtl88e_enable_fw_download(adapt, true); - usb_write8(adapt, REG_MCUFWDL, usb_read8(adapt, REG_MCUFWDL) | FWDL_ChkSum_rpt); + usb_write8(adapt, REG_MCUFWDL, + usb_read8(adapt, REG_MCUFWDL) | FWDL_ChkSum_rpt); same here also. check your patch with --strict option of checkpatch thanks sudip _rtl88e_write_fw(adapt, pfwdata, fwsize); _rtl88e_enable_fw_download(adapt, false); -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
[PATCH] staging: rtl8188eu: coding style fixup
checkpatch.pl tiny fix get rid of 2 warnings and 2 errors for hal/fw.c Signed-off-by: Igor Bogomazov yg...@ygrex.ru Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: navin patidar navin.pati...@gmail.com Cc: Stephen Rothwell s...@canb.auug.org.au --- for linux-next 3.17.0 this is an updated version of the patch I've sent two days ago, that one did not apply cleanly diff --git a/drivers/staging/rtl8188eu/hal/fw.c b/drivers/staging/rtl8188eu/hal/fw.c --- a/drivers/staging/rtl8188eu/hal/fw.c +++ b/drivers/staging/rtl8188eu/hal/fw.c @@ -154,9 +154,8 @@ static int _rtl88e_fw_free_to_go(struct adapter *adapt) break; } while (counter++ POLLING_READY_TIMEOUT_COUNT); - if (counter = POLLING_READY_TIMEOUT_COUNT) { + if (counter = POLLING_READY_TIMEOUT_COUNT) goto exit; - } value32 = usb_read32(adapt, REG_MCUFWDL); value32 |= MCUFWDL_RDY; @@ -193,13 +192,13 @@ int rtl88eu_download_fw(struct adapter *adapt) u32 fwsize; int err; - if (request_firmware(fw, fw_name, device)){ + if (request_firmware(fw, fw_name, device)) { dev_err(device, Firmware %s not available\n, fw_name); return -ENOENT; } if (fw-size FW_8188E_SIZE) { - dev_err(device,Firmware size exceed 0x%X. Check it.\n, + dev_err(device, Firmware size exceed 0x%X. Check it.\n, FW_8188E_SIZE); return -1; } @@ -226,7 +225,8 @@ int rtl88eu_download_fw(struct adapter *adapt) rtl88e_firmware_selfreset(adapt); } _rtl88e_enable_fw_download(adapt, true); - usb_write8(adapt, REG_MCUFWDL, usb_read8(adapt, REG_MCUFWDL) | FWDL_ChkSum_rpt); + usb_write8(adapt, REG_MCUFWDL, + usb_read8(adapt, REG_MCUFWDL) | FWDL_ChkSum_rpt); _rtl88e_write_fw(adapt, pfwdata, fwsize); _rtl88e_enable_fw_download(adapt, false); ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH] staging: rtl8188eu: coding style fixup
But you haven't CCed people in the CC list On 10/9/14, Igor Bogomazov yg...@ygrex.ru wrote: checkpatch.pl tiny fix get rid of 2 warnings and 2 errors for hal/fw.c Signed-off-by: Igor Bogomazov yg...@ygrex.ru Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: navin patidar navin.pati...@gmail.com Cc: Stephen Rothwell s...@canb.auug.org.au --- for linux-next 3.17.0 this is an updated version of the patch I've sent two days ago, that one did not apply cleanly diff --git a/drivers/staging/rtl8188eu/hal/fw.c b/drivers/staging/rtl8188eu/hal/fw.c --- a/drivers/staging/rtl8188eu/hal/fw.c +++ b/drivers/staging/rtl8188eu/hal/fw.c @@ -154,9 +154,8 @@ static int _rtl88e_fw_free_to_go(struct adapter *adapt) break; } while (counter++ POLLING_READY_TIMEOUT_COUNT); - if (counter = POLLING_READY_TIMEOUT_COUNT) { + if (counter = POLLING_READY_TIMEOUT_COUNT) goto exit; - } value32 = usb_read32(adapt, REG_MCUFWDL); value32 |= MCUFWDL_RDY; @@ -193,13 +192,13 @@ int rtl88eu_download_fw(struct adapter *adapt) u32 fwsize; int err; - if (request_firmware(fw, fw_name, device)){ + if (request_firmware(fw, fw_name, device)) { dev_err(device, Firmware %s not available\n, fw_name); return -ENOENT; } if (fw-size FW_8188E_SIZE) { - dev_err(device,Firmware size exceed 0x%X. Check it.\n, + dev_err(device, Firmware size exceed 0x%X. Check it.\n, FW_8188E_SIZE); return -1; } @@ -226,7 +225,8 @@ int rtl88eu_download_fw(struct adapter *adapt) rtl88e_firmware_selfreset(adapt); } _rtl88e_enable_fw_download(adapt, true); - usb_write8(adapt, REG_MCUFWDL, usb_read8(adapt, REG_MCUFWDL) | FWDL_ChkSum_rpt); + usb_write8(adapt, REG_MCUFWDL, + usb_read8(adapt, REG_MCUFWDL) | FWDL_ChkSum_rpt); _rtl88e_write_fw(adapt, pfwdata, fwsize); _rtl88e_enable_fw_download(adapt, false); ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel -- Regards, Denis ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH] staging: rtl8188eu: coding style fixup
you are right, would be wise to do that, added On Thu, 9 Oct 2014 10:32:37 +0400 Denis Kirjanov kirja...@gmail.com wrote: But you haven't CCed people in the CC list On 10/9/14, Igor Bogomazov yg...@ygrex.ru wrote: checkpatch.pl tiny fix get rid of 2 warnings and 2 errors for hal/fw.c Signed-off-by: Igor Bogomazov yg...@ygrex.ru Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: navin patidar navin.pati...@gmail.com Cc: Stephen Rothwell s...@canb.auug.org.au --- for linux-next 3.17.0 this is an updated version of the patch I've sent two days ago, that one did not apply cleanly diff --git a/drivers/staging/rtl8188eu/hal/fw.c b/drivers/staging/rtl8188eu/hal/fw.c --- a/drivers/staging/rtl8188eu/hal/fw.c +++ b/drivers/staging/rtl8188eu/hal/fw.c @@ -154,9 +154,8 @@ static int _rtl88e_fw_free_to_go(struct adapter *adapt) break; } while (counter++ POLLING_READY_TIMEOUT_COUNT); - if (counter = POLLING_READY_TIMEOUT_COUNT) { + if (counter = POLLING_READY_TIMEOUT_COUNT) goto exit; - } value32 = usb_read32(adapt, REG_MCUFWDL); value32 |= MCUFWDL_RDY; @@ -193,13 +192,13 @@ int rtl88eu_download_fw(struct adapter *adapt) u32 fwsize; int err; - if (request_firmware(fw, fw_name, device)){ + if (request_firmware(fw, fw_name, device)) { dev_err(device, Firmware %s not available\n, fw_name); return -ENOENT; } if (fw-size FW_8188E_SIZE) { - dev_err(device,Firmware size exceed 0x%X. Check it.\n, + dev_err(device, Firmware size exceed 0x%X. Check it.\n, FW_8188E_SIZE); return -1; } @@ -226,7 +225,8 @@ int rtl88eu_download_fw(struct adapter *adapt) rtl88e_firmware_selfreset(adapt); } _rtl88e_enable_fw_download(adapt, true); - usb_write8(adapt, REG_MCUFWDL, usb_read8(adapt, REG_MCUFWDL) | FWDL_ChkSum_rpt); + usb_write8(adapt, REG_MCUFWDL, + usb_read8(adapt, REG_MCUFWDL) | FWDL_ChkSum_rpt); _rtl88e_write_fw(adapt, pfwdata, fwsize); _rtl88e_enable_fw_download(adapt, false); ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel -- Sincerely yours, Igor Bogomazov Saint-Petersburg, Russia ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH] staging: rtl8188eu: coding style fixup
On Thu, Oct 09, 2014 at 10:30:06AM +0400, Igor Bogomazov wrote: checkpatch.pl tiny fix get rid of 2 warnings and 2 errors for hal/fw.c What warnings? What errors? Be specific please. Trailing whitespace :( ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH] staging: rtl8188eu: coding style fixup
On Tue, Oct 07, 2014 at 01:11:42PM +0400, Igor Bogomazov wrote: checkpatch.pk tiny fix get rid of 2 warnings and 2 errors for hal/fw.c Signed-off-by: Igor Bogomazov yg...@ygrex.ru Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: navin patidar navin.pati...@gmail.com Cc: Stephen Rothwell s...@canb.auug.org.au --- linux-next 3.17.0 I think someone else already did this before you sent it in, as this doesn't apply to my tree, sorry :( ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
[PATCH] staging: rtl8188eu: coding style fixup
checkpatch.pk tiny fix get rid of 2 warnings and 2 errors for hal/fw.c Signed-off-by: Igor Bogomazov yg...@ygrex.ru Cc: Greg Kroah-Hartman gre...@linuxfoundation.org Cc: navin patidar navin.pati...@gmail.com Cc: Stephen Rothwell s...@canb.auug.org.au --- linux-next 3.17.0 diff --git a/drivers/staging/rtl8188eu/hal/fw.c b/drivers/staging/rtl8188eu/hal/fw.c index 17b7f37..c4293df 100644 --- a/drivers/staging/rtl8188eu/hal/fw.c +++ b/drivers/staging/rtl8188eu/hal/fw.c @@ -154,9 +154,8 @@ static int _rtl88e_fw_free_to_go(struct adapter *adapt) break; } while (counter++ POLLING_READY_TIMEOUT_COUNT); - if (counter = POLLING_READY_TIMEOUT_COUNT) { + if (counter = POLLING_READY_TIMEOUT_COUNT) goto exit; - } value32 = usb_read32(adapt, REG_MCUFWDL); value32 |= MCUFWDL_RDY; @@ -193,13 +192,13 @@ int rtl88eu_download_fw(struct adapter *adapt) u32 fwsize; int err; - if (request_firmware(fw, fw_name, device)){ + if (request_firmware(fw, fw_name, device)) { dev_err(device, Firmware %s not available\n, fw_name); return -ENOENT; } if (fw-size FW_8188E_SIZE) { - dev_err(device,Firmware size exceed 0x%X. Check it.\n, + dev_err(device, Firmware size exceed 0x%X. Check it.\n, FW_8188E_SIZE); return -1; } @@ -226,7 +225,8 @@ int rtl88eu_download_fw(struct adapter *adapt) rtl88e_firmware_selfreset(adapt); } _rtl88e_enable_fw_download(adapt, true); - usb_write8(adapt, REG_MCUFWDL, usb_read8(adapt, REG_MCUFWDL) | FWDL_ChkSum_rpt); + usb_write8(adapt, REG_MCUFWDL, + usb_read8(adapt, REG_MCUFWDL) | FWDL_ChkSum_rpt); _rtl88e_write_fw(adapt, pfwdata, fwsize); _rtl88e_enable_fw_download(adapt, false); ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel