* [PATCH][next] usb: gadget: f_uac2: Fix spelling mistake "maxpctksize" -> "maxpcktsize" @ 2022-02-02 9:19 Colin Ian King 2022-02-02 9:57 ` Joe Perches 0 siblings, 1 reply; 3+ messages in thread From: Colin Ian King @ 2022-02-02 9:19 UTC (permalink / raw) To: Felipe Balbi, Greg Kroah-Hartman, linux-usb; +Cc: kernel-janitors, linux-kernel There is a spelling mistake in a deb_dbg message. Fix it. Signed-off-by: Colin Ian King <colin.i.king@gmail.com> --- drivers/usb/gadget/function/f_uac2.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/gadget/function/f_uac2.c b/drivers/usb/gadget/function/f_uac2.c index f2237bcdba7c..c9b8d11f9870 100644 --- a/drivers/usb/gadget/function/f_uac2.c +++ b/drivers/usb/gadget/function/f_uac2.c @@ -755,7 +755,7 @@ static int set_ep_max_packet_size_bint(struct device *dev, const struct f_uac2_o if (max_size_bw <= max_size_ep) dev_dbg(dev, - "%s %s: Would use maxpctksize %d and bInterval %d\n", + "%s %s: Would use maxpcktsize %d and bInterval %d\n", speed_names[speed], dir, max_size_bw, bint); else { dev_warn(dev, -- 2.34.1 ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH][next] usb: gadget: f_uac2: Fix spelling mistake "maxpctksize" -> "maxpcktsize" 2022-02-02 9:19 [PATCH][next] usb: gadget: f_uac2: Fix spelling mistake "maxpctksize" -> "maxpcktsize" Colin Ian King @ 2022-02-02 9:57 ` Joe Perches 2022-02-02 10:31 ` Pavel Hofman 0 siblings, 1 reply; 3+ messages in thread From: Joe Perches @ 2022-02-02 9:57 UTC (permalink / raw) To: Colin Ian King, Felipe Balbi, Greg Kroah-Hartman, linux-usb Cc: kernel-janitors, linux-kernel On Wed, 2022-02-02 at 09:19 +0000, Colin Ian King wrote: > There is a spelling mistake in a deb_dbg message. Fix it. [] > diff --git a/drivers/usb/gadget/function/f_uac2.c b/drivers/usb/gadget/function/f_uac2.c [] > @@ -755,7 +755,7 @@ static int set_ep_max_packet_size_bint(struct device *dev, const struct f_uac2_o > > if (max_size_bw <= max_size_ep) > dev_dbg(dev, > - "%s %s: Would use maxpctksize %d and bInterval %d\n", > + "%s %s: Would use maxpcktsize %d and bInterval %d\n", why not just spell it out? or use wMaxPacketSize from the uapi include? --- drivers/usb/gadget/function/f_uac2.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/usb/gadget/function/f_uac2.c b/drivers/usb/gadget/function/f_uac2.c index f2237bcdba7c0..13b59128121a2 100644 --- a/drivers/usb/gadget/function/f_uac2.c +++ b/drivers/usb/gadget/function/f_uac2.c @@ -755,12 +755,12 @@ static int set_ep_max_packet_size_bint(struct device *dev, const struct f_uac2_o if (max_size_bw <= max_size_ep) dev_dbg(dev, - "%s %s: Would use maxpctksize %d and bInterval %d\n", + "%s %s: Would use wMaxPacketSize %d and bInterval %d\n", speed_names[speed], dir, max_size_bw, bint); else { dev_warn(dev, - "%s %s: Req. maxpcktsize %d at bInterval %d > max ISOC %d, may drop data!\n", - speed_names[speed], dir, max_size_bw, bint, max_size_ep); + "%s %s: Req. wMaxPacketSize %d at bInterval %d > max ISOC %d, may drop data!\n", + speed_names[speed], dir, max_size_bw, bint, max_size_ep); max_size_bw = max_size_ep; } ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH][next] usb: gadget: f_uac2: Fix spelling mistake "maxpctksize" -> "maxpcktsize" 2022-02-02 9:57 ` Joe Perches @ 2022-02-02 10:31 ` Pavel Hofman 0 siblings, 0 replies; 3+ messages in thread From: Pavel Hofman @ 2022-02-02 10:31 UTC (permalink / raw) To: Joe Perches, Colin Ian King, Felipe Balbi, Greg Kroah-Hartman, linux-usb Cc: kernel-janitors, linux-kernel Dne 02. 02. 22 v 10:57 Joe Perches napsal(a): > On Wed, 2022-02-02 at 09:19 +0000, Colin Ian King wrote: >> There is a spelling mistake in a deb_dbg message. Fix it. > [] >> diff --git a/drivers/usb/gadget/function/f_uac2.c b/drivers/usb/gadget/function/f_uac2.c > [] >> @@ -755,7 +755,7 @@ static int set_ep_max_packet_size_bint(struct device *dev, const struct f_uac2_o >> >> if (max_size_bw <= max_size_ep) >> dev_dbg(dev, >> - "%s %s: Would use maxpctksize %d and bInterval %d\n", >> + "%s %s: Would use maxpcktsize %d and bInterval %d\n", > > why not just spell it out? or use wMaxPacketSize from the uapi include? My reason for the ugly abbreviation (prone to misspelling for which I apologize) was to keep the line length limit. I would be happy to see wMaxPacketSize instead :-) Pavel. > --- > drivers/usb/gadget/function/f_uac2.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/usb/gadget/function/f_uac2.c b/drivers/usb/gadget/function/f_uac2.c > index f2237bcdba7c0..13b59128121a2 100644 > --- a/drivers/usb/gadget/function/f_uac2.c > +++ b/drivers/usb/gadget/function/f_uac2.c > @@ -755,12 +755,12 @@ static int set_ep_max_packet_size_bint(struct device *dev, const struct f_uac2_o > > if (max_size_bw <= max_size_ep) > dev_dbg(dev, > - "%s %s: Would use maxpctksize %d and bInterval %d\n", > + "%s %s: Would use wMaxPacketSize %d and bInterval %d\n", > speed_names[speed], dir, max_size_bw, bint); > else { > dev_warn(dev, > - "%s %s: Req. maxpcktsize %d at bInterval %d > max ISOC %d, may drop data!\n", > - speed_names[speed], dir, max_size_bw, bint, max_size_ep); > + "%s %s: Req. wMaxPacketSize %d at bInterval %d > max ISOC %d, may drop data!\n", > + speed_names[speed], dir, max_size_bw, bint, max_size_ep); > max_size_bw = max_size_ep; > } > > > ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-02-02 10:31 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-02-02 9:19 [PATCH][next] usb: gadget: f_uac2: Fix spelling mistake "maxpctksize" -> "maxpcktsize" Colin Ian King 2022-02-02 9:57 ` Joe Perches 2022-02-02 10:31 ` Pavel Hofman
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®