| Message ID | 1394618518-13803-4-git-send-email-denis@eukrea.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
> This adds the i.MX25 and the i.MX35 support in the ChipIdea usbmisc > driver. > > The i.MX25 and i.MX35 usb controllers are similar enough to be able to > use the same code. > > Signed-off-by: Denis Carikli <denis@eukrea.com> > --- > Changelog v2->v3: > - Add a commit log > > Changelog v1->v2: > - converted two remaining defines to BIT() > - Removed a variable declaration that was not used in usbmisc_imx25_init > --- > drivers/usb/chipidea/usbmisc_imx.c | 58 > ++++++++++++++++++++++++++++++++++++ > 1 file changed, 58 insertions(+) > > diff --git a/drivers/usb/chipidea/usbmisc_imx.c > b/drivers/usb/chipidea/usbmisc_imx.c > index cd061ab..3523c09 100644 > --- a/drivers/usb/chipidea/usbmisc_imx.c > +++ b/drivers/usb/chipidea/usbmisc_imx.c > @@ -21,6 +21,26 @@ > #define MX25_USB_PHY_CTRL_OFFSET 0x08 > #define MX25_BM_EXTERNAL_VBUS_DIVIDER BIT(23) > > +#define MXC_EHCI_INTERFACE_SINGLE_UNI (2 << 0) > +#define MXC_EHCI_INTERFACE_DIFF_UNI (0 << 0) > +#define MXC_EHCI_INTERFACE_MASK (0xf) > + Have a check for my comment for above Macros please. Peter > +#define MX25_OTG_SIC_SHIFT 29 > +#define MX25_OTG_SIC_MASK (0x3 << MX25_OTG_SIC_SHIFT) > +#define MX25_OTG_PM_BIT BIT(24) > +#define MX25_OTG_PP_BIT BIT(11) > +#define MX25_OTG_OCPOL_BIT BIT(3) > + > +#define MX25_H1_SIC_SHIFT 21 > +#define MX25_H1_SIC_MASK (0x3 << MX25_H1_SIC_SHIFT) > +#define MX25_H1_PP_BIT BIT(18) > +#define MX25_H1_PM_BIT BIT(16) > +#define MX25_H1_IPPUE_UP_BIT BIT(7) > +#define MX25_H1_IPPUE_DOWN_BIT BIT(6) > +#define MX25_H1_TLL_BIT BIT(5) > +#define MX25_H1_USBTE_BIT BIT(4) > +#define MX25_H1_OCPOL_BIT BIT(2) > + > #define MX27_H1_PM_BIT BIT(8) > #define MX27_H2_PM_BIT BIT(16) > #define MX27_OTG_PM_BIT BIT(24) > @@ -50,6 +70,39 @@ struct imx_usbmisc { > > static struct imx_usbmisc *usbmisc; > > +static int usbmisc_imx25_init(struct imx_usbmisc_data *data) { > + unsigned long flags; > + u32 val = 0; > + > + if (data->index > 1) > + return -EINVAL; > + > + spin_lock_irqsave(&usbmisc->lock, flags); > + switch (data->index) { > + case 0: > + val = readl(usbmisc->base); > + val &= ~(MX25_OTG_SIC_MASK | MX25_OTG_PP_BIT); > + val |= (MXC_EHCI_INTERFACE_DIFF_UNI & MXC_EHCI_INTERFACE_MASK) > << MX25_OTG_SIC_SHIFT; > + val |= (MX25_OTG_PM_BIT | MX25_OTG_OCPOL_BIT); > + writel(val, usbmisc->base); > + break; > + case 1: > + val = readl(usbmisc->base); > + val &= ~(MX25_H1_SIC_MASK | MX25_H1_PP_BIT | > MX25_H1_IPPUE_UP_BIT); > + val |= (MXC_EHCI_INTERFACE_SINGLE_UNI & > MXC_EHCI_INTERFACE_MASK) << MX25_H1_SIC_SHIFT; > + val |= (MX25_H1_PM_BIT | MX25_H1_OCPOL_BIT | MX25_H1_TLL_BIT > | > + MX25_H1_USBTE_BIT | MX25_H1_IPPUE_DOWN_BIT); > + > + writel(val, usbmisc->base); > + > + break; > + } > + spin_unlock_irqrestore(&usbmisc->lock, flags); > + > + return 0; > +} > + > static int usbmisc_imx25_post(struct imx_usbmisc_data *data) { > void __iomem *reg; > @@ -159,6 +212,7 @@ static int usbmisc_imx6q_init(struct imx_usbmisc_data > *data) } > > static const struct usbmisc_ops imx25_usbmisc_ops = { > + .init = usbmisc_imx25_init, > .post = usbmisc_imx25_post, > }; > > @@ -200,6 +254,10 @@ static const struct of_device_id usbmisc_imx_dt_ids[] > = { > .data = &imx25_usbmisc_ops, > }, > { > + .compatible = "fsl,imx35-usbmisc", > + .data = &imx25_usbmisc_ops, > + }, > + { > .compatible = "fsl,imx27-usbmisc", > .data = &imx27_usbmisc_ops, > }, > -- > 1.7.9.5 > >
diff --git a/drivers/usb/chipidea/usbmisc_imx.c b/drivers/usb/chipidea/usbmisc_imx.c index cd061ab..3523c09 100644 --- a/drivers/usb/chipidea/usbmisc_imx.c +++ b/drivers/usb/chipidea/usbmisc_imx.c @@ -21,6 +21,26 @@ #define MX25_USB_PHY_CTRL_OFFSET 0x08 #define MX25_BM_EXTERNAL_VBUS_DIVIDER BIT(23) +#define MXC_EHCI_INTERFACE_SINGLE_UNI (2 << 0) +#define MXC_EHCI_INTERFACE_DIFF_UNI (0 << 0) +#define MXC_EHCI_INTERFACE_MASK (0xf) + +#define MX25_OTG_SIC_SHIFT 29 +#define MX25_OTG_SIC_MASK (0x3 << MX25_OTG_SIC_SHIFT) +#define MX25_OTG_PM_BIT BIT(24) +#define MX25_OTG_PP_BIT BIT(11) +#define MX25_OTG_OCPOL_BIT BIT(3) + +#define MX25_H1_SIC_SHIFT 21 +#define MX25_H1_SIC_MASK (0x3 << MX25_H1_SIC_SHIFT) +#define MX25_H1_PP_BIT BIT(18) +#define MX25_H1_PM_BIT BIT(16) +#define MX25_H1_IPPUE_UP_BIT BIT(7) +#define MX25_H1_IPPUE_DOWN_BIT BIT(6) +#define MX25_H1_TLL_BIT BIT(5) +#define MX25_H1_USBTE_BIT BIT(4) +#define MX25_H1_OCPOL_BIT BIT(2) + #define MX27_H1_PM_BIT BIT(8) #define MX27_H2_PM_BIT BIT(16) #define MX27_OTG_PM_BIT BIT(24) @@ -50,6 +70,39 @@ struct imx_usbmisc { static struct imx_usbmisc *usbmisc; +static int usbmisc_imx25_init(struct imx_usbmisc_data *data) +{ + unsigned long flags; + u32 val = 0; + + if (data->index > 1) + return -EINVAL; + + spin_lock_irqsave(&usbmisc->lock, flags); + switch (data->index) { + case 0: + val = readl(usbmisc->base); + val &= ~(MX25_OTG_SIC_MASK | MX25_OTG_PP_BIT); + val |= (MXC_EHCI_INTERFACE_DIFF_UNI & MXC_EHCI_INTERFACE_MASK) << MX25_OTG_SIC_SHIFT; + val |= (MX25_OTG_PM_BIT | MX25_OTG_OCPOL_BIT); + writel(val, usbmisc->base); + break; + case 1: + val = readl(usbmisc->base); + val &= ~(MX25_H1_SIC_MASK | MX25_H1_PP_BIT | MX25_H1_IPPUE_UP_BIT); + val |= (MXC_EHCI_INTERFACE_SINGLE_UNI & MXC_EHCI_INTERFACE_MASK) << MX25_H1_SIC_SHIFT; + val |= (MX25_H1_PM_BIT | MX25_H1_OCPOL_BIT | MX25_H1_TLL_BIT | + MX25_H1_USBTE_BIT | MX25_H1_IPPUE_DOWN_BIT); + + writel(val, usbmisc->base); + + break; + } + spin_unlock_irqrestore(&usbmisc->lock, flags); + + return 0; +} + static int usbmisc_imx25_post(struct imx_usbmisc_data *data) { void __iomem *reg; @@ -159,6 +212,7 @@ static int usbmisc_imx6q_init(struct imx_usbmisc_data *data) } static const struct usbmisc_ops imx25_usbmisc_ops = { + .init = usbmisc_imx25_init, .post = usbmisc_imx25_post, }; @@ -200,6 +254,10 @@ static const struct of_device_id usbmisc_imx_dt_ids[] = { .data = &imx25_usbmisc_ops, }, { + .compatible = "fsl,imx35-usbmisc", + .data = &imx25_usbmisc_ops, + }, + { .compatible = "fsl,imx27-usbmisc", .data = &imx27_usbmisc_ops, },
This adds the i.MX25 and the i.MX35 support in the ChipIdea usbmisc driver. The i.MX25 and i.MX35 usb controllers are similar enough to be able to use the same code. Signed-off-by: Denis Carikli <denis@eukrea.com> --- Changelog v2->v3: - Add a commit log Changelog v1->v2: - converted two remaining defines to BIT() - Removed a variable declaration that was not used in usbmisc_imx25_init --- drivers/usb/chipidea/usbmisc_imx.c | 58 ++++++++++++++++++++++++++++++++++++ 1 file changed, 58 insertions(+)