Re: [PATCH V8 1/1] i2c: i2c-qcom-geni: Add shutdown callback for i2c

2021-04-20 Thread rojay
Hi Stephen, On 2021-02-24 12:36, Stephen Boyd wrote: Quoting ro...@codeaurora.org (2021-02-18 06:15:17) Hi Stephen, On 2021-01-13 12:24, Stephen Boyd wrote: > Quoting Roja Rani Yarubandi (2021-01-08 07:05:45) >> diff --git a/drivers/i2c/busses/i2c-qcom-geni.c >>

Re: [PATCH 3/3] i2c: i2c-qcom-geni: Add support for 'assigned-performance-states'

2021-04-01 Thread rojay
On 2021-02-12 14:51, ro...@codeaurora.org wrote: On 2021-01-20 19:01, Ulf Hansson wrote: On Tue, 19 Jan 2021 at 12:05, Viresh Kumar wrote: On 19-01-21, 12:02, Ulf Hansson wrote: > As a matter of fact this was quite recently discussed [1], which also > pointed out some issues when using the

Re: [PATCH V3 1/2] soc: qcom-geni-se: Cleanup the code to remove proxy votes

2021-03-24 Thread rojay
On 2021-03-23 15:00, Greg KH wrote: On Mon, Mar 22, 2021 at 04:34:28PM +0530, Roja Rani Yarubandi wrote: This reverts commit 048eb908a1f2 ("soc: qcom-geni-se: Add interconnect support to fix earlycon crash") ICC core and platforms drivers supports sync_state feature with commit 7d3b0b0d8184

Re: [PATCH V2 1/2] soc: qcom-geni-se: Cleanup the code to remove proxy votes

2021-03-22 Thread rojay
On 2021-03-18 22:43, Matthias Kaehlcke wrote: On Thu, Mar 18, 2021 at 04:40:08PM +0530, Roja Rani Yarubandi wrote: ICC core and platforms drivers supports sync_state feature, which ensures that the default ICC BW votes from the bootloader is not removed until all it's consumers are probes. The

Re: [PATCH V8 1/1] i2c: i2c-qcom-geni: Add shutdown callback for i2c

2021-02-18 Thread rojay
Hi Stephen, On 2021-01-13 12:24, Stephen Boyd wrote: Quoting Roja Rani Yarubandi (2021-01-08 07:05:45) diff --git a/drivers/i2c/busses/i2c-qcom-geni.c b/drivers/i2c/busses/i2c-qcom-geni.c index 214b4c913a13..c3f584795911 100644 --- a/drivers/i2c/busses/i2c-qcom-geni.c +++

Re: [PATCH 3/3] i2c: i2c-qcom-geni: Add support for 'assigned-performance-states'

2021-02-12 Thread rojay
On 2021-01-20 19:01, Ulf Hansson wrote: On Tue, 19 Jan 2021 at 12:05, Viresh Kumar wrote: On 19-01-21, 12:02, Ulf Hansson wrote: > As a matter of fact this was quite recently discussed [1], which also > pointed out some issues when using the "required-opps" in combination, > but perhaps that

Re: [PATCH V7 2/2] i2c: i2c-qcom-geni: Add shutdown callback for i2c

2021-01-08 Thread rojay
Hi Wolfram, On 2021-01-05 20:57, Wolfram Sang wrote: + geni_status = readl_relaxed(gi2c->se.base + SE_GENI_STATUS); + if (!(geni_status & M_GENI_CMD_ACTIVE)) + goto out; + + cur = gi2c->cur; + geni_i2c_abort_xfer(gi2c); + if (cur->flags & I2C_M_RD) +

Re: [RESEND PATCH V6 1/2] i2c: i2c-qcom-geni: Store DMA mapping data in geni_i2c_dev struct

2020-12-21 Thread rojay
On 2020-12-09 18:29, Akash Asthana wrote: Hi Roja, On 12/3/2020 4:01 PM, Roja Rani Yarubandi wrote: Store DMA mapping data in geni_i2c_dev struct to enhance DMA mapping data scope. For example during shutdown callback to unmap DMA mapping, this stored DMA mapping data can be used to call

Re: [PATCH V5 3/3] i2c: i2c-qcom-geni: Add shutdown callback for i2c

2020-10-30 Thread rojay
Hi Stephen, On 2020-10-03 07:09, Stephen Boyd wrote: Quoting Roja Rani Yarubandi (2020-10-01 01:44:25) diff --git a/drivers/i2c/busses/i2c-qcom-geni.c b/drivers/i2c/busses/i2c-qcom-geni.c index aee2a1dd2c62..56d3fbfe7eb6 100644 --- a/drivers/i2c/busses/i2c-qcom-geni.c +++

Re: [PATCH V4] i2c: i2c-qcom-geni: Add shutdown callback for i2c

2020-09-21 Thread rojay
Hi Stephen, On 2020-09-18 01:53, Stephen Boyd wrote: Quoting Roja Rani Yarubandi (2020-09-17 05:25:58) diff --git a/drivers/i2c/busses/i2c-qcom-geni.c b/drivers/i2c/busses/i2c-qcom-geni.c index dead5db3315a..b0d8043c8cb2 100644 --- a/drivers/i2c/busses/i2c-qcom-geni.c +++

Re: [PATCH V3] i2c: i2c-qcom-geni: Add shutdown callback for i2c

2020-09-17 Thread rojay
Hi Stephen, On 2020-09-09 01:00, Stephen Boyd wrote: Why is dri-devel on here? And linaro-mm-sig? Ok, I will remove these lists. Quoting Roja Rani Yarubandi (2020-09-07 06:07:31) diff --git a/drivers/i2c/busses/i2c-qcom-geni.c b/drivers/i2c/busses/i2c-qcom-geni.c index

Re: [PATCH V2 2/2] i2c: i2c-qcom-geni: Add shutdown callback for i2c

2020-09-07 Thread rojay
On 2020-08-26 17:26, Akash Asthana wrote: Hi Roja, On 8/20/2020 4:05 PM, Roja Rani Yarubandi wrote: If the hardware is still accessing memory after SMMU translation is disabled (as part of smmu shutdown callback), then the IOVAs (I/O virtual address) which it was using will go on the bus as

Re: [PATCH V2 1/2] i2c: i2c-qcom-geni: Store DMA mapping data in geni_i2c_dev struct

2020-09-07 Thread rojay
On 2020-08-26 17:25, Akash Asthana wrote: Hi Roja, On 8/20/2020 4:05 PM, Roja Rani Yarubandi wrote: Store DMA mapping data in geni_i2c_dev struct to enhance DMA mapping data scope. For example during shutdown callback to unmap DMA mapping, this stored DMA mapping data can be used to call

Re: [PATCH V2 2/2] i2c: i2c-qcom-geni: Add shutdown callback for i2c

2020-08-28 Thread rojay
On 2020-08-21 05:52, Stephen Boyd wrote: Quoting Roja Rani Yarubandi (2020-08-20 03:35:22) If the hardware is still accessing memory after SMMU translation is disabled (as part of smmu shutdown callback), then the IOVAs (I/O virtual address) which it was using will go on the bus as the physical

Re: [PATCH V2 1/2] i2c: i2c-qcom-geni: Store DMA mapping data in geni_i2c_dev struct

2020-08-28 Thread rojay
On 2020-08-21 05:48, Stephen Boyd wrote: Quoting Roja Rani Yarubandi (2020-08-20 03:35:21) Store DMA mapping data in geni_i2c_dev struct to enhance DMA mapping data scope. For example during shutdown callback to unmap DMA mapping, this stored DMA mapping data can be used to call

Re: [PATCH 2/2] i2c: i2c-qcom-geni: Add shutdown callback for i2c

2020-08-20 Thread rojay
On 2020-08-19 09:13, Stephen Boyd wrote: Quoting Roja Rani Yarubandi (2020-08-14 02:55:40) If the hardware is still accessing memory after SMMU translation is disabled(as part of smmu shutdown callback), then the Put a space before ( Ok. IOVAs(I/O virtual address) which it was using will

Re: [PATCH 1/2] i2c: i2c-qcom-geni: Add tx_dma, rx_dma and xfer_len to geni_i2c_dev struct

2020-08-20 Thread rojay
Hi Stephen, Thanks for reviewing the patches. On 2020-08-19 09:09, Stephen Boyd wrote: Quoting Roja Rani Yarubandi (2020-08-14 02:55:39) Adding tx_dma, rx_dma and xfer length in geni_i2c_dev struct to store DMA mapping data to enhance its scope. For example during shutdown callback to unmap