diff mbox series

[v2,1/2] dt-bindings: remoteproc: k3-r5f: Update bindings for AM64x SoCs

Message ID 20210327143117.1840-2-s-anna@ti.com
State Accepted
Commit c16ced60f3bf4aeba85e638f2186c468d7892ee0
Headers show
Series TI K3 R5F remoteproc support on AM64x SoCs | expand

Commit Message

Suman Anna March 27, 2021, 2:31 p.m. UTC
The K3 AM64x SoCs have two dual-core Arm R5F clusters/subsystems, with
2 R5F cores each, both in the MAIN voltage domain.

These clusters are a revised IP version compared to those present on
J721E and J7200 SoCs, and supports a new "Single-CPU" mode instead of
LockStep mode. Update the K3 R5F remoteproc bindings with the compatible
info relevant to these R5F clusters/subsystems on K3 AM64x SoCs.

Signed-off-by: Suman Anna <s-anna@ti.com>
---
v2: No changes

 .../bindings/remoteproc/ti,k3-r5f-rproc.yaml  | 31 ++++++++++++++++---
 1 file changed, 26 insertions(+), 5 deletions(-)

Comments

Suman Anna April 19, 2021, 1:55 p.m. UTC | #1
Hi Rob,

On 3/27/21 9:31 AM, Suman Anna wrote:
> The K3 AM64x SoCs have two dual-core Arm R5F clusters/subsystems, with

> 2 R5F cores each, both in the MAIN voltage domain.

> 

> These clusters are a revised IP version compared to those present on

> J721E and J7200 SoCs, and supports a new "Single-CPU" mode instead of

> LockStep mode. Update the K3 R5F remoteproc bindings with the compatible

> info relevant to these R5F clusters/subsystems on K3 AM64x SoCs.

> 

> Signed-off-by: Suman Anna <s-anna@ti.com>

> ---

> v2: No changes

> 

>  .../bindings/remoteproc/ti,k3-r5f-rproc.yaml  | 31 ++++++++++++++++---


Looks like this patch has fallen through the cracks, can you please review and
give your ack for this patch so that Bjorn can pick up the series for 5.13?

regards
Suman

>  1 file changed, 26 insertions(+), 5 deletions(-)

> 

> diff --git a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

> index d905d614502b..130fbaacc4b1 100644

> --- a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

> +++ b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

> @@ -14,8 +14,12 @@ description: |

>    processor subsystems/clusters (R5FSS). The dual core cluster can be used

>    either in a LockStep mode providing safety/fault tolerance features or in a

>    Split mode providing two individual compute cores for doubling the compute

> -  capacity. These are used together with other processors present on the SoC

> -  to achieve various system level goals.

> +  capacity on most SoCs. These are used together with other processors present

> +  on the SoC to achieve various system level goals.

> +

> +  AM64x SoCs do not support LockStep mode, but rather a new non-safety mode

> +  called "Single-CPU" mode, where only Core0 is used, but with ability to use

> +  Core1's TCMs as well.

>  

>    Each Dual-Core R5F sub-system is represented as a single DTS node

>    representing the cluster, with a pair of child DT nodes representing

> @@ -33,6 +37,7 @@ properties:

>        - ti,am654-r5fss

>        - ti,j721e-r5fss

>        - ti,j7200-r5fss

> +      - ti,am64-r5fss

>  

>    power-domains:

>      description: |

> @@ -56,11 +61,12 @@ properties:

>  

>    ti,cluster-mode:

>      $ref: /schemas/types.yaml#/definitions/uint32

> -    enum: [0, 1]

>      description: |

>        Configuration Mode for the Dual R5F cores within the R5F cluster.

> -      Should be either a value of 1 (LockStep mode) or 0 (Split mode),

> -      default is LockStep mode if omitted.

> +      Should be either a value of 1 (LockStep mode) or 0 (Split mode) on

> +      most SoCs (AM65x, J721E, J7200), default is LockStep mode if omitted;

> +      and should be either a value of 0 (Split mode) or 2 (Single-CPU mode)

> +      on AM64x SoCs, default is Split mode if omitted.

>  

>  # R5F Processor Child Nodes:

>  # ==========================

> @@ -97,6 +103,7 @@ patternProperties:

>            - ti,am654-r5f

>            - ti,j721e-r5f

>            - ti,j7200-r5f

> +          - ti,am64-r5f

>  

>        reg:

>          items:

> @@ -198,6 +205,20 @@ patternProperties:

>  

>      unevaluatedProperties: false

>  

> +if:

> +  properties:

> +    compatible:

> +      enum:

> +        - ti,am64-r5fss

> +then:

> +  properties:

> +    ti,cluster-mode:

> +      enum: [0, 2]

> +else:

> +  properties:

> +    ti,cluster-mode:

> +      enum: [0, 1]

> +

>  required:

>    - compatible

>    - power-domains

>
Suman Anna May 13, 2021, 2:47 a.m. UTC | #2
Hi Rob,

On 4/19/21 8:55 AM, Suman Anna wrote:
> Hi Rob,

> 

> On 3/27/21 9:31 AM, Suman Anna wrote:

>> The K3 AM64x SoCs have two dual-core Arm R5F clusters/subsystems, with

>> 2 R5F cores each, both in the MAIN voltage domain.

>>

>> These clusters are a revised IP version compared to those present on

>> J721E and J7200 SoCs, and supports a new "Single-CPU" mode instead of

>> LockStep mode. Update the K3 R5F remoteproc bindings with the compatible

>> info relevant to these R5F clusters/subsystems on K3 AM64x SoCs.

>>

>> Signed-off-by: Suman Anna <s-anna@ti.com>

>> ---

>> v2: No changes

>>

>>  .../bindings/remoteproc/ti,k3-r5f-rproc.yaml  | 31 ++++++++++++++++---

> 

> Looks like this patch has fallen through the cracks, can you please review and

> give your ack for this patch so that Bjorn can pick up the series for 5.13?


Gentle reminder, do you have any comments on this patch. Appreciate your ack so
that we can get this in for 5.14?

regards
Suman

> 

> regards

> Suman

> 

>>  1 file changed, 26 insertions(+), 5 deletions(-)

>>

>> diff --git a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

>> index d905d614502b..130fbaacc4b1 100644

>> --- a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

>> +++ b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

>> @@ -14,8 +14,12 @@ description: |

>>    processor subsystems/clusters (R5FSS). The dual core cluster can be used

>>    either in a LockStep mode providing safety/fault tolerance features or in a

>>    Split mode providing two individual compute cores for doubling the compute

>> -  capacity. These are used together with other processors present on the SoC

>> -  to achieve various system level goals.

>> +  capacity on most SoCs. These are used together with other processors present

>> +  on the SoC to achieve various system level goals.

>> +

>> +  AM64x SoCs do not support LockStep mode, but rather a new non-safety mode

>> +  called "Single-CPU" mode, where only Core0 is used, but with ability to use

>> +  Core1's TCMs as well.

>>  

>>    Each Dual-Core R5F sub-system is represented as a single DTS node

>>    representing the cluster, with a pair of child DT nodes representing

>> @@ -33,6 +37,7 @@ properties:

>>        - ti,am654-r5fss

>>        - ti,j721e-r5fss

>>        - ti,j7200-r5fss

>> +      - ti,am64-r5fss

>>  

>>    power-domains:

>>      description: |

>> @@ -56,11 +61,12 @@ properties:

>>  

>>    ti,cluster-mode:

>>      $ref: /schemas/types.yaml#/definitions/uint32

>> -    enum: [0, 1]

>>      description: |

>>        Configuration Mode for the Dual R5F cores within the R5F cluster.

>> -      Should be either a value of 1 (LockStep mode) or 0 (Split mode),

>> -      default is LockStep mode if omitted.

>> +      Should be either a value of 1 (LockStep mode) or 0 (Split mode) on

>> +      most SoCs (AM65x, J721E, J7200), default is LockStep mode if omitted;

>> +      and should be either a value of 0 (Split mode) or 2 (Single-CPU mode)

>> +      on AM64x SoCs, default is Split mode if omitted.

>>  

>>  # R5F Processor Child Nodes:

>>  # ==========================

>> @@ -97,6 +103,7 @@ patternProperties:

>>            - ti,am654-r5f

>>            - ti,j721e-r5f

>>            - ti,j7200-r5f

>> +          - ti,am64-r5f

>>  

>>        reg:

>>          items:

>> @@ -198,6 +205,20 @@ patternProperties:

>>  

>>      unevaluatedProperties: false

>>  

>> +if:

>> +  properties:

>> +    compatible:

>> +      enum:

>> +        - ti,am64-r5fss

>> +then:

>> +  properties:

>> +    ti,cluster-mode:

>> +      enum: [0, 2]

>> +else:

>> +  properties:

>> +    ti,cluster-mode:

>> +      enum: [0, 1]

>> +

>>  required:

>>    - compatible

>>    - power-domains

>>

>
Mathieu Poirier May 21, 2021, 8:40 p.m. UTC | #3
Hi suman,

On Wed, May 12, 2021 at 09:47:44PM -0500, Suman Anna wrote:
> Hi Rob,

> 

> On 4/19/21 8:55 AM, Suman Anna wrote:

> > Hi Rob,

> > 

> > On 3/27/21 9:31 AM, Suman Anna wrote:

> >> The K3 AM64x SoCs have two dual-core Arm R5F clusters/subsystems, with

> >> 2 R5F cores each, both in the MAIN voltage domain.

> >>

> >> These clusters are a revised IP version compared to those present on

> >> J721E and J7200 SoCs, and supports a new "Single-CPU" mode instead of

> >> LockStep mode. Update the K3 R5F remoteproc bindings with the compatible

> >> info relevant to these R5F clusters/subsystems on K3 AM64x SoCs.

> >>

> >> Signed-off-by: Suman Anna <s-anna@ti.com>

> >> ---

> >> v2: No changes

> >>

> >>  .../bindings/remoteproc/ti,k3-r5f-rproc.yaml  | 31 ++++++++++++++++---

> > 

> > Looks like this patch has fallen through the cracks, can you please review and

> > give your ack for this patch so that Bjorn can pick up the series for 5.13?

> 

> Gentle reminder, do you have any comments on this patch. Appreciate your ack so

> that we can get this in for 5.14?


If memory serves me well Rob indicated that he would not review or comment on
bindings related to multi-core remote processors.  On the flip side he also
mentioned that he would not object to their presence.  And since this is an
increment to an existing binding rather than a new one, I think it is fair for
us to pick it up.  

Rob - please intervene if my recollections are not accurate and accept my honest
apologies.  Otherwise: 

Reviewed-by: Mathieu Poirier <mathieu.poirier@linaro.org>  


> 

> regards

> Suman

> 

> > 

> > regards

> > Suman

> > 

> >>  1 file changed, 26 insertions(+), 5 deletions(-)

> >>

> >> diff --git a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

> >> index d905d614502b..130fbaacc4b1 100644

> >> --- a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

> >> +++ b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

> >> @@ -14,8 +14,12 @@ description: |

> >>    processor subsystems/clusters (R5FSS). The dual core cluster can be used

> >>    either in a LockStep mode providing safety/fault tolerance features or in a

> >>    Split mode providing two individual compute cores for doubling the compute

> >> -  capacity. These are used together with other processors present on the SoC

> >> -  to achieve various system level goals.

> >> +  capacity on most SoCs. These are used together with other processors present

> >> +  on the SoC to achieve various system level goals.

> >> +

> >> +  AM64x SoCs do not support LockStep mode, but rather a new non-safety mode

> >> +  called "Single-CPU" mode, where only Core0 is used, but with ability to use

> >> +  Core1's TCMs as well.

> >>  

> >>    Each Dual-Core R5F sub-system is represented as a single DTS node

> >>    representing the cluster, with a pair of child DT nodes representing

> >> @@ -33,6 +37,7 @@ properties:

> >>        - ti,am654-r5fss

> >>        - ti,j721e-r5fss

> >>        - ti,j7200-r5fss

> >> +      - ti,am64-r5fss

> >>  

> >>    power-domains:

> >>      description: |

> >> @@ -56,11 +61,12 @@ properties:

> >>  

> >>    ti,cluster-mode:

> >>      $ref: /schemas/types.yaml#/definitions/uint32

> >> -    enum: [0, 1]

> >>      description: |

> >>        Configuration Mode for the Dual R5F cores within the R5F cluster.

> >> -      Should be either a value of 1 (LockStep mode) or 0 (Split mode),

> >> -      default is LockStep mode if omitted.

> >> +      Should be either a value of 1 (LockStep mode) or 0 (Split mode) on

> >> +      most SoCs (AM65x, J721E, J7200), default is LockStep mode if omitted;

> >> +      and should be either a value of 0 (Split mode) or 2 (Single-CPU mode)

> >> +      on AM64x SoCs, default is Split mode if omitted.

> >>  

> >>  # R5F Processor Child Nodes:

> >>  # ==========================

> >> @@ -97,6 +103,7 @@ patternProperties:

> >>            - ti,am654-r5f

> >>            - ti,j721e-r5f

> >>            - ti,j7200-r5f

> >> +          - ti,am64-r5f

> >>  

> >>        reg:

> >>          items:

> >> @@ -198,6 +205,20 @@ patternProperties:

> >>  

> >>      unevaluatedProperties: false

> >>  

> >> +if:

> >> +  properties:

> >> +    compatible:

> >> +      enum:

> >> +        - ti,am64-r5fss

> >> +then:

> >> +  properties:

> >> +    ti,cluster-mode:

> >> +      enum: [0, 2]

> >> +else:

> >> +  properties:

> >> +    ti,cluster-mode:

> >> +      enum: [0, 1]

> >> +

> >>  required:

> >>    - compatible

> >>    - power-domains

> >>

> > 

>
Suman Anna May 24, 2021, 3:47 p.m. UTC | #4
On 5/21/21 3:40 PM, Mathieu Poirier wrote:
> Hi suman,

> 

> On Wed, May 12, 2021 at 09:47:44PM -0500, Suman Anna wrote:

>> Hi Rob,

>>

>> On 4/19/21 8:55 AM, Suman Anna wrote:

>>> Hi Rob,

>>>

>>> On 3/27/21 9:31 AM, Suman Anna wrote:

>>>> The K3 AM64x SoCs have two dual-core Arm R5F clusters/subsystems, with

>>>> 2 R5F cores each, both in the MAIN voltage domain.

>>>>

>>>> These clusters are a revised IP version compared to those present on

>>>> J721E and J7200 SoCs, and supports a new "Single-CPU" mode instead of

>>>> LockStep mode. Update the K3 R5F remoteproc bindings with the compatible

>>>> info relevant to these R5F clusters/subsystems on K3 AM64x SoCs.

>>>>

>>>> Signed-off-by: Suman Anna <s-anna@ti.com>

>>>> ---

>>>> v2: No changes

>>>>

>>>>  .../bindings/remoteproc/ti,k3-r5f-rproc.yaml  | 31 ++++++++++++++++---

>>>

>>> Looks like this patch has fallen through the cracks, can you please review and

>>> give your ack for this patch so that Bjorn can pick up the series for 5.13?

>>

>> Gentle reminder, do you have any comments on this patch. Appreciate your ack so

>> that we can get this in for 5.14?

> 

> If memory serves me well Rob indicated that he would not review or comment on

> bindings related to multi-core remote processors.  On the flip side he also

> mentioned that he would not object to their presence.  And since this is an

> increment to an existing binding rather than a new one, I think it is fair for

> us to pick it up.  

> 

> Rob - please intervene if my recollections are not accurate and accept my honest

> apologies.  Otherwise: 

> 

> Reviewed-by: Mathieu Poirier <mathieu.poirier@linaro.org> 

> 


Thank you Mathieu.

Bjorn,
Is it possible for you to give an immutable branch with just this bindings so we
can add the R5F nodes as well and avoid any checkpatch warnings on Nishanth's
tree with our K3 dts patches?

regards
Suman

>>

>> regards

>> Suman

>>

>>>

>>> regards

>>> Suman

>>>

>>>>  1 file changed, 26 insertions(+), 5 deletions(-)

>>>>

>>>> diff --git a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

>>>> index d905d614502b..130fbaacc4b1 100644

>>>> --- a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

>>>> +++ b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

>>>> @@ -14,8 +14,12 @@ description: |

>>>>    processor subsystems/clusters (R5FSS). The dual core cluster can be used

>>>>    either in a LockStep mode providing safety/fault tolerance features or in a

>>>>    Split mode providing two individual compute cores for doubling the compute

>>>> -  capacity. These are used together with other processors present on the SoC

>>>> -  to achieve various system level goals.

>>>> +  capacity on most SoCs. These are used together with other processors present

>>>> +  on the SoC to achieve various system level goals.

>>>> +

>>>> +  AM64x SoCs do not support LockStep mode, but rather a new non-safety mode

>>>> +  called "Single-CPU" mode, where only Core0 is used, but with ability to use

>>>> +  Core1's TCMs as well.

>>>>  

>>>>    Each Dual-Core R5F sub-system is represented as a single DTS node

>>>>    representing the cluster, with a pair of child DT nodes representing

>>>> @@ -33,6 +37,7 @@ properties:

>>>>        - ti,am654-r5fss

>>>>        - ti,j721e-r5fss

>>>>        - ti,j7200-r5fss

>>>> +      - ti,am64-r5fss

>>>>  

>>>>    power-domains:

>>>>      description: |

>>>> @@ -56,11 +61,12 @@ properties:

>>>>  

>>>>    ti,cluster-mode:

>>>>      $ref: /schemas/types.yaml#/definitions/uint32

>>>> -    enum: [0, 1]

>>>>      description: |

>>>>        Configuration Mode for the Dual R5F cores within the R5F cluster.

>>>> -      Should be either a value of 1 (LockStep mode) or 0 (Split mode),

>>>> -      default is LockStep mode if omitted.

>>>> +      Should be either a value of 1 (LockStep mode) or 0 (Split mode) on

>>>> +      most SoCs (AM65x, J721E, J7200), default is LockStep mode if omitted;

>>>> +      and should be either a value of 0 (Split mode) or 2 (Single-CPU mode)

>>>> +      on AM64x SoCs, default is Split mode if omitted.

>>>>  

>>>>  # R5F Processor Child Nodes:

>>>>  # ==========================

>>>> @@ -97,6 +103,7 @@ patternProperties:

>>>>            - ti,am654-r5f

>>>>            - ti,j721e-r5f

>>>>            - ti,j7200-r5f

>>>> +          - ti,am64-r5f

>>>>  

>>>>        reg:

>>>>          items:

>>>> @@ -198,6 +205,20 @@ patternProperties:

>>>>  

>>>>      unevaluatedProperties: false

>>>>  

>>>> +if:

>>>> +  properties:

>>>> +    compatible:

>>>> +      enum:

>>>> +        - ti,am64-r5fss

>>>> +then:

>>>> +  properties:

>>>> +    ti,cluster-mode:

>>>> +      enum: [0, 2]

>>>> +else:

>>>> +  properties:

>>>> +    ti,cluster-mode:

>>>> +      enum: [0, 1]

>>>> +

>>>>  required:

>>>>    - compatible

>>>>    - power-domains

>>>>

>>>

>>
Bjorn Andersson May 28, 2021, 3:17 a.m. UTC | #5
On Mon 24 May 10:47 CDT 2021, Suman Anna wrote:

> On 5/21/21 3:40 PM, Mathieu Poirier wrote:

> > Hi suman,

> > 

> > On Wed, May 12, 2021 at 09:47:44PM -0500, Suman Anna wrote:

> >> Hi Rob,

> >>

> >> On 4/19/21 8:55 AM, Suman Anna wrote:

> >>> Hi Rob,

> >>>

> >>> On 3/27/21 9:31 AM, Suman Anna wrote:

> >>>> The K3 AM64x SoCs have two dual-core Arm R5F clusters/subsystems, with

> >>>> 2 R5F cores each, both in the MAIN voltage domain.

> >>>>

> >>>> These clusters are a revised IP version compared to those present on

> >>>> J721E and J7200 SoCs, and supports a new "Single-CPU" mode instead of

> >>>> LockStep mode. Update the K3 R5F remoteproc bindings with the compatible

> >>>> info relevant to these R5F clusters/subsystems on K3 AM64x SoCs.

> >>>>

> >>>> Signed-off-by: Suman Anna <s-anna@ti.com>

> >>>> ---

> >>>> v2: No changes

> >>>>

> >>>>  .../bindings/remoteproc/ti,k3-r5f-rproc.yaml  | 31 ++++++++++++++++---

> >>>

> >>> Looks like this patch has fallen through the cracks, can you please review and

> >>> give your ack for this patch so that Bjorn can pick up the series for 5.13?

> >>

> >> Gentle reminder, do you have any comments on this patch. Appreciate your ack so

> >> that we can get this in for 5.14?

> > 

> > If memory serves me well Rob indicated that he would not review or comment on

> > bindings related to multi-core remote processors.  On the flip side he also

> > mentioned that he would not object to their presence.  And since this is an

> > increment to an existing binding rather than a new one, I think it is fair for

> > us to pick it up.  

> > 

> > Rob - please intervene if my recollections are not accurate and accept my honest

> > apologies.  Otherwise: 

> > 

> > Reviewed-by: Mathieu Poirier <mathieu.poirier@linaro.org> 

> > 

> 

> Thank you Mathieu.

> 

> Bjorn,

> Is it possible for you to give an immutable branch with just this bindings so we

> can add the R5F nodes as well and avoid any checkpatch warnings on Nishanth's

> tree with our K3 dts patches?

> 


Hi Suman,

That sounds rather ambitious, but you can now find this at:
https://git.kernel.org/pub/scm/linux/kernel/git/andersson/remoteproc.git/tag/?h=20210327143117.1840-2-s-anna@ti.com

Regards,
Bjorn

> regards

> Suman

> 

> >>

> >> regards

> >> Suman

> >>

> >>>

> >>> regards

> >>> Suman

> >>>

> >>>>  1 file changed, 26 insertions(+), 5 deletions(-)

> >>>>

> >>>> diff --git a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

> >>>> index d905d614502b..130fbaacc4b1 100644

> >>>> --- a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

> >>>> +++ b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

> >>>> @@ -14,8 +14,12 @@ description: |

> >>>>    processor subsystems/clusters (R5FSS). The dual core cluster can be used

> >>>>    either in a LockStep mode providing safety/fault tolerance features or in a

> >>>>    Split mode providing two individual compute cores for doubling the compute

> >>>> -  capacity. These are used together with other processors present on the SoC

> >>>> -  to achieve various system level goals.

> >>>> +  capacity on most SoCs. These are used together with other processors present

> >>>> +  on the SoC to achieve various system level goals.

> >>>> +

> >>>> +  AM64x SoCs do not support LockStep mode, but rather a new non-safety mode

> >>>> +  called "Single-CPU" mode, where only Core0 is used, but with ability to use

> >>>> +  Core1's TCMs as well.

> >>>>  

> >>>>    Each Dual-Core R5F sub-system is represented as a single DTS node

> >>>>    representing the cluster, with a pair of child DT nodes representing

> >>>> @@ -33,6 +37,7 @@ properties:

> >>>>        - ti,am654-r5fss

> >>>>        - ti,j721e-r5fss

> >>>>        - ti,j7200-r5fss

> >>>> +      - ti,am64-r5fss

> >>>>  

> >>>>    power-domains:

> >>>>      description: |

> >>>> @@ -56,11 +61,12 @@ properties:

> >>>>  

> >>>>    ti,cluster-mode:

> >>>>      $ref: /schemas/types.yaml#/definitions/uint32

> >>>> -    enum: [0, 1]

> >>>>      description: |

> >>>>        Configuration Mode for the Dual R5F cores within the R5F cluster.

> >>>> -      Should be either a value of 1 (LockStep mode) or 0 (Split mode),

> >>>> -      default is LockStep mode if omitted.

> >>>> +      Should be either a value of 1 (LockStep mode) or 0 (Split mode) on

> >>>> +      most SoCs (AM65x, J721E, J7200), default is LockStep mode if omitted;

> >>>> +      and should be either a value of 0 (Split mode) or 2 (Single-CPU mode)

> >>>> +      on AM64x SoCs, default is Split mode if omitted.

> >>>>  

> >>>>  # R5F Processor Child Nodes:

> >>>>  # ==========================

> >>>> @@ -97,6 +103,7 @@ patternProperties:

> >>>>            - ti,am654-r5f

> >>>>            - ti,j721e-r5f

> >>>>            - ti,j7200-r5f

> >>>> +          - ti,am64-r5f

> >>>>  

> >>>>        reg:

> >>>>          items:

> >>>> @@ -198,6 +205,20 @@ patternProperties:

> >>>>  

> >>>>      unevaluatedProperties: false

> >>>>  

> >>>> +if:

> >>>> +  properties:

> >>>> +    compatible:

> >>>> +      enum:

> >>>> +        - ti,am64-r5fss

> >>>> +then:

> >>>> +  properties:

> >>>> +    ti,cluster-mode:

> >>>> +      enum: [0, 2]

> >>>> +else:

> >>>> +  properties:

> >>>> +    ti,cluster-mode:

> >>>> +      enum: [0, 1]

> >>>> +

> >>>>  required:

> >>>>    - compatible

> >>>>    - power-domains

> >>>>

> >>>

> >>

>
Suman Anna May 28, 2021, 12:06 p.m. UTC | #6
On 5/27/21 10:17 PM, Bjorn Andersson wrote:
> On Mon 24 May 10:47 CDT 2021, Suman Anna wrote:

> 

>> On 5/21/21 3:40 PM, Mathieu Poirier wrote:

>>> Hi suman,

>>>

>>> On Wed, May 12, 2021 at 09:47:44PM -0500, Suman Anna wrote:

>>>> Hi Rob,

>>>>

>>>> On 4/19/21 8:55 AM, Suman Anna wrote:

>>>>> Hi Rob,

>>>>>

>>>>> On 3/27/21 9:31 AM, Suman Anna wrote:

>>>>>> The K3 AM64x SoCs have two dual-core Arm R5F clusters/subsystems, with

>>>>>> 2 R5F cores each, both in the MAIN voltage domain.

>>>>>>

>>>>>> These clusters are a revised IP version compared to those present on

>>>>>> J721E and J7200 SoCs, and supports a new "Single-CPU" mode instead of

>>>>>> LockStep mode. Update the K3 R5F remoteproc bindings with the compatible

>>>>>> info relevant to these R5F clusters/subsystems on K3 AM64x SoCs.

>>>>>>

>>>>>> Signed-off-by: Suman Anna <s-anna@ti.com>

>>>>>> ---

>>>>>> v2: No changes

>>>>>>

>>>>>>  .../bindings/remoteproc/ti,k3-r5f-rproc.yaml  | 31 ++++++++++++++++---

>>>>>

>>>>> Looks like this patch has fallen through the cracks, can you please review and

>>>>> give your ack for this patch so that Bjorn can pick up the series for 5.13?

>>>>

>>>> Gentle reminder, do you have any comments on this patch. Appreciate your ack so

>>>> that we can get this in for 5.14?

>>>

>>> If memory serves me well Rob indicated that he would not review or comment on

>>> bindings related to multi-core remote processors.  On the flip side he also

>>> mentioned that he would not object to their presence.  And since this is an

>>> increment to an existing binding rather than a new one, I think it is fair for

>>> us to pick it up.  

>>>

>>> Rob - please intervene if my recollections are not accurate and accept my honest

>>> apologies.  Otherwise: 

>>>

>>> Reviewed-by: Mathieu Poirier <mathieu.poirier@linaro.org> 

>>>

>>

>> Thank you Mathieu.

>>

>> Bjorn,

>> Is it possible for you to give an immutable branch with just this bindings so we

>> can add the R5F nodes as well and avoid any checkpatch warnings on Nishanth's

>> tree with our K3 dts patches?

>>

> 

> Hi Suman,

> 

> That sounds rather ambitious, but you can now find this at:

> https://git.kernel.org/pub/scm/linux/kernel/git/andersson/remoteproc.git/tag/?h=20210327143117.1840-2-s-anna@ti.com


Thanks a lot Bjorn. Appreciate this a lot.

regards
Suman

> 

> Regards,

> Bjorn

> 

>> regards

>> Suman

>>

>>>>

>>>> regards

>>>> Suman

>>>>

>>>>>

>>>>> regards

>>>>> Suman

>>>>>

>>>>>>  1 file changed, 26 insertions(+), 5 deletions(-)

>>>>>>

>>>>>> diff --git a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

>>>>>> index d905d614502b..130fbaacc4b1 100644

>>>>>> --- a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

>>>>>> +++ b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml

>>>>>> @@ -14,8 +14,12 @@ description: |

>>>>>>    processor subsystems/clusters (R5FSS). The dual core cluster can be used

>>>>>>    either in a LockStep mode providing safety/fault tolerance features or in a

>>>>>>    Split mode providing two individual compute cores for doubling the compute

>>>>>> -  capacity. These are used together with other processors present on the SoC

>>>>>> -  to achieve various system level goals.

>>>>>> +  capacity on most SoCs. These are used together with other processors present

>>>>>> +  on the SoC to achieve various system level goals.

>>>>>> +

>>>>>> +  AM64x SoCs do not support LockStep mode, but rather a new non-safety mode

>>>>>> +  called "Single-CPU" mode, where only Core0 is used, but with ability to use

>>>>>> +  Core1's TCMs as well.

>>>>>>  

>>>>>>    Each Dual-Core R5F sub-system is represented as a single DTS node

>>>>>>    representing the cluster, with a pair of child DT nodes representing

>>>>>> @@ -33,6 +37,7 @@ properties:

>>>>>>        - ti,am654-r5fss

>>>>>>        - ti,j721e-r5fss

>>>>>>        - ti,j7200-r5fss

>>>>>> +      - ti,am64-r5fss

>>>>>>  

>>>>>>    power-domains:

>>>>>>      description: |

>>>>>> @@ -56,11 +61,12 @@ properties:

>>>>>>  

>>>>>>    ti,cluster-mode:

>>>>>>      $ref: /schemas/types.yaml#/definitions/uint32

>>>>>> -    enum: [0, 1]

>>>>>>      description: |

>>>>>>        Configuration Mode for the Dual R5F cores within the R5F cluster.

>>>>>> -      Should be either a value of 1 (LockStep mode) or 0 (Split mode),

>>>>>> -      default is LockStep mode if omitted.

>>>>>> +      Should be either a value of 1 (LockStep mode) or 0 (Split mode) on

>>>>>> +      most SoCs (AM65x, J721E, J7200), default is LockStep mode if omitted;

>>>>>> +      and should be either a value of 0 (Split mode) or 2 (Single-CPU mode)

>>>>>> +      on AM64x SoCs, default is Split mode if omitted.

>>>>>>  

>>>>>>  # R5F Processor Child Nodes:

>>>>>>  # ==========================

>>>>>> @@ -97,6 +103,7 @@ patternProperties:

>>>>>>            - ti,am654-r5f

>>>>>>            - ti,j721e-r5f

>>>>>>            - ti,j7200-r5f

>>>>>> +          - ti,am64-r5f

>>>>>>  

>>>>>>        reg:

>>>>>>          items:

>>>>>> @@ -198,6 +205,20 @@ patternProperties:

>>>>>>  

>>>>>>      unevaluatedProperties: false

>>>>>>  

>>>>>> +if:

>>>>>> +  properties:

>>>>>> +    compatible:

>>>>>> +      enum:

>>>>>> +        - ti,am64-r5fss

>>>>>> +then:

>>>>>> +  properties:

>>>>>> +    ti,cluster-mode:

>>>>>> +      enum: [0, 2]

>>>>>> +else:

>>>>>> +  properties:

>>>>>> +    ti,cluster-mode:

>>>>>> +      enum: [0, 1]

>>>>>> +

>>>>>>  required:

>>>>>>    - compatible

>>>>>>    - power-domains

>>>>>>

>>>>>

>>>>

>>
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml
index d905d614502b..130fbaacc4b1 100644
--- a/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/ti,k3-r5f-rproc.yaml
@@ -14,8 +14,12 @@  description: |
   processor subsystems/clusters (R5FSS). The dual core cluster can be used
   either in a LockStep mode providing safety/fault tolerance features or in a
   Split mode providing two individual compute cores for doubling the compute
-  capacity. These are used together with other processors present on the SoC
-  to achieve various system level goals.
+  capacity on most SoCs. These are used together with other processors present
+  on the SoC to achieve various system level goals.
+
+  AM64x SoCs do not support LockStep mode, but rather a new non-safety mode
+  called "Single-CPU" mode, where only Core0 is used, but with ability to use
+  Core1's TCMs as well.
 
   Each Dual-Core R5F sub-system is represented as a single DTS node
   representing the cluster, with a pair of child DT nodes representing
@@ -33,6 +37,7 @@  properties:
       - ti,am654-r5fss
       - ti,j721e-r5fss
       - ti,j7200-r5fss
+      - ti,am64-r5fss
 
   power-domains:
     description: |
@@ -56,11 +61,12 @@  properties:
 
   ti,cluster-mode:
     $ref: /schemas/types.yaml#/definitions/uint32
-    enum: [0, 1]
     description: |
       Configuration Mode for the Dual R5F cores within the R5F cluster.
-      Should be either a value of 1 (LockStep mode) or 0 (Split mode),
-      default is LockStep mode if omitted.
+      Should be either a value of 1 (LockStep mode) or 0 (Split mode) on
+      most SoCs (AM65x, J721E, J7200), default is LockStep mode if omitted;
+      and should be either a value of 0 (Split mode) or 2 (Single-CPU mode)
+      on AM64x SoCs, default is Split mode if omitted.
 
 # R5F Processor Child Nodes:
 # ==========================
@@ -97,6 +103,7 @@  patternProperties:
           - ti,am654-r5f
           - ti,j721e-r5f
           - ti,j7200-r5f
+          - ti,am64-r5f
 
       reg:
         items:
@@ -198,6 +205,20 @@  patternProperties:
 
     unevaluatedProperties: false
 
+if:
+  properties:
+    compatible:
+      enum:
+        - ti,am64-r5fss
+then:
+  properties:
+    ti,cluster-mode:
+      enum: [0, 2]
+else:
+  properties:
+    ti,cluster-mode:
+      enum: [0, 1]
+
 required:
   - compatible
   - power-domains