Re: [PATCH 4/5] pwm: tegra: Add DT node compatible for Tegra186

From: Thierry Reding
Date: Wed Jun 22 2016 - 08:47:46 EST


On Wed, Jun 22, 2016 at 05:17:22PM +0530, Laxman Dewangan wrote:
> Tegra186 has 8 different PWM controller and each controller has only
> one output. Earlier generation SoCs have the 4 PWM output per controller.
>
> Add DT node compatible for Tegra186.
>
> Signed-off-by: Laxman Dewangan <ldewangan@xxxxxxxxxx>
> ---
> Documentation/devicetree/bindings/pwm/nvidia,tegra20-pwm.txt | 10 ++++++----
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/pwm/nvidia,tegra20-pwm.txt b/Documentation/devicetree/bindings/pwm/nvidia,tegra20-pwm.txt
> index c52f03b..2851b2d 100644
> --- a/Documentation/devicetree/bindings/pwm/nvidia,tegra20-pwm.txt
> +++ b/Documentation/devicetree/bindings/pwm/nvidia,tegra20-pwm.txt
> @@ -1,10 +1,12 @@
> Tegra SoC PWFM controller
>
> Required properties:
> -- compatible: For Tegra20, must contain "nvidia,tegra20-pwm". For Tegra30,
> - must contain "nvidia,tegra30-pwm". Otherwise, must contain
> - "nvidia,<chip>-pwm", plus one of the above, where <chip> is tegra114,
> - tegra124, tegra132, or tegra210.
> +- compatible: For Tegra20, must contain "nvidia,tegra20-pwm".
> + For Tegra30, must contain "nvidia,tegra30-pwm".
> + For Tegra114, Tegra124, Tegra132, Tegra210 must contain
> + "nvidia,<chip>-pwm", plus one of the above, where <chip> is
> + tegra114, tegra124, tegra132, or tegra210.
> + For Tegra186, must contain "nvidia,tegra186-pwm".

Rob, I recall discussing this with you a couple of weeks ago, but fail
to remember the outcome and can't find a link to the discussion either.
Wasn't there a new standard way of documenting this kind of compatible
string list?

Or did you say it didn't matter much until we moved to a YAML-based
description?

Thierry

Attachment: signature.asc
Description: PGP signature