Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S967068Ab3E2VPX (ORCPT ); Wed, 29 May 2013 17:15:23 -0400 Received: from h1446028.stratoserver.net ([85.214.92.142]:48932 "EHLO mail.ahsoftware.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933570Ab3E2VPU (ORCPT ); Wed, 29 May 2013 17:15:20 -0400 From: Alexander Holler To: linux-kernel@vger.kernel.org Cc: linux-usb@vger.kernel.org, Sarah Sharp , Alexander Holler Subject: [PATCH 1/2] USB: xhci: rename ambiguous named XHCI_NEC_HOST to XHCI_NEC_SHOW_FW Date: Wed, 29 May 2013 23:14:32 +0200 Message-Id: <1369862073-3685-1-git-send-email-holler@ahsoftware.de> X-Mailer: git-send-email 1.8.1.5 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3694 Lines: 100 Current Renesas Electronics XHCI hosts (which were formerly NEC) do support the same vendor command to show the firmware. Rename the ambigious named define XHCI_NEC_HOST to XHCI_NEC_SHOW_FW because it's only used to display the firmware version. Besides that, change the output "... NEC firmware version x.y" to "... firmware version x.y" to not confuse owners of Renesas USB hosts. (so only cosmetic, no functional changes) Signed-off-by: Alexander Holler --- drivers/usb/host/xhci-pci.c | 2 +- drivers/usb/host/xhci-ring.c | 6 +++--- drivers/usb/host/xhci.c | 4 ++-- drivers/usb/host/xhci.h | 2 +- 4 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/usb/host/xhci-pci.c b/drivers/usb/host/xhci-pci.c index 1a30c38..f47d3d2 100644 --- a/drivers/usb/host/xhci-pci.c +++ b/drivers/usb/host/xhci-pci.c @@ -79,7 +79,7 @@ static void xhci_pci_quirks(struct device *dev, struct xhci_hcd *xhci) } if (pdev->vendor == PCI_VENDOR_ID_NEC) - xhci->quirks |= XHCI_NEC_HOST; + xhci->quirks |= XHCI_NEC_SHOW_FW; if (pdev->vendor == PCI_VENDOR_ID_AMD && xhci->hci_version == 0x96) xhci->quirks |= XHCI_AMD_0x96_HOST; diff --git a/drivers/usb/host/xhci-ring.c b/drivers/usb/host/xhci-ring.c index 1969c00..761d566 100644 --- a/drivers/usb/host/xhci-ring.c +++ b/drivers/usb/host/xhci-ring.c @@ -1489,11 +1489,11 @@ bandwidth_change: "for disabled slot %u\n", slot_id); break; case TRB_TYPE(TRB_NEC_GET_FW): - if (!(xhci->quirks & XHCI_NEC_HOST)) { + if (!(xhci->quirks & XHCI_NEC_SHOW_FW)) { xhci->error_bitmask |= 1 << 6; break; } - xhci_dbg(xhci, "NEC firmware version %2x.%02x\n", + xhci_dbg(xhci, "firmware version %2x.%02x\n", NEC_FW_MAJOR(le32_to_cpu(event->status)), NEC_FW_MINOR(le32_to_cpu(event->status))); break; @@ -1512,7 +1512,7 @@ static void handle_vendor_event(struct xhci_hcd *xhci, trb_type = TRB_FIELD_TO_TYPE(le32_to_cpu(event->generic.field[3])); xhci_dbg(xhci, "Vendor specific event TRB type = %u\n", trb_type); - if (trb_type == TRB_NEC_CMD_COMP && (xhci->quirks & XHCI_NEC_HOST)) + if (trb_type == TRB_NEC_CMD_COMP && (xhci->quirks & XHCI_NEC_SHOW_FW)) handle_cmd_completion(xhci, &event->event_cmd); } diff --git a/drivers/usb/host/xhci.c b/drivers/usb/host/xhci.c index b4aa79d..b961e9a 100644 --- a/drivers/usb/host/xhci.c +++ b/drivers/usb/host/xhci.c @@ -588,7 +588,7 @@ static int xhci_run_finished(struct xhci_hcd *xhci) xhci->shared_hcd->state = HC_STATE_RUNNING; xhci->cmd_ring_state = CMD_RING_STATE_RUNNING; - if (xhci->quirks & XHCI_NEC_HOST) + if (xhci->quirks & XHCI_NEC_SHOW_FW) xhci_ring_cmd_db(xhci); xhci_dbg(xhci, "Finished xhci_run for USB3 roothub\n"); @@ -673,7 +673,7 @@ int xhci_run(struct usb_hcd *hcd) &xhci->ir_set->irq_pending); xhci_print_ir_set(xhci, 0); - if (xhci->quirks & XHCI_NEC_HOST) + if (xhci->quirks & XHCI_NEC_SHOW_FW) xhci_queue_vendor_command(xhci, 0, 0, 0, TRB_TYPE(TRB_NEC_GET_FW)); diff --git a/drivers/usb/host/xhci.h b/drivers/usb/host/xhci.h index 29c978e..e483f12 100644 --- a/drivers/usb/host/xhci.h +++ b/drivers/usb/host/xhci.h @@ -1493,7 +1493,7 @@ struct xhci_hcd { unsigned int quirks; #define XHCI_LINK_TRB_QUIRK (1 << 0) #define XHCI_RESET_EP_QUIRK (1 << 1) -#define XHCI_NEC_HOST (1 << 2) +#define XHCI_NEC_SHOW_FW (1 << 2) #define XHCI_AMD_PLL_FIX (1 << 3) #define XHCI_SPURIOUS_SUCCESS (1 << 4) /* -- 1.8.1.5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/