| Message ID | 1363124563-24784-1-git-send-email-achew@nvidia.com |
|---|---|
| State | Superseded, archived |
| Headers | show |
On 03/12/2013 03:42 PM, Andrew Chew wrote: > This patch adds a device tree node for the four PWM controllers present > on Tegra114. > diff --git a/arch/arm/boot/dts/tegra114.dtsi b/arch/arm/boot/dts/tegra114.dtsi > + pwm: pwm { > + compatible = "nvidia,tegra114-pwm", "nvidia,tegra20-pwm"; > + reg = <0x7000a000 0x40>; I notice that the size is 0x100 in previous chips. Which is more correct? > + #pwm-cells = <2>; > + clocks = <&tegra_car 17>; > + status = "disabled"; Previous chips don't disable this module by default. Perhaps they should? > + }; -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
> From: Stephen Warren [mailto:swarren@wwwdotorg.org] > Sent: Tuesday, March 12, 2013 4:13 PM > To: Andrew Chew > Cc: Hiroshi Doyu; linux-tegra@vger.kernel.org > Subject: Re: [PATCH 1/1 v3] ARM: tegra: add PWM nodes to Tegra114 DT > > On 03/12/2013 03:42 PM, Andrew Chew wrote: > > This patch adds a device tree node for the four PWM controllers > > present on Tegra114. > > > diff --git a/arch/arm/boot/dts/tegra114.dtsi > > b/arch/arm/boot/dts/tegra114.dtsi > > > + pwm: pwm { > > + compatible = "nvidia,tegra114-pwm", "nvidia,tegra20-pwm"; > > + reg = <0x7000a000 0x40>; > > I notice that the size is 0x100 in previous chips. Which is more correct? Looking at the TRM, it seems to me that 0x40 is correct. There are four PWM controllers, as far as I know. Each PWM controller has exactly one 4-byte register that controls it. They are located at 0x00, 0x10, 0x20, and 0x30. Since the stride is 0x10, I decided to use a total size of 0x40 (rather than something strange like 0x34, which would have been sufficient). I don't know why 0x100 was used in previous chips. > > + #pwm-cells = <2>; > > + clocks = <&tegra_car 17>; > > + status = "disabled"; > > Previous chips don't disable this module by default. Perhaps they should? I thought this was the appropriate thing to do. -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 03/12/2013 05:17 PM, Andrew Chew wrote: >> From: Stephen Warren [mailto:swarren@wwwdotorg.org] >> Sent: Tuesday, March 12, 2013 4:13 PM >> To: Andrew Chew >> Cc: Hiroshi Doyu; linux-tegra@vger.kernel.org >> Subject: Re: [PATCH 1/1 v3] ARM: tegra: add PWM nodes to Tegra114 DT >> >> On 03/12/2013 03:42 PM, Andrew Chew wrote: >>> This patch adds a device tree node for the four PWM controllers >>> present on Tegra114. >> >>> diff --git a/arch/arm/boot/dts/tegra114.dtsi >>> b/arch/arm/boot/dts/tegra114.dtsi >> >>> + pwm: pwm { >>> + compatible = "nvidia,tegra114-pwm", "nvidia,tegra20-pwm"; >>> + reg = <0x7000a000 0x40>; >> >> I notice that the size is 0x100 in previous chips. Which is more correct? > > Looking at the TRM, it seems to me that 0x40 is correct. There are four > PWM controllers, as far as I know. Each PWM controller has exactly one > 4-byte register that controls it. They are located at 0x00, 0x10, 0x20, and > 0x30. Since the stride is 0x10, I decided to use a total size of 0x40 (rather > than something strange like 0x34, which would have been sufficient). > > I don't know why 0x100 was used in previous chips. I guess it's because the TRM documents the address map allocation as being 0x100. Due to that, I guess we should continue to use the 0x100 value. Both are smaller than a page anyway, so even though registers aren't present up to 0x100, we won't waste any virtual memory space. >>> + #pwm-cells = <2>; >>> + clocks = <&tegra_car 17>; >>> + status = "disabled"; >> >> Previous chips don't disable this module by default. Perhaps they should? > > I thought this was the appropriate thing to do. That's probably true. Can you please also submit a patch to add the status property to the previous chips, for consistency. Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/arch/arm/boot/dts/tegra114.dtsi b/arch/arm/boot/dts/tegra114.dtsi index 1dfaf28..71783be 100644 --- a/arch/arm/boot/dts/tegra114.dtsi +++ b/arch/arm/boot/dts/tegra114.dtsi @@ -92,6 +92,14 @@ status = "disabled"; }; + pwm: pwm { + compatible = "nvidia,tegra114-pwm", "nvidia,tegra20-pwm"; + reg = <0x7000a000 0x40>; + #pwm-cells = <2>; + clocks = <&tegra_car 17>; + status = "disabled"; + }; + rtc { compatible = "nvidia,tegra114-rtc", "nvidia,tegra20-rtc"; reg = <0x7000e000 0x100>;
This patch adds a device tree node for the four PWM controllers present on Tegra114. Signed-off-by: Andrew Chew <achew@nvidia.com> --- Changed from specifying four PWM nodes to specifying a single node for all four PWM comtrollers. The first member of the cells tuple is used to select which specific PWM controller. arch/arm/boot/dts/tegra114.dtsi | 8 ++++++++ 1 file changed, 8 insertions(+)