staging: rtl8712: Fixed Multiple Parenthesis Alignment
PATCH 1/1 staging: rtl8712: Fixed Multiple Parenthesis Alignment
Fixed multiple parenthesis alignment warnings.
--- drivers/staging/rtl8712/hal_init.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-)
drivers/staging/rtl8712/hal_init.c +4 -3
@@ -197,7 +197,7 @@ static u8 rtl8712_dl_fw(struct _adapter *padapter) 0x0000ffff);memcpy (ppayload, ptr, dump_imem_sz);r8712_write_mem (padapter,RTL8712_DMA_VOQ , - dump_imem_sz +TXDESC_SIZE , + dump_imem_sz +TXDESC_SIZE , (u8 *)ptx_desc); ptr += dump_imem_sz; imem_sz -= dump_imem_sz;
@@ -226,7 +226,8 @@ static u8 rtl8712_dl_fw(struct _adapter *padapter) 0x0000ffff);memcpy (ppayload, ptr, dump_emem_sz);r8712_write_mem (padapter,RTL8712_DMA_VOQ , - dump_emem_sz +TXDESC_SIZE , (u8 *)ptx_desc); + dump_emem_sz +TXDESC_SIZE , + (u8 *)ptx_desc); ptr += dump_emem_sz; emem_sz -= dump_emem_sz; }while (emem_sz >0 );
@@ -278,7 +279,7 @@ static u8 rtl8712_dl_fw(struct _adapter *padapter) ptx_desc->txdw0 |=cpu_to_le32 (BIT (28 ));memcpy (ppayload, &fwhdr.fwpriv, fwhdr.fw_priv_sz);r8712_write_mem (padapter,RTL8712_DMA_VOQ , - fwhdr.fw_priv_sz +TXDESC_SIZE , (u8 *)ptx_desc); + fwhdr.fw_priv_sz +TXDESC_SIZE , (u8 *)ptx_desc);/* polling dmem code done */ i =100 ;
Signature · 1 line
1.9.1
Quoted 2 lines · Parth Sane
Fixed multiple parenthesis alignment warnings.
Signed-off-by Parth Sane <[email protected]>
Please always add a blank line after your changelog and before your "Signed-off-by:" line
drivers/staging/rtl8712/hal_init.c
Quoted 1 line
drivers/staging/rtl8712/hal_init.c +0
-0
[]
drivers/staging/rtl8712/hal_init.c static u8 rtl8712_dl_fw(struct _adapter *padapter) · line 201
Quoted 7 lines
drivers/staging/rtl8712/hal_init.c +1
-1@@ -197,7 +197,7 @@ static u8 rtl8712_dl_fw(struct _adapter *padapter) 0x0000ffff); memcpy(ppayload, ptr, dump_imem_sz); r8712_write_mem(padapter, RTL8712_DMA_VOQ, - dump_imem_sz + TXDESC_SIZE, + dump_imem_sz +TXDESC_SIZE , (u8 *)ptx_desc);
Please align all lines of multi-line statements.
checkpatch is a stupid little script.
You have to make sure your own patches are sensible.
There are several things you should read and understand in the Documentation directory.
Documentation/CodingStyle Documentation/SubmittingPatches Documentation/SubmitChecklist
Fixing checkpatch messages is sometimes a decent way to understand this process, but the patches that can be generated fixing these messages tend to have low overall value.
After doing a few of these, please move on to some part of the kernel that interests you and submit new code or start to fix actual defects.
Maybe if you are interested in wireless drivers, buy a NIC that isn't supported by the mac80211 drivers and adapt the existing staging driver for that variant.
Code, test, validate, submit, repeat...
Quoted 41 lines · Joe Perches
Quoted 2 lines · Parth Sane
Fixed multiple parenthesis alignment warnings.
Signed-off-by Parth Sane <[email protected]>Please always add a blank line after your changelog and before your "Signed-off-by:" line
drivers/staging/rtl8712/hal_init.c
Quoted 1 line
drivers/staging/rtl8712/hal_init.c +0
-0[]
drivers/staging/rtl8712/hal_init.c static u8 rtl8712_dl_fw(struct _adapter *padapter) · line 201
Quoted 7 lines
drivers/staging/rtl8712/hal_init.c +1
-1@@ -197,7 +197,7 @@ static u8 rtl8712_dl_fw(struct _adapter *padapter) 0x0000ffff);memcpy (ppayload, ptr, dump_imem_sz);r8712_write_mem (padapter,RTL8712_DMA_VOQ , - dump_imem_sz +TXDESC_SIZE , + dump_imem_sz +TXDESC_SIZE , (u8 *)ptx_desc);Please align all lines of multi-line statements.
checkpatch is a stupid little script.
You have to make sure your own patches are sensible.There are several things you should read and understand in the Documentation directory.
Documentation/CodingStyleDocumentation/SubmittingPatchesDocumentation/SubmitChecklistFixing checkpatch messages is sometimes a decent way to understand this process, but the patches that can be generated fixing these messages tend to have low overall value.
After doing a few of these, please move on to some part of the kernel that interests you and submit new code or start to fix actual defects.
Quoted 7 lines
Maybe if you are interested in wireless drivers, buy a NIC that isn't supported by the mac80211 drivers and adapt the existing staging driver for that variant.
Code, test, validate, submit, repeat...
Hi,
I’m first trying my hands at easy stuff before I actually move on to more difficult stuff. I will surely look in that direction. Thanks for your valuable inputs. Regards,
Parth Sane
A message that is not in the archive (sent off-list or not yet received): CAEE9rjYEj98=riyLLrWg9ErLwVV5sMbT27MpZ2-UajUC066AQw@mail.gmail.com
Quoted 65 lines · Roger H Newell
Quoted 54 lines · Parth Sane
Quoted 42 lines · Joe Perches
Quoted 2 lines · Parth Sane
Fixed multiple parenthesis alignment warnings.
Signed-off-by Parth Sane <[email protected]>Please always add a blank line after your changelog and before your "Signed-off-by:" line
drivers/staging/rtl8712/hal_init.c
Quoted 1 line
drivers/staging/rtl8712/hal_init.c +0
-0[]
drivers/staging/rtl8712/hal_init.c static u8 rtl8712_dl_fw(struct _adapter *padapter) · line 201
Quoted 7 lines
drivers/staging/rtl8712/hal_init.c +1
-1@@ -197,7 +197,7 @@ static u8 rtl8712_dl_fw(struct _adapter *padapter) 0x0000ffff);memcpy (ppayload, ptr, dump_imem_sz);r8712_write_mem (padapter,RTL8712_DMA_VOQ , - dump_imem_sz +TXDESC_SIZE , + dump_imem_sz +TXDESC_SIZE , (u8 *)ptx_desc);Please align all lines of multi-line statements.
checkpatch is a stupid little script.
You have to make sure your own patches are sensible.There are several things you should read and understand in the Documentation directory.
Documentation/CodingStyleDocumentation/SubmittingPatchesDocumentation/SubmitChecklistFixing checkpatch messages is sometimes a decent way to understand this process, but the patches that can be generated fixing these messages tend to have low overall value.
After doing a few of these, please move on to some part of the kernel that interests you and submit new code or start to fix actual defects.
Quoted 7 lines
Maybe if you are interested in wireless drivers, buy a NIC that isn't supported by the mac80211 drivers and adapt the existing staging driver for that variant.
Code, test, validate, submit, repeat...
Hi,
I’m first trying my hands at easy stuff before I actually move on to more difficult stuff. I will surely look in that direction. Thanks for your valuable inputs. Regards,
Parth SaneQuoted 5 lines
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Quoted 4 lines
www.kernelnewbies.org is an excellent resource for getting started. Check out the first patch tutorial, it walks you through all the basics of creating and sending a properly formated patch. You're better off learning to do it right the first time, if its not right the patch simply won't be accepted and you'll have to redo it anyway. Take some time to go through the tutorial I mentioned. Good luck
Roger H. Newell
Hi Roger,
Thanks for your comments. I’ve correctly submitted a patchset for the multiple FSF notices, maybe you can comment if its any good?. I’m through the rough patch and now understand the process now quite fairly. I did checkout kernelnewbies.org before I submitted the patch, but alas I needed some practical XP.
Regards,
Parth Sane