[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <7130e38e-0c9f-4d34-b427-f652619e53fa@quicinc.com>
Date: Wed, 4 Sep 2024 23:38:46 +0530
From: Mukesh Kumar Savaliya <quic_msavaliy@...cinc.com>
To: Bryan O'Donoghue <bryan.odonoghue@...aro.org>, <konrad.dybcio@...aro.org>,
<andersson@...nel.org>, <andi.shyti@...nel.org>,
<linux-arm-msm@...r.kernel.org>, <dmaengine@...r.kernel.org>,
<linux-kernel@...r.kernel.org>, <linux-i2c@...r.kernel.org>
CC: <quic_vdadhani@...cinc.com>
Subject: Re: [PATCH v1 0/4] Enable shared SE support over I2C
On 8/29/2024 3:40 PM, Bryan O'Donoghue wrote:
> On 29/08/2024 10:24, Mukesh Kumar Savaliya wrote:
>> This Series adds support to share QUP based I2C SE between subsystems.
>> Each subsystem should have its own GPII which interacts between SE and
>> GSI DMA HW engine.
>>
>> Subsystem must acquire Lock over the SE on GPII channel so that it
>> gets uninterrupted control till it unlocks the SE. It also makes sure
>> the commonly shared TLMM GPIOs are not touched which can impact other
>> subsystem or cause any interruption. Generally, GPIOs are being
>> unconfigured during suspend time.
>>
>> GSI DMA engine is capable to perform requested transfer operations
>> from any of the SE in a seamless way and its transparent to the
>> subsystems. Make sure to enable “qcom,shared-se” flag only while
>> enabling this feature. I2C client should add in its respective parent
>> node.
>>
>> ---
>> Mukesh Kumar Savaliya (4):
>> dt-bindindgs: i2c: qcom,i2c-geni: Document shared flag
>> dma: gpi: Add Lock and Unlock TRE support to access SE exclusively
>> soc: qcom: geni-se: Export function geni_se_clks_off()
>> i2c: i2c-qcom-geni: Enable i2c controller sharing between two
>> subsystems
>>
>> .../bindings/i2c/qcom,i2c-geni-qcom.yaml | 4 ++
>> drivers/dma/qcom/gpi.c | 37 ++++++++++++++++++-
>> drivers/i2c/busses/i2c-qcom-geni.c | 29 +++++++++++----
>> drivers/soc/qcom/qcom-geni-se.c | 4 +-
>> include/linux/dma/qcom-gpi-dma.h | 6 +++
>> include/linux/soc/qcom/geni-se.h | 3 ++
>> 6 files changed, 74 insertions(+), 9 deletions(-)
>>
>
> In the cover letter please give an example of Serial Engine sharing.
>
Sure Bryan, Noted. In next patch will update in cover letter.
>
>
Powered by blists - more mailing lists