[PATCH] unbreak ehci-mxc on otg port of i.MX27

Sergei Shtylyov sshtylyov at mvista.com
Thu Jan 13 07:17:43 EST 2011


Hello.

On 13-01-2011 12:06, Eric Bénard wrote:

> commit 711669e5b80b6f2d88f61ed8a9681f83d8cbd201 fixed port 0 support
> for i.MX51 but broke it for (at least) i.MX27 which doesn't have
> a usb_phy1 clock but has a pdev->id 0.

> Signed-off-by: Eric Bénard<eric at eukrea.com>
> Cc: Arnaud Patard<arnaud.patard at rtp-net.org>
> Cc: Sascha Hauer<s.hauer at pengutronix.de>
> ---
>   drivers/usb/host/ehci-mxc.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)

> diff --git a/drivers/usb/host/ehci-mxc.c b/drivers/usb/host/ehci-mxc.c
> index fa59b26..18eb597 100644
> --- a/drivers/usb/host/ehci-mxc.c
> +++ b/drivers/usb/host/ehci-mxc.c
> @@ -178,7 +178,7 @@ static int ehci_mxc_drv_probe(struct platform_device *pdev)
>   	}
>
>   	/* "dr" device has its own clock */
> -	if (pdev->id == 0) {
> +	if ((cpu_is_mx51()) & (pdev->id == 0)) {

    Why enclose the function call in parens at all? You also don't need parens 
around 'pdev->id == 0'...

WBR, Sergei



More information about the linux-arm-kernel mailing list