From: kishon@ti•com (Kishon Vijay Abraham I)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH RESEND v4 4/8] phy: usb: sunxi: Introduce Allwinner A31 USB PHY support
Date: Wed, 14 May 2014 14:09:37 +0530 [thread overview]
Message-ID: <53732BC9.3020507@ti.com> (raw)
In-Reply-To: <1399995862-17788-5-git-send-email-maxime.ripard@free-electrons.com>
Hi,
On Tuesday 13 May 2014 09:14 PM, Maxime Ripard wrote:
> The USB phy controller in the A31 differs mostly from the older controllers
> because it has a clock dedicated for each phy, while the older ones were having
> a single clock for all the phys.
>
> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons•com>
> Reviewed-by: Hans de Goede <hdegoede@redhat•com>
> ---
> drivers/phy/phy-sun4i-usb.c | 33 ++++++++++++++++++++++-----------
> 1 file changed, 22 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/phy/phy-sun4i-usb.c b/drivers/phy/phy-sun4i-usb.c
> index 66a87d50512b..115d8d5190d5 100644
> --- a/drivers/phy/phy-sun4i-usb.c
> +++ b/drivers/phy/phy-sun4i-usb.c
> @@ -61,7 +61,6 @@
> #define MAX_PHYS 3
>
> struct sun4i_usb_phy_data {
> - struct clk *clk;
> void __iomem *base;
> struct mutex mutex;
> int num_phys;
> @@ -71,6 +70,7 @@ struct sun4i_usb_phy_data {
> void __iomem *pmu;
> struct regulator *vbus;
> struct reset_control *reset;
> + struct clk *clk;
> int index;
> } phys[MAX_PHYS];
> };
> @@ -146,13 +146,13 @@ static int sun4i_usb_phy_init(struct phy *_phy)
> struct sun4i_usb_phy_data *data = to_sun4i_usb_phy_data(phy);
> int ret;
>
> - ret = clk_prepare_enable(data->clk);
> + ret = clk_prepare_enable(phy->clk);
> if (ret)
> return ret;
>
> ret = reset_control_deassert(phy->reset);
> if (ret) {
> - clk_disable_unprepare(data->clk);
> + clk_disable_unprepare(phy->clk);
> return ret;
> }
>
> @@ -170,11 +170,10 @@ static int sun4i_usb_phy_init(struct phy *_phy)
> static int sun4i_usb_phy_exit(struct phy *_phy)
> {
> struct sun4i_usb_phy *phy = phy_get_drvdata(_phy);
> - struct sun4i_usb_phy_data *data = to_sun4i_usb_phy_data(phy);
>
> sun4i_usb_phy_passby(phy, 0);
> reset_control_assert(phy->reset);
> - clk_disable_unprepare(data->clk);
> + clk_disable_unprepare(phy->clk);
>
> return 0;
> }
> @@ -225,6 +224,7 @@ static int sun4i_usb_phy_probe(struct platform_device *pdev)
> struct device *dev = &pdev->dev;
> struct device_node *np = dev->of_node;
> struct phy_provider *phy_provider;
> + bool dedicated_clocks;
> struct resource *res;
> int i;
>
> @@ -244,17 +244,16 @@ static int sun4i_usb_phy_probe(struct platform_device *pdev)
> else
> data->disc_thresh = 2;
>
> + if (of_device_is_compatible(np, "allwinner,sun6i-a31-usb-phy"))
> + dedicated_clocks = true;
> + else
> + dedicated_clocks = false;
> +
> res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "phy_ctrl");
> data->base = devm_ioremap_resource(dev, res);
> if (IS_ERR(data->base))
> return PTR_ERR(data->base);
>
> - data->clk = devm_clk_get(dev, "usb_phy");
> - if (IS_ERR(data->clk)) {
> - dev_err(dev, "could not get usb_phy clock\n");
> - return PTR_ERR(data->clk);
> - }
> -
> /* Skip 0, 0 is the phy for otg which is not yet supported. */
> for (i = 1; i < data->num_phys; i++) {
> struct sun4i_usb_phy *phy = data->phys + i;
> @@ -268,6 +267,17 @@ static int sun4i_usb_phy_probe(struct platform_device *pdev)
> phy->vbus = NULL;
> }
>
> + if (dedicated_clocks)
> + snprintf(name, sizeof(name), "usb%d_phy", i);
> + else
> + strlcpy(name, "usb_phy", sizeof(name));
> +
> + phy->clk = devm_clk_get(dev, name);
> + if (IS_ERR(phy->clk)) {
> + dev_err(dev, "failed to get clock %s\n", name);
> + return PTR_ERR(phy->clk);
> + }
> +
> snprintf(name, sizeof(name), "usb%d_reset", i);
> phy->reset = devm_reset_control_get(dev, name);
> if (IS_ERR(phy->reset)) {
> @@ -305,6 +315,7 @@ static int sun4i_usb_phy_probe(struct platform_device *pdev)
> static const struct of_device_id sun4i_usb_phy_of_match[] = {
> { .compatible = "allwinner,sun4i-a10-usb-phy" },
> { .compatible = "allwinner,sun5i-a13-usb-phy" },
> + { .compatible = "allwinner,sun6i-a31-usb-phy" },
Do you have Documentation for this comptible binding? Would be good to mention
that in the commit log.
Thanks
Kishon
next prev parent reply other threads:[~2014-05-14 8:39 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-13 15:44 [PATCH RESEND v4 0/8] Add Allwinner A31 USB support Maxime Ripard
2014-05-13 15:44 ` [PATCH RESEND v4 1/8] clk: sunxi: Implement A31 USB clock Maxime Ripard
2014-05-13 16:40 ` Emilio López
2014-05-13 15:44 ` [PATCH RESEND v4 2/8] ARM: sun6i: Add the USB clocks to the DTSI Maxime Ripard
2014-05-13 15:44 ` [PATCH RESEND v4 3/8] phy: sunxi: Rework phy initialization Maxime Ripard
2014-05-14 5:48 ` Kishon Vijay Abraham I
2014-05-14 6:49 ` Maxime Ripard
2014-05-15 9:44 ` Kishon Vijay Abraham I
2014-05-13 15:44 ` [PATCH RESEND v4 4/8] phy: usb: sunxi: Introduce Allwinner A31 USB PHY support Maxime Ripard
2014-05-14 8:39 ` Kishon Vijay Abraham I [this message]
2014-05-14 12:30 ` Maxime Ripard
2014-05-13 15:44 ` [PATCH RESEND v4 5/8] usb: ehci-platform: add optional reset controller retrieval Maxime Ripard
2014-05-13 15:44 ` [PATCH RESEND v4 6/8] usb: ohci-platform: Enable optional use of reset controller Maxime Ripard
2014-05-13 15:44 ` [PATCH RESEND v4 7/8] ARM: sun6i: dt: Add support for the USB controllers Maxime Ripard
2014-05-13 15:44 ` [PATCH RESEND v4 8/8] ARM: sunxi: dt: add APP4-EVB1 board support Maxime Ripard
2014-05-14 12:34 ` [PATCH RESEND v4 0/8] Add Allwinner A31 USB support Maxime Ripard
2014-05-14 16:05 ` Greg Kroah-Hartman
2014-05-15 9:14 ` Maxime Ripard
2014-05-27 22:53 ` Greg Kroah-Hartman
2014-05-28 7:47 ` Maxime Ripard
2014-05-23 18:33 ` Maxime Ripard
2014-05-23 22:19 ` Greg Kroah-Hartman
2014-05-24 15:09 ` Maxime Ripard
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=53732BC9.3020507@ti.com \
--to=kishon@ti$(echo .)com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox