[PATCH v4 1/7] arm64: dts: ti: k3-j7200-som-p0: Change timer nodes status to reserved

Beleswar Padhi posted 7 patches 1 year, 3 months ago
[PATCH v4 1/7] arm64: dts: ti: k3-j7200-som-p0: Change timer nodes status to reserved
Posted by Beleswar Padhi 1 year, 3 months ago
The remoteproc firmware of R5F in the MAIN voltage domain use timers.
Therefore, change the status of the timer nodes to "reserved" to avoid
any clash. Usage is described as below:

	+===================+==========================+
	|  Remoteproc node  |        Timer Node        |
	+===================+==========================+
	| main_r5fss0_core0 | main_timer0, main_timer2 |
	+-------------------+--------------------------+
	| main_r5fss0_core1 | main_timer1              |
	+-------------------+--------------------------+

Fixes: c8a28ed4837c ("arm64: dts: ti: k3-j7200: Add general purpose timers")
Signed-off-by: Beleswar Padhi <b-padhi@ti.com>
---
 arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi
index deb380a83ea1..e78b4622a7d1 100644
--- a/arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi
@@ -266,6 +266,19 @@ &main_r5fss0 {
 	ti,cluster-mode = <0>;
 };
 
+/* Timers are used by Remoteproc firmware */
+&main_timer0 {
+	status = "reserved";
+};
+
+&main_timer1 {
+	status = "reserved";
+};
+
+&main_timer2 {
+	status = "reserved";
+};
+
 &main_r5fss0_core0 {
 	mboxes = <&mailbox0_cluster1 &mbox_main_r5fss0_core0>;
 	memory-region = <&main_r5fss0_core0_dma_memory_region>,
-- 
2.34.1
Re: [PATCH v4 1/7] arm64: dts: ti: k3-j7200-som-p0: Change timer nodes status to reserved
Posted by Krzysztof Kozlowski 1 year, 3 months ago
On 26/08/2024 12:48, Beleswar Padhi wrote:
> The remoteproc firmware of R5F in the MAIN voltage domain use timers.
> Therefore, change the status of the timer nodes to "reserved" to avoid
> any clash. Usage is described as below:
> 
> 	+===================+==========================+
> 	|  Remoteproc node  |        Timer Node        |
> 	+===================+==========================+
> 	| main_r5fss0_core0 | main_timer0, main_timer2 |
> 	+-------------------+--------------------------+
> 	| main_r5fss0_core1 | main_timer1              |
> 	+-------------------+--------------------------+
> 
> Fixes: c8a28ed4837c ("arm64: dts: ti: k3-j7200: Add general purpose timers")
> Signed-off-by: Beleswar Padhi <b-padhi@ti.com>

You keep sending the same. Where is the changelog? Why so many same
patchsets? Why so many resends without changes?

Best regards,
Krzysztof
Re: [PATCH v4 1/7] arm64: dts: ti: k3-j7200-som-p0: Change timer nodes status to reserved
Posted by Krzysztof Kozlowski 1 year, 3 months ago
On 26/08/2024 12:54, Krzysztof Kozlowski wrote:
> On 26/08/2024 12:48, Beleswar Padhi wrote:
>> The remoteproc firmware of R5F in the MAIN voltage domain use timers.
>> Therefore, change the status of the timer nodes to "reserved" to avoid
>> any clash. Usage is described as below:
>>
>> 	+===================+==========================+
>> 	|  Remoteproc node  |        Timer Node        |
>> 	+===================+==========================+
>> 	| main_r5fss0_core0 | main_timer0, main_timer2 |
>> 	+-------------------+--------------------------+
>> 	| main_r5fss0_core1 | main_timer1              |
>> 	+-------------------+--------------------------+
>>
>> Fixes: c8a28ed4837c ("arm64: dts: ti: k3-j7200: Add general purpose timers")
>> Signed-off-by: Beleswar Padhi <b-padhi@ti.com>
> 
> You keep sending the same. Where is the changelog? Why so many same
> patchsets? Why so many resends without changes?

OK, now cover letter came and I see the changelog. Some hickup on mail
servers.

Best regards,
Krzysztof
Re: [PATCH v4 1/7] arm64: dts: ti: k3-j7200-som-p0: Change timer nodes status to reserved
Posted by Beleswar Prasad Padhi 1 year, 3 months ago
On 26/08/24 18:07, Krzysztof Kozlowski wrote:
> On 26/08/2024 12:54, Krzysztof Kozlowski wrote:
>> On 26/08/2024 12:48, Beleswar Padhi wrote:
>>> The remoteproc firmware of R5F in the MAIN voltage domain use timers.
>>> Therefore, change the status of the timer nodes to "reserved" to avoid
>>> any clash. Usage is described as below:
>>>
>>> 	+===================+==========================+
>>> 	|  Remoteproc node  |        Timer Node        |
>>> 	+===================+==========================+
>>> 	| main_r5fss0_core0 | main_timer0, main_timer2 |
>>> 	+-------------------+--------------------------+
>>> 	| main_r5fss0_core1 | main_timer1              |
>>> 	+-------------------+--------------------------+
>>>
>>> Fixes: c8a28ed4837c ("arm64: dts: ti: k3-j7200: Add general purpose timers")
>>> Signed-off-by: Beleswar Padhi <b-padhi@ti.com>
>> You keep sending the same. Where is the changelog? Why so many same
>> patchsets? Why so many resends without changes?
> OK, now cover letter came and I see the changelog. Some hickup on mail
> servers.


I understand the mistake. From next time, will mention changelogs in 
patches as well as cover letter.

>
> Best regards,
> Krzysztof
>