Re: [PATCH v1 1/4] dt-bindindgs: i2c: qcom,i2c-geni: Document shared flag

Mukesh Kumar Savaliya
2024-09-04T18:27:06Z

On 8/29/2024 3:31 PM, Bryan O'Donoghue wrote:
> On 29/08/2024 10:58, Bryan O'Donoghue wrote:
>> On 29/08/2024 10:24, Mukesh Kumar Savaliya wrote:
>>> Adds qcom,shared-se flag usage. Use this when particular I2C serial
>>> controller needs to be shared between two subsystems.
>>>
>>> Signed-off-by: Mukesh Kumar Savaliya <quic_msavaliy@quicinc.com>
>>> ---
>>>   Documentation/devicetree/bindings/i2c/qcom,i2c-geni-qcom.yaml | 4 ++++
>>>   1 file changed, 4 insertions(+)
>>>
>>> diff --git
>>> a/Documentation/devicetree/bindings/i2c/qcom,i2c-geni-qcom.yaml
>>> b/Documentation/devicetree/bindings/i2c/qcom,i2c-geni-qcom.yaml
>>> index 9f66a3bb1f80..ae423127f736 100644
>>> --- a/Documentation/devicetree/bindings/i2c/qcom,i2c-geni-qcom.yaml
>>> +++ b/Documentation/devicetree/bindings/i2c/qcom,i2c-geni-qcom.yaml
>>> @@ -60,6 +60,10 @@ properties:
>>>     power-domains:
>>>       maxItems: 1
>>> +  qcom,shared-se:
>>> +    description: True if I2C needs to be shared between two or more
>>> subsystems.
>>> +    type: boolean
>>> +
>>>     reg:
>>>       maxItems: 1
>>
>> SE = shared execution ?
>>
>> ---
>> bod
>>
>
> Serial Engines
>
Sorry for short name. yes, SE = Serial Engine. I shall mention in
brackets in next patch. Noted below TLAs too.
> This is a good example of defining TLAs
>
> commit eddac5af06546d2e7a0730e3dc02dde3dc91098a
> Author: Karthikeyan Ramasubramanian <kramasub@codeaurora.org>
> Date:   Fri Mar 30 11:08:17 2018 -0600
>
>     soc: qcom: Add GENI based QUP Wrapper driver
>
>     This driver manages the Generic Interface (GENI) firmware based
>     Qualcomm Universal Peripheral (QUP) Wrapper. GENI based QUP is the
>     next generation programmable module composed of multiple Serial
>     Engines (SE)
>
> ...