diff mbox series

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

Message ID 20240826104821.1516344-2-b-padhi@ti.com (mailing list archive)
State New, archived
Headers show
Series Reserve Timer Nodes to avoid clash with Remoteproc firmware | expand

Commit Message

Beleswar Prasad Padhi Aug. 26, 2024, 10:48 a.m. UTC
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(+)

Comments

Krzysztof Kozlowski Aug. 26, 2024, 10:54 a.m. UTC | #1
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
Krzysztof Kozlowski Aug. 26, 2024, 12:37 p.m. UTC | #2
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
Beleswar Prasad Padhi Aug. 27, 2024, 4:54 a.m. UTC | #3
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
>
diff mbox series

Patch

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>,