| Message ID | 20240816073908.2343650-6-b-padhi@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Reserve Timer Nodes to avoid clash with Remoteproc firmware | expand |
On 8/16/2024 1:09 PM, Beleswar Padhi wrote: > The remoteproc firmware like of R5F and DSPs 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 | > +-------------------+-------------+ > | c7x_0 | main_timer1 | > +-------------------+-------------+ > | c7x_1 | main_timer2 | > +-------------------+-------------+ > > Fixes: 29075cc09f43 ("arm64: dts: ti: Introduce AM62P5 family of SoCs") > Signed-off-by: Beleswar Padhi <b-padhi@ti.com> > --- > arch/arm64/boot/dts/ti/k3-j722s-evm.dts | 13 +++++++++++++ > 1 file changed, 13 insertions(+) > > diff --git a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts > index dd3b5f7039d7..e03beb0b5aad 100644 > --- a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts > +++ b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts > @@ -566,3 +566,16 @@ &mcasp1 { > 0 0 0 0 > >; > }; > + > +/* Timers are used by Remoteproc firmware */ > +&main_timer0 { > + status = "reserved"; > +}; > + > +&main_timer1 { > + status = "reserved"; > +}; > + > +&main_timer2 { > + status = "reserved"; > +}; Since IPC is not up-streamed for J722S , So I suggest to drop this patch from this series and include this when you upstream IPC support for J722S board .
On 26/08/24 10:30, Kumar, Udit wrote: > > On 8/16/2024 1:09 PM, Beleswar Padhi wrote: >> The remoteproc firmware like of R5F and DSPs 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 | >> +-------------------+-------------+ >> | c7x_0 | main_timer1 | >> +-------------------+-------------+ >> | c7x_1 | main_timer2 | >> +-------------------+-------------+ >> >> Fixes: 29075cc09f43 ("arm64: dts: ti: Introduce AM62P5 family of SoCs") >> Signed-off-by: Beleswar Padhi <b-padhi@ti.com> >> --- >> arch/arm64/boot/dts/ti/k3-j722s-evm.dts | 13 +++++++++++++ >> 1 file changed, 13 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts >> b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts >> index dd3b5f7039d7..e03beb0b5aad 100644 >> --- a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts >> +++ b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts >> @@ -566,3 +566,16 @@ &mcasp1 { >> 0 0 0 0 >> >; >> }; >> + >> +/* Timers are used by Remoteproc firmware */ >> +&main_timer0 { >> + status = "reserved"; >> +}; >> + >> +&main_timer1 { >> + status = "reserved"; >> +}; >> + >> +&main_timer2 { >> + status = "reserved"; >> +}; > > > Since IPC is not up-streamed for J722S , So I suggest to drop this > patch from this series Thanks for catching this! I have dropped this patch in v4 revision of this series. > > and include this when you upstream IPC support for J722S board . > >
diff --git a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts index dd3b5f7039d7..e03beb0b5aad 100644 --- a/arch/arm64/boot/dts/ti/k3-j722s-evm.dts +++ b/arch/arm64/boot/dts/ti/k3-j722s-evm.dts @@ -566,3 +566,16 @@ &mcasp1 { 0 0 0 0 >; }; + +/* Timers are used by Remoteproc firmware */ +&main_timer0 { + status = "reserved"; +}; + +&main_timer1 { + status = "reserved"; +}; + +&main_timer2 { + status = "reserved"; +};
The remoteproc firmware like of R5F and DSPs 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 | +-------------------+-------------+ | c7x_0 | main_timer1 | +-------------------+-------------+ | c7x_1 | main_timer2 | +-------------------+-------------+ Fixes: 29075cc09f43 ("arm64: dts: ti: Introduce AM62P5 family of SoCs") Signed-off-by: Beleswar Padhi <b-padhi@ti.com> --- arch/arm64/boot/dts/ti/k3-j722s-evm.dts | 13 +++++++++++++ 1 file changed, 13 insertions(+)