Re: [PATCH v3] usb: dwc3: remove unused sg struct member

[Thinh Nguyen <Thinh.Nguyen@synopsys.com>]

On Tue, Nov 19, 2024, Luis Felipe Hernandez wrote:
> 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

Remove this paragraph. This context only makes sense in this thread
where the previous patch version is visible.

> 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>

I'm not the one who should be attributed to by the Reported-by tag.

I think Greg was alluding to this report by Stephen Rothwell:
https://lore.kernel.org/all/20241118194006.77c7b126@canb.auug.org.au/

It wasn't Cc to the linux-usb list, so I'm not sure if you've seen it.
If your change was base on the report above, then you can update the
Reported-by tag accordingly.


> 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://urldefense.com/v3/__https://lore.kernel.org/all/20241119020807.cn7ugxnhbkqwrr2b@synopsys.com/__;!!A4F2R9G_pg!cbEnDmWN9_VzHUAotB8hMCEm9jnK0F1SwtvDH-Ts4uicWlPUif59FPkZWzCzRBcFH05bhdanwXhAPrydadc9ItOJWTpTSrs$ 
> [2] https://urldefense.com/v3/__https://lore.kernel.org/all/2024111922-pantyhose-panorama-6f16@gregkh/__;!!A4F2R9G_pg!cbEnDmWN9_VzHUAotB8hMCEm9jnK0F1SwtvDH-Ts4uicWlPUif59FPkZWzCzRBcFH05bhdanwXhAPrydadc9ItOJGBu8ksU$ 
> [3] https://urldefense.com/v3/__https://lore.kernel.org/all/5l65sdskdzbehxamff5ax4ptiqhaxh7ewi4umtpp6ynen45nj6@nebuxjg4c4rx/__;!!A4F2R9G_pg!cbEnDmWN9_VzHUAotB8hMCEm9jnK0F1SwtvDH-Ts4uicWlPUif59FPkZWzCzRBcFH05bhdanwXhAPrydadc9ItOJ5nkwD-I$ 
> ---
>  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
> 

The rest looks good.

Thanks,
Thinh