Re: [PATCH 3/9v2] usb: host: ehci.h: remove space before open square bracket

2015-12-10 Thread Peter Senna Tschudin
On Wed, Dec 09, 2015 at 07:32:32PM -0300, Geyslan G. Bem wrote:
> Get rid of space before open square bracket.
> 
> Caught by checkpatch: "ERROR: space prohibited before open square
> bracket '['"
> 
> Signed-off-by: Geyslan G. Bem 
As with previous, missing the changelog, but other than that the patch
looks good to me.

Acked-by: Peter Senna Tschudin 
> ---
>  drivers/usb/host/ehci.h | 20 ++--
>  1 file changed, 10 insertions(+), 10 deletions(-)
> 
> diff --git a/drivers/usb/host/ehci.h b/drivers/usb/host/ehci.h
> index 46982df..cfeebd8 100644
> --- a/drivers/usb/host/ehci.h
> +++ b/drivers/usb/host/ehci.h
> @@ -185,7 +185,7 @@ struct ehci_hcd { /* one per controller */
>   struct ehci_sitd*last_sitd_to_free;
>  
>   /* per root hub port */
> - unsigned long   reset_done [EHCI_MAX_ROOT_PORTS];
> + unsigned long   reset_done[EHCI_MAX_ROOT_PORTS];
>  
>   /* bit vectors (one bit per port) */
>   unsigned long   bus_suspended;  /* which ports were
> @@ -316,8 +316,8 @@ struct ehci_qtd {
>  #define HALT_BIT(ehci)   cpu_to_hc32(ehci, QTD_STS_HALT)
>  #define STATUS_BIT(ehci) cpu_to_hc32(ehci, QTD_STS_STS)
>  
> - __hc32  hw_buf [5];/* see EHCI 3.5.4 */
> - __hc32  hw_buf_hi [5];/* Appendix B */
> + __hc32  hw_buf[5];/* see EHCI 3.5.4 */
> + __hc32  hw_buf_hi[5];/* Appendix B */
>  
>   /* the rest is HCD-private */
>   dma_addr_t  qtd_dma;/* qtd address */
> @@ -405,8 +405,8 @@ struct ehci_qh_hw {
>   __hc32  hw_qtd_next;
>   __hc32  hw_alt_next;
>   __hc32  hw_token;
> - __hc32  hw_buf [5];
> - __hc32  hw_buf_hi [5];
> + __hc32  hw_buf[5];
> + __hc32  hw_buf_hi[5];
>  } __attribute__ ((aligned(32)));
>  
>  struct ehci_qh {
> @@ -510,7 +510,7 @@ struct ehci_iso_stream {
>  struct ehci_itd {
>   /* first part defined by EHCI spec */
>   __hc32  hw_next;   /* see EHCI 3.3.1 */
> - __hc32  hw_transaction [8]; /* see EHCI 3.3.2 */
> + __hc32  hw_transaction[8]; /* see EHCI 3.3.2 */
>  #define EHCI_ISOC_ACTIVE(1<<31)/* activate transfer this 
> slot */
>  #define EHCI_ISOC_BUF_ERR   (1<<30)/* Data buffer error */
>  #define EHCI_ISOC_BABBLE(1<<29)/* babble detected */
> @@ -520,8 +520,8 @@ struct ehci_itd {
>  
>  #define ITD_ACTIVE(ehci) cpu_to_hc32(ehci, EHCI_ISOC_ACTIVE)
>  
> - __hc32  hw_bufp [7];/* see EHCI 3.3.3 */
> - __hc32  hw_bufp_hi [7]; /* Appendix B */
> + __hc32  hw_bufp[7]; /* see EHCI 3.3.3 */
> + __hc32  hw_bufp_hi[7];  /* Appendix B */
>  
>   /* the rest is HCD-private */
>   dma_addr_t  itd_dma;/* for this itd */
> @@ -565,9 +565,9 @@ struct ehci_sitd {
>  
>  #define SITD_ACTIVE(ehci)cpu_to_hc32(ehci, SITD_STS_ACTIVE)
>  
> - __hc32  hw_buf [2]; /* EHCI table 3-12 */
> + __hc32  hw_buf[2];  /* EHCI table 3-12 */
>   __hc32  hw_backpointer; /* EHCI table 3-13 */
> - __hc32  hw_buf_hi [2];  /* Appendix B */
> + __hc32  hw_buf_hi[2];   /* Appendix B */
>  
>   /* the rest is HCD-private */
>   dma_addr_t  sitd_dma;
> -- 
> 2.6.3
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 3/9v2] usb: host: ehci.h: remove space before open square bracket

2015-12-09 Thread Geyslan G. Bem
Get rid of space before open square bracket.

Caught by checkpatch: "ERROR: space prohibited before open square
bracket '['"

Signed-off-by: Geyslan G. Bem 
---
 drivers/usb/host/ehci.h | 20 ++--
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/drivers/usb/host/ehci.h b/drivers/usb/host/ehci.h
index 46982df..cfeebd8 100644
--- a/drivers/usb/host/ehci.h
+++ b/drivers/usb/host/ehci.h
@@ -185,7 +185,7 @@ struct ehci_hcd {   /* one per controller */
struct ehci_sitd*last_sitd_to_free;
 
/* per root hub port */
-   unsigned long   reset_done [EHCI_MAX_ROOT_PORTS];
+   unsigned long   reset_done[EHCI_MAX_ROOT_PORTS];
 
/* bit vectors (one bit per port) */
unsigned long   bus_suspended;  /* which ports were
@@ -316,8 +316,8 @@ struct ehci_qtd {
 #define HALT_BIT(ehci) cpu_to_hc32(ehci, QTD_STS_HALT)
 #define STATUS_BIT(ehci)   cpu_to_hc32(ehci, QTD_STS_STS)
 
-   __hc32  hw_buf [5];/* see EHCI 3.5.4 */
-   __hc32  hw_buf_hi [5];/* Appendix B */
+   __hc32  hw_buf[5];/* see EHCI 3.5.4 */
+   __hc32  hw_buf_hi[5];/* Appendix B */
 
/* the rest is HCD-private */
dma_addr_t  qtd_dma;/* qtd address */
@@ -405,8 +405,8 @@ struct ehci_qh_hw {
__hc32  hw_qtd_next;
__hc32  hw_alt_next;
__hc32  hw_token;
-   __hc32  hw_buf [5];
-   __hc32  hw_buf_hi [5];
+   __hc32  hw_buf[5];
+   __hc32  hw_buf_hi[5];
 } __attribute__ ((aligned(32)));
 
 struct ehci_qh {
@@ -510,7 +510,7 @@ struct ehci_iso_stream {
 struct ehci_itd {
/* first part defined by EHCI spec */
__hc32  hw_next;   /* see EHCI 3.3.1 */
-   __hc32  hw_transaction [8]; /* see EHCI 3.3.2 */
+   __hc32  hw_transaction[8]; /* see EHCI 3.3.2 */
 #define EHCI_ISOC_ACTIVE(1<<31)/* activate transfer this slot 
*/
 #define EHCI_ISOC_BUF_ERR   (1<<30)/* Data buffer error */
 #define EHCI_ISOC_BABBLE(1<<29)/* babble detected */
@@ -520,8 +520,8 @@ struct ehci_itd {
 
 #define ITD_ACTIVE(ehci)   cpu_to_hc32(ehci, EHCI_ISOC_ACTIVE)
 
-   __hc32  hw_bufp [7];/* see EHCI 3.3.3 */
-   __hc32  hw_bufp_hi [7]; /* Appendix B */
+   __hc32  hw_bufp[7]; /* see EHCI 3.3.3 */
+   __hc32  hw_bufp_hi[7];  /* Appendix B */
 
/* the rest is HCD-private */
dma_addr_t  itd_dma;/* for this itd */
@@ -565,9 +565,9 @@ struct ehci_sitd {
 
 #define SITD_ACTIVE(ehci)  cpu_to_hc32(ehci, SITD_STS_ACTIVE)
 
-   __hc32  hw_buf [2]; /* EHCI table 3-12 */
+   __hc32  hw_buf[2];  /* EHCI table 3-12 */
__hc32  hw_backpointer; /* EHCI table 3-13 */
-   __hc32  hw_buf_hi [2];  /* Appendix B */
+   __hc32  hw_buf_hi[2];   /* Appendix B */
 
/* the rest is HCD-private */
dma_addr_t  sitd_dma;
-- 
2.6.3

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html