lkml.org 
[lkml]   [2016]   [Mar]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] staging: rtl8712: Fixed Multiple Parenthesis Alignment
From
Date

> On 20-Mar-2016, at 9:18 PM, Joe Perches <joe@perches.com> wrote:
>
> On Sun, 2016-03-20 at 14:33 +0000, Parth Sane wrote:
>> Fixed multiple parenthesis alignment warnings.
>> Signed-off-by: Parth Sane <laerdevstudios@gmail.com>
>
> Please always add a blank line after your changelog and
> before your "Signed-off-by:" line
>
>> diff --git a/drivers/staging/rtl8712/hal_init.c b/drivers/staging/rtl8712/hal_init.c
> []
>> @@ -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...
>
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
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2016-03-20 17:41    [W:0.201 / U:0.056 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site