| Message ID | 20241121130134.29408-2-quic_jseerapu@quicinc.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Add Block event interrupt support for I2C protocol | expand |
On 21-11-24, 18:31, Jyothi Kumar Seerapu wrote: > GSI hardware generates an interrupt for each transfer completion. > For multiple messages within a single transfer, this results in > N interrupts for N messages, leading to significant software > interrupt latency. > > To mitigate this latency, utilize Block Event Interrupt (BEI) mechanism. > Enabling BEI instructs the GSI hardware to prevent interrupt generation > and BEI is disabled when an interrupt is necessary. > > When using BEI, consider splitting a single multi-message transfer into > chunks of 8 internally. Interrupts are not expected for the first 7 message > completions, only the last message triggers an interrupt,indicating > the completion of 8 messages. > > This BEI mechanism enhances overall transfer efficiency. > > Signed-off-by: Jyothi Kumar Seerapu <quic_jseerapu@quicinc.com> > --- > > v2-> v3: > - Renamed gpi_multi_desc_process to gpi_multi_xfer_timeout_handler > - MIN_NUM_OF_MSGS_MULTI_DESC changed from 4 to 2 > - Added documentation for newly added changes in "qcom-gpi-dma.h" file > - Updated commit description. > > v1 -> v2: > - Changed dma_addr type from array of pointers to array. > - To support BEI functionality with the TRE size of 64 defined in GPI driver, > updated QCOM_GPI_MAX_NUM_MSGS to 16 and NUM_MSGS_PER_IRQ to 4. > > drivers/dma/qcom/gpi.c | 48 ++++++++++++++++++++ > include/linux/dma/qcom-gpi-dma.h | 76 ++++++++++++++++++++++++++++++++ > 2 files changed, 124 insertions(+) > > diff --git a/drivers/dma/qcom/gpi.c b/drivers/dma/qcom/gpi.c > index 52a7c8f2498f..5442b65b1638 100644 > --- a/drivers/dma/qcom/gpi.c > +++ b/drivers/dma/qcom/gpi.c > @@ -1693,6 +1693,9 @@ static int gpi_create_i2c_tre(struct gchan *chan, struct gpi_desc *desc, > > tre->dword[3] = u32_encode_bits(TRE_TYPE_DMA, TRE_FLAGS_TYPE); > tre->dword[3] |= u32_encode_bits(1, TRE_FLAGS_IEOT); > + > + if (i2c->flags & QCOM_GPI_BLOCK_EVENT_IRQ) > + tre->dword[3] |= u32_encode_bits(1, TRE_FLAGS_BEI); > } > > for (i = 0; i < tre_idx; i++) > @@ -2098,6 +2101,51 @@ static int gpi_find_avail_gpii(struct gpi_dev *gpi_dev, u32 seid) > return -EIO; > } > > +/** > + * gpi_multi_xfer_timeout_handler() - Handle multi message transfer timeout > + * @dev: pointer to the corresponding dev node > + * @multi_xfer: pointer to the gpi_multi_xfer > + * @num_xfers: total number of transfers > + * @transfer_timeout_msecs: transfer timeout value > + * @transfer_comp: completion object of the transfer > + * > + * This function is used to wait for the processed transfers based on > + * the interrupts generated upon transfer completion. > + * Return: On success returns 0, otherwise return error code (-ETIMEDOUT) > + */ > +int gpi_multi_xfer_timeout_handler(struct device *dev, struct gpi_multi_xfer *multi_xfer, > + u32 num_xfers, u32 transfer_timeout_msecs, > + struct completion *transfer_comp) > +{ > + int i; > + u32 max_irq_cnt, time_left; > + > + max_irq_cnt = num_xfers / NUM_MSGS_PER_IRQ; > + if (num_xfers % NUM_MSGS_PER_IRQ) > + max_irq_cnt++; > + > + /* > + * Wait for the interrupts of the processed transfers in multiple > + * of 8 and for the last transfer. If the hardware is fast and > + * already processed all the transfers then no need to wait. > + */ > + for (i = 0; i < max_irq_cnt; i++) { > + reinit_completion(transfer_comp); > + if (max_irq_cnt != multi_xfer->irq_cnt) { > + time_left = wait_for_completion_timeout(transfer_comp, > + transfer_timeout_msecs); > + if (!time_left) { > + dev_err(dev, "%s: Transfer timeout\n", __func__); > + return -ETIMEDOUT; > + } > + } > + if (num_xfers > multi_xfer->msg_idx_cnt) > + return 0; > + } > + return 0; > +} > +EXPORT_SYMBOL_GPL(gpi_multi_xfer_timeout_handler); > + > /* gpi_of_dma_xlate: open client requested channel */ > static struct dma_chan *gpi_of_dma_xlate(struct of_phandle_args *args, > struct of_dma *of_dma) > diff --git a/include/linux/dma/qcom-gpi-dma.h b/include/linux/dma/qcom-gpi-dma.h > index 6680dd1a43c6..f001a8ac1887 100644 > --- a/include/linux/dma/qcom-gpi-dma.h > +++ b/include/linux/dma/qcom-gpi-dma.h > @@ -15,6 +15,38 @@ enum spi_transfer_cmd { > SPI_DUPLEX, > }; > > +/** > + * define QCOM_GPI_BLOCK_EVENT_IRQ - Block event interrupt support > + * > + * This is used to enable/disable the Block event interrupt mechanism. > + */ > +#define QCOM_GPI_BLOCK_EVENT_IRQ BIT(0) > + > +/** > + * define QCOM_GPI_MAX_NUM_MSGS - maximum number of messages support > + * > + * This indicates maximum number of messages can allocate and > + * submit to hardware. To handle more messages beyond this, > + * need to unmap the processed messages. > + */ > +#define QCOM_GPI_MAX_NUM_MSGS 16 > + > +/** > + * define NUM_MSGS_PER_IRQ - interrupt per messages completion > + * > + * This indicates that trigger an interrupt, after the completion of 8 messages. > + */ > +#define NUM_MSGS_PER_IRQ 8 > + > +/** > + * define MIN_NUM_OF_MSGS_MULTI_DESC - \ > + * minimum number of messages to support Block evenet interrupt > + * > + * This indicates minimum number of messages in a trenafer required to > + * process it using block event interrupt mechanism. > + */ > +#define MIN_NUM_OF_MSGS_MULTI_DESC 2 > + > /** > * struct gpi_spi_config - spi config for peripheral > * > @@ -51,6 +83,29 @@ enum i2c_op { > I2C_READ, > }; why should these be exposed to user? > > +/** > + * struct gpi_multi_xfer - Used for multi transfer support > + * > + * @msg_idx_cnt: message index for the transfer > + * @buf_idx: dma buffer index > + * @unmap_msg_cnt: unmapped transfer index > + * @freed_msg_cnt: freed transfer index > + * @irq_cnt: received interrupt count > + * @irq_msg_cnt: transfer message count for the received irqs > + * @dma_buf: virtual addresses of the buffers > + * @dma_addr: dma addresses of the buffers > + */ > +struct gpi_multi_xfer { > + u32 msg_idx_cnt; > + u32 buf_idx; > + u32 unmap_msg_cnt; > + u32 freed_msg_cnt; > + u32 irq_cnt; > + u32 irq_msg_cnt; > + void *dma_buf[QCOM_GPI_MAX_NUM_MSGS]; > + dma_addr_t dma_addr[QCOM_GPI_MAX_NUM_MSGS]; > +}; DMAengine API can do multiple transfers and we already have flags for interrupts, pls use that instead of usual behaviour of defining custom interfaces to handle everything. That is not recommended > + > /** > * struct gpi_i2c_config - i2c config for peripheral > * > @@ -65,6 +120,8 @@ enum i2c_op { > * @rx_len: receive length for buffer > * @op: i2c cmd > * @muli-msg: is part of multi i2c r-w msgs > + * @flags: true for block event interrupt support > + * @multi_xfer: indicates transfer has multi messages > */ > struct gpi_i2c_config { > u8 set_config; > @@ -78,6 +135,25 @@ struct gpi_i2c_config { > u32 rx_len; > enum i2c_op op; > bool multi_msg; > + u8 flags; > + struct gpi_multi_xfer multi_xfer; > }; > > +/** > + * gpi_multi_timeout_handler() - Handle multi message transfer timeout > + * @dev: pointer to the corresponding dev node > + * @multi_xfer: pointer to the gpi_multi_xfer > + * @num_xfers: total number of transfers > + * @transfer_timeout_msecs: transfer timeout value > + * @transfer_comp: completion object of the transfer > + * > + * This function is used to wait for the processed transfers based on > + * the interrupts generated upon transfer completion. > + * > + * Return: On success returns 0, otherwise return error code (-ETIMEDOUT) > + */ > +int gpi_multi_xfer_timeout_handler(struct device *dev, struct gpi_multi_xfer *multi_xfer, > + u32 num_xfers, u32 tranfer_timeout_msecs, > + struct completion *transfer_comp); Why should a handler be here?
On 12/2/2024 12:51 PM, Vinod Koul wrote: > On 21-11-24, 18:31, Jyothi Kumar Seerapu wrote: >> GSI hardware generates an interrupt for each transfer completion. >> For multiple messages within a single transfer, this results in >> N interrupts for N messages, leading to significant software >> interrupt latency. >> >> To mitigate this latency, utilize Block Event Interrupt (BEI) mechanism. >> Enabling BEI instructs the GSI hardware to prevent interrupt generation >> and BEI is disabled when an interrupt is necessary. >> >> When using BEI, consider splitting a single multi-message transfer into >> chunks of 8 internally. Interrupts are not expected for the first 7 message >> completions, only the last message triggers an interrupt,indicating >> the completion of 8 messages. >> >> This BEI mechanism enhances overall transfer efficiency. >> >> Signed-off-by: Jyothi Kumar Seerapu <quic_jseerapu@quicinc.com> >> --- >> >> v2-> v3: >> - Renamed gpi_multi_desc_process to gpi_multi_xfer_timeout_handler >> - MIN_NUM_OF_MSGS_MULTI_DESC changed from 4 to 2 >> - Added documentation for newly added changes in "qcom-gpi-dma.h" file >> - Updated commit description. >> >> v1 -> v2: >> - Changed dma_addr type from array of pointers to array. >> - To support BEI functionality with the TRE size of 64 defined in GPI driver, >> updated QCOM_GPI_MAX_NUM_MSGS to 16 and NUM_MSGS_PER_IRQ to 4. >> >> drivers/dma/qcom/gpi.c | 48 ++++++++++++++++++++ >> include/linux/dma/qcom-gpi-dma.h | 76 ++++++++++++++++++++++++++++++++ >> 2 files changed, 124 insertions(+) >> >> diff --git a/drivers/dma/qcom/gpi.c b/drivers/dma/qcom/gpi.c >> index 52a7c8f2498f..5442b65b1638 100644 >> --- a/drivers/dma/qcom/gpi.c >> +++ b/drivers/dma/qcom/gpi.c >> @@ -1693,6 +1693,9 @@ static int gpi_create_i2c_tre(struct gchan *chan, struct gpi_desc *desc, >> >> tre->dword[3] = u32_encode_bits(TRE_TYPE_DMA, TRE_FLAGS_TYPE); >> tre->dword[3] |= u32_encode_bits(1, TRE_FLAGS_IEOT); >> + >> + if (i2c->flags & QCOM_GPI_BLOCK_EVENT_IRQ) >> + tre->dword[3] |= u32_encode_bits(1, TRE_FLAGS_BEI); >> } >> >> for (i = 0; i < tre_idx; i++) >> @@ -2098,6 +2101,51 @@ static int gpi_find_avail_gpii(struct gpi_dev *gpi_dev, u32 seid) >> return -EIO; >> } >> >> +/** >> + * gpi_multi_xfer_timeout_handler() - Handle multi message transfer timeout >> + * @dev: pointer to the corresponding dev node >> + * @multi_xfer: pointer to the gpi_multi_xfer >> + * @num_xfers: total number of transfers >> + * @transfer_timeout_msecs: transfer timeout value >> + * @transfer_comp: completion object of the transfer >> + * >> + * This function is used to wait for the processed transfers based on >> + * the interrupts generated upon transfer completion. >> + * Return: On success returns 0, otherwise return error code (-ETIMEDOUT) >> + */ >> +int gpi_multi_xfer_timeout_handler(struct device *dev, struct gpi_multi_xfer *multi_xfer, >> + u32 num_xfers, u32 transfer_timeout_msecs, >> + struct completion *transfer_comp) >> +{ >> + int i; >> + u32 max_irq_cnt, time_left; >> + >> + max_irq_cnt = num_xfers / NUM_MSGS_PER_IRQ; >> + if (num_xfers % NUM_MSGS_PER_IRQ) >> + max_irq_cnt++; >> + >> + /* >> + * Wait for the interrupts of the processed transfers in multiple >> + * of 8 and for the last transfer. If the hardware is fast and >> + * already processed all the transfers then no need to wait. >> + */ >> + for (i = 0; i < max_irq_cnt; i++) { >> + reinit_completion(transfer_comp); >> + if (max_irq_cnt != multi_xfer->irq_cnt) { >> + time_left = wait_for_completion_timeout(transfer_comp, >> + transfer_timeout_msecs); >> + if (!time_left) { >> + dev_err(dev, "%s: Transfer timeout\n", __func__); >> + return -ETIMEDOUT; >> + } >> + } >> + if (num_xfers > multi_xfer->msg_idx_cnt) >> + return 0; >> + } >> + return 0; >> +} >> +EXPORT_SYMBOL_GPL(gpi_multi_xfer_timeout_handler); >> + >> /* gpi_of_dma_xlate: open client requested channel */ >> static struct dma_chan *gpi_of_dma_xlate(struct of_phandle_args *args, >> struct of_dma *of_dma) >> diff --git a/include/linux/dma/qcom-gpi-dma.h b/include/linux/dma/qcom-gpi-dma.h >> index 6680dd1a43c6..f001a8ac1887 100644 >> --- a/include/linux/dma/qcom-gpi-dma.h >> +++ b/include/linux/dma/qcom-gpi-dma.h >> @@ -15,6 +15,38 @@ enum spi_transfer_cmd { >> SPI_DUPLEX, >> }; >> >> +/** >> + * define QCOM_GPI_BLOCK_EVENT_IRQ - Block event interrupt support >> + * >> + * This is used to enable/disable the Block event interrupt mechanism. >> + */ >> +#define QCOM_GPI_BLOCK_EVENT_IRQ BIT(0) >> + >> +/** >> + * define QCOM_GPI_MAX_NUM_MSGS - maximum number of messages support >> + * >> + * This indicates maximum number of messages can allocate and >> + * submit to hardware. To handle more messages beyond this, >> + * need to unmap the processed messages. >> + */ >> +#define QCOM_GPI_MAX_NUM_MSGS 16 >> + >> +/** >> + * define NUM_MSGS_PER_IRQ - interrupt per messages completion >> + * >> + * This indicates that trigger an interrupt, after the completion of 8 messages. >> + */ >> +#define NUM_MSGS_PER_IRQ 8 >> + >> +/** >> + * define MIN_NUM_OF_MSGS_MULTI_DESC - \ >> + * minimum number of messages to support Block evenet interrupt >> + * >> + * This indicates minimum number of messages in a trenafer required to >> + * process it using block event interrupt mechanism. >> + */ >> +#define MIN_NUM_OF_MSGS_MULTI_DESC 2 >> + >> /** >> * struct gpi_spi_config - spi config for peripheral >> * >> @@ -51,6 +83,29 @@ enum i2c_op { >> I2C_READ, >> }; > > why should these be exposed to user? This (struct gpi_multi_xfer) has been added in this file to provide common support for other protocols which uses the Block event interrupt mechanism. Please let me know instead of GPI, if these need to handle in I2C driver itself. > >> >> +/** >> + * struct gpi_multi_xfer - Used for multi transfer support >> + * >> + * @msg_idx_cnt: message index for the transfer >> + * @buf_idx: dma buffer index >> + * @unmap_msg_cnt: unmapped transfer index >> + * @freed_msg_cnt: freed transfer index >> + * @irq_cnt: received interrupt count >> + * @irq_msg_cnt: transfer message count for the received irqs >> + * @dma_buf: virtual addresses of the buffers >> + * @dma_addr: dma addresses of the buffers >> + */ >> +struct gpi_multi_xfer { >> + u32 msg_idx_cnt; >> + u32 buf_idx; >> + u32 unmap_msg_cnt; >> + u32 freed_msg_cnt; >> + u32 irq_cnt; >> + u32 irq_msg_cnt; >> + void *dma_buf[QCOM_GPI_MAX_NUM_MSGS]; >> + dma_addr_t dma_addr[QCOM_GPI_MAX_NUM_MSGS]; >> +}; > > DMAengine API can do multiple transfers and we already have flags for > interrupts, pls use that instead of usual behaviour of defining custom > interfaces to handle everything. That is not recommended > Hi Vinod, if i understand correctly you are referring to DMA with device to memory transfers and scatter-gather transfers and "DMA_INTERRUPT" for interrupts. Please correct me if this is not the case. The plan for these changes to use the Qualcomm GPI DMA hardware feature, specifically the Block Event Interrupt (BEI). This feature instructs the GSI hardware to prevent interrupt generation with BEI being disabled and enable BEI when an interrupt is required. For example, if an I2C transfer is initiated with 100 messages, we would typically expect 100 interrupts for the completion of these messages. However, with the Block Event Interrupt mechanism, we will only receive 13 interrupts. Additionally, to handle I2C transfer with 100 or more messages using the existing channel TRE size of 64, we can have possiblity of utilize 16 I2C messages (16 messages can fit with channel TRE size of 64 for config TRE, go TRE, and DMA TRE), and so use an array of 16 DMA buffers. After the completion of the 16 I2C messages, we can unmap the processed messages based on the interrupt count (unmapping 8 messages for each interrupt count). This process helps to handle all messages in a large I2C transfer, improving throughput and overall transfer efficiency. Please let me know if you have any other comments. > >> + >> /** >> * struct gpi_i2c_config - i2c config for peripheral >> * >> @@ -65,6 +120,8 @@ enum i2c_op { >> * @rx_len: receive length for buffer >> * @op: i2c cmd >> * @muli-msg: is part of multi i2c r-w msgs >> + * @flags: true for block event interrupt support >> + * @multi_xfer: indicates transfer has multi messages >> */ >> struct gpi_i2c_config { >> u8 set_config; >> @@ -78,6 +135,25 @@ struct gpi_i2c_config { >> u32 rx_len; >> enum i2c_op op; >> bool multi_msg; >> + u8 flags; >> + struct gpi_multi_xfer multi_xfer; >> }; >> >> +/** >> + * gpi_multi_timeout_handler() - Handle multi message transfer timeout >> + * @dev: pointer to the corresponding dev node >> + * @multi_xfer: pointer to the gpi_multi_xfer >> + * @num_xfers: total number of transfers >> + * @transfer_timeout_msecs: transfer timeout value >> + * @transfer_comp: completion object of the transfer >> + * >> + * This function is used to wait for the processed transfers based on >> + * the interrupts generated upon transfer completion. >> + * >> + * Return: On success returns 0, otherwise return error code (-ETIMEDOUT) >> + */ >> +int gpi_multi_xfer_timeout_handler(struct device *dev, struct gpi_multi_xfer *multi_xfer, >> + u32 num_xfers, u32 tranfer_timeout_msecs, >> + struct completion *transfer_comp); > > Why should a handler be here? I intended to use this function as a common utility to support other protocols, so I included it in the GPI module. However, I got to know that GPI functions cannot be invoked directly and must be called through an existing DMA engine API. Unfortunately, this function does not fit into any DMA engine API. Therefore, I am considering moving this function to the I2C driver. Please let me know if this is acceptable or if you have any suggestions. >
diff --git a/drivers/dma/qcom/gpi.c b/drivers/dma/qcom/gpi.c index 52a7c8f2498f..5442b65b1638 100644 --- a/drivers/dma/qcom/gpi.c +++ b/drivers/dma/qcom/gpi.c @@ -1693,6 +1693,9 @@ static int gpi_create_i2c_tre(struct gchan *chan, struct gpi_desc *desc, tre->dword[3] = u32_encode_bits(TRE_TYPE_DMA, TRE_FLAGS_TYPE); tre->dword[3] |= u32_encode_bits(1, TRE_FLAGS_IEOT); + + if (i2c->flags & QCOM_GPI_BLOCK_EVENT_IRQ) + tre->dword[3] |= u32_encode_bits(1, TRE_FLAGS_BEI); } for (i = 0; i < tre_idx; i++) @@ -2098,6 +2101,51 @@ static int gpi_find_avail_gpii(struct gpi_dev *gpi_dev, u32 seid) return -EIO; } +/** + * gpi_multi_xfer_timeout_handler() - Handle multi message transfer timeout + * @dev: pointer to the corresponding dev node + * @multi_xfer: pointer to the gpi_multi_xfer + * @num_xfers: total number of transfers + * @transfer_timeout_msecs: transfer timeout value + * @transfer_comp: completion object of the transfer + * + * This function is used to wait for the processed transfers based on + * the interrupts generated upon transfer completion. + * Return: On success returns 0, otherwise return error code (-ETIMEDOUT) + */ +int gpi_multi_xfer_timeout_handler(struct device *dev, struct gpi_multi_xfer *multi_xfer, + u32 num_xfers, u32 transfer_timeout_msecs, + struct completion *transfer_comp) +{ + int i; + u32 max_irq_cnt, time_left; + + max_irq_cnt = num_xfers / NUM_MSGS_PER_IRQ; + if (num_xfers % NUM_MSGS_PER_IRQ) + max_irq_cnt++; + + /* + * Wait for the interrupts of the processed transfers in multiple + * of 8 and for the last transfer. If the hardware is fast and + * already processed all the transfers then no need to wait. + */ + for (i = 0; i < max_irq_cnt; i++) { + reinit_completion(transfer_comp); + if (max_irq_cnt != multi_xfer->irq_cnt) { + time_left = wait_for_completion_timeout(transfer_comp, + transfer_timeout_msecs); + if (!time_left) { + dev_err(dev, "%s: Transfer timeout\n", __func__); + return -ETIMEDOUT; + } + } + if (num_xfers > multi_xfer->msg_idx_cnt) + return 0; + } + return 0; +} +EXPORT_SYMBOL_GPL(gpi_multi_xfer_timeout_handler); + /* gpi_of_dma_xlate: open client requested channel */ static struct dma_chan *gpi_of_dma_xlate(struct of_phandle_args *args, struct of_dma *of_dma) diff --git a/include/linux/dma/qcom-gpi-dma.h b/include/linux/dma/qcom-gpi-dma.h index 6680dd1a43c6..f001a8ac1887 100644 --- a/include/linux/dma/qcom-gpi-dma.h +++ b/include/linux/dma/qcom-gpi-dma.h @@ -15,6 +15,38 @@ enum spi_transfer_cmd { SPI_DUPLEX, }; +/** + * define QCOM_GPI_BLOCK_EVENT_IRQ - Block event interrupt support + * + * This is used to enable/disable the Block event interrupt mechanism. + */ +#define QCOM_GPI_BLOCK_EVENT_IRQ BIT(0) + +/** + * define QCOM_GPI_MAX_NUM_MSGS - maximum number of messages support + * + * This indicates maximum number of messages can allocate and + * submit to hardware. To handle more messages beyond this, + * need to unmap the processed messages. + */ +#define QCOM_GPI_MAX_NUM_MSGS 16 + +/** + * define NUM_MSGS_PER_IRQ - interrupt per messages completion + * + * This indicates that trigger an interrupt, after the completion of 8 messages. + */ +#define NUM_MSGS_PER_IRQ 8 + +/** + * define MIN_NUM_OF_MSGS_MULTI_DESC - \ + * minimum number of messages to support Block evenet interrupt + * + * This indicates minimum number of messages in a trenafer required to + * process it using block event interrupt mechanism. + */ +#define MIN_NUM_OF_MSGS_MULTI_DESC 2 + /** * struct gpi_spi_config - spi config for peripheral * @@ -51,6 +83,29 @@ enum i2c_op { I2C_READ, }; +/** + * struct gpi_multi_xfer - Used for multi transfer support + * + * @msg_idx_cnt: message index for the transfer + * @buf_idx: dma buffer index + * @unmap_msg_cnt: unmapped transfer index + * @freed_msg_cnt: freed transfer index + * @irq_cnt: received interrupt count + * @irq_msg_cnt: transfer message count for the received irqs + * @dma_buf: virtual addresses of the buffers + * @dma_addr: dma addresses of the buffers + */ +struct gpi_multi_xfer { + u32 msg_idx_cnt; + u32 buf_idx; + u32 unmap_msg_cnt; + u32 freed_msg_cnt; + u32 irq_cnt; + u32 irq_msg_cnt; + void *dma_buf[QCOM_GPI_MAX_NUM_MSGS]; + dma_addr_t dma_addr[QCOM_GPI_MAX_NUM_MSGS]; +}; + /** * struct gpi_i2c_config - i2c config for peripheral * @@ -65,6 +120,8 @@ enum i2c_op { * @rx_len: receive length for buffer * @op: i2c cmd * @muli-msg: is part of multi i2c r-w msgs + * @flags: true for block event interrupt support + * @multi_xfer: indicates transfer has multi messages */ struct gpi_i2c_config { u8 set_config; @@ -78,6 +135,25 @@ struct gpi_i2c_config { u32 rx_len; enum i2c_op op; bool multi_msg; + u8 flags; + struct gpi_multi_xfer multi_xfer; }; +/** + * gpi_multi_timeout_handler() - Handle multi message transfer timeout + * @dev: pointer to the corresponding dev node + * @multi_xfer: pointer to the gpi_multi_xfer + * @num_xfers: total number of transfers + * @transfer_timeout_msecs: transfer timeout value + * @transfer_comp: completion object of the transfer + * + * This function is used to wait for the processed transfers based on + * the interrupts generated upon transfer completion. + * + * Return: On success returns 0, otherwise return error code (-ETIMEDOUT) + */ +int gpi_multi_xfer_timeout_handler(struct device *dev, struct gpi_multi_xfer *multi_xfer, + u32 num_xfers, u32 tranfer_timeout_msecs, + struct completion *transfer_comp); + #endif /* QCOM_GPI_DMA_H */
GSI hardware generates an interrupt for each transfer completion. For multiple messages within a single transfer, this results in N interrupts for N messages, leading to significant software interrupt latency. To mitigate this latency, utilize Block Event Interrupt (BEI) mechanism. Enabling BEI instructs the GSI hardware to prevent interrupt generation and BEI is disabled when an interrupt is necessary. When using BEI, consider splitting a single multi-message transfer into chunks of 8 internally. Interrupts are not expected for the first 7 message completions, only the last message triggers an interrupt,indicating the completion of 8 messages. This BEI mechanism enhances overall transfer efficiency. Signed-off-by: Jyothi Kumar Seerapu <quic_jseerapu@quicinc.com> --- v2-> v3: - Renamed gpi_multi_desc_process to gpi_multi_xfer_timeout_handler - MIN_NUM_OF_MSGS_MULTI_DESC changed from 4 to 2 - Added documentation for newly added changes in "qcom-gpi-dma.h" file - Updated commit description. v1 -> v2: - Changed dma_addr type from array of pointers to array. - To support BEI functionality with the TRE size of 64 defined in GPI driver, updated QCOM_GPI_MAX_NUM_MSGS to 16 and NUM_MSGS_PER_IRQ to 4. drivers/dma/qcom/gpi.c | 48 ++++++++++++++++++++ include/linux/dma/qcom-gpi-dma.h | 76 ++++++++++++++++++++++++++++++++ 2 files changed, 124 insertions(+)