Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1555923 > unrolled thread
| Started by | Shuah Khan <shuahkh@osg.samsung.com> |
|---|---|
| First post | 2017-01-10 22:30 +0100 |
| Last post | 2017-01-11 01:30 +0100 |
| Articles | 3 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH] usb: dwc3-exynos remove suspend clock unspecified debug message Shuah Khan <shuahkh@osg.samsung.com> - 2017-01-10 22:30 +0100
Re: [PATCH] usb: dwc3-exynos remove suspend clock unspecified debug message Javier Martinez Canillas <javier@osg.samsung.com> - 2017-01-11 01:20 +0100
Re: [PATCH] usb: dwc3-exynos remove suspend clock unspecified debug message Shuah Khan <shuahkh@osg.samsung.com> - 2017-01-11 01:30 +0100
| From | Shuah Khan <shuahkh@osg.samsung.com> |
|---|---|
| Date | 2017-01-10 22:30 +0100 |
| Subject | [PATCH] usb: dwc3-exynos remove suspend clock unspecified debug message |
| Message-ID | <sYc9s-6qU-37@gated-at.bofh.it> |
dwc3-exynos prints debug message when suspend clock is not specified.
The suspend clock is optional and driver can work without it.
This debug message doesn't add any value and leads to confusion and
concern. Remove it.
Signed-off-by: Shuah Khan <shuahkh@osg.samsung.com>
---
This patch is a result of the disussion on the following patch.
https://lkml.org/lkml/2017/1/9/891
drivers/usb/dwc3/dwc3-exynos.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/usb/dwc3/dwc3-exynos.c b/drivers/usb/dwc3/dwc3-exynos.c
index e27899b..3e8407a 100644
--- a/drivers/usb/dwc3/dwc3-exynos.c
+++ b/drivers/usb/dwc3/dwc3-exynos.c
@@ -128,10 +128,8 @@ static int dwc3_exynos_probe(struct platform_device *pdev)
clk_prepare_enable(exynos->clk);
exynos->susp_clk = devm_clk_get(dev, "usbdrd30_susp_clk");
- if (IS_ERR(exynos->susp_clk)) {
- dev_info(dev, "no suspend clk specified\n");
+ if (IS_ERR(exynos->susp_clk))
exynos->susp_clk = NULL;
- }
clk_prepare_enable(exynos->susp_clk);
if (of_device_is_compatible(node, "samsung,exynos7-dwusb3")) {
--
2.7.4
[toc] | [next] | [standalone]
| From | Javier Martinez Canillas <javier@osg.samsung.com> |
|---|---|
| Date | 2017-01-11 01:20 +0100 |
| Subject | Re: [PATCH] usb: dwc3-exynos remove suspend clock unspecified debug message |
| Message-ID | <sYeNX-8br-9@gated-at.bofh.it> |
| In reply to | #1555923 |
Hello Shuah,
On 01/10/2017 06:20 PM, Shuah Khan wrote:
> dwc3-exynos prints debug message when suspend clock is not specified.
> The suspend clock is optional and driver can work without it.
>
> This debug message doesn't add any value and leads to confusion and
> concern. Remove it.
>
> Signed-off-by: Shuah Khan <shuahkh@osg.samsung.com>
> ---
> This patch is a result of the disussion on the following patch.
> https://lkml.org/lkml/2017/1/9/891
>
> drivers/usb/dwc3/dwc3-exynos.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/usb/dwc3/dwc3-exynos.c b/drivers/usb/dwc3/dwc3-exynos.c
> index e27899b..3e8407a 100644
> --- a/drivers/usb/dwc3/dwc3-exynos.c
> +++ b/drivers/usb/dwc3/dwc3-exynos.c
> @@ -128,10 +128,8 @@ static int dwc3_exynos_probe(struct platform_device *pdev)
> clk_prepare_enable(exynos->clk);
>
> exynos->susp_clk = devm_clk_get(dev, "usbdrd30_susp_clk");
> - if (IS_ERR(exynos->susp_clk)) {
> - dev_info(dev, "no suspend clk specified\n");
Sorry for being late in the discussion, but what about making it a
dev_dbg() instead? That way the message may not confuse others but
still keep information that could be useful for debugging purposes.
I know the current message isn't great but keeping it as debug allows
to improve it later.
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
[toc] | [prev] | [next] | [standalone]
| From | Shuah Khan <shuahkh@osg.samsung.com> |
|---|---|
| Date | 2017-01-11 01:30 +0100 |
| Subject | Re: [PATCH] usb: dwc3-exynos remove suspend clock unspecified debug message |
| Message-ID | <sYeXD-8ep-1@gated-at.bofh.it> |
| In reply to | #1556067 |
On 01/10/2017 05:18 PM, Javier Martinez Canillas wrote:
> Hello Shuah,
>
> On 01/10/2017 06:20 PM, Shuah Khan wrote:
>> dwc3-exynos prints debug message when suspend clock is not specified.
>> The suspend clock is optional and driver can work without it.
>>
>> This debug message doesn't add any value and leads to confusion and
>> concern. Remove it.
>>
>> Signed-off-by: Shuah Khan <shuahkh@osg.samsung.com>
>> ---
>> This patch is a result of the disussion on the following patch.
>> https://lkml.org/lkml/2017/1/9/891
>>
>> drivers/usb/dwc3/dwc3-exynos.c | 4 +---
>> 1 file changed, 1 insertion(+), 3 deletions(-)
>>
>> diff --git a/drivers/usb/dwc3/dwc3-exynos.c b/drivers/usb/dwc3/dwc3-exynos.c
>> index e27899b..3e8407a 100644
>> --- a/drivers/usb/dwc3/dwc3-exynos.c
>> +++ b/drivers/usb/dwc3/dwc3-exynos.c
>> @@ -128,10 +128,8 @@ static int dwc3_exynos_probe(struct platform_device *pdev)
>> clk_prepare_enable(exynos->clk);
>>
>> exynos->susp_clk = devm_clk_get(dev, "usbdrd30_susp_clk");
>> - if (IS_ERR(exynos->susp_clk)) {
>> - dev_info(dev, "no suspend clk specified\n");
>
> Sorry for being late in the discussion, but what about making it a
> dev_dbg() instead? That way the message may not confuse others but
> still keep information that could be useful for debugging purposes.
>
> I know the current message isn't great but keeping it as debug allows
> to improve it later.
>
> Best regards,
>
I proposed dev debug and the recommendation from Krzysztof was to
just remove it. Please see response from Krzysztof's response.
thanks,
-- Shuah
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web