| Message ID | 20230619094759.21013-4-minda.chen@starfivetech.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Add JH7110 USB PHY driver support | expand |
| Context | Check | Description |
|---|---|---|
| conchuod/cover_letter | success | Series has a cover letter |
| conchuod/tree_selection | success | Guessed tree name to be for-next at HEAD d5e45e810e0e |
| conchuod/fixes_present | success | Fixes tag not required for -next series |
| conchuod/maintainers_pattern | success | MAINTAINERS pattern errors before the patch: 6 and now 6 |
| conchuod/verify_signedoff | success | Signed-off-by tag matches author and committer |
| conchuod/kdoc | success | Errors and warnings before: 0 this patch: 0 |
| conchuod/build_rv64_clang_allmodconfig | success | Errors and warnings before: 8 this patch: 8 |
| conchuod/module_param | success | Was 0 now: 0 |
| conchuod/build_rv64_gcc_allmodconfig | success | Errors and warnings before: 8 this patch: 8 |
| conchuod/build_rv32_defconfig | success | Build OK |
| conchuod/dtb_warn_rv64 | success | Errors and warnings before: 20 this patch: 20 |
| conchuod/header_inline | success | No static functions without inline keyword in header files |
| conchuod/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 193 lines checked |
| conchuod/build_rv64_nommu_k210_defconfig | success | Build OK |
| conchuod/verify_fixes | success | No Fixes tag |
| conchuod/build_rv64_nommu_virt_defconfig | success | Build OK |
On 19-06-23, 17:47, Minda Chen wrote: > Add Starfive JH7110 SoC USB 2.0 PHY driver support. > USB 2.0 PHY default connect to Cadence USB controller. > > Signed-off-by: Minda Chen <minda.chen@starfivetech.com> > Reviewed-by: Roger Quadros <rogerq@kernel.org> > --- > MAINTAINERS | 6 ++ > drivers/phy/Kconfig | 1 + > drivers/phy/Makefile | 1 + > drivers/phy/starfive/Kconfig | 15 +++ > drivers/phy/starfive/Makefile | 2 + > drivers/phy/starfive/phy-jh7110-usb.c | 150 ++++++++++++++++++++++++++ > 6 files changed, 175 insertions(+) > create mode 100644 drivers/phy/starfive/Kconfig > create mode 100644 drivers/phy/starfive/Makefile > create mode 100644 drivers/phy/starfive/phy-jh7110-usb.c > > diff --git a/MAINTAINERS b/MAINTAINERS > index f794002a192e..d2ce89a8d31c 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -20174,6 +20174,12 @@ S: Supported > F: Documentation/devicetree/bindings/watchdog/starfive* > F: drivers/watchdog/starfive-wdt.c > > +STARFIVE JH71X0 USB PHY DRIVER > +M: Minda Chen <minda.chen@starfivetech.com> > +S: Supported > +F: Documentation/devicetree/bindings/phy/starfive,jh7110-usb-phy.yaml > +F: drivers/phy/starfive/phy-jh7110-usb.c > + > STATIC BRANCH/CALL > M: Peter Zijlstra <peterz@infradead.org> > M: Josh Poimboeuf <jpoimboe@kernel.org> > diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig > index f46e3148d286..0000149edbc4 100644 > --- a/drivers/phy/Kconfig > +++ b/drivers/phy/Kconfig > @@ -91,6 +91,7 @@ source "drivers/phy/rockchip/Kconfig" > source "drivers/phy/samsung/Kconfig" > source "drivers/phy/socionext/Kconfig" > source "drivers/phy/st/Kconfig" > +source "drivers/phy/starfive/Kconfig" > source "drivers/phy/sunplus/Kconfig" > source "drivers/phy/tegra/Kconfig" > source "drivers/phy/ti/Kconfig" > diff --git a/drivers/phy/Makefile b/drivers/phy/Makefile > index 54f312c10a40..fb3dc9de6111 100644 > --- a/drivers/phy/Makefile > +++ b/drivers/phy/Makefile > @@ -31,6 +31,7 @@ obj-y += allwinner/ \ > samsung/ \ > socionext/ \ > st/ \ > + starfive/ \ > sunplus/ \ > tegra/ \ > ti/ \ > diff --git a/drivers/phy/starfive/Kconfig b/drivers/phy/starfive/Kconfig > new file mode 100644 > index 000000000000..2283feadfc76 > --- /dev/null > +++ b/drivers/phy/starfive/Kconfig > @@ -0,0 +1,15 @@ > +# SPDX-License-Identifier: GPL-2.0-only > +# > +# Phy drivers for StarFive platforms > +# > + > +config PHY_STARFIVE_JH7110_USB > + tristate "Starfive JH7110 USB 2.0 PHY support" > + depends on USB_SUPPORT > + select GENERIC_PHY > + select USB_PHY > + help > + Enable this to support the StarFive USB 2.0 PHY, > + used with the Cadence USB controller. > + If M is selected, the module will be called > + phy-jh7110-usb.ko. > diff --git a/drivers/phy/starfive/Makefile b/drivers/phy/starfive/Makefile > new file mode 100644 > index 000000000000..52e9a09cc619 > --- /dev/null > +++ b/drivers/phy/starfive/Makefile > @@ -0,0 +1,2 @@ > +# SPDX-License-Identifier: GPL-2.0 > +obj-$(CONFIG_PHY_STARFIVE_JH7110_USB) += phy-jh7110-usb.o > diff --git a/drivers/phy/starfive/phy-jh7110-usb.c b/drivers/phy/starfive/phy-jh7110-usb.c > new file mode 100644 > index 000000000000..90d788423705 > --- /dev/null > +++ b/drivers/phy/starfive/phy-jh7110-usb.c > @@ -0,0 +1,150 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * StarFive JH7110 USB 2.0 PHY driver > + * > + * Copyright (C) 2023 StarFive Technology Co., Ltd. > + * Author: Minda Chen <minda.chen@starfivetech.com> > + */ > + > +#include <linux/bits.h> > +#include <linux/clk.h> > +#include <linux/err.h> > +#include <linux/io.h> > +#include <linux/module.h> > +#include <linux/phy/phy.h> > +#include <linux/platform_device.h> > +#include <linux/usb/of.h> > + > +#define USB_125M_CLK_RATE 125000000 > +#define USB_LS_KEEPALIVE_OFF 0x4 > +#define USB_LS_KEEPALIVE_ENABLE BIT(4) > + > +struct jh7110_usb2_phy { > + struct phy *phy; > + void __iomem *regs; > + struct clk *usb_125m_clk; > + struct clk *app_125m; > + enum phy_mode mode; > +}; > + > +static void jh7110_usb2_mode_set(struct jh7110_usb2_phy *phy) > +{ > + unsigned int val; > + > + if (phy->mode != PHY_MODE_USB_HOST) { > + /* Enable the LS speed keep-alive signal */ > + val = readl(phy->regs + USB_LS_KEEPALIVE_OFF); > + val |= USB_LS_KEEPALIVE_ENABLE; > + writel(val, phy->regs + USB_LS_KEEPALIVE_OFF); > + } looks like this sets only for host, so why not call it jh7110_usb2_set_host_mode() rather than get confused about jh7110_usb2_mode_set/jh7110_usb2_phy_set_mode > +} > + > +static int jh7110_usb2_phy_set_mode(struct phy *_phy, > + enum phy_mode mode, int submode) > +{ > + struct jh7110_usb2_phy *phy = phy_get_drvdata(_phy); > + > + switch (mode) { > + case PHY_MODE_USB_HOST: > + case PHY_MODE_USB_DEVICE: > + case PHY_MODE_USB_OTG: > + break; > + default: > + return -EINVAL; > + } > + > + if (mode != phy->mode) { > + dev_dbg(&_phy->dev, "Changing phy to %d\n", mode); > + phy->mode = mode; > + jh7110_usb2_mode_set(phy); > + } > + > + return 0; > +} > + > +static int jh7110_usb2_phy_init(struct phy *_phy) > +{ > + struct jh7110_usb2_phy *phy = phy_get_drvdata(_phy); > + int ret; > + > + ret = clk_set_rate(phy->usb_125m_clk, USB_125M_CLK_RATE); > + if (ret) > + return ret; > + > + ret = clk_prepare_enable(phy->app_125m); > + if (ret) > + return ret; > + > + return 0; > +} > + > +static int jh7110_usb2_phy_exit(struct phy *_phy) > +{ > + struct jh7110_usb2_phy *phy = phy_get_drvdata(_phy); > + > + clk_disable_unprepare(phy->app_125m); > + > + return 0; > +} > + > +static const struct phy_ops jh7110_usb2_phy_ops = { > + .init = jh7110_usb2_phy_init, > + .exit = jh7110_usb2_phy_exit, > + .set_mode = jh7110_usb2_phy_set_mode, > + .owner = THIS_MODULE, > +}; > + > +static int jh7110_usb_phy_probe(struct platform_device *pdev) > +{ > + struct jh7110_usb2_phy *phy; > + struct device *dev = &pdev->dev; > + struct phy_provider *phy_provider; > + > + phy = devm_kzalloc(dev, sizeof(*phy), GFP_KERNEL); > + if (!phy) > + return -ENOMEM; > + > + phy->usb_125m_clk = devm_clk_get(dev, "125m"); > + if (IS_ERR(phy->usb_125m_clk)) > + return dev_err_probe(dev, PTR_ERR(phy->usb_125m_clk), > + "Failed to get 125m clock\n"); > + > + phy->app_125m = devm_clk_get(dev, "app_125m"); > + if (IS_ERR(phy->app_125m)) > + return dev_err_probe(dev, PTR_ERR(phy->app_125m), > + "Failed to get app 125m clock\n"); > + > + phy->regs = devm_platform_ioremap_resource(pdev, 0); > + if (IS_ERR(phy->regs)) > + return dev_err_probe(dev, PTR_ERR(phy->regs), > + "Failed to map phy base\n"); > + > + phy->phy = devm_phy_create(dev, NULL, &jh7110_usb2_phy_ops); > + if (IS_ERR(phy->phy)) > + return dev_err_probe(dev, PTR_ERR(phy->phy), > + "Failed to create phy\n"); > + > + phy_set_drvdata(phy->phy, phy); > + phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate); > + > + return PTR_ERR_OR_ZERO(phy_provider); > +} > + > +static const struct of_device_id jh7110_usb_phy_of_match[] = { > + { .compatible = "starfive,jh7110-usb-phy" }, > + { /* sentinel */ }, > +}; > +MODULE_DEVICE_TABLE(of, jh7110_usb_phy_of_match); > + > +static struct platform_driver jh7110_usb_phy_driver = { > + .probe = jh7110_usb_phy_probe, > + .driver = { > + .of_match_table = jh7110_usb_phy_of_match, > + .name = "jh7110-usb-phy", > + } > +}; > +module_platform_driver(jh7110_usb_phy_driver); A very light driver, only setting couple of things for hw. Can you explain how phy registers are configured, am sure there would be many more..? Do you rely on bootloader or some other entity for that? > + > +MODULE_DESCRIPTION("StarFive JH7110 USB 2.0 PHY driver"); > +MODULE_AUTHOR("Minda Chen <minda.chen@starfivetech.com>"); > +MODULE_LICENSE("GPL"); > -- > 2.17.1
On 2023/6/21 19:12, Vinod Koul wrote: > On 19-06-23, 17:47, Minda Chen wrote: >> Add Starfive JH7110 SoC USB 2.0 PHY driver support. >> USB 2.0 PHY default connect to Cadence USB controller. >> >> Signed-off-by: Minda Chen <minda.chen@starfivetech.com> >> Reviewed-by: Roger Quadros <rogerq@kernel.org> >> --- >> MAINTAINERS | 6 ++ >> drivers/phy/Kconfig | 1 + >> drivers/phy/Makefile | 1 + >> drivers/phy/starfive/Kconfig | 15 +++ >> drivers/phy/starfive/Makefile | 2 + >> drivers/phy/starfive/phy-jh7110-usb.c | 150 ++++++++++++++++++++++++++ >> 6 files changed, 175 insertions(+) >> create mode 100644 drivers/phy/starfive/Kconfig >> create mode 100644 drivers/phy/starfive/Makefile >> create mode 100644 drivers/phy/starfive/phy-jh7110-usb.c >> >> diff --git a/MAINTAINERS b/MAINTAINERS >> index f794002a192e..d2ce89a8d31c 100644 >> --- a/MAINTAINERS >> +++ b/MAINTAINERS >> @@ -20174,6 +20174,12 @@ S: Supported >> F: Documentation/devicetree/bindings/watchdog/starfive* >> F: drivers/watchdog/starfive-wdt.c >> >> +STARFIVE JH71X0 USB PHY DRIVER >> +M: Minda Chen <minda.chen@starfivetech.com> >> +S: Supported >> +F: Documentation/devicetree/bindings/phy/starfive,jh7110-usb-phy.yaml >> +F: drivers/phy/starfive/phy-jh7110-usb.c >> + >> STATIC BRANCH/CALL >> M: Peter Zijlstra <peterz@infradead.org> >> M: Josh Poimboeuf <jpoimboe@kernel.org> >> diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig >> index f46e3148d286..0000149edbc4 100644 >> --- a/drivers/phy/Kconfig >> +++ b/drivers/phy/Kconfig >> @@ -91,6 +91,7 @@ source "drivers/phy/rockchip/Kconfig" >> source "drivers/phy/samsung/Kconfig" >> source "drivers/phy/socionext/Kconfig" >> source "drivers/phy/st/Kconfig" >> +source "drivers/phy/starfive/Kconfig" >> source "drivers/phy/sunplus/Kconfig" >> source "drivers/phy/tegra/Kconfig" >> source "drivers/phy/ti/Kconfig" >> diff --git a/drivers/phy/Makefile b/drivers/phy/Makefile >> index 54f312c10a40..fb3dc9de6111 100644 >> --- a/drivers/phy/Makefile >> +++ b/drivers/phy/Makefile >> @@ -31,6 +31,7 @@ obj-y += allwinner/ \ >> samsung/ \ >> socionext/ \ >> st/ \ >> + starfive/ \ >> sunplus/ \ >> tegra/ \ >> ti/ \ >> diff --git a/drivers/phy/starfive/Kconfig b/drivers/phy/starfive/Kconfig >> new file mode 100644 >> index 000000000000..2283feadfc76 >> --- /dev/null >> +++ b/drivers/phy/starfive/Kconfig >> @@ -0,0 +1,15 @@ >> +# SPDX-License-Identifier: GPL-2.0-only >> +# >> +# Phy drivers for StarFive platforms >> +# >> + >> +config PHY_STARFIVE_JH7110_USB >> + tristate "Starfive JH7110 USB 2.0 PHY support" >> + depends on USB_SUPPORT >> + select GENERIC_PHY >> + select USB_PHY >> + help >> + Enable this to support the StarFive USB 2.0 PHY, >> + used with the Cadence USB controller. >> + If M is selected, the module will be called >> + phy-jh7110-usb.ko. >> diff --git a/drivers/phy/starfive/Makefile b/drivers/phy/starfive/Makefile >> new file mode 100644 >> index 000000000000..52e9a09cc619 >> --- /dev/null >> +++ b/drivers/phy/starfive/Makefile >> @@ -0,0 +1,2 @@ >> +# SPDX-License-Identifier: GPL-2.0 >> +obj-$(CONFIG_PHY_STARFIVE_JH7110_USB) += phy-jh7110-usb.o >> diff --git a/drivers/phy/starfive/phy-jh7110-usb.c b/drivers/phy/starfive/phy-jh7110-usb.c >> new file mode 100644 >> index 000000000000..90d788423705 >> --- /dev/null >> +++ b/drivers/phy/starfive/phy-jh7110-usb.c >> @@ -0,0 +1,150 @@ >> +// SPDX-License-Identifier: GPL-2.0+ >> +/* >> + * StarFive JH7110 USB 2.0 PHY driver >> + * >> + * Copyright (C) 2023 StarFive Technology Co., Ltd. >> + * Author: Minda Chen <minda.chen@starfivetech.com> >> + */ >> + >> +#include <linux/bits.h> >> +#include <linux/clk.h> >> +#include <linux/err.h> >> +#include <linux/io.h> >> +#include <linux/module.h> >> +#include <linux/phy/phy.h> >> +#include <linux/platform_device.h> >> +#include <linux/usb/of.h> >> + >> +#define USB_125M_CLK_RATE 125000000 >> +#define USB_LS_KEEPALIVE_OFF 0x4 >> +#define USB_LS_KEEPALIVE_ENABLE BIT(4) >> + >> +struct jh7110_usb2_phy { >> + struct phy *phy; >> + void __iomem *regs; >> + struct clk *usb_125m_clk; >> + struct clk *app_125m; >> + enum phy_mode mode; >> +}; >> + >> +static void jh7110_usb2_mode_set(struct jh7110_usb2_phy *phy) >> +{ >> + unsigned int val; >> + >> + if (phy->mode != PHY_MODE_USB_HOST) { >> + /* Enable the LS speed keep-alive signal */ >> + val = readl(phy->regs + USB_LS_KEEPALIVE_OFF); >> + val |= USB_LS_KEEPALIVE_ENABLE; >> + writel(val, phy->regs + USB_LS_KEEPALIVE_OFF); >> + } > > looks like this sets only for host, so why not call it > jh7110_usb2_set_host_mode() rather than get confused about > jh7110_usb2_mode_set/jh7110_usb2_phy_set_mode > Thanks , It is only for host, I will change it. >> +} >> + >> +static int jh7110_usb2_phy_set_mode(struct phy *_phy, >> + enum phy_mode mode, int submode) >> +{ >> + struct jh7110_usb2_phy *phy = phy_get_drvdata(_phy); >> + >> + switch (mode) { >> + case PHY_MODE_USB_HOST: >> + case PHY_MODE_USB_DEVICE: >> + case PHY_MODE_USB_OTG: >> + break; >> + default: >> + return -EINVAL; >> + } >> + >> + if (mode != phy->mode) { >> + dev_dbg(&_phy->dev, "Changing phy to %d\n", mode); >> + phy->mode = mode; >> + jh7110_usb2_mode_set(phy); >> + } >> + >> + return 0; >> +} >> + >> +static int jh7110_usb2_phy_init(struct phy *_phy) >> +{ >> + struct jh7110_usb2_phy *phy = phy_get_drvdata(_phy); >> + int ret; >> + >> + ret = clk_set_rate(phy->usb_125m_clk, USB_125M_CLK_RATE); >> + if (ret) >> + return ret; >> + >> + ret = clk_prepare_enable(phy->app_125m); >> + if (ret) >> + return ret; >> + >> + return 0; >> +} >> + >> +static int jh7110_usb2_phy_exit(struct phy *_phy) >> +{ >> + struct jh7110_usb2_phy *phy = phy_get_drvdata(_phy); >> + >> + clk_disable_unprepare(phy->app_125m); >> + >> + return 0; >> +} >> + >> +static const struct phy_ops jh7110_usb2_phy_ops = { >> + .init = jh7110_usb2_phy_init, >> + .exit = jh7110_usb2_phy_exit, >> + .set_mode = jh7110_usb2_phy_set_mode, >> + .owner = THIS_MODULE, >> +}; >> + >> +static int jh7110_usb_phy_probe(struct platform_device *pdev) >> +{ >> + struct jh7110_usb2_phy *phy; >> + struct device *dev = &pdev->dev; >> + struct phy_provider *phy_provider; >> + >> + phy = devm_kzalloc(dev, sizeof(*phy), GFP_KERNEL); >> + if (!phy) >> + return -ENOMEM; >> + >> + phy->usb_125m_clk = devm_clk_get(dev, "125m"); >> + if (IS_ERR(phy->usb_125m_clk)) >> + return dev_err_probe(dev, PTR_ERR(phy->usb_125m_clk), >> + "Failed to get 125m clock\n"); >> + >> + phy->app_125m = devm_clk_get(dev, "app_125m"); >> + if (IS_ERR(phy->app_125m)) >> + return dev_err_probe(dev, PTR_ERR(phy->app_125m), >> + "Failed to get app 125m clock\n"); >> + >> + phy->regs = devm_platform_ioremap_resource(pdev, 0); >> + if (IS_ERR(phy->regs)) >> + return dev_err_probe(dev, PTR_ERR(phy->regs), >> + "Failed to map phy base\n"); >> + >> + phy->phy = devm_phy_create(dev, NULL, &jh7110_usb2_phy_ops); >> + if (IS_ERR(phy->phy)) >> + return dev_err_probe(dev, PTR_ERR(phy->phy), >> + "Failed to create phy\n"); >> + >> + phy_set_drvdata(phy->phy, phy); >> + phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate); >> + >> + return PTR_ERR_OR_ZERO(phy_provider); >> +} >> + >> +static const struct of_device_id jh7110_usb_phy_of_match[] = { >> + { .compatible = "starfive,jh7110-usb-phy" }, >> + { /* sentinel */ }, >> +}; >> +MODULE_DEVICE_TABLE(of, jh7110_usb_phy_of_match); >> + >> +static struct platform_driver jh7110_usb_phy_driver = { >> + .probe = jh7110_usb_phy_probe, >> + .driver = { >> + .of_match_table = jh7110_usb_phy_of_match, >> + .name = "jh7110-usb-phy", >> + } >> +}; >> +module_platform_driver(jh7110_usb_phy_driver); > > A very light driver, only setting couple of things for hw. Can you > explain how phy registers are configured, am sure there would be many > more..? Do you rely on bootloader or some other entity for that? > Phy Setting is set by reset. Because it is for StarFive JH7110 only. The same as PCIe PHY. >> + >> +MODULE_DESCRIPTION("StarFive JH7110 USB 2.0 PHY driver"); >> +MODULE_AUTHOR("Minda Chen <minda.chen@starfivetech.com>"); >> +MODULE_LICENSE("GPL"); >> -- >> 2.17.1 >
diff --git a/MAINTAINERS b/MAINTAINERS index f794002a192e..d2ce89a8d31c 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -20174,6 +20174,12 @@ S: Supported F: Documentation/devicetree/bindings/watchdog/starfive* F: drivers/watchdog/starfive-wdt.c +STARFIVE JH71X0 USB PHY DRIVER +M: Minda Chen <minda.chen@starfivetech.com> +S: Supported +F: Documentation/devicetree/bindings/phy/starfive,jh7110-usb-phy.yaml +F: drivers/phy/starfive/phy-jh7110-usb.c + STATIC BRANCH/CALL M: Peter Zijlstra <peterz@infradead.org> M: Josh Poimboeuf <jpoimboe@kernel.org> diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig index f46e3148d286..0000149edbc4 100644 --- a/drivers/phy/Kconfig +++ b/drivers/phy/Kconfig @@ -91,6 +91,7 @@ source "drivers/phy/rockchip/Kconfig" source "drivers/phy/samsung/Kconfig" source "drivers/phy/socionext/Kconfig" source "drivers/phy/st/Kconfig" +source "drivers/phy/starfive/Kconfig" source "drivers/phy/sunplus/Kconfig" source "drivers/phy/tegra/Kconfig" source "drivers/phy/ti/Kconfig" diff --git a/drivers/phy/Makefile b/drivers/phy/Makefile index 54f312c10a40..fb3dc9de6111 100644 --- a/drivers/phy/Makefile +++ b/drivers/phy/Makefile @@ -31,6 +31,7 @@ obj-y += allwinner/ \ samsung/ \ socionext/ \ st/ \ + starfive/ \ sunplus/ \ tegra/ \ ti/ \ diff --git a/drivers/phy/starfive/Kconfig b/drivers/phy/starfive/Kconfig new file mode 100644 index 000000000000..2283feadfc76 --- /dev/null +++ b/drivers/phy/starfive/Kconfig @@ -0,0 +1,15 @@ +# SPDX-License-Identifier: GPL-2.0-only +# +# Phy drivers for StarFive platforms +# + +config PHY_STARFIVE_JH7110_USB + tristate "Starfive JH7110 USB 2.0 PHY support" + depends on USB_SUPPORT + select GENERIC_PHY + select USB_PHY + help + Enable this to support the StarFive USB 2.0 PHY, + used with the Cadence USB controller. + If M is selected, the module will be called + phy-jh7110-usb.ko. diff --git a/drivers/phy/starfive/Makefile b/drivers/phy/starfive/Makefile new file mode 100644 index 000000000000..52e9a09cc619 --- /dev/null +++ b/drivers/phy/starfive/Makefile @@ -0,0 +1,2 @@ +# SPDX-License-Identifier: GPL-2.0 +obj-$(CONFIG_PHY_STARFIVE_JH7110_USB) += phy-jh7110-usb.o diff --git a/drivers/phy/starfive/phy-jh7110-usb.c b/drivers/phy/starfive/phy-jh7110-usb.c new file mode 100644 index 000000000000..90d788423705 --- /dev/null +++ b/drivers/phy/starfive/phy-jh7110-usb.c @@ -0,0 +1,150 @@ +// SPDX-License-Identifier: GPL-2.0+ +/* + * StarFive JH7110 USB 2.0 PHY driver + * + * Copyright (C) 2023 StarFive Technology Co., Ltd. + * Author: Minda Chen <minda.chen@starfivetech.com> + */ + +#include <linux/bits.h> +#include <linux/clk.h> +#include <linux/err.h> +#include <linux/io.h> +#include <linux/module.h> +#include <linux/phy/phy.h> +#include <linux/platform_device.h> +#include <linux/usb/of.h> + +#define USB_125M_CLK_RATE 125000000 +#define USB_LS_KEEPALIVE_OFF 0x4 +#define USB_LS_KEEPALIVE_ENABLE BIT(4) + +struct jh7110_usb2_phy { + struct phy *phy; + void __iomem *regs; + struct clk *usb_125m_clk; + struct clk *app_125m; + enum phy_mode mode; +}; + +static void jh7110_usb2_mode_set(struct jh7110_usb2_phy *phy) +{ + unsigned int val; + + if (phy->mode != PHY_MODE_USB_HOST) { + /* Enable the LS speed keep-alive signal */ + val = readl(phy->regs + USB_LS_KEEPALIVE_OFF); + val |= USB_LS_KEEPALIVE_ENABLE; + writel(val, phy->regs + USB_LS_KEEPALIVE_OFF); + } +} + +static int jh7110_usb2_phy_set_mode(struct phy *_phy, + enum phy_mode mode, int submode) +{ + struct jh7110_usb2_phy *phy = phy_get_drvdata(_phy); + + switch (mode) { + case PHY_MODE_USB_HOST: + case PHY_MODE_USB_DEVICE: + case PHY_MODE_USB_OTG: + break; + default: + return -EINVAL; + } + + if (mode != phy->mode) { + dev_dbg(&_phy->dev, "Changing phy to %d\n", mode); + phy->mode = mode; + jh7110_usb2_mode_set(phy); + } + + return 0; +} + +static int jh7110_usb2_phy_init(struct phy *_phy) +{ + struct jh7110_usb2_phy *phy = phy_get_drvdata(_phy); + int ret; + + ret = clk_set_rate(phy->usb_125m_clk, USB_125M_CLK_RATE); + if (ret) + return ret; + + ret = clk_prepare_enable(phy->app_125m); + if (ret) + return ret; + + return 0; +} + +static int jh7110_usb2_phy_exit(struct phy *_phy) +{ + struct jh7110_usb2_phy *phy = phy_get_drvdata(_phy); + + clk_disable_unprepare(phy->app_125m); + + return 0; +} + +static const struct phy_ops jh7110_usb2_phy_ops = { + .init = jh7110_usb2_phy_init, + .exit = jh7110_usb2_phy_exit, + .set_mode = jh7110_usb2_phy_set_mode, + .owner = THIS_MODULE, +}; + +static int jh7110_usb_phy_probe(struct platform_device *pdev) +{ + struct jh7110_usb2_phy *phy; + struct device *dev = &pdev->dev; + struct phy_provider *phy_provider; + + phy = devm_kzalloc(dev, sizeof(*phy), GFP_KERNEL); + if (!phy) + return -ENOMEM; + + phy->usb_125m_clk = devm_clk_get(dev, "125m"); + if (IS_ERR(phy->usb_125m_clk)) + return dev_err_probe(dev, PTR_ERR(phy->usb_125m_clk), + "Failed to get 125m clock\n"); + + phy->app_125m = devm_clk_get(dev, "app_125m"); + if (IS_ERR(phy->app_125m)) + return dev_err_probe(dev, PTR_ERR(phy->app_125m), + "Failed to get app 125m clock\n"); + + phy->regs = devm_platform_ioremap_resource(pdev, 0); + if (IS_ERR(phy->regs)) + return dev_err_probe(dev, PTR_ERR(phy->regs), + "Failed to map phy base\n"); + + phy->phy = devm_phy_create(dev, NULL, &jh7110_usb2_phy_ops); + if (IS_ERR(phy->phy)) + return dev_err_probe(dev, PTR_ERR(phy->phy), + "Failed to create phy\n"); + + phy_set_drvdata(phy->phy, phy); + phy_provider = devm_of_phy_provider_register(dev, of_phy_simple_xlate); + + return PTR_ERR_OR_ZERO(phy_provider); +} + +static const struct of_device_id jh7110_usb_phy_of_match[] = { + { .compatible = "starfive,jh7110-usb-phy" }, + { /* sentinel */ }, +}; +MODULE_DEVICE_TABLE(of, jh7110_usb_phy_of_match); + +static struct platform_driver jh7110_usb_phy_driver = { + .probe = jh7110_usb_phy_probe, + .driver = { + .of_match_table = jh7110_usb_phy_of_match, + .name = "jh7110-usb-phy", + } +}; +module_platform_driver(jh7110_usb_phy_driver); + +MODULE_DESCRIPTION("StarFive JH7110 USB 2.0 PHY driver"); +MODULE_AUTHOR("Minda Chen <minda.chen@starfivetech.com>"); +MODULE_LICENSE("GPL");