Re: [PATCH v3] Staging: rtl8192e: rtllib_rx: fix alignment

[Yusef Aslam]

On Thu, 2024-06-27 at 15:43 +0200, Greg KH wrote:
> This all should not be in the body of the email, please use a tool
> like
> 'git send-email' to send patches.

Oh k thanks, I was trying to use git send-email but in the end I just
copy pasted the email into Evolution as msmtp was not sending emails
for some reason.

> > Fix alignment.
> 
> We need more description than that, right?  What would you want to
> see
> here?

Yeah, I guess I would like to see a more detailed description like "Fix
alignment to improve readability" or something.

> > 
> > Signed-off-by: Yusef Aslam <YUZi54780@xxxxxxxxxxx>
> > ---
> >  v3:
> >  - Used the correct email addresses.
> >  - Developed against the correct git repository.
> >  v2:
> >  - The email address of Greg Kroah-Hartman was wrong.
> >  - Developed against the wrong git repository.
> >  v1:
> >  - Developed against the wrong git repository.
> > 
> >  drivers/staging/rtl8192e/rtllib_rx.c | 110 +++++++++++++----------
> > ----
> >  1 file changed, 55 insertions(+), 55 deletions(-)
> > 
> > diff --git a/drivers/staging/rtl8192e/rtllib_rx.c
> > b/drivers/staging/rtl8192e/rtllib_rx.c
> > index 84ca5d769b7e..1f6c4a3de5c2 100644
> > --- a/drivers/staging/rtl8192e/rtllib_rx.c
> > +++ b/drivers/staging/rtl8192e/rtllib_rx.c
> > @@ -410,7 +410,7 @@ static bool add_reorder_entry(struct
> > rx_ts_record *ts,
> >  	while (list->next != &ts->rx_pending_pkt_list) {
> >  		if (SN_LESS(pReorderEntry->SeqNum, ((struct
> > rx_reorder_entry *)
> >  		    list_entry(list->next, struct
> > rx_reorder_entry,
> > -		    list))->SeqNum))
> > +			       list))->SeqNum))
> >  			list = list->next;
> >  		else if (SN_EQUAL(pReorderEntry->SeqNum,
> >  			((struct rx_reorder_entry
> > *)list_entry(list->next,
> > @@ -736,7 +736,7 @@ static u8 parse_subframe(struct rtllib_device
> > *ieee, struct sk_buff *skb,
> >  	/* just for debug purpose */
> >  	SeqNum = WLAN_GET_SEQ_SEQ(le16_to_cpu(hdr->seq_ctrl));
> >  	if ((RTLLIB_QOS_HAS_SEQ(fc)) &&
> > -	   (((union frameqos *)(skb->data + RTLLIB_3ADDR_LEN))-
> > >field.reserved))
> > +	    (((union frameqos *)(skb->data + RTLLIB_3ADDR_LEN))-
> > >field.reserved))
> >  		is_aggregate_frame = true;
> >  
> >  	if (RTLLIB_QOS_HAS_SEQ(fc))
> > @@ -876,13 +876,13 @@ static int rtllib_rx_check_duplicate(struct
> > rtllib_device *ieee,
> >  	frag = WLAN_GET_SEQ_FRAG(sc);
> >  
> >  	if (!ieee->ht_info->cur_rx_reorder_enable ||
> > -		!ieee->current_network.qos_data.active ||
> > -		!is_data_frame(skb->data) ||
> > -		is_legacy_data_frame(skb->data)) {
> > -		if (!ieee80211_is_beacon(hdr->frame_control)) {
> > -			if (is_duplicate_packet(ieee, hdr))
> > -				return -1;
> > -		}
> > +	    !ieee->current_network.qos_data.active ||
> > +	    !is_data_frame(skb->data) ||
> > +	    is_legacy_data_frame(skb->data)) {
> > +	  if (!ieee80211_is_beacon(hdr->frame_control)) {
> > +	    if (is_duplicate_packet(ieee, hdr))
> > +	      return -1;
> > +	  }
> 
> That's obviously not correct.  Did you run your patch through
> checkpatch.pl after creating it?

I did not, thank you.

> thanks,
> 
> greg k-h