* [PATCH 0/4] Staging: octeon-usb: Fixes and Coding style applied.
@ 2018-07-24 22:38 Georgios Tsotsos
2018-07-24 22:38 ` [PATCH 1/4] Staging: octeon-usb: Adds SPDX License Identifier Georgios Tsotsos
` (4 more replies)
0 siblings, 5 replies; 8+ messages in thread
From: Georgios Tsotsos @ 2018-07-24 22:38 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Georgios Tsotsos, Aaro Koskinen, James Hogan, devel, linux-kernel
Hello,
Previously patches were sent with wrong cover format, please ignore.
I am sending four patches which trying to resolve TODO list requirements
no 45 about octeon-usb.
There are SPDX licence additions on c and header files.
Checkpatch warnings are resolved,also a notice about CVMX_WAIT_FOR_FIELD32 macro.
It could improve readability and maintenance of this module if function
cvmx_usb_poll_channel() break down (and probably some others too)
Georgios Tsotsos (4):
Staging: octeon-usb: Adds SPDX License Identifier
Staging: octeon-usb: Applying coding style for column width.
Staging: octeon-usb: Change coding style for CVMX_WAIT_FOR_FIELD32 marco.
Staging: octeon-usb: Adds SPDX License Identifier for header file
drivers/staging/octeon-usb/octeon-hcd.c | 55 ++++++++++++++++++---------------
drivers/staging/octeon-usb/octeon-hcd.h | 1 +
2 files changed, 31 insertions(+), 25 deletions(-)
--
2.16.4
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 1/4] Staging: octeon-usb: Adds SPDX License Identifier
2018-07-24 22:38 [PATCH 0/4] Staging: octeon-usb: Fixes and Coding style applied Georgios Tsotsos
@ 2018-07-24 22:38 ` Georgios Tsotsos
2018-07-24 22:38 ` [PATCH 2/4] Staging: octeon-usb: Applying coding style for column width Georgios Tsotsos
` (3 subsequent siblings)
4 siblings, 0 replies; 8+ messages in thread
From: Georgios Tsotsos @ 2018-07-24 22:38 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Georgios Tsotsos, Aaro Koskinen, James Hogan, devel, linux-kernel
Signed-off-by: Georgios Tsotsos <tsotsos@gmail.com>
---
drivers/staging/octeon-usb/octeon-hcd.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/staging/octeon-usb/octeon-hcd.c
index cded30f145aa..cff5e790b196 100644
--- a/drivers/staging/octeon-usb/octeon-hcd.c
+++ b/drivers/staging/octeon-usb/octeon-hcd.c
@@ -1,3 +1,4 @@
+// SPDX-License-Identifier: GPL-2.0
/*
* This file is subject to the terms and conditions of the GNU General Public
* License. See the file "COPYING" in the main directory of this archive
--
2.16.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 2/4] Staging: octeon-usb: Applying coding style for column width.
2018-07-24 22:38 [PATCH 0/4] Staging: octeon-usb: Fixes and Coding style applied Georgios Tsotsos
2018-07-24 22:38 ` [PATCH 1/4] Staging: octeon-usb: Adds SPDX License Identifier Georgios Tsotsos
@ 2018-07-24 22:38 ` Georgios Tsotsos
2018-07-24 22:38 ` [PATCH 3/4] Staging: octeon-usb: Change coding style for CVMX_WAIT_FOR_FIELD32 marco Georgios Tsotsos
` (2 subsequent siblings)
4 siblings, 0 replies; 8+ messages in thread
From: Georgios Tsotsos @ 2018-07-24 22:38 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Georgios Tsotsos, Aaro Koskinen, James Hogan, devel, linux-kernel
This patch resolves the warnings as its refereed to TODO list.
There are also a code style issues that could be resolved via breaking
down cvmx_usb_poll_channel().
Signed-off-by: Georgios Tsotsos <tsotsos@gmail.com>
---
drivers/staging/octeon-usb/octeon-hcd.c | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/staging/octeon-usb/octeon-hcd.c
index cff5e790b196..0d3ec12788fc 100644
--- a/drivers/staging/octeon-usb/octeon-hcd.c
+++ b/drivers/staging/octeon-usb/octeon-hcd.c
@@ -2637,12 +2637,14 @@ static int cvmx_usb_poll_channel(struct octeon_hcd *usb, int channel)
hcintmsk.u32 = 0;
hcintmsk.s.chhltdmsk = 1;
cvmx_usb_write_csr32(usb,
- CVMX_USBCX_HCINTMSKX(channel, usb->index),
- hcintmsk.u32);
+ CVMX_USBCX_HCINTMSKX(channel,
+ usb->index),
+ hcintmsk.u32);
usbc_hcchar.s.chdis = 1;
cvmx_usb_write_csr32(usb,
- CVMX_USBCX_HCCHARX(channel, usb->index),
- usbc_hcchar.u32);
+ CVMX_USBCX_HCCHARX(channel,
+ usb->index),
+ usbc_hcchar.u32);
return 0;
} else if (usbc_hcint.s.xfercompl) {
/*
--
2.16.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 3/4] Staging: octeon-usb: Change coding style for CVMX_WAIT_FOR_FIELD32 marco.
2018-07-24 22:38 [PATCH 0/4] Staging: octeon-usb: Fixes and Coding style applied Georgios Tsotsos
2018-07-24 22:38 ` [PATCH 1/4] Staging: octeon-usb: Adds SPDX License Identifier Georgios Tsotsos
2018-07-24 22:38 ` [PATCH 2/4] Staging: octeon-usb: Applying coding style for column width Georgios Tsotsos
@ 2018-07-24 22:38 ` Georgios Tsotsos
2018-07-24 22:38 ` [PATCH 4/4] Staging: octeon-usb: Adds SPDX License Identifier for header file Georgios Tsotsos
2018-07-25 7:57 ` [PATCH 0/4] Staging: octeon-usb: Fixes and Coding style applied Greg Kroah-Hartman
4 siblings, 0 replies; 8+ messages in thread
From: Georgios Tsotsos @ 2018-07-24 22:38 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Georgios Tsotsos, Aaro Koskinen, James Hogan, devel, linux-kernel
Changes coding style and encapsulate into parentheses timeout_usec.
Signed-off-by: Georgios Tsotsos <tsotsos@gmail.com>
---
drivers/staging/octeon-usb/octeon-hcd.c | 44 +++++++++++++++++----------------
1 file changed, 23 insertions(+), 21 deletions(-)
diff --git a/drivers/staging/octeon-usb/octeon-hcd.c b/drivers/staging/octeon-usb/octeon-hcd.c
index 0d3ec12788fc..84986252720d 100644
--- a/drivers/staging/octeon-usb/octeon-hcd.c
+++ b/drivers/staging/octeon-usb/octeon-hcd.c
@@ -378,27 +378,29 @@ struct octeon_hcd {
};
/* This macro spins on a register waiting for it to reach a condition. */
-#define CVMX_WAIT_FOR_FIELD32(address, _union, cond, timeout_usec) \
- ({int result; \
- do { \
- u64 done = cvmx_get_cycle() + (u64)timeout_usec * \
- octeon_get_clock_rate() / 1000000; \
- union _union c; \
- \
- while (1) { \
- c.u32 = cvmx_usb_read_csr32(usb, address); \
- \
- if (cond) { \
- result = 0; \
- break; \
- } else if (cvmx_get_cycle() > done) { \
- result = -1; \
- break; \
- } else \
- __delay(100); \
- } \
- } while (0); \
- result; })
+#define CVMX_WAIT_FOR_FIELD32(address, _union, cond, timeout_usec) \
+({ \
+ int result; \
+ do { \
+ u64 done = cvmx_get_cycle() + (u64)(timeout_usec) * \
+ octeon_get_clock_rate() / 1000000; \
+ union _union c; \
+ \
+ while (1) { \
+ c.u32 = cvmx_usb_read_csr32(usb, address); \
+ \
+ if (cond) { \
+ result = 0; \
+ break; \
+ } else if (cvmx_get_cycle() > done) { \
+ result = -1; \
+ break; \
+ } else \
+ __delay(100); \
+ } \
+ } while (0); \
+ result; \
+})
/*
* This macro logically sets a single field in a CSR. It does the sequence
--
2.16.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 4/4] Staging: octeon-usb: Adds SPDX License Identifier for header file
2018-07-24 22:38 [PATCH 0/4] Staging: octeon-usb: Fixes and Coding style applied Georgios Tsotsos
` (2 preceding siblings ...)
2018-07-24 22:38 ` [PATCH 3/4] Staging: octeon-usb: Change coding style for CVMX_WAIT_FOR_FIELD32 marco Georgios Tsotsos
@ 2018-07-24 22:38 ` Georgios Tsotsos
2018-07-25 7:57 ` [PATCH 0/4] Staging: octeon-usb: Fixes and Coding style applied Greg Kroah-Hartman
4 siblings, 0 replies; 8+ messages in thread
From: Georgios Tsotsos @ 2018-07-24 22:38 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Georgios Tsotsos, Aaro Koskinen, James Hogan, devel, linux-kernel
Signed-off-by: Georgios Tsotsos <tsotsos@gmail.com>
---
drivers/staging/octeon-usb/octeon-hcd.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/octeon-usb/octeon-hcd.h b/drivers/staging/octeon-usb/octeon-hcd.h
index 3353aefe662e..769c36cf6614 100644
--- a/drivers/staging/octeon-usb/octeon-hcd.h
+++ b/drivers/staging/octeon-usb/octeon-hcd.h
@@ -1,3 +1,4 @@
+/* SPDX-License-Identifier: GPL-2.0 */
/*
* Octeon HCD hardware register definitions.
*
--
2.16.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 0/4] Staging: octeon-usb: Fixes and Coding style applied.
2018-07-24 22:38 [PATCH 0/4] Staging: octeon-usb: Fixes and Coding style applied Georgios Tsotsos
` (3 preceding siblings ...)
2018-07-24 22:38 ` [PATCH 4/4] Staging: octeon-usb: Adds SPDX License Identifier for header file Georgios Tsotsos
@ 2018-07-25 7:57 ` Greg Kroah-Hartman
4 siblings, 0 replies; 8+ messages in thread
From: Greg Kroah-Hartman @ 2018-07-25 7:57 UTC (permalink / raw)
To: Georgios Tsotsos; +Cc: devel, James Hogan, linux-kernel, Aaro Koskinen
On Wed, Jul 25, 2018 at 01:38:49AM +0300, Georgios Tsotsos wrote:
> Hello,
>
> Previously patches were sent with wrong cover format, please ignore.
>
> I am sending four patches which trying to resolve TODO list requirements
> no 45 about octeon-usb.
> There are SPDX licence additions on c and header files.
> Checkpatch warnings are resolved,also a notice about CVMX_WAIT_FOR_FIELD32 macro.
> It could improve readability and maintenance of this module if function
> cvmx_usb_poll_channel() break down (and probably some others too)
My same comments hold true for the "no changelog" patches :(
thanks,
greg k-h
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 4/4] Staging: octeon-usb: Adds SPDX License Identifier for header file
2018-07-24 22:22 ` [PATCH 4/4] Staging: octeon-usb: Adds SPDX License Identifier for header file Georgios Tsotsos
@ 2018-07-25 7:40 ` Greg Kroah-Hartman
0 siblings, 0 replies; 8+ messages in thread
From: Greg Kroah-Hartman @ 2018-07-25 7:40 UTC (permalink / raw)
To: Georgios Tsotsos; +Cc: devel, James Hogan, linux-kernel, Aaro Koskinen
On Wed, Jul 25, 2018 at 01:22:11AM +0300, Georgios Tsotsos wrote:
> Signed-off-by: Georgios Tsotsos <tsotsos@gmail.com>
> ---
I can not take patches without any changelog text, sorry :(
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 4/4] Staging: octeon-usb: Adds SPDX License Identifier for header file
[not found] <20180724222212.8742-1-tsotsos@gmail.com>
@ 2018-07-24 22:22 ` Georgios Tsotsos
2018-07-25 7:40 ` Greg Kroah-Hartman
0 siblings, 1 reply; 8+ messages in thread
From: Georgios Tsotsos @ 2018-07-24 22:22 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Georgios Tsotsos, Aaro Koskinen, James Hogan, devel, linux-kernel
Signed-off-by: Georgios Tsotsos <tsotsos@gmail.com>
---
drivers/staging/octeon-usb/octeon-hcd.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/octeon-usb/octeon-hcd.h b/drivers/staging/octeon-usb/octeon-hcd.h
index 3353aefe662e..769c36cf6614 100644
--- a/drivers/staging/octeon-usb/octeon-hcd.h
+++ b/drivers/staging/octeon-usb/octeon-hcd.h
@@ -1,3 +1,4 @@
+/* SPDX-License-Identifier: GPL-2.0 */
/*
* Octeon HCD hardware register definitions.
*
--
2.16.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
end of thread, other threads:[~2018-07-25 7:57 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-07-24 22:38 [PATCH 0/4] Staging: octeon-usb: Fixes and Coding style applied Georgios Tsotsos
2018-07-24 22:38 ` [PATCH 1/4] Staging: octeon-usb: Adds SPDX License Identifier Georgios Tsotsos
2018-07-24 22:38 ` [PATCH 2/4] Staging: octeon-usb: Applying coding style for column width Georgios Tsotsos
2018-07-24 22:38 ` [PATCH 3/4] Staging: octeon-usb: Change coding style for CVMX_WAIT_FOR_FIELD32 marco Georgios Tsotsos
2018-07-24 22:38 ` [PATCH 4/4] Staging: octeon-usb: Adds SPDX License Identifier for header file Georgios Tsotsos
2018-07-25 7:57 ` [PATCH 0/4] Staging: octeon-usb: Fixes and Coding style applied Greg Kroah-Hartman
[not found] <20180724222212.8742-1-tsotsos@gmail.com>
2018-07-24 22:22 ` [PATCH 4/4] Staging: octeon-usb: Adds SPDX License Identifier for header file Georgios Tsotsos
2018-07-25 7:40 ` Greg Kroah-Hartman
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).