2013-09-10 22:00:03

by Jon Bernard

Subject: [PATCH] Staging: fwserial: wrap a line that exceeds 80 characters

This is a patch to fwserial.c that wraps a line which previously exceeded the 80
character limit warning found by checkpatch.pl. This driver is now warning and
error free, according to checkpatch.pl

Signed-off-by: Jon Bernard <[email protected]>
---
drivers/staging/fwserial/fwserial.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/fwserial/fwserial.c b/drivers/staging/fwserial/fwserial.c
index ff92f34..62df009 100644
--- a/drivers/staging/fwserial/fwserial.c
+++ b/drivers/staging/fwserial/fwserial.c
@@ -2394,7 +2394,8 @@ static int fwserial_create(struct fw_unit *unit)

list_del_rcu(&serial->list);
if (create_loop_dev)
- tty_unregister_device(fwloop_driver, loop_idx(serial->ports[j]));
+ tty_unregister_device(fwloop_driver,
+ loop_idx(serial->ports[j]));
unregister_ttys:
for (--j; j >= 0; --j)
tty_unregister_device(fwtty_driver, serial->ports[j]->index);
--
1.8.4

2013-09-11 23:19:00

by Peter Hurley

Subject: Re: [PATCH] Staging: fwserial: wrap a line that exceeds 80 characters

On 09/10/2013 06:00 PM, Jon Bernard wrote:
> This is a patch to fwserial.c that wraps a line which previously exceeded the 80
> character limit warning found by checkpatch.pl. This driver is now warning and
> error free, according to checkpatch.pl
>
> Signed-off-by: Jon Bernard <[email protected]>

Ok with me, thanks.

Reviewed-by: Peter Hurley <[email protected]>