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 |
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
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
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 --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>,
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(+)