On Fri, 2017-06-16 at 02:19 -0700, Joe Perches wrote: > > > if (frag_size == > > PN533_CMD_DATAFRAME_MAXLEN) > > - *skb_push(frag, sizeof(u8)) = > > - (PN533_CMD > > _MI_MASK | 1); > > - else > > - *skb_push(frag, sizeof(u8)) = 1; > > /* TG */ > > + *(u8 *)skb_push(frag, sizeof(u8)) > > = > > + > > (PN533_CMD_MI_MASK | 1); > > + else > > + *(u8 *)skb_push(frag, > > sizeof(u8)) = 1; /* TG */ > > Can you check all the if/else uses. > This one is not formatted correctly. > I didn't look at any others.
Hmm, that's annoying. I did fix up a few, but I guess I missed this one. johannes
