linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: ti: k3-am64-main: add timesync router node
@ 2021-11-09 15:01 Christian Gmeiner
  2021-12-02 10:44 ` Vignesh Raghavendra
  0 siblings, 1 reply; 3+ messages in thread
From: Christian Gmeiner @ 2021-11-09 15:01 UTC (permalink / raw)
  To: linux-kernel
  Cc: Christian Gmeiner, Grygorii Strashko, Vignesh Raghavendra,
	Nishanth Menon, Tero Kristo, Rob Herring, linux-arm-kernel,
	devicetree

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 <grygorii.strashko@ti.com>
Signed-off-by: Vignesh Raghavendra <vigneshr@ti.com>
Signed-off-by: Christian Gmeiner <christian.gmeiner@gmail.com>
---
 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


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: dts: ti: k3-am64-main: add timesync router node
  2021-11-09 15:01 [PATCH] arm64: dts: ti: k3-am64-main: add timesync router node Christian Gmeiner
@ 2021-12-02 10:44 ` Vignesh Raghavendra
  2021-12-02 17:31   ` Christian Gmeiner
  0 siblings, 1 reply; 3+ messages in thread
From: Vignesh Raghavendra @ 2021-12-02 10:44 UTC (permalink / raw)
  To: Christian Gmeiner, linux-kernel
  Cc: Grygorii Strashko, Nishanth Menon, Tero Kristo, Rob Herring,
	linux-arm-kernel, devicetree

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 <grygorii.strashko@ti.com>
> Signed-off-by: Vignesh Raghavendra <vigneshr@ti.com>
> Signed-off-by: Christian Gmeiner <christian.gmeiner@gmail.com>
> ---
>  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

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] arm64: dts: ti: k3-am64-main: add timesync router node
  2021-12-02 10:44 ` Vignesh Raghavendra
@ 2021-12-02 17:31   ` Christian Gmeiner
  0 siblings, 0 replies; 3+ messages in thread
From: Christian Gmeiner @ 2021-12-02 17:31 UTC (permalink / raw)
  To: Vignesh Raghavendra
  Cc: LKML, Grygorii Strashko, Nishanth Menon, Tero Kristo,
	Rob Herring, linux-arm-kernel, devicetree

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 <grygorii.strashko@ti.com>
> > Signed-off-by: Vignesh Raghavendra <vigneshr@ti.com>
> > Signed-off-by: Christian Gmeiner <christian.gmeiner@gmail.com>
> > ---
> >  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

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-12-02 17:31 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-09 15:01 [PATCH] arm64: dts: ti: k3-am64-main: add timesync router node Christian Gmeiner
2021-12-02 10:44 ` Vignesh Raghavendra
2021-12-02 17:31   ` Christian Gmeiner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).