From: Wang Dongsheng <Dongsheng.Wang@freescale.com>
To: Stuart Yoder <b08248@gmail.com>
Cc: "shawn.guo@linaro.org" <shawn.guo@linaro.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
Huan Wang <alison.wang@freescale.com>
Subject: RE: [PATCH v3 1/2] Documentation: DT: FTM: add FTM0 be used as alarm timer
Date: Fri, 14 Aug 2015 03:24:35 +0000 [thread overview]
Message-ID: <SN1PR0301MB16165DCFD09CEB424A91F79F9D7C0@SN1PR0301MB1616.namprd03.prod.outlook.com> (raw)
In-Reply-To: <CALRxmdDGgYh+_cOPB43gyuZwu_zo+wE0qHh8nWJTjiQN__hZ3w@mail.gmail.com>
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 3395 bytes --]
Thank Yoder,
Fix them in next version.
Regards,
-Dongsheng
> -----Original Message-----
> From: Stuart Yoder [mailto:b08248@gmail.com]
> Sent: Friday, August 14, 2015 12:23 AM
> To: Wang Dongsheng-B40534
> Cc: shawn.guo@linaro.org; devicetree@vger.kernel.org; linux-
> kernel@vger.kernel.org; linux-arm-kernel@lists.infradead.org; Wang Huan-B18965
> Subject: Re: [PATCH v3 1/2] Documentation: DT: FTM: add FTM0 be used as alarm
> timer
>
> On Tue, Aug 11, 2015 at 11:01 PM, Dongsheng Wang
> <dongsheng.wang@freescale.com> wrote:
> > From: Wang Dongsheng <dongsheng.wang@freescale.com>
> >
> > In freescale layerscape platform there is only FTM0 can be used as
> > alarm timer to wake up system. So add FTM0 description for devicetree
> > document.
>
> Suggestion:
>
> In the Freescale Layerscape platform a flextimer module can be used
> as an alarm timer to wake up the system. Define a binding for this
> alarm.
>
> > Signed-off-by: Wang Dongsheng <dongsheng.wang@freescale.com>
> > ---
> > V3:
> > Include this patch in V3.
> >
> > diff --git a/Documentation/devicetree/bindings/timer/fsl,ftm-timer.txt
> b/Documentation/devicetree/bindings/timer/fsl,ftm-timer.txt
> > index aa8c402..380a0b3d 100644
> > --- a/Documentation/devicetree/bindings/timer/fsl,ftm-timer.txt
> > +++ b/Documentation/devicetree/bindings/timer/fsl,ftm-timer.txt
> > @@ -1,5 +1,7 @@
> > Freescale FlexTimer Module (FTM) Timer
> >
> > +* Default FTM Timer
> > +
> > Required properties:
> >
> > - compatible : should be "fsl,ftm-timer"
> > @@ -29,3 +31,26 @@ ftm: ftm@400b8000 {
> > <&clks VF610_CLK_FTM3_EXT_FIX_EN>;
> > big-endian;
> > };
> > +
> > +* FTM Alarm Timer
> > + The default FTM device contains eight FlexTimer modules. FlexTimer1 is in
> > + on-domain(power is not switched off in deep sleep mode). Other seven
> FlexTimer
> > + modules(flextimer2/3/4/5/6/7/8) are in off-domain (power is switched off in
> > + deep sleep mode).
>
> Suggestion:
>
> This binding describes a Flextimer (FTM) instance that can be used as an alarm
> to wake up a system in deep sleep. The flextimer module with alarm
> support is in a power domain that remains awake when the SoC is in
> deep sleep mode.
>
> (don't use flextimer1 in the binding...for all you know in the next SoC it
> will be flextimer7 that has this capability)
>
> > +Required properties:
> > +
> > +- compatible : should be "fsl,ftm-alarm".
> > +- reg : should contain base address and length of FTM timer 0 register.
>
> "should contain the address and size of the FTM alarm module registers"
>
> (don't use the number 0)
>
> > +- interrupts : Should contain FTM 0 interrupt.
>
> "describes the FTM alarm interrupt"
>
> > +- big-endian: One boolean property, the big endian mode will be in use if it
> is
> > + present, or the little endian mode will be in use for all the device
> registers.
> > +
> > +Example:
> > +ftm0: ftm0@29d0000 {
> > + compatible = "fsl,ftm-alarm";
> > + reg = <0x0 0x29d0000 0x0 0x10000>;
> > + interrupts = <GIC_SPI 118 IRQ_TYPE_LEVEL_HIGH>;
> > + big-endian;
> > + status = "disabled";
> > +};
>
> Drop the number 0 from the names in your example.
>
> Thanks,
> Stuart
ÿôèº{.nÇ+·®+%Ëÿ±éݶ\x17¥wÿº{.nÇ+·¥{±þG«éÿ{ayº\x1dÊÚë,j\a¢f£¢·hïêÿêçz_è®\x03(éÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?¨èÚ&£ø§~á¶iOæ¬z·vØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?I¥
prev parent reply other threads:[~2015-08-14 3:40 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-12 4:01 [PATCH v3 1/2] Documentation: DT: FTM: add FTM0 be used as alarm timer Dongsheng Wang
2015-08-12 4:01 ` [PATCH v3 2/2] ARM: dts: ls1021a: add wakeup device ftm0 node for ls1021a Dongsheng Wang
2015-08-13 16:23 ` [PATCH v3 1/2] Documentation: DT: FTM: add FTM0 be used as alarm timer Stuart Yoder
2015-08-14 3:24 ` Wang Dongsheng [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=SN1PR0301MB16165DCFD09CEB424A91F79F9D7C0@SN1PR0301MB1616.namprd03.prod.outlook.com \
--to=dongsheng.wang@freescale.com \
--cc=alison.wang@freescale.com \
--cc=b08248@gmail.com \
--cc=devicetree@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=shawn.guo@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).