[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID:
<CWLP265MB6516AB499D407C0AB0CF106EFDD72@CWLP265MB6516.GBRP265.PROD.OUTLOOK.COM>
Date: Thu, 27 Jun 2024 19:09:50 +0100
From: Yusef Aslam <yuzi54780@...look.com>
To: Greg KH <gregkh@...uxfoundation.org>
Cc: linux-staging@...ts.linux.dev, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3] Staging: rtl8192e: rtllib_rx: fix alignment
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@...look.com>
> > ---
> > 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
Powered by blists - more mailing lists