All of lore.kernel.org
 help / color / mirror / Atom feed
* [RESEND PATCH v2 0/2] add multiple clock handling for dwc2 driver
@ 2017-02-10  3:06 ` Frank Wang
  0 siblings, 0 replies; 14+ messages in thread
From: Frank Wang @ 2017-02-10  3:06 UTC (permalink / raw)
  To: johnyoun, gregkh, robh+dt, mark.rutland, heiko
  Cc: linux-kernel, linux-usb, devicetree, linux-rockchip, huangtao,
	kever.yang, william.wu, daniel.meng, frank.wang

Resend this series to involve device tree maintainer and add 'Reviewed-by' tag for driver.

The Current default dwc2 just handle one clock named otg, however, it may have
two or more clock need to manage for some new SoCs(such as RK3328), so this
adds change clk to clk's array of dwc2_hsotg to handle more clocks operation.

Changes in v2:
 - amend dwc2 clocks property in DT.
 - change DWC2_MAX_CLKS to 5 and amend commit message.

Frank Wang (2):
  Documentation: dt: dwc2: amend clocks property
  usb: dwc2: add multiple clocks handling

 Documentation/devicetree/bindings/usb/dwc2.txt | 13 +++++++--
 drivers/usb/dwc2/core.h                        |  5 +++-
 drivers/usb/dwc2/platform.c                    | 39 +++++++++++++++++---------
 3 files changed, 40 insertions(+), 17 deletions(-)

-- 
1.9.1

^ permalink raw reply	[flat|nested] 14+ messages in thread

* [RESEND PATCH v2 0/2] add multiple clock handling for dwc2 driver
@ 2017-02-10  3:06 ` Frank Wang
  0 siblings, 0 replies; 14+ messages in thread
From: Frank Wang @ 2017-02-10  3:06 UTC (permalink / raw)
  To: johnyoun-HKixBCOQz3hWk0Htik3J/w,
	gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, mark.rutland-5wv7dgnIgG8,
	heiko-4mtYJXux2i+zQB+pC5nmwQ
  Cc: huangtao-TNX95d0MmH7DzftRWevZcw,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	frank.wang-TNX95d0MmH7DzftRWevZcw,
	linux-usb-u79uwXL29TY76Z2rM5mHXA,
	kever.yang-TNX95d0MmH7DzftRWevZcw,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	william.wu-TNX95d0MmH7DzftRWevZcw,
	daniel.meng-TNX95d0MmH7DzftRWevZcw

Resend this series to involve device tree maintainer and add 'Reviewed-by' tag for driver.

The Current default dwc2 just handle one clock named otg, however, it may have
two or more clock need to manage for some new SoCs(such as RK3328), so this
adds change clk to clk's array of dwc2_hsotg to handle more clocks operation.

Changes in v2:
 - amend dwc2 clocks property in DT.
 - change DWC2_MAX_CLKS to 5 and amend commit message.

Frank Wang (2):
  Documentation: dt: dwc2: amend clocks property
  usb: dwc2: add multiple clocks handling

 Documentation/devicetree/bindings/usb/dwc2.txt | 13 +++++++--
 drivers/usb/dwc2/core.h                        |  5 +++-
 drivers/usb/dwc2/platform.c                    | 39 +++++++++++++++++---------
 3 files changed, 40 insertions(+), 17 deletions(-)

-- 
1.9.1

^ permalink raw reply	[flat|nested] 14+ messages in thread

* [RESEND PATCH v2 1/2] Documentation: dt: dwc2: amend clocks property
@ 2017-02-10  3:06   ` Frank Wang
  0 siblings, 0 replies; 14+ messages in thread
From: Frank Wang @ 2017-02-10  3:06 UTC (permalink / raw)
  To: johnyoun, gregkh, robh+dt, mark.rutland, heiko
  Cc: linux-kernel, linux-usb, devicetree, linux-rockchip, huangtao,
	kever.yang, william.wu, daniel.meng, frank.wang

Move 'clocks' to optional properties since not every platform
is going to need to specify it and add pmu/utmi/ulpi/utmifs
optional clocks into 'clock-names' list.

Signed-off-by: Frank Wang <frank.wang@rock-chips.com>
---
 Documentation/devicetree/bindings/usb/dwc2.txt | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/usb/dwc2.txt b/Documentation/devicetree/bindings/usb/dwc2.txt
index 6c7c2bce..4d8b6dd 100644
--- a/Documentation/devicetree/bindings/usb/dwc2.txt
+++ b/Documentation/devicetree/bindings/usb/dwc2.txt
@@ -16,11 +16,18 @@ Required properties:
   - snps,dwc2: A generic DWC2 USB controller with default parameters.
 - reg : Should contain 1 register range (address and length)
 - interrupts : Should contain 1 interrupt
-- clocks: clock provider specifier
-- clock-names: shall be "otg"
-Refer to clk/clock-bindings.txt for generic clock consumer properties
 
 Optional properties:
+- clocks: A list of phandle + clock-specifier pairs for the clocks listed in clock-names.
+- clock-names: Contain the following optional clocks on demand:
+  - "otg":	AHB clock. It is the scan clock for the core's AHB domain.
+  - "pmu":	PMU hclk. Clock to PMU module when hibernation and/or ADP is enabled.
+  - "utmi":	UTMI+ clock. Functionally used only when a UTMI PHY is selected,
+		but always used as the PHY domain clock during DFT Scan mode.
+  - "ulpi":	ULPI PHY clock. Present only when a ULPI PHY is selected; negedge
+		is also used in DDR mode.
+  - "utmifs":	UTMI FS clock. Present only when OTG_FSPHY_INTERFACE != 0.
+Refer to clk/clock-bindings.txt for generic clock consumer properties
 - phys: phy provider specifier
 - phy-names: shall be "usb2-phy"
 Refer to phy/phy-bindings.txt for generic phy consumer properties
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 14+ messages in thread

* [RESEND PATCH v2 1/2] Documentation: dt: dwc2: amend clocks property
@ 2017-02-10  3:06   ` Frank Wang
  0 siblings, 0 replies; 14+ messages in thread
From: Frank Wang @ 2017-02-10  3:06 UTC (permalink / raw)
  To: johnyoun-HKixBCOQz3hWk0Htik3J/w,
	gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, mark.rutland-5wv7dgnIgG8,
	heiko-4mtYJXux2i+zQB+pC5nmwQ
  Cc: huangtao-TNX95d0MmH7DzftRWevZcw,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	frank.wang-TNX95d0MmH7DzftRWevZcw,
	linux-usb-u79uwXL29TY76Z2rM5mHXA,
	kever.yang-TNX95d0MmH7DzftRWevZcw,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	william.wu-TNX95d0MmH7DzftRWevZcw,
	daniel.meng-TNX95d0MmH7DzftRWevZcw

Move 'clocks' to optional properties since not every platform
is going to need to specify it and add pmu/utmi/ulpi/utmifs
optional clocks into 'clock-names' list.

Signed-off-by: Frank Wang <frank.wang-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
---
 Documentation/devicetree/bindings/usb/dwc2.txt | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/usb/dwc2.txt b/Documentation/devicetree/bindings/usb/dwc2.txt
index 6c7c2bce..4d8b6dd 100644
--- a/Documentation/devicetree/bindings/usb/dwc2.txt
+++ b/Documentation/devicetree/bindings/usb/dwc2.txt
@@ -16,11 +16,18 @@ Required properties:
   - snps,dwc2: A generic DWC2 USB controller with default parameters.
 - reg : Should contain 1 register range (address and length)
 - interrupts : Should contain 1 interrupt
-- clocks: clock provider specifier
-- clock-names: shall be "otg"
-Refer to clk/clock-bindings.txt for generic clock consumer properties
 
 Optional properties:
+- clocks: A list of phandle + clock-specifier pairs for the clocks listed in clock-names.
+- clock-names: Contain the following optional clocks on demand:
+  - "otg":	AHB clock. It is the scan clock for the core's AHB domain.
+  - "pmu":	PMU hclk. Clock to PMU module when hibernation and/or ADP is enabled.
+  - "utmi":	UTMI+ clock. Functionally used only when a UTMI PHY is selected,
+		but always used as the PHY domain clock during DFT Scan mode.
+  - "ulpi":	ULPI PHY clock. Present only when a ULPI PHY is selected; negedge
+		is also used in DDR mode.
+  - "utmifs":	UTMI FS clock. Present only when OTG_FSPHY_INTERFACE != 0.
+Refer to clk/clock-bindings.txt for generic clock consumer properties
 - phys: phy provider specifier
 - phy-names: shall be "usb2-phy"
 Refer to phy/phy-bindings.txt for generic phy consumer properties
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 14+ messages in thread

* [RESEND PATCH v2 2/2] usb: dwc2: add multiple clocks handling
@ 2017-02-10  3:06   ` Frank Wang
  0 siblings, 0 replies; 14+ messages in thread
From: Frank Wang @ 2017-02-10  3:06 UTC (permalink / raw)
  To: johnyoun, gregkh, robh+dt, mark.rutland, heiko
  Cc: linux-kernel, linux-usb, devicetree, linux-rockchip, huangtao,
	kever.yang, william.wu, daniel.meng, frank.wang

Since dwc2 may have one or more input clocks need to manage for some
platform, so this adds change clk to clk's array of struct dwc2_hsotg
to handle more clocks operation.

Signed-off-by: Frank Wang <frank.wang@rock-chips.com>
Reviewed-by: Heiko Stuebner <heiko@sntech.de>
---
 drivers/usb/dwc2/core.h     |  5 ++++-
 drivers/usb/dwc2/platform.c | 39 ++++++++++++++++++++++++++-------------
 2 files changed, 30 insertions(+), 14 deletions(-)

diff --git a/drivers/usb/dwc2/core.h b/drivers/usb/dwc2/core.h
index 1a7e830..afde8a1 100644
--- a/drivers/usb/dwc2/core.h
+++ b/drivers/usb/dwc2/core.h
@@ -121,6 +121,9 @@ static inline void dwc2_writel(u32 value, void __iomem *addr)
 /* Maximum number of Endpoints/HostChannels */
 #define MAX_EPS_CHANNELS	16
 
+/* Maximum number of dwc2 input clocks */
+#define DWC2_MAX_CLKS	5
+
 /* dwc2-hsotg declarations */
 static const char * const dwc2_hsotg_supply_names[] = {
 	"vusb_d",               /* digital USB supply, 1.2V */
@@ -913,7 +916,7 @@ struct dwc2_hsotg {
 	spinlock_t lock;
 	void *priv;
 	int     irq;
-	struct clk *clk;
+	struct clk *clks[DWC2_MAX_CLKS];
 	struct reset_control *reset;
 
 	unsigned int queuing_high_bandwidth:1;
diff --git a/drivers/usb/dwc2/platform.c b/drivers/usb/dwc2/platform.c
index 9564bc7..795fc43b 100644
--- a/drivers/usb/dwc2/platform.c
+++ b/drivers/usb/dwc2/platform.c
@@ -123,17 +123,20 @@ static int dwc2_get_dr_mode(struct dwc2_hsotg *hsotg)
 static int __dwc2_lowlevel_hw_enable(struct dwc2_hsotg *hsotg)
 {
 	struct platform_device *pdev = to_platform_device(hsotg->dev);
-	int ret;
+	int clk, ret;
 
 	ret = regulator_bulk_enable(ARRAY_SIZE(hsotg->supplies),
 				    hsotg->supplies);
 	if (ret)
 		return ret;
 
-	if (hsotg->clk) {
-		ret = clk_prepare_enable(hsotg->clk);
-		if (ret)
+	for (clk = 0; clk < DWC2_MAX_CLKS && hsotg->clks[clk]; clk++) {
+		ret = clk_prepare_enable(hsotg->clks[clk]);
+		if (ret) {
+			while (--clk >= 0)
+				clk_disable_unprepare(hsotg->clks[clk]);
 			return ret;
+		}
 	}
 
 	if (hsotg->uphy) {
@@ -168,7 +171,7 @@ int dwc2_lowlevel_hw_enable(struct dwc2_hsotg *hsotg)
 static int __dwc2_lowlevel_hw_disable(struct dwc2_hsotg *hsotg)
 {
 	struct platform_device *pdev = to_platform_device(hsotg->dev);
-	int ret = 0;
+	int clk, ret = 0;
 
 	if (hsotg->uphy) {
 		usb_phy_shutdown(hsotg->uphy);
@@ -182,8 +185,9 @@ static int __dwc2_lowlevel_hw_disable(struct dwc2_hsotg *hsotg)
 	if (ret)
 		return ret;
 
-	if (hsotg->clk)
-		clk_disable_unprepare(hsotg->clk);
+	for (clk = DWC2_MAX_CLKS - 1; clk >= 0; clk--)
+		if (hsotg->clks[clk])
+			clk_disable_unprepare(hsotg->clks[clk]);
 
 	ret = regulator_bulk_disable(ARRAY_SIZE(hsotg->supplies),
 				     hsotg->supplies);
@@ -209,7 +213,7 @@ int dwc2_lowlevel_hw_disable(struct dwc2_hsotg *hsotg)
 
 static int dwc2_lowlevel_hw_init(struct dwc2_hsotg *hsotg)
 {
-	int i, ret;
+	int i, clk, ret;
 
 	hsotg->reset = devm_reset_control_get_optional(hsotg->dev, "dwc2");
 	if (IS_ERR(hsotg->reset)) {
@@ -282,11 +286,20 @@ static int dwc2_lowlevel_hw_init(struct dwc2_hsotg *hsotg)
 			hsotg->phyif = GUSBCFG_PHYIF8;
 	}
 
-	/* Clock */
-	hsotg->clk = devm_clk_get(hsotg->dev, "otg");
-	if (IS_ERR(hsotg->clk)) {
-		hsotg->clk = NULL;
-		dev_dbg(hsotg->dev, "cannot get otg clock\n");
+	/* Clocks */
+	for (clk = 0; clk < DWC2_MAX_CLKS; clk++) {
+		hsotg->clks[clk] = of_clk_get(hsotg->dev->of_node, clk);
+		if (IS_ERR(hsotg->clks[clk])) {
+			ret = PTR_ERR(hsotg->clks[clk]);
+			if (ret == -EPROBE_DEFER) {
+				while (--clk >= 0)
+					clk_put(hsotg->clks[clk]);
+				return ret;
+			}
+
+			hsotg->clks[clk] = NULL;
+			break;
+		}
 	}
 
 	/* Regulators */
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 14+ messages in thread

* [RESEND PATCH v2 2/2] usb: dwc2: add multiple clocks handling
@ 2017-02-10  3:06   ` Frank Wang
  0 siblings, 0 replies; 14+ messages in thread
From: Frank Wang @ 2017-02-10  3:06 UTC (permalink / raw)
  To: johnyoun-HKixBCOQz3hWk0Htik3J/w,
	gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, mark.rutland-5wv7dgnIgG8,
	heiko-4mtYJXux2i+zQB+pC5nmwQ
  Cc: huangtao-TNX95d0MmH7DzftRWevZcw,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	frank.wang-TNX95d0MmH7DzftRWevZcw,
	linux-usb-u79uwXL29TY76Z2rM5mHXA,
	kever.yang-TNX95d0MmH7DzftRWevZcw,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	william.wu-TNX95d0MmH7DzftRWevZcw,
	daniel.meng-TNX95d0MmH7DzftRWevZcw

Since dwc2 may have one or more input clocks need to manage for some
platform, so this adds change clk to clk's array of struct dwc2_hsotg
to handle more clocks operation.

Signed-off-by: Frank Wang <frank.wang-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
Reviewed-by: Heiko Stuebner <heiko-4mtYJXux2i+zQB+pC5nmwQ@public.gmane.org>
---
 drivers/usb/dwc2/core.h     |  5 ++++-
 drivers/usb/dwc2/platform.c | 39 ++++++++++++++++++++++++++-------------
 2 files changed, 30 insertions(+), 14 deletions(-)

diff --git a/drivers/usb/dwc2/core.h b/drivers/usb/dwc2/core.h
index 1a7e830..afde8a1 100644
--- a/drivers/usb/dwc2/core.h
+++ b/drivers/usb/dwc2/core.h
@@ -121,6 +121,9 @@ static inline void dwc2_writel(u32 value, void __iomem *addr)
 /* Maximum number of Endpoints/HostChannels */
 #define MAX_EPS_CHANNELS	16
 
+/* Maximum number of dwc2 input clocks */
+#define DWC2_MAX_CLKS	5
+
 /* dwc2-hsotg declarations */
 static const char * const dwc2_hsotg_supply_names[] = {
 	"vusb_d",               /* digital USB supply, 1.2V */
@@ -913,7 +916,7 @@ struct dwc2_hsotg {
 	spinlock_t lock;
 	void *priv;
 	int     irq;
-	struct clk *clk;
+	struct clk *clks[DWC2_MAX_CLKS];
 	struct reset_control *reset;
 
 	unsigned int queuing_high_bandwidth:1;
diff --git a/drivers/usb/dwc2/platform.c b/drivers/usb/dwc2/platform.c
index 9564bc7..795fc43b 100644
--- a/drivers/usb/dwc2/platform.c
+++ b/drivers/usb/dwc2/platform.c
@@ -123,17 +123,20 @@ static int dwc2_get_dr_mode(struct dwc2_hsotg *hsotg)
 static int __dwc2_lowlevel_hw_enable(struct dwc2_hsotg *hsotg)
 {
 	struct platform_device *pdev = to_platform_device(hsotg->dev);
-	int ret;
+	int clk, ret;
 
 	ret = regulator_bulk_enable(ARRAY_SIZE(hsotg->supplies),
 				    hsotg->supplies);
 	if (ret)
 		return ret;
 
-	if (hsotg->clk) {
-		ret = clk_prepare_enable(hsotg->clk);
-		if (ret)
+	for (clk = 0; clk < DWC2_MAX_CLKS && hsotg->clks[clk]; clk++) {
+		ret = clk_prepare_enable(hsotg->clks[clk]);
+		if (ret) {
+			while (--clk >= 0)
+				clk_disable_unprepare(hsotg->clks[clk]);
 			return ret;
+		}
 	}
 
 	if (hsotg->uphy) {
@@ -168,7 +171,7 @@ int dwc2_lowlevel_hw_enable(struct dwc2_hsotg *hsotg)
 static int __dwc2_lowlevel_hw_disable(struct dwc2_hsotg *hsotg)
 {
 	struct platform_device *pdev = to_platform_device(hsotg->dev);
-	int ret = 0;
+	int clk, ret = 0;
 
 	if (hsotg->uphy) {
 		usb_phy_shutdown(hsotg->uphy);
@@ -182,8 +185,9 @@ static int __dwc2_lowlevel_hw_disable(struct dwc2_hsotg *hsotg)
 	if (ret)
 		return ret;
 
-	if (hsotg->clk)
-		clk_disable_unprepare(hsotg->clk);
+	for (clk = DWC2_MAX_CLKS - 1; clk >= 0; clk--)
+		if (hsotg->clks[clk])
+			clk_disable_unprepare(hsotg->clks[clk]);
 
 	ret = regulator_bulk_disable(ARRAY_SIZE(hsotg->supplies),
 				     hsotg->supplies);
@@ -209,7 +213,7 @@ int dwc2_lowlevel_hw_disable(struct dwc2_hsotg *hsotg)
 
 static int dwc2_lowlevel_hw_init(struct dwc2_hsotg *hsotg)
 {
-	int i, ret;
+	int i, clk, ret;
 
 	hsotg->reset = devm_reset_control_get_optional(hsotg->dev, "dwc2");
 	if (IS_ERR(hsotg->reset)) {
@@ -282,11 +286,20 @@ static int dwc2_lowlevel_hw_init(struct dwc2_hsotg *hsotg)
 			hsotg->phyif = GUSBCFG_PHYIF8;
 	}
 
-	/* Clock */
-	hsotg->clk = devm_clk_get(hsotg->dev, "otg");
-	if (IS_ERR(hsotg->clk)) {
-		hsotg->clk = NULL;
-		dev_dbg(hsotg->dev, "cannot get otg clock\n");
+	/* Clocks */
+	for (clk = 0; clk < DWC2_MAX_CLKS; clk++) {
+		hsotg->clks[clk] = of_clk_get(hsotg->dev->of_node, clk);
+		if (IS_ERR(hsotg->clks[clk])) {
+			ret = PTR_ERR(hsotg->clks[clk]);
+			if (ret == -EPROBE_DEFER) {
+				while (--clk >= 0)
+					clk_put(hsotg->clks[clk]);
+				return ret;
+			}
+
+			hsotg->clks[clk] = NULL;
+			break;
+		}
 	}
 
 	/* Regulators */
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 14+ messages in thread

* Re: [RESEND PATCH v2 1/2] Documentation: dt: dwc2: amend clocks property
@ 2017-02-16  2:16     ` Rob Herring
  0 siblings, 0 replies; 14+ messages in thread
From: Rob Herring @ 2017-02-16  2:16 UTC (permalink / raw)
  To: Frank Wang
  Cc: johnyoun, gregkh, mark.rutland, heiko, linux-kernel, linux-usb,
	devicetree, linux-rockchip, huangtao, kever.yang, william.wu,
	daniel.meng

On Fri, Feb 10, 2017 at 11:06:26AM +0800, Frank Wang wrote:
> Move 'clocks' to optional properties since not every platform
> is going to need to specify it and add pmu/utmi/ulpi/utmifs
> optional clocks into 'clock-names' list.
> 
> Signed-off-by: Frank Wang <frank.wang@rock-chips.com>
> ---
>  Documentation/devicetree/bindings/usb/dwc2.txt | 13 ++++++++++---
>  1 file changed, 10 insertions(+), 3 deletions(-)

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

It would be good to get some acks from others familar with this block.

Rob

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [RESEND PATCH v2 1/2] Documentation: dt: dwc2: amend clocks property
@ 2017-02-16  2:16     ` Rob Herring
  0 siblings, 0 replies; 14+ messages in thread
From: Rob Herring @ 2017-02-16  2:16 UTC (permalink / raw)
  To: Frank Wang
  Cc: johnyoun-HKixBCOQz3hWk0Htik3J/w,
	gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r,
	mark.rutland-5wv7dgnIgG8, heiko-4mtYJXux2i+zQB+pC5nmwQ,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	linux-usb-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	huangtao-TNX95d0MmH7DzftRWevZcw,
	kever.yang-TNX95d0MmH7DzftRWevZcw,
	william.wu-TNX95d0MmH7DzftRWevZcw,
	daniel.meng-TNX95d0MmH7DzftRWevZcw

On Fri, Feb 10, 2017 at 11:06:26AM +0800, Frank Wang wrote:
> Move 'clocks' to optional properties since not every platform
> is going to need to specify it and add pmu/utmi/ulpi/utmifs
> optional clocks into 'clock-names' list.
> 
> Signed-off-by: Frank Wang <frank.wang-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
> ---
>  Documentation/devicetree/bindings/usb/dwc2.txt | 13 ++++++++++---
>  1 file changed, 10 insertions(+), 3 deletions(-)

Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> 

It would be good to get some acks from others familar with this block.

Rob
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [RESEND PATCH v2 0/2] add multiple clock handling for dwc2 driver
@ 2017-02-22  1:29   ` Frank Wang
  0 siblings, 0 replies; 14+ messages in thread
From: Frank Wang @ 2017-02-22  1:29 UTC (permalink / raw)
  To: johnyoun, gregkh, robh+dt, mark.rutland, heiko
  Cc: linux-kernel, linux-usb, devicetree, linux-rockchip, huangtao,
	kever.yang, william.wu, daniel.meng, frank.wang

Hi John and Greg,

Friendly ping... :-)

On 2017/2/10 11:06, Frank Wang wrote:
> Resend this series to involve device tree maintainer and add 'Reviewed-by' tag for driver.
>
> The Current default dwc2 just handle one clock named otg, however, it may have
> two or more clock need to manage for some new SoCs(such as RK3328), so this
> adds change clk to clk's array of dwc2_hsotg to handle more clocks operation.
>
> Changes in v2:
>   - amend dwc2 clocks property in DT.
>   - change DWC2_MAX_CLKS to 5 and amend commit message.
>
> Frank Wang (2):
>    Documentation: dt: dwc2: amend clocks property
>    usb: dwc2: add multiple clocks handling
>
>   Documentation/devicetree/bindings/usb/dwc2.txt | 13 +++++++--
>   drivers/usb/dwc2/core.h                        |  5 +++-
>   drivers/usb/dwc2/platform.c                    | 39 +++++++++++++++++---------
>   3 files changed, 40 insertions(+), 17 deletions(-)
>

BR.
Frank

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [RESEND PATCH v2 0/2] add multiple clock handling for dwc2 driver
@ 2017-02-22  1:29   ` Frank Wang
  0 siblings, 0 replies; 14+ messages in thread
From: Frank Wang @ 2017-02-22  1:29 UTC (permalink / raw)
  To: johnyoun-HKixBCOQz3hWk0Htik3J/w,
	gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, mark.rutland-5wv7dgnIgG8,
	heiko-4mtYJXux2i+zQB+pC5nmwQ
  Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	linux-usb-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	huangtao-TNX95d0MmH7DzftRWevZcw,
	kever.yang-TNX95d0MmH7DzftRWevZcw,
	william.wu-TNX95d0MmH7DzftRWevZcw,
	daniel.meng-TNX95d0MmH7DzftRWevZcw,
	frank.wang-TNX95d0MmH7DzftRWevZcw

Hi John and Greg,

Friendly ping... :-)

On 2017/2/10 11:06, Frank Wang wrote:
> Resend this series to involve device tree maintainer and add 'Reviewed-by' tag for driver.
>
> The Current default dwc2 just handle one clock named otg, however, it may have
> two or more clock need to manage for some new SoCs(such as RK3328), so this
> adds change clk to clk's array of dwc2_hsotg to handle more clocks operation.
>
> Changes in v2:
>   - amend dwc2 clocks property in DT.
>   - change DWC2_MAX_CLKS to 5 and amend commit message.
>
> Frank Wang (2):
>    Documentation: dt: dwc2: amend clocks property
>    usb: dwc2: add multiple clocks handling
>
>   Documentation/devicetree/bindings/usb/dwc2.txt | 13 +++++++--
>   drivers/usb/dwc2/core.h                        |  5 +++-
>   drivers/usb/dwc2/platform.c                    | 39 +++++++++++++++++---------
>   3 files changed, 40 insertions(+), 17 deletions(-)
>

BR.
Frank

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [RESEND PATCH v2 0/2] add multiple clock handling for dwc2 driver
@ 2017-02-22 14:58     ` Greg KH
  0 siblings, 0 replies; 14+ messages in thread
From: Greg KH @ 2017-02-22 14:58 UTC (permalink / raw)
  To: Frank Wang
  Cc: johnyoun, robh+dt, mark.rutland, heiko, linux-kernel, linux-usb,
	devicetree, linux-rockchip, huangtao, kever.yang, william.wu,
	daniel.meng

On Wed, Feb 22, 2017 at 09:29:43AM +0800, Frank Wang wrote:
> Hi John and Greg,
> 
> Friendly ping... :-)

I'm not the one to take dwc2 patches, sorry, nothing I can do here,
especially in the middle of the merge window...

greg k-h

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [RESEND PATCH v2 0/2] add multiple clock handling for dwc2 driver
@ 2017-02-22 14:58     ` Greg KH
  0 siblings, 0 replies; 14+ messages in thread
From: Greg KH @ 2017-02-22 14:58 UTC (permalink / raw)
  To: Frank Wang
  Cc: johnyoun-HKixBCOQz3hWk0Htik3J/w, robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
	mark.rutland-5wv7dgnIgG8, heiko-4mtYJXux2i+zQB+pC5nmwQ,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	linux-usb-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	huangtao-TNX95d0MmH7DzftRWevZcw,
	kever.yang-TNX95d0MmH7DzftRWevZcw,
	william.wu-TNX95d0MmH7DzftRWevZcw,
	daniel.meng-TNX95d0MmH7DzftRWevZcw

On Wed, Feb 22, 2017 at 09:29:43AM +0800, Frank Wang wrote:
> Hi John and Greg,
> 
> Friendly ping... :-)

I'm not the one to take dwc2 patches, sorry, nothing I can do here,
especially in the middle of the merge window...

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [RESEND PATCH v2 0/2] add multiple clock handling for dwc2 driver
  2017-02-22  1:29   ` Frank Wang
@ 2017-02-22 20:19     ` John Youn
  -1 siblings, 0 replies; 14+ messages in thread
From: John Youn @ 2017-02-22 20:19 UTC (permalink / raw)
  To: Frank Wang, John.Youn, gregkh, robh+dt, mark.rutland, heiko
  Cc: linux-kernel, linux-usb, devicetree, linux-rockchip, huangtao,
	kever.yang, william.wu, daniel.meng

On 2/21/2017 5:30 PM, Frank Wang wrote:
> Hi John and Greg,
>
> Friendly ping... :-)
>

Hi Frank,

We'll take a look at this and get back to you soon.

Regards,
John


> On 2017/2/10 11:06, Frank Wang wrote:
>> Resend this series to involve device tree maintainer and add 'Reviewed-by' tag for driver.
>>
>> The Current default dwc2 just handle one clock named otg, however, it may have
>> two or more clock need to manage for some new SoCs(such as RK3328), so this
>> adds change clk to clk's array of dwc2_hsotg to handle more clocks operation.
>>
>> Changes in v2:
>>   - amend dwc2 clocks property in DT.
>>   - change DWC2_MAX_CLKS to 5 and amend commit message.
>>
>> Frank Wang (2):
>>    Documentation: dt: dwc2: amend clocks property
>>    usb: dwc2: add multiple clocks handling
>>
>>   Documentation/devicetree/bindings/usb/dwc2.txt | 13 +++++++--
>>   drivers/usb/dwc2/core.h                        |  5 +++-
>>   drivers/usb/dwc2/platform.c                    | 39 +++++++++++++++++---------
>>   3 files changed, 40 insertions(+), 17 deletions(-)
>>
>
> BR.
> Frank
>
>

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [RESEND PATCH v2 0/2] add multiple clock handling for dwc2 driver
@ 2017-02-22 20:19     ` John Youn
  0 siblings, 0 replies; 14+ messages in thread
From: John Youn @ 2017-02-22 20:19 UTC (permalink / raw)
  To: Frank Wang, John.Youn-HKixBCOQz3hWk0Htik3J/w,
	gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, mark.rutland-5wv7dgnIgG8,
	heiko-4mtYJXux2i+zQB+pC5nmwQ
  Cc: huangtao-TNX95d0MmH7DzftRWevZcw,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-usb-u79uwXL29TY76Z2rM5mHXA,
	kever.yang-TNX95d0MmH7DzftRWevZcw,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	william.wu-TNX95d0MmH7DzftRWevZcw,
	daniel.meng-TNX95d0MmH7DzftRWevZcw

On 2/21/2017 5:30 PM, Frank Wang wrote:
> Hi John and Greg,
>
> Friendly ping... :-)
>

Hi Frank,

We'll take a look at this and get back to you soon.

Regards,
John


> On 2017/2/10 11:06, Frank Wang wrote:
>> Resend this series to involve device tree maintainer and add 'Reviewed-by' tag for driver.
>>
>> The Current default dwc2 just handle one clock named otg, however, it may have
>> two or more clock need to manage for some new SoCs(such as RK3328), so this
>> adds change clk to clk's array of dwc2_hsotg to handle more clocks operation.
>>
>> Changes in v2:
>>   - amend dwc2 clocks property in DT.
>>   - change DWC2_MAX_CLKS to 5 and amend commit message.
>>
>> Frank Wang (2):
>>    Documentation: dt: dwc2: amend clocks property
>>    usb: dwc2: add multiple clocks handling
>>
>>   Documentation/devicetree/bindings/usb/dwc2.txt | 13 +++++++--
>>   drivers/usb/dwc2/core.h                        |  5 +++-
>>   drivers/usb/dwc2/platform.c                    | 39 +++++++++++++++++---------
>>   3 files changed, 40 insertions(+), 17 deletions(-)
>>
>
> BR.
> Frank
>
>

^ permalink raw reply	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2017-02-22 20:20 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-10  3:06 [RESEND PATCH v2 0/2] add multiple clock handling for dwc2 driver Frank Wang
2017-02-10  3:06 ` Frank Wang
2017-02-10  3:06 ` [RESEND PATCH v2 1/2] Documentation: dt: dwc2: amend clocks property Frank Wang
2017-02-10  3:06   ` Frank Wang
2017-02-16  2:16   ` Rob Herring
2017-02-16  2:16     ` Rob Herring
2017-02-10  3:06 ` [RESEND PATCH v2 2/2] usb: dwc2: add multiple clocks handling Frank Wang
2017-02-10  3:06   ` Frank Wang
2017-02-22  1:29 ` [RESEND PATCH v2 0/2] add multiple clock handling for dwc2 driver Frank Wang
2017-02-22  1:29   ` Frank Wang
2017-02-22 14:58   ` Greg KH
2017-02-22 14:58     ` Greg KH
2017-02-22 20:19   ` John Youn
2017-02-22 20:19     ` John Youn

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.