* [U-Boot] [PATCH] USB: ehci-mx6: Add proper IO accessors
@ 2012-03-19 22:36 Fabio Estevam
2012-03-20 6:29 ` Stefano Babic
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Fabio Estevam @ 2012-03-19 22:36 UTC (permalink / raw)
To: u-boot
Add proper IO accessors for mx6 usb registers.
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
This patch depends on "[PATCH v5] mx6: Read silicon revision from register"
drivers/usb/host/ehci-mx6.c | 13 ++++---------
1 files changed, 4 insertions(+), 9 deletions(-)
diff --git a/drivers/usb/host/ehci-mx6.c b/drivers/usb/host/ehci-mx6.c
index b7bf49d..5dec673 100644
--- a/drivers/usb/host/ehci-mx6.c
+++ b/drivers/usb/host/ehci-mx6.c
@@ -46,13 +46,9 @@
#define USBPHY_CTRL_ENUTMILEVEL3 0x00008000
#define USBPHY_CTRL_ENUTMILEVEL2 0x00004000
-#define ANADIG_USB2_CHRG_DETECT 0x00000210
#define ANADIG_USB2_CHRG_DETECT_EN_B 0x00100000
#define ANADIG_USB2_CHRG_DETECT_CHK_CHRG_B 0x00080000
-#define ANADIG_USB2_PLL_480_CTRL 0x00000020
-#define ANADIG_USB2_PLL_480_CTRL_SET 0x00000024
-#define ANADIG_USB2_PLL_480_CTRL_CLR 0x00000028
#define ANADIG_USB2_PLL_480_CTRL_BYPASS 0x00010000
#define ANADIG_USB2_PLL_480_CTRL_ENABLE 0x00002000
#define ANADIG_USB2_PLL_480_CTRL_POWER 0x00001000
@@ -77,8 +73,7 @@ static void usbh1_internal_phy_clock_gate(int on)
static void usbh1_power_config(void)
{
- void __iomem *anatop_base = (void __iomem *)ANATOP_BASE_ADDR;
-
+ struct anatop_regs *anatop = (struct anatop_regs *)ANATOP_BASE_ADDR;
/*
* Some phy and power's special controls for host1
* 1. The external charger detector needs to be disabled
@@ -89,15 +84,15 @@ static void usbh1_power_config(void)
*/
__raw_writel(ANADIG_USB2_CHRG_DETECT_EN_B |
ANADIG_USB2_CHRG_DETECT_CHK_CHRG_B,
- anatop_base + ANADIG_USB2_CHRG_DETECT);
+ &anatop->usb2_chrg_detect);
__raw_writel(ANADIG_USB2_PLL_480_CTRL_BYPASS,
- anatop_base + ANADIG_USB2_PLL_480_CTRL_CLR);
+ &anatop->usb2_pll_480_ctrl);
__raw_writel(ANADIG_USB2_PLL_480_CTRL_ENABLE |
ANADIG_USB2_PLL_480_CTRL_POWER |
ANADIG_USB2_PLL_480_CTRL_EN_USB_CLKS,
- anatop_base + ANADIG_USB2_PLL_480_CTRL_SET);
+ &anatop->usb2_pll_480_ctrl_set);
}
static int usbh1_phy_enable(void)
--
1.7.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* [U-Boot] [PATCH] USB: ehci-mx6: Add proper IO accessors
2012-03-19 22:36 [U-Boot] [PATCH] USB: ehci-mx6: Add proper IO accessors Fabio Estevam
@ 2012-03-20 6:29 ` Stefano Babic
2012-03-20 6:48 ` Marek Vasut
2012-03-22 8:30 ` Liu Hui-R64343
2012-03-22 16:09 ` Stefano Babic
2 siblings, 1 reply; 5+ messages in thread
From: Stefano Babic @ 2012-03-20 6:29 UTC (permalink / raw)
To: u-boot
On 19/03/2012 23:36, Fabio Estevam wrote:
> Add proper IO accessors for mx6 usb registers.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
> This patch depends on "[PATCH v5] mx6: Read silicon revision from register"
>
> drivers/usb/host/ehci-mx6.c | 13 ++++---------
> 1 files changed, 4 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/usb/host/ehci-mx6.c b/drivers/usb/host/ehci-mx6.c
> index b7bf49d..5dec673 100644
> --- a/drivers/usb/host/ehci-mx6.c
> +++ b/drivers/usb/host/ehci-mx6.c
> @@ -46,13 +46,9 @@
> #define USBPHY_CTRL_ENUTMILEVEL3 0x00008000
> #define USBPHY_CTRL_ENUTMILEVEL2 0x00004000
>
> -#define ANADIG_USB2_CHRG_DETECT 0x00000210
> #define ANADIG_USB2_CHRG_DETECT_EN_B 0x00100000
> #define ANADIG_USB2_CHRG_DETECT_CHK_CHRG_B 0x00080000
>
> -#define ANADIG_USB2_PLL_480_CTRL 0x00000020
> -#define ANADIG_USB2_PLL_480_CTRL_SET 0x00000024
> -#define ANADIG_USB2_PLL_480_CTRL_CLR 0x00000028
> #define ANADIG_USB2_PLL_480_CTRL_BYPASS 0x00010000
> #define ANADIG_USB2_PLL_480_CTRL_ENABLE 0x00002000
> #define ANADIG_USB2_PLL_480_CTRL_POWER 0x00001000
> @@ -77,8 +73,7 @@ static void usbh1_internal_phy_clock_gate(int on)
>
> static void usbh1_power_config(void)
> {
> - void __iomem *anatop_base = (void __iomem *)ANATOP_BASE_ADDR;
> -
> + struct anatop_regs *anatop = (struct anatop_regs *)ANATOP_BASE_ADDR;
> /*
> * Some phy and power's special controls for host1
> * 1. The external charger detector needs to be disabled
> @@ -89,15 +84,15 @@ static void usbh1_power_config(void)
> */
> __raw_writel(ANADIG_USB2_CHRG_DETECT_EN_B |
> ANADIG_USB2_CHRG_DETECT_CHK_CHRG_B,
> - anatop_base + ANADIG_USB2_CHRG_DETECT);
> + &anatop->usb2_chrg_detect);
>
> __raw_writel(ANADIG_USB2_PLL_480_CTRL_BYPASS,
> - anatop_base + ANADIG_USB2_PLL_480_CTRL_CLR);
> + &anatop->usb2_pll_480_ctrl);
>
> __raw_writel(ANADIG_USB2_PLL_480_CTRL_ENABLE |
> ANADIG_USB2_PLL_480_CTRL_POWER |
> ANADIG_USB2_PLL_480_CTRL_EN_USB_CLKS,
> - anatop_base + ANADIG_USB2_PLL_480_CTRL_SET);
> + &anatop->usb2_pll_480_ctrl_set);
> }
>
> static int usbh1_phy_enable(void)
Acked-by: Stefano Babic <sbabic@denx.de>
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 5+ messages in thread* [U-Boot] [PATCH] USB: ehci-mx6: Add proper IO accessors
2012-03-20 6:29 ` Stefano Babic
@ 2012-03-20 6:48 ` Marek Vasut
0 siblings, 0 replies; 5+ messages in thread
From: Marek Vasut @ 2012-03-20 6:48 UTC (permalink / raw)
To: u-boot
Dear Stefano Babic,
> On 19/03/2012 23:36, Fabio Estevam wrote:
> > Add proper IO accessors for mx6 usb registers.
> >
> > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> > ---
> > This patch depends on "[PATCH v5] mx6: Read silicon revision from
> > register"
> >
> > drivers/usb/host/ehci-mx6.c | 13 ++++---------
> > 1 files changed, 4 insertions(+), 9 deletions(-)
> >
> > diff --git a/drivers/usb/host/ehci-mx6.c b/drivers/usb/host/ehci-mx6.c
> > index b7bf49d..5dec673 100644
> > --- a/drivers/usb/host/ehci-mx6.c
> > +++ b/drivers/usb/host/ehci-mx6.c
> > @@ -46,13 +46,9 @@
> >
> > #define USBPHY_CTRL_ENUTMILEVEL3 0x00008000
> > #define USBPHY_CTRL_ENUTMILEVEL2 0x00004000
> >
> > -#define ANADIG_USB2_CHRG_DETECT 0x00000210
> >
> > #define ANADIG_USB2_CHRG_DETECT_EN_B 0x00100000
> > #define ANADIG_USB2_CHRG_DETECT_CHK_CHRG_B 0x00080000
> >
> > -#define ANADIG_USB2_PLL_480_CTRL 0x00000020
> > -#define ANADIG_USB2_PLL_480_CTRL_SET 0x00000024
> > -#define ANADIG_USB2_PLL_480_CTRL_CLR 0x00000028
> >
> > #define ANADIG_USB2_PLL_480_CTRL_BYPASS 0x00010000
> > #define ANADIG_USB2_PLL_480_CTRL_ENABLE 0x00002000
> > #define ANADIG_USB2_PLL_480_CTRL_POWER 0x00001000
> >
> > @@ -77,8 +73,7 @@ static void usbh1_internal_phy_clock_gate(int on)
> >
> > static void usbh1_power_config(void)
> > {
> >
> > - void __iomem *anatop_base = (void __iomem *)ANATOP_BASE_ADDR;
> > -
> > + struct anatop_regs *anatop = (struct anatop_regs *)ANATOP_BASE_ADDR;
> >
> > /*
> >
> > * Some phy and power's special controls for host1
> > * 1. The external charger detector needs to be disabled
> >
> > @@ -89,15 +84,15 @@ static void usbh1_power_config(void)
> >
> > */
> >
> > __raw_writel(ANADIG_USB2_CHRG_DETECT_EN_B |
> >
> > ANADIG_USB2_CHRG_DETECT_CHK_CHRG_B,
> >
> > - anatop_base + ANADIG_USB2_CHRG_DETECT);
> > + &anatop->usb2_chrg_detect);
> >
> > __raw_writel(ANADIG_USB2_PLL_480_CTRL_BYPASS,
> >
> > - anatop_base + ANADIG_USB2_PLL_480_CTRL_CLR);
> > + &anatop->usb2_pll_480_ctrl);
> >
> > __raw_writel(ANADIG_USB2_PLL_480_CTRL_ENABLE |
> >
> > ANADIG_USB2_PLL_480_CTRL_POWER |
> > ANADIG_USB2_PLL_480_CTRL_EN_USB_CLKS,
> >
> > - anatop_base + ANADIG_USB2_PLL_480_CTRL_SET);
> > + &anatop->usb2_pll_480_ctrl_set);
> >
> > }
> >
> > static int usbh1_phy_enable(void)
>
> Acked-by: Stefano Babic <sbabic@denx.de>
>
Agreed
Acked-by: Marek Vasut <marex@denx.de>
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] [PATCH] USB: ehci-mx6: Add proper IO accessors
2012-03-19 22:36 [U-Boot] [PATCH] USB: ehci-mx6: Add proper IO accessors Fabio Estevam
2012-03-20 6:29 ` Stefano Babic
@ 2012-03-22 8:30 ` Liu Hui-R64343
2012-03-22 16:09 ` Stefano Babic
2 siblings, 0 replies; 5+ messages in thread
From: Liu Hui-R64343 @ 2012-03-22 8:30 UTC (permalink / raw)
To: u-boot
>-----Original Message-----
From: u-boot-bounces@lists.denx.de [mailto:u-boot-bounces at lists.denx.de]
>On Behalf Of Fabio Estevam
>Sent: Tuesday, March 20, 2012 6:36 AM
>To: u-boot at lists.denx.de
>Cc: marex at denx.de; Estevam Fabio-R49496; wg at denx.de
>Subject: [U-Boot] [PATCH] USB: ehci-mx6: Add proper IO accessors
>
>Add proper IO accessors for mx6 usb registers.
>
>Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
>---
>This patch depends on "[PATCH v5] mx6: Read silicon revision from register"
>
> drivers/usb/host/ehci-mx6.c | 13 ++++---------
> 1 files changed, 4 insertions(+), 9 deletions(-)
>
>diff --git a/drivers/usb/host/ehci-mx6.c b/drivers/usb/host/ehci-mx6.c index
>b7bf49d..5dec673 100644
>--- a/drivers/usb/host/ehci-mx6.c
>+++ b/drivers/usb/host/ehci-mx6.c
>@@ -46,13 +46,9 @@
> #define USBPHY_CTRL_ENUTMILEVEL3 0x00008000
> #define USBPHY_CTRL_ENUTMILEVEL2 0x00004000
>
>-#define ANADIG_USB2_CHRG_DETECT 0x00000210
> #define ANADIG_USB2_CHRG_DETECT_EN_B 0x00100000
> #define ANADIG_USB2_CHRG_DETECT_CHK_CHRG_B 0x00080000
>
>-#define ANADIG_USB2_PLL_480_CTRL 0x00000020
>-#define ANADIG_USB2_PLL_480_CTRL_SET 0x00000024
>-#define ANADIG_USB2_PLL_480_CTRL_CLR 0x00000028
> #define ANADIG_USB2_PLL_480_CTRL_BYPASS 0x00010000
> #define ANADIG_USB2_PLL_480_CTRL_ENABLE 0x00002000
> #define ANADIG_USB2_PLL_480_CTRL_POWER 0x00001000
>@@ -77,8 +73,7 @@ static void usbh1_internal_phy_clock_gate(int on)
>
> static void usbh1_power_config(void)
> {
>- void __iomem *anatop_base = (void __iomem *)ANATOP_BASE_ADDR;
>-
>+ struct anatop_regs *anatop = (struct anatop_regs
>*)ANATOP_BASE_ADDR;
> /*
> * Some phy and power's special controls for host1
> * 1. The external charger detector needs to be disabled @@ -89,15
>+84,15 @@ static void usbh1_power_config(void)
> */
> __raw_writel(ANADIG_USB2_CHRG_DETECT_EN_B |
> ANADIG_USB2_CHRG_DETECT_CHK_CHRG_B,
>- anatop_base + ANADIG_USB2_CHRG_DETECT);
>+ &anatop->usb2_chrg_detect);
>
> __raw_writel(ANADIG_USB2_PLL_480_CTRL_BYPASS,
>- anatop_base + ANADIG_USB2_PLL_480_CTRL_CLR);
>+ &anatop->usb2_pll_480_ctrl);
>
> __raw_writel(ANADIG_USB2_PLL_480_CTRL_ENABLE |
> ANADIG_USB2_PLL_480_CTRL_POWER |
> ANADIG_USB2_PLL_480_CTRL_EN_USB_CLKS,
>- anatop_base + ANADIG_USB2_PLL_480_CTRL_SET);
>+ &anatop->usb2_pll_480_ctrl_set);
> }
>
> static int usbh1_phy_enable(void)
Acked-by: Jason Liu <r64343@freescale.com>
>--
>1.7.1
>
>_______________________________________________
>U-Boot mailing list
>U-Boot at lists.denx.de
>http://lists.denx.de/mailman/listinfo/u-boot
^ permalink raw reply [flat|nested] 5+ messages in thread* [U-Boot] [PATCH] USB: ehci-mx6: Add proper IO accessors
2012-03-19 22:36 [U-Boot] [PATCH] USB: ehci-mx6: Add proper IO accessors Fabio Estevam
2012-03-20 6:29 ` Stefano Babic
2012-03-22 8:30 ` Liu Hui-R64343
@ 2012-03-22 16:09 ` Stefano Babic
2 siblings, 0 replies; 5+ messages in thread
From: Stefano Babic @ 2012-03-22 16:09 UTC (permalink / raw)
To: u-boot
On 19/03/2012 23:36, Fabio Estevam wrote:
> Add proper IO accessors for mx6 usb registers.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
> This patch depends on "[PATCH v5] mx6: Read silicon revision from register"
>
> drivers/usb/host/ehci-mx6.c | 13 ++++---------
> 1 files changed, 4 insertions(+), 9 deletions(-)
>
Applied to u-boot-imx, thanks.
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-03-22 16:09 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-03-19 22:36 [U-Boot] [PATCH] USB: ehci-mx6: Add proper IO accessors Fabio Estevam
2012-03-20 6:29 ` Stefano Babic
2012-03-20 6:48 ` Marek Vasut
2012-03-22 8:30 ` Liu Hui-R64343
2012-03-22 16:09 ` Stefano Babic
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox