* [v2] arm: dts: ls1021a: fix the value of TMR_FIPER1
@ 2017-11-29 6:54 Yangbo Lu
[not found] ` <20171129065458.4235-1-yangbo.lu-3arQi8VN3Tc@public.gmane.org>
0 siblings, 1 reply; 4+ messages in thread
From: Yangbo Lu @ 2017-11-29 6:54 UTC (permalink / raw)
To: devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
Cc: Rob Herring, Mark Rutland, Russell King, Shawn Guo, Yangbo Lu
The timer fixed interval period pulse generator register
is used to generate periodic pulses. The down count
register loads the value programmed in the fixed period
interval (FIPER). At every tick of the timer accumulator
overflow, the counter decrements by the value of
TMR_CTRL[TCLK_PERIOD]. It generates a pulse when the down
counter value reaches zero. It reloads the down counter
in the cycle following a pulse. To use the TMR_FIPER1
register to generate a 1 PPS event, the value
(10^9 nanoseconds) - TCLK_PERIOD should be programmed.
It should be 999999995 not 999999990 since TCLK_PERIOD
is 5.
Signed-off-by: Yangbo Lu <yangbo.lu-3arQi8VN3Tc@public.gmane.org>
---
Changes for v2:
- Added Shawn into cc list.
---
arch/arm/boot/dts/ls1021a.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/ls1021a.dtsi b/arch/arm/boot/dts/ls1021a.dtsi
index 64249726b3cb..a861a4b9e319 100644
--- a/arch/arm/boot/dts/ls1021a.dtsi
+++ b/arch/arm/boot/dts/ls1021a.dtsi
@@ -589,7 +589,7 @@
fsl,tclk-period = <5>;
fsl,tmr-prsc = <2>;
fsl,tmr-add = <0xaaaaaaab>;
- fsl,tmr-fiper1 = <999999990>;
+ fsl,tmr-fiper1 = <999999995>;
fsl,tmr-fiper2 = <99990>;
fsl,max-adj = <499999999>;
};
--
2.14.1
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 4+ messages in thread[parent not found: <20171129065458.4235-1-yangbo.lu-3arQi8VN3Tc@public.gmane.org>]
* Re: [v2] arm: dts: ls1021a: fix the value of TMR_FIPER1 [not found] ` <20171129065458.4235-1-yangbo.lu-3arQi8VN3Tc@public.gmane.org> @ 2017-11-30 0:29 ` Shawn Guo 2017-12-18 2:51 ` Y.b. Lu 0 siblings, 1 reply; 4+ messages in thread From: Shawn Guo @ 2017-11-30 0:29 UTC (permalink / raw) To: Yangbo Lu Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Mark Rutland, Rob Herring, Russell King On Wed, Nov 29, 2017 at 02:54:58PM +0800, Yangbo Lu wrote: > The timer fixed interval period pulse generator register > is used to generate periodic pulses. The down count > register loads the value programmed in the fixed period > interval (FIPER). At every tick of the timer accumulator > overflow, the counter decrements by the value of > TMR_CTRL[TCLK_PERIOD]. It generates a pulse when the down > counter value reaches zero. It reloads the down counter > in the cycle following a pulse. To use the TMR_FIPER1 > register to generate a 1 PPS event, the value > (10^9 nanoseconds) - TCLK_PERIOD should be programmed. > It should be 999999995 not 999999990 since TCLK_PERIOD > is 5. > > Signed-off-by: Yangbo Lu <yangbo.lu-3arQi8VN3Tc@public.gmane.org> Again, for arm dts, we usually use prefix like 'ARM: dts: ...' I fixed it up and applied the patch. Shawn > --- > Changes for v2: > - Added Shawn into cc list. > --- > arch/arm/boot/dts/ls1021a.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/ls1021a.dtsi b/arch/arm/boot/dts/ls1021a.dtsi > index 64249726b3cb..a861a4b9e319 100644 > --- a/arch/arm/boot/dts/ls1021a.dtsi > +++ b/arch/arm/boot/dts/ls1021a.dtsi > @@ -589,7 +589,7 @@ > fsl,tclk-period = <5>; > fsl,tmr-prsc = <2>; > fsl,tmr-add = <0xaaaaaaab>; > - fsl,tmr-fiper1 = <999999990>; > + fsl,tmr-fiper1 = <999999995>; > fsl,tmr-fiper2 = <99990>; > fsl,max-adj = <499999999>; > }; > -- > 2.14.1 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 4+ messages in thread
* RE: [v2] arm: dts: ls1021a: fix the value of TMR_FIPER1 2017-11-30 0:29 ` Shawn Guo @ 2017-12-18 2:51 ` Y.b. Lu [not found] ` <DB6PR0401MB2536CAFE776097EDEBE82C49F80E0-2mNvjAGDOPkZcyyZo0JLBI3W/0Ik+aLCnBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org> 0 siblings, 1 reply; 4+ messages in thread From: Y.b. Lu @ 2017-12-18 2:51 UTC (permalink / raw) To: Shawn Guo Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Mark Rutland, Rob Herring, Russell King Hi Shawn, Sorry for bother. I just couldn’t find this patch on your git tree. Could you help to check? Thanks a lot. Best regards, Yangbo Lu > -----Original Message----- > From: Shawn Guo [mailto:shawnguo@kernel.org] > Sent: 2017年11月30日 8:29 > To: Y.b. Lu <yangbo.lu@nxp.com> > Cc: devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; > linux-arm-kernel@lists.infradead.org; Mark Rutland <mark.rutland@arm.com>; > Rob Herring <robh+dt@kernel.org>; Russell King <linux@armlinux.org.uk> > Subject: Re: [v2] arm: dts: ls1021a: fix the value of TMR_FIPER1 > > On Wed, Nov 29, 2017 at 02:54:58PM +0800, Yangbo Lu wrote: > > The timer fixed interval period pulse generator register is used to > > generate periodic pulses. The down count register loads the value > > programmed in the fixed period interval (FIPER). At every tick of the > > timer accumulator overflow, the counter decrements by the value of > > TMR_CTRL[TCLK_PERIOD]. It generates a pulse when the down counter > > value reaches zero. It reloads the down counter in the cycle following > > a pulse. To use the TMR_FIPER1 register to generate a 1 PPS event, the > > value > > (10^9 nanoseconds) - TCLK_PERIOD should be programmed. > > It should be 999999995 not 999999990 since TCLK_PERIOD is 5. > > > > Signed-off-by: Yangbo Lu <yangbo.lu@nxp.com> > > > Again, for arm dts, we usually use prefix like 'ARM: dts: ...' > > I fixed it up and applied the patch. > > Shawn > > > --- > > Changes for v2: > > - Added Shawn into cc list. > > --- > > arch/arm/boot/dts/ls1021a.dtsi | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/arch/arm/boot/dts/ls1021a.dtsi > > b/arch/arm/boot/dts/ls1021a.dtsi index 64249726b3cb..a861a4b9e319 > > 100644 > > --- a/arch/arm/boot/dts/ls1021a.dtsi > > +++ b/arch/arm/boot/dts/ls1021a.dtsi > > @@ -589,7 +589,7 @@ > > fsl,tclk-period = <5>; > > fsl,tmr-prsc = <2>; > > fsl,tmr-add = <0xaaaaaaab>; > > - fsl,tmr-fiper1 = <999999990>; > > + fsl,tmr-fiper1 = <999999995>; > > fsl,tmr-fiper2 = <99990>; > > fsl,max-adj = <499999999>; > > }; > > -- > > 2.14.1 > > > > > > _______________________________________________ > > linux-arm-kernel mailing list > > linux-arm-kernel@lists.infradead.org > > https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Flist > > > s.infradead.org%2Fmailman%2Flistinfo%2Flinux-arm-kernel&data=02%7C01% > 7 > > > Cyangbo.lu%40nxp.com%7C703a7f8c490745cac3d708d537896a63%7C686ea > 1d3bc2b > > > 4c6fa92cd99c5c301635%7C0%7C0%7C636475985694945094&sdata=XLw6lJ > ONCoo2jq > > Hoixb%2BtlYTDcLC4tPbKWT3eQ7RI7c%3D&reserved=0 ^ permalink raw reply [flat|nested] 4+ messages in thread
[parent not found: <DB6PR0401MB2536CAFE776097EDEBE82C49F80E0-2mNvjAGDOPkZcyyZo0JLBI3W/0Ik+aLCnBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org>]
* Re: [v2] arm: dts: ls1021a: fix the value of TMR_FIPER1 [not found] ` <DB6PR0401MB2536CAFE776097EDEBE82C49F80E0-2mNvjAGDOPkZcyyZo0JLBI3W/0Ik+aLCnBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org> @ 2017-12-20 3:33 ` Shawn Guo 0 siblings, 0 replies; 4+ messages in thread From: Shawn Guo @ 2017-12-20 3:33 UTC (permalink / raw) To: Y.b. Lu Cc: Mark Rutland, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Russell King, Rob Herring, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org On Mon, Dec 18, 2017 at 02:51:06AM +0000, Y.b. Lu wrote: > Hi Shawn, > > Sorry for bother. I just couldn’t find this patch on your git tree. > Could you help to check? Sorry. I forgot to push the update. Just pushed now. Shawn -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-12-20 3:33 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-11-29 6:54 [v2] arm: dts: ls1021a: fix the value of TMR_FIPER1 Yangbo Lu
[not found] ` <20171129065458.4235-1-yangbo.lu-3arQi8VN3Tc@public.gmane.org>
2017-11-30 0:29 ` Shawn Guo
2017-12-18 2:51 ` Y.b. Lu
[not found] ` <DB6PR0401MB2536CAFE776097EDEBE82C49F80E0-2mNvjAGDOPkZcyyZo0JLBI3W/0Ik+aLCnBOFsp37pqbUKgpGm//BTAC/G2K4zDHf@public.gmane.org>
2017-12-20 3:33 ` Shawn Guo
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).