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

[Mukesh Kumar Savaliya]

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@xxxxxxxxxxx>
> > > ---
> > >   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@xxxxxxxxxxxxxx>
> 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)
>
> ...