diff mbox

[V4] ARM64: dts: hi6220-hikey: Add clock binding for the pmic mfd

Message ID 1493066439-6889-1-git-send-email-daniel.lezcano@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Daniel Lezcano April 24, 2017, 8:40 p.m. UTC
The hi655x PMIC provides the regulators but also a clock. The latter is missing
in the definition and in the DT, thus it is no possible to enable the WiFi which
depends on this clock.

The hi655x's clock has been added and the hi655x multifunction driver has
updated with a clock-cell.

This patch adds the clock-cells for the PMIC in the DT and updates the
documentation.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Acked-by: Rob Herring <robh@kernel.org>
Acked-by: Lee Jones <lee.jones@linaro.org>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Stephen Boyd <sboyd@codeaurora.org>
Cc: Michael Turquette <mturquette@baylibre.com>
---
 Changelog:
	v4:
	   - Added Acked-by's
	   - Updated the commit message with a better description
---
 Documentation/devicetree/bindings/mfd/hisilicon,hi655x.txt | 6 ++++++
 arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts             | 1 +
 2 files changed, 7 insertions(+)

Comments

Daniel Lezcano May 4, 2017, 10:07 p.m. UTC | #1
On Mon, Apr 24, 2017 at 10:40:22PM +0200, Daniel Lezcano wrote:
> The hi655x PMIC provides the regulators but also a clock. The latter is missing
> in the definition and in the DT, thus it is no possible to enable the WiFi which
> depends on this clock.
> 
> The hi655x's clock has been added and the hi655x multifunction driver has
> updated with a clock-cell.
> 
> This patch adds the clock-cells for the PMIC in the DT and updates the
> documentation.
> 
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> Acked-by: Rob Herring <robh@kernel.org>
> Acked-by: Lee Jones <lee.jones@linaro.org>
> Cc: Arnd Bergmann <arnd@arndb.de>
> Cc: Stephen Boyd <sboyd@codeaurora.org>
> Cc: Michael Turquette <mturquette@baylibre.com>
> ---
>  Changelog:
> 	v4:
> 	   - Added Acked-by's
> 	   - Updated the commit message with a better description
> ---

Hi Xu Wei,

a gentle reminder for this patch. Is it ok for you?

Thanks.

  -- Daniel
Lee Jones May 5, 2017, 7:48 a.m. UTC | #2
On Fri, 05 May 2017, Daniel Lezcano wrote:

> On Mon, Apr 24, 2017 at 10:40:22PM +0200, Daniel Lezcano wrote:
> > The hi655x PMIC provides the regulators but also a clock. The latter is missing
> > in the definition and in the DT, thus it is no possible to enable the WiFi which
> > depends on this clock.
> > 
> > The hi655x's clock has been added and the hi655x multifunction driver has
> > updated with a clock-cell.
> > 
> > This patch adds the clock-cells for the PMIC in the DT and updates the
> > documentation.
> > 
> > Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> > Acked-by: Rob Herring <robh@kernel.org>
> > Acked-by: Lee Jones <lee.jones@linaro.org>
> > Cc: Arnd Bergmann <arnd@arndb.de>
> > Cc: Stephen Boyd <sboyd@codeaurora.org>
> > Cc: Michael Turquette <mturquette@baylibre.com>
> > ---
> >  Changelog:
> > 	v4:
> > 	   - Added Acked-by's
> > 	   - Updated the commit message with a better description
> > ---
> 
> Hi Xu Wei,
> 
> a gentle reminder for this patch. Is it ok for you?

a) Pings aren't usually received well.  If you wish to remind
maintainers of a patch's existence a [RESEND] is usually the most
accepted method.

b) There is usually little point (at least it's significantly reduced)
in sending pings/resends whilst the merge-window is open.  Maintainers
usually take this time to take a little breather (rest) from their
duties.
Daniel Lezcano May 5, 2017, 8:32 a.m. UTC | #3
On Fri, May 05, 2017 at 08:48:21AM +0100, Lee Jones wrote:
> On Fri, 05 May 2017, Daniel Lezcano wrote:
> 
> > On Mon, Apr 24, 2017 at 10:40:22PM +0200, Daniel Lezcano wrote:
> > > The hi655x PMIC provides the regulators but also a clock. The latter is missing
> > > in the definition and in the DT, thus it is no possible to enable the WiFi which
> > > depends on this clock.
> > > 
> > > The hi655x's clock has been added and the hi655x multifunction driver has
> > > updated with a clock-cell.
> > > 
> > > This patch adds the clock-cells for the PMIC in the DT and updates the
> > > documentation.
> > > 
> > > Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> > > Acked-by: Rob Herring <robh@kernel.org>
> > > Acked-by: Lee Jones <lee.jones@linaro.org>
> > > Cc: Arnd Bergmann <arnd@arndb.de>
> > > Cc: Stephen Boyd <sboyd@codeaurora.org>
> > > Cc: Michael Turquette <mturquette@baylibre.com>
> > > ---
> > >  Changelog:
> > > 	v4:
> > > 	   - Added Acked-by's
> > > 	   - Updated the commit message with a better description
> > > ---
> > 
> > Hi Xu Wei,
> > 
> > a gentle reminder for this patch. Is it ok for you?
> 
> a) Pings aren't usually received well.  If you wish to remind
> maintainers of a patch's existence a [RESEND] is usually the most
> accepted method.
> b) There is usually little point (at least it's significantly reduced)
> in sending pings/resends whilst the merge-window is open.  Maintainers
> usually take this time to take a little breather (rest) from their
> duties.

This patch is the last one of a series which was already merged. As the
series has been resend and split with a different submission path, I just
wanted to make sure there were no confusion.

The sentence "a gentle reminder" is clumsy.
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/mfd/hisilicon,hi655x.txt b/Documentation/devicetree/bindings/mfd/hisilicon,hi655x.txt
index 0548569..9630ac0 100644
--- a/Documentation/devicetree/bindings/mfd/hisilicon,hi655x.txt
+++ b/Documentation/devicetree/bindings/mfd/hisilicon,hi655x.txt
@@ -16,6 +16,11 @@  Required properties:
 - reg:                  Base address of PMIC on Hi6220 SoC.
 - interrupt-controller: Hi655x has internal IRQs (has own IRQ domain).
 - pmic-gpios:           The GPIO used by PMIC IRQ.
+- #clock-cells:		From common clock binding; shall be set to 0
+
+Optional properties:
+- clock-output-names: From common clock binding to override the
+  default output clock name
 
 Example:
 	pmic: pmic@f8000000 {
@@ -24,4 +29,5 @@  Example:
 		interrupt-controller;
 		#interrupt-cells = <2>;
 		pmic-gpios = <&gpio1 2 GPIO_ACTIVE_HIGH>;
+		#clock-cells = <0>;
 	}
diff --git a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
index dba3c13..e0496f7 100644
--- a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
+++ b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
@@ -325,6 +325,7 @@ 
 	pmic: pmic@f8000000 {
 		compatible = "hisilicon,hi655x-pmic";
 		reg = <0x0 0xf8000000 0x0 0x1000>;
+		#clock-cells = <0>;
 		interrupt-controller;
 		#interrupt-cells = <2>;
 		pmic-gpios = <&gpio1 2 GPIO_ACTIVE_HIGH>;