* [PATCH] staging: winbond: use dev_err() instead of printk()
@ 2014-05-08 11:38 Martin Kepplinger
2014-05-08 11:48 ` Dan Carpenter
2014-05-09 8:21 ` Pavel Machek
0 siblings, 2 replies; 5+ messages in thread
From: Martin Kepplinger @ 2014-05-08 11:38 UTC (permalink / raw)
To: gregkh; +Cc: pavel, devel, linux-kernel, Martin Kepplinger
For obvious error messages, use dev_err() in order to provide userspace
with more useful information and use the common kernel coding style.
Signed-off-by: Martin Kepplinger <martink@posteo.de>
---
this applies to v3.15-rc4
greetings from Linuxdays Vienna, Austria
drivers/staging/winbond/wb35tx.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/winbond/wb35tx.c b/drivers/staging/winbond/wb35tx.c
index 708c5b0..870cff3 100644
--- a/drivers/staging/winbond/wb35tx.c
+++ b/drivers/staging/winbond/wb35tx.c
@@ -49,7 +49,7 @@ static void Wb35Tx_complete(struct urb *pUrb)
/* The URB is completed, check the result */
if (pWb35Tx->EP4VM_status != 0) {
- printk("URB submission failed\n");
+ dev_err(&pUrb->dev->dev, "URB submission failed\n");
pWb35Tx->EP4vm_state = VM_STOP;
goto error;
}
@@ -96,7 +96,7 @@ static void Wb35Tx(struct wbsoft_priv *adapter)
pWb35Tx->EP4vm_state = VM_RUNNING;
retv = usb_submit_urb(pUrb, GFP_ATOMIC);
if (retv < 0) {
- printk("EP4 Tx Irp sending error\n");
+ dev_err(&pUrb->dev->dev, "EP4 Tx Irp sending error\n");
goto cleanup;
}
@@ -218,7 +218,7 @@ static void Wb35Tx_EP2VM_complete(struct urb *pUrb)
/* The Urb is completed, check the result */
if (pWb35Tx->EP2VM_status != 0) {
- printk("EP2 IoCompleteRoutine return error\n");
+ dev_err(&pUrb->dev->dev, "EP2 IoCompleteRoutine return error\n");
pWb35Tx->EP2vm_state = VM_STOP;
goto error;
}
--
1.7.10.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] staging: winbond: use dev_err() instead of printk()
2014-05-08 11:38 [PATCH] staging: winbond: use dev_err() instead of printk() Martin Kepplinger
@ 2014-05-08 11:48 ` Dan Carpenter
2014-05-09 8:21 ` Pavel Machek
1 sibling, 0 replies; 5+ messages in thread
From: Dan Carpenter @ 2014-05-08 11:48 UTC (permalink / raw)
To: Martin Kepplinger; +Cc: gregkh, devel, linux-kernel, pavel
On Thu, May 08, 2014 at 01:38:33PM +0200, Martin Kepplinger wrote:
> For obvious error messages, use dev_err() in order to provide userspace
> with more useful information and use the common kernel coding style.
>
> Signed-off-by: Martin Kepplinger <martink@posteo.de>
> ---
> this applies to v3.15-rc4
>
Do these kinds of patches against linux-next. This patch applies fine
so it's not a worry.
regards,
dan carpenter
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] staging: winbond: use dev_err() instead of printk()
2014-05-08 11:38 [PATCH] staging: winbond: use dev_err() instead of printk() Martin Kepplinger
2014-05-08 11:48 ` Dan Carpenter
@ 2014-05-09 8:21 ` Pavel Machek
2014-05-19 10:55 ` [PATCH][RESEND] " Martin Kepplinger
1 sibling, 1 reply; 5+ messages in thread
From: Pavel Machek @ 2014-05-09 8:21 UTC (permalink / raw)
To: Martin Kepplinger; +Cc: gregkh, devel, linux-kernel
On Thu 2014-05-08 13:38:33, Martin Kepplinger wrote:
> For obvious error messages, use dev_err() in order to provide userspace
> with more useful information and use the common kernel coding style.
>
Acked-by: Pavel Machek <pavel@ucw.cz>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH][RESEND] staging: winbond: use dev_err() instead of printk()
2014-05-09 8:21 ` Pavel Machek
@ 2014-05-19 10:55 ` Martin Kepplinger
2014-05-19 10:59 ` Dan Carpenter
0 siblings, 1 reply; 5+ messages in thread
From: Martin Kepplinger @ 2014-05-19 10:55 UTC (permalink / raw)
To: gregkh; +Cc: pavel, dan.carpenter, devel, linux-kernel, Martin Kepplinger
For obvious error messages, use dev_err() in order to provide userspace
with more useful information and use the common kernel coding style.
Signed-off-by: Martin Kepplinger <martink@posteo.de>
---
i just waited a week or so. this applies to next-20140516.
greetings from Linuxdays Vienna, Austria
drivers/staging/winbond/wb35tx.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/winbond/wb35tx.c b/drivers/staging/winbond/wb35tx.c
index 708c5b0..870cff3 100644
--- a/drivers/staging/winbond/wb35tx.c
+++ b/drivers/staging/winbond/wb35tx.c
@@ -49,7 +49,7 @@ static void Wb35Tx_complete(struct urb *pUrb)
/* The URB is completed, check the result */
if (pWb35Tx->EP4VM_status != 0) {
- printk("URB submission failed\n");
+ dev_err(&pUrb->dev->dev, "URB submission failed\n");
pWb35Tx->EP4vm_state = VM_STOP;
goto error;
}
@@ -96,7 +96,7 @@ static void Wb35Tx(struct wbsoft_priv *adapter)
pWb35Tx->EP4vm_state = VM_RUNNING;
retv = usb_submit_urb(pUrb, GFP_ATOMIC);
if (retv < 0) {
- printk("EP4 Tx Irp sending error\n");
+ dev_err(&pUrb->dev->dev, "EP4 Tx Irp sending error\n");
goto cleanup;
}
@@ -218,7 +218,7 @@ static void Wb35Tx_EP2VM_complete(struct urb *pUrb)
/* The Urb is completed, check the result */
if (pWb35Tx->EP2VM_status != 0) {
- printk("EP2 IoCompleteRoutine return error\n");
+ dev_err(&pUrb->dev->dev, "EP2 IoCompleteRoutine return error\n");
pWb35Tx->EP2vm_state = VM_STOP;
goto error;
}
--
1.7.10.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH][RESEND] staging: winbond: use dev_err() instead of printk()
2014-05-19 10:55 ` [PATCH][RESEND] " Martin Kepplinger
@ 2014-05-19 10:59 ` Dan Carpenter
0 siblings, 0 replies; 5+ messages in thread
From: Dan Carpenter @ 2014-05-19 10:59 UTC (permalink / raw)
To: Martin Kepplinger; +Cc: gregkh, pavel, devel, linux-kernel
On Mon, May 19, 2014 at 12:55:11PM +0200, Martin Kepplinger wrote:
> For obvious error messages, use dev_err() in order to provide userspace
> with more useful information and use the common kernel coding style.
>
> Signed-off-by: Martin Kepplinger <martink@posteo.de>
> ---
> i just waited a week or so. this applies to next-20140516.
A week is too short a time. You will always get a message that your
patch was either applied or dropped.
>
> greetings from Linuxdays Vienna, Austria
Hi hi. Greetings from Africa. :)
regards,
dan carpenter
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-05-19 11:00 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-05-08 11:38 [PATCH] staging: winbond: use dev_err() instead of printk() Martin Kepplinger
2014-05-08 11:48 ` Dan Carpenter
2014-05-09 8:21 ` Pavel Machek
2014-05-19 10:55 ` [PATCH][RESEND] " Martin Kepplinger
2014-05-19 10:59 ` Dan Carpenter
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).