From: Roger Quadros <[email protected]>
The PRUSS module has a SYSCFG which is unique. The SYSCFG
has two additional unique fields called STANDBY_INIT and
SUB_MWAIT in addition to regular IDLE_MODE and STANDBY_MODE
fields. Add the bindings for this new sysc type.
Signed-off-by: Roger Quadros <[email protected]>
Signed-off-by: Suman Anna <[email protected]>
---
Documentation/devicetree/bindings/bus/ti-sysc.txt | 1 +
include/dt-bindings/bus/ti-sysc.h | 4 ++++
2 files changed, 5 insertions(+)
diff --git a/Documentation/devicetree/bindings/bus/ti-sysc.txt b/Documentation/devicetree/bindings/bus/ti-sysc.txt
index 233eb8294204..c984143d08d2 100644
--- a/Documentation/devicetree/bindings/bus/ti-sysc.txt
+++ b/Documentation/devicetree/bindings/bus/ti-sysc.txt
@@ -38,6 +38,7 @@ Required standard properties:
"ti,sysc-dra7-mcasp"
"ti,sysc-usb-host-fs"
"ti,sysc-dra7-mcan"
+ "ti,sysc-pruss"
- reg shall have register areas implemented for the interconnect
target module in question such as revision, sysc and syss
diff --git a/include/dt-bindings/bus/ti-sysc.h b/include/dt-bindings/bus/ti-sysc.h
index babd08a1d226..76b07826ed05 100644
--- a/include/dt-bindings/bus/ti-sysc.h
+++ b/include/dt-bindings/bus/ti-sysc.h
@@ -18,6 +18,10 @@
#define SYSC_DRA7_MCAN_ENAWAKEUP (1 << 4)
+/* PRUSS sysc found on AM33xx/AM43xx/AM57xx */
+#define SYSC_PRUSS_SUB_MWAIT (1 << 5)
+#define SYSC_PRUSS_STANDBY_INIT (1 << 4)
+
/* SYSCONFIG STANDBYMODE/MIDLEMODE/SIDLEMODE supported by hardware */
#define SYSC_IDLE_FORCE 0
#define SYSC_IDLE_NO 1
--
2.23.0
Hi Suman,
On 25/02/2020 22:46, Suman Anna wrote:
> From: Roger Quadros <[email protected]>
>
> The PRUSS module has a SYSCFG which is unique. The SYSCFG
> has two additional unique fields called STANDBY_INIT and
> SUB_MWAIT in addition to regular IDLE_MODE and STANDBY_MODE
> fields. Add the bindings for this new sysc type.
>
> Signed-off-by: Roger Quadros <[email protected]>
> Signed-off-by: Suman Anna <[email protected]>
> ---
> Documentation/devicetree/bindings/bus/ti-sysc.txt | 1 +
> include/dt-bindings/bus/ti-sysc.h | 4 ++++
> 2 files changed, 5 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/bus/ti-sysc.txt b/Documentation/devicetree/bindings/bus/ti-sysc.txt
> index 233eb8294204..c984143d08d2 100644
> --- a/Documentation/devicetree/bindings/bus/ti-sysc.txt
> +++ b/Documentation/devicetree/bindings/bus/ti-sysc.txt
> @@ -38,6 +38,7 @@ Required standard properties:
> "ti,sysc-dra7-mcasp"
> "ti,sysc-usb-host-fs"
> "ti,sysc-dra7-mcan"
> + "ti,sysc-pruss"
>
> - reg shall have register areas implemented for the interconnect
> target module in question such as revision, sysc and syss
> diff --git a/include/dt-bindings/bus/ti-sysc.h b/include/dt-bindings/bus/ti-sysc.h
Did you intentionally leave this here? It should be part of 2nd patch?
> index babd08a1d226..76b07826ed05 100644
> --- a/include/dt-bindings/bus/ti-sysc.h
> +++ b/include/dt-bindings/bus/ti-sysc.h
> @@ -18,6 +18,10 @@
>
> #define SYSC_DRA7_MCAN_ENAWAKEUP (1 << 4)
>
> +/* PRUSS sysc found on AM33xx/AM43xx/AM57xx */
> +#define SYSC_PRUSS_SUB_MWAIT (1 << 5)
> +#define SYSC_PRUSS_STANDBY_INIT (1 << 4)
> +
> /* SYSCONFIG STANDBYMODE/MIDLEMODE/SIDLEMODE supported by hardware */
> #define SYSC_IDLE_FORCE 0
> #define SYSC_IDLE_NO 1
>
--
cheers,
-roger
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
Hi Roger,
On 2/26/20 2:42 AM, Roger Quadros wrote:
> Hi Suman,
>
> On 25/02/2020 22:46, Suman Anna wrote:
>> From: Roger Quadros <[email protected]>
>>
>> The PRUSS module has a SYSCFG which is unique. The SYSCFG
>> has two additional unique fields called STANDBY_INIT and
>> SUB_MWAIT in addition to regular IDLE_MODE and STANDBY_MODE
>> fields. Add the bindings for this new sysc type.
>>
>> Signed-off-by: Roger Quadros <[email protected]>
>> Signed-off-by: Suman Anna <[email protected]>
>> ---
>> Documentation/devicetree/bindings/bus/ti-sysc.txt | 1 +
>> include/dt-bindings/bus/ti-sysc.h | 4 ++++
>> 2 files changed, 5 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/bus/ti-sysc.txt
>> b/Documentation/devicetree/bindings/bus/ti-sysc.txt
>> index 233eb8294204..c984143d08d2 100644
>> --- a/Documentation/devicetree/bindings/bus/ti-sysc.txt
>> +++ b/Documentation/devicetree/bindings/bus/ti-sysc.txt
>> @@ -38,6 +38,7 @@ Required standard properties:
>> "ti,sysc-dra7-mcasp"
>> "ti,sysc-usb-host-fs"
>> "ti,sysc-dra7-mcan"
>> + "ti,sysc-pruss"
>> - reg shall have register areas implemented for the
>> interconnect
>> target module in question such as revision, sysc and syss
>> diff --git a/include/dt-bindings/bus/ti-sysc.h
>> b/include/dt-bindings/bus/ti-sysc.h
>
> Did you intentionally leave this here? It should be part of 2nd patch?
No, not really, include/bindings are also considered part of bindings.
This patch alone should be enough for you to add the DT nodes.
regards
Suman
>
>> index babd08a1d226..76b07826ed05 100644
>> --- a/include/dt-bindings/bus/ti-sysc.h
>> +++ b/include/dt-bindings/bus/ti-sysc.h
>> @@ -18,6 +18,10 @@
>> #define SYSC_DRA7_MCAN_ENAWAKEUP (1 << 4)
>> +/* PRUSS sysc found on AM33xx/AM43xx/AM57xx */
>> +#define SYSC_PRUSS_SUB_MWAIT (1 << 5)
>> +#define SYSC_PRUSS_STANDBY_INIT (1 << 4)
>> +
>> /* SYSCONFIG STANDBYMODE/MIDLEMODE/SIDLEMODE supported by hardware */
>> #define SYSC_IDLE_FORCE 0
>> #define SYSC_IDLE_NO 1
>>
>
* Suman Anna <[email protected]> [200226 16:39]:
> Hi Roger,
>
> On 2/26/20 2:42 AM, Roger Quadros wrote:
> > Hi Suman,
> >
> > On 25/02/2020 22:46, Suman Anna wrote:
> >> From: Roger Quadros <[email protected]>
> >>
> >> The PRUSS module has a SYSCFG which is unique. The SYSCFG
> >> has two additional unique fields called STANDBY_INIT and
> >> SUB_MWAIT in addition to regular IDLE_MODE and STANDBY_MODE
> >> fields. Add the bindings for this new sysc type.
> >>
> >> Signed-off-by: Roger Quadros <[email protected]>
> >> Signed-off-by: Suman Anna <[email protected]>
> >> ---
> >> Documentation/devicetree/bindings/bus/ti-sysc.txt | 1 +
> >> include/dt-bindings/bus/ti-sysc.h | 4 ++++
> >> 2 files changed, 5 insertions(+)
> >>
> >> diff --git a/Documentation/devicetree/bindings/bus/ti-sysc.txt
> >> b/Documentation/devicetree/bindings/bus/ti-sysc.txt
> >> index 233eb8294204..c984143d08d2 100644
> >> --- a/Documentation/devicetree/bindings/bus/ti-sysc.txt
> >> +++ b/Documentation/devicetree/bindings/bus/ti-sysc.txt
> >> @@ -38,6 +38,7 @@ Required standard properties:
> >> "ti,sysc-dra7-mcasp"
> >> "ti,sysc-usb-host-fs"
> >> "ti,sysc-dra7-mcan"
> >> + "ti,sysc-pruss"
> >> - reg shall have register areas implemented for the
> >> interconnect
> >> target module in question such as revision, sysc and syss
> >> diff --git a/include/dt-bindings/bus/ti-sysc.h
> >> b/include/dt-bindings/bus/ti-sysc.h
> >
> > Did you intentionally leave this here? It should be part of 2nd patch?
>
> No, not really, include/bindings are also considered part of bindings.
> This patch alone should be enough for you to add the DT nodes.
Well I don't care either way, sort of would prefer to have this
with the device patch in case somebody starts back porting
driver changes.
Anyways, let's wayt for Rob's ack on this, maybe he has also
preference on the ti-sysc.h changes.
Regards,
Tony