Commit 9bf24605 authored by David Brownell's avatar David Brownell Committed by Kevin Hilman

musb_hdrc: compile fixes for davinci

Minor musb_hdrc updates:

  - so it'll build on DaVinci, given relevant platform updates;
      * remove support for an un-shipped OTG prototype
      * rely on gpiolib framework conversion for the I2C GPIOs
      * the <asm/arch/hdrc_cnf.h> mechanism has been removed

  - catch comments up to the recent removal of the per-SOC header
    with the silicon configuration data;

  - and remove two inappropriate "inline" declarations which
    just bloat host side code.

There are still some more <asm/arch/XYZ.h> ==> <mach/XYZ.h>
changes needed in this driver, catching up to the removal of
most of the include/asm-arm/arch-* directories; this patch
has the Davinci parts of those.
Signed-off-by: default avatarDavid Brownell <dbrownell@users.sourceforge.net>
parent 8efc87d8
...@@ -32,10 +32,6 @@ config USB_MUSB_SOC ...@@ -32,10 +32,6 @@ config USB_MUSB_SOC
default y if ARCH_DAVINCI default y if ARCH_DAVINCI
default y if ARCH_OMAP2430 default y if ARCH_OMAP2430
default y if ARCH_OMAP34XX default y if ARCH_OMAP34XX
help
Use a static <asm/arch/hdrc_cnf.h> file to describe how the
controller is configured (endpoints, mechanisms, etc) on the
current iteration of a given system-on-chip.
comment "DaVinci 644x USB support" comment "DaVinci 644x USB support"
depends on USB_MUSB_HDRC && ARCH_DAVINCI depends on USB_MUSB_HDRC && ARCH_DAVINCI
......
...@@ -119,8 +119,8 @@ struct cppi { ...@@ -119,8 +119,8 @@ struct cppi {
void __iomem *mregs; /* Mentor regs */ void __iomem *mregs; /* Mentor regs */
void __iomem *tibase; /* TI/CPPI regs */ void __iomem *tibase; /* TI/CPPI regs */
struct cppi_channel tx[MUSB_C_NUM_EPT - 1]; struct cppi_channel tx[4];
struct cppi_channel rx[MUSB_C_NUM_EPR - 1]; struct cppi_channel rx[4];
struct dma_pool *pool; struct dma_pool *pool;
......
...@@ -30,16 +30,17 @@ ...@@ -30,16 +30,17 @@
#include <linux/delay.h> #include <linux/delay.h>
#include <linux/clk.h> #include <linux/clk.h>
#include <linux/io.h> #include <linux/io.h>
#include <linux/gpio.h>
#include <mach/hardware.h>
#include <mach/memory.h>
#include <asm/arch/hardware.h>
#include <asm/arch/memory.h>
#include <asm/arch/gpio.h>
#include <asm/mach-types.h> #include <asm/mach-types.h>
#include "musb_core.h" #include "musb_core.h"
#ifdef CONFIG_MACH_DAVINCI_EVM #ifdef CONFIG_MACH_DAVINCI_EVM
#include <asm/arch/i2c-client.h> #define GPIO_nVBUS_DRV 87
#endif #endif
#include "davinci.h" #include "davinci.h"
...@@ -138,7 +139,6 @@ static int vbus_state = -1; ...@@ -138,7 +139,6 @@ static int vbus_state = -1;
/* VBUS SWITCHING IS BOARD-SPECIFIC */ /* VBUS SWITCHING IS BOARD-SPECIFIC */
#ifdef CONFIG_MACH_DAVINCI_EVM #ifdef CONFIG_MACH_DAVINCI_EVM
#ifndef CONFIG_MACH_DAVINCI_EVM_OTG
/* I2C operations are always synchronous, and require a task context. /* I2C operations are always synchronous, and require a task context.
* With unloaded systems, using the shared workqueue seems to suffice * With unloaded systems, using the shared workqueue seems to suffice
...@@ -146,12 +146,11 @@ static int vbus_state = -1; ...@@ -146,12 +146,11 @@ static int vbus_state = -1;
*/ */
static void evm_deferred_drvvbus(struct work_struct *ignored) static void evm_deferred_drvvbus(struct work_struct *ignored)
{ {
davinci_i2c_expander_op(0x3a, USB_DRVVBUS, vbus_state); gpio_set_value_cansleep(GPIO_nVBUS_DRV, vbus_state);
vbus_state = !vbus_state; vbus_state = !vbus_state;
} }
static DECLARE_WORK(evm_vbus_work, evm_deferred_drvvbus); static DECLARE_WORK(evm_vbus_work, evm_deferred_drvvbus);
#endif /* modified board */
#endif /* EVM */ #endif /* EVM */
static void davinci_source_power(struct musb *musb, int is_on, int immediate) static void davinci_source_power(struct musb *musb, int is_on, int immediate)
...@@ -165,21 +164,10 @@ static void davinci_source_power(struct musb *musb, int is_on, int immediate) ...@@ -165,21 +164,10 @@ static void davinci_source_power(struct musb *musb, int is_on, int immediate)
#ifdef CONFIG_MACH_DAVINCI_EVM #ifdef CONFIG_MACH_DAVINCI_EVM
if (machine_is_davinci_evm()) { if (machine_is_davinci_evm()) {
#ifdef CONFIG_MACH_DAVINCI_EVM_OTG
/* modified EVM board switching VBUS with GPIO(6) not I2C
* NOTE: PINMUX0.RGB888 (bit23) must be clear
*/
if (is_on)
gpio_set(GPIO(6));
else
gpio_clear(GPIO(6));
immediate = 1;
#else
if (immediate) if (immediate)
davinci_i2c_expander_op(0x3a, USB_DRVVBUS, !is_on); gpio_set_value_cansleep(GPIO_nVBUS_DRV, vbus_state);
else else
schedule_work(&evm_vbus_work); schedule_work(&evm_vbus_work);
#endif
} }
#endif #endif
if (immediate) if (immediate)
......
...@@ -82,9 +82,9 @@ ...@@ -82,9 +82,9 @@
/* /*
* This gets many kinds of configuration information: * This gets many kinds of configuration information:
* - Kconfig for everything user-configurable * - Kconfig for everything user-configurable
* - <asm/arch/hdrc_cnf.h> for SOC or family details
* - platform_device for addressing, irq, and platform_data * - platform_device for addressing, irq, and platform_data
* - platform_data is mostly for board-specific informarion * - platform_data is mostly for board-specific information
* (plus recently, SOC or family details)
* *
* Most of the conditional compilation will (someday) vanish. * Most of the conditional compilation will (someday) vanish.
*/ */
...@@ -100,8 +100,8 @@ ...@@ -100,8 +100,8 @@
#include <linux/io.h> #include <linux/io.h>
#ifdef CONFIG_ARM #ifdef CONFIG_ARM
#include <asm/arch/hardware.h> #include <mach/hardware.h>
#include <asm/arch/memory.h> #include <mach/memory.h>
#include <asm/mach-types.h> #include <asm/mach-types.h>
#endif #endif
...@@ -974,9 +974,9 @@ static void musb_shutdown(struct platform_device *pdev) ...@@ -974,9 +974,9 @@ static void musb_shutdown(struct platform_device *pdev)
/* /*
* The silicon either has hard-wired endpoint configurations, or else * The silicon either has hard-wired endpoint configurations, or else
* "dynamic fifo" sizing. The driver has support for both, though at this * "dynamic fifo" sizing. The driver has support for both, though at this
* writing only the dynamic sizing is very well tested. We use normal * writing only the dynamic sizing is very well tested. Since we switched
* idioms to so both modes are compile-tested, but dead code elimination * away from compile-time hardware parameters, we can no longer rely on
* leaves only the relevant one in the object file. * dead code elimination to leave only the relevant one in the object file.
* *
* We don't currently use dynamic fifo setup capability to do anything * We don't currently use dynamic fifo setup capability to do anything
* more than selecting one of a bunch of predefined configurations. * more than selecting one of a bunch of predefined configurations.
......
...@@ -108,7 +108,7 @@ static void musb_ep_program(struct musb *musb, u8 epnum, ...@@ -108,7 +108,7 @@ static void musb_ep_program(struct musb *musb, u8 epnum,
/* /*
* Clear TX fifo. Needed to avoid BABBLE errors. * Clear TX fifo. Needed to avoid BABBLE errors.
*/ */
static inline void musb_h_tx_flush_fifo(struct musb_hw_ep *ep) static void musb_h_tx_flush_fifo(struct musb_hw_ep *ep)
{ {
void __iomem *epio = ep->regs; void __iomem *epio = ep->regs;
u16 csr; u16 csr;
...@@ -436,7 +436,7 @@ musb_advance_schedule(struct musb *musb, struct urb *urb, ...@@ -436,7 +436,7 @@ musb_advance_schedule(struct musb *musb, struct urb *urb,
} }
} }
static inline u16 musb_h_flush_rxfifo(struct musb_hw_ep *hw_ep, u16 csr) static u16 musb_h_flush_rxfifo(struct musb_hw_ep *hw_ep, u16 csr)
{ {
/* we don't want fifo to fill itself again; /* we don't want fifo to fill itself again;
* ignore dma (various models), * ignore dma (various models),
...@@ -1005,7 +1005,7 @@ static bool musb_h_ep0_continue(struct musb *musb, u16 len, struct urb *urb) ...@@ -1005,7 +1005,7 @@ static bool musb_h_ep0_continue(struct musb *musb, u16 len, struct urb *urb)
/* /*
* Handle default endpoint interrupt as host. Only called in IRQ time * Handle default endpoint interrupt as host. Only called in IRQ time
* from the LinuxIsr() interrupt service routine. * from musb_interrupt().
* *
* called with controller irqlocked * called with controller irqlocked
*/ */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment