xhci: sort out TRB Endpoint ID bitfield macros

xhci macros that read and write endpoint ID bitfields of TRBs are mixing
the 1-based Endpoint ID as described in the xHCI specification, and
0-based endpoint index used by driver as an array index.

Sort this out by naming macros that deal with 1 based Endpoint ID fields
to *_EP_ID_*, and 0 based endpoint index values to *_EP_INDEX_*.

Signed-off-by: Mathias Nyman <mathias.nyman@linux.intel.com>
Link: https://lore.kernel.org/r/20240626124835.1023046-22-mathias.nyman@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/usb/host/xhci-ring.c b/drivers/usb/host/xhci-ring.c
index 49f8f980776bad4b713a792d573db741a461c482..b7517c3c8059f4afb00e270bcbb288a67fdcffab 100644 (file)
--- a/drivers/usb/host/xhci-ring.c
+++ b/drivers/usb/host/xhci-ring.c
@@ -715,7 +715,7 @@ static int xhci_move_dequeue_past_td(struct xhci_hcd *xhci,
                lower_32_bits(addr) | trb_sct | new_cycle,
                upper_32_bits(addr),
                STREAM_ID_FOR_TRB(stream_id), SLOT_ID_FOR_TRB(slot_id) |
-               EP_ID_FOR_TRB(ep_index) | TRB_TYPE(TRB_SET_DEQ), false);
+               EP_INDEX_FOR_TRB(ep_index) | TRB_TYPE(TRB_SET_DEQ), false);
        if (ret < 0) {
                xhci_free_command(xhci, cmd);
                return ret;
@@ -4379,7 +4379,7 @@ int xhci_queue_stop_endpoint(struct xhci_hcd *xhci, struct xhci_command *cmd,
                             int slot_id, unsigned int ep_index, int suspend)
 {
        u32 trb_slot_id = SLOT_ID_FOR_TRB(slot_id);
-       u32 trb_ep_index = EP_ID_FOR_TRB(ep_index);
+       u32 trb_ep_index = EP_INDEX_FOR_TRB(ep_index);
        u32 type = TRB_TYPE(TRB_STOP_RING);
        u32 trb_suspend = SUSPEND_PORT_FOR_TRB(suspend);
 
@@ -4392,7 +4392,7 @@ int xhci_queue_reset_ep(struct xhci_hcd *xhci, struct xhci_command *cmd,
                        enum xhci_ep_reset_type reset_type)
 {
        u32 trb_slot_id = SLOT_ID_FOR_TRB(slot_id);
-       u32 trb_ep_index = EP_ID_FOR_TRB(ep_index);
+       u32 trb_ep_index = EP_INDEX_FOR_TRB(ep_index);
        u32 type = TRB_TYPE(TRB_RESET_EP);
 
        if (reset_type == EP_SOFT_RESET)

diff --git a/drivers/usb/host/xhci.h b/drivers/usb/host/xhci.h
index 413ce0b3a213d937534da7ce74d5a2b4b150dca0..ebd0afd59a60be205f692de03029bcfe6969c86c 100644 (file)
--- a/drivers/usb/host/xhci.h
+++ b/drivers/usb/host/xhci.h
@@ -806,13 +806,19 @@ struct xhci_transfer_event {
        __le32  flags;
 };
 
+/* Transfer event flags bitfield, also for select command completion events */
+#define TRB_TO_SLOT_ID(p)      (((p) >> 24) & 0xff)
+#define SLOT_ID_FOR_TRB(p)     (((p) & 0xff) << 24)
+
+#define TRB_TO_EP_ID(p)                (((p) >> 16) & 0x1f) /* Endpoint ID 1 - 31 */
+#define EP_ID_FOR_TRB(p)       (((p) & 0x1f) << 16)
+
+#define TRB_TO_EP_INDEX(p)     (TRB_TO_EP_ID(p) - 1) /* Endpoint index 0 - 30 */
+#define EP_INDEX_FOR_TRB(p)    ((((p) + 1) & 0x1f) << 16)
+
 /* Transfer event TRB length bit mask */
-/* bits 0:23 */
 #define        EVENT_TRB_LEN(p)                ((p) & 0xffffff)
 
-/** Transfer Event bit fields **/
-#define        TRB_TO_EP_ID(p) (((p) >> 16) & 0x1f)
-
 /* Completion Code - only applicable for some types of TRBs */
 #define        COMP_CODE_MASK          (0xff << 24)
 #define GET_COMP_CODE(p)       (((p) & COMP_CODE_MASK) >> 24)
@@ -951,8 +957,6 @@ struct xhci_event_cmd {
        __le32 flags;
 };
 
-/* flags bitmasks */
-
 /* Address device - disable SetAddress */
 #define TRB_BSR                (1<<9)
 
@@ -988,13 +992,8 @@ enum xhci_setup_dev {
 
 /* bits 16:23 are the virtual function ID */
 /* bits 24:31 are the slot ID */
-#define TRB_TO_SLOT_ID(p)      (((p) & (0xff<<24)) >> 24)
-#define SLOT_ID_FOR_TRB(p)     (((p) & 0xff) << 24)
 
 /* Stop Endpoint TRB - ep_index to endpoint ID for this TRB */
-#define TRB_TO_EP_INDEX(p)             ((((p) & (0x1f << 16)) >> 16) - 1)
-#define        EP_ID_FOR_TRB(p)                ((((p) + 1) & 0x1f) << 16)
-
 #define SUSPEND_PORT_FOR_TRB(p)                (((p) & 1) << 23)
 #define TRB_TO_SUSPEND_PORT(p)         (((p) & (1 << 23)) >> 23)
 #define LAST_EP_INDEX                  30
@@ -2025,8 +2024,7 @@ static inline const char *xhci_decode_trb(char *str, size_t size,
                        field1, field0,
                        xhci_trb_comp_code_string(GET_COMP_CODE(field2)),
                        EVENT_TRB_LEN(field2), TRB_TO_SLOT_ID(field3),
-                       /* Macro decrements 1, maybe it shouldn't?!? */
-                       TRB_TO_EP_INDEX(field3) + 1,
+                       TRB_TO_EP_ID(field3),
                        xhci_trb_type_string(type),
                        field3 & EVENT_DATA ? 'E' : 'e',
                        field3 & TRB_CYCLE ? 'C' : 'c');
@@ -2141,8 +2139,7 @@ static inline const char *xhci_decode_trb(char *str, size_t size,
                        xhci_trb_type_string(type),
                        field1, field0,
                        TRB_TO_SLOT_ID(field3),
-                       /* Macro decrements 1, maybe it shouldn't?!? */
-                       TRB_TO_EP_INDEX(field3) + 1,
+                       TRB_TO_EP_ID(field3),
                        field3 & TRB_TSP ? 'T' : 't',
                        field3 & TRB_CYCLE ? 'C' : 'c');
                break;
@@ -2152,8 +2149,7 @@ static inline const char *xhci_decode_trb(char *str, size_t size,
                        xhci_trb_type_string(type),
                        TRB_TO_SLOT_ID(field3),
                        TRB_TO_SUSPEND_PORT(field3),
-                       /* Macro decrements 1, maybe it shouldn't?!? */
-                       TRB_TO_EP_INDEX(field3) + 1,
+                       TRB_TO_EP_ID(field3),
                        field3 & TRB_CYCLE ? 'C' : 'c');
                break;
        case TRB_SET_DEQ:
@@ -2163,8 +2159,7 @@ static inline const char *xhci_decode_trb(char *str, size_t size,
                        field1, field0,
                        TRB_TO_STREAM_ID(field2),
                        TRB_TO_SLOT_ID(field3),
-                       /* Macro decrements 1, maybe it shouldn't?!? */
-                       TRB_TO_EP_INDEX(field3) + 1,
+                       TRB_TO_EP_ID(field3),
                        field3 & TRB_CYCLE ? 'C' : 'c');
                break;
        case TRB_RESET_DEV: