Re: [PATCH 2/4] ARM: qcom: add description of KPSS WDT for IPQ8064

2014-10-08 Thread Josh Cartwright
On Tue, Oct 07, 2014 at 04:07:43PM -0700, Stephen Boyd wrote: > On 10/07/2014 03:10 PM, Josh Cartwright wrote: > >On Thu, Oct 02, 2014 at 12:08:38PM -0700, Stephen Boyd wrote: [..] > >>I'm thinking: > >> > >> timer@200a000 { > >> compatible = "qcom,kpss-timer

Re: [PATCH 2/4] ARM: qcom: add description of KPSS WDT for IPQ8064

2014-10-07 Thread Stephen Boyd
On 10/07/2014 03:10 PM, Josh Cartwright wrote: On Thu, Oct 02, 2014 at 12:08:38PM -0700, Stephen Boyd wrote: [..] On 10/01/14 11:15, Josh Cartwright wrote: Something like this perhaps: timer@200a000 { compatible = "qcom,kpss-timer", "qcom,msm-timer"; int

Re: [PATCH 2/4] ARM: qcom: add description of KPSS WDT for IPQ8064

2014-10-07 Thread Josh Cartwright
On Thu, Oct 02, 2014 at 12:08:38PM -0700, Stephen Boyd wrote: [..] > On 10/01/14 11:15, Josh Cartwright wrote: > > The percpu-ness of the two WDTs makes configuration even more > > interesting, as it's possible you'd want to independently configure > > timeouts for CPU0_WDT0 and CPU1_WDT0, supporti

Re: [PATCH 2/4] ARM: qcom: add description of KPSS WDT for IPQ8064

2014-10-02 Thread Stephen Boyd
On 10/01/14 11:15, Josh Cartwright wrote: > Yeah, the description of this thing is a bit awkward. :-/ I tried to make the binding future proof. > > I'm not sure how I'd feel about just just adding "qcom,kpss-wdt" to the > timer node compatible. I'm wondering if the WDT(s) should be a > subnode(s

Re: [PATCH 2/4] ARM: qcom: add description of KPSS WDT for IPQ8064

2014-10-01 Thread Josh Cartwright
Hey Stephen- Thanks for taking a look. On Wed, Oct 01, 2014 at 10:28:55AM -0700, Stephen Boyd wrote: > On 10/01, Josh Cartwright wrote: > > @@ -96,6 +104,13 @@ > > cpu-offset = <0x8>; > > }; > > > > + watchdog@208a038 { > > + compati

Re: [PATCH 2/4] ARM: qcom: add description of KPSS WDT for IPQ8064

2014-10-01 Thread Stephen Boyd
On 10/01, Josh Cartwright wrote: > @@ -96,6 +104,13 @@ > cpu-offset = <0x8>; > }; > > + watchdog@208a038 { > + compatible = "qcom,kpss-wdt-ipq8064"; > + reg = <0x0208a038 0x40>; Not being aligned to 4k or

[PATCH 2/4] ARM: qcom: add description of KPSS WDT for IPQ8064

2014-10-01 Thread Josh Cartwright
Describe the Krait Processor Sub-system (KPSS) Watchdog timer in the IPQ8064 device tree. Also, add a fixed-clock description of SLEEP_CLK, which will do for now. Signed-off-by: Josh Cartwright --- arch/arm/boot/dts/qcom-ipq8064.dtsi | 15 +++ 1 file changed, 15 insertions(+) diff