On 9/2/20 8:13 AM, Frank Wunderlich wrote:
> From: Chunfeng Yun <chunfeng....@mediatek.com>
>
> Add a member to save xHCI version, it's used some times.
>
> Signed-off-by: Chunfeng Yun <chunfeng....@mediatek.com>
> ---
> drivers/usb/host/xhci-ring.c | 4 ++--
> drivers/usb/host/xhci.c | 1 +
> include/usb/xhci.h | 1 +
> 3 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/usb/host/xhci-ring.c b/drivers/usb/host/xhci-ring.c
> index 092ed6eaf1..79bfc349f4 100644
> --- a/drivers/usb/host/xhci-ring.c
> +++ b/drivers/usb/host/xhci-ring.c
> @@ -682,7 +682,7 @@ int xhci_bulk_tx(struct usb_device *udev, unsigned long
> pipe,
> field |= TRB_ISP;
>
> /* Set the TRB length, TD size, and interrupter fields. */
> - if (HC_VERSION(xhci_readl(&ctrl->hccr->cr_capbase)) < 0x100)
> + if (ctrl->hci_version < 0x100)
> remainder = xhci_td_remainder(length - running_total);
> else
> remainder = xhci_v1_0_td_remainder(running_total,
> @@ -830,7 +830,7 @@ int xhci_ctrl_tx(struct usb_device *udev, unsigned long
> pipe,
> field |= 0x1;
>
> /* xHCI 1.0 6.4.1.2.1: Transfer Type field */
> - if (HC_VERSION(xhci_readl(&ctrl->hccr->cr_capbase)) >= 0x100) {
> + if (ctrl->hci_version >= 0x100) {
> if (length > 0) {
> if (req->requesttype & USB_DIR_IN)
> field |= (TRB_DATA_IN << TRB_TX_TYPE_SHIFT);
> diff --git a/drivers/usb/host/xhci.c b/drivers/usb/host/xhci.c
> index 126dabc11b..4be1411243 100644
> --- a/drivers/usb/host/xhci.c
> +++ b/drivers/usb/host/xhci.c
> @@ -1283,6 +1283,7 @@ static int xhci_lowlevel_init(struct xhci_ctrl *ctrl)
>
> reg = HC_VERSION(xhci_readl(&hccr->cr_capbase));
> printf("USB XHCI %x.%02x\n", reg >> 8, reg & 0xff);
> + ctrl->hci_version = reg;
>
> return 0;
> }
> diff --git a/include/usb/xhci.h b/include/usb/xhci.h
> index 7d34103fd5..a3e5914b10 100644
> --- a/include/usb/xhci.h
> +++ b/include/usb/xhci.h
> @@ -1227,6 +1227,7 @@ struct xhci_ctrl {
> struct xhci_scratchpad *scratchpad;
> struct xhci_virt_device *devs[MAX_HC_SLOTS];
> int rootdev;
> + u16 hci_version;
> };
>
> unsigned long trb_addr(struct xhci_segment *seg, union xhci_trb *trb);
>
+CC Bin
