mbox series

[v6,0/7] arm64: ti: k3-j7: Add the ESM & main domain watchdog nodes

Message ID 20230927023357.9883-1-j-keerthy@ti.com
Headers
Series arm64: ti: k3-j7: Add the ESM & main domain watchdog nodes |

Message

J, KEERTHY Sept. 27, 2023, 2:33 a.m. UTC 
  The series add the ESM & main domain watchdog nodes for j721s2,
j784s4 SOCs.

Changes in v5/v6:

        * Updated commit log and added comments for MCU & non-A72 watchdog
          instances disabling.

Changes in v4:

        * Added bootph-pre-ram for all the ESM instances needed for SPL.

Changes in v3:

        * Added all the RTI events for MAIN_ESM for j784s4 as 8 instances
          are enabled.
        * Rebased on top of 6.6-rc1
        * Tested for the watchdog reset

RESEND series - corrected krzysztof.kozlowski+dt@linaro.org ID

Changes in v2:

        * Added all the instances of watchdog on j784s4/j721s2
        * Fixed all 0x0 in dts to 0x00
        * Fixed couple of ESM event numbers for j721s2
        * Rebased to linux-next branch


Keerthy (7):
  arm64: dts: ti: k3-j721s2: Add ESM instances
  arm64: dts: ti: k3-j784s4: Add ESM instances
  arm64: dts: ti: k3-j7200: Add MCU domain ESM instance
  arm64: dts: ti: k3-j784s4-main: Add the main domain watchdog instances
  arm64: dts: ti: k3-j784s4-mcu: Add the mcu domain watchdog instances
  arm64: dts: ti: k3-j721s2-main: Add the main domain watchdog instances
  dts: ti: k3-j712s2-mcu: Add the mcu domain watchdog instances

 .../boot/dts/ti/k3-j7200-mcu-wakeup.dtsi      |   7 +
 arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi    | 100 +++++++++
 .../boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi     |  38 ++++
 arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi    | 195 ++++++++++++++++++
 .../boot/dts/ti/k3-j784s4-mcu-wakeup.dtsi     |  38 ++++
 5 files changed, 378 insertions(+)

Comments

Krzysztof Kozlowski Sept. 27, 2023, 7:53 a.m. UTC | #1 
On 27/09/2023 04:33, Keerthy wrote:
> There are totally 2 instances of watchdog module in MCU domain.
> These instances are coupled with the MCU domain R5F instances.
> Disabling them as they are not used by Linux.
> 
> Signed-off-by: Keerthy <j-keerthy@ti.com>

Still incorrect subject prefix.

Best regards,
Krzysztof
J, KEERTHY Sept. 27, 2023, 8:16 a.m. UTC | #2 
On 9/27/2023 1:23 PM, Krzysztof Kozlowski wrote:
> On 27/09/2023 04:33, Keerthy wrote:
>> There are totally 2 instances of watchdog module in MCU domain.
>> These instances are coupled with the MCU domain R5F instances.
>> Disabling them as they are not used by Linux.
>>
>> Signed-off-by: Keerthy <j-keerthy@ti.com>
> 
> Still incorrect subject prefix.

Sorry. I missed this :-(. I will fix all your comments in the next version.

> 
> Best regards,
> Krzysztof
>
J, KEERTHY Sept. 27, 2023, 8:25 a.m. UTC | #3 
On 9/27/2023 1:46 PM, J, KEERTHY wrote:
> 
> 
> On 9/27/2023 1:23 PM, Krzysztof Kozlowski wrote:
>> On 27/09/2023 04:33, Keerthy wrote:
>>> There are totally 2 instances of watchdog module in MCU domain.
>>> These instances are coupled with the MCU domain R5F instances.
>>> Disabling them as they are not used by Linux.
>>>
>>> Signed-off-by: Keerthy <j-keerthy@ti.com>
>>
>> Still incorrect subject prefix.
> 
> Sorry. I missed this :-(. I will fix all your comments in the next version.

Hi Krzysztof,

If there are no further comments on V6. Can i resend this patch alone 
with subject corrected?

Regards,
Keerthy

> 
>>
>> Best regards,
>> Krzysztof
>>