Received-SPF: pass (google.com: domain of linux-kernel+bounces-68981-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <c6a3fc6a-d21e-4b76-bfc0-82de2f729ad7@siemens.com>
Date: Fri, 16 Feb 2024 17:48:51 +0100
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] arm64: dts: ti: Add reserved memory for watchdog
Content-Language: en-US
To: huaqian.li@siemens.com, nm@ti.com, vigneshr@ti.com, kristo@kernel.org,
 robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org
Cc: linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
 linux-kernel@vger.kernel.org, baocheng.su@siemens.com
References: <20240117060654.109424-1-huaqian.li@siemens.com>
From: Jan Kiszka <jan.kiszka@siemens.com>
In-Reply-To: <20240117060654.109424-1-huaqian.li@siemens.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?T1lJQnh2eGFrVUZEam1jWjB2Z3l4QjVXWWtiK1JKOXNyeTNTZ2FuNE1TMTNV?=
 =?utf-8?B?QS9xT3BtUnJsOEVnUlBCZm00cEswYlBFNmdyZXFUa1Z5czF2dmZoZHJjSzZ5?=
 =?utf-8?B?SCtpMmJGaWN5WFQ5M01oTEQ0VGd2bko1UzBQcmQ2eEhpNjZOd05Jcy8yV25s?=
 =?utf-8?B?cGs3RWE2Mzd3Nkd4NWJvRis0d0V2amxqdE1rcE56S1dVWnJOVmxmYmtCYWxH?=
 =?utf-8?B?THBEN0VSYVhWa1hsOGJpYTQ2TlVSTGM3bWtsUlpYcEIwSkJ4K0lYeU1WVjJC?=
 =?utf-8?B?L3IwOHJNTVdNTGxrcXNpVjJhT0VCeDJsKzhCaUZ2WXJqVkJieUl2WnZhNk05?=
 =?utf-8?B?Z0t6RnJzMkprbEVYZzZveTJCVGZMVGdOZk4waDl0R0xYWFA0N080QkNLYXY5?=
 =?utf-8?B?NlN5ZlpGZFpsMzFLOXhUZzVxQ2krTWRMUXNhUEluZXRocUcrQWVrME5QM092?=
 =?utf-8?B?d0tKd1ZWSmNMcTdKZEZ4N3NqQVJEeGFhazRJSGJYdVdDZDBJYTNrOS8wVVZK?=
 =?utf-8?B?MjRtakM2RVBsSWkzRks0MjVWUU4vVWlWMC84WXltRG9lUVRkN0l1aHQ2QXZ3?=
 =?utf-8?B?am42b2RMYS9lWU84eEk3MmowS0Y3L1ZSTzBMQ1VvVm5IODlkOWM3eUFWZDNp?=
 =?utf-8?B?WTBpUlgrWmpnZWZOL0dkQnNoZ0xySEdnaEJhTXBHa1NIWjdoRlJ0SWVzZXBo?=
 =?utf-8?B?dXhzVkxOVlg3azRPRUJWbkg0RWk1TzQ1Rm9WMHpEeXMraExTV28zUk1tYzY5?=
 =?utf-8?B?T21CeU05N3NGTU1HM3c4SnBNZlMrT2N2ZlhtZ09qakZoMzBuaWFlaHZtUXNq?=
 =?utf-8?B?aWFWV0ZDZVJ4MEpPRE5DaEhMU1RqSmljWmp6NzBIVXlhcTRSeUl1aUZzcGxW?=
 =?utf-8?B?SGw2L0JqQXpDdXllV0JoRmxvSzFubS9xeHBXd3JUWVR4cE9UZGRKQy9RdzEx?=
 =?utf-8?B?c1VPMldjOTFzOVV0OU5uWVcxbStLaXk1cW1UTE5oOVRvTXRtbzZ0SDh0eUI4?=
 =?utf-8?B?TGhGY1dGWWFnUi9uUjY5OSs4N2NFNW9lQ3o2RlVNYjB0elVsSU56RjhDQWJG?=
 =?utf-8?B?c25XMElIZ1NwZHZrWnpIeGljNHMrL3YrelVlcklhd1l4K2lrOTNkeHBrQ0ZC?=
 =?utf-8?B?b3JFK3EvdnU5NUZUTzM1RFh5bXZHZzNsSkF1UDJjTUVsaDRrZm1odDRaRWpK?=
 =?utf-8?B?bDc3cjc5MGttaVZObXZnekkwK2djZGhXNkJmM0JZTmxCcW1QdGlIcWNydEZ6?=
 =?utf-8?B?TVZmdFNzTklsRWQ4QWc4UVZiK1RUbmN4cEF1QmRWRlZMcmNFRUkyUHB2bm1Y?=
 =?utf-8?B?QUtEOHJJQVdabDh5d0M4eGZ0OGxFdExzUUhGVXB1UnJPZmxhK0tqY2Nkd3J6?=
 =?utf-8?B?a2ZXRHBRNGk0Mlk0NmVVZ0pBdHB1NUZJTnNXUnl5a0JVQ2xzaXY0cUl4dXN1?=
 =?utf-8?B?UmlBTHB6M2ErdGhybS9GbzRnVDFoYzlUN081RXFNbGh2YkNRM29NelZBclZP?=
 =?utf-8?B?U3Y5MlJab1VTUStvRWVwQnZUZ1dLTmNDVkFnak0rWjBuZVp2VWZXUXM5YStV?=
 =?utf-8?B?eExPSDF4M29kd3pSQ1Mzd0ZhY3I2R2tkWEs5azFwa2V2SG5sZmY2M2xXQkZq?=
 =?utf-8?B?eEh0NFBIUkxMMkk2dmtKVUg5Um1Yb0xRMkl6eFhzNTNEbXZOZ3NtUTFWZ2lK?=
 =?utf-8?B?VlFzTDVjY1VXREhZazYxeGc5Mkh2anhBenBFNFZvWkl5d1Q2R3A3STZ4c0c0?=
 =?utf-8?B?N2pydHVLbzRLdDh3Y2dwdDkyM0RLY21sZ3Y1cXg1ZEliZ3d3U1hSS0tkODRk?=
 =?utf-8?B?RFFob0V5azJpZE9jZTR6cEJ1WCtMeTZQTlFMRWVGbHVwRDZhTjEwWWovcXMy?=
 =?utf-8?B?NmI3MTJaTC9rbzJTT2hZTnUyck5NN2NaWGRaUGNtUm1EVm9WRjhGN2tHVDNj?=
 =?utf-8?B?UHg3ZVZkRGRkSjAwcmI0VWU0aWp6bTlhcWNwbFNlQlJncGk1djFSdTlCZXZl?=
 =?utf-8?B?SFh1Y2NrdzNXK210V3dGN2VTNDNGVlA4ZS9kTzNtRFhVdUY1Y2RYQ01EZlM2?=
 =?utf-8?B?cDZYcXN3NTd3WkZQNDE4djV5YS9BZ2JnbWI4MTlYbWhTRlp5dGNRYTBsQ0tQ?=
 =?utf-8?B?dDU4NDVwRDZtbTZUcDBmYzBka04wUXlIemF1d3YzbVo5VEFIUDd2ckx4S2hm?=
 =?utf-8?B?N2c9PQ==?=
X-OriginatorOrg: siemens.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 77632b14-1095-43ed-7920-08dc2f0f28f0
X-MS-Exchange-CrossTenant-AuthSource: AS4PR10MB6181.EURPRD10.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Feb 2024 16:48:53.8856
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 38ae3bcd-9579-4fd4-adda-b42e1495d55a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: T0v4XtenadS2jIGZ+unh0fd2O0nahM4j6nEByQA9Yr92yA4FkblIJ8fqv6f8JoWhh+obEN2npkPTucovxS25rQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAWPR10MB7101

On 17.01.24 07:06, huaqian.li@siemens.com wrote:
> From: Li Hua Qian <huaqian.li@siemens.com>
> 
> This patch adds a reserved memory for the TI AM65X platform watchdog
> to reserve the specific info, triggering the watchdog reset in last
> boot, to know if the board reboot is due to a watchdog reset.
> 
> Signed-off-by: Li Hua Qian <huaqian.li@siemens.com>
> ---
> The driver and binding have hit the master, which could be found in the
> following link.
> - watchdog:rit_wdt: Add support for WDIOF_CARDRESET
> - dt-bindings: watchdog: ti,rti-wdt: Add support for WDIOF_CARDRESET
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=29057cc5bddc785ea0a11534d7ad2546fa0872d3
> 
>  arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
> index 61a634afaa4f..88321799d468 100644
> --- a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
> @@ -75,6 +75,12 @@ rtos_ipc_memory_region: ipc-memories@a2000000 {
>  			alignment = <0x1000>;
>  			no-map;
>  		};
> +
> +		/* To reserve the power-on(PON) reason for watchdog reset */
> +		wdt_reset_memory_region: wdt-memory@a2200000 {
> +			reg = <0x00 0xa2200000 0x00 0x1000>;
> +			no-map;
> +		};
>  	};
>  
>  	leds {
> @@ -1427,6 +1433,10 @@ &mcu_r5fss0_core1 {
>  	mboxes = <&mailbox0_cluster1>, <&mbox_mcu_r5fss0_core1>;
>  };
>  
> +&mcu_rti1 {
> +	memory-region = <&wdt_reset_memory_region>;
> +};
> +
>  &icssg0_mdio {
>  	status = "okay";
>  	pinctrl-names = "default";

Is this not forgotten (now that my IOT2050 series was merged)?

Thanks,
Jan

-- 
Siemens AG, Technology
Linux Expert Center