On Mon, Mar 27, 2023 at 06:12:20PM +0800, Linyu Yuan wrote: > show ffs->dev_name in all possible debug message. > > Signed-off-by: Linyu Yuan <quic_linyyuan@xxxxxxxxxxx> > --- > v3: change according comments > v2: split to several changes according to v1 comments > v1: https://lore.kernel.org/linux-usb/1679481369-30094-1-git-send-email-quic_linyyuan@xxxxxxxxxxx/ > > drivers/usb/gadget/function/f_fs.c | 141 ++++++++++++++++++++----------------- > 1 file changed, 75 insertions(+), 66 deletions(-) > > diff --git a/drivers/usb/gadget/function/f_fs.c b/drivers/usb/gadget/function/f_fs.c > index 0761eaa..383343d 100644 > --- a/drivers/usb/gadget/function/f_fs.c > +++ b/drivers/usb/gadget/function/f_fs.c > @@ -317,12 +317,12 @@ static int __ffs_ep0_queue_wait(struct ffs_data *ffs, char *data, size_t len) > static int __ffs_ep0_stall(struct ffs_data *ffs) > { > if (ffs->ev.can_stall) { > - dev_vdbg(ffs->dev, "ep0 stall\n"); > + dev_vdbg(ffs->dev, "%s: ep0 stall\n", ffs->dev_name); You already have the name here, it's in the usb-gadget structure, why do you need to print it out again? What is the before and after output of this change? I think it should have the same information already in it. thanks, greg k-h
