[PATCH v3] phy: rcar-gen3-usb3: add support for R-Car Gen3 USB 3.0 PHY

[PATCH v3] phy: rcar-gen3-usb3: add support for R-Car Gen3 USB 3.0 PHY

[Yoshihiro Shimoda]

The USB 3.0 PHY modules of R-Car Gen3 SoCs have:
 - Spread spectrum clock (ssc).
 - Using USB 2.0 EXTAL clock instead of USB 3.0 clock.
 - Enabling VBUS detection for usb3.0 peripheral.

So, this driver supports these features.

Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
---
 This patch is based on the latest linux-phy.git / next branch and
 "[PATCH v5 3/3] phy: Group vendor specific phy drivers" patch.

 I discussed with Geert-san about this driver on the ML:
 https://patchwork.kernel.org/patch/9731759/

 Changes from v2:
 - Revise the device tree doc.
 - Improve initialization method in rcar_gen3_phy_usb3_init().
 - remove some values from rcar_gen3_phy_usb3_match_table.

 Changes from v1:
 - Changes from dev_info to dev_vdbg in rcar_gen3_phy_usb3_init().

 .../devicetree/bindings/phy/rcar-gen3-phy-usb3.txt |  46 +++++
 MAINTAINERS                                        |   4 +-
 drivers/phy/renesas/Kconfig                        |   7 +
 drivers/phy/renesas/Makefile                       |   1 +
 drivers/phy/renesas/phy-rcar-gen3-usb3.c           | 226 +++++++++++++++++++++
 5 files changed, 282 insertions(+), 2 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/phy/rcar-gen3-phy-usb3.txt
 create mode 100644 drivers/phy/renesas/phy-rcar-gen3-usb3.c

diff --git a/Documentation/devicetree/bindings/phy/rcar-gen3-phy-usb3.txt b/Documentation/devicetree/bindings/phy/rcar-gen3-phy-usb3.txt
new file mode 100644
index 0000000..f94cea4
--- /dev/null
+++ b/Documentation/devicetree/bindings/phy/rcar-gen3-phy-usb3.txt
@@ -0,0 +1,46 @@
+* Renesas R-Car generation 3 USB 3.0 PHY
+
+This file provides information on what the device node for the R-Car generation
+3 USB 3.0 PHY contains.
+If you want to enable spread spectrum clock (ssc), you should use USB_EXTAL
+instead of USB3_CLK. However, if you don't want to these features, you don't
+need this driver.
+
+Required properties:
+- compatible: "renesas,r8a7795-usb3-phy" if the device is a part of an R8A7795
+	      SoC.
+	      "renesas,r8a7796-usb3-phy" if the device is a part of an R8A7796
+	      SoC.
+	      "renesas,rcar-gen3-usb3-phy" for a generic R-Car Gen3 compatible
+	      device.
+
+	      When compatible with the generic version, nodes must list the
+	      SoC-specific version corresponding to the platform first
+	      followed by the generic version.
+
+- reg: offset and length of the USB 3.0 PHY register block.
+- clocks: A list of phandles and clock-specifier pairs.
+- clock-names: Name of the clocks.
+  - The funcional clock must be "usb3-if".
+  - The usb3's external clock must be "usb3s_clk".
+  - The usb2's external clock must be "usb_extal". If you want to use the ssc,
+    the clock-frequency must not be 0.
+- #phy-cells: see phy-bindings.txt in the same directory, must be <0>.
+
+Optional properties:
+- renesas,ssc-range: Enable/disable spread spectrum clock (ssc) by using
+		     the following values as u32:
+			- 0 (or the property doesn't exist): disable the ssc
+			- 4980: enable the ssc as -4980 ppm
+			- 4492: enable the ssc as -4492 ppm
+			- 4003: enable the ssc as -4003 ppm
+
+Example (R-Car H3):
+
+	usb-phy@e65ee000 {
+		compatible = "renesas,r8a7795-usb3-phy",
+			     "renesas,rcar-gen3-usb3-phy";
+		reg = <0 0xe65ee000 0 0x90>;
+		clocks = <&cpg CPG_MOD 328>, <&usb3s0_clk>, <&usb_extal>;
+		clock-names = "usb3-if", "usb3s_clk", "usb_extal";
+	};
diff --git a/MAINTAINERS b/MAINTAINERS
index a47d3da..d711d53 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -10829,11 +10829,11 @@ L:	linux-iio@vger.kernel.org
 S:	Supported
 F:	drivers/iio/adc/rcar_gyro_adc.c
 
-RENESAS USB2 PHY DRIVER
+RENESAS USB PHY DRIVER
 M:	Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
 L:	linux-renesas-soc@vger.kernel.org
 S:	Maintained
-F:	drivers/phy/renesas/phy-rcar-gen3-usb2.c
+F:	drivers/phy/renesas/phy-rcar-gen3-usb*.c
 
 RESET CONTROLLER FRAMEWORK
 M:	Philipp Zabel <p.zabel@pengutronix.de>
diff --git a/drivers/phy/renesas/Kconfig b/drivers/phy/renesas/Kconfig
index 432e271..4377064 100644
--- a/drivers/phy/renesas/Kconfig
+++ b/drivers/phy/renesas/Kconfig
@@ -15,3 +15,10 @@ config PHY_RCAR_GEN3_USB2
 	select GENERIC_PHY
 	help
 	  Support for USB 2.0 PHY found on Renesas R-Car generation 3 SoCs.
+
+config PHY_RCAR_GEN3_USB3
+	tristate "Renesas R-Car generation 3 USB 3.0 PHY driver"
+	depends on ARCH_RENESAS
+	select GENERIC_PHY
+	help
+	  Support for USB 3.0 PHY found on Renesas R-Car generation 3 SoCs.
diff --git a/drivers/phy/renesas/Makefile b/drivers/phy/renesas/Makefile
index 695241a..8b60259 100644
--- a/drivers/phy/renesas/Makefile
+++ b/drivers/phy/renesas/Makefile
@@ -1,2 +1,3 @@
 obj-$(CONFIG_PHY_RCAR_GEN2)		+= phy-rcar-gen2.o
 obj-$(CONFIG_PHY_RCAR_GEN3_USB2)	+= phy-rcar-gen3-usb2.o
+obj-$(CONFIG_PHY_RCAR_GEN3_USB3)	+= phy-rcar-gen3-usb3.o
diff --git a/drivers/phy/renesas/phy-rcar-gen3-usb3.c b/drivers/phy/renesas/phy-rcar-gen3-usb3.c
new file mode 100644
index 0000000..88c83c9
--- /dev/null
+++ b/drivers/phy/renesas/phy-rcar-gen3-usb3.c
@@ -0,0 +1,226 @@
+/*
+ * Renesas R-Car Gen3 for USB3.0 PHY driver
+ *
+ * Copyright (C) 2017 Renesas Electronics Corporation
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ */
+
+#include <linux/clk.h>
+#include <linux/delay.h>
+#include <linux/io.h>
+#include <linux/module.h>
+#include <linux/of.h>
+#include <linux/phy/phy.h>
+#include <linux/platform_device.h>
+#include <linux/pm_runtime.h>
+
+#define USB30_CLKSET0		0x034
+#define USB30_CLKSET1		0x036
+#define USB30_SSC_SET		0x038
+#define USB30_PHY_ENABLE	0x060
+#define USB30_VBUS_EN		0x064
+
+/* USB30_CLKSET0 */
+#define CLKSET0_PRIVATE			0x05c0
+#define CLKSET0_USB30_FSEL_USB_EXTAL	0x0002
+
+/* USB30_CLKSET1 */
+#define CLKSET1_USB30_PLL_MULTI_SHIFT		6
+#define CLKSET1_USB30_PLL_MULTI_USB_EXTAL	(0x64 << \
+						 CLKSET1_USB30_PLL_MULTI_SHIFT)
+#define CLKSET1_PHYRESET	BIT(4)	/* 1: reset */
+#define CLKSET1_REF_CLKDIV	BIT(3)	/* 1: USB_EXTAL */
+#define CLKSET1_PRIVATE_2_1	BIT(1)	/* Write B'01 */
+#define CLKSET1_REF_CLK_SEL	BIT(0)	/* 1: USB3S0_CLK_P */
+
+/* USB30_SSC_SET */
+#define SSC_SET_SSC_EN		BIT(12)
+#define SSC_SET_RANGE_SHIFT	9
+#define SSC_SET_RANGE_4980	(0x0 << SSC_SET_RANGE_SHIFT)
+#define SSC_SET_RANGE_4492	(0x1 << SSC_SET_RANGE_SHIFT)
+#define SSC_SET_RANGE_4003	(0x2 << SSC_SET_RANGE_SHIFT)
+
+/* USB30_PHY_ENABLE */
+#define PHY_ENABLE_RESET_EN	BIT(4)
+
+/* USB30_VBUS_EN */
+#define VBUS_EN_VBUS_EN		BIT(1)
+
+struct rcar_gen3_usb3 {
+	void __iomem *base;
+	struct phy *phy;
+	u32 ssc_range;
+	bool usb3s_clk;
+	bool usb_extal;
+};
+
+static void write_clkset1_for_usb_extal(struct rcar_gen3_usb3 *r, bool reset)
+{
+	u16 val = CLKSET1_USB30_PLL_MULTI_USB_EXTAL |
+		  CLKSET1_REF_CLKDIV | CLKSET1_PRIVATE_2_1;
+
+	if (reset)
+		val |= CLKSET1_PHYRESET;
+
+	writew(val, r->base + USB30_CLKSET1);
+}
+
+static void rcar_gen3_phy_usb3_enable_ssc(struct rcar_gen3_usb3 *r)
+{
+	u16 val = SSC_SET_SSC_EN;
+
+	switch (r->ssc_range) {
+	case 4980:
+		val |= SSC_SET_RANGE_4980;
+		break;
+	case 4492:
+		val |= SSC_SET_RANGE_4492;
+		break;
+	case 4003:
+		val |= SSC_SET_RANGE_4003;
+		break;
+	default:
+		dev_err(&r->phy->dev, "%s: unsupported range (%x)\n", __func__,
+			r->ssc_range);
+		return;
+	}
+
+	writew(val, r->base + USB30_SSC_SET);
+}
+
+static void rcar_gen3_phy_usb3_select_usb_extal(struct rcar_gen3_usb3 *r)
+{
+	write_clkset1_for_usb_extal(r, false);
+	if (r->ssc_range)
+		rcar_gen3_phy_usb3_enable_ssc(r);
+	writew(CLKSET0_PRIVATE | CLKSET0_USB30_FSEL_USB_EXTAL,
+	       r->base + USB30_CLKSET0);
+	writew(PHY_ENABLE_RESET_EN, r->base + USB30_PHY_ENABLE);
+	write_clkset1_for_usb_extal(r, true);
+	usleep_range(10, 20);
+	write_clkset1_for_usb_extal(r, false);
+}
+
+static int rcar_gen3_phy_usb3_init(struct phy *p)
+{
+	struct rcar_gen3_usb3 *r = phy_get_drvdata(p);
+
+	dev_vdbg(&r->phy->dev, "%s: enter (%d, %d, %d)\n", __func__,
+		 r->usb3s_clk, r->usb_extal, r->ssc_range);
+
+	if (!r->usb3s_clk && r->usb_extal)
+		rcar_gen3_phy_usb3_select_usb_extal(r);
+
+	/* Enables VBUS detection anyway */
+	writew(VBUS_EN_VBUS_EN, r->base + USB30_VBUS_EN);
+
+	return 0;
+}
+
+static const struct phy_ops rcar_gen3_phy_usb3_ops = {
+	.init		= rcar_gen3_phy_usb3_init,
+	.owner		= THIS_MODULE,
+};
+
+static const struct of_device_id rcar_gen3_phy_usb3_match_table[] = {
+	{ .compatible = "renesas,rcar-gen3-usb3-phy" },
+	{ }
+};
+MODULE_DEVICE_TABLE(of, rcar_gen3_phy_usb3_match_table);
+
+static int rcar_gen3_phy_usb3_probe(struct platform_device *pdev)
+{
+	struct device *dev = &pdev->dev;
+	struct rcar_gen3_usb3 *r;
+	struct phy_provider *provider;
+	struct resource *res;
+	int ret = 0;
+	struct clk *clk;
+
+	if (!dev->of_node) {
+		dev_err(dev, "This driver needs device tree\n");
+		return -EINVAL;
+	}
+
+	r = devm_kzalloc(dev, sizeof(*r), GFP_KERNEL);
+	if (!r)
+		return -ENOMEM;
+
+	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+	r->base = devm_ioremap_resource(dev, res);
+	if (IS_ERR(r->base))
+		return PTR_ERR(r->base);
+
+	clk = devm_clk_get(dev, "usb3s_clk");
+	if (!IS_ERR(clk) && !clk_prepare_enable(clk)) {
+		r->usb3s_clk = !!clk_get_rate(clk);
+		clk_disable_unprepare(clk);
+	}
+	clk = devm_clk_get(dev, "usb_extal");
+	if (!IS_ERR(clk) && !clk_prepare_enable(clk)) {
+		r->usb_extal = !!clk_get_rate(clk);
+		clk_disable_unprepare(clk);
+	}
+
+	if (!r->usb3s_clk && !r->usb_extal) {
+		dev_err(dev, "This driver needs usb3s_clk and/or usb_extal\n");
+		ret = -EINVAL;
+		goto error;
+	}
+
+	/*
+	 * devm_phy_create() will call pm_runtime_enable(&phy->dev);
+	 * And then, phy-core will manage runtime pm for this device.
+	 */
+	pm_runtime_enable(dev);
+
+	r->phy = devm_phy_create(dev, NULL, &rcar_gen3_phy_usb3_ops);
+	if (IS_ERR(r->phy)) {
+		dev_err(dev, "Failed to create USB3 PHY\n");
+		ret = PTR_ERR(r->phy);
+		goto error;
+	}
+
+	of_property_read_u32(dev->of_node, "renesas,ssc-range", &r->ssc_range);
+
+	platform_set_drvdata(pdev, r);
+	phy_set_drvdata(r->phy, r);
+
+	provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate);
+	if (IS_ERR(provider)) {
+		dev_err(dev, "Failed to register PHY provider\n");
+		ret = PTR_ERR(provider);
+		goto error;
+	}
+
+	return 0;
+
+error:
+	pm_runtime_disable(dev);
+
+	return ret;
+}
+
+static int rcar_gen3_phy_usb3_remove(struct platform_device *pdev)
+{
+	pm_runtime_disable(&pdev->dev);
+
+	return 0;
+};
+
+static struct platform_driver rcar_gen3_phy_usb3_driver = {
+	.driver = {
+		.name		= "phy_rcar_gen3_usb3",
+		.of_match_table	= rcar_gen3_phy_usb3_match_table,
+	},
+	.probe	= rcar_gen3_phy_usb3_probe,
+	.remove = rcar_gen3_phy_usb3_remove,
+};
+module_platform_driver(rcar_gen3_phy_usb3_driver);
+
+MODULE_LICENSE("GPL v2");
+MODULE_DESCRIPTION("Renesas R-Car Gen3 USB 3.0 PHY");
+MODULE_AUTHOR("Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>");
-- 
1.9.1

RE: [PATCH v3] phy: rcar-gen3-usb3: add support for R-Car Gen3 USB 3.0 PHY

[Yoshihiro Shimoda]

Hi Kishon,

> From: Yoshihiro Shimoda
> Sent: Monday, May 29, 2017 7:42 PM
> 
> The USB 3.0 PHY modules of R-Car Gen3 SoCs have:
>  - Spread spectrum clock (ssc).
>  - Using USB 2.0 EXTAL clock instead of USB 3.0 clock.
>  - Enabling VBUS detection for usb3.0 peripheral.
> 
> So, this driver supports these features.
> 
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>

Would you review this patch?
I confirmed this patch could be applied on the latest linux-phy.git / next branch.

Best regards.
Yoshihiro Shimoda

> ---
>  This patch is based on the latest linux-phy.git / next branch and
>  "[PATCH v5 3/3] phy: Group vendor specific phy drivers" patch.
> 
>  I discussed with Geert-san about this driver on the ML:
>  https://patchwork.kernel.org/patch/9731759/
> 
>  Changes from v2:
>  - Revise the device tree doc.
>  - Improve initialization method in rcar_gen3_phy_usb3_init().
>  - remove some values from rcar_gen3_phy_usb3_match_table.
> 
>  Changes from v1:
>  - Changes from dev_info to dev_vdbg in rcar_gen3_phy_usb3_init().
> 
>  .../devicetree/bindings/phy/rcar-gen3-phy-usb3.txt |  46 +++++
>  MAINTAINERS                                        |   4 +-
>  drivers/phy/renesas/Kconfig                        |   7 +
>  drivers/phy/renesas/Makefile                       |   1 +
>  drivers/phy/renesas/phy-rcar-gen3-usb3.c           | 226 +++++++++++++++++++++
>  5 files changed, 282 insertions(+), 2 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/phy/rcar-gen3-phy-usb3.txt
>  create mode 100644 drivers/phy/renesas/phy-rcar-gen3-usb3.c
> 
> diff --git a/Documentation/devicetree/bindings/phy/rcar-gen3-phy-usb3.txt
> b/Documentation/devicetree/bindings/phy/rcar-gen3-phy-usb3.txt
> new file mode 100644
> index 0000000..f94cea4
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/phy/rcar-gen3-phy-usb3.txt
> @@ -0,0 +1,46 @@
> +* Renesas R-Car generation 3 USB 3.0 PHY
> +
> +This file provides information on what the device node for the R-Car generation
> +3 USB 3.0 PHY contains.
> +If you want to enable spread spectrum clock (ssc), you should use USB_EXTAL
> +instead of USB3_CLK. However, if you don't want to these features, you don't
> +need this driver.
> +
> +Required properties:
> +- compatible: "renesas,r8a7795-usb3-phy" if the device is a part of an R8A7795
> +	      SoC.
> +	      "renesas,r8a7796-usb3-phy" if the device is a part of an R8A7796
> +	      SoC.
> +	      "renesas,rcar-gen3-usb3-phy" for a generic R-Car Gen3 compatible
> +	      device.
> +
> +	      When compatible with the generic version, nodes must list the
> +	      SoC-specific version corresponding to the platform first
> +	      followed by the generic version.
> +
> +- reg: offset and length of the USB 3.0 PHY register block.
> +- clocks: A list of phandles and clock-specifier pairs.
> +- clock-names: Name of the clocks.
> +  - The funcional clock must be "usb3-if".
> +  - The usb3's external clock must be "usb3s_clk".
> +  - The usb2's external clock must be "usb_extal". If you want to use the ssc,
> +    the clock-frequency must not be 0.
> +- #phy-cells: see phy-bindings.txt in the same directory, must be <0>.
> +
> +Optional properties:
> +- renesas,ssc-range: Enable/disable spread spectrum clock (ssc) by using
> +		     the following values as u32:
> +			- 0 (or the property doesn't exist): disable the ssc
> +			- 4980: enable the ssc as -4980 ppm
> +			- 4492: enable the ssc as -4492 ppm
> +			- 4003: enable the ssc as -4003 ppm
> +
> +Example (R-Car H3):
> +
> +	usb-phy@e65ee000 {
> +		compatible = "renesas,r8a7795-usb3-phy",
> +			     "renesas,rcar-gen3-usb3-phy";
> +		reg = <0 0xe65ee000 0 0x90>;
> +		clocks = <&cpg CPG_MOD 328>, <&usb3s0_clk>, <&usb_extal>;
> +		clock-names = "usb3-if", "usb3s_clk", "usb_extal";
> +	};
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a47d3da..d711d53 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -10829,11 +10829,11 @@ L:	linux-iio@vger.kernel.org
>  S:	Supported
>  F:	drivers/iio/adc/rcar_gyro_adc.c
> 
> -RENESAS USB2 PHY DRIVER
> +RENESAS USB PHY DRIVER
>  M:	Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
>  L:	linux-renesas-soc@vger.kernel.org
>  S:	Maintained
> -F:	drivers/phy/renesas/phy-rcar-gen3-usb2.c
> +F:	drivers/phy/renesas/phy-rcar-gen3-usb*.c
> 
>  RESET CONTROLLER FRAMEWORK
>  M:	Philipp Zabel <p.zabel@pengutronix.de>
> diff --git a/drivers/phy/renesas/Kconfig b/drivers/phy/renesas/Kconfig
> index 432e271..4377064 100644
> --- a/drivers/phy/renesas/Kconfig
> +++ b/drivers/phy/renesas/Kconfig
> @@ -15,3 +15,10 @@ config PHY_RCAR_GEN3_USB2
>  	select GENERIC_PHY
>  	help
>  	  Support for USB 2.0 PHY found on Renesas R-Car generation 3 SoCs.
> +
> +config PHY_RCAR_GEN3_USB3
> +	tristate "Renesas R-Car generation 3 USB 3.0 PHY driver"
> +	depends on ARCH_RENESAS
> +	select GENERIC_PHY
> +	help
> +	  Support for USB 3.0 PHY found on Renesas R-Car generation 3 SoCs.
> diff --git a/drivers/phy/renesas/Makefile b/drivers/phy/renesas/Makefile
> index 695241a..8b60259 100644
> --- a/drivers/phy/renesas/Makefile
> +++ b/drivers/phy/renesas/Makefile
> @@ -1,2 +1,3 @@
>  obj-$(CONFIG_PHY_RCAR_GEN2)		+= phy-rcar-gen2.o
>  obj-$(CONFIG_PHY_RCAR_GEN3_USB2)	+= phy-rcar-gen3-usb2.o
> +obj-$(CONFIG_PHY_RCAR_GEN3_USB3)	+= phy-rcar-gen3-usb3.o
> diff --git a/drivers/phy/renesas/phy-rcar-gen3-usb3.c b/drivers/phy/renesas/phy-rcar-gen3-usb3.c
> new file mode 100644
> index 0000000..88c83c9
> --- /dev/null
> +++ b/drivers/phy/renesas/phy-rcar-gen3-usb3.c
> @@ -0,0 +1,226 @@
> +/*
> + * Renesas R-Car Gen3 for USB3.0 PHY driver
> + *
> + * Copyright (C) 2017 Renesas Electronics Corporation
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#include <linux/clk.h>
> +#include <linux/delay.h>
> +#include <linux/io.h>
> +#include <linux/module.h>
> +#include <linux/of.h>
> +#include <linux/phy/phy.h>
> +#include <linux/platform_device.h>
> +#include <linux/pm_runtime.h>
> +
> +#define USB30_CLKSET0		0x034
> +#define USB30_CLKSET1		0x036
> +#define USB30_SSC_SET		0x038
> +#define USB30_PHY_ENABLE	0x060
> +#define USB30_VBUS_EN		0x064
> +
> +/* USB30_CLKSET0 */
> +#define CLKSET0_PRIVATE			0x05c0
> +#define CLKSET0_USB30_FSEL_USB_EXTAL	0x0002
> +
> +/* USB30_CLKSET1 */
> +#define CLKSET1_USB30_PLL_MULTI_SHIFT		6
> +#define CLKSET1_USB30_PLL_MULTI_USB_EXTAL	(0x64 << \
> +						 CLKSET1_USB30_PLL_MULTI_SHIFT)
> +#define CLKSET1_PHYRESET	BIT(4)	/* 1: reset */
> +#define CLKSET1_REF_CLKDIV	BIT(3)	/* 1: USB_EXTAL */
> +#define CLKSET1_PRIVATE_2_1	BIT(1)	/* Write B'01 */
> +#define CLKSET1_REF_CLK_SEL	BIT(0)	/* 1: USB3S0_CLK_P */
> +
> +/* USB30_SSC_SET */
> +#define SSC_SET_SSC_EN		BIT(12)
> +#define SSC_SET_RANGE_SHIFT	9
> +#define SSC_SET_RANGE_4980	(0x0 << SSC_SET_RANGE_SHIFT)
> +#define SSC_SET_RANGE_4492	(0x1 << SSC_SET_RANGE_SHIFT)
> +#define SSC_SET_RANGE_4003	(0x2 << SSC_SET_RANGE_SHIFT)
> +
> +/* USB30_PHY_ENABLE */
> +#define PHY_ENABLE_RESET_EN	BIT(4)
> +
> +/* USB30_VBUS_EN */
> +#define VBUS_EN_VBUS_EN		BIT(1)
> +
> +struct rcar_gen3_usb3 {
> +	void __iomem *base;
> +	struct phy *phy;
> +	u32 ssc_range;
> +	bool usb3s_clk;
> +	bool usb_extal;
> +};
> +
> +static void write_clkset1_for_usb_extal(struct rcar_gen3_usb3 *r, bool reset)
> +{
> +	u16 val = CLKSET1_USB30_PLL_MULTI_USB_EXTAL |
> +		  CLKSET1_REF_CLKDIV | CLKSET1_PRIVATE_2_1;
> +
> +	if (reset)
> +		val |= CLKSET1_PHYRESET;
> +
> +	writew(val, r->base + USB30_CLKSET1);
> +}
> +
> +static void rcar_gen3_phy_usb3_enable_ssc(struct rcar_gen3_usb3 *r)
> +{
> +	u16 val = SSC_SET_SSC_EN;
> +
> +	switch (r->ssc_range) {
> +	case 4980:
> +		val |= SSC_SET_RANGE_4980;
> +		break;
> +	case 4492:
> +		val |= SSC_SET_RANGE_4492;
> +		break;
> +	case 4003:
> +		val |= SSC_SET_RANGE_4003;
> +		break;
> +	default:
> +		dev_err(&r->phy->dev, "%s: unsupported range (%x)\n", __func__,
> +			r->ssc_range);
> +		return;
> +	}
> +
> +	writew(val, r->base + USB30_SSC_SET);
> +}
> +
> +static void rcar_gen3_phy_usb3_select_usb_extal(struct rcar_gen3_usb3 *r)
> +{
> +	write_clkset1_for_usb_extal(r, false);
> +	if (r->ssc_range)
> +		rcar_gen3_phy_usb3_enable_ssc(r);
> +	writew(CLKSET0_PRIVATE | CLKSET0_USB30_FSEL_USB_EXTAL,
> +	       r->base + USB30_CLKSET0);
> +	writew(PHY_ENABLE_RESET_EN, r->base + USB30_PHY_ENABLE);
> +	write_clkset1_for_usb_extal(r, true);
> +	usleep_range(10, 20);
> +	write_clkset1_for_usb_extal(r, false);
> +}
> +
> +static int rcar_gen3_phy_usb3_init(struct phy *p)
> +{
> +	struct rcar_gen3_usb3 *r = phy_get_drvdata(p);
> +
> +	dev_vdbg(&r->phy->dev, "%s: enter (%d, %d, %d)\n", __func__,
> +		 r->usb3s_clk, r->usb_extal, r->ssc_range);
> +
> +	if (!r->usb3s_clk && r->usb_extal)
> +		rcar_gen3_phy_usb3_select_usb_extal(r);
> +
> +	/* Enables VBUS detection anyway */
> +	writew(VBUS_EN_VBUS_EN, r->base + USB30_VBUS_EN);
> +
> +	return 0;
> +}
> +
> +static const struct phy_ops rcar_gen3_phy_usb3_ops = {
> +	.init		= rcar_gen3_phy_usb3_init,
> +	.owner		= THIS_MODULE,
> +};
> +
> +static const struct of_device_id rcar_gen3_phy_usb3_match_table[] = {
> +	{ .compatible = "renesas,rcar-gen3-usb3-phy" },
> +	{ }
> +};
> +MODULE_DEVICE_TABLE(of, rcar_gen3_phy_usb3_match_table);
> +
> +static int rcar_gen3_phy_usb3_probe(struct platform_device *pdev)
> +{
> +	struct device *dev = &pdev->dev;
> +	struct rcar_gen3_usb3 *r;
> +	struct phy_provider *provider;
> +	struct resource *res;
> +	int ret = 0;
> +	struct clk *clk;
> +
> +	if (!dev->of_node) {
> +		dev_err(dev, "This driver needs device tree\n");
> +		return -EINVAL;
> +	}
> +
> +	r = devm_kzalloc(dev, sizeof(*r), GFP_KERNEL);
> +	if (!r)
> +		return -ENOMEM;
> +
> +	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> +	r->base = devm_ioremap_resource(dev, res);
> +	if (IS_ERR(r->base))
> +		return PTR_ERR(r->base);
> +
> +	clk = devm_clk_get(dev, "usb3s_clk");
> +	if (!IS_ERR(clk) && !clk_prepare_enable(clk)) {
> +		r->usb3s_clk = !!clk_get_rate(clk);
> +		clk_disable_unprepare(clk);
> +	}
> +	clk = devm_clk_get(dev, "usb_extal");
> +	if (!IS_ERR(clk) && !clk_prepare_enable(clk)) {
> +		r->usb_extal = !!clk_get_rate(clk);
> +		clk_disable_unprepare(clk);
> +	}
> +
> +	if (!r->usb3s_clk && !r->usb_extal) {
> +		dev_err(dev, "This driver needs usb3s_clk and/or usb_extal\n");
> +		ret = -EINVAL;
> +		goto error;
> +	}
> +
> +	/*
> +	 * devm_phy_create() will call pm_runtime_enable(&phy->dev);
> +	 * And then, phy-core will manage runtime pm for this device.
> +	 */
> +	pm_runtime_enable(dev);
> +
> +	r->phy = devm_phy_create(dev, NULL, &rcar_gen3_phy_usb3_ops);
> +	if (IS_ERR(r->phy)) {
> +		dev_err(dev, "Failed to create USB3 PHY\n");
> +		ret = PTR_ERR(r->phy);
> +		goto error;
> +	}
> +
> +	of_property_read_u32(dev->of_node, "renesas,ssc-range", &r->ssc_range);
> +
> +	platform_set_drvdata(pdev, r);
> +	phy_set_drvdata(r->phy, r);
> +
> +	provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate);
> +	if (IS_ERR(provider)) {
> +		dev_err(dev, "Failed to register PHY provider\n");
> +		ret = PTR_ERR(provider);
> +		goto error;
> +	}
> +
> +	return 0;
> +
> +error:
> +	pm_runtime_disable(dev);
> +
> +	return ret;
> +}
> +
> +static int rcar_gen3_phy_usb3_remove(struct platform_device *pdev)
> +{
> +	pm_runtime_disable(&pdev->dev);
> +
> +	return 0;
> +};
> +
> +static struct platform_driver rcar_gen3_phy_usb3_driver = {
> +	.driver = {
> +		.name		= "phy_rcar_gen3_usb3",
> +		.of_match_table	= rcar_gen3_phy_usb3_match_table,
> +	},
> +	.probe	= rcar_gen3_phy_usb3_probe,
> +	.remove = rcar_gen3_phy_usb3_remove,
> +};
> +module_platform_driver(rcar_gen3_phy_usb3_driver);
> +
> +MODULE_LICENSE("GPL v2");
> +MODULE_DESCRIPTION("Renesas R-Car Gen3 USB 3.0 PHY");
> +MODULE_AUTHOR("Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>");
> --
> 1.9.1

Re: [PATCH v3] phy: rcar-gen3-usb3: add support for R-Car Gen3 USB 3.0 PHY

[Kishon Vijay Abraham I]

On Monday 29 May 2017 04:12 PM, Yoshihiro Shimoda wrote:
> The USB 3.0 PHY modules of R-Car Gen3 SoCs have:
>  - Spread spectrum clock (ssc).
>  - Using USB 2.0 EXTAL clock instead of USB 3.0 clock.
>  - Enabling VBUS detection for usb3.0 peripheral.
> 
> So, this driver supports these features.
> 
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>

merged after adding COMPILE_TEST.

Thanks
Kishon
> ---
>  This patch is based on the latest linux-phy.git / next branch and
>  "[PATCH v5 3/3] phy: Group vendor specific phy drivers" patch.
> 
>  I discussed with Geert-san about this driver on the ML:
>  https://patchwork.kernel.org/patch/9731759/
> 
>  Changes from v2:
>  - Revise the device tree doc.
>  - Improve initialization method in rcar_gen3_phy_usb3_init().
>  - remove some values from rcar_gen3_phy_usb3_match_table.
> 
>  Changes from v1:
>  - Changes from dev_info to dev_vdbg in rcar_gen3_phy_usb3_init().
> 
>  .../devicetree/bindings/phy/rcar-gen3-phy-usb3.txt |  46 +++++
>  MAINTAINERS                                        |   4 +-
>  drivers/phy/renesas/Kconfig                        |   7 +
>  drivers/phy/renesas/Makefile                       |   1 +
>  drivers/phy/renesas/phy-rcar-gen3-usb3.c           | 226 +++++++++++++++++++++
>  5 files changed, 282 insertions(+), 2 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/phy/rcar-gen3-phy-usb3.txt
>  create mode 100644 drivers/phy/renesas/phy-rcar-gen3-usb3.c
> 
> diff --git a/Documentation/devicetree/bindings/phy/rcar-gen3-phy-usb3.txt b/Documentation/devicetree/bindings/phy/rcar-gen3-phy-usb3.txt
> new file mode 100644
> index 0000000..f94cea4
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/phy/rcar-gen3-phy-usb3.txt
> @@ -0,0 +1,46 @@
> +* Renesas R-Car generation 3 USB 3.0 PHY
> +
> +This file provides information on what the device node for the R-Car generation
> +3 USB 3.0 PHY contains.
> +If you want to enable spread spectrum clock (ssc), you should use USB_EXTAL
> +instead of USB3_CLK. However, if you don't want to these features, you don't
> +need this driver.
> +
> +Required properties:
> +- compatible: "renesas,r8a7795-usb3-phy" if the device is a part of an R8A7795
> +	      SoC.
> +	      "renesas,r8a7796-usb3-phy" if the device is a part of an R8A7796
> +	      SoC.
> +	      "renesas,rcar-gen3-usb3-phy" for a generic R-Car Gen3 compatible
> +	      device.
> +
> +	      When compatible with the generic version, nodes must list the
> +	      SoC-specific version corresponding to the platform first
> +	      followed by the generic version.
> +
> +- reg: offset and length of the USB 3.0 PHY register block.
> +- clocks: A list of phandles and clock-specifier pairs.
> +- clock-names: Name of the clocks.
> +  - The funcional clock must be "usb3-if".
> +  - The usb3's external clock must be "usb3s_clk".
> +  - The usb2's external clock must be "usb_extal". If you want to use the ssc,
> +    the clock-frequency must not be 0.
> +- #phy-cells: see phy-bindings.txt in the same directory, must be <0>.
> +
> +Optional properties:
> +- renesas,ssc-range: Enable/disable spread spectrum clock (ssc) by using
> +		     the following values as u32:
> +			- 0 (or the property doesn't exist): disable the ssc
> +			- 4980: enable the ssc as -4980 ppm
> +			- 4492: enable the ssc as -4492 ppm
> +			- 4003: enable the ssc as -4003 ppm
> +
> +Example (R-Car H3):
> +
> +	usb-phy@e65ee000 {
> +		compatible = "renesas,r8a7795-usb3-phy",
> +			     "renesas,rcar-gen3-usb3-phy";
> +		reg = <0 0xe65ee000 0 0x90>;
> +		clocks = <&cpg CPG_MOD 328>, <&usb3s0_clk>, <&usb_extal>;
> +		clock-names = "usb3-if", "usb3s_clk", "usb_extal";
> +	};
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a47d3da..d711d53 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -10829,11 +10829,11 @@ L:	linux-iio@vger.kernel.org
>  S:	Supported
>  F:	drivers/iio/adc/rcar_gyro_adc.c
>  
> -RENESAS USB2 PHY DRIVER
> +RENESAS USB PHY DRIVER
>  M:	Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
>  L:	linux-renesas-soc@vger.kernel.org
>  S:	Maintained
> -F:	drivers/phy/renesas/phy-rcar-gen3-usb2.c
> +F:	drivers/phy/renesas/phy-rcar-gen3-usb*.c
>  
>  RESET CONTROLLER FRAMEWORK
>  M:	Philipp Zabel <p.zabel@pengutronix.de>
> diff --git a/drivers/phy/renesas/Kconfig b/drivers/phy/renesas/Kconfig
> index 432e271..4377064 100644
> --- a/drivers/phy/renesas/Kconfig
> +++ b/drivers/phy/renesas/Kconfig
> @@ -15,3 +15,10 @@ config PHY_RCAR_GEN3_USB2
>  	select GENERIC_PHY
>  	help
>  	  Support for USB 2.0 PHY found on Renesas R-Car generation 3 SoCs.
> +
> +config PHY_RCAR_GEN3_USB3
> +	tristate "Renesas R-Car generation 3 USB 3.0 PHY driver"
> +	depends on ARCH_RENESAS
> +	select GENERIC_PHY
> +	help
> +	  Support for USB 3.0 PHY found on Renesas R-Car generation 3 SoCs.
> diff --git a/drivers/phy/renesas/Makefile b/drivers/phy/renesas/Makefile
> index 695241a..8b60259 100644
> --- a/drivers/phy/renesas/Makefile
> +++ b/drivers/phy/renesas/Makefile
> @@ -1,2 +1,3 @@
>  obj-$(CONFIG_PHY_RCAR_GEN2)		+= phy-rcar-gen2.o
>  obj-$(CONFIG_PHY_RCAR_GEN3_USB2)	+= phy-rcar-gen3-usb2.o
> +obj-$(CONFIG_PHY_RCAR_GEN3_USB3)	+= phy-rcar-gen3-usb3.o
> diff --git a/drivers/phy/renesas/phy-rcar-gen3-usb3.c b/drivers/phy/renesas/phy-rcar-gen3-usb3.c
> new file mode 100644
> index 0000000..88c83c9
> --- /dev/null
> +++ b/drivers/phy/renesas/phy-rcar-gen3-usb3.c
> @@ -0,0 +1,226 @@
> +/*
> + * Renesas R-Car Gen3 for USB3.0 PHY driver
> + *
> + * Copyright (C) 2017 Renesas Electronics Corporation
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#include <linux/clk.h>
> +#include <linux/delay.h>
> +#include <linux/io.h>
> +#include <linux/module.h>
> +#include <linux/of.h>
> +#include <linux/phy/phy.h>
> +#include <linux/platform_device.h>
> +#include <linux/pm_runtime.h>
> +
> +#define USB30_CLKSET0		0x034
> +#define USB30_CLKSET1		0x036
> +#define USB30_SSC_SET		0x038
> +#define USB30_PHY_ENABLE	0x060
> +#define USB30_VBUS_EN		0x064
> +
> +/* USB30_CLKSET0 */
> +#define CLKSET0_PRIVATE			0x05c0
> +#define CLKSET0_USB30_FSEL_USB_EXTAL	0x0002
> +
> +/* USB30_CLKSET1 */
> +#define CLKSET1_USB30_PLL_MULTI_SHIFT		6
> +#define CLKSET1_USB30_PLL_MULTI_USB_EXTAL	(0x64 << \
> +						 CLKSET1_USB30_PLL_MULTI_SHIFT)
> +#define CLKSET1_PHYRESET	BIT(4)	/* 1: reset */
> +#define CLKSET1_REF_CLKDIV	BIT(3)	/* 1: USB_EXTAL */
> +#define CLKSET1_PRIVATE_2_1	BIT(1)	/* Write B'01 */
> +#define CLKSET1_REF_CLK_SEL	BIT(0)	/* 1: USB3S0_CLK_P */
> +
> +/* USB30_SSC_SET */
> +#define SSC_SET_SSC_EN		BIT(12)
> +#define SSC_SET_RANGE_SHIFT	9
> +#define SSC_SET_RANGE_4980	(0x0 << SSC_SET_RANGE_SHIFT)
> +#define SSC_SET_RANGE_4492	(0x1 << SSC_SET_RANGE_SHIFT)
> +#define SSC_SET_RANGE_4003	(0x2 << SSC_SET_RANGE_SHIFT)
> +
> +/* USB30_PHY_ENABLE */
> +#define PHY_ENABLE_RESET_EN	BIT(4)
> +
> +/* USB30_VBUS_EN */
> +#define VBUS_EN_VBUS_EN		BIT(1)
> +
> +struct rcar_gen3_usb3 {
> +	void __iomem *base;
> +	struct phy *phy;
> +	u32 ssc_range;
> +	bool usb3s_clk;
> +	bool usb_extal;
> +};
> +
> +static void write_clkset1_for_usb_extal(struct rcar_gen3_usb3 *r, bool reset)
> +{
> +	u16 val = CLKSET1_USB30_PLL_MULTI_USB_EXTAL |
> +		  CLKSET1_REF_CLKDIV | CLKSET1_PRIVATE_2_1;
> +
> +	if (reset)
> +		val |= CLKSET1_PHYRESET;
> +
> +	writew(val, r->base + USB30_CLKSET1);
> +}
> +
> +static void rcar_gen3_phy_usb3_enable_ssc(struct rcar_gen3_usb3 *r)
> +{
> +	u16 val = SSC_SET_SSC_EN;
> +
> +	switch (r->ssc_range) {
> +	case 4980:
> +		val |= SSC_SET_RANGE_4980;
> +		break;
> +	case 4492:
> +		val |= SSC_SET_RANGE_4492;
> +		break;
> +	case 4003:
> +		val |= SSC_SET_RANGE_4003;
> +		break;
> +	default:
> +		dev_err(&r->phy->dev, "%s: unsupported range (%x)\n", __func__,
> +			r->ssc_range);
> +		return;
> +	}
> +
> +	writew(val, r->base + USB30_SSC_SET);
> +}
> +
> +static void rcar_gen3_phy_usb3_select_usb_extal(struct rcar_gen3_usb3 *r)
> +{
> +	write_clkset1_for_usb_extal(r, false);
> +	if (r->ssc_range)
> +		rcar_gen3_phy_usb3_enable_ssc(r);
> +	writew(CLKSET0_PRIVATE | CLKSET0_USB30_FSEL_USB_EXTAL,
> +	       r->base + USB30_CLKSET0);
> +	writew(PHY_ENABLE_RESET_EN, r->base + USB30_PHY_ENABLE);
> +	write_clkset1_for_usb_extal(r, true);
> +	usleep_range(10, 20);
> +	write_clkset1_for_usb_extal(r, false);
> +}
> +
> +static int rcar_gen3_phy_usb3_init(struct phy *p)
> +{
> +	struct rcar_gen3_usb3 *r = phy_get_drvdata(p);
> +
> +	dev_vdbg(&r->phy->dev, "%s: enter (%d, %d, %d)\n", __func__,
> +		 r->usb3s_clk, r->usb_extal, r->ssc_range);
> +
> +	if (!r->usb3s_clk && r->usb_extal)
> +		rcar_gen3_phy_usb3_select_usb_extal(r);
> +
> +	/* Enables VBUS detection anyway */
> +	writew(VBUS_EN_VBUS_EN, r->base + USB30_VBUS_EN);
> +
> +	return 0;
> +}
> +
> +static const struct phy_ops rcar_gen3_phy_usb3_ops = {
> +	.init		= rcar_gen3_phy_usb3_init,
> +	.owner		= THIS_MODULE,
> +};
> +
> +static const struct of_device_id rcar_gen3_phy_usb3_match_table[] = {
> +	{ .compatible = "renesas,rcar-gen3-usb3-phy" },
> +	{ }
> +};
> +MODULE_DEVICE_TABLE(of, rcar_gen3_phy_usb3_match_table);
> +
> +static int rcar_gen3_phy_usb3_probe(struct platform_device *pdev)
> +{
> +	struct device *dev = &pdev->dev;
> +	struct rcar_gen3_usb3 *r;
> +	struct phy_provider *provider;
> +	struct resource *res;
> +	int ret = 0;
> +	struct clk *clk;
> +
> +	if (!dev->of_node) {
> +		dev_err(dev, "This driver needs device tree\n");
> +		return -EINVAL;
> +	}
> +
> +	r = devm_kzalloc(dev, sizeof(*r), GFP_KERNEL);
> +	if (!r)
> +		return -ENOMEM;
> +
> +	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> +	r->base = devm_ioremap_resource(dev, res);
> +	if (IS_ERR(r->base))
> +		return PTR_ERR(r->base);
> +
> +	clk = devm_clk_get(dev, "usb3s_clk");
> +	if (!IS_ERR(clk) && !clk_prepare_enable(clk)) {
> +		r->usb3s_clk = !!clk_get_rate(clk);
> +		clk_disable_unprepare(clk);
> +	}
> +	clk = devm_clk_get(dev, "usb_extal");
> +	if (!IS_ERR(clk) && !clk_prepare_enable(clk)) {
> +		r->usb_extal = !!clk_get_rate(clk);
> +		clk_disable_unprepare(clk);
> +	}
> +
> +	if (!r->usb3s_clk && !r->usb_extal) {
> +		dev_err(dev, "This driver needs usb3s_clk and/or usb_extal\n");
> +		ret = -EINVAL;
> +		goto error;
> +	}
> +
> +	/*
> +	 * devm_phy_create() will call pm_runtime_enable(&phy->dev);
> +	 * And then, phy-core will manage runtime pm for this device.
> +	 */
> +	pm_runtime_enable(dev);
> +
> +	r->phy = devm_phy_create(dev, NULL, &rcar_gen3_phy_usb3_ops);
> +	if (IS_ERR(r->phy)) {
> +		dev_err(dev, "Failed to create USB3 PHY\n");
> +		ret = PTR_ERR(r->phy);
> +		goto error;
> +	}
> +
> +	of_property_read_u32(dev->of_node, "renesas,ssc-range", &r->ssc_range);
> +
> +	platform_set_drvdata(pdev, r);
> +	phy_set_drvdata(r->phy, r);
> +
> +	provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate);
> +	if (IS_ERR(provider)) {
> +		dev_err(dev, "Failed to register PHY provider\n");
> +		ret = PTR_ERR(provider);
> +		goto error;
> +	}
> +
> +	return 0;
> +
> +error:
> +	pm_runtime_disable(dev);
> +
> +	return ret;
> +}
> +
> +static int rcar_gen3_phy_usb3_remove(struct platform_device *pdev)
> +{
> +	pm_runtime_disable(&pdev->dev);
> +
> +	return 0;
> +};
> +
> +static struct platform_driver rcar_gen3_phy_usb3_driver = {
> +	.driver = {
> +		.name		= "phy_rcar_gen3_usb3",
> +		.of_match_table	= rcar_gen3_phy_usb3_match_table,
> +	},
> +	.probe	= rcar_gen3_phy_usb3_probe,
> +	.remove = rcar_gen3_phy_usb3_remove,
> +};
> +module_platform_driver(rcar_gen3_phy_usb3_driver);
> +
> +MODULE_LICENSE("GPL v2");
> +MODULE_DESCRIPTION("Renesas R-Car Gen3 USB 3.0 PHY");
> +MODULE_AUTHOR("Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>");
>