Message ID | a5d13acba646de9abe2ce138067a405de0058a60.1427881003.git.maitysanchayan@gmail.com |
---|---|
State | Superseded |
Delegated to: | Tom Rini |
Headers | show |
On Wednesday, April 01, 2015 at 11:54:22 AM, Sanchayan Maity wrote: The commit message is missing, please fix in v2. > Signed-off-by: Sanchayan Maity <maitysanchayan@gmail.com> [...] > +#define USB_NC_REG_OFFSET 0x00000800 > +#define USBCx_CTRL_OFFSET 0x00000000 > +#define USBCx_PHY_CTRL_OFFSET 0x00000018 Please define the register offsets using the regular struct {} method, see for example struct mxs_usbphy_regs and it's usage in ehci-mxs.c . > +#define USBPHY_CTRL 0x00000030 > +#define USBPHY_CTRL_SET 0x00000034 > +#define USBPHY_CTRL_CLR 0x00000038 > +#define USBPHY_CTRL_TOG 0x0000003c > + > +#define USBPHY_PWD 0x00000000 > +#define USBPHY_TX 0x00000010 > +#define USBPHY_RX 0x00000020 > +#define USBPHY_DEBUG 0x00000050 > +#define USBPHY_CTRL_SFTRST 0x80000000 > +#define USBPHY_CTRL_CLKGATE 0x40000000 > +#define USBPHY_CTRL_ENUTMILEVEL3 0x00008000 > +#define USBPHY_CTRL_ENUTMILEVEL2 0x00004000 > +#define USBPHY_CTRL_OTG_ID 0x08000000 > + > +#define ANADIG_PLL_CTRL_BYPASS 0x00010000 > +#define ANADIG_PLL_CTRL_ENABLE 0x00002000 > +#define ANADIG_PLL_CTRL_POWER 0x00001000 > +#define ANADIG_PLL_CTRL_EN_USB_CLKS 0x00000040 > + > +#define UCTRL_OVER_CUR_POL (1 << 8) /* OTG Polarity of Overcurrent */ > +#define UCTRL_OVER_CUR_DIS (1 << 7) /* Disable OTG Overcurrent Detection > */ + > +/* USBCMD */ > +#define UCMD_RUN_STOP (1 << 0) /* controller run/stop */ > +#define UCMD_RESET (1 << 1) /* controller reset */ This looks very much like the USB PHY used on MX28 , can you double-check this please ? Best regards, Marek Vasut
Hello, On 15-04-01 21:15:21, Marek Vasut wrote: > On Wednesday, April 01, 2015 at 11:54:22 AM, Sanchayan Maity wrote: > > The commit message is missing, please fix in v2. Ok. Will add the commit message in v2. > > > Signed-off-by: Sanchayan Maity <maitysanchayan@gmail.com> > > [...] > > > +#define USB_NC_REG_OFFSET 0x00000800 > > +#define USBCx_CTRL_OFFSET 0x00000000 > > +#define USBCx_PHY_CTRL_OFFSET 0x00000018 > > Please define the register offsets using the regular struct {} method, > see for example struct mxs_usbphy_regs and it's usage in ehci-mxs.c . Ok. Will fix in v2. > > > +#define USBPHY_CTRL 0x00000030 > > +#define USBPHY_CTRL_SET 0x00000034 > > +#define USBPHY_CTRL_CLR 0x00000038 > > +#define USBPHY_CTRL_TOG 0x0000003c > > + > > +#define USBPHY_PWD 0x00000000 > > +#define USBPHY_TX 0x00000010 > > +#define USBPHY_RX 0x00000020 > > +#define USBPHY_DEBUG 0x00000050 > > +#define USBPHY_CTRL_SFTRST 0x80000000 > > +#define USBPHY_CTRL_CLKGATE 0x40000000 > > +#define USBPHY_CTRL_ENUTMILEVEL3 0x00008000 > > +#define USBPHY_CTRL_ENUTMILEVEL2 0x00004000 > > +#define USBPHY_CTRL_OTG_ID 0x08000000 > > + > > +#define ANADIG_PLL_CTRL_BYPASS 0x00010000 > > +#define ANADIG_PLL_CTRL_ENABLE 0x00002000 > > +#define ANADIG_PLL_CTRL_POWER 0x00001000 > > +#define ANADIG_PLL_CTRL_EN_USB_CLKS 0x00000040 > > + > > +#define UCTRL_OVER_CUR_POL (1 << 8) /* OTG Polarity of Overcurrent */ > > +#define UCTRL_OVER_CUR_DIS (1 << 7) /* Disable OTG Overcurrent Detection > > */ + > > +/* USBCMD */ > > +#define UCMD_RUN_STOP (1 << 0) /* controller run/stop */ > > +#define UCMD_RESET (1 << 1) /* controller reset */ > > This looks very much like the USB PHY used on MX28 , can you double-check this > please ? I have not yet looked at MX28, but, the Vybrid USB IP is in fact very similar to i.MX6, however, the non core registers are spread in two different register areas. Will have a look at MX28 as well. This ehci-vf driver is very closely based on ehci-mx6 driver already in uboot. > > Best regards, > Marek Vasut - Sanchayan.
On Thursday, April 02, 2015 at 12:55:57 PM, maitysanchayan@gmail.com wrote: > Hello, Hi! [...] > > > +#define UCTRL_OVER_CUR_DIS (1 << 7) /* Disable OTG Overcurrent > > > Detection */ + > > > +/* USBCMD */ > > > +#define UCMD_RUN_STOP (1 << 0) /* controller run/stop */ > > > +#define UCMD_RESET (1 << 1) /* controller reset */ > > > > This looks very much like the USB PHY used on MX28 , can you double-check > > this please ? > > I have not yet looked at MX28, but, the Vybrid USB IP is in fact very > similar to i.MX6, however, the non core registers are spread in two > different register areas. Will have a look at MX28 as well. > > This ehci-vf driver is very closely based on ehci-mx6 driver already in > uboot. Understood, thanks! Best regards, Marek Vasut
Hello, On 15-04-01 21:15:21, Marek Vasut wrote: > On Wednesday, April 01, 2015 at 11:54:22 AM, Sanchayan Maity wrote: > > The commit message is missing, please fix in v2. > > > Signed-off-by: Sanchayan Maity <maitysanchayan@gmail.com> > > [...] > > > +#define USB_NC_REG_OFFSET 0x00000800 > > +#define USBCx_CTRL_OFFSET 0x00000000 > > +#define USBCx_PHY_CTRL_OFFSET 0x00000018 > > Please define the register offsets using the regular struct {} method, > see for example struct mxs_usbphy_regs and it's usage in ehci-mxs.c . I had a query here, just to be sure and avoid rework. The vybrid defines would be similar to mxs. I assume I can add them to the regs-common.h file along with a note that the VF610 also has the same _set, _clr, _tog register? Or perhaps it would be more appropriate to have the file have generic names which mxs, vf and imx can all leverage? Though for now this would require reworking all the three drivers. The USB phy definitions part is ok, as they would go in the arch specific folder. Thoughts? > > > +#define USBPHY_CTRL 0x00000030 > > +#define USBPHY_CTRL_SET 0x00000034 > > +#define USBPHY_CTRL_CLR 0x00000038 > > +#define USBPHY_CTRL_TOG 0x0000003c > > + > > +#define USBPHY_PWD 0x00000000 > > +#define USBPHY_TX 0x00000010 > > +#define USBPHY_RX 0x00000020 > > +#define USBPHY_DEBUG 0x00000050 > > +#define USBPHY_CTRL_SFTRST 0x80000000 > > +#define USBPHY_CTRL_CLKGATE 0x40000000 > > +#define USBPHY_CTRL_ENUTMILEVEL3 0x00008000 > > +#define USBPHY_CTRL_ENUTMILEVEL2 0x00004000 > > +#define USBPHY_CTRL_OTG_ID 0x08000000 > > + > > +#define ANADIG_PLL_CTRL_BYPASS 0x00010000 > > +#define ANADIG_PLL_CTRL_ENABLE 0x00002000 > > +#define ANADIG_PLL_CTRL_POWER 0x00001000 > > +#define ANADIG_PLL_CTRL_EN_USB_CLKS 0x00000040 > > + > > +#define UCTRL_OVER_CUR_POL (1 << 8) /* OTG Polarity of Overcurrent */ > > +#define UCTRL_OVER_CUR_DIS (1 << 7) /* Disable OTG Overcurrent Detection > > */ + > > +/* USBCMD */ > > +#define UCMD_RUN_STOP (1 << 0) /* controller run/stop */ > > +#define UCMD_RESET (1 << 1) /* controller reset */ > > This looks very much like the USB PHY used on MX28 , can you double-check this > please ? MX28 IP also seems similar to the Vybrid USB IP except for a few registers and the non core registers. Perhaps to be expected as they all have a common chipidea IP core, though having a different version thereof. > > Best regards, > Marek Vasut - Sanchayan.
On Tuesday, April 07, 2015 at 09:03:45 AM, maitysanchayan@gmail.com wrote: > Hello, > > On 15-04-01 21:15:21, Marek Vasut wrote: > > On Wednesday, April 01, 2015 at 11:54:22 AM, Sanchayan Maity wrote: > > > > The commit message is missing, please fix in v2. > > > > > Signed-off-by: Sanchayan Maity <maitysanchayan@gmail.com> > > > > [...] > > > > > +#define USB_NC_REG_OFFSET 0x00000800 > > > +#define USBCx_CTRL_OFFSET 0x00000000 > > > +#define USBCx_PHY_CTRL_OFFSET 0x00000018 > > > > Please define the register offsets using the regular struct {} method, > > see for example struct mxs_usbphy_regs and it's usage in ehci-mxs.c . > > I had a query here, just to be sure and avoid rework. The vybrid defines > would be similar to mxs. I assume I can add them to the regs-common.h > file along with a note that the VF610 also has the same _set, _clr, > _tog register? Or perhaps it would be more appropriate to have the file > have generic names which mxs, vf and imx can all leverage? Though for > now this would require reworking all the three drivers. > > The USB phy definitions part is ok, as they would go in the arch > specific folder. If these are really IMX/MXS/VF specific, then the defines should go into arch/arm/include/asm/imx-common/ . Otherwise, you can make chipidea specific file in include/usb/ . > > > +#define USBPHY_CTRL 0x00000030 > > > +#define USBPHY_CTRL_SET 0x00000034 > > > +#define USBPHY_CTRL_CLR 0x00000038 > > > +#define USBPHY_CTRL_TOG 0x0000003c > > > + > > > +#define USBPHY_PWD 0x00000000 > > > +#define USBPHY_TX 0x00000010 > > > +#define USBPHY_RX 0x00000020 > > > +#define USBPHY_DEBUG 0x00000050 > > > +#define USBPHY_CTRL_SFTRST 0x80000000 > > > +#define USBPHY_CTRL_CLKGATE 0x40000000 > > > +#define USBPHY_CTRL_ENUTMILEVEL3 0x00008000 > > > +#define USBPHY_CTRL_ENUTMILEVEL2 0x00004000 > > > +#define USBPHY_CTRL_OTG_ID 0x08000000 > > > + > > > +#define ANADIG_PLL_CTRL_BYPASS 0x00010000 > > > +#define ANADIG_PLL_CTRL_ENABLE 0x00002000 > > > +#define ANADIG_PLL_CTRL_POWER 0x00001000 > > > +#define ANADIG_PLL_CTRL_EN_USB_CLKS 0x00000040 > > > + > > > +#define UCTRL_OVER_CUR_POL (1 << 8) /* OTG Polarity of Overcurrent */ > > > +#define UCTRL_OVER_CUR_DIS (1 << 7) /* Disable OTG Overcurrent > > > Detection */ + > > > +/* USBCMD */ > > > +#define UCMD_RUN_STOP (1 << 0) /* controller run/stop */ > > > +#define UCMD_RESET (1 << 1) /* controller reset */ > > > > This looks very much like the USB PHY used on MX28 , can you double-check > > this please ? > > MX28 IP also seems similar to the Vybrid USB IP except for a few > registers and the non core registers. Perhaps to be expected as they all > have a common chipidea IP core, though having a different version > thereof. Yep, I agree :) Thanks!
Hi Marek, On Wed, Apr 1, 2015 at 4:15 PM, Marek Vasut <marex@denx.de> wrote: >> +#define USB_NC_REG_OFFSET 0x00000800 >> +#define USBCx_CTRL_OFFSET 0x00000000 >> +#define USBCx_PHY_CTRL_OFFSET 0x00000018 > > Please define the register offsets using the regular struct {} method, > see for example struct mxs_usbphy_regs and it's usage in ehci-mxs.c . I thought we were trying to get rid of the 'no register access via offset' rule in U-boot. https://www.marc.info/?l=u-boot&m=142609602127309&w=2 Please see: Regards, Fabio Estevam
Hello, On 15-04-09 18:09:46, Marek Vasut wrote: > On Tuesday, April 07, 2015 at 09:03:45 AM, maitysanchayan@gmail.com wrote: > > Hello, > > > > On 15-04-01 21:15:21, Marek Vasut wrote: > > > On Wednesday, April 01, 2015 at 11:54:22 AM, Sanchayan Maity wrote: > > > > > > The commit message is missing, please fix in v2. > > > > > > > Signed-off-by: Sanchayan Maity <maitysanchayan@gmail.com> > > > > > > [...] > > > > > > > +#define USB_NC_REG_OFFSET 0x00000800 > > > > +#define USBCx_CTRL_OFFSET 0x00000000 > > > > +#define USBCx_PHY_CTRL_OFFSET 0x00000018 > > > > > > Please define the register offsets using the regular struct {} method, > > > see for example struct mxs_usbphy_regs and it's usage in ehci-mxs.c . > > > > I had a query here, just to be sure and avoid rework. The vybrid defines > > would be similar to mxs. I assume I can add them to the regs-common.h > > file along with a note that the VF610 also has the same _set, _clr, > > _tog register? Or perhaps it would be more appropriate to have the file > > have generic names which mxs, vf and imx can all leverage? Though for > > now this would require reworking all the three drivers. > > > > The USB phy definitions part is ok, as they would go in the arch > > specific folder. > > If these are really IMX/MXS/VF specific, then the defines should go into > arch/arm/include/asm/imx-common/ . Otherwise, you can make chipidea specific > file in include/usb/ . Not really much VF specific. I was not sure about using the mxs_ prefix accessors for VF as well. In the end I found usage in one iMX6 file and decided that it makes better sense to actually use the mxs_ existing defines. They can be used for Vybrid as well. Except for the non core regsiters and a few others no other difference. I send a v2 version of the patchset based taking your suggestion into account. The driver looks cleaner in comparison to the previous :) IMHO. Thanks. https://www.mail-archive.com/u-boot@lists.denx.de/msg168727.html - Sanchayan. > > > > > +#define USBPHY_CTRL > 0x00000030 > > > > +#define USBPHY_CTRL_SET 0x00000034 > > > > +#define USBPHY_CTRL_CLR 0x00000038 > > > > +#define USBPHY_CTRL_TOG 0x0000003c > > > > + > > > > +#define USBPHY_PWD > 0x00000000 > > > > +#define USBPHY_TX > 0x00000010 > > > > +#define USBPHY_RX > 0x00000020 > > > > +#define USBPHY_DEBUG 0x00000050 > > > > +#define USBPHY_CTRL_SFTRST 0x80000000 > > > > +#define USBPHY_CTRL_CLKGATE 0x40000000 > > > > +#define USBPHY_CTRL_ENUTMILEVEL3 0x00008000 > > > > +#define USBPHY_CTRL_ENUTMILEVEL2 0x00004000 > > > > +#define USBPHY_CTRL_OTG_ID 0x08000000 > > > > + > > > > +#define ANADIG_PLL_CTRL_BYPASS 0x00010000 > > > > +#define ANADIG_PLL_CTRL_ENABLE 0x00002000 > > > > +#define ANADIG_PLL_CTRL_POWER 0x00001000 > > > > +#define ANADIG_PLL_CTRL_EN_USB_CLKS 0x00000040 > > > > + > > > > +#define UCTRL_OVER_CUR_POL (1 << 8) /* OTG Polarity of Overcurrent > */ > > > > +#define UCTRL_OVER_CUR_DIS (1 << 7) /* Disable OTG Overcurrent > > > > Detection */ + > > > > +/* USBCMD */ > > > > +#define UCMD_RUN_STOP (1 << 0) /* controller run/stop */ > > > > +#define UCMD_RESET (1 << 1) /* controller reset */ > > > > > > This looks very much like the USB PHY used on MX28 , can you double-check > > > this please ? > > > > MX28 IP also seems similar to the Vybrid USB IP except for a few > > registers and the non core registers. Perhaps to be expected as they all > > have a common chipidea IP core, though having a different version > > thereof. > > Yep, I agree :) > > Thanks!
On Thursday, April 09, 2015 at 06:51:45 PM, Fabio Estevam wrote: > Hi Marek, > > On Wed, Apr 1, 2015 at 4:15 PM, Marek Vasut <marex@denx.de> wrote: > >> +#define USB_NC_REG_OFFSET 0x00000800 > >> +#define USBCx_CTRL_OFFSET 0x00000000 > >> +#define USBCx_PHY_CTRL_OFFSET 0x00000018 > > > > Please define the register offsets using the regular struct {} method, > > see for example struct mxs_usbphy_regs and it's usage in ehci-mxs.c . > > I thought we were trying to get rid of the 'no register access via > offset' rule in U-boot. > https://www.marc.info/?l=u-boot&m=142609602127309&w=2 Thanks! Did we reach some agreement on this ? Best regards, Marek Vasut
On Thursday, April 09, 2015 at 07:05:15 PM, maitysanchayan@gmail.com wrote: > Hello, > > On 15-04-09 18:09:46, Marek Vasut wrote: > > On Tuesday, April 07, 2015 at 09:03:45 AM, maitysanchayan@gmail.com wrote: > > > Hello, > > > > > > On 15-04-01 21:15:21, Marek Vasut wrote: > > > > On Wednesday, April 01, 2015 at 11:54:22 AM, Sanchayan Maity wrote: > > > > > > > > The commit message is missing, please fix in v2. > > > > > > > > > Signed-off-by: Sanchayan Maity <maitysanchayan@gmail.com> > > > > > > > > [...] > > > > > > > > > +#define USB_NC_REG_OFFSET 0x00000800 > > > > > +#define USBCx_CTRL_OFFSET 0x00000000 > > > > > +#define USBCx_PHY_CTRL_OFFSET 0x00000018 > > > > > > > > Please define the register offsets using the regular struct {} > > > > method, see for example struct mxs_usbphy_regs and it's usage in > > > > ehci-mxs.c . > > > > > > I had a query here, just to be sure and avoid rework. The vybrid > > > defines would be similar to mxs. I assume I can add them to the > > > regs-common.h file along with a note that the VF610 also has the same > > > _set, _clr, _tog register? Or perhaps it would be more appropriate to > > > have the file have generic names which mxs, vf and imx can all > > > leverage? Though for now this would require reworking all the three > > > drivers. > > > > > > The USB phy definitions part is ok, as they would go in the arch > > > specific folder. > > > > If these are really IMX/MXS/VF specific, then the defines should go into > > arch/arm/include/asm/imx-common/ . Otherwise, you can make chipidea > > specific file in include/usb/ . > > Not really much VF specific. I was not sure about using the mxs_ prefix > accessors for VF as well. In the end I found usage in one iMX6 file and > decided that it makes better sense to actually use the mxs_ existing > defines. They can be used for Vybrid as well. Except for the non core > regsiters and a few others no other difference. > > I send a v2 version of the patchset based taking your suggestion into > account. The driver looks cleaner in comparison to the previous :) > IMHO. Thanks. > > https://www.mail-archive.com/u-boot@lists.denx.de/msg168727.html Roger. If you drop the MXS prefix, feel free to do that in a separate patch, so this set can go in first :)
diff --git a/arch/arm/include/asm/arch-vf610/crm_regs.h b/arch/arm/include/asm/arch-vf610/crm_regs.h index 78708e2..851c3b9 100644 --- a/arch/arm/include/asm/arch-vf610/crm_regs.h +++ b/arch/arm/include/asm/arch-vf610/crm_regs.h @@ -124,6 +124,10 @@ struct anadig_reg { #define CCM_CCSR_PLL1_PFD_CLK_SEL_MASK (0x7 << 16) #define CCM_CCSR_PLL1_PFD_CLK_SEL(v) (((v) & 0x7) << 16) +#define CCM_CCSR_PLL3_PFD4_EN (1 << 31) +#define CCM_CCSR_PLL3_PFD3_EN (1 << 30) +#define CCM_CCSR_PLL3_PFD2_EN (1 << 29) +#define CCM_CCSR_PLL3_PFD1_EN (1 << 28) #define CCM_CCSR_PLL2_PFD4_EN (1 << 15) #define CCM_CCSR_PLL2_PFD3_EN (1 << 14) #define CCM_CCSR_PLL2_PFD2_EN (1 << 13) @@ -189,6 +193,7 @@ struct anadig_reg { #define CCM_REG_CTRL_MASK 0xffffffff #define CCM_CCGR0_UART0_CTRL_MASK (0x3 << 14) #define CCM_CCGR0_UART1_CTRL_MASK (0x3 << 16) +#define CCM_CCGR1_USBC0_CTRL_MASK (0x3 << 8) #define CCM_CCGR1_PIT_CTRL_MASK (0x3 << 14) #define CCM_CCGR1_WDOGA5_CTRL_MASK (0x3 << 28) #define CCM_CCGR2_QSPI0_CTRL_MASK (0x3 << 8) @@ -207,14 +212,23 @@ struct anadig_reg { #define CCM_CCGR6_OCOTP_CTRL_MASK (0x3 << 10) #define CCM_CCGR6_DDRMC_CTRL_MASK (0x3 << 28) #define CCM_CCGR7_SDHC1_CTRL_MASK (0x3 << 4) +#define CCM_CCGR7_USBC1_CTRL_MASK (0x3 << 8) #define CCM_CCGR9_FEC0_CTRL_MASK 0x3 #define CCM_CCGR9_FEC1_CTRL_MASK (0x3 << 2) #define CCM_CCGR10_NFC_CTRL_MASK 0x3 +#define ANADIG_PLL7_CTRL_BYPASS (1 << 16) +#define ANADIG_PLL7_CTRL_ENABLE (1 << 13) +#define ANADIG_PLL7_CTRL_POWERDOWN (1 << 12) +#define ANADIG_PLL7_CTRL_DIV_SELECT (1 << 1) #define ANADIG_PLL5_CTRL_BYPASS (1 << 16) #define ANADIG_PLL5_CTRL_ENABLE (1 << 13) #define ANADIG_PLL5_CTRL_POWERDOWN (1 << 12) #define ANADIG_PLL5_CTRL_DIV_SELECT 1 +#define ANADIG_PLL3_CTRL_BYPASS (1 << 16) +#define ANADIG_PLL3_CTRL_ENABLE (1 << 13) +#define ANADIG_PLL3_CTRL_POWERDOWN (1 << 12) +#define ANADIG_PLL3_CTRL_DIV_SELECT (1 << 1) #define ANADIG_PLL2_CTRL_ENABLE (1 << 13) #define ANADIG_PLL2_CTRL_POWERDOWN (1 << 12) #define ANADIG_PLL2_CTRL_DIV_SELECT 1 diff --git a/arch/arm/include/asm/arch-vf610/imx-regs.h b/arch/arm/include/asm/arch-vf610/imx-regs.h index a5908ca..26057fe 100644 --- a/arch/arm/include/asm/arch-vf610/imx-regs.h +++ b/arch/arm/include/asm/arch-vf610/imx-regs.h @@ -52,6 +52,7 @@ #define SAI2_BASE_ADDR (AIPS0_BASE_ADDR + 0x00031000) #define SAI3_BASE_ADDR (AIPS0_BASE_ADDR + 0x00032000) #define CRC_BASE_ADDR (AIPS0_BASE_ADDR + 0x00033000) +#define USBC0_BASE_ADDR (AIPS0_BASE_ADDR + 0x00034000) #define PDB_BASE_ADDR (AIPS0_BASE_ADDR + 0x00036000) #define PIT_BASE_ADDR (AIPS0_BASE_ADDR + 0x00037000) #define FTM0_BASE_ADDR (AIPS0_BASE_ADDR + 0x00038000) @@ -65,6 +66,8 @@ #define QSPI0_BASE_ADDR (AIPS0_BASE_ADDR + 0x00044000) #define IOMUXC_BASE_ADDR (AIPS0_BASE_ADDR + 0x00048000) #define ANADIG_BASE_ADDR (AIPS0_BASE_ADDR + 0x00050000) +#define USB_PHY0_BASE_ADDR (AIPS0_BASE_ADDR + 0x00050800) +#define USB_PHY1_BASE_ADDR (AIPS0_BASE_ADDR + 0x00050C00) #define SCSC_BASE_ADDR (AIPS0_BASE_ADDR + 0x00052000) #define ASRC_BASE_ADDR (AIPS0_BASE_ADDR + 0x00060000) #define SPDIF_BASE_ADDR (AIPS0_BASE_ADDR + 0x00061000) @@ -84,6 +87,7 @@ #define DDR_BASE_ADDR (AIPS1_BASE_ADDR + 0x0002E000) #define ESDHC0_BASE_ADDR (AIPS1_BASE_ADDR + 0x00031000) #define ESDHC1_BASE_ADDR (AIPS1_BASE_ADDR + 0x00032000) +#define USBC1_BASE_ADDR (AIPS1_BASE_ADDR + 0x00034000) #define ENET_BASE_ADDR (AIPS1_BASE_ADDR + 0x00050000) #define ENET1_BASE_ADDR (AIPS1_BASE_ADDR + 0x00051000) #define NFC_BASE_ADDR (AIPS1_BASE_ADDR + 0x00060000) diff --git a/drivers/usb/host/Makefile b/drivers/usb/host/Makefile index eb6f34b..4bd9321 100644 --- a/drivers/usb/host/Makefile +++ b/drivers/usb/host/Makefile @@ -39,6 +39,7 @@ obj-$(CONFIG_USB_EHCI_SUNXI) += ehci-sunxi.o obj-$(CONFIG_USB_EHCI_TEGRA) += ehci-tegra.o obj-$(CONFIG_USB_EHCI_UNIPHIER) += ehci-uniphier.o obj-$(CONFIG_USB_EHCI_VCT) += ehci-vct.o +obj-$(CONFIG_USB_EHCI_VF) += ehci-vf.o obj-$(CONFIG_USB_EHCI_RMOBILE) += ehci-rmobile.o obj-$(CONFIG_USB_EHCI_ZYNQ) += ehci-zynq.o diff --git a/drivers/usb/host/ehci-vf.c b/drivers/usb/host/ehci-vf.c new file mode 100644 index 0000000..4aa3b9c --- /dev/null +++ b/drivers/usb/host/ehci-vf.c @@ -0,0 +1,204 @@ +/* + * Copyright (c) 2015 Sanchayan Maity <sanchayan.maity@toradex.com> + * Copyright (C) 2015 Toradex AG + * + * Based on ehci-mx6 driver + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include <common.h> +#include <usb.h> +#include <errno.h> +#include <linux/compiler.h> +#include <asm/io.h> +#include <asm/arch/clock.h> +#include <asm/arch/imx-regs.h> +#include <asm/arch/crm_regs.h> +#include <asm/imx-common/iomux-v3.h> +#include <usb/ehci-fsl.h> + +#include "ehci.h" + +#define USB_NC_REG_OFFSET 0x00000800 +#define USBCx_CTRL_OFFSET 0x00000000 +#define USBCx_PHY_CTRL_OFFSET 0x00000018 + +#define USBPHY_CTRL 0x00000030 +#define USBPHY_CTRL_SET 0x00000034 +#define USBPHY_CTRL_CLR 0x00000038 +#define USBPHY_CTRL_TOG 0x0000003c + +#define USBPHY_PWD 0x00000000 +#define USBPHY_TX 0x00000010 +#define USBPHY_RX 0x00000020 +#define USBPHY_DEBUG 0x00000050 +#define USBPHY_CTRL_SFTRST 0x80000000 +#define USBPHY_CTRL_CLKGATE 0x40000000 +#define USBPHY_CTRL_ENUTMILEVEL3 0x00008000 +#define USBPHY_CTRL_ENUTMILEVEL2 0x00004000 +#define USBPHY_CTRL_OTG_ID 0x08000000 + +#define ANADIG_PLL_CTRL_BYPASS 0x00010000 +#define ANADIG_PLL_CTRL_ENABLE 0x00002000 +#define ANADIG_PLL_CTRL_POWER 0x00001000 +#define ANADIG_PLL_CTRL_EN_USB_CLKS 0x00000040 + +#define UCTRL_OVER_CUR_POL (1 << 8) /* OTG Polarity of Overcurrent */ +#define UCTRL_OVER_CUR_DIS (1 << 7) /* Disable OTG Overcurrent Detection */ + +/* USBCMD */ +#define UCMD_RUN_STOP (1 << 0) /* controller run/stop */ +#define UCMD_RESET (1 << 1) /* controller reset */ + +static const unsigned phy_bases[] = { + USB_PHY0_BASE_ADDR, + USB_PHY1_BASE_ADDR, +}; + +static const unsigned nc_reg_bases[] = { + USBC0_BASE_ADDR, + USBC1_BASE_ADDR, +}; + +static void usb_internal_phy_clock_gate(int index) +{ + u32 reg; + void __iomem *phy_reg; + + phy_reg = (void __iomem *)phy_bases[index]; + reg = __raw_readl(phy_reg + USBPHY_CTRL); + reg &= ~USBPHY_CTRL_CLKGATE; + __raw_writel(reg, phy_reg + USBPHY_CTRL_SET); +} + +static void usb_power_config(int index) +{ + struct anadig_reg __iomem *anadig = + (struct anadig_reg __iomem *)ANADIG_BASE_ADDR; + void __iomem *pll_ctrl; + u32 reg; + + switch (index) { + case 0: + pll_ctrl = &anadig->pll3_ctrl; + break; + case 1: + pll_ctrl = &anadig->pll7_ctrl; + break; + default: + return; + } + + reg = __raw_readl(pll_ctrl); + reg &= ~ANADIG_PLL_CTRL_BYPASS; + __raw_writel(reg, pll_ctrl); + + reg = __raw_readl(pll_ctrl); + reg |= ANADIG_PLL_CTRL_ENABLE | ANADIG_PLL_CTRL_POWER + | ANADIG_PLL_CTRL_EN_USB_CLKS; + __raw_writel(reg, pll_ctrl); +} + +static void usb_phy_enable(int index, struct usb_ehci *ehci) +{ + void __iomem *phy_reg; + void __iomem *phy_ctrl; + void __iomem *usb_cmd; + u32 val; + + phy_reg = (void __iomem *)phy_bases[index]; + phy_ctrl = (void __iomem *)(phy_reg + USBPHY_CTRL); + usb_cmd = (void __iomem *)&ehci->usbcmd; + + /* Stop then Reset */ + val = __raw_readl(usb_cmd); + val &= ~UCMD_RUN_STOP; + __raw_writel(val, usb_cmd); + while (__raw_readl(usb_cmd) & UCMD_RUN_STOP) + ; + + val = __raw_readl(usb_cmd); + val |= UCMD_RESET; + __raw_writel(val, usb_cmd); + while (__raw_readl(usb_cmd) & UCMD_RESET) + ; + + /* Reset USBPHY module */ + val = __raw_readl(phy_ctrl); + val |= USBPHY_CTRL_SFTRST; + __raw_writel(val, phy_ctrl); + udelay(10); + + /* Remove CLKGATE and SFTRST */ + val = __raw_readl(phy_ctrl); + val &= ~(USBPHY_CTRL_CLKGATE | USBPHY_CTRL_SFTRST); + __raw_writel(val, phy_ctrl); + udelay(10); + + /* Power up the PHY */ + __raw_writel(0, phy_reg + USBPHY_PWD); + /* enable FS/LS device */ + val = __raw_readl(phy_ctrl); + val |= (USBPHY_CTRL_ENUTMILEVEL2 | USBPHY_CTRL_ENUTMILEVEL3); + __raw_writel(val, phy_ctrl); +} + +static void usb_oc_config(int index) +{ + void __iomem *ctrl; + u32 val; + + ctrl = (void __iomem *)(nc_reg_bases[index] + USB_NC_REG_OFFSET + + USBCx_CTRL_OFFSET); + + val = __raw_readl(ctrl); + val |= UCTRL_OVER_CUR_POL; + __raw_writel(val, ctrl); + + val = __raw_readl(ctrl); + val |= UCTRL_OVER_CUR_DIS; + __raw_writel(val, ctrl); +} + +int ehci_hcd_init(int index, enum usb_init_type init, + struct ehci_hccr **hccr, struct ehci_hcor **hcor) +{ + struct usb_ehci *ehci; + + if (index >= ARRAY_SIZE(nc_reg_bases)) + return -EINVAL; + + if (init == USB_INIT_DEVICE && index == 1) + return -ENODEV; + if (init == USB_INIT_HOST && index == 0) + return -ENODEV; + + ehci = (struct usb_ehci *)nc_reg_bases[index]; + + usb_power_config(index); + usb_oc_config(index); + usb_internal_phy_clock_gate(index); + usb_phy_enable(index, ehci); + + *hccr = (struct ehci_hccr *)((uint32_t)&ehci->caplength); + *hcor = (struct ehci_hcor *)((uint32_t)*hccr + + HC_LENGTH(ehci_readl(&(*hccr)->cr_capbase))); + + if (init == USB_INIT_DEVICE) { + setbits_le32(&ehci->usbmode, CM_DEVICE); + __raw_writel((PORT_PTS_UTMI | PORT_PTS_PTW), &ehci->portsc); + setbits_le32(&ehci->portsc, USB_EN); + } else if (init == USB_INIT_HOST) { + setbits_le32(&ehci->usbmode, CM_HOST); + __raw_writel((PORT_PTS_UTMI | PORT_PTS_PTW), &ehci->portsc); + setbits_le32(&ehci->portsc, USB_EN); + } + + return 0; +} + +int ehci_hcd_stop(int index) +{ + return 0; +}
Signed-off-by: Sanchayan Maity <maitysanchayan@gmail.com> --- arch/arm/include/asm/arch-vf610/crm_regs.h | 14 ++ arch/arm/include/asm/arch-vf610/imx-regs.h | 4 + drivers/usb/host/Makefile | 1 + drivers/usb/host/ehci-vf.c | 204 +++++++++++++++++++++++++++++ 4 files changed, 223 insertions(+) create mode 100644 drivers/usb/host/ehci-vf.c