From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932350AbbHNDkP (ORCPT ); Thu, 13 Aug 2015 23:40:15 -0400 Received: from mail-bl2on0119.outbound.protection.outlook.com ([65.55.169.119]:28384 "EHLO na01-bl2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932081AbbHNDkN (ORCPT ); Thu, 13 Aug 2015 23:40:13 -0400 From: Wang Dongsheng To: Stuart Yoder CC: "shawn.guo@linaro.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Huan Wang Subject: RE: [PATCH v3 1/2] Documentation: DT: FTM: add FTM0 be used as alarm timer Thread-Topic: [PATCH v3 1/2] Documentation: DT: FTM: add FTM0 be used as alarm timer Thread-Index: AQHQ1LRDTidhxTDJkE+gLo55/5YyCJ4KH0iAgAC4k4A= Date: Fri, 14 Aug 2015 03:24:35 +0000 Message-ID: References: <1439352061-14656-1-git-send-email-dongsheng.wang@freescale.com> In-Reply-To: Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=Dongsheng.Wang@freescale.com; x-originating-ip: [123.151.195.52] x-microsoft-exchange-diagnostics: 1;DM2PR0301MB0877;5:QG3V/XiauOLB4CAw4478XTEyVO8hWBT8xr4UG0J5vJYPALxCFtTDRTDl4TzsghKPoYcUrewlf9QzP23F+DCVfu5ipOya3kmWU0MJhtL6rYDUS+DV9gVmBP/LmsBkTeolS7zMHTk3r/1tVzF9W0tjjw==;24:934uSfo+c02sRgFdhg+OcXiQuNJiXJM4vPfuJVb5P1V0goe+4i/UwYKjZgoh7o2V7nXrwG64G875kQGUGzOQbAcuEwHEWCh6YG90DRSI4MY=;20:H6PQKUx0ndD0BDwaSrzKJpp8GcEJVO9KJPQ4wHadCsIqToj7o3Wv1EUblH7p/RuG8aba0rPf9pYlfoKFxb/WGA== x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:DM2PR0301MB0877; x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:; x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(601004)(5005006)(3002001);SRVR:DM2PR0301MB0877;BCL:0;PCL:0;RULEID:;SRVR:DM2PR0301MB0877; x-forefront-prvs: 066898046A x-forefront-antispam-report: SFV:NSPM;SFS:(10019020)(6009001)(164054003)(13464003)(24454002)(377454003)(199003)(189002)(76576001)(10400500002)(5001830100001)(110136002)(97736004)(5001960100002)(4001540100001)(81156007)(5001860100001)(46102003)(105586002)(106116001)(99286002)(106356001)(5003600100002)(77156002)(62966003)(40100003)(122556002)(107886002)(50986999)(2656002)(68736005)(77096005)(102836002)(19580395003)(92566002)(1411001)(2900100001)(2950100001)(54356999)(64706001)(19580405001)(76176999)(74316001)(86362001)(575784001)(66066001)(189998001)(5002640100001)(87936001)(33656002)(101416001)(4001430100001);DIR:OUT;SFP:1102;SCL:1;SRVR:DM2PR0301MB0877;H:SN1PR0301MB1616.namprd03.prod.outlook.com;FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 X-OriginatorOrg: freescale.com X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Aug 2015 03:24:35.6968 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 710a03f5-10f6-4d38-9ff4-a80b81da590d X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM2PR0301MB0877 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id t7E3emmh029372 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 > wrote: > > From: Wang Dongsheng > > > > 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 > > --- > > 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 = ; > > + big-endian; > > + status = "disabled"; > > +}; > > Drop the number 0 from the names in your example. > > Thanks, > Stuart {.n++%ݶw{.n+{G{ayʇڙ,jfhz_(階ݢj"mG?&~iOzv^m ?I