Re: [PATCH v1] ARM: tegra: add nvidia,wdt-timer-id optionalproperty

From: Mark Rutland
Date: Mon Feb 03 2014 - 12:11:09 EST

On Fri, Jan 31, 2014 at 09:46:51PM +0000, Andrew Chew wrote:
> This optional property can be used to specify which timers are to be used
> for hardware watchdog timeouts (via a tegra wdt driver).

Is there any reason that a particular timer should be used?

This shouldn't even mention the driver, as the binding should describe
the HW, not how it's used by Linux at the moment.

> Signed-off-by: Andrew Chew <achew@xxxxxxxxxx>
> ---
> Documentation/devicetree/bindings/timer/nvidia,tegra30-timer.txt | 8 ++++++++
> 1 file changed, 8 insertions(+)
> diff --git a/Documentation/devicetree/bindings/timer/nvidia,tegra30-timer.txt b/Documentation/devicetree/bindings/timer/nvidia,tegra30-timer.txt
> index b5082a1..e87fa70 100644
> --- a/Documentation/devicetree/bindings/timer/nvidia,tegra30-timer.txt
> +++ b/Documentation/devicetree/bindings/timer/nvidia,tegra30-timer.txt
> @@ -13,6 +13,13 @@ Required properties:
> - clocks : Must contain one entry, for the module clock.
> See ../clocks/clock-bindings.txt for details.
> +Optional properties:
> +
> +- nvidia,wdt-timer-id: A list of timer IDs to be used for watchdogs.
> + Watchdog 0 will be assigned to the first timer listed, watchdog 1 will
> + be assigned to the second timer listed, etc. up to the number of watchdogs
> + available.

This sounds like a description of what software should do. Is there any
reason this order is important?

Also, it feels odd for the proerty name to be singular given it's a

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at