Merge branch 'ipa-reg-versions'
authorDavid S. Miller <davem@davemloft.net>
Fri, 26 Mar 2021 00:22:30 +0000 (17:22 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 26 Mar 2021 00:22:30 +0000 (17:22 -0700)
Alex Elder says:

====================
net: ipa: update registers for other versions

This series updates IPA and GSI register definitions to permit more
versions of IPA hardware to be supported.  Most of the updates are
informational, updating comments to indicate which IPA versions
support each register and field.  But some registers are new and
others are deprecated.  In a few cases register fields are laid
out differently, and in these cases the changes are a little more
substantive.

I won't claim the result is 100% correct, but it's close, and should
allow all IPA versions 3.x through 4.x to be supported by the driver.
====================

Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ipa/gsi.c
drivers/net/ipa/gsi_reg.h
drivers/net/ipa/ipa_interrupt.c
drivers/net/ipa/ipa_main.c
drivers/net/ipa/ipa_reg.h
drivers/net/ipa/ipa_uc.c

index 7f2a8fce5e0dbf8528f87950d44485bd03d49d3d..585574af36ecd3c17de692d774125fb11e96c955 100644 (file)
@@ -701,7 +701,7 @@ static void gsi_evt_ring_program(struct gsi *gsi, u32 evt_ring_id)
        val |= u32_encode_bits(GSI_RING_ELEMENT_SIZE, EV_ELEMENT_SIZE_FMASK);
        iowrite32(val, gsi->virt + GSI_EV_CH_E_CNTXT_0_OFFSET(evt_ring_id));
 
-       val = u32_encode_bits(size, EV_R_LENGTH_FMASK);
+       val = ev_r_length_encoded(gsi->version, size);
        iowrite32(val, gsi->virt + GSI_EV_CH_E_CNTXT_1_OFFSET(evt_ring_id));
 
        /* The context 2 and 3 registers store the low-order and
@@ -801,14 +801,14 @@ static void gsi_channel_program(struct gsi_channel *channel, bool doorbell)
        channel->tre_ring.index = 0;
 
        /* We program all channels as GPI type/protocol */
-       val = u32_encode_bits(GSI_CHANNEL_TYPE_GPI, CHTYPE_PROTOCOL_FMASK);
+       val = chtype_protocol_encoded(gsi->version, GSI_CHANNEL_TYPE_GPI);
        if (channel->toward_ipa)
                val |= CHTYPE_DIR_FMASK;
        val |= u32_encode_bits(channel->evt_ring_id, ERINDEX_FMASK);
        val |= u32_encode_bits(GSI_RING_ELEMENT_SIZE, ELEMENT_SIZE_FMASK);
        iowrite32(val, gsi->virt + GSI_CH_C_CNTXT_0_OFFSET(channel_id));
 
-       val = u32_encode_bits(size, R_LENGTH_FMASK);
+       val = r_length_encoded(gsi->version, size);
        iowrite32(val, gsi->virt + GSI_CH_C_CNTXT_1_OFFSET(channel_id));
 
        /* The context 2 and 3 registers store the low-order and
@@ -842,6 +842,9 @@ static void gsi_channel_program(struct gsi_channel *channel, bool doorbell)
                        val |= u32_encode_bits(GSI_ESCAPE_BUF_ONLY,
                                               PREFETCH_MODE_FMASK);
        }
+       /* All channels set DB_IN_BYTES */
+       if (gsi->version >= IPA_VERSION_4_9)
+               val |= DB_IN_BYTES;
 
        iowrite32(val, gsi->virt + GSI_CH_C_QOS_OFFSET(channel_id));
 
index 1622d8cf8dea407016aa84aa622b69e1c5b33736..b4ac0258d6e1088a4ee15225c6a9f904e04cf365 100644 (file)
                        (0x0000c01c + 0x1000 * (ee))
 
 /* All other register offsets are relative to gsi->virt */
+
+/** enum gsi_channel_type - CHTYPE_PROTOCOL field values in CH_C_CNTXT_0 */
+enum gsi_channel_type {
+       GSI_CHANNEL_TYPE_MHI                    = 0x0,
+       GSI_CHANNEL_TYPE_XHCI                   = 0x1,
+       GSI_CHANNEL_TYPE_GPI                    = 0x2,
+       GSI_CHANNEL_TYPE_XDCI                   = 0x3,
+       GSI_CHANNEL_TYPE_WDI2                   = 0x4,
+       GSI_CHANNEL_TYPE_GCI                    = 0x5,
+       GSI_CHANNEL_TYPE_WDI3                   = 0x6,
+       GSI_CHANNEL_TYPE_MHIP                   = 0x7,
+       GSI_CHANNEL_TYPE_AQC                    = 0x8,
+       GSI_CHANNEL_TYPE_11AD                   = 0x9,
+};
+
 #define GSI_CH_C_CNTXT_0_OFFSET(ch) \
                GSI_EE_N_CH_C_CNTXT_0_OFFSET((ch), GSI_EE_AP)
 #define GSI_EE_N_CH_C_CNTXT_0_OFFSET(ch, ee) \
 #define CHSTATE_FMASK                  GENMASK(23, 20)
 #define ELEMENT_SIZE_FMASK             GENMASK(31, 24)
 
-/** enum gsi_channel_type - CHTYPE_PROTOCOL field values in CH_C_CNTXT_0 */
-enum gsi_channel_type {
-       GSI_CHANNEL_TYPE_MHI                    = 0x0,
-       GSI_CHANNEL_TYPE_XHCI                   = 0x1,
-       GSI_CHANNEL_TYPE_GPI                    = 0x2,
-       GSI_CHANNEL_TYPE_XDCI                   = 0x3,
-};
+/* Encoded value for CH_C_CNTXT_0 register channel protocol fields */
+static inline u32
+chtype_protocol_encoded(enum ipa_version version, enum gsi_channel_type type)
+{
+       u32 val;
+
+       val = u32_encode_bits(type, CHTYPE_PROTOCOL_FMASK);
+       if (version < IPA_VERSION_4_5)
+               return val;
+
+       /* Encode upper bit(s) as well */
+       type >>= hweight32(CHTYPE_PROTOCOL_FMASK);
+       val |= u32_encode_bits(type, CHTYPE_PROTOCOL_MSB_FMASK);
+
+       return val;
+}
 
 #define GSI_CH_C_CNTXT_1_OFFSET(ch) \
                GSI_EE_N_CH_C_CNTXT_1_OFFSET((ch), GSI_EE_AP)
 #define GSI_EE_N_CH_C_CNTXT_1_OFFSET(ch, ee) \
                (0x0001c004 + 0x4000 * (ee) + 0x80 * (ch))
-#define R_LENGTH_FMASK                 GENMASK(15, 0)
+
+/* Encoded value for CH_C_CNTXT_1 register R_LENGTH field */
+static inline u32 r_length_encoded(enum ipa_version version, u32 length)
+{
+       if (version < IPA_VERSION_4_9)
+               return u32_encode_bits(length, GENMASK(15, 0));
+       return u32_encode_bits(length, GENMASK(19, 0));
+}
 
 #define GSI_CH_C_CNTXT_2_OFFSET(ch) \
                GSI_EE_N_CH_C_CNTXT_2_OFFSET((ch), GSI_EE_AP)
@@ -114,6 +145,9 @@ enum gsi_channel_type {
 /* The next two fields are present for IPA v4.5 and above */
 #define PREFETCH_MODE_FMASK            GENMASK(13, 10)
 #define EMPTY_LVL_THRSHOLD_FMASK       GENMASK(23, 16)
+/* The next field is present for IPA v4.9 and above */
+#define DB_IN_BYTES                    GENMASK(24, 24)
+
 /** enum gsi_prefetch_mode - PREFETCH_MODE field in CH_C_QOS */
 enum gsi_prefetch_mode {
        GSI_USE_PREFETCH_BUFS                   = 0x0,
@@ -146,19 +180,25 @@ enum gsi_prefetch_mode {
                GSI_EE_N_EV_CH_E_CNTXT_0_OFFSET((ev), GSI_EE_AP)
 #define GSI_EE_N_EV_CH_E_CNTXT_0_OFFSET(ev, ee) \
                (0x0001d000 + 0x4000 * (ee) + 0x80 * (ev))
+/* enum gsi_channel_type defines EV_CHTYPE field values in EV_CH_E_CNTXT_0 */
 #define EV_CHTYPE_FMASK                        GENMASK(3, 0)
 #define EV_EE_FMASK                    GENMASK(7, 4)
 #define EV_EVCHID_FMASK                        GENMASK(15, 8)
 #define EV_INTYPE_FMASK                        GENMASK(16, 16)
 #define EV_CHSTATE_FMASK               GENMASK(23, 20)
 #define EV_ELEMENT_SIZE_FMASK          GENMASK(31, 24)
-/* enum gsi_channel_type defines EV_CHTYPE field values in EV_CH_E_CNTXT_0 */
 
 #define GSI_EV_CH_E_CNTXT_1_OFFSET(ev) \
                GSI_EE_N_EV_CH_E_CNTXT_1_OFFSET((ev), GSI_EE_AP)
 #define GSI_EE_N_EV_CH_E_CNTXT_1_OFFSET(ev, ee) \
                (0x0001d004 + 0x4000 * (ee) + 0x80 * (ev))
-#define EV_R_LENGTH_FMASK              GENMASK(15, 0)
+/* Encoded value for EV_CH_C_CNTXT_1 register EV_R_LENGTH field */
+static inline u32 ev_r_length_encoded(enum ipa_version version, u32 length)
+{
+       if (version < IPA_VERSION_4_9)
+               return u32_encode_bits(length, GENMASK(15, 0));
+       return u32_encode_bits(length, GENMASK(19, 0));
+}
 
 #define GSI_EV_CH_E_CNTXT_2_OFFSET(ev) \
                GSI_EE_N_EV_CH_E_CNTXT_2_OFFSET((ev), GSI_EE_AP)
@@ -248,6 +288,7 @@ enum gsi_ch_cmd_opcode {
        GSI_CH_STOP                             = 0x2,
        GSI_CH_RESET                            = 0x9,
        GSI_CH_DE_ALLOC                         = 0xa,
+       GSI_CH_DB_STOP                          = 0xb,
 };
 
 #define GSI_EV_CH_CMD_OFFSET \
@@ -278,6 +319,7 @@ enum gsi_generic_cmd_opcode {
        GSI_GENERIC_ALLOCATE_CHANNEL            = 0x2,
 };
 
+/* The next register is present for IPA v3.5.1 and above */
 #define GSI_GSI_HW_PARAM_2_OFFSET \
                        GSI_EE_N_GSI_HW_PARAM_2_OFFSET(GSI_EE_AP)
 #define GSI_EE_N_GSI_HW_PARAM_2_OFFSET(ee) \
@@ -300,7 +342,7 @@ enum gsi_generic_cmd_opcode {
 enum gsi_iram_size {
        IRAM_SIZE_ONE_KB                        = 0x0,
        IRAM_SIZE_TWO_KB                        = 0x1,
-/* The next two values are available for IPA v4.0 and above */
+       /* The next two values are available for IPA v4.0 and above */
        IRAM_SIZE_TWO_N_HALF_KB                 = 0x2,
        IRAM_SIZE_THREE_KB                      = 0x3,
        /* The next two values are available for IPA v4.5 and above */
@@ -424,6 +466,8 @@ enum gsi_general_id {
                        GSI_EE_N_ERROR_LOG_OFFSET(GSI_EE_AP)
 #define GSI_EE_N_ERROR_LOG_OFFSET(ee) \
                        (0x0001f200 + 0x4000 * (ee))
+
+/* Fields below are present for IPA v3.5.1 and above */
 #define ERR_ARG3_FMASK                 GENMASK(3, 0)
 #define ERR_ARG2_FMASK                 GENMASK(7, 4)
 #define ERR_ARG1_FMASK                 GENMASK(11, 8)
@@ -474,7 +518,4 @@ enum gsi_generic_ee_result {
        GENERIC_EE_NO_RESOURCES                 = 0x7,
 };
 
-#define USB_MAX_PACKET_FMASK           GENMASK(15, 15) /* 0: HS; 1: SS */
-#define MHI_BASE_CHANNEL_FMASK         GENMASK(31, 24)
-
 #endif /* _GSI_REG_H_ */
index 61dd7605bcb6616138418e729ceee84c78c0dc6b..c46df0b7c4e5047db31f987410409ea99095115a 100644 (file)
@@ -54,12 +54,14 @@ static void ipa_interrupt_process(struct ipa_interrupt *interrupt, u32 irq_id)
        bool uc_irq = ipa_interrupt_uc(interrupt, irq_id);
        struct ipa *ipa = interrupt->ipa;
        u32 mask = BIT(irq_id);
+       u32 offset;
 
        /* For microcontroller interrupts, clear the interrupt right away,
         * "to avoid clearing unhandled interrupts."
         */
+       offset = ipa_reg_irq_clr_offset(ipa->version);
        if (uc_irq)
-               iowrite32(mask, ipa->reg_virt + IPA_REG_IRQ_CLR_OFFSET);
+               iowrite32(mask, ipa->reg_virt + offset);
 
        if (irq_id < IPA_IRQ_COUNT && interrupt->handler[irq_id])
                interrupt->handler[irq_id](interrupt->ipa, irq_id);
@@ -69,7 +71,7 @@ static void ipa_interrupt_process(struct ipa_interrupt *interrupt, u32 irq_id)
         * so defer clearing until after the handler has been called.
         */
        if (!uc_irq)
-               iowrite32(mask, ipa->reg_virt + IPA_REG_IRQ_CLR_OFFSET);
+               iowrite32(mask, ipa->reg_virt + offset);
 }
 
 /* Process all IPA interrupt types that have been signaled */
@@ -77,13 +79,15 @@ static void ipa_interrupt_process_all(struct ipa_interrupt *interrupt)
 {
        struct ipa *ipa = interrupt->ipa;
        u32 enabled = interrupt->enabled;
+       u32 offset;
        u32 mask;
 
        /* The status register indicates which conditions are present,
         * including conditions whose interrupt is not enabled.  Handle
         * only the enabled ones.
         */
-       mask = ioread32(ipa->reg_virt + IPA_REG_IRQ_STTS_OFFSET);
+       offset = ipa_reg_irq_stts_offset(ipa->version);
+       mask = ioread32(ipa->reg_virt + offset);
        while ((mask &= enabled)) {
                do {
                        u32 irq_id = __ffs(mask);
@@ -92,7 +96,7 @@ static void ipa_interrupt_process_all(struct ipa_interrupt *interrupt)
 
                        ipa_interrupt_process(interrupt, irq_id);
                } while (mask);
-               mask = ioread32(ipa->reg_virt + IPA_REG_IRQ_STTS_OFFSET);
+               mask = ioread32(ipa->reg_virt + offset);
        }
 }
 
@@ -115,14 +119,17 @@ static irqreturn_t ipa_isr(int irq, void *dev_id)
 {
        struct ipa_interrupt *interrupt = dev_id;
        struct ipa *ipa = interrupt->ipa;
+       u32 offset;
        u32 mask;
 
-       mask = ioread32(ipa->reg_virt + IPA_REG_IRQ_STTS_OFFSET);
+       offset = ipa_reg_irq_stts_offset(ipa->version);
+       mask = ioread32(ipa->reg_virt + offset);
        if (mask & interrupt->enabled)
                return IRQ_WAKE_THREAD;
 
        /* Nothing in the mask was supposed to cause an interrupt */
-       iowrite32(mask, ipa->reg_virt + IPA_REG_IRQ_CLR_OFFSET);
+       offset = ipa_reg_irq_clr_offset(ipa->version);
+       iowrite32(mask, ipa->reg_virt + offset);
 
        dev_err(&ipa->pdev->dev, "%s: unexpected interrupt, mask 0x%08x\n",
                __func__, mask);
@@ -136,15 +143,22 @@ static void ipa_interrupt_suspend_control(struct ipa_interrupt *interrupt,
 {
        struct ipa *ipa = interrupt->ipa;
        u32 mask = BIT(endpoint_id);
+       u32 offset;
        u32 val;
 
        /* assert(mask & ipa->available); */
-       val = ioread32(ipa->reg_virt + IPA_REG_IRQ_SUSPEND_EN_OFFSET);
+
+       /* IPA version 3.0 does not support TX_SUSPEND interrupt control */
+       if (ipa->version == IPA_VERSION_3_0)
+               return;
+
+       offset = ipa_reg_irq_suspend_en_offset(ipa->version);
+       val = ioread32(ipa->reg_virt + offset);
        if (enable)
                val |= mask;
        else
                val &= ~mask;
-       iowrite32(val, ipa->reg_virt + IPA_REG_IRQ_SUSPEND_EN_OFFSET);
+       iowrite32(val, ipa->reg_virt + offset);
 }
 
 /* Enable TX_SUSPEND for an endpoint */
@@ -165,10 +179,18 @@ ipa_interrupt_suspend_disable(struct ipa_interrupt *interrupt, u32 endpoint_id)
 void ipa_interrupt_suspend_clear_all(struct ipa_interrupt *interrupt)
 {
        struct ipa *ipa = interrupt->ipa;
+       u32 offset;
        u32 val;
 
-       val = ioread32(ipa->reg_virt + IPA_REG_IRQ_SUSPEND_INFO_OFFSET);
-       iowrite32(val, ipa->reg_virt + IPA_REG_IRQ_SUSPEND_CLR_OFFSET);
+       offset = ipa_reg_irq_suspend_info_offset(ipa->version);
+       val = ioread32(ipa->reg_virt + offset);
+
+       /* SUSPEND interrupt status isn't cleared on IPA version 3.0 */
+       if (ipa->version == IPA_VERSION_3_0)
+               return;
+
+       offset = ipa_reg_irq_suspend_clr_offset(ipa->version);
+       iowrite32(val, ipa->reg_virt + offset);
 }
 
 /* Simulate arrival of an IPA TX_SUSPEND interrupt */
@@ -182,13 +204,15 @@ void ipa_interrupt_add(struct ipa_interrupt *interrupt,
                       enum ipa_irq_id ipa_irq, ipa_irq_handler_t handler)
 {
        struct ipa *ipa = interrupt->ipa;
+       u32 offset;
 
        /* assert(ipa_irq < IPA_IRQ_COUNT); */
        interrupt->handler[ipa_irq] = handler;
 
        /* Update the IPA interrupt mask to enable it */
        interrupt->enabled |= BIT(ipa_irq);
-       iowrite32(interrupt->enabled, ipa->reg_virt + IPA_REG_IRQ_EN_OFFSET);
+       offset = ipa_reg_irq_en_offset(ipa->version);
+       iowrite32(interrupt->enabled, ipa->reg_virt + offset);
 }
 
 /* Remove the handler for an IPA interrupt type */
@@ -196,11 +220,13 @@ void
 ipa_interrupt_remove(struct ipa_interrupt *interrupt, enum ipa_irq_id ipa_irq)
 {
        struct ipa *ipa = interrupt->ipa;
+       u32 offset;
 
        /* assert(ipa_irq < IPA_IRQ_COUNT); */
        /* Update the IPA interrupt mask to disable it */
        interrupt->enabled &= ~BIT(ipa_irq);
-       iowrite32(interrupt->enabled, ipa->reg_virt + IPA_REG_IRQ_EN_OFFSET);
+       offset = ipa_reg_irq_en_offset(ipa->version);
+       iowrite32(interrupt->enabled, ipa->reg_virt + offset);
 
        interrupt->handler[ipa_irq] = NULL;
 }
@@ -211,6 +237,7 @@ struct ipa_interrupt *ipa_interrupt_setup(struct ipa *ipa)
        struct device *dev = &ipa->pdev->dev;
        struct ipa_interrupt *interrupt;
        unsigned int irq;
+       u32 offset;
        int ret;
 
        ret = platform_get_irq_byname(ipa->pdev, "ipa");
@@ -228,7 +255,8 @@ struct ipa_interrupt *ipa_interrupt_setup(struct ipa *ipa)
        interrupt->irq = irq;
 
        /* Start with all IPA interrupts disabled */
-       iowrite32(0, ipa->reg_virt + IPA_REG_IRQ_EN_OFFSET);
+       offset = ipa_reg_irq_en_offset(ipa->version);
+       iowrite32(0, ipa->reg_virt + offset);
 
        ret = request_threaded_irq(irq, ipa_isr, ipa_isr_thread, IRQF_ONESHOT,
                                   "ipa", interrupt);
index ba1bfc30210a3aab2a40a2efb2287f61bc4d050f..f071e90de5409e1c9be43ac1d0a167f5117a7a9c 100644 (file)
@@ -222,7 +222,7 @@ static void ipa_teardown(struct ipa *ipa)
        gsi_teardown(&ipa->gsi);
 }
 
-/* Configure QMB Core Master Port selection */
+/* Configure bus access behavior for IPA components */
 static void ipa_hardware_config_comp(struct ipa *ipa)
 {
        u32 val;
index 86fe2978e81028d18b58006bafd20a21c2187b4a..8820e08d2535ec9c4f18327819f55a4459dfc9fc 100644 (file)
@@ -66,14 +66,16 @@ struct ipa;
  */
 
 #define IPA_REG_COMP_CFG_OFFSET                                0x0000003c
-/* The next field is not supported for IPA v4.1 */
+/* The next field is not supported for IPA v4.0+, not present for IPA v4.5+ */
 #define ENABLE_FMASK                           GENMASK(0, 0)
+/* The next field is present for IPA v4.7+ */
+#define RAM_ARB_PRI_CLIENT_SAMP_FIX_DIS_FMASK  GENMASK(0, 0)
 #define GSI_SNOC_BYPASS_DIS_FMASK              GENMASK(1, 1)
 #define GEN_QMB_0_SNOC_BYPASS_DIS_FMASK                GENMASK(2, 2)
 #define GEN_QMB_1_SNOC_BYPASS_DIS_FMASK                GENMASK(3, 3)
-/* The next field is not present for IPA v4.5 */
+/* The next field is not present for IPA v4.5+ */
 #define IPA_DCMP_FAST_CLK_EN_FMASK             GENMASK(4, 4)
-/* The remaining fields are not present for IPA v3.5.1 */
+/* The next twelve fields are present for IPA v4.0+ */
 #define IPA_QMB_SELECT_CONS_EN_FMASK           GENMASK(5, 5)
 #define IPA_QMB_SELECT_PROD_EN_FMASK           GENMASK(6, 6)
 #define GSI_MULTI_INORDER_RD_DIS_FMASK         GENMASK(7, 7)
@@ -86,9 +88,41 @@ struct ipa;
 #define GSI_SNOC_CNOC_LOOP_PROT_DISABLE_FMASK  GENMASK(14, 14)
 #define GSI_MULTI_AXI_MASTERS_DIS_FMASK                GENMASK(15, 15)
 #define IPA_QMB_SELECT_GLOBAL_EN_FMASK         GENMASK(16, 16)
-#define IPA_ATOMIC_FETCHER_ARB_LOCK_DIS_FMASK  GENMASK(20, 17)
-/* The next field is present for IPA v4.5 */
-#define IPA_FULL_FLUSH_WAIT_RSC_CLOSE_EN_FMASK GENMASK(21, 21)
+/* The next five fields are present for IPA v4.9+ */
+#define QMB_RAM_RD_CACHE_DISABLE_FMASK         GENMASK(19, 19)
+#define GENQMB_AOOOWR_FMASK                    GENMASK(20, 20)
+#define IF_OUT_OF_BUF_STOP_RESET_MASK_EN_FMASK GENMASK(21, 21)
+#define GEN_QMB_1_DYNAMIC_ASIZE_FMASK          GENMASK(30, 30)
+#define GEN_QMB_0_DYNAMIC_ASIZE_FMASK          GENMASK(31, 31)
+
+/* Encoded value for COMP_CFG register ATOMIC_FETCHER_ARB_LOCK_DIS field */
+static inline u32 arbitration_lock_disable_encoded(enum ipa_version version,
+                                                  u32 mask)
+{
+       /* assert(version >= IPA_VERSION_4_0); */
+
+       if (version < IPA_VERSION_4_9)
+               return u32_encode_bits(mask, GENMASK(20, 17));
+
+       if (version == IPA_VERSION_4_9)
+               return u32_encode_bits(mask, GENMASK(24, 22));
+
+       return u32_encode_bits(mask, GENMASK(23, 22));
+}
+
+/* Encoded value for COMP_CFG register FULL_FLUSH_WAIT_RS_CLOSURE_EN field */
+static inline u32 full_flush_rsc_closure_en_encoded(enum ipa_version version,
+                                                   bool enable)
+{
+       u32 val = enable ? 1 : 0;
+
+       /* assert(version >= IPA_VERSION_4_5); */
+
+       if (version == IPA_VERSION_4_5 || version == IPA_VERSION_4_7)
+               return u32_encode_bits(val, GENMASK(21, 21));
+
+       return u32_encode_bits(val, GENMASK(17, 17));
+}
 
 #define IPA_REG_CLKON_CFG_OFFSET                       0x00000044
 #define RX_FMASK                               GENMASK(0, 0)
@@ -108,13 +142,15 @@ struct ipa;
 #define ACK_MNGR_FMASK                         GENMASK(14, 14)
 #define D_DCPH_FMASK                           GENMASK(15, 15)
 #define H_DCPH_FMASK                           GENMASK(16, 16)
-/* The next field is not present for IPA v4.5 */
+/* The next field is not present for IPA v4.5+ */
 #define DCMP_FMASK                             GENMASK(17, 17)
+/* The next three fields are present for IPA v3.5+ */
 #define NTF_TX_CMDQS_FMASK                     GENMASK(18, 18)
 #define TX_0_FMASK                             GENMASK(19, 19)
 #define TX_1_FMASK                             GENMASK(20, 20)
+/* The next field is present for IPA v3.5.1+ */
 #define FNR_FMASK                              GENMASK(21, 21)
-/* The remaining fields are not present for IPA v3.5.1 */
+/* The next eight fields are present for IPA v4.0+ */
 #define QSB2AXI_CMDQ_L_FMASK                   GENMASK(22, 22)
 #define AGGR_WRAPPER_FMASK                     GENMASK(23, 23)
 #define RAM_SLAVEWAY_FMASK                     GENMASK(24, 24)
@@ -123,8 +159,10 @@ struct ipa;
 #define GSI_IF_FMASK                           GENMASK(27, 27)
 #define GLOBAL_FMASK                           GENMASK(28, 28)
 #define GLOBAL_2X_CLK_FMASK                    GENMASK(29, 29)
-/* The next field is present for IPA v4.5 */
+/* The next field is present for IPA v4.5+ */
 #define DPL_FIFO_FMASK                         GENMASK(30, 30)
+/* The next field is present for IPA v4.7+ */
+#define DRBIP_FMASK                            GENMASK(31, 31)
 
 #define IPA_REG_ROUTE_OFFSET                           0x00000048
 #define ROUTE_DIS_FMASK                                GENMASK(0, 0)
@@ -145,13 +183,13 @@ struct ipa;
 #define IPA_REG_QSB_MAX_READS_OFFSET                   0x00000078
 #define GEN_QMB_0_MAX_READS_FMASK              GENMASK(3, 0)
 #define GEN_QMB_1_MAX_READS_FMASK              GENMASK(7, 4)
-/* The next two fields are not present for IPA v3.5.1 */
+/* The next two fields are present for IPA v4.0+ */
 #define GEN_QMB_0_MAX_READS_BEATS_FMASK                GENMASK(23, 16)
 #define GEN_QMB_1_MAX_READS_BEATS_FMASK                GENMASK(31, 24)
 
 static inline u32 ipa_reg_filt_rout_hash_en_offset(enum ipa_version version)
 {
-       if (version == IPA_VERSION_3_5_1)
+       if (version < IPA_VERSION_4_0)
                return 0x000008c;
 
        return 0x0000148;
@@ -159,7 +197,7 @@ static inline u32 ipa_reg_filt_rout_hash_en_offset(enum ipa_version version)
 
 static inline u32 ipa_reg_filt_rout_hash_flush_offset(enum ipa_version version)
 {
-       if (version == IPA_VERSION_3_5_1)
+       if (version < IPA_VERSION_4_0)
                return 0x0000090;
 
        return 0x000014c;
@@ -174,22 +212,23 @@ static inline u32 ipa_reg_filt_rout_hash_flush_offset(enum ipa_version version)
 /* ipa->available defines the valid bits in the STATE_AGGR_ACTIVE register */
 static inline u32 ipa_reg_state_aggr_active_offset(enum ipa_version version)
 {
-       if (version == IPA_VERSION_3_5_1)
+       if (version < IPA_VERSION_4_0)
                return 0x0000010c;
 
        return 0x000000b4;
 }
 
-/* The next register is not present for IPA v4.5 */
+/* The next register is not present for IPA v4.5+ */
 #define IPA_REG_BCR_OFFSET                             0x000001d0
-/* The next two fields are not present for IPA v4.2 */
+/* The next two fields are not present for IPA v4.2+ */
 #define BCR_CMDQ_L_LACK_ONE_ENTRY_FMASK                GENMASK(0, 0)
 #define BCR_TX_NOT_USING_BRESP_FMASK           GENMASK(1, 1)
-/* The next field is invalid for IPA v4.1 */
+/* The next field is invalid for IPA v4.0+ */
 #define BCR_TX_SUSPEND_IRQ_ASSERT_ONCE_FMASK   GENMASK(2, 2)
-/* The next two fields are not present for IPA v4.2 */
+/* The next two fields are not present for IPA v4.2+ */
 #define BCR_SUSPEND_L2_IRQ_FMASK               GENMASK(3, 3)
 #define BCR_HOLB_DROP_L2_IRQ_FMASK             GENMASK(4, 4)
+/* The next five fields are present for IPA v3.5+ */
 #define BCR_DUAL_TX_FMASK                      GENMASK(5, 5)
 #define BCR_ENABLE_FILTER_DATA_CACHE_FMASK     GENMASK(6, 6)
 #define BCR_NOTIF_PRIORITY_OVER_ZLT_FMASK      GENMASK(7, 7)
@@ -233,35 +272,40 @@ static inline u32 proc_cntxt_base_addr_encoded(enum ipa_version version,
 /* ipa->available defines the valid bits in the AGGR_FORCE_CLOSE register */
 #define IPA_REG_AGGR_FORCE_CLOSE_OFFSET                        0x000001ec
 
-/* The next register is not present for IPA v4.5 */
+/* The next register is not present for IPA v4.5+ */
 #define IPA_REG_COUNTER_CFG_OFFSET                     0x000001f0
+/* The next field is not present for IPA v3.5+ */
+#define EOT_COAL_GRANULARITY                   GENMASK(3, 0)
 #define AGGR_GRANULARITY_FMASK                 GENMASK(8, 4)
 
-/* The next register is not present for IPA v4.5 */
+/* The next register is present for IPA v3.5+ */
 #define IPA_REG_TX_CFG_OFFSET                          0x000001fc
-/* The first three fields are present for IPA v3.5.1 only */
+/* The next three fields are not present for IPA v4.0+ */
 #define TX0_PREFETCH_DISABLE_FMASK             GENMASK(0, 0)
 #define TX1_PREFETCH_DISABLE_FMASK             GENMASK(1, 1)
 #define PREFETCH_ALMOST_EMPTY_SIZE_FMASK       GENMASK(4, 2)
-/* The next six fields are present for IPA v4.0 and above */
+/* The next six fields are present for IPA v4.0+ */
 #define PREFETCH_ALMOST_EMPTY_SIZE_TX0_FMASK   GENMASK(5, 2)
 #define DMAW_SCND_OUTSD_PRED_THRESHOLD_FMASK   GENMASK(9, 6)
 #define DMAW_SCND_OUTSD_PRED_EN_FMASK          GENMASK(10, 10)
 #define DMAW_MAX_BEATS_256_DIS_FMASK           GENMASK(11, 11)
 #define PA_MASK_EN_FMASK                       GENMASK(12, 12)
 #define PREFETCH_ALMOST_EMPTY_SIZE_TX1_FMASK   GENMASK(16, 13)
-/* The next field is present for IPA v4.5 */
+/* The next field is present for IPA v4.5+ */
 #define DUAL_TX_ENABLE_FMASK                   GENMASK(17, 17)
-/* The next two fields are present for IPA v4.2 only */
+/* The next field is present for IPA v4.2+, but not IPA v4.5 */
 #define SSPND_PA_NO_START_STATE_FMASK          GENMASK(18, 18)
+/* The next field is present for IPA v4.2 only */
 #define SSPND_PA_NO_BQ_STATE_FMASK             GENMASK(19, 19)
 
+/* The next register is present for IPA v3.5+ */
 #define IPA_REG_FLAVOR_0_OFFSET                                0x00000210
 #define IPA_MAX_PIPES_FMASK                    GENMASK(3, 0)
 #define IPA_MAX_CONS_PIPES_FMASK               GENMASK(12, 8)
 #define IPA_MAX_PROD_PIPES_FMASK               GENMASK(20, 16)
 #define IPA_PROD_LOWEST_FMASK                  GENMASK(27, 24)
 
+/* The next register is present for IPA v3.5+ */
 static inline u32 ipa_reg_idle_indication_cfg_offset(enum ipa_version version)
 {
        if (version >= IPA_VERSION_4_2)
@@ -273,19 +317,19 @@ static inline u32 ipa_reg_idle_indication_cfg_offset(enum ipa_version version)
 #define ENTER_IDLE_DEBOUNCE_THRESH_FMASK       GENMASK(15, 0)
 #define CONST_NON_IDLE_ENABLE_FMASK            GENMASK(16, 16)
 
-/* The next register is present for IPA v4.5 */
+/* The next register is present for IPA v4.5+ */
 #define IPA_REG_QTIME_TIMESTAMP_CFG_OFFSET             0x0000024c
 #define DPL_TIMESTAMP_LSB_FMASK                        GENMASK(4, 0)
 #define DPL_TIMESTAMP_SEL_FMASK                        GENMASK(7, 7)
 #define TAG_TIMESTAMP_LSB_FMASK                        GENMASK(12, 8)
 #define NAT_TIMESTAMP_LSB_FMASK                        GENMASK(20, 16)
 
-/* The next register is present for IPA v4.5 */
+/* The next register is present for IPA v4.5+ */
 #define IPA_REG_TIMERS_XO_CLK_DIV_CFG_OFFSET           0x00000250
 #define DIV_VALUE_FMASK                                GENMASK(8, 0)
 #define DIV_ENABLE_FMASK                       GENMASK(31, 31)
 
-/* The next register is present for IPA v4.5 */
+/* The next register is present for IPA v4.5+ */
 #define IPA_REG_TIMERS_PULSE_GRAN_CFG_OFFSET           0x00000254
 #define GRAN_0_FMASK                           GENMASK(2, 0)
 #define GRAN_1_FMASK                           GENMASK(5, 3)
@@ -344,19 +388,17 @@ static inline u32 ipa_resource_group_dst_count(enum ipa_version version)
        }
 }
 
-/* Not all of the following are valid (depends on the count, above) */
+/* Not all of the following are present (depends on IPA version) */
 #define IPA_REG_SRC_RSRC_GRP_01_RSRC_TYPE_N_OFFSET(rt) \
                                        (0x00000400 + 0x0020 * (rt))
 #define IPA_REG_SRC_RSRC_GRP_23_RSRC_TYPE_N_OFFSET(rt) \
                                        (0x00000404 + 0x0020 * (rt))
-/* The next register is only present for IPA v4.5 */
 #define IPA_REG_SRC_RSRC_GRP_45_RSRC_TYPE_N_OFFSET(rt) \
                                        (0x00000408 + 0x0020 * (rt))
 #define IPA_REG_DST_RSRC_GRP_01_RSRC_TYPE_N_OFFSET(rt) \
                                        (0x00000500 + 0x0020 * (rt))
 #define IPA_REG_DST_RSRC_GRP_23_RSRC_TYPE_N_OFFSET(rt) \
                                        (0x00000504 + 0x0020 * (rt))
-/* The next register is only present for IPA v4.5 */
 #define IPA_REG_DST_RSRC_GRP_45_RSRC_TYPE_N_OFFSET(rt) \
                                        (0x00000508 + 0x0020 * (rt))
 /* The next four fields are used for all resource group registers */
@@ -368,8 +410,9 @@ static inline u32 ipa_resource_group_dst_count(enum ipa_version version)
 
 #define IPA_REG_ENDP_INIT_CTRL_N_OFFSET(ep) \
                                        (0x00000800 + 0x0070 * (ep))
-/* The next field should only used for IPA v3.5.1 */
+/* Valid only for RX (IPA producer) endpoints (do not use for IPA v4.0+) */
 #define ENDP_SUSPEND_FMASK                     GENMASK(0, 0)
+/* Valid only for TX (IPA consumer) endpoints */
 #define ENDP_DELAY_FMASK                       GENMASK(1, 1)
 
 #define IPA_REG_ENDP_INIT_CFG_N_OFFSET(ep) \
@@ -379,11 +422,11 @@ static inline u32 ipa_resource_group_dst_count(enum ipa_version version)
 #define CS_METADATA_HDR_OFFSET_FMASK           GENMASK(6, 3)
 #define CS_GEN_QMB_MASTER_SEL_FMASK            GENMASK(8, 8)
 
-/** enum ipa_cs_offload_en - checksum offload field in ENDP_INIT_CFG_N */
+/** enum ipa_cs_offload_en - ENDP_INIT_CFG register CS_OFFLOAD_EN field value */
 enum ipa_cs_offload_en {
        IPA_CS_OFFLOAD_NONE             = 0x0,
-       IPA_CS_OFFLOAD_UL               = 0x1,
-       IPA_CS_OFFLOAD_DL               = 0x2,
+       IPA_CS_OFFLOAD_UL               = 0x1,  /* Before IPA v4.5 (TX) */
+       IPA_CS_OFFLOAD_DL               = 0x2,  /* Before IPA v4.5 (RX) */
 };
 
 /* Valid only for TX (IPA consumer) endpoints */
@@ -406,11 +449,12 @@ enum ipa_nat_en {
 #define HDR_ADDITIONAL_CONST_LEN_FMASK         GENMASK(18, 13)
 #define HDR_OFST_PKT_SIZE_VALID_FMASK          GENMASK(19, 19)
 #define HDR_OFST_PKT_SIZE_FMASK                        GENMASK(25, 20)
+/* The next field is not present for IPA v4.9+ */
 #define HDR_A5_MUX_FMASK                       GENMASK(26, 26)
 #define HDR_LEN_INC_DEAGG_HDR_FMASK            GENMASK(27, 27)
-/* The next field is not present for IPA v4.5 */
+/* The next field is not present for IPA v4.5+ */
 #define HDR_METADATA_REG_VALID_FMASK           GENMASK(28, 28)
-/* The next two fields are present for IPA v4.5 */
+/* The next two fields are present for IPA v4.5+ */
 #define HDR_LEN_MSB_FMASK                      GENMASK(29, 28)
 #define HDR_OFST_METADATA_MSB_FMASK            GENMASK(31, 30)
 
@@ -462,7 +506,7 @@ static inline u32 ipa_metadata_offset_encoded(enum ipa_version version,
 #define HDR_PAYLOAD_LEN_INC_PADDING_FMASK      GENMASK(3, 3)
 #define HDR_TOTAL_LEN_OR_PAD_OFFSET_FMASK      GENMASK(9, 4)
 #define HDR_PAD_TO_ALIGNMENT_FMASK             GENMASK(13, 10)
-/* The next three fields are present for IPA v4.5 */
+/* The next three fields are present for IPA v4.5+ */
 #define HDR_TOTAL_LEN_OR_PAD_OFFSET_MSB_FMASK  GENMASK(17, 16)
 #define HDR_OFST_PKT_SIZE_MSB_FMASK            GENMASK(19, 18)
 #define HDR_ADDITIONAL_CONST_LEN_MSB_FMASK     GENMASK(21, 20)
@@ -475,16 +519,18 @@ static inline u32 ipa_metadata_offset_encoded(enum ipa_version version,
 #define IPA_REG_ENDP_INIT_MODE_N_OFFSET(txep) \
                                        (0x00000820 + 0x0070 * (txep))
 #define MODE_FMASK                             GENMASK(2, 0)
-/* The next field is present for IPA v4.5 */
+/* The next field is present for IPA v4.5+ */
 #define DCPH_ENABLE_FMASK                      GENMASK(3, 3)
 #define DEST_PIPE_INDEX_FMASK                  GENMASK(8, 4)
 #define BYTE_THRESHOLD_FMASK                   GENMASK(27, 12)
 #define PIPE_REPLICATION_EN_FMASK              GENMASK(28, 28)
 #define PAD_EN_FMASK                           GENMASK(29, 29)
-/* The next register is not present for IPA v4.5 */
+/* The next field is not present for IPA v4.5+ */
 #define HDR_FTCH_DISABLE_FMASK                 GENMASK(30, 30)
+/* The next field is present for IPA v4.9+ */
+#define DRBIP_ACL_ENABLE                       GENMASK(30, 30)
 
-/** enum ipa_mode - mode field in ENDP_INIT_MODE_N */
+/** enum ipa_mode - ENDP_INIT_MODE register MODE field value */
 enum ipa_mode {
        IPA_BASIC                       = 0x0,
        IPA_ENABLE_FRAMING_HDLC         = 0x1,
@@ -496,47 +542,54 @@ enum ipa_mode {
                                        (0x00000824 +  0x0070 * (ep))
 #define AGGR_EN_FMASK                          GENMASK(1, 0)
 #define AGGR_TYPE_FMASK                                GENMASK(4, 2)
+
+/* The legacy value is used for IPA hardware before IPA v4.5 */
 static inline u32 aggr_byte_limit_fmask(bool legacy)
 {
        return legacy ? GENMASK(9, 5) : GENMASK(10, 5);
 }
 
+/* The legacy value is used for IPA hardware before IPA v4.5 */
 static inline u32 aggr_time_limit_fmask(bool legacy)
 {
        return legacy ? GENMASK(14, 10) : GENMASK(16, 12);
 }
 
+/* The legacy value is used for IPA hardware before IPA v4.5 */
 static inline u32 aggr_pkt_limit_fmask(bool legacy)
 {
        return legacy ? GENMASK(20, 15) : GENMASK(22, 17);
 }
 
+/* The legacy value is used for IPA hardware before IPA v4.5 */
 static inline u32 aggr_sw_eof_active_fmask(bool legacy)
 {
        return legacy ? GENMASK(21, 21) : GENMASK(23, 23);
 }
 
+/* The legacy value is used for IPA hardware before IPA v4.5 */
 static inline u32 aggr_force_close_fmask(bool legacy)
 {
        return legacy ? GENMASK(22, 22) : GENMASK(24, 24);
 }
 
+/* The legacy value is used for IPA hardware before IPA v4.5 */
 static inline u32 aggr_hard_byte_limit_enable_fmask(bool legacy)
 {
        return legacy ? GENMASK(24, 24) : GENMASK(26, 26);
 }
 
-/* The next field is present for IPA v4.5 */
+/* The next field is present for IPA v4.5+ */
 #define AGGR_GRAN_SEL_FMASK                    GENMASK(27, 27)
 
-/** enum ipa_aggr_en - aggregation enable field in ENDP_INIT_AGGR_N */
+/** enum ipa_aggr_en - ENDP_INIT_AGGR register AGGR_EN field value */
 enum ipa_aggr_en {
-       IPA_BYPASS_AGGR                 = 0x0,
-       IPA_ENABLE_AGGR                 = 0x1,
-       IPA_ENABLE_DEAGGR               = 0x2,
+       IPA_BYPASS_AGGR                 = 0x0,  /* (TX, RX) */
+       IPA_ENABLE_AGGR                 = 0x1,  /* (RX) */
+       IPA_ENABLE_DEAGGR               = 0x2,  /* (TX) */
 };
 
-/** enum ipa_aggr_type - aggregation type field in ENDP_INIT_AGGR_N */
+/** enum ipa_aggr_type - ENDP_INIT_AGGR register AGGR_TYPE field value */
 enum ipa_aggr_type {
        IPA_MBIM_16                     = 0x0,
        IPA_HDLC                        = 0x1,
@@ -577,14 +630,13 @@ enum ipa_aggr_type {
 /* Encoded value for ENDP_INIT_RSRC_GRP register RSRC_GRP field */
 static inline u32 rsrc_grp_encoded(enum ipa_version version, u32 rsrc_grp)
 {
-       switch (version) {
-       case IPA_VERSION_4_2:
-               return u32_encode_bits(rsrc_grp, GENMASK(0, 0));
-       case IPA_VERSION_4_5:
+       if (version < IPA_VERSION_3_5 || version == IPA_VERSION_4_5)
                return u32_encode_bits(rsrc_grp, GENMASK(2, 0));
-       default:
-               return u32_encode_bits(rsrc_grp, GENMASK(1, 0));
-       }
+
+       if (version == IPA_VERSION_4_2 || version == IPA_VERSION_4_7)
+               return u32_encode_bits(rsrc_grp, GENMASK(0, 0));
+
+       return u32_encode_bits(rsrc_grp, GENMASK(1, 0));
 }
 
 /* Valid only for TX (IPA consumer) endpoints */
@@ -595,6 +647,13 @@ static inline u32 rsrc_grp_encoded(enum ipa_version version, u32 rsrc_grp)
 
 /**
  * enum ipa_seq_type - HPS and DPS sequencer type
+ * @IPA_SEQ_DMA:                Perform DMA only
+ * @IPA_SEQ_1_PASS:             One pass through the pipeline
+ * @IPA_SEQ_2_PASS_SKIP_LAST_UC: Two passes, skip the microcprocessor
+ * @IPA_SEQ_1_PASS_SKIP_LAST_UC: One pass, skip the microcprocessor
+ * @IPA_SEQ_2_PASS:             Two passes through the pipeline
+ * @IPA_SEQ_3_PASS_SKIP_LAST_UC: Three passes, skip the microcprocessor
+ * @IPA_SEQ_DECIPHER:           Optional deciphering step (combined)
  *
  * The low-order byte of the sequencer type register defines the number of
  * passes a packet takes through the IPA pipeline.  The last pass through can
@@ -604,7 +663,6 @@ static inline u32 rsrc_grp_encoded(enum ipa_version version, u32 rsrc_grp)
  * Note: not all combinations of ipa_seq_type and ipa_seq_rep_type are
  * supported (or meaningful).
  */
-#define IPA_SEQ_DECIPHER                       0x11
 enum ipa_seq_type {
        IPA_SEQ_DMA                             = 0x00,
        IPA_SEQ_1_PASS                          = 0x02,
@@ -612,10 +670,13 @@ enum ipa_seq_type {
        IPA_SEQ_1_PASS_SKIP_LAST_UC             = 0x06,
        IPA_SEQ_2_PASS                          = 0x0a,
        IPA_SEQ_3_PASS_SKIP_LAST_UC             = 0x0c,
+       /* The next value can be ORed with the above */
+       IPA_SEQ_DECIPHER                        = 0x11,
 };
 
 /**
  * enum ipa_seq_rep_type - replicated packet sequencer type
+ * @IPA_SEQ_REP_DMA_PARSER:    DMA parser for replicated packets
  *
  * This goes in the second byte of the endpoint sequencer type register.
  *
@@ -630,12 +691,12 @@ enum ipa_seq_rep_type {
                                        (0x00000840 + 0x0070 * (ep))
 #define STATUS_EN_FMASK                                GENMASK(0, 0)
 #define STATUS_ENDP_FMASK                      GENMASK(5, 1)
-/* The next field is not present for IPA v4.5 */
+/* The next field is not present for IPA v4.5+ */
 #define STATUS_LOCATION_FMASK                  GENMASK(8, 8)
-/* The next field is not present for IPA v3.5.1 */
+/* The next field is present for IPA v4.0+ */
 #define STATUS_PKT_SUPPRESS_FMASK              GENMASK(9, 9)
 
-/* The next register is only present for IPA versions that support hashing */
+/* The next register is not present for IPA v4.2 (which no hashing support) */
 #define IPA_REG_ENDP_FILTER_ROUTER_HSH_CFG_N_OFFSET(er) \
                                        (0x0000085c + 0x0070 * (er))
 #define FILTER_HASH_MSK_SRC_ID_FMASK           GENMASK(0, 0)
@@ -656,31 +717,87 @@ enum ipa_seq_rep_type {
 #define ROUTER_HASH_MSK_METADATA_FMASK         GENMASK(22, 22)
 #define IPA_REG_ENDP_ROUTER_HASH_MSK_ALL       GENMASK(22, 16)
 
-#define IPA_REG_IRQ_STTS_OFFSET        \
-                               IPA_REG_IRQ_STTS_EE_N_OFFSET(GSI_EE_AP)
-#define IPA_REG_IRQ_STTS_EE_N_OFFSET(ee) \
-                                       (0x00003008 + 0x1000 * (ee))
+static inline u32 ipa_reg_irq_stts_ee_n_offset(enum ipa_version version,
+                                              u32 ee)
+{
+       if (version < IPA_VERSION_4_9)
+               return 0x00003008 + 0x1000 * ee;
+
+       return 0x00004008 + 0x1000 * ee;
+}
 
-#define IPA_REG_IRQ_EN_OFFSET \
-                               IPA_REG_IRQ_EN_EE_N_OFFSET(GSI_EE_AP)
-#define IPA_REG_IRQ_EN_EE_N_OFFSET(ee) \
-                                       (0x0000300c + 0x1000 * (ee))
+static inline u32 ipa_reg_irq_stts_offset(enum ipa_version version)
+{
+       return ipa_reg_irq_stts_ee_n_offset(version, GSI_EE_AP);
+}
+
+static inline u32 ipa_reg_irq_en_ee_n_offset(enum ipa_version version, u32 ee)
+{
+       if (version < IPA_VERSION_4_9)
+               return 0x0000300c + 0x1000 * ee;
+
+       return 0x0000400c + 0x1000 * ee;
+}
+
+static inline u32 ipa_reg_irq_en_offset(enum ipa_version version)
+{
+       return ipa_reg_irq_en_ee_n_offset(version, GSI_EE_AP);
+}
+
+static inline u32 ipa_reg_irq_clr_ee_n_offset(enum ipa_version version, u32 ee)
+{
+       if (version < IPA_VERSION_4_9)
+               return 0x00003010 + 0x1000 * ee;
+
+       return 0x00004010 + 0x1000 * ee;
+}
+
+static inline u32 ipa_reg_irq_clr_offset(enum ipa_version version)
+{
+       return ipa_reg_irq_clr_ee_n_offset(version, GSI_EE_AP);
+}
 
-#define IPA_REG_IRQ_CLR_OFFSET \
-                               IPA_REG_IRQ_CLR_EE_N_OFFSET(GSI_EE_AP)
-#define IPA_REG_IRQ_CLR_EE_N_OFFSET(ee) \
-                                       (0x00003010 + 0x1000 * (ee))
 /**
  * enum ipa_irq_id - Bit positions representing type of IPA IRQ
  * @IPA_IRQ_UC_0:      Microcontroller event interrupt
  * @IPA_IRQ_UC_1:      Microcontroller response interrupt
  * @IPA_IRQ_TX_SUSPEND:        Data ready interrupt
+ * @IPA_IRQ_COUNT:     Number of IRQ ids (must be last)
  *
  * IRQ types not described above are not currently used.
+ *
+ * @IPA_IRQ_BAD_SNOC_ACCESS:           (Not currently used)
+ * @IPA_IRQ_EOT_COAL:                  (Not currently used)
+ * @IPA_IRQ_UC_2:                      (Not currently used)
+ * @IPA_IRQ_UC_3:                      (Not currently used)
+ * @IPA_IRQ_UC_IN_Q_NOT_EMPTY:         (Not currently used)
+ * @IPA_IRQ_UC_RX_CMD_Q_NOT_FULL:      (Not currently used)
+ * @IPA_IRQ_PROC_UC_ACK_Q_NOT_EMPTY:   (Not currently used)
+ * @IPA_IRQ_RX_ERR:                    (Not currently used)
+ * @IPA_IRQ_DEAGGR_ERR:                        (Not currently used)
+ * @IPA_IRQ_TX_ERR:                    (Not currently used)
+ * @IPA_IRQ_STEP_MODE:                 (Not currently used)
+ * @IPA_IRQ_PROC_ERR:                  (Not currently used)
+ * @IPA_IRQ_TX_HOLB_DROP:              (Not currently used)
+ * @IPA_IRQ_BAM_GSI_IDLE:              (Not currently used)
+ * @IPA_IRQ_PIPE_YELLOW_BELOW:         (Not currently used)
+ * @IPA_IRQ_PIPE_RED_BELOW:            (Not currently used)
+ * @IPA_IRQ_PIPE_YELLOW_ABOVE:         (Not currently used)
+ * @IPA_IRQ_PIPE_RED_ABOVE:            (Not currently used)
+ * @IPA_IRQ_UCP:                       (Not currently used)
+ * @IPA_IRQ_DCMP:                      (Not currently used)
+ * @IPA_IRQ_GSI_EE:                    (Not currently used)
+ * @IPA_IRQ_GSI_IPA_IF_TLV_RCVD:       (Not currently used)
+ * @IPA_IRQ_GSI_UC:                    (Not currently used)
+ * @IPA_IRQ_TLV_LEN_MIN_DSM:           (Not currently used)
+ * @IPA_IRQ_DRBIP_PKT_EXCEED_MAX_SIZE_EN: (Not currently used)
+ * @IPA_IRQ_DRBIP_DATA_SCTR_CFG_ERROR_EN: (Not currently used)
+ * @IPA_IRQ_DRBIP_IMM_CMD_NO_FLSH_HZRD_EN: (Not currently used)
  */
 enum ipa_irq_id {
        IPA_IRQ_BAD_SNOC_ACCESS                 = 0x0,
-       /* Type (bit) 0x1 is not defined */
+       /* The next bit is not present for IPA v3.5+ */
+       IPA_IRQ_EOT_COAL                        = 0x1,
        IPA_IRQ_UC_0                            = 0x2,
        IPA_IRQ_UC_1                            = 0x3,
        IPA_IRQ_UC_2                            = 0x4,
@@ -701,38 +818,89 @@ enum ipa_irq_id {
        IPA_IRQ_PIPE_YELLOW_ABOVE               = 0x13,
        IPA_IRQ_PIPE_RED_ABOVE                  = 0x14,
        IPA_IRQ_UCP                             = 0x15,
+       /* The next bit is not present for IPA v4.5+ */
        IPA_IRQ_DCMP                            = 0x16,
        IPA_IRQ_GSI_EE                          = 0x17,
        IPA_IRQ_GSI_IPA_IF_TLV_RCVD             = 0x18,
        IPA_IRQ_GSI_UC                          = 0x19,
-       /* The next bit is present for IPA v4.5 */
+       /* The next bit is present for IPA v4.5+ */
        IPA_IRQ_TLV_LEN_MIN_DSM                 = 0x1a,
+       /* The next three bits are present for IPA v4.9+ */
+       IPA_IRQ_DRBIP_PKT_EXCEED_MAX_SIZE_EN    = 0x1b,
+       IPA_IRQ_DRBIP_DATA_SCTR_CFG_ERROR_EN    = 0x1c,
+       IPA_IRQ_DRBIP_IMM_CMD_NO_FLSH_HZRD_EN   = 0x1d,
        IPA_IRQ_COUNT,                          /* Last; not an id */
 };
 
-#define IPA_REG_IRQ_UC_OFFSET \
-                               IPA_REG_IRQ_UC_EE_N_OFFSET(GSI_EE_AP)
-#define IPA_REG_IRQ_UC_EE_N_OFFSET(ee) \
-                                       (0x0000301c + 0x1000 * (ee))
+static inline u32 ipa_reg_irq_uc_ee_n_offset(enum ipa_version version, u32 ee)
+{
+       if (version < IPA_VERSION_4_9)
+               return 0x0000301c + 0x1000 * ee;
+
+       return 0x0000401c + 0x1000 * ee;
+}
+
+static inline u32 ipa_reg_irq_uc_offset(enum ipa_version version)
+{
+       return ipa_reg_irq_uc_ee_n_offset(version, GSI_EE_AP);
+}
+
 #define UC_INTR_FMASK                          GENMASK(0, 0)
 
 /* ipa->available defines the valid bits in the SUSPEND_INFO register */
-#define IPA_REG_IRQ_SUSPEND_INFO_OFFSET \
-                               IPA_REG_IRQ_SUSPEND_INFO_EE_N_OFFSET(GSI_EE_AP)
-#define IPA_REG_IRQ_SUSPEND_INFO_EE_N_OFFSET(ee) \
-                                       (0x00003030 + 0x1000 * (ee))
-
-/* ipa->available defines the valid bits in the IRQ_SUSPEND_EN register */
-#define IPA_REG_IRQ_SUSPEND_EN_OFFSET \
-                               IPA_REG_IRQ_SUSPEND_EN_EE_N_OFFSET(GSI_EE_AP)
-#define IPA_REG_IRQ_SUSPEND_EN_EE_N_OFFSET(ee) \
-                                       (0x00003034 + 0x1000 * (ee))
-
-/* ipa->available defines the valid bits in the IRQ_SUSPEND_CLR register */
-#define IPA_REG_IRQ_SUSPEND_CLR_OFFSET \
-                               IPA_REG_IRQ_SUSPEND_CLR_EE_N_OFFSET(GSI_EE_AP)
-#define IPA_REG_IRQ_SUSPEND_CLR_EE_N_OFFSET(ee) \
-                                       (0x00003038 + 0x1000 * (ee))
+static inline u32
+ipa_reg_irq_suspend_info_ee_n_offset(enum ipa_version version, u32 ee)
+{
+       if (version == IPA_VERSION_3_0)
+               return 0x00003098 + 0x1000 * ee;
+
+       if (version < IPA_VERSION_4_9)
+               return 0x00003030 + 0x1000 * ee;
+
+       return 0x00004030 + 0x1000 * ee;
+}
+
+static inline u32
+ipa_reg_irq_suspend_info_offset(enum ipa_version version)
+{
+       return ipa_reg_irq_suspend_info_ee_n_offset(version, GSI_EE_AP);
+}
+
+/* ipa->available defines the valid bits in the SUSPEND_EN register */
+static inline u32
+ipa_reg_irq_suspend_en_ee_n_offset(enum ipa_version version, u32 ee)
+{
+       /* assert(version != IPA_VERSION_3_0); */
+
+       if (version < IPA_VERSION_4_9)
+               return 0x00003034 + 0x1000 * ee;
+
+       return 0x00004034 + 0x1000 * ee;
+}
+
+static inline u32
+ipa_reg_irq_suspend_en_offset(enum ipa_version version)
+{
+       return ipa_reg_irq_suspend_en_ee_n_offset(version, GSI_EE_AP);
+}
+
+/* ipa->available defines the valid bits in the SUSPEND_CLR register */
+static inline u32
+ipa_reg_irq_suspend_clr_ee_n_offset(enum ipa_version version, u32 ee)
+{
+       /* assert(version != IPA_VERSION_3_0); */
+
+       if (version < IPA_VERSION_4_9)
+               return 0x00003038 + 0x1000 * ee;
+
+       return 0x00004038 + 0x1000 * ee;
+}
+
+static inline u32
+ipa_reg_irq_suspend_clr_offset(enum ipa_version version)
+{
+       return ipa_reg_irq_suspend_clr_ee_n_offset(version, GSI_EE_AP);
+}
 
 int ipa_reg_init(struct ipa *ipa);
 void ipa_reg_exit(struct ipa *ipa);
index dee58a6596d41d07646485b710830959faa7af8b..2756363e6938545293669533c05ae0a68faf3090 100644 (file)
@@ -192,6 +192,7 @@ void ipa_uc_teardown(struct ipa *ipa)
 static void send_uc_command(struct ipa *ipa, u32 command, u32 command_param)
 {
        struct ipa_uc_mem_area *shared = ipa_uc_shared(ipa);
+       u32 offset;
        u32 val;
 
        /* Fill in the command data */
@@ -203,8 +204,8 @@ static void send_uc_command(struct ipa *ipa, u32 command, u32 command_param)
 
        /* Use an interrupt to tell the microcontroller the command is ready */
        val = u32_encode_bits(1, UC_INTR_FMASK);
-
-       iowrite32(val, ipa->reg_virt + IPA_REG_IRQ_UC_OFFSET);
+       offset = ipa_reg_irq_uc_offset(ipa->version);
+       iowrite32(val, ipa->reg_virt + offset);
 }
 
 /* Tell the microcontroller the AP is shutting down */