From patchwork Wed Mar 20 17:11:47 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Felipe Balbi X-Patchwork-Id: 229436 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 1453C2C00AF for ; Thu, 21 Mar 2013 04:12:11 +1100 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757617Ab3CTRMH (ORCPT ); Wed, 20 Mar 2013 13:12:07 -0400 Received: from comal.ext.ti.com ([198.47.26.152]:56039 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755601Ab3CTRMF (ORCPT ); Wed, 20 Mar 2013 13:12:05 -0400 Received: from dlelxv30.itg.ti.com ([172.17.2.17]) by comal.ext.ti.com (8.13.7/8.13.7) with ESMTP id r2KHC4BB007924; Wed, 20 Mar 2013 12:12:04 -0500 Received: from DLEE70.ent.ti.com (dlee70.ent.ti.com [157.170.170.113]) by dlelxv30.itg.ti.com (8.13.8/8.13.8) with ESMTP id r2KHC4nl022678; Wed, 20 Mar 2013 12:12:04 -0500 Received: from dlelxv22.itg.ti.com (172.17.1.197) by DLEE70.ent.ti.com (157.170.170.113) with Microsoft SMTP Server id 14.2.342.3; Wed, 20 Mar 2013 12:12:04 -0500 Received: from localhost (h79-2.vpn.ti.com [172.24.79.2]) by dlelxv22.itg.ti.com (8.13.8/8.13.8) with ESMTP id r2KHC2lG011941; Wed, 20 Mar 2013 12:12:03 -0500 Date: Wed, 20 Mar 2013 19:11:47 +0200 From: Felipe Balbi To: Stephen Warren CC: , Venu Byravarasu , USB list , "linux-next@vger.kernel.org" , "linux-tegra@vger.kernel.org" Subject: Re: USB-related build errors on Tegra in next-20130319 Message-ID: <20130320171147.GM1567@arwen.pp.htv.fi> Reply-To: References: <51489251.5040504@wwwdotorg.org> <20130320122031.GI29659@arwen.pp.htv.fi> <5149E62F.9090705@wwwdotorg.org> <20130320164553.GL1567@arwen.pp.htv.fi> <5149E925.7030302@wwwdotorg.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <5149E925.7030302@wwwdotorg.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-tegra-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-tegra@vger.kernel.org Hi, On Wed, Mar 20, 2013 at 10:51:49AM -0600, Stephen Warren wrote: > On 03/20/2013 10:45 AM, Felipe Balbi wrote: > > Hi, > > > > On Wed, Mar 20, 2013 at 10:39:11AM -0600, Stephen Warren wrote: > >> On 03/20/2013 06:20 AM, Felipe Balbi wrote: > ... > >> The patch you gave fixes the Kconfig warnings, but not the build > >> failure. > ... > >> For now, I think the correct solution is to have USB_EHCI_TEGRA > >> select USB_PHY. I validated that solves the build problem, and > >> has the bonus of not needing any changes to tegra_defconfig. > ... > > Alright, do you want to send that patch ? My bad that I missed > > that build error... > > Sure. Do you want to roll it into your patch and repost it, or me to > send a separate patch? updated patch below: From c2b3334a624e4231952ab050e29445b8c62f662d Mon Sep 17 00:00:00 2001 From: Felipe Balbi Date: Wed, 20 Mar 2013 14:18:28 +0200 Subject: [PATCH] arm: tegra: fix Kconfig select clauses USB_ULPI and USB_ULPI_VIEWPORT shouldn't really be selected directly by anyone, but since Tegra still needs some time before turning ulpi viewport into a proper PHY driver, we need to keep the selects in place. This patch just fixes the conditional select so that it will continue to build after merging the latest PHY layer changes. Signed-off-by: Felipe Balbi Acked-by: Stephen Warren Tested-by: Stephen Warren --- arch/arm/mach-tegra/Kconfig | 8 ++++---- drivers/usb/host/Kconfig | 1 + 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/arch/arm/mach-tegra/Kconfig b/arch/arm/mach-tegra/Kconfig index d1c4893..dbc653e 100644 --- a/arch/arm/mach-tegra/Kconfig +++ b/arch/arm/mach-tegra/Kconfig @@ -18,8 +18,8 @@ config ARCH_TEGRA_2x_SOC select PL310_ERRATA_727915 if CACHE_L2X0 select PL310_ERRATA_769419 if CACHE_L2X0 select USB_ARCH_HAS_EHCI if USB_SUPPORT - select USB_ULPI if USB - select USB_ULPI_VIEWPORT if USB_SUPPORT + select USB_ULPI if USB_PHY + select USB_ULPI_VIEWPORT if USB_PHY help Support for NVIDIA Tegra AP20 and T20 processors, based on the ARM CortexA9MP CPU and the ARM PL310 L2 cache controller @@ -37,8 +37,8 @@ config ARCH_TEGRA_3x_SOC select PINCTRL_TEGRA30 select PL310_ERRATA_769419 if CACHE_L2X0 select USB_ARCH_HAS_EHCI if USB_SUPPORT - select USB_ULPI if USB - select USB_ULPI_VIEWPORT if USB_SUPPORT + select USB_ULPI if USB_PHY + select USB_ULPI_VIEWPORT if USB_PHY help Support for NVIDIA Tegra T30 processor family, based on the ARM CortexA9MP CPU and the ARM PL310 L2 cache controller diff --git a/drivers/usb/host/Kconfig b/drivers/usb/host/Kconfig index ba1347c..1b58587 100644 --- a/drivers/usb/host/Kconfig +++ b/drivers/usb/host/Kconfig @@ -179,6 +179,7 @@ config USB_EHCI_TEGRA boolean "NVIDIA Tegra HCD support" depends on USB_EHCI_HCD && ARCH_TEGRA select USB_EHCI_ROOT_HUB_TT + select USB_PHY help This driver enables support for the internal USB Host Controllers found in NVIDIA Tegra SoCs. The controllers are EHCI compliant.