From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Sent: Tuesday, October 5, 2021 7:12 PM
To: Linyu Yuan (QUIC) <redacted>
Cc: Felipe Balbi <balbi@kernel.org>; linux-usb@vger.kernel.org
Subject: Re: [PATCH v5 2/3] usb: gadget: configfs: add gadget_info for config
group

On Tue, Sep 07, 2021 at 09:09:36AM +0800, Linyu Yuan wrote:

quoted

add gadget_info pointer in struct config_usb_cfg
to allow common gadget info trace from configfs layer.

Again, I do not understand this description, can you please try to
reword it?

quoted

Signed-off-by: Linyu Yuan <redacted>
---
v2: no change
v3: change struct inside configfs.c
v4: no change
v5: no change

 drivers/usb/gadget/configfs.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/usb/gadget/configfs.c b/drivers/usb/gadget/configfs.c
index 5b2e6f9..cea12c3 100644
--- a/drivers/usb/gadget/configfs.c
+++ b/drivers/usb/gadget/configfs.c

@@ -70,6 +70,7 @@ struct config_usb_cfg {
 	struct usb_configuration c;
 	struct list_head func_list;
 	struct usb_gadget_strings *gstrings[MAX_USB_STRING_LANGS + 1];
+	struct gadget_info *gi;

You add a pointer, where are you removing it?

Currently configfs layer seem do nothing when configfs drop of config group...
(it assume the configuration group will not remove)
Need to test rmdir operation  and send patch if possible later.

thanks,

greg k-h