Re: [PATCH v2 1/3] usb: add find_raw_port_number callback to struct hc_driver()

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

 



On 2013年01月25日 08:52, Sarah Sharp wrote:
> Please combine this patch with patch three, so that I can clearly see
> what is being replaced in xhci-mem.c:xhci_find_real_port_number.
> 
Ok, I will refresh it soon.
> Sarah Sharp
> 
> On Sat, Jan 19, 2013 at 12:43:32AM +0800, Lan Tianyu wrote:
>> xhci driver divides the root hub into two logical hubs which work
>> respectively for usb 2.0 and usb 3.0 devices. They are independent
>> devices in the usb core. But in the ACPI table, it's one device node
>> and all usb2.0 and usb3.0 ports are under it. Binding usb port with
>> its acpi node needs the raw port number which is reflected in the xhci
>> extended capabilities table. This patch is to add find_raw_port_number
>> callback to struct hc_driver() and fill it with xhci_find_raw_port_number().
>>
>> Signed-off-by: Lan Tianyu <tianyu.lan@xxxxxxxxx>
>> ---
>> Change since v1:
>> 	Don't export usb_hcd_find_raw_port_number() symbol since there is no
>> its user outside of usb core.
>>
>> This patchset is rebased on the usb-next tree commit 74ff31b81d9
>> "usb: misc: usb3503_probe() can be static"
>> ---
>>  drivers/usb/core/hcd.c      |    8 ++++++++
>>  drivers/usb/host/xhci-pci.c |    1 +
>>  drivers/usb/host/xhci.c     |   22 ++++++++++++++++++++++
>>  drivers/usb/host/xhci.h     |    1 +
>>  include/linux/usb/hcd.h     |    2 ++
>>  5 files changed, 34 insertions(+)
>>
>> diff --git a/drivers/usb/core/hcd.c b/drivers/usb/core/hcd.c
>> index 5f6da8b..0277bd2 100644
>> --- a/drivers/usb/core/hcd.c
>> +++ b/drivers/usb/core/hcd.c
>> @@ -2364,6 +2364,14 @@ int usb_hcd_is_primary_hcd(struct usb_hcd *hcd)
>>  }
>>  EXPORT_SYMBOL_GPL(usb_hcd_is_primary_hcd);
>>  
>> +int usb_hcd_find_raw_port_number(struct usb_hcd *hcd, int port1)
>> +{
>> +	if (!hcd->driver->find_raw_port_number)
>> +		return port1;
>> +
>> +	return hcd->driver->find_raw_port_number(hcd, port1);
>> +}
>> +
>>  static int usb_hcd_request_irqs(struct usb_hcd *hcd,
>>  		unsigned int irqnum, unsigned long irqflags)
>>  {
>> diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c
>> index af259e0..1a30c38 100644
>> --- a/drivers/usb/host/xhci-pci.c
>> +++ b/drivers/usb/host/xhci-pci.c
>> @@ -313,6 +313,7 @@ static const struct hc_driver xhci_pci_hc_driver = {
>>  	.set_usb2_hw_lpm =	xhci_set_usb2_hardware_lpm,
>>  	.enable_usb3_lpm_timeout =	xhci_enable_usb3_lpm_timeout,
>>  	.disable_usb3_lpm_timeout =	xhci_disable_usb3_lpm_timeout,
>> +	.find_raw_port_number =	xhci_find_raw_port_number,
>>  };
>>  
>>  /*-------------------------------------------------------------------------*/
>> diff --git a/drivers/usb/host/xhci.c b/drivers/usb/host/xhci.c
>> index f1f01a8..0780d8f 100644
>> --- a/drivers/usb/host/xhci.c
>> +++ b/drivers/usb/host/xhci.c
>> @@ -3778,6 +3778,28 @@ int xhci_address_device(struct usb_hcd *hcd, struct usb_device *udev)
>>  	return 0;
>>  }
>>  
>> +/*
>> + * Transfer the port index into real index in the HW port status
>> + * registers. Caculate offset between the port's PORTSC register
>> + * and port status base. Divide the number of per port register
>> + * to get the real index. The raw port number bases 1.
>> + */
>> +int xhci_find_raw_port_number(struct usb_hcd *hcd, int port1)
>> +{
>> +	struct xhci_hcd *xhci = hcd_to_xhci(hcd);
>> +	__le32 __iomem *base_addr = &xhci->op_regs->port_status_base;
>> +	__le32 __iomem *addr;
>> +	int raw_port;
>> +
>> +	if (hcd->speed != HCD_USB3)
>> +		addr = xhci->usb2_ports[port1 - 1];
>> +	else
>> +		addr = xhci->usb3_ports[port1 - 1];
>> +
>> +	raw_port = (addr - base_addr)/NUM_PORT_REGS + 1;
>> +	return raw_port;
>> +}
>> +
>>  #ifdef CONFIG_USB_SUSPEND
>>  
>>  /* BESL to HIRD Encoding array for USB2 LPM */
>> diff --git a/drivers/usb/host/xhci.h b/drivers/usb/host/xhci.h
>> index f791bd0..55345d9 100644
>> --- a/drivers/usb/host/xhci.h
>> +++ b/drivers/usb/host/xhci.h
>> @@ -1829,6 +1829,7 @@ void xhci_test_and_clear_bit(struct xhci_hcd *xhci, __le32 __iomem **port_array,
>>  int xhci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue, u16 wIndex,
>>  		char *buf, u16 wLength);
>>  int xhci_hub_status_data(struct usb_hcd *hcd, char *buf);
>> +int xhci_find_raw_port_number(struct usb_hcd *hcd, int port1);
>>  
>>  #ifdef CONFIG_PM
>>  int xhci_bus_suspend(struct usb_hcd *hcd);
>> diff --git a/include/linux/usb/hcd.h b/include/linux/usb/hcd.h
>> index 608050b..61b88b5 100644
>> --- a/include/linux/usb/hcd.h
>> +++ b/include/linux/usb/hcd.h
>> @@ -357,6 +357,7 @@ struct hc_driver {
>>  		 */
>>  	int	(*disable_usb3_lpm_timeout)(struct usb_hcd *,
>>  			struct usb_device *, enum usb3_link_state state);
>> +	int	(*find_raw_port_number)(struct usb_hcd *, int);
>>  };
>>  
>>  extern int usb_hcd_link_urb_to_ep(struct usb_hcd *hcd, struct urb *urb);
>> @@ -396,6 +397,7 @@ extern int usb_hcd_is_primary_hcd(struct usb_hcd *hcd);
>>  extern int usb_add_hcd(struct usb_hcd *hcd,
>>  		unsigned int irqnum, unsigned long irqflags);
>>  extern void usb_remove_hcd(struct usb_hcd *hcd);
>> +extern int usb_hcd_find_raw_port_number(struct usb_hcd *hcd, int port1);
>>  
>>  struct platform_device;
>>  extern void usb_hcd_platform_shutdown(struct platform_device *dev);
>> -- 
>> 1.7.10.4
>>


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


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

  Powered by Linux