Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1402748 > unrolled thread

[PATCH INTERNAL v2 0/3] pwm support for Broadcom iProc SoC's

Started byYendapally Reddy Dhananjaya Reddy <yendapally.reddy@broadcom.com>
First post2016-05-18 09:00 +0200
Last post2016-05-18 09:00 +0200
Articles 5 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH INTERNAL v2 0/3] pwm support for Broadcom iProc SoC's Yendapally Reddy Dhananjaya Reddy <yendapally.reddy@broadcom.com> - 2016-05-18 09:00 +0200
    [PATCH INTERNAL v2 1/3] Documentation: dt: Add Broadcom iproc pwm controller binding Yendapally Reddy Dhananjaya Reddy <yendapally.reddy@broadcom.com> - 2016-05-18 09:00 +0200
      Re: [PATCH INTERNAL v2 1/3] Documentation: dt: Add Broadcom iproc  pwm controller binding Rob Herring <robh@kernel.org> - 2016-05-19 00:20 +0200
        Re: [PATCH INTERNAL v2 1/3] Documentation: dt: Add Broadcom iproc pwm  controller binding Scott Branden <scott.branden@broadcom.com> - 2016-05-19 01:10 +0200
    [PATCH INTERNAL v2 3/3] ARM: dts: NSP: Add PWM Support to DT Yendapally Reddy Dhananjaya Reddy <yendapally.reddy@broadcom.com> - 2016-05-18 09:00 +0200

#1402748 — [PATCH INTERNAL v2 0/3] pwm support for Broadcom iProc SoC's

FromYendapally Reddy Dhananjaya Reddy <yendapally.reddy@broadcom.com>
Date2016-05-18 09:00 +0200
Subject[PATCH INTERNAL v2 0/3] pwm support for Broadcom iProc SoC's
Message-ID<rA3Cx-6WR-3@gated-at.bofh.it>
This patchset contains the pwm support for the Broadcom's iProc SoC's.

The first patch provides the documentation details and the
second patch contains the controller support details. The third patch
contains the enable method for Northstar Plus SoC.

This patch series has been tested on NSP bcm958625HR board.
The pwm is not enabled for this board as the pins are open and not used.

This patch series is based on v4.6.0-rc1 and is available from github
repo: https://github.com/Broadcom/cygnus-linux.git
branch: iproc-pwm-v2

Changes since v1

Addressed the review comments from Thierry
 1. Use separate driver for iProc SoC's.

Yendapally Reddy Dhananjaya Reddy (3):
  Documentation: dt: Add Broadcom iproc pwm controller binding
  pwm: iproc: Add support for Broadcom iproc pwm controller
  ARM: dts: NSP: Add PWM Support to DT

 .../devicetree/bindings/pwm/brcm,iproc-pwm.txt     |  21 ++
 arch/arm/boot/dts/bcm-nsp.dtsi                     |   8 +
 drivers/pwm/Kconfig                                |  10 +
 drivers/pwm/Makefile                               |   1 +
 drivers/pwm/pwm-bcm-iproc.c                        | 320 +++++++++++++++++++++
 5 files changed, 360 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/pwm/brcm,iproc-pwm.txt
 create mode 100644 drivers/pwm/pwm-bcm-iproc.c

-- 
2.1.0

[toc] | [next] | [standalone]


#1402749 — [PATCH INTERNAL v2 1/3] Documentation: dt: Add Broadcom iproc pwm controller binding

FromYendapally Reddy Dhananjaya Reddy <yendapally.reddy@broadcom.com>
Date2016-05-18 09:00 +0200
Subject[PATCH INTERNAL v2 1/3] Documentation: dt: Add Broadcom iproc pwm controller binding
Message-ID<rA3Cx-6WR-15@gated-at.bofh.it>
In reply to#1402748
Add a binding for Broadcom iproc pwm controller

Signed-off-by: Yendapally Reddy Dhananjaya Reddy <yendapally.reddy@broadcom.com>
---
 .../devicetree/bindings/pwm/brcm,iproc-pwm.txt      | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/pwm/brcm,iproc-pwm.txt

diff --git a/Documentation/devicetree/bindings/pwm/brcm,iproc-pwm.txt b/Documentation/devicetree/bindings/pwm/brcm,iproc-pwm.txt
new file mode 100644
index 0000000..21f75bb
--- /dev/null
+++ b/Documentation/devicetree/bindings/pwm/brcm,iproc-pwm.txt
@@ -0,0 +1,21 @@
+Broadcom iProc PWM controller device tree bindings
+
+This controller has 4 channels.
+
+Required Properties :
+- compatible: must be "brcm,iproc-pwm"
+- reg: physical base address and length of the controller's registers
+- clocks: phandle + clock specifier pair for the external clock
+- #pwm-cells: Should be 3. See pwm.txt in this directory for a
+  description of the cells format.
+
+Refer to clocks/clock-bindings.txt for generic clock consumer properties.
+
+Example:
+
+pwm: pwm@18031000 {
+	compatible = "brcm,iproc-pwm";
+	reg = <0x18031000 0x28>;
+	clocks = <&osc>;
+	#pwm-cells = <3>;
+};
-- 
2.1.0

[toc] | [prev] | [next] | [standalone]


#1403322 — Re: [PATCH INTERNAL v2 1/3] Documentation: dt: Add Broadcom iproc pwm controller binding

FromRob Herring <robh@kernel.org>
Date2016-05-19 00:20 +0200
SubjectRe: [PATCH INTERNAL v2 1/3] Documentation: dt: Add Broadcom iproc pwm controller binding
Message-ID<rAhYR-81A-11@gated-at.bofh.it>
In reply to#1402749
Am I supposed to ignore since this is "internal"?

On Wed, May 18, 2016 at 02:53:59AM -0400, Yendapally Reddy Dhananjaya Reddy wrote:
> Add a binding for Broadcom iproc pwm controller
> 
> Signed-off-by: Yendapally Reddy Dhananjaya Reddy <yendapally.reddy@broadcom.com>
> ---
>  .../devicetree/bindings/pwm/brcm,iproc-pwm.txt      | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/pwm/brcm,iproc-pwm.txt

Acked-by: Rob Herring <robh@kernel.org>

[toc] | [prev] | [next] | [standalone]


#1403340 — Re: [PATCH INTERNAL v2 1/3] Documentation: dt: Add Broadcom iproc pwm controller binding

FromScott Branden <scott.branden@broadcom.com>
Date2016-05-19 01:10 +0200
SubjectRe: [PATCH INTERNAL v2 1/3] Documentation: dt: Add Broadcom iproc pwm controller binding
Message-ID<rAiLg-7m-23@gated-at.bofh.it>
In reply to#1403322
Hi Rob,

On 16-05-18 03:15 PM, Rob Herring wrote:
> Am I supposed to ignore since this is "internal"?
The "INTERNAL" was supposed to be removed after internal review before 
sending out to the public mailing list.

Also, Yendapally's git email config was not setup properly so some are 
tagging the patchset as spam.

Other than that, everything is fine with the PATCH sent out to the open 
source community :)

I'm sure Yendapally will send out an updated version with those items 
corrected (and can add your Acked-by now).

Thanks,
  Scott

>
> On Wed, May 18, 2016 at 02:53:59AM -0400, Yendapally Reddy Dhananjaya Reddy wrote:
>> Add a binding for Broadcom iproc pwm controller
>>
>> Signed-off-by: Yendapally Reddy Dhananjaya Reddy <yendapally.reddy@broadcom.com>
>> ---
>>   .../devicetree/bindings/pwm/brcm,iproc-pwm.txt      | 21 +++++++++++++++++++++
>>   1 file changed, 21 insertions(+)
>>   create mode 100644 Documentation/devicetree/bindings/pwm/brcm,iproc-pwm.txt
>
> Acked-by: Rob Herring <robh@kernel.org>
>

[toc] | [prev] | [next] | [standalone]


#1402750 — [PATCH INTERNAL v2 3/3] ARM: dts: NSP: Add PWM Support to DT

FromYendapally Reddy Dhananjaya Reddy <yendapally.reddy@broadcom.com>
Date2016-05-18 09:00 +0200
Subject[PATCH INTERNAL v2 3/3] ARM: dts: NSP: Add PWM Support to DT
Message-ID<rA3Cy-6WR-19@gated-at.bofh.it>
In reply to#1402748
Add PWM support to the device tree for the Broadcom Northstar Plus SoC.

Signed-off-by: Yendapally Reddy Dhananjaya Reddy <yendapally.reddy@broadcom.com>
---
 arch/arm/boot/dts/bcm-nsp.dtsi | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/arch/arm/boot/dts/bcm-nsp.dtsi b/arch/arm/boot/dts/bcm-nsp.dtsi
index def9e78..f9c8341 100644
--- a/arch/arm/boot/dts/bcm-nsp.dtsi
+++ b/arch/arm/boot/dts/bcm-nsp.dtsi
@@ -206,6 +206,14 @@
 			brcm,nand-has-wp;
 		};
 
+		pwm: pwm@31000 {
+			compatible = "brcm,iproc-pwm";
+			reg = <0x31000 0x28>;
+			clocks = <&osc>;
+			#pwm-cells = <3>;
+			status = "disabled";
+		};
+
 		ccbtimer0: timer@34000 {
 			compatible = "arm,sp804";
 			reg = <0x34000 0x1000>;
-- 
2.1.0

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web