Re: [PATCH v2] usb: gadget: Address review comments and include missing files

[Greg KH <gregkh@linuxfoundation.org>]

On Fri, Jul 25, 2025 at 11:09:26PM +0200, Olivier Tuchon wrote:
> This patch addresses feedback on v1 from greg k-h and Alan Stern. It
> also includes files that were missing from the initial submission.
> 
> The following changes have been made:
>   - correct initial coding style issue (spaces for tabs)
>   - add missing file required for the feature to build and run
>   - improve FIFO full handling by truncating event data instead of
>     dropping the event entirely
>   - avoid capturing redundant data for IN submissions

This is not a proper way to write a subject line or comments at all.

Look at patches on the list and in the tree, nothing starts with
"address review comments", right?

Please go back and start again...


> 
> Signed-off-by: Olivier Tuchon <tcn@google.com>
> ---
>  drivers/usb/Kconfig                     |  2 +
>  drivers/usb/Makefile                    |  1 +
>  drivers/usb/gadget/udc/core.c           | 78 +++++++++++++++++++++++++
>  drivers/usb/gadget_mon/Kconfig          | 20 +++----
>  drivers/usb/gadget_mon/gadgetmon_main.c | 41 ++++++++-----
>  include/linux/usb/gadget.h              | 23 ++++++++
>  6 files changed, 141 insertions(+), 24 deletions(-)

Please read the in-kernel documentation about how to submit a kernel
patch, it should explain all of this, right?  If not, please work with
someone in your group that knows how to create kernel patches, don't
require the community to do that kind of basic review/training for you.

thanks,

greg k-h