From: u.kleine-koenig@pengutronix.de (Uwe Kleine-König)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] plat-mxc/ehci.c: fix compile breakage
Date: Sat, 27 Nov 2010 20:15:00 +0100 [thread overview]
Message-ID: <20101127191500.GC25293@pengutronix.de> (raw)
In-Reply-To: <1290845738-21586-1-git-send-email-eric@eukrea.com>
On Sat, Nov 27, 2010 at 09:15:38AM +0100, Eric B?nard wrote:
> commits 2eb42d5c287f5e883a4b3ebe668ba880caa351e5 and
> 9e1dde33876ba83ad586c336647fff133d0f5472 renamed some defines
> but didn't fix all the places where these defines are used
> leading to a compile failure for USB on i.MX31, 35 and 27.
>
> Signed-off-by: Eric B?nard <eric@eukrea.com>
grml, it seems the defconfigs I test with have support for ehci and
usb-gadget disabled. :-|
You probably need to split your patch.
Other than that
Acked-by: Uwe Kleine-K?nig <u.kleine-koenig@pengutronix.de>
and thanks
Uwe
> ---
> arch/arm/plat-mxc/ehci.c | 12 ++++++------
> drivers/usb/gadget/fsl_mxc_udc.c | 4 ++--
> 2 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/arch/arm/plat-mxc/ehci.c b/arch/arm/plat-mxc/ehci.c
> index 4bac3d5..b79c809 100644
> --- a/arch/arm/plat-mxc/ehci.c
> +++ b/arch/arm/plat-mxc/ehci.c
> @@ -115,7 +115,7 @@ int mxc_initialize_usb_hw(int port, unsigned int flags)
> #endif /* if defined(CONFIG_SOC_IMX25) */
> #if defined(CONFIG_ARCH_MX3)
> if (cpu_is_mx31()) {
> - v = readl(MX31_IO_ADDRESS(MX31_OTG_BASE_ADDR +
> + v = readl(MX31_IO_ADDRESS(MX31_USB_BASE_ADDR +
> USBCTRL_OTGBASE_OFFSET));
>
> switch (port) {
> @@ -153,13 +153,13 @@ int mxc_initialize_usb_hw(int port, unsigned int flags)
> return -EINVAL;
> }
>
> - writel(v, MX31_IO_ADDRESS(MX31_OTG_BASE_ADDR +
> + writel(v, MX31_IO_ADDRESS(MX31_USB_BASE_ADDR +
> USBCTRL_OTGBASE_OFFSET));
> return 0;
> }
>
> if (cpu_is_mx35()) {
> - v = readl(MX35_IO_ADDRESS(MX35_OTG_BASE_ADDR +
> + v = readl(MX35_IO_ADDRESS(MX35_USB_BASE_ADDR +
> USBCTRL_OTGBASE_OFFSET));
>
> switch (port) {
> @@ -196,7 +196,7 @@ int mxc_initialize_usb_hw(int port, unsigned int flags)
> return -EINVAL;
> }
>
> - writel(v, MX35_IO_ADDRESS(MX35_OTG_BASE_ADDR +
> + writel(v, MX35_IO_ADDRESS(MX35_USB_BASE_ADDR +
> USBCTRL_OTGBASE_OFFSET));
> return 0;
> }
> @@ -206,7 +206,7 @@ int mxc_initialize_usb_hw(int port, unsigned int flags)
> /* On i.MX27 we can use the i.MX31 USBCTRL bits, they
> * are identical
> */
> - v = readl(MX27_IO_ADDRESS(MX27_OTG_BASE_ADDR +
> + v = readl(MX27_IO_ADDRESS(MX27_USB_BASE_ADDR +
> USBCTRL_OTGBASE_OFFSET));
> switch (port) {
> case 0: /* OTG port */
> @@ -241,7 +241,7 @@ int mxc_initialize_usb_hw(int port, unsigned int flags)
> default:
> return -EINVAL;
> }
> - writel(v, MX27_IO_ADDRESS(MX27_OTG_BASE_ADDR +
> + writel(v, MX27_IO_ADDRESS(MX27_USB_BASE_ADDR +
> USBCTRL_OTGBASE_OFFSET));
> return 0;
> }
> diff --git a/drivers/usb/gadget/fsl_mxc_udc.c b/drivers/usb/gadget/fsl_mxc_udc.c
> index 5bdbfe6..77b1eb5 100644
> --- a/drivers/usb/gadget/fsl_mxc_udc.c
> +++ b/drivers/usb/gadget/fsl_mxc_udc.c
> @@ -93,9 +93,9 @@ void fsl_udc_clk_finalize(struct platform_device *pdev)
>
> /* workaround ENGcm09152 for i.MX35 */
> if (pdata->workaround & FLS_USB2_WORKAROUND_ENGCM09152) {
> - v = readl(MX35_IO_ADDRESS(MX35_OTG_BASE_ADDR +
> + v = readl(MX35_IO_ADDRESS(MX35_USB_BASE_ADDR +
> USBPHYCTRL_OTGBASE_OFFSET));
> - writel(v | USBPHYCTRL_EVDO, MX35_IO_ADDRESS(MX35_OTG_BASE_ADDR +
> + writel(v | USBPHYCTRL_EVDO, MX35_IO_ADDRESS(MX35_USB_BASE_ADDR +
> USBPHYCTRL_OTGBASE_OFFSET));
> }
> #endif
> --
> 1.7.3.2
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
--
Pengutronix e.K. | Uwe Kleine-K?nig |
Industrial Linux Solutions | http://www.pengutronix.de/ |
prev parent reply other threads:[~2010-11-27 19:15 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-11-27 8:15 [PATCH] plat-mxc/ehci.c: fix compile breakage Eric Bénard
2010-11-27 19:15 ` Uwe Kleine-König [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20101127191500.GC25293@pengutronix.de \
--to=u.kleine-koenig@pengutronix.de \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).