From: Jagan Teki <jagan@amarulasolutions.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v6 18/20] musb-new: sunxi: Use CLK and RESET support
Date: Fri, 11 Jan 2019 00:10:14 +0530 [thread overview]
Message-ID: <20190110184016.17027-19-jagan@amarulasolutions.com> (raw)
In-Reply-To: <20190110184016.17027-1-jagan@amarulasolutions.com>
Now clock and reset drivers are available for respective
SoC's so use clk and reset ops on musb driver.
Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
Acked-by: Maxime Ripard <maxime.ripard@bootlin.com>
Reviewed-by: Marek Vasut <marex@denx.de>
---
drivers/usb/musb-new/sunxi.c | 79 ++++++++++++++++++------------------
1 file changed, 39 insertions(+), 40 deletions(-)
diff --git a/drivers/usb/musb-new/sunxi.c b/drivers/usb/musb-new/sunxi.c
index f3deb9bc66..906c08ece9 100644
--- a/drivers/usb/musb-new/sunxi.c
+++ b/drivers/usb/musb-new/sunxi.c
@@ -16,9 +16,11 @@
* This file is part of the Inventra Controller Driver for Linux.
*/
#include <common.h>
+#include <clk.h>
#include <dm.h>
#include <generic-phy.h>
#include <phy-sun4i-usb.h>
+#include <reset.h>
#include <asm/arch/cpu.h>
#include <asm/arch/clock.h>
#include <asm/arch/gpio.h>
@@ -80,16 +82,12 @@
struct sunxi_musb_config {
struct musb_hdrc_config *config;
- bool has_reset;
- u8 rst_bit;
- u8 clkgate_bit;
- u32 off_reset0;
};
struct sunxi_glue {
struct musb_host_data mdata;
- struct sunxi_ccm_reg *ccm;
- u32 *reg_reset0;
+ struct clk clk;
+ struct reset_ctl rst;
struct sunxi_musb_config *cfg;
struct device dev;
struct phy phy;
@@ -296,24 +294,27 @@ static int sunxi_musb_init(struct musb *musb)
pr_debug("%s():\n", __func__);
- ret = generic_phy_init(&glue->phy);
+ ret = clk_enable(&glue->clk);
if (ret) {
- pr_err("failed to init USB PHY\n");
+ dev_err(dev, "failed to enable clock\n");
return ret;
}
- musb->isr = sunxi_musb_interrupt;
-
- setbits_le32(&glue->ccm->ahb_gate0, BIT(AHB_GATE_OFFSET_USB0));
- if (glue->cfg->clkgate_bit)
- setbits_le32(&glue->ccm->ahb_gate0,
- BIT(glue->cfg->clkgate_bit));
+ if (reset_valid(&glue->rst)) {
+ ret = reset_deassert(&glue->rst);
+ if (ret) {
+ dev_err(dev, "failed to deassert reset\n");
+ goto err_clk;
+ }
+ }
- if (glue->cfg->has_reset)
- setbits_le32(glue->reg_reset0, BIT(AHB_GATE_OFFSET_USB0));
+ ret = generic_phy_init(&glue->phy);
+ if (ret) {
+ dev_err(dev, "failed to init USB PHY\n");
+ goto err_rst;
+ }
- if (glue->cfg->rst_bit)
- setbits_le32(glue->reg_reset0, BIT(glue->cfg->rst_bit));
+ musb->isr = sunxi_musb_interrupt;
USBC_ConfigFIFO_Base();
USBC_EnableDpDmPullUp(musb->mregs);
@@ -329,6 +330,13 @@ static int sunxi_musb_init(struct musb *musb)
USBC_ForceVbusValidToHigh(musb->mregs);
return 0;
+
+err_rst:
+ if (reset_valid(&glue->rst))
+ reset_assert(&glue->rst);
+err_clk:
+ clk_disable(&glue->clk);
+ return ret;
}
static int sunxi_musb_exit(struct musb *musb)
@@ -344,16 +352,9 @@ static int sunxi_musb_exit(struct musb *musb)
}
}
- if (glue->cfg->has_reset)
- clrbits_le32(glue->reg_reset0, BIT(AHB_GATE_OFFSET_USB0));
-
- if (glue->cfg->rst_bit)
- clrbits_le32(glue->reg_reset0, BIT(glue->cfg->rst_bit));
-
- clrbits_le32(&glue->ccm->ahb_gate0, BIT(AHB_GATE_OFFSET_USB0));
- if (glue->cfg->clkgate_bit)
- clrbits_le32(&glue->ccm->ahb_gate0,
- BIT(glue->cfg->clkgate_bit));
+ if (reset_valid(&glue->rst))
+ reset_assert(&glue->rst);
+ clk_disable(&glue->clk);
return 0;
}
@@ -450,11 +451,17 @@ static int musb_usb_probe(struct udevice *dev)
if (!glue->cfg)
return -EINVAL;
- glue->ccm = (struct sunxi_ccm_reg *)SUNXI_CCM_BASE;
- if (IS_ERR(glue->ccm))
- return PTR_ERR(glue->ccm);
+ ret = clk_get_by_index(dev, 0, &glue->clk);
+ if (ret) {
+ dev_err(dev, "failed to get clock\n");
+ return ret;
+ }
- glue->reg_reset0 = (void *)glue->ccm + glue->cfg->off_reset0;
+ ret = reset_get_by_index(dev, 0, &glue->rst);
+ if (ret && ret != -ENOENT) {
+ dev_err(dev, "failed to get reset\n");
+ return ret;
+ }
ret = generic_phy_get_by_name(dev, "usb", &glue->phy);
if (ret) {
@@ -462,7 +469,6 @@ static int musb_usb_probe(struct udevice *dev)
return ret;
}
-
memset(&pdata, 0, sizeof(pdata));
pdata.power = 250;
pdata.platform_ops = &sunxi_musb_ops;
@@ -505,21 +511,14 @@ static int musb_usb_remove(struct udevice *dev)
static const struct sunxi_musb_config sun4i_a10_cfg = {
.config = &musb_config,
- .has_reset = false,
};
static const struct sunxi_musb_config sun6i_a31_cfg = {
.config = &musb_config,
- .has_reset = true,
- .off_reset0 = OFF_SUN6I_AHB_RESET0,
};
static const struct sunxi_musb_config sun8i_h3_cfg = {
.config = &musb_config_h3,
- .has_reset = true,
- .rst_bit = 23,
- .clkgate_bit = 23,
- .off_reset0 = OFF_SUN6I_AHB_RESET0,
};
static const struct udevice_id sunxi_musb_ids[] = {
--
2.18.0.321.gffc6fa0e3
next prev parent reply other threads:[~2019-01-10 18:40 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-10 18:39 [U-Boot] [PATCH v6 00/20] clk: Add Allwinner CLK, RESET support Jagan Teki
2019-01-10 18:39 ` [U-Boot] [PATCH v6 01/20] clk: Add Allwinner A64 CLK driver Jagan Teki
2019-01-10 18:39 ` [U-Boot] [PATCH v6 02/20] reset: Add Allwinner RESET driver Jagan Teki
2019-01-10 18:39 ` [U-Boot] [PATCH v6 03/20] clk: sunxi: Add Allwinner H3/H5 CLK driver Jagan Teki
2019-01-10 18:40 ` [U-Boot] [PATCH v6 04/20] clk: sunxi: Add Allwinner A10/A20 " Jagan Teki
2019-01-10 18:40 ` [U-Boot] [PATCH v6 05/20] clk: sunxi: Add Allwinner A10s/A13 " Jagan Teki
2019-01-10 18:40 ` [U-Boot] [PATCH v6 06/20] clk: sunxi: Add Allwinner A31 " Jagan Teki
2019-01-10 18:40 ` [U-Boot] [PATCH v6 07/20] clk: sunxi: Add Allwinner A23/A33 " Jagan Teki
2019-01-10 18:40 ` [U-Boot] [PATCH v6 08/20] clk: sunxi: Add Allwinner A83T " Jagan Teki
2019-01-10 18:40 ` [U-Boot] [PATCH v6 09/20] clk: sunxi: Add Allwinner R40 " Jagan Teki
2019-01-10 18:40 ` [U-Boot] [PATCH v6 10/20] clk: sunxi: Add Allwinner V3S " Jagan Teki
2019-01-10 18:40 ` [U-Boot] [PATCH v6 11/20] clk: sunxi: Implement UART clocks Jagan Teki
2019-01-10 22:52 ` André Przywara
2019-01-10 18:40 ` [U-Boot] [PATCH v6 12/20] clk: sunxi: Implement UART resets Jagan Teki
2019-01-10 22:50 ` André Przywara
2019-01-10 18:40 ` [U-Boot] [PATCH v6 13/20] clk: sunxi: Add Allwinner H6 CLK driver Jagan Teki
2019-01-10 23:55 ` André Przywara
2019-01-10 18:40 ` [U-Boot] [PATCH v6 14/20] sunxi: A64: Update sun50i-a64-ccu.h Jagan Teki
2019-01-10 23:58 ` André Przywara
2019-01-10 18:40 ` [U-Boot] [PATCH v6 15/20] sunxi: Enable CLK Jagan Teki
2019-01-11 0:09 ` André Przywara
2019-01-10 18:40 ` [U-Boot] [PATCH v6 16/20] phy: sun4i-usb: Use CLK and RESET support Jagan Teki
2019-01-10 18:40 ` [U-Boot] [PATCH v6 17/20] reset: Add reset valid Jagan Teki
2019-01-10 18:40 ` Jagan Teki [this message]
2019-01-10 18:40 ` [U-Boot] [PATCH v6 19/20] sunxi: usb: Switch to Generic host controllers Jagan Teki
2019-01-10 18:40 ` [U-Boot] [PATCH v6 20/20] usb: host: Drop [e-o]hci-sunxi drivers Jagan Teki
2019-01-18 16:52 ` [U-Boot] [PATCH v6 00/20] clk: Add Allwinner CLK, RESET support Jagan Teki
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=20190110184016.17027-19-jagan@amarulasolutions.com \
--to=jagan@amarulasolutions.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
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