Messages in this thread | | | Subject | Re: [PATCH 2/2] drivers:staging:gdm724x Fix Alignment should match open parenthesis | From | Joe Perches <> | Date | Wed, 28 Oct 2015 11:23:51 -0700 |
| |
On Wed, 2015-10-28 at 19:13 +0100, Bogicevic Sasa wrote: > This fixes all Alignment should match open parenthesis > messages from checkpatch.pl
What you suggest is not at all generally accepted kernel style.
> diff --git a/drivers/staging/gdm724x/gdm_lte.c b/drivers/staging/gdm724x/gdm_lte.c [] > @@ -161,12 +161,15 @@ static int gdm_lte_emulate_arp(struct sk_buff *skb_in, u32 nic_type) > return -ENOMEM; > skb_reserve(skb_out, NET_IP_ALIGN); > > - memcpy(skb_put(skb_out, mac_header_len), mac_header_data, > + memcpy( > + skb_put(skb_out, mac_header_len), mac_header_data, > mac_header_len);
What's desired here is:
memcpy(skb_put(skb_out, mac_header_len), mac_header_data, mac_header_len);
Second line should be: [tab][7 spaces]mac_header_len);
etc...
btw: Please run your _patches_ through checkpatch.pl, not just use it to find style issues in files.
| |