Re: [PATCH] drivers:staging:octeon-usb: Fixed Bitfields coding style errors

2014-01-31 Thread Aaro Koskinen
Hi,

On Thu, Jan 30, 2014 at 11:17:47PM -0800, Surendra Patil wrote:
> Fixed Below coding style errors -
> octeon-hcd.h:146: ERROR: spaces prohibited around that ':' (ctx:WxW)
> octeon-hcd.h:147: ERROR: spaces prohibited around that ':' (ctx:WxW)

Not an error, checkpatch is wrong here.

A.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] drivers:staging:octeon-usb: Fixed Bitfields coding style errors

2014-01-31 Thread Aaro Koskinen
Hi,

On Thu, Jan 30, 2014 at 11:17:47PM -0800, Surendra Patil wrote:
 Fixed Below coding style errors -
 octeon-hcd.h:146: ERROR: spaces prohibited around that ':' (ctx:WxW)
 octeon-hcd.h:147: ERROR: spaces prohibited around that ':' (ctx:WxW)

Not an error, checkpatch is wrong here.

A.
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH] drivers:staging:octeon-usb: Fixed Bitfields coding style errors

2014-01-30 Thread Surendra Patil
Fixed Below coding style errors -
octeon-hcd.h:146: ERROR: spaces prohibited around that ':' (ctx:WxW)
octeon-hcd.h:147: ERROR: spaces prohibited around that ':' (ctx:WxW)
total: 243 errors, 0 warnings, 1819 lines checked - fixed all errors

Signed-off-by: Surendra Patil 
---
 drivers/staging/octeon-usb/octeon-hcd.h | 486 
 1 file changed, 243 insertions(+), 243 deletions(-)

diff --git a/drivers/staging/octeon-usb/octeon-hcd.h 
b/drivers/staging/octeon-usb/octeon-hcd.h
index 42fe4fe..c534e317 100644
--- a/drivers/staging/octeon-usb/octeon-hcd.h
+++ b/drivers/staging/octeon-usb/octeon-hcd.h
@@ -143,13 +143,13 @@ union cvmx_usbcx_gahbcfg {
 *  * 1'b1: Unmask the interrupt assertion to the application.
 */
struct cvmx_usbcx_gahbcfg_s {
-   uint32_t reserved_9_31  : 23;
-   uint32_t ptxfemplvl : 1;
-   uint32_t nptxfemplvl: 1;
-   uint32_t reserved_6_6   : 1;
-   uint32_t dmaen  : 1;
-   uint32_t hbstlen: 4;
-   uint32_t glblintrmsk: 1;
+   uint32_t reserved_9_31:23;
+   uint32_t ptxfemplvl:1;
+   uint32_t nptxfemplvl:1;
+   uint32_t reserved_6_6:1;
+   uint32_t dmaen:1;
+   uint32_t hbstlen:4;
+   uint32_t glblintrmsk:1;
} s;
 };
 
@@ -209,16 +209,16 @@ union cvmx_usbcx_ghwcfg3 {
 *  * Others: Reserved
 */
struct cvmx_usbcx_ghwcfg3_s {
-   uint32_t dfifodepth : 16;
-   uint32_t reserved_13_15 : 3;
-   uint32_t ahbphysync : 1;
-   uint32_t rsttype: 1;
-   uint32_t optfeature : 1;
-   uint32_t vendor_control_interface_support   : 1;
-   uint32_t i2c_selection  : 1;
-   uint32_t otgen  : 1;
-   uint32_t pktsizewidth   : 3;
-   uint32_t xfersizewidth  : 4;
+   uint32_t dfifodepth:16;
+   uint32_t reserved_13_15:3;
+   uint32_t ahbphysync:1;
+   uint32_t rsttype:1;
+   uint32_t optfeature:1;
+   uint32_t vendor_control_interface_support:1;
+   uint32_t i2c_selection:1;
+   uint32_t otgen:1;
+   uint32_t pktsizewidth:3;
+   uint32_t xfersizewidth:4;
} s;
 };
 
@@ -275,38 +275,38 @@ union cvmx_usbcx_gintmsk {
 * @modemismsk: Mode Mismatch Interrupt Mask (ModeMisMsk)
 */
struct cvmx_usbcx_gintmsk_s {
-   uint32_t wkupintmsk : 1;
-   uint32_t sessreqintmsk  : 1;
-   uint32_t disconnintmsk  : 1;
-   uint32_t conidstschngmsk: 1;
-   uint32_t reserved_27_27 : 1;
-   uint32_t ptxfempmsk : 1;
-   uint32_t hchintmsk  : 1;
-   uint32_t prtintmsk  : 1;
-   uint32_t reserved_23_23 : 1;
-   uint32_t fetsuspmsk : 1;
-   uint32_t incomplpmsk: 1;
-   uint32_t incompisoinmsk : 1;
-   uint32_t oepintmsk  : 1;
-   uint32_t inepintmsk : 1;
-   uint32_t epmismsk   : 1;
-   uint32_t reserved_16_16 : 1;
-   uint32_t eopfmsk: 1;
-   uint32_t isooutdropmsk  : 1;
-   uint32_t enumdonemsk: 1;
-   uint32_t usbrstmsk  : 1;
-   uint32_t usbsuspmsk : 1;
-   uint32_t erlysuspmsk: 1;
-   uint32_t i2cint : 1;
-   uint32_t ulpickintmsk   : 1;
-   uint32_t goutnakeffmsk  : 1;
-   uint32_t ginnakeffmsk   : 1;
-   uint32_t nptxfempmsk: 1;
-   uint32_t rxflvlmsk  : 1;
-   uint32_t sofmsk : 1;
-   uint32_t otgintmsk  : 1;
-   uint32_t modemismsk : 1;
-   uint32_t reserved_0_0   : 1;
+   uint32_t wkupintmsk:1;
+   uint32_t sessreqintmsk:1;
+   uint32_t disconnintmsk:1;
+   uint32_t conidstschngmsk:1;
+   uint32_t reserved_27_27:1;
+   uint32_t ptxfempmsk:1;
+   uint32_t hchintmsk:1;
+   uint32_t prtintmsk:1;
+   uint32_t reserved_23_23:1;
+   uint32_t fetsuspmsk:1;
+   uint32_t incomplpmsk:1;
+   

[PATCH] drivers:staging:octeon-usb: Fixed Bitfields coding style errors

2014-01-30 Thread Surendra Patil
Fixed Below coding style errors -
octeon-hcd.h:146: ERROR: spaces prohibited around that ':' (ctx:WxW)
octeon-hcd.h:147: ERROR: spaces prohibited around that ':' (ctx:WxW)
total: 243 errors, 0 warnings, 1819 lines checked - fixed all errors

Signed-off-by: Surendra Patil surendra@gmail.com
---
 drivers/staging/octeon-usb/octeon-hcd.h | 486 
 1 file changed, 243 insertions(+), 243 deletions(-)

diff --git a/drivers/staging/octeon-usb/octeon-hcd.h 
b/drivers/staging/octeon-usb/octeon-hcd.h
index 42fe4fe..c534e317 100644
--- a/drivers/staging/octeon-usb/octeon-hcd.h
+++ b/drivers/staging/octeon-usb/octeon-hcd.h
@@ -143,13 +143,13 @@ union cvmx_usbcx_gahbcfg {
 *  * 1'b1: Unmask the interrupt assertion to the application.
 */
struct cvmx_usbcx_gahbcfg_s {
-   uint32_t reserved_9_31  : 23;
-   uint32_t ptxfemplvl : 1;
-   uint32_t nptxfemplvl: 1;
-   uint32_t reserved_6_6   : 1;
-   uint32_t dmaen  : 1;
-   uint32_t hbstlen: 4;
-   uint32_t glblintrmsk: 1;
+   uint32_t reserved_9_31:23;
+   uint32_t ptxfemplvl:1;
+   uint32_t nptxfemplvl:1;
+   uint32_t reserved_6_6:1;
+   uint32_t dmaen:1;
+   uint32_t hbstlen:4;
+   uint32_t glblintrmsk:1;
} s;
 };
 
@@ -209,16 +209,16 @@ union cvmx_usbcx_ghwcfg3 {
 *  * Others: Reserved
 */
struct cvmx_usbcx_ghwcfg3_s {
-   uint32_t dfifodepth : 16;
-   uint32_t reserved_13_15 : 3;
-   uint32_t ahbphysync : 1;
-   uint32_t rsttype: 1;
-   uint32_t optfeature : 1;
-   uint32_t vendor_control_interface_support   : 1;
-   uint32_t i2c_selection  : 1;
-   uint32_t otgen  : 1;
-   uint32_t pktsizewidth   : 3;
-   uint32_t xfersizewidth  : 4;
+   uint32_t dfifodepth:16;
+   uint32_t reserved_13_15:3;
+   uint32_t ahbphysync:1;
+   uint32_t rsttype:1;
+   uint32_t optfeature:1;
+   uint32_t vendor_control_interface_support:1;
+   uint32_t i2c_selection:1;
+   uint32_t otgen:1;
+   uint32_t pktsizewidth:3;
+   uint32_t xfersizewidth:4;
} s;
 };
 
@@ -275,38 +275,38 @@ union cvmx_usbcx_gintmsk {
 * @modemismsk: Mode Mismatch Interrupt Mask (ModeMisMsk)
 */
struct cvmx_usbcx_gintmsk_s {
-   uint32_t wkupintmsk : 1;
-   uint32_t sessreqintmsk  : 1;
-   uint32_t disconnintmsk  : 1;
-   uint32_t conidstschngmsk: 1;
-   uint32_t reserved_27_27 : 1;
-   uint32_t ptxfempmsk : 1;
-   uint32_t hchintmsk  : 1;
-   uint32_t prtintmsk  : 1;
-   uint32_t reserved_23_23 : 1;
-   uint32_t fetsuspmsk : 1;
-   uint32_t incomplpmsk: 1;
-   uint32_t incompisoinmsk : 1;
-   uint32_t oepintmsk  : 1;
-   uint32_t inepintmsk : 1;
-   uint32_t epmismsk   : 1;
-   uint32_t reserved_16_16 : 1;
-   uint32_t eopfmsk: 1;
-   uint32_t isooutdropmsk  : 1;
-   uint32_t enumdonemsk: 1;
-   uint32_t usbrstmsk  : 1;
-   uint32_t usbsuspmsk : 1;
-   uint32_t erlysuspmsk: 1;
-   uint32_t i2cint : 1;
-   uint32_t ulpickintmsk   : 1;
-   uint32_t goutnakeffmsk  : 1;
-   uint32_t ginnakeffmsk   : 1;
-   uint32_t nptxfempmsk: 1;
-   uint32_t rxflvlmsk  : 1;
-   uint32_t sofmsk : 1;
-   uint32_t otgintmsk  : 1;
-   uint32_t modemismsk : 1;
-   uint32_t reserved_0_0   : 1;
+   uint32_t wkupintmsk:1;
+   uint32_t sessreqintmsk:1;
+   uint32_t disconnintmsk:1;
+   uint32_t conidstschngmsk:1;
+   uint32_t reserved_27_27:1;
+   uint32_t ptxfempmsk:1;
+   uint32_t hchintmsk:1;
+   uint32_t prtintmsk:1;
+   uint32_t reserved_23_23:1;
+   uint32_t fetsuspmsk:1;
+   uint32_t