[U-Boot] [PATCH v5 6/7] phy: Add USB PHY driver for the cadence USB3
Jean-Jacques Hiblot
jjhiblot at ti.com
Wed Aug 28 09:23:47 UTC 2019
+Kishon who worked on this PHY under linux
Hi Sherry,
On 28/08/2019 10:05, Sherry Sun wrote:
> Hi Jean,
>
>> Hi Sherry,
>>
>> On 21/08/2019 16:36, Sherry Sun wrote:
>>> The cdns3-usb-phy driver supports both host and peripheral mode of usb
>>> driver which use cadence usb3 IP.
>>>
>>> Signed-off-by: Sherry Sun <sherry.sun at nxp.com>
>>> ---
>>> drivers/phy/Kconfig | 8 ++
>>> drivers/phy/Makefile | 1 +
>>> drivers/phy/cdns3-usb-phy.c | 241
>> ++++++++++++++++++++++++++++++++++++
>>> 3 files changed, 250 insertions(+)
>>> create mode 100644 drivers/phy/cdns3-usb-phy.c
>>>
>>> diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig index
>>> 3942f035eb..1837b32c31 100644
>>> --- a/drivers/phy/Kconfig
>>> +++ b/drivers/phy/Kconfig
>>> @@ -205,4 +205,12 @@ config MT76X8_USB_PHY
>>>
>>> This PHY is found on MT76x8 devices supporting USB.
>>>
>>> +config CDNS3_USB_PHY
>>> + bool "Support CDNS3 USB PHY"
>>> + depends on PHY
>>> + help
>>> + Support for the USB PHY in CDNS3 IP.
>>> +
>>> + This PHY is found on CDNS3 IP devices supporting USB.
>>> +
>>> endmenu
>>> diff --git a/drivers/phy/Makefile b/drivers/phy/Makefile index
>>> 3157f1b7ee..0e062214d3 100644
>>> --- a/drivers/phy/Makefile
>>> +++ b/drivers/phy/Makefile
>>> @@ -22,4 +22,5 @@ obj-$(CONFIG_MSM8916_USB_PHY) += msm8916-
>> usbh-phy.o
>>> obj-$(CONFIG_OMAP_USB2_PHY) += omap-usb2-phy.o
>>> obj-$(CONFIG_KEYSTONE_USB_PHY) += keystone-usb-phy.o
>>> obj-$(CONFIG_MT76X8_USB_PHY) += mt76x8-usb-phy.o
>>> +obj-$(CONFIG_CDNS3_USB_PHY) += cdns3-usb-phy.o
>>> obj-$(CONFIG_PHY_DA8XX_USB) += phy-da8xx-usb.o diff --git
>>> a/drivers/phy/cdns3-usb-phy.c b/drivers/phy/cdns3-usb-phy.c new file
>>> mode 100644 index 0000000000..c0d308075b
>>> --- /dev/null
>>> +++ b/drivers/phy/cdns3-usb-phy.c
>>> @@ -0,0 +1,241 @@
>>> +// SPDX-License-Identifier: GPL-2.0+
>>> +/*
>>> + * Copyright 2019 NXP
>>> + *
>>> + * Cadence3 USB PHY driver
>>> + *
>>> + * Author: Sherry Sun <sherry.sun at nxp.com> */
>>> +
>>> +#include <common.h>
>>> +#include <clk.h>
>>> +#include <dm.h>
>>> +#include <generic-phy.h>
>>> +#include <asm/io.h>
>>> +
>>> +/* PHY registers */
>>> +#define PHY_PMA_CMN_CTRL1 (0xC800 * 4)
>>> +#define TB_ADDR_CMN_DIAG_HSCLK_SEL (0x01e0 * 4)
>>> +#define TB_ADDR_CMN_PLL0_VCOCAL_INIT_TMR (0x0084 * 4)
>>> +#define TB_ADDR_CMN_PLL0_VCOCAL_ITER_TMR (0x0085 * 4)
>>> +#define TB_ADDR_CMN_PLL0_INTDIV (0x0094 * 4)
>>> +#define TB_ADDR_CMN_PLL0_FRACDIV (0x0095 * 4)
>>> +#define TB_ADDR_CMN_PLL0_HIGH_THR (0x0096 * 4)
>>> +#define TB_ADDR_CMN_PLL0_SS_CTRL1 (0x0098 * 4)
>>> +#define TB_ADDR_CMN_PLL0_SS_CTRL2 (0x0099 * 4)
>>> +#define TB_ADDR_CMN_PLL0_DSM_DIAG (0x0097 * 4)
>>> +#define TB_ADDR_CMN_DIAG_PLL0_OVRD (0x01c2 * 4)
>>> +#define TB_ADDR_CMN_DIAG_PLL0_FBH_OVRD (0x01c0 * 4)
>>> +#define TB_ADDR_CMN_DIAG_PLL0_FBL_OVRD (0x01c1 * 4)
>>> +#define TB_ADDR_CMN_DIAG_PLL0_V2I_TUNE (0x01C5 * 4)
>>> +#define TB_ADDR_CMN_DIAG_PLL0_CP_TUNE (0x01C6 * 4)
>>> +#define TB_ADDR_CMN_DIAG_PLL0_LF_PROG (0x01C7 * 4)
>>> +#define TB_ADDR_CMN_DIAG_PLL0_TEST_MODE (0x01c4 * 4)
>>> +#define TB_ADDR_CMN_PSM_CLK_CTRL (0x0061 * 4)
>>> +#define TB_ADDR_XCVR_DIAG_RX_LANE_CAL_RST_TMR (0x40ea * 4)
>>> +#define TB_ADDR_XCVR_PSM_RCTRL (0x4001 * 4)
>>> +#define TB_ADDR_TX_PSC_A0 (0x4100 * 4)
>>> +#define TB_ADDR_TX_PSC_A1 (0x4101 * 4)
>>> +#define TB_ADDR_TX_PSC_A2 (0x4102 * 4)
>>> +#define TB_ADDR_TX_PSC_A3 (0x4103 * 4)
>>> +#define TB_ADDR_TX_DIAG_ECTRL_OVRD (0x41f5 * 4)
>>> +#define TB_ADDR_TX_PSC_CAL (0x4106 * 4)
>>> +#define TB_ADDR_TX_PSC_RDY (0x4107 * 4)
>>> +#define TB_ADDR_RX_PSC_A0 (0x8000 * 4)
>>> +#define TB_ADDR_RX_PSC_A1 (0x8001 * 4)
>>> +#define TB_ADDR_RX_PSC_A2 (0x8002 * 4)
>>> +#define TB_ADDR_RX_PSC_A3 (0x8003 * 4)
>>> +#define TB_ADDR_RX_PSC_CAL (0x8006 * 4)
>>> +#define TB_ADDR_RX_PSC_RDY (0x8007 * 4)
>>> +#define TB_ADDR_TX_TXCC_MGNLS_MULT_000 (0x4058 * 4)
>>> +#define TB_ADDR_TX_DIAG_BGREF_PREDRV_DELAY (0x41e7 * 4)
>>> +#define TB_ADDR_RX_SLC_CU_ITER_TMR (0x80e3 * 4)
>>> +#define TB_ADDR_RX_SIGDET_HL_FILT_TMR (0x8090 * 4)
>>> +#define TB_ADDR_RX_SAMP_DAC_CTRL (0x8058 * 4)
>>> +#define TB_ADDR_RX_DIAG_SIGDET_TUNE (0x81dc * 4)
>>> +#define TB_ADDR_RX_DIAG_LFPSDET_TUNE2 (0x81df * 4)
>>> +#define TB_ADDR_RX_DIAG_BS_TM (0x81f5 * 4)
>>> +#define TB_ADDR_RX_DIAG_DFE_CTRL1 (0x81d3 * 4)
>>> +#define TB_ADDR_RX_DIAG_ILL_IQE_TRIM4 (0x81c7 * 4)
>>> +#define TB_ADDR_RX_DIAG_ILL_E_TRIM0 (0x81c2 * 4)
>>> +#define TB_ADDR_RX_DIAG_ILL_IQ_TRIM0 (0x81c1 * 4)
>>> +#define TB_ADDR_RX_DIAG_ILL_IQE_TRIM6 (0x81c9 * 4)
>>> +#define TB_ADDR_RX_DIAG_RXFE_TM3 (0x81f8 * 4)
>>> +#define TB_ADDR_RX_DIAG_RXFE_TM4 (0x81f9 * 4)
>>> +#define TB_ADDR_RX_DIAG_LFPSDET_TUNE (0x81dd * 4)
>>> +#define TB_ADDR_RX_DIAG_DFE_CTRL3 (0x81d5 * 4)
>>> +#define TB_ADDR_RX_DIAG_SC2C_DELAY (0x81e1 * 4)
>>> +#define TB_ADDR_RX_REE_VGA_GAIN_NODFE (0x81bf * 4)
>>> +#define TB_ADDR_XCVR_PSM_CAL_TMR (0x4002 * 4)
>>> +#define TB_ADDR_XCVR_PSM_A0BYP_TMR (0x4004 * 4)
>>> +#define TB_ADDR_XCVR_PSM_A0IN_TMR (0x4003 * 4)
>>> +#define TB_ADDR_XCVR_PSM_A1IN_TMR (0x4005 * 4)
>>> +#define TB_ADDR_XCVR_PSM_A2IN_TMR (0x4006 * 4)
>>> +#define TB_ADDR_XCVR_PSM_A3IN_TMR (0x4007 * 4)
>>> +#define TB_ADDR_XCVR_PSM_A4IN_TMR (0x4008 * 4)
>>> +#define TB_ADDR_XCVR_PSM_A5IN_TMR (0x4009 * 4)
>>> +#define TB_ADDR_XCVR_PSM_A0OUT_TMR (0x400a * 4)
>>> +#define TB_ADDR_XCVR_PSM_A1OUT_TMR (0x400b * 4)
>>> +#define TB_ADDR_XCVR_PSM_A2OUT_TMR (0x400c * 4)
>>> +#define TB_ADDR_XCVR_PSM_A3OUT_TMR (0x400d * 4)
>>> +#define TB_ADDR_XCVR_PSM_A4OUT_TMR (0x400e * 4)
>>> +#define TB_ADDR_XCVR_PSM_A5OUT_TMR (0x400f * 4)
>>> +#define TB_ADDR_TX_RCVDET_EN_TMR (0x4122 * 4)
>>> +#define TB_ADDR_TX_RCVDET_ST_TMR (0x4123 * 4)
>>> +#define TB_ADDR_XCVR_DIAG_LANE_FCM_EN_MGN_TMR (0x40f2 * 4)
>>> +
>>> +struct cdns3_usb_phy {
>>> + struct clk phy_clk;
>>> + void __iomem *phy_regs;
>>> +};
>>> +
>>> +static int cdns3_usb_phy_init(struct phy *phy) {
>>> + struct udevice *dev = phy->dev;
>>> + struct cdns3_usb_phy *priv = dev_get_priv(dev);
>>> + void __iomem *regs = priv->phy_regs;
>>> +
>>> + writel(0x0830, regs + PHY_PMA_CMN_CTRL1);
>>> + writel(0x10, regs + TB_ADDR_CMN_DIAG_HSCLK_SEL);
>>> + writel(0x00F0, regs + TB_ADDR_CMN_PLL0_VCOCAL_INIT_TMR);
>>> + writel(0x0018, regs + TB_ADDR_CMN_PLL0_VCOCAL_ITER_TMR);
>>> + writel(0x00D0, regs + TB_ADDR_CMN_PLL0_INTDIV);
>>> + writel(0x4aaa, regs + TB_ADDR_CMN_PLL0_FRACDIV);
>>> + writel(0x0034, regs + TB_ADDR_CMN_PLL0_HIGH_THR);
>>> + writel(0x1ee, regs + TB_ADDR_CMN_PLL0_SS_CTRL1);
>>> + writel(0x7F03, regs + TB_ADDR_CMN_PLL0_SS_CTRL2);
>>> + writel(0x0020, regs + TB_ADDR_CMN_PLL0_DSM_DIAG);
>>> + writel(0x0000, regs + TB_ADDR_CMN_DIAG_PLL0_OVRD);
>>> + writel(0x0000, regs + TB_ADDR_CMN_DIAG_PLL0_FBH_OVRD);
>>> + writel(0x0000, regs + TB_ADDR_CMN_DIAG_PLL0_FBL_OVRD);
>>> + writel(0x0007, regs + TB_ADDR_CMN_DIAG_PLL0_V2I_TUNE);
>>> + writel(0x0027, regs + TB_ADDR_CMN_DIAG_PLL0_CP_TUNE);
>>> + writel(0x0008, regs + TB_ADDR_CMN_DIAG_PLL0_LF_PROG);
>>> + writel(0x0022, regs + TB_ADDR_CMN_DIAG_PLL0_TEST_MODE);
>>> + writel(0x000a, regs + TB_ADDR_CMN_PSM_CLK_CTRL);
>>> + writel(0x139, regs + TB_ADDR_XCVR_DIAG_RX_LANE_CAL_RST_TMR);
>>> + writel(0xbefc, regs + TB_ADDR_XCVR_PSM_RCTRL);
>>> +
>>> + writel(0x7799, regs + TB_ADDR_TX_PSC_A0);
>>> + writel(0x7798, regs + TB_ADDR_TX_PSC_A1);
>>> + writel(0x509b, regs + TB_ADDR_TX_PSC_A2);
>>> + writel(0x3, regs + TB_ADDR_TX_DIAG_ECTRL_OVRD);
>>> + writel(0x509b, regs + TB_ADDR_TX_PSC_A3);
>>> + writel(0x2090, regs + TB_ADDR_TX_PSC_CAL);
>>> + writel(0x2090, regs + TB_ADDR_TX_PSC_RDY);
>>> +
>>> + writel(0xA6FD, regs + TB_ADDR_RX_PSC_A0);
>>> + writel(0xA6FD, regs + TB_ADDR_RX_PSC_A1);
>>> + writel(0xA410, regs + TB_ADDR_RX_PSC_A2);
>>> + writel(0x2410, regs + TB_ADDR_RX_PSC_A3);
>>> +
>>> + writel(0x23FF, regs + TB_ADDR_RX_PSC_CAL);
>>> + writel(0x2010, regs + TB_ADDR_RX_PSC_RDY);
>>> +
>>> + writel(0x0020, regs + TB_ADDR_TX_TXCC_MGNLS_MULT_000);
>>> + writel(0x00ff, regs + TB_ADDR_TX_DIAG_BGREF_PREDRV_DELAY);
>>> + writel(0x0002, regs + TB_ADDR_RX_SLC_CU_ITER_TMR);
>>> + writel(0x0013, regs + TB_ADDR_RX_SIGDET_HL_FILT_TMR);
>>> + writel(0x0000, regs + TB_ADDR_RX_SAMP_DAC_CTRL);
>>> + writel(0x1004, regs + TB_ADDR_RX_DIAG_SIGDET_TUNE);
>>> + writel(0x4041, regs + TB_ADDR_RX_DIAG_LFPSDET_TUNE2);
>>> + writel(0x0480, regs + TB_ADDR_RX_DIAG_BS_TM);
>>> + writel(0x8006, regs + TB_ADDR_RX_DIAG_DFE_CTRL1);
>>> + writel(0x003f, regs + TB_ADDR_RX_DIAG_ILL_IQE_TRIM4);
>>> + writel(0x543f, regs + TB_ADDR_RX_DIAG_ILL_E_TRIM0);
>>> + writel(0x543f, regs + TB_ADDR_RX_DIAG_ILL_IQ_TRIM0);
>>> + writel(0x0000, regs + TB_ADDR_RX_DIAG_ILL_IQE_TRIM6);
>>> + writel(0x8000, regs + TB_ADDR_RX_DIAG_RXFE_TM3);
>>> + writel(0x0003, regs + TB_ADDR_RX_DIAG_RXFE_TM4);
>>> + writel(0x2408, regs + TB_ADDR_RX_DIAG_LFPSDET_TUNE);
>>> + writel(0x05ca, regs + TB_ADDR_RX_DIAG_DFE_CTRL3);
>>> + writel(0x0258, regs + TB_ADDR_RX_DIAG_SC2C_DELAY);
>>> + writel(0x1fff, regs + TB_ADDR_RX_REE_VGA_GAIN_NODFE);
>>> +
>>> + writel(0x02c6, regs + TB_ADDR_XCVR_PSM_CAL_TMR);
>>> + writel(0x0002, regs + TB_ADDR_XCVR_PSM_A0BYP_TMR);
>>> + writel(0x02c6, regs + TB_ADDR_XCVR_PSM_A0IN_TMR);
>>> + writel(0x0010, regs + TB_ADDR_XCVR_PSM_A1IN_TMR);
>>> + writel(0x0010, regs + TB_ADDR_XCVR_PSM_A2IN_TMR);
>>> + writel(0x0010, regs + TB_ADDR_XCVR_PSM_A3IN_TMR);
>>> + writel(0x0010, regs + TB_ADDR_XCVR_PSM_A4IN_TMR);
>>> + writel(0x0010, regs + TB_ADDR_XCVR_PSM_A5IN_TMR);
>>> +
>>> + writel(0x0002, regs + TB_ADDR_XCVR_PSM_A0OUT_TMR);
>>> + writel(0x0002, regs + TB_ADDR_XCVR_PSM_A1OUT_TMR);
>>> + writel(0x0002, regs + TB_ADDR_XCVR_PSM_A2OUT_TMR);
>>> + writel(0x0002, regs + TB_ADDR_XCVR_PSM_A3OUT_TMR);
>>> + writel(0x0002, regs + TB_ADDR_XCVR_PSM_A4OUT_TMR);
>>> + writel(0x0002, regs + TB_ADDR_XCVR_PSM_A5OUT_TMR);
>>> +
>>> + /* Change rx detect parameter */
>>> + writel(0x960, regs + TB_ADDR_TX_RCVDET_EN_TMR);
>>> + writel(0x01e0, regs + TB_ADDR_TX_RCVDET_ST_TMR);
>>> + writel(0x0090, regs +
>> TB_ADDR_XCVR_DIAG_LANE_FCM_EN_MGN_TMR);
>>> +
>>> + udelay(10);
>>> + return 0;
>>> +}
>> Where do all those hard-coded values come from? Is this something that
>> would work for all platforms implementing this PHY ?
>>
> These values are from Cadence USB 3.0 PHY User Guide, and certainly it can work for all platforms which use this PHY.
>
>> Is there a linux driver for this PHY ?
> Yes, the cdns3 phy driver in kernel is phy-cadence-sierra.c,
> but it is shared with pcie, and it has a lot of differences with this PHY driver in uboot.
This is indeed a different driver. IMO it would be better to keep the
driver closer to its linux version. The reason for that is that it is
quite new and will be adapted when platforms starts supporting it. If we
start with something too far from the original, it will hard to update.
Already I see that the registers initialized are not all the same as in
the linux driver and not all the values are the same either, nor are the
register names
TI's J7 platforms is using this PHY driver for USB3/PCIe support, and
the linux driver has already seen quite a few modifications to get it to
work
(https://git.ti.com/ti-linux-kernel/ti-linux-kernel/blobs/history/ti-linux-4.19.y/drivers/phy/cadence/phy-cadence-sierra.c)
Adding support for it on top of this current version will be rather
difficult.
JJ
>
> Best regards
> Sherry sun
>
>> JJ
>>
>>> +
>>> +struct phy_ops cdns3_usb_phy_ops = {
>>> + .init = cdns3_usb_phy_init,
>>> +};
>>> +
>>> +static int cdns3_usb_phy_remove(struct udevice *dev) { #if
>>> +CONFIG_IS_ENABLED(CLK)
>>> + struct cdns3_usb_phy *priv = dev_get_priv(dev);
>>> + int ret;
>>> +
>>> + if (priv->phy_clk.dev) {
>>> + ret = clk_disable(&priv->phy_clk);
>>> + if (ret)
>>> + return ret;
>>> +
>>> + ret = clk_free(&priv->phy_clk);
>>> + if (ret)
>>> + return ret;
>>> + }
>>> +#endif
>>> +
>>> + return 0;
>>> +}
>>> +
>>> +static int cdns3_usb_phy_probe(struct udevice *dev) {
>>> + struct cdns3_usb_phy *priv = dev_get_priv(dev);
>>> +
>>> +#if CONFIG_IS_ENABLED(CLK)
>>> + int ret;
>>> +
>>> + ret = clk_get_by_name(dev, "main_clk", &priv->phy_clk);
>>> + if (ret) {
>>> + printf("Failed to get phy_clk\n");
>>> + return ret;
>>> + }
>>> +
>>> + ret = clk_enable(&priv->phy_clk);
>>> + if (ret) {
>>> + printf("Failed to enable phy_clk\n");
>>> + return ret;
>>> + }
>>> +#endif
>>> + priv->phy_regs = (void *__iomem)devfdt_get_addr(dev);
>>> +
>>> + return 0;
>>> +}
>>> +
>>> +static const struct udevice_id cdns3_usb_phy_ids[] = {
>>> + { .compatible = "cdns,usb3-phy" },
>>
>>> + { }
>>> +};
>>> +
>>> +U_BOOT_DRIVER(cdns3_usb_phy) = {
>>> + .name = "cdns3_usb_phy",
>>> + .id = UCLASS_PHY,
>>> + .of_match = cdns3_usb_phy_ids,
>>> + .probe = cdns3_usb_phy_probe,
>>> + .remove = cdns3_usb_phy_remove,
>>> + .ops = &cdns3_usb_phy_ops,
>>> + .priv_auto_alloc_size = sizeof(struct cdns3_usb_phy), };
More information about the U-Boot
mailing list