From patchwork Thu May 16 16:05:46 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tomi Valkeinen X-Patchwork-Id: 2579041 Return-Path: X-Original-To: patchwork-linux-omap@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork2.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork2.kernel.org (Postfix) with ESMTP id 67870DFB7B for ; Thu, 16 May 2013 16:05:54 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752476Ab3EPQFw (ORCPT ); Thu, 16 May 2013 12:05:52 -0400 Received: from mail-la0-f49.google.com ([209.85.215.49]:39443 "EHLO mail-la0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752163Ab3EPQFv (ORCPT ); Thu, 16 May 2013 12:05:51 -0400 Received: by mail-la0-f49.google.com with SMTP id fp13so2534028lab.8 for ; Thu, 16 May 2013 09:05:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=x-received:sender:message-id:date:from:user-agent:mime-version:to :cc:subject:references:in-reply-to:content-type :content-transfer-encoding; bh=aNplWXGZzfUX+XTNVSYjVWKiPzv0eSKccwm5f9wUIFA=; b=vLTBgjURmribLSgZ/kGmUrNx7kvhGXFK+hdPW1tY9tI+70xJA2knD/QRnl9GxQ1rhr 8IZRnJ9SiIlHxWCswsM9iX6GLzZo4b9T3F3qfD1kmCtCF9Raai7O30ir3YoHqFi4iv5c r+twme4r43tXsJkX0QrPJ+mdFerHC12QFpxO97zni8LB/VXWljBc+lDMdwJILAivzhbG yirm2nfCZdc650VVQ/BWXRU/a+yFpbKZWadxeCIBsquIrSGNEKYZGSca/zKG8X2p4ub+ T7hd5IQ2aQXq3HYKQwlwW2ugymrkGNWbi5dqpps0x5Xk+dueNg5J2pn+sV3F6JRbpJmb wusw== X-Received: by 10.112.126.9 with SMTP id mu9mr20040774lbb.99.1368720349643; Thu, 16 May 2013 09:05:49 -0700 (PDT) Received: from [192.168.1.3] (a91-156-160-115.elisa-laajakaista.fi. [91.156.160.115]) by mx.google.com with ESMTPSA id ky7sm3359804lbc.16.2013.05.16.09.05.47 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Thu, 16 May 2013 09:05:48 -0700 (PDT) Message-ID: <519503DA.5070303@iki.fi> Date: Thu, 16 May 2013 19:05:46 +0300 From: Tomi Valkeinen User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130510 Thunderbird/17.0.6 MIME-Version: 1.0 To: Tony Lindgren CC: Kishon Vijay Abraham I , linux@arm.linux.org.uk, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, balbi@ti.com Subject: Re: [PATCH] ARM: OMAP4: change the device names in usb_bind_phy References: <1366697656-14315-1-git-send-email-kishon@ti.com> <51936946.3030507@iki.fi> <20130516155857.GV5600@atomide.com> In-Reply-To: <20130516155857.GV5600@atomide.com> Sender: linux-omap-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-omap@vger.kernel.org On 16/05/13 18:58, Tony Lindgren wrote: > * Tomi Valkeinen [130515 03:59]: >> On 23/04/13 09:14, Kishon Vijay Abraham I wrote: >>> After the device names are created using PLATFORM_DEVID_AUTO, the old >>> device names given in usb_bind_phy are no longer valid causing the musb >>> controller not to get the phy reference. Updated the usb_bind_phy with >>> the new device names to get MUSB functional in omap4 panda. >>> >>> Signed-off-by: Kishon Vijay Abraham I >>> --- >>> Tested in OMAP4 PANDA. >>> arch/arm/mach-omap2/board-4430sdp.c | 2 +- >>> arch/arm/mach-omap2/board-omap4panda.c | 2 +- >>> 2 files changed, 2 insertions(+), 2 deletions(-) >>> >>> diff --git a/arch/arm/mach-omap2/board-4430sdp.c b/arch/arm/mach-omap2/board-4430sdp.c >>> index 00d7290..56a9a4f 100644 >>> --- a/arch/arm/mach-omap2/board-4430sdp.c >>> +++ b/arch/arm/mach-omap2/board-4430sdp.c >>> @@ -730,7 +730,7 @@ static void __init omap_4430sdp_init(void) >>> omap4_sdp4430_wifi_init(); >>> omap4_twl6030_hsmmc_init(mmc); >>> >>> - usb_bind_phy("musb-hdrc.0.auto", 0, "omap-usb2.1.auto"); >>> + usb_bind_phy("musb-hdrc.2.auto", 0, "omap-usb2.3.auto"); >>> usb_musb_init(&musb_board_data); >>> >>> status = omap_ethernet_init(); >> >> I'm seeing >> >> [ 2.190155] unable to find transceiver >> [ 2.190155] HS USB OTG: no transceiver configured >> [ 2.190155] musb-hdrc musb-hdrc.0.auto: musb_init_controller failed >> with status -517 >> [ 2.207458] platform musb-hdrc.0.auto: Driver musb-hdrc requests >> probe deferral >> >> on 4430sdp with v3.10-rc1. Does that mean that the musb-hdrc.0.auto was >> indeed correct, and the new value of musb-hdrc.2.auto is not? > > Just checking.. Do you have CONFIG_OMAP_OCP2SCP=y in your .config? Sounds > like the some transceivers should depend on that for omap4. Yes, I have OCP2SCP=y. >> The musb-hdrc id is wrong on overo also. > > Hmm has there been a fix posted for that? I couldn't find with a quick look. We debugged and discussed this on an irc channel with Kishon, who said he'll send a patch. Changing the musb-hdrc ID on overo fixed the issue, and it looks very similar to the error on 4430sdp. The overo fix was just: Is that ID "randomly" chosen? Doesn't that mean that it'll just get broken every now and then? Tomi --- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/arch/arm/mach-omap2/board-overo.c b/arch/arm/mach-omap2/board-overo.c index 4ca6b68..a496774 100644 --- a/arch/arm/mach-omap2/board-overo.c +++ b/arch/arm/mach-omap2/board-overo.c @@ -472,7 +472,7 @@ static void __init overo_init(void) mt46h32m32lf6_sdrc_params); board_nand_init(overo_nand_partitions, ARRAY_SIZE(overo_nand_partitions), NAND_CS, 0, NULL); - usb_bind_phy("musb-hdrc.0.auto", 0, "twl4030_usb"); + usb_bind_phy("musb-hdrc.1.auto", 0, "twl4030_usb"); usb_musb_init(NULL); usbhs_init_phys(phy_data, ARRAY_SIZE(phy_data));