* [PATCH octeon-usb] fix 'line over 80 characters'
@ 2015-03-18 14:42 Amitoj Kaur Chawla
2015-03-18 14:53 ` Greg KH
2015-03-18 18:17 ` Aaro Koskinen
0 siblings, 2 replies; 3+ messages in thread
From: Amitoj Kaur Chawla @ 2015-03-18 14:42 UTC (permalink / raw)
To: devel, linux-kernel
This file contained a warning of a line being over 80 characters and
so the file has been edited to remove that warning.
Signed-off-by: Amitoj Kaur Chawla <amitoj1606@gmail.com>
---
drivers/staging/octeon-usb/octeon-hcd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/staging/octeon-usb/octeon-hcd.c
index 1daeb31..c8df0c3 100644
--- a/drivers/staging/octeon-usb/octeon-hcd.c
+++ b/drivers/staging/octeon-usb/octeon-hcd.c
@@ -2884,7 +2884,7 @@ static int __cvmx_usb_poll_channel(struct cvmx_usb_state *usb, int channel)
pipe->pid_toggle = 1;
if (__cvmx_usb_pipe_needs_split(usb, pipe))
transaction->stage =
- CVMX_USB_STAGE_SETUP_SPLIT_COMPLETE;
+ CVMX_USB_STAGE_SETUP_SPLIT_COMPLETE;
else {
struct usb_ctrlrequest *header =
cvmx_phys_to_ptr(transaction->control_header);
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH octeon-usb] fix 'line over 80 characters'
2015-03-18 14:42 [PATCH octeon-usb] fix 'line over 80 characters' Amitoj Kaur Chawla
@ 2015-03-18 14:53 ` Greg KH
2015-03-18 18:17 ` Aaro Koskinen
1 sibling, 0 replies; 3+ messages in thread
From: Greg KH @ 2015-03-18 14:53 UTC (permalink / raw)
To: Amitoj Kaur Chawla; +Cc: devel, linux-kernel
On Wed, Mar 18, 2015 at 08:12:32PM +0530, Amitoj Kaur Chawla wrote:
>
> This file contained a warning of a line being over 80 characters and
> so the file has been edited to remove that warning.
>
> Signed-off-by: Amitoj Kaur Chawla <amitoj1606@gmail.com>
Why did you put the driver name within the []? It needs to go outside
of it.
> ---
> drivers/staging/octeon-usb/octeon-hcd.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/staging/octeon-usb/octeon-hcd.c
> index 1daeb31..c8df0c3 100644
> --- a/drivers/staging/octeon-usb/octeon-hcd.c
> +++ b/drivers/staging/octeon-usb/octeon-hcd.c
> @@ -2884,7 +2884,7 @@ static int __cvmx_usb_poll_channel(struct cvmx_usb_state *usb, int channel)
> pipe->pid_toggle = 1;
> if (__cvmx_usb_pipe_needs_split(usb, pipe))
> transaction->stage =
> - CVMX_USB_STAGE_SETUP_SPLIT_COMPLETE;
> + CVMX_USB_STAGE_SETUP_SPLIT_COMPLETE;
This really doesn't help much, the original code is fine, there are
better things to fix in this file :)
thanks,
greg k-h
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH octeon-usb] fix 'line over 80 characters'
2015-03-18 14:42 [PATCH octeon-usb] fix 'line over 80 characters' Amitoj Kaur Chawla
2015-03-18 14:53 ` Greg KH
@ 2015-03-18 18:17 ` Aaro Koskinen
1 sibling, 0 replies; 3+ messages in thread
From: Aaro Koskinen @ 2015-03-18 18:17 UTC (permalink / raw)
To: Amitoj Kaur Chawla; +Cc: devel, linux-kernel
Hi,
On Wed, Mar 18, 2015 at 08:12:32PM +0530, Amitoj Kaur Chawla wrote:
> This file contained a warning of a line being over 80 characters and
> so the file has been edited to remove that warning.
>
> Signed-off-by: Amitoj Kaur Chawla <amitoj1606@gmail.com>
> ---
> drivers/staging/octeon-usb/octeon-hcd.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/staging/octeon-usb/octeon-hcd.c
> index 1daeb31..c8df0c3 100644
> --- a/drivers/staging/octeon-usb/octeon-hcd.c
> +++ b/drivers/staging/octeon-usb/octeon-hcd.c
> @@ -2884,7 +2884,7 @@ static int __cvmx_usb_poll_channel(struct cvmx_usb_state *usb, int channel)
> pipe->pid_toggle = 1;
> if (__cvmx_usb_pipe_needs_split(usb, pipe))
> transaction->stage =
> - CVMX_USB_STAGE_SETUP_SPLIT_COMPLETE;
> + CVMX_USB_STAGE_SETUP_SPLIT_COMPLETE;
This is not an improvement. The proper fix is to refactor the code
to avoid deep nesting.
A.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-03-18 18:18 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-03-18 14:42 [PATCH octeon-usb] fix 'line over 80 characters' Amitoj Kaur Chawla
2015-03-18 14:53 ` Greg KH
2015-03-18 18:17 ` Aaro Koskinen
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.