- * [PATCH v3 1/5] ARM: tegra: do not hide dma declarations
  2012-03-02 22:24 ` [PATCH v3 0/5] " Alan Ott
@ 2012-03-02 22:25   ` Alan Ott
  2012-03-02 22:25   ` [PATCH v3 2/5] ARM: tegra: select CPU_FREQ_TABLE Alan Ott
                     ` (4 subsequent siblings)
  5 siblings, 0 replies; 13+ messages in thread
From: Alan Ott @ 2012-03-02 22:25 UTC (permalink / raw)
  To: linux-arm-kernel
From: Arnd Bergmann <arnd@arndb.de>
The declarations are actually required for the device
definitions, and are still valid even if the dma controller
is disabled:
arch/arm/mach-tegra/devices.c:559:12: error: 'TEGRA_DMA_REQ_SEL_I2S_1' undeclared here (not in a function)
arch/arm/mach-tegra/devices.c:577:12: error: 'TEGRA_DMA_REQ_SEL_I2S2_1' undeclared here (not in a function)
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Alan Ott <alan@signal11.us>
---
 arch/arm/mach-tegra/include/mach/dma.h |    4 ----
 1 files changed, 0 insertions(+), 4 deletions(-)
diff --git a/arch/arm/mach-tegra/include/mach/dma.h b/arch/arm/mach-tegra/include/mach/dma.h
index 3c93390..9077092 100644
--- a/arch/arm/mach-tegra/include/mach/dma.h
+++ b/arch/arm/mach-tegra/include/mach/dma.h
@@ -51,8 +51,6 @@
 #define TEGRA_DMA_REQ_SEL_OWR			25
 #define TEGRA_DMA_REQ_SEL_INVALID		31
 
-#if defined(CONFIG_TEGRA_SYSTEM_DMA)
-
 struct tegra_dma_req;
 struct tegra_dma_channel;
 
@@ -151,5 +149,3 @@ void tegra_dma_free_channel(struct tegra_dma_channel *ch);
 int __init tegra_dma_init(void);
 
 #endif
-
-#endif
-- 
1.7.0.4
^ permalink raw reply related	[flat|nested] 13+ messages in thread
- * [PATCH v3 2/5] ARM: tegra: select CPU_FREQ_TABLE
  2012-03-02 22:24 ` [PATCH v3 0/5] " Alan Ott
  2012-03-02 22:25   ` [PATCH v3 1/5] ARM: tegra: do not hide dma declarations Alan Ott
@ 2012-03-02 22:25   ` Alan Ott
  2012-03-02 22:25   ` [PATCH v3 3/5] ARM: tegra: build localtimer support only when needed Alan Ott
                     ` (3 subsequent siblings)
  5 siblings, 0 replies; 13+ messages in thread
From: Alan Ott @ 2012-03-02 22:25 UTC (permalink / raw)
  To: linux-arm-kernel
From: Arnd Bergmann <arnd@arndb.de>
The tegra cpufreq implementation relies on the cpu_freq_table
code, so make sure that this is always there when needed.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Alan Ott <alan@signal11.us>
---
 arch/arm/mach-tegra/Kconfig |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-tegra/Kconfig b/arch/arm/mach-tegra/Kconfig
index 373652d..60f3e4b 100644
--- a/arch/arm/mach-tegra/Kconfig
+++ b/arch/arm/mach-tegra/Kconfig
@@ -10,6 +10,7 @@ config ARCH_TEGRA_2x_SOC
 	select USB_ARCH_HAS_EHCI if USB_SUPPORT
 	select USB_ULPI if USB_SUPPORT
 	select USB_ULPI_VIEWPORT if USB_SUPPORT
+	select CPU_FREQ_TABLE if CPU_FREQ
 	help
 	  Support for NVIDIA Tegra AP20 and T20 processors, based on the
 	  ARM CortexA9MP CPU and the ARM PL310 L2 cache controller
@@ -23,6 +24,7 @@ config ARCH_TEGRA_3x_SOC
 	select USB_ULPI if USB_SUPPORT
 	select USB_ULPI_VIEWPORT if USB_SUPPORT
 	select USE_OF
+	select CPU_FREQ_TABLE if CPU_FREQ
 	help
 	  Support for NVIDIA Tegra T30 processor family, based on the
 	  ARM CortexA9MP CPU and the ARM PL310 L2 cache controller
-- 
1.7.0.4
^ permalink raw reply related	[flat|nested] 13+ messages in thread
- * [PATCH v3 3/5] ARM: tegra: build localtimer support only when needed
  2012-03-02 22:24 ` [PATCH v3 0/5] " Alan Ott
  2012-03-02 22:25   ` [PATCH v3 1/5] ARM: tegra: do not hide dma declarations Alan Ott
  2012-03-02 22:25   ` [PATCH v3 2/5] ARM: tegra: select CPU_FREQ_TABLE Alan Ott
@ 2012-03-02 22:25   ` Alan Ott
  2012-03-02 22:26   ` [PATCH v3 4/5] ARM: tegra: select USB_ULPI if USB is selected Alan Ott
                     ` (2 subsequent siblings)
  5 siblings, 0 replies; 13+ messages in thread
From: Alan Ott @ 2012-03-02 22:25 UTC (permalink / raw)
  To: linux-arm-kernel
From: Arnd Bergmann <arnd@arndb.de>
It is possible to build a tegra kernel without localtimer
support, so the tegra specific parts should only be built
when that is indeed enabled.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Alan Ott <alan@signal11.us>
---
 arch/arm/mach-tegra/Makefile |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-tegra/Makefile b/arch/arm/mach-tegra/Makefile
index e120ff5..829066f 100644
--- a/arch/arm/mach-tegra/Makefile
+++ b/arch/arm/mach-tegra/Makefile
@@ -13,7 +13,8 @@ obj-$(CONFIG_ARCH_TEGRA_2x_SOC)		+= tegra2_emc.o
 obj-$(CONFIG_ARCH_TEGRA_2x_SOC)		+= pinmux-tegra20-tables.o
 obj-$(CONFIG_ARCH_TEGRA_3x_SOC)		+= pinmux-tegra30-tables.o
 obj-$(CONFIG_ARCH_TEGRA_3x_SOC)		+= board-dt-tegra30.o
-obj-$(CONFIG_SMP)                       += platsmp.o localtimer.o headsmp.o
+obj-$(CONFIG_SMP)			+= platsmp.o headsmp.o
+obj-$(CONFIG_LOCAL_TIMERS)		+= localtimer.o
 obj-$(CONFIG_HOTPLUG_CPU)               += hotplug.o
 obj-$(CONFIG_TEGRA_SYSTEM_DMA)		+= dma.o
 obj-$(CONFIG_CPU_FREQ)                  += cpu-tegra.o
-- 
1.7.0.4
^ permalink raw reply related	[flat|nested] 13+ messages in thread
- * [PATCH v3 4/5] ARM: tegra: select USB_ULPI if USB is selected
  2012-03-02 22:24 ` [PATCH v3 0/5] " Alan Ott
                     ` (2 preceding siblings ...)
  2012-03-02 22:25   ` [PATCH v3 3/5] ARM: tegra: build localtimer support only when needed Alan Ott
@ 2012-03-02 22:26   ` Alan Ott
  2012-03-18 17:36     ` Olof Johansson
  2012-03-02 22:26   ` [PATCH v3 5/5] ARM: tegra: export usb phy symbols Alan Ott
  2012-03-02 22:30   ` [PATCH v3 0/5] ARM: Tegra: cleanup patches from Arnd Bergmann Stephen Warren
  5 siblings, 1 reply; 13+ messages in thread
From: Alan Ott @ 2012-03-02 22:26 UTC (permalink / raw)
  To: linux-arm-kernel
From: Arnd Bergmann <arnd@arndb.de>
Automatically select USB_ULPI if USB is enabled on Tegra.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Alan Ott <alan@signal11.us>
---
 arch/arm/mach-tegra/Kconfig |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/mach-tegra/Kconfig b/arch/arm/mach-tegra/Kconfig
index 60f3e4b..fdef6b2 100644
--- a/arch/arm/mach-tegra/Kconfig
+++ b/arch/arm/mach-tegra/Kconfig
@@ -8,7 +8,7 @@ config ARCH_TEGRA_2x_SOC
 	select ARM_GIC
 	select ARCH_REQUIRE_GPIOLIB
 	select USB_ARCH_HAS_EHCI if USB_SUPPORT
-	select USB_ULPI if USB_SUPPORT
+	select USB_ULPI if USB
 	select USB_ULPI_VIEWPORT if USB_SUPPORT
 	select CPU_FREQ_TABLE if CPU_FREQ
 	help
@@ -21,7 +21,7 @@ config ARCH_TEGRA_3x_SOC
 	select ARM_GIC
 	select ARCH_REQUIRE_GPIOLIB
 	select USB_ARCH_HAS_EHCI if USB_SUPPORT
-	select USB_ULPI if USB_SUPPORT
+	select USB_ULPI if USB
 	select USB_ULPI_VIEWPORT if USB_SUPPORT
 	select USE_OF
 	select CPU_FREQ_TABLE if CPU_FREQ
-- 
1.7.0.4
^ permalink raw reply related	[flat|nested] 13+ messages in thread
- * [PATCH v3 5/5] ARM: tegra: export usb phy symbols
  2012-03-02 22:24 ` [PATCH v3 0/5] " Alan Ott
                     ` (3 preceding siblings ...)
  2012-03-02 22:26   ` [PATCH v3 4/5] ARM: tegra: select USB_ULPI if USB is selected Alan Ott
@ 2012-03-02 22:26   ` Alan Ott
  2012-03-02 22:30   ` [PATCH v3 0/5] ARM: Tegra: cleanup patches from Arnd Bergmann Stephen Warren
  5 siblings, 0 replies; 13+ messages in thread
From: Alan Ott @ 2012-03-02 22:26 UTC (permalink / raw)
  To: linux-arm-kernel
From: Arnd Bergmann <arnd@arndb.de>
The ehci driver can be a module, so the functions provided
by the tegra platform code used by ehci-tegra need to
be exported.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Alan Ott <alan@signal11.us>
---
 arch/arm/mach-tegra/usb_phy.c |   11 +++++++++++
 1 files changed, 11 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-tegra/usb_phy.c b/arch/arm/mach-tegra/usb_phy.c
index 37576a7..6091cf3 100644
--- a/arch/arm/mach-tegra/usb_phy.c
+++ b/arch/arm/mach-tegra/usb_phy.c
@@ -22,6 +22,7 @@
 #include <linux/delay.h>
 #include <linux/slab.h>
 #include <linux/err.h>
+#include <linux/export.h>
 #include <linux/platform_device.h>
 #include <linux/io.h>
 #include <linux/gpio.h>
@@ -730,6 +731,7 @@ err0:
 	kfree(phy);
 	return ERR_PTR(err);
 }
+EXPORT_SYMBOL_GPL(tegra_usb_phy_open);
 
 int tegra_usb_phy_power_on(struct tegra_usb_phy *phy)
 {
@@ -738,6 +740,7 @@ int tegra_usb_phy_power_on(struct tegra_usb_phy *phy)
 	else
 		return utmi_phy_power_on(phy);
 }
+EXPORT_SYMBOL_GPL(tegra_usb_phy_power_on);
 
 void tegra_usb_phy_power_off(struct tegra_usb_phy *phy)
 {
@@ -746,18 +749,21 @@ void tegra_usb_phy_power_off(struct tegra_usb_phy *phy)
 	else
 		utmi_phy_power_off(phy);
 }
+EXPORT_SYMBOL_GPL(tegra_usb_phy_power_off);
 
 void tegra_usb_phy_preresume(struct tegra_usb_phy *phy)
 {
 	if (!phy_is_ulpi(phy))
 		utmi_phy_preresume(phy);
 }
+EXPORT_SYMBOL_GPL(tegra_usb_phy_preresume);
 
 void tegra_usb_phy_postresume(struct tegra_usb_phy *phy)
 {
 	if (!phy_is_ulpi(phy))
 		utmi_phy_postresume(phy);
 }
+EXPORT_SYMBOL_GPL(tegra_usb_phy_postresume);
 
 void tegra_ehci_phy_restore_start(struct tegra_usb_phy *phy,
 				 enum tegra_usb_phy_port_speed port_speed)
@@ -765,24 +771,28 @@ void tegra_ehci_phy_restore_start(struct tegra_usb_phy *phy,
 	if (!phy_is_ulpi(phy))
 		utmi_phy_restore_start(phy, port_speed);
 }
+EXPORT_SYMBOL_GPL(tegra_ehci_phy_restore_start);
 
 void tegra_ehci_phy_restore_end(struct tegra_usb_phy *phy)
 {
 	if (!phy_is_ulpi(phy))
 		utmi_phy_restore_end(phy);
 }
+EXPORT_SYMBOL_GPL(tegra_ehci_phy_restore_end);
 
 void tegra_usb_phy_clk_disable(struct tegra_usb_phy *phy)
 {
 	if (!phy_is_ulpi(phy))
 		utmi_phy_clk_disable(phy);
 }
+EXPORT_SYMBOL_GPL(tegra_usb_phy_clk_disable);
 
 void tegra_usb_phy_clk_enable(struct tegra_usb_phy *phy)
 {
 	if (!phy_is_ulpi(phy))
 		utmi_phy_clk_enable(phy);
 }
+EXPORT_SYMBOL_GPL(tegra_usb_phy_clk_enable);
 
 void tegra_usb_phy_close(struct tegra_usb_phy *phy)
 {
@@ -794,3 +804,4 @@ void tegra_usb_phy_close(struct tegra_usb_phy *phy)
 	clk_put(phy->pll_u);
 	kfree(phy);
 }
+EXPORT_SYMBOL_GPL(tegra_usb_phy_close);
-- 
1.7.0.4
^ permalink raw reply related	[flat|nested] 13+ messages in thread
- * [PATCH v3 0/5] ARM: Tegra: cleanup patches from Arnd Bergmann
  2012-03-02 22:24 ` [PATCH v3 0/5] " Alan Ott
                     ` (4 preceding siblings ...)
  2012-03-02 22:26   ` [PATCH v3 5/5] ARM: tegra: export usb phy symbols Alan Ott
@ 2012-03-02 22:30   ` Stephen Warren
  5 siblings, 0 replies; 13+ messages in thread
From: Stephen Warren @ 2012-03-02 22:30 UTC (permalink / raw)
  To: linux-arm-kernel
Alan Ott wrote at Friday, March 02, 2012 3:25 PM:
> Version 3 of the patch set.
> 
> Changes:
> 	* change of logic (and commit message change) for
> 	  "select USB_ULPI if USB is selected"
> 	* dropped THUMB2 patch
> 
> Arnd Bergmann (5):
>   ARM: tegra: do not hide dma declarations
>   ARM: tegra: select CPU_FREQ_TABLE
>   ARM: tegra: build localtimer support only when needed
>   ARM: tegra: select USB_ULPI if USB is selected
>   ARM: tegra: export usb phy symbols
The series,
Acked-by: Stephen Warren <swarren@nvidia.com>
Thanks Arnd and Alan for cleaning Tegra up!
-- 
nvpublic
^ permalink raw reply	[flat|nested] 13+ messages in thread