From: Luis Felipe Hernandez <luis.hernandez093@gmail.com>
To: Thinh.Nguyen@synopsys.com, gregkh@linuxfoundation.org, rbm@suse.com
Cc: Luis Felipe Hernandez <luis.hernandez093@gmail.com>,
skhan@linuxfoundation.org, linux-usb@vger.kernel.org,
linux-kernel-mentees@lists.linuxfoundation.org,
linux-kernel@vger.kernel.org
Subject: [PATCH v3] usb: dwc3: remove unused sg struct member
Date: Tue, 19 Nov 2024 16:24:51 -0500 [thread overview]
Message-ID: <20241119212452.269255-1-luis.hernandez093@gmail.com> (raw)
The sg member of the dwc3_request struct is no longer used and should be
removed. This patch drops the unused member, cleaning up the struct.
Previously, this patch addressed a documentation warning caused by the
missing description for the sg member. However, upon review,
it was determined that the member itself is deprecated and can be removed
entirely.
This change improves code clarity and avoids maintaining unnecessary members
in the structure.
Signed-off-by: Luis Felipe Hernandez <luis.hernandez093@gmail.com>
Reported-by: Thinh Nguyen <Thinh.Nguyen@synopsys.com>
Reviewed-by: Ricardo Marliere <rbm@suse.com>
---
v2: remove unused sg struct member as per review[1]
v3:
- Add reported-by tag as per review [2]
- Carry over reviewed-by tag from v2 [3]
- Update commit subject to reflect maintainers
- Update commit message to reflect actual change gathered from
Thinh Nguyen's feedback
[1] https://lore.kernel.org/all/20241119020807.cn7ugxnhbkqwrr2b@synopsys.com/
[2] https://lore.kernel.org/all/2024111922-pantyhose-panorama-6f16@gregkh/
[3] https://lore.kernel.org/all/5l65sdskdzbehxamff5ax4ptiqhaxh7ewi4umtpp6ynen45nj6@nebuxjg4c4rx/
---
drivers/usb/dwc3/core.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h
index ee73789326bc..3be069c4520e 100644
--- a/drivers/usb/dwc3/core.h
+++ b/drivers/usb/dwc3/core.h
@@ -956,7 +956,6 @@ struct dwc3_request {
struct usb_request request;
struct list_head list;
struct dwc3_ep *dep;
- struct scatterlist *sg;
struct scatterlist *start_sg;
unsigned int num_pending_sgs;
--
2.47.0
next reply other threads:[~2024-11-19 21:24 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-11-19 21:24 Luis Felipe Hernandez [this message]
2024-11-19 22:19 ` [PATCH v3] usb: dwc3: remove unused sg struct member Thinh Nguyen
2024-11-20 14:17 ` Luis Felipe Hernandez
2024-11-20 14:25 ` gregkh
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20241119212452.269255-1-luis.hernandez093@gmail.com \
--to=luis.hernandez093@gmail.com \
--cc=Thinh.Nguyen@synopsys.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel-mentees@lists.linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=rbm@suse.com \
--cc=skhan@linuxfoundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox