* [PATCH][next] usb: cdnsp: Fix spelling mistake "delagete" -> "delegate"
@ 2021-02-03 11:12 Colin King
2021-02-03 13:06 ` Dan Carpenter
0 siblings, 1 reply; 7+ messages in thread
From: Colin King @ 2021-02-03 11:12 UTC (permalink / raw)
To: Pawel Laszczak, Greg Kroah-Hartman, linux-usb
Cc: kernel-janitors, linux-kernel
From: Colin Ian King <colin.king@canonical.com>
There is a spelling mistake in a literal string. Fix it.
Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
drivers/usb/cdns3/cdnsp-ep0.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/cdns3/cdnsp-ep0.c b/drivers/usb/cdns3/cdnsp-ep0.c
index e2b1bcb3f80e..e30931ebc870 100644
--- a/drivers/usb/cdns3/cdnsp-ep0.c
+++ b/drivers/usb/cdns3/cdnsp-ep0.c
@@ -45,7 +45,7 @@ static int cdnsp_ep0_delegate_req(struct cdnsp_device *pdev,
{
int ret;
- trace_cdnsp_ep0_request("delagete");
+ trace_cdnsp_ep0_request("delegate");
spin_unlock(&pdev->lock);
ret = pdev->gadget_driver->setup(&pdev->gadget, ctrl);
--
2.29.2
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH][next] usb: cdnsp: Fix spelling mistake "delagete" -> "delegate"
2021-02-03 11:12 [PATCH][next] usb: cdnsp: Fix spelling mistake "delagete" -> "delegate" Colin King
@ 2021-02-03 13:06 ` Dan Carpenter
2021-02-04 5:07 ` Pawel Laszczak
0 siblings, 1 reply; 7+ messages in thread
From: Dan Carpenter @ 2021-02-03 13:06 UTC (permalink / raw)
To: Colin King
Cc: Pawel Laszczak, Greg Kroah-Hartman, linux-usb, kernel-janitors,
linux-kernel
On Wed, Feb 03, 2021 at 11:12:39AM +0000, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> There is a spelling mistake in a literal string. Fix it.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
> drivers/usb/cdns3/cdnsp-ep0.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/usb/cdns3/cdnsp-ep0.c b/drivers/usb/cdns3/cdnsp-ep0.c
> index e2b1bcb3f80e..e30931ebc870 100644
> --- a/drivers/usb/cdns3/cdnsp-ep0.c
> +++ b/drivers/usb/cdns3/cdnsp-ep0.c
> @@ -45,7 +45,7 @@ static int cdnsp_ep0_delegate_req(struct cdnsp_device *pdev,
> {
> int ret;
>
> - trace_cdnsp_ep0_request("delagete");
> + trace_cdnsp_ep0_request("delegate");
>
This printk is useless and should just be deleted. Use ftrace instead.
regards,
dan carpenter
^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [PATCH][next] usb: cdnsp: Fix spelling mistake "delagete" -> "delegate"
2021-02-03 13:06 ` Dan Carpenter
@ 2021-02-04 5:07 ` Pawel Laszczak
2021-02-04 6:30 ` Greg Kroah-Hartman
0 siblings, 1 reply; 7+ messages in thread
From: Pawel Laszczak @ 2021-02-04 5:07 UTC (permalink / raw)
To: Dan Carpenter, Colin King
Cc: Greg Kroah-Hartman, linux-usb, kernel-janitors, linux-kernel
Hi Dan,
>> From: Colin Ian King <colin.king@canonical.com>
>>
>> There is a spelling mistake in a literal string. Fix it.
>>
>> Signed-off-by: Colin Ian King <colin.king@canonical.com>
>> ---
>> drivers/usb/cdns3/cdnsp-ep0.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/usb/cdns3/cdnsp-ep0.c b/drivers/usb/cdns3/cdnsp-ep0.c
>> index e2b1bcb3f80e..e30931ebc870 100644
>> --- a/drivers/usb/cdns3/cdnsp-ep0.c
>> +++ b/drivers/usb/cdns3/cdnsp-ep0.c
>> @@ -45,7 +45,7 @@ static int cdnsp_ep0_delegate_req(struct cdnsp_device *pdev,
>> {
>> int ret;
>>
>> - trace_cdnsp_ep0_request("delagete");
>> + trace_cdnsp_ep0_request("delegate");
>>
>
>This printk is useless and should just be deleted. Use ftrace instead.
Maybe this printk is redundant but it's more comfortable in use.
To debug I can simply enable cdns-dev events (echo cdnsp-dev:* > set_event)
and I will get the full picture of what the driver is doing.
Otherwise, I must remember which function I need to add to set_ftrace_filter.
Of course, by default I can simply add all cdnsp* functions (echo cdnsp* > set_ftrace_filter) but it
increases the trace log and makes it a little more difficult to analyze.
So maybe in some cases we shouldn't complain for such printk ?
It's my private opinion and not necessarily correct :)
Thanks,
Pawel Laszczak
>
>regards,
>dan carpenter
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH][next] usb: cdnsp: Fix spelling mistake "delagete" -> "delegate"
2021-02-04 5:07 ` Pawel Laszczak
@ 2021-02-04 6:30 ` Greg Kroah-Hartman
2021-02-04 9:25 ` Pawel Laszczak
0 siblings, 1 reply; 7+ messages in thread
From: Greg Kroah-Hartman @ 2021-02-04 6:30 UTC (permalink / raw)
To: Pawel Laszczak
Cc: Dan Carpenter, Colin King, linux-usb, kernel-janitors, linux-kernel
On Thu, Feb 04, 2021 at 05:07:16AM +0000, Pawel Laszczak wrote:
> Hi Dan,
>
> >> From: Colin Ian King <colin.king@canonical.com>
> >>
> >> There is a spelling mistake in a literal string. Fix it.
> >>
> >> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> >> ---
> >> drivers/usb/cdns3/cdnsp-ep0.c | 2 +-
> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/drivers/usb/cdns3/cdnsp-ep0.c b/drivers/usb/cdns3/cdnsp-ep0.c
> >> index e2b1bcb3f80e..e30931ebc870 100644
> >> --- a/drivers/usb/cdns3/cdnsp-ep0.c
> >> +++ b/drivers/usb/cdns3/cdnsp-ep0.c
> >> @@ -45,7 +45,7 @@ static int cdnsp_ep0_delegate_req(struct cdnsp_device *pdev,
> >> {
> >> int ret;
> >>
> >> - trace_cdnsp_ep0_request("delagete");
> >> + trace_cdnsp_ep0_request("delegate");
> >>
> >
> >This printk is useless and should just be deleted. Use ftrace instead.
>
> Maybe this printk is redundant but it's more comfortable in use.
> To debug I can simply enable cdns-dev events (echo cdnsp-dev:* > set_event)
> and I will get the full picture of what the driver is doing.
>
> Otherwise, I must remember which function I need to add to set_ftrace_filter.
> Of course, by default I can simply add all cdnsp* functions (echo cdnsp* > set_ftrace_filter) but it
> increases the trace log and makes it a little more difficult to analyze.
>
> So maybe in some cases we shouldn't complain for such printk ?
>
> It's my private opinion and not necessarily correct :)
Please don't have duplicate tracepoints for something like "this
function is now called", it's redundant.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [PATCH][next] usb: cdnsp: Fix spelling mistake "delagete" -> "delegate"
2021-02-04 6:30 ` Greg Kroah-Hartman
@ 2021-02-04 9:25 ` Pawel Laszczak
2021-02-04 9:25 ` Colin Ian King
0 siblings, 1 reply; 7+ messages in thread
From: Pawel Laszczak @ 2021-02-04 9:25 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Dan Carpenter, Colin King, linux-usb, kernel-janitors, linux-kernel
I've sent the patch that remove this one and others similar printk from driver.
>
>
>On Thu, Feb 04, 2021 at 05:07:16AM +0000, Pawel Laszczak wrote:
>> Hi Dan,
>>
>> >> From: Colin Ian King <colin.king@canonical.com>
>> >>
>> >> There is a spelling mistake in a literal string. Fix it.
>> >>
>> >> Signed-off-by: Colin Ian King <colin.king@canonical.com>
>> >> ---
>> >> drivers/usb/cdns3/cdnsp-ep0.c | 2 +-
>> >> 1 file changed, 1 insertion(+), 1 deletion(-)
>> >>
>> >> diff --git a/drivers/usb/cdns3/cdnsp-ep0.c b/drivers/usb/cdns3/cdnsp-ep0.c
>> >> index e2b1bcb3f80e..e30931ebc870 100644
>> >> --- a/drivers/usb/cdns3/cdnsp-ep0.c
>> >> +++ b/drivers/usb/cdns3/cdnsp-ep0.c
>> >> @@ -45,7 +45,7 @@ static int cdnsp_ep0_delegate_req(struct cdnsp_device *pdev,
>> >> {
>> >> int ret;
>> >>
>> >> - trace_cdnsp_ep0_request("delagete");
>> >> + trace_cdnsp_ep0_request("delegate");
>> >>
>> >
>> >This printk is useless and should just be deleted. Use ftrace instead.
>>
>> Maybe this printk is redundant but it's more comfortable in use.
>> To debug I can simply enable cdns-dev events (echo cdnsp-dev:* > set_event)
>> and I will get the full picture of what the driver is doing.
>>
>> Otherwise, I must remember which function I need to add to set_ftrace_filter.
>> Of course, by default I can simply add all cdnsp* functions (echo cdnsp* > set_ftrace_filter) but it
>> increases the trace log and makes it a little more difficult to analyze.
>>
>> So maybe in some cases we shouldn't complain for such printk ?
>>
>> It's my private opinion and not necessarily correct :)
>
>Please don't have duplicate tracepoints for something like "this
>function is now called", it's redundant.
>
Thanks,
Pawel Laszczak
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH][next] usb: cdnsp: Fix spelling mistake "delagete" -> "delegate"
2021-02-04 9:25 ` Pawel Laszczak
@ 2021-02-04 9:25 ` Colin Ian King
0 siblings, 0 replies; 7+ messages in thread
From: Colin Ian King @ 2021-02-04 9:25 UTC (permalink / raw)
To: Pawel Laszczak, Greg Kroah-Hartman
Cc: Dan Carpenter, linux-usb, kernel-janitors, linux-kernel
On 04/02/2021 09:25, Pawel Laszczak wrote:
>
> I've sent the patch that remove this one and others similar printk from driver.
Thanks Pawel.
Colin
>
>>
>>
>> On Thu, Feb 04, 2021 at 05:07:16AM +0000, Pawel Laszczak wrote:
>>> Hi Dan,
>>>
>>>>> From: Colin Ian King <colin.king@canonical.com>
>>>>>
>>>>> There is a spelling mistake in a literal string. Fix it.
>>>>>
>>>>> Signed-off-by: Colin Ian King <colin.king@canonical.com>
>>>>> ---
>>>>> drivers/usb/cdns3/cdnsp-ep0.c | 2 +-
>>>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/drivers/usb/cdns3/cdnsp-ep0.c b/drivers/usb/cdns3/cdnsp-ep0.c
>>>>> index e2b1bcb3f80e..e30931ebc870 100644
>>>>> --- a/drivers/usb/cdns3/cdnsp-ep0.c
>>>>> +++ b/drivers/usb/cdns3/cdnsp-ep0.c
>>>>> @@ -45,7 +45,7 @@ static int cdnsp_ep0_delegate_req(struct cdnsp_device *pdev,
>>>>> {
>>>>> int ret;
>>>>>
>>>>> - trace_cdnsp_ep0_request("delagete");
>>>>> + trace_cdnsp_ep0_request("delegate");
>>>>>
>>>>
>>>> This printk is useless and should just be deleted. Use ftrace instead.
>>>
>>> Maybe this printk is redundant but it's more comfortable in use.
>>> To debug I can simply enable cdns-dev events (echo cdnsp-dev:* > set_event)
>>> and I will get the full picture of what the driver is doing.
>>>
>>> Otherwise, I must remember which function I need to add to set_ftrace_filter.
>>> Of course, by default I can simply add all cdnsp* functions (echo cdnsp* > set_ftrace_filter) but it
>>> increases the trace log and makes it a little more difficult to analyze.
>>>
>>> So maybe in some cases we shouldn't complain for such printk ?
>>>
>>> It's my private opinion and not necessarily correct :)
>>
>> Please don't have duplicate tracepoints for something like "this
>> function is now called", it's redundant.
>>
>
> Thanks,
> Pawel Laszczak
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH][next] usb: cdnsp: fix spelling mistake "delagete" -> "delegate"
@ 2020-12-14 22:28 Colin King
0 siblings, 0 replies; 7+ messages in thread
From: Colin King @ 2020-12-14 22:28 UTC (permalink / raw)
To: Pawel Laszczak, Greg Kroah-Hartman, linux-usb
Cc: kernel-janitors, linux-kernel
From: Colin Ian King <colin.king@canonical.com>
There is a spelling mistake in a trace message. Fix it.
Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
drivers/usb/cdns3/cdnsp-ep0.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/cdns3/cdnsp-ep0.c b/drivers/usb/cdns3/cdnsp-ep0.c
index d55b59ed7381..b6a06e6210ba 100644
--- a/drivers/usb/cdns3/cdnsp-ep0.c
+++ b/drivers/usb/cdns3/cdnsp-ep0.c
@@ -45,7 +45,7 @@ static int cdnsp_ep0_delegate_req(struct cdnsp_device *pdev,
{
int ret;
- trace_cdnsp_ep0_request("delagete");
+ trace_cdnsp_ep0_request("delegate");
spin_unlock(&pdev->lock);
ret = pdev->gadget_driver->setup(&pdev->gadget, ctrl);
--
2.29.2
^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-02-04 9:26 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-03 11:12 [PATCH][next] usb: cdnsp: Fix spelling mistake "delagete" -> "delegate" Colin King
2021-02-03 13:06 ` Dan Carpenter
2021-02-04 5:07 ` Pawel Laszczak
2021-02-04 6:30 ` Greg Kroah-Hartman
2021-02-04 9:25 ` Pawel Laszczak
2021-02-04 9:25 ` Colin Ian King
-- strict thread matches above, loose matches on Subject: below --
2020-12-14 22:28 [PATCH][next] usb: cdnsp: fix " Colin King
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).