Re: [PATCH 5/9v2] usb: host: ehci.h: remove direct use of __attribute__ keyword

[Peter Senna Tschudin]

On Wed, Dec 09, 2015 at 07:32:34PM -0300, Geyslan G. Bem wrote:
> Prefer to use __aligned(size) macro instead of
> __attribute__((aligned(size))).
> 
> Caught by checkpatch: "WARNING"
Not sure about the WARNING here, maybe just remove this line?

> 
> Signed-off-by: Geyslan G. Bem <geyslan@xxxxxxxxx>
As with previous, missing the changelog, but other than that the patch
looks good to me.

Acked-by: Peter Senna Tschudin <peter.senna@xxxxxxxxx>
> ---
>  drivers/usb/host/ehci.h | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/usb/host/ehci.h b/drivers/usb/host/ehci.h
> index 945000a..7b2b213 100644
> --- a/drivers/usb/host/ehci.h
> +++ b/drivers/usb/host/ehci.h
> @@ -324,7 +324,7 @@ struct ehci_qtd {
>  	struct list_head	qtd_list;		/* sw qtd list */
>  	struct urb		*urb;			/* qtd's urb */
>  	size_t			length;			/* length of buffer */
> -} __attribute__ ((aligned (32)));
> +} __aligned(32);
>  
>  /* mask NakCnt+T in qh->hw_alt_next */
>  #define QTD_MASK(ehci)	cpu_to_hc32(ehci, ~0x1f)
> @@ -407,7 +407,7 @@ struct ehci_qh_hw {
>  	__hc32			hw_token;
>  	__hc32			hw_buf[5];
>  	__hc32			hw_buf_hi[5];
> -} __attribute__ ((aligned(32)));
> +} __aligned(32);
>  
>  struct ehci_qh {
>  	struct ehci_qh_hw	*hw;		/* Must come first */
> @@ -535,7 +535,7 @@ struct ehci_itd {
>  	unsigned		frame;		/* where scheduled */
>  	unsigned		pg;
>  	unsigned		index[8];	/* in urb->iso_frame_desc */
> -} __attribute__ ((aligned (32)));
> +} __aligned(32);
>  
>  /*-------------------------------------------------------------------------*/
>  
> @@ -578,7 +578,7 @@ struct ehci_sitd {
>  	struct list_head	sitd_list;	/* list of stream's sitds */
>  	unsigned		frame;
>  	unsigned		index;
> -} __attribute__ ((aligned (32)));
> +} __aligned(32);
>  
>  /*-------------------------------------------------------------------------*/
>  
> @@ -598,7 +598,7 @@ struct ehci_fstn {
>  	/* the rest is HCD-private */
>  	dma_addr_t		fstn_dma;
>  	union ehci_shadow	fstn_next;	/* ptr to periodic q entry */
> -} __attribute__ ((aligned (32)));
> +} __aligned(32);
>  
>  /*-------------------------------------------------------------------------*/
>  
> -- 
> 2.6.3
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/