Re: [PATCH 10/17] usb: isp1760: Prefix driver data structures with isp1760_

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Oct 06, 2014 at 06:54:58PM +0300, Laurent Pinchart wrote:
> The slotinfo and memory_chunk structures are specific to the driver and
> defined in a header file. Prefix them with isp1760_ to avoid namespace
> clashes.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>

Reviewed-by: Felipe Balbi <balbi@xxxxxx>

> ---
>  drivers/usb/host/isp1760-hcd.c | 15 ++++++++-------
>  drivers/usb/host/isp1760-hcd.h |  4 ++--
>  2 files changed, 10 insertions(+), 9 deletions(-)
> 
> diff --git a/drivers/usb/host/isp1760-hcd.c b/drivers/usb/host/isp1760-hcd.c
> index df715b6..5695ffc 100644
> --- a/drivers/usb/host/isp1760-hcd.c
> +++ b/drivers/usb/host/isp1760-hcd.c
> @@ -44,11 +44,11 @@ struct isp1760_hcd {
>  
>  	u32 hcs_params;
>  	spinlock_t		lock;
> -	struct slotinfo		atl_slots[32];
> +	struct isp1760_slotinfo	atl_slots[32];
>  	int			atl_done_map;
> -	struct slotinfo		int_slots[32];
> +	struct isp1760_slotinfo	int_slots[32];
>  	int			int_done_map;
> -	struct memory_chunk memory_pool[BLOCKS];
> +	struct isp1760_memory_chunk memory_pool[BLOCKS];
>  	struct list_head	qh_list[QH_END];
>  
>  	/* periodic schedule support */
> @@ -748,8 +748,9 @@ static void qtd_free(struct isp1760_qtd *qtd)
>  }
>  
>  static void start_bus_transfer(struct usb_hcd *hcd, u32 ptd_offset, int slot,
> -				struct slotinfo *slots, struct isp1760_qtd *qtd,
> -				struct isp1760_qh *qh, struct ptd *ptd)
> +				struct isp1760_slotinfo *slots,
> +				struct isp1760_qtd *qtd, struct isp1760_qh *qh,
> +				struct ptd *ptd)
>  {
>  	struct isp1760_hcd *priv = hcd_to_priv(hcd);
>  	int skip_map;
> @@ -862,7 +863,7 @@ static void enqueue_qtds(struct usb_hcd *hcd, struct isp1760_qh *qh)
>  {
>  	struct isp1760_hcd *priv = hcd_to_priv(hcd);
>  	int ptd_offset;
> -	struct slotinfo *slots;
> +	struct isp1760_slotinfo *slots;
>  	int curr_slot, free_slot;
>  	int n;
>  	struct ptd ptd;
> @@ -1102,7 +1103,7 @@ static void handle_done_ptds(struct usb_hcd *hcd)
>  	struct isp1760_qh *qh;
>  	int slot;
>  	int state;
> -	struct slotinfo *slots;
> +	struct isp1760_slotinfo *slots;
>  	u32 ptd_offset;
>  	struct isp1760_qtd *qtd;
>  	int modified;
> diff --git a/drivers/usb/host/isp1760-hcd.h b/drivers/usb/host/isp1760-hcd.h
> index 9c55e93..2a918d6 100644
> --- a/drivers/usb/host/isp1760-hcd.h
> +++ b/drivers/usb/host/isp1760-hcd.h
> @@ -103,7 +103,7 @@ struct ptd {
>  #define ATL_PTD_OFFSET		0x0c00
>  #define PAYLOAD_OFFSET		0x1000
>  
> -struct slotinfo {
> +struct isp1760_slotinfo {
>  	struct isp1760_qh *qh;
>  	struct isp1760_qtd *qtd;
>  	unsigned long timestamp;
> @@ -126,7 +126,7 @@ struct slotinfo {
>  #define ISP1760_FLAG_RESET_ACTIVE_HIGH	0x80000000 /* RESET GPIO active high */
>  
>  /* chip memory management */
> -struct memory_chunk {
> +struct isp1760_memory_chunk {
>  	unsigned int start;
>  	unsigned int size;
>  	unsigned int free;
> -- 
> 2.0.4
> 

-- 
balbi

Attachment: signature.asc
Description: Digital signature


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux