2021-11-09 23:58:27

by Christian Gmeiner

[permalink] [raw]
Subject: [PATCH] arm64: dts: ti: k3-am64-main: add timesync router node

The Time Sync Event Router (TIMESYNC_INTRTR0) implements a set of
multiplexers to provide selection of active CPTS time sync events for
routing to CPTS capable modules.

This patch adds DT node TIMESYNC_INTRTR0 using "pinctrl-single" bindings.

Signed-off-by: Grygorii Strashko <[email protected]>
Signed-off-by: Vignesh Raghavendra <[email protected]>
Signed-off-by: Christian Gmeiner <[email protected]>
---
arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 8 ++++++++
1 file changed, 8 insertions(+)

diff --git a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
index 42d1d219a3fd..c9321a80353a 100644
--- a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
@@ -564,6 +564,14 @@ cpts@39000000 {
ti,cpts-ext-ts-inputs = <8>;
};

+ timesync_router: timesync-router@a40000 {
+ compatible = "pinctrl-single";
+ reg = <0x0 0xa40000 0x0 0x800>;
+ #pinctrl-cells = <1>;
+ pinctrl-single,register-width = <32>;
+ pinctrl-single,function-mask = <0x000107ff>;
+ };
+
usbss0: cdns-usb@f900000{
compatible = "ti,am64-usb";
reg = <0x00 0xf900000 0x00 0x100>;
--
2.33.1


2021-12-02 10:44:38

by Vignesh Raghavendra

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: ti: k3-am64-main: add timesync router node

Hi,

On 11/9/21 8:31 PM, Christian Gmeiner wrote:
> The Time Sync Event Router (TIMESYNC_INTRTR0) implements a set of
> multiplexers to provide selection of active CPTS time sync events for
> routing to CPTS capable modules.
>
> This patch adds DT node TIMESYNC_INTRTR0 using "pinctrl-single" bindings.
>
> Signed-off-by: Grygorii Strashko <[email protected]>
> Signed-off-by: Vignesh Raghavendra <[email protected]>
> Signed-off-by: Christian Gmeiner <[email protected]>
> ---
> arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> index 42d1d219a3fd..c9321a80353a 100644
> --- a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> @@ -564,6 +564,14 @@ cpts@39000000 {
> ti,cpts-ext-ts-inputs = <8>;
> };
>
> + timesync_router: timesync-router@a40000 {

Node with "pinctrl-single" would need to be named pinctrl@reg.
Else this would probably fail dtbs_check once pinctrl-single bindings is
converted to YAML format.

> + compatible = "pinctrl-single";
> + reg = <0x0 0xa40000 0x0 0x800>;
> + #pinctrl-cells = <1>;
> + pinctrl-single,register-width = <32>;
> + pinctrl-single,function-mask = <0x000107ff>;
> + };
> +
> usbss0: cdns-usb@f900000{
> compatible = "ti,am64-usb";
> reg = <0x00 0xf900000 0x00 0x100>;
>

Regards
Vignesh

2021-12-02 17:31:44

by Christian Gmeiner

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: ti: k3-am64-main: add timesync router node

Hi

>
> On 11/9/21 8:31 PM, Christian Gmeiner wrote:
> > The Time Sync Event Router (TIMESYNC_INTRTR0) implements a set of
> > multiplexers to provide selection of active CPTS time sync events for
> > routing to CPTS capable modules.
> >
> > This patch adds DT node TIMESYNC_INTRTR0 using "pinctrl-single" bindings.
> >
> > Signed-off-by: Grygorii Strashko <[email protected]>
> > Signed-off-by: Vignesh Raghavendra <[email protected]>
> > Signed-off-by: Christian Gmeiner <[email protected]>
> > ---
> > arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 8 ++++++++
> > 1 file changed, 8 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> > index 42d1d219a3fd..c9321a80353a 100644
> > --- a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> > +++ b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> > @@ -564,6 +564,14 @@ cpts@39000000 {
> > ti,cpts-ext-ts-inputs = <8>;
> > };
> >
> > + timesync_router: timesync-router@a40000 {
>
> Node with "pinctrl-single" would need to be named pinctrl@reg.
> Else this would probably fail dtbs_check once pinctrl-single bindings is
> converted to YAML format.
>

Okay.. will send a v2.

> > + compatible = "pinctrl-single";
> > + reg = <0x0 0xa40000 0x0 0x800>;
> > + #pinctrl-cells = <1>;
> > + pinctrl-single,register-width = <32>;
> > + pinctrl-single,function-mask = <0x000107ff>;
> > + };
> > +
> > usbss0: cdns-usb@f900000{
> > compatible = "ti,am64-usb";
> > reg = <0x00 0xf900000 0x00 0x100>;
> >
>

--
greets
--
Christian Gmeiner, MSc

https://christian-gmeiner.info/privacypolicy