[RESEND PATCH 4/8] layerscape/ftm: Add compatible string for FTM0 be used as alarm timer.

[mark.rutland@arm.com (Mark Rutland)]

On Tue, Apr 21, 2015 at 12:01:06PM +0100, Dongsheng.Wang at freescale.com wrote:
> 
> 
> > -----Original Message-----
> > From: Mark Rutland [mailto:mark.rutland at arm.com]
> > Sent: Tuesday, April 21, 2015 6:42 PM
> > To: Wang Dongsheng-B40534
> > Cc: Sharma Bhupesh-B45370; arnd at arndb.de; linux-arm-kernel at lists.infradead.org;
> > Marc Zyngier; bhupesh.linux at gmail.com; Catalin Marinas; Yoder Stuart-B08248;
> > olof at lixom.net; Will Deacon
> > Subject: Re: [RESEND PATCH 4/8] layerscape/ftm: Add compatible string for FTM0
> > be used as alarm timer.
> > 
> > > > > +	ftm0: ftm0 at 2800000 {
> > > > > +		compatible = "fsl,ftm-alarm";
> > > > > +		reg = <0x0 0x2800000 0x0 0x10000>;
> > > > > +		interrupts = <GIC_SPI 76 IRQ_TYPE_LEVEL_HIGH>;
> > > > > +	};
> > > >
> > > > Missing properties?
> > > >
> > >
> > > The FTM0 alarm timer only need "compatible", "reg" and "interrupts" properties.
> > > I miss something about the properties?
> > 
> > As far as I could tell from the original patch, clocks were also listed as
> > required properties regardless.
> > 
> 
> Yes, But only one clock can be used for FTM0 as alarm timer. So this node not need to
> "clock" properties. :)

If the clock property is not necessary in some cases, please add those
caveats in the clock property description.

I'm not sure I follow why you don't need a clock in this case. Is there
a clock internal to the unit? Or is it simply that there is only one
possible external clock? For the latter it should still be described
explicitly.

Mark.