* [PATCH 0/3] my pending patches @ 2008-08-13 17:36 Felipe Balbi 2008-08-13 17:36 ` [PATCH 1/3] input: keypad: General fixes to omap-twl4030keypad.c Felipe Balbi 0 siblings, 1 reply; 10+ messages in thread From: Felipe Balbi @ 2008-08-13 17:36 UTC (permalink / raw) To: linux-omap; +Cc: Felipe Balbi Hi tony, following are my three pending patches since last merge. At least patch #2 for musb should be applied before next merge with mainline. Another version of that patch (without board-file changes) is already in Greg's queue and without this version of that patch in l-o, musb will be broken. It's also a good idea to apply this before you format omap2/3 patches to mainline since I'm getting rid of hdrc_conf.h file, so we don't polute mailine with an extra unused file. ps: I'm on vacations until next week and I don't have devices with me to test them on. The patches were working last time I tried, but after merge I only rebased them and made sure they build. So, please, give it a try before applying. Felipe Balbi (3): input: keypad: General fixes to omap-twl4030keypad.c usb: musb: pass configuration specifics via pdata input: add more error checks to lm8323 driver arch/arm/mach-omap2/board-2430sdp.c | 5 +- arch/arm/mach-omap2/board-3430sdp.c | 5 +- arch/arm/mach-omap2/board-n800-usb.c | 46 ++++++- arch/arm/mach-omap2/usb-musb.c | 51 +++++++- arch/arm/mach-omap2/usb-tusb6010.c | 1 - drivers/input/keyboard/lm8323.c | 42 ++++-- drivers/input/keyboard/omap-twl4030keypad.c | 204 ++++++++++++++++----------- drivers/usb/musb/musb_core.c | 37 ++--- drivers/usb/musb/musb_core.h | 14 +-- drivers/usb/musb/tusb6010.h | 169 ---------------------- include/asm-arm/arch-omap/hdrc_cnf.h | 177 ----------------------- include/asm-arm/arch-omap/keypad.h | 1 + include/linux/usb/musb.h | 38 +++++- 13 files changed, 305 insertions(+), 485 deletions(-) delete mode 100644 include/asm-arm/arch-omap/hdrc_cnf.h ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 1/3] input: keypad: General fixes to omap-twl4030keypad.c 2008-08-13 17:36 [PATCH 0/3] my pending patches Felipe Balbi @ 2008-08-13 17:36 ` Felipe Balbi 2008-08-13 17:36 ` [PATCH 2/3] usb: musb: pass configuration specifics via pdata Felipe Balbi 0 siblings, 1 reply; 10+ messages in thread From: Felipe Balbi @ 2008-08-13 17:36 UTC (permalink / raw) To: linux-omap; +Cc: Felipe Balbi The following patch fixes some problems in T2 keypad driver. Basically we're passing irq number via platform_data, moving globals to a structure and fixing a problem while iterating over the keymap. It might be that we still have a few locking issues that might be solved on a later version of this same patch. Signed-off-by: Felipe Balbi <felipe.balbi@nokia.com> --- arch/arm/mach-omap2/board-2430sdp.c | 5 +- arch/arm/mach-omap2/board-3430sdp.c | 5 +- drivers/input/keyboard/omap-twl4030keypad.c | 204 ++++++++++++++++----------- include/asm-arm/arch-omap/keypad.h | 1 + 4 files changed, 129 insertions(+), 86 deletions(-) diff --git a/arch/arm/mach-omap2/board-2430sdp.c b/arch/arm/mach-omap2/board-2430sdp.c index b2f8b9c..3270ea6 100644 --- a/arch/arm/mach-omap2/board-2430sdp.c +++ b/arch/arm/mach-omap2/board-2430sdp.c @@ -176,9 +176,10 @@ static int sdp2430_keymap[] = { static struct omap_kp_platform_data sdp2430_kp_data = { .rows = 5, .cols = 6, - .keymap = sdp2430_keymap, - .keymapsize = ARRAY_SIZE(sdp2430_keymap), + .keymap = sdp2430_keymap, + .keymapsize = ARRAY_SIZE(sdp2430_keymap), .rep = 1, + .irq = TWL4030_MODIRQ_KEYPAD, }; static struct platform_device sdp2430_kp_device = { diff --git a/arch/arm/mach-omap2/board-3430sdp.c b/arch/arm/mach-omap2/board-3430sdp.c index ee4ec18..94ad839 100644 --- a/arch/arm/mach-omap2/board-3430sdp.c +++ b/arch/arm/mach-omap2/board-3430sdp.c @@ -113,9 +113,10 @@ static int sdp3430_keymap[] = { static struct omap_kp_platform_data sdp3430_kp_data = { .rows = 5, .cols = 6, - .keymap = sdp3430_keymap, - .keymapsize = ARRAY_SIZE(sdp3430_keymap), + .keymap = sdp3430_keymap, + .keymapsize = ARRAY_SIZE(sdp3430_keymap), .rep = 1, + .irq = TWL4030_MODIRQ_KEYPAD, }; static struct platform_device sdp3430_kp_device = { diff --git a/drivers/input/keyboard/omap-twl4030keypad.c b/drivers/input/keyboard/omap-twl4030keypad.c index 68c5b8e..05c7cf8 100644 --- a/drivers/input/keyboard/omap-twl4030keypad.c +++ b/drivers/input/keyboard/omap-twl4030keypad.c @@ -31,6 +31,7 @@ #include <linux/types.h> #include <linux/input.h> #include <linux/kernel.h> +#include <linux/mutex.h> #include <linux/delay.h> #include <linux/bitops.h> #include <linux/platform_device.h> @@ -47,52 +48,65 @@ #define KEYNUM_MASK 0x00FFFFFF /* Global variables */ -static int *keymap; -static u16 kp_state[MAX_ROWS]; -static int n_rows, n_cols; -static struct device *dbg_dev; -static struct input_dev *omap_twl4030kp; +struct omap_keypad { + int *keymap; + unsigned int keymapsize; + u16 kp_state[MAX_ROWS]; + int n_rows; + int n_cols; + int irq; -static int twl4030_kpread(u32 module, u8 *data, u32 reg, u8 num_bytes) + struct device *dbg_dev; + struct input_dev *omap_twl4030kp; + + /* sync read/write */ + struct mutex mutex; +}; + +static int twl4030_kpread(struct omap_keypad *kp, + u32 module, u8 *data, u32 reg, u8 num_bytes) { int ret; ret = twl4030_i2c_read(module, data, reg, num_bytes); if (ret < 0) { - dev_warn(dbg_dev, "Couldn't read TWL4030: %X - ret %d[%x]\n", + dev_warn(kp->dbg_dev, + "Couldn't read TWL4030: %X - ret %d[%x]\n", reg, ret, ret); return ret; } return ret; } -static int twl4030_kpwrite_u8(u32 module, u8 data, u32 reg) +static int twl4030_kpwrite_u8(struct omap_keypad *kp, + u32 module, u8 data, u32 reg) { int ret; ret = twl4030_i2c_write_u8(module, data, reg); if (ret < 0) { - dev_warn(dbg_dev, "Could not write TWL4030: %X - ret %d[%x]\n", + dev_warn(kp->dbg_dev, + "Could not write TWL4030: %X - ret %d[%x]\n", reg, ret, ret); return ret; } return ret; } -static int omap_kp_find_key(int col, int row) +static int omap_kp_find_key(struct omap_keypad *kp, int col, int row) { int i, rc; rc = KEY(col, row, 0); - for (i = 0; keymap[i] != 0; i++) - if ((keymap[i] & ROWCOL_MASK) == rc) - return keymap[i] & KEYNUM_MASK; + for (i = 0; i < kp->keymapsize; i++) + if ((kp->keymap[i] & ROWCOL_MASK) == rc) + return kp->keymap[i] & KEYNUM_MASK; return -EINVAL; } -static inline u16 omap_kp_col_xlate(u8 col) +static inline u16 omap_kp_col_xlate(struct omap_keypad *kp, u8 col) { /* If all bits in a row are active for all coloumns then * we have that row line connected to gnd. Mark this @@ -100,30 +114,30 @@ static inline u16 omap_kp_col_xlate(u8 col) * one higher than the size of the matrix). */ if (col == 0xFF) - return (1 << n_cols); + return 1 << kp->n_cols; else - return col & ((1 << n_cols) - 1); + return col & ((1 << kp->n_cols) - 1); } -static int omap_kp_read_kp_matrix_state(u16 *state) +static int omap_kp_read_kp_matrix_state(struct omap_keypad *kp, u16 *state) { u8 new_state[MAX_ROWS]; int row; - int ret = twl4030_kpread(TWL4030_MODULE_KEYPAD, - new_state, KEYP_FULL_CODE_7_0, n_rows); + int ret = twl4030_kpread(kp, TWL4030_MODULE_KEYPAD, + new_state, KEYP_FULL_CODE_7_0, kp->n_rows); if (ret >= 0) { - for (row = 0; row < n_rows; row++) - state[row] = omap_kp_col_xlate(new_state[row]); + for (row = 0; row < kp->n_rows; row++) + state[row] = omap_kp_col_xlate(kp, new_state[row]); } return ret; } -static int omap_kp_is_in_ghost_state(u16 *key_state) +static int omap_kp_is_in_ghost_state(struct omap_keypad *kp, u16 *key_state) { int i; u16 check = 0; - for (i = 0; i < n_rows; i++) { + for (i = 0; i < kp->n_rows; i++) { u16 col = key_state[i]; if ((col & check) && hweight16(col) > 1) @@ -134,7 +148,7 @@ static int omap_kp_is_in_ghost_state(u16 *key_state) return 0; } -static void twl4030_kp_scan(int release_all) +static void twl4030_kp_scan(struct omap_keypad *kp, int release_all) { u16 new_state[MAX_ROWS]; int col, row; @@ -143,60 +157,66 @@ static void twl4030_kp_scan(int release_all) memset(new_state, 0, sizeof(new_state)); else { /* check for any changes */ - int ret = omap_kp_read_kp_matrix_state(new_state); + int ret = omap_kp_read_kp_matrix_state(kp, new_state); if (ret < 0) /* panic ... */ return; - if (omap_kp_is_in_ghost_state(new_state)) + if (omap_kp_is_in_ghost_state(kp, new_state)) return; } + mutex_lock(&kp->mutex); + /* check for changes and print those */ - for (row = 0; row < n_rows; row++) { - int changed = new_state[row] ^ kp_state[row]; + for (row = 0; row < kp->n_rows; row++) { + int changed = new_state[row] ^ kp->kp_state[row]; if (!changed) continue; - for (col = 0; col < n_cols; col++) { + for (col = 0; col < kp->n_cols + 1; col++) { int key; if (!(changed & (1 << col))) continue; - dev_dbg(dbg_dev, "key [%d:%d] %s\n", row, col, + dev_dbg(kp->dbg_dev, "key [%d:%d] %s\n", row, col, (new_state[row] & (1 << col)) ? "press" : "release"); - key = omap_kp_find_key(col, row); + key = omap_kp_find_key(kp, col, row); if (key < 0) - dev_warn(dbg_dev, "Spurious key event %d-%d\n", + dev_warn(kp->dbg_dev, + "Spurious key event %d-%d\n", col, row); else - input_report_key(omap_twl4030kp, key, + input_report_key(kp->omap_twl4030kp, key, new_state[row] & (1 << col)); } - kp_state[row] = new_state[row]; + kp->kp_state[row] = new_state[row]; } + + mutex_unlock(&kp->mutex); } /* * Keypad interrupt handler */ -static irqreturn_t do_kp_irq(int irq, void *dev_id) +static irqreturn_t do_kp_irq(int irq, void *_kp) { + struct omap_keypad *kp = _kp; u8 reg; int ret; /* Read & Clear TWL4030 pending interrupt */ - ret = twl4030_kpread(TWL4030_MODULE_KEYPAD, ®, KEYP_ISR1, 1); + ret = twl4030_kpread(kp, TWL4030_MODULE_KEYPAD, ®, KEYP_ISR1, 1); /* Release all keys if I2C has gone bad or * the KEYP has gone to idle state */ if ((ret >= 0) && (reg & KEYP_IMR1_KP)) - twl4030_kp_scan(0); + twl4030_kp_scan(kp, 0); else - twl4030_kp_scan(1); + twl4030_kp_scan(kp, 1); return IRQ_HANDLED; } @@ -210,92 +230,108 @@ static int __init omap_kp_probe(struct platform_device *pdev) u8 reg; int i; int ret = 0; + struct omap_keypad *kp; struct omap_kp_platform_data *pdata = pdev->dev.platform_data; - /* Get the debug Device */ - dbg_dev = &(pdev->dev); + kp = kzalloc(sizeof(*kp), GFP_KERNEL); + if (!kp) + return -ENOMEM; if (!pdata->rows || !pdata->cols || !pdata->keymap) { - dev_err(dbg_dev, "No rows, cols or keymap from pdata\n"); + dev_err(kp->dbg_dev, "No rows, cols or keymap from pdata\n"); + kfree(kp); return -EINVAL; } - omap_twl4030kp = input_allocate_device(); - if (omap_twl4030kp == NULL) + dev_set_drvdata(&pdev->dev, kp); + + /* Get the debug Device */ + kp->dbg_dev = &pdev->dev; + + kp->omap_twl4030kp = input_allocate_device(); + if (!kp->omap_twl4030kp) { + kfree(kp); return -ENOMEM; + } - keymap = pdata->keymap; - n_rows = pdata->rows; - n_cols = pdata->cols; + mutex_init(&kp->mutex); + + kp->keymap = pdata->keymap; + kp->keymapsize = pdata->keymapsize; + kp->n_rows = pdata->rows; + kp->n_cols = pdata->cols; + kp->irq = pdata->irq; /* setup input device */ - set_bit(EV_KEY, omap_twl4030kp->evbit); + set_bit(EV_KEY, kp->omap_twl4030kp->evbit); /* Enable auto repeat feature of Linux input subsystem */ if (pdata->rep) - set_bit(EV_REP, omap_twl4030kp->evbit); + set_bit(EV_REP, kp->omap_twl4030kp->evbit); - for (i = 0; keymap[i] != 0; i++) - set_bit(keymap[i] & KEYNUM_MASK, omap_twl4030kp->keybit); + for (i = 0; i < kp->keymapsize; i++) + set_bit(kp->keymap[i] & KEYNUM_MASK, + kp->omap_twl4030kp->keybit); - omap_twl4030kp->name = "omap_twl4030keypad"; - omap_twl4030kp->phys = "omap_twl4030keypad/input0"; - omap_twl4030kp->dev.parent = &pdev->dev; + kp->omap_twl4030kp->name = "omap_twl4030keypad"; + kp->omap_twl4030kp->phys = "omap_twl4030keypad/input0"; + kp->omap_twl4030kp->dev.parent = &pdev->dev; - omap_twl4030kp->id.bustype = BUS_HOST; - omap_twl4030kp->id.vendor = 0x0001; - omap_twl4030kp->id.product = 0x0001; - omap_twl4030kp->id.version = 0x0003; + kp->omap_twl4030kp->id.bustype = BUS_HOST; + kp->omap_twl4030kp->id.vendor = 0x0001; + kp->omap_twl4030kp->id.product = 0x0001; + kp->omap_twl4030kp->id.version = 0x0003; - omap_twl4030kp->keycode = keymap; - omap_twl4030kp->keycodesize = sizeof(unsigned int); - omap_twl4030kp->keycodemax = pdata->keymapsize; + kp->omap_twl4030kp->keycode = kp->keymap; + kp->omap_twl4030kp->keycodesize = sizeof(unsigned int); + kp->omap_twl4030kp->keycodemax = kp->keymapsize; - ret = input_register_device(omap_twl4030kp); + ret = input_register_device(kp->omap_twl4030kp); if (ret < 0) { - dev_err(dbg_dev, "Unable to register twl4030 keypad device\n"); + dev_err(kp->dbg_dev, + "Unable to register twl4030 keypad device\n"); goto err2; } /* Disable auto-repeat */ reg = KEYP_CTRL_NOAUTORPT; - ret = twl4030_kpwrite_u8(TWL4030_MODULE_KEYPAD, reg, KEYP_CTRL); + ret = twl4030_kpwrite_u8(kp, TWL4030_MODULE_KEYPAD, reg, KEYP_CTRL); if (ret < 0) goto err3; /* Enable TO rising and KP rising and falling edge detection */ reg = KEYP_EDR_KP_BOTH | KEYP_EDR_TO_RISING; - ret = twl4030_kpwrite_u8(TWL4030_MODULE_KEYPAD, reg, KEYP_EDR); + ret = twl4030_kpwrite_u8(kp, TWL4030_MODULE_KEYPAD, reg, KEYP_EDR); if (ret < 0) goto err3; /* Set PTV prescaler Field */ reg = (PTV_PRESCALER << KEYP_LK_PTV_PTV_SHIFT); - ret = twl4030_kpwrite_u8(TWL4030_MODULE_KEYPAD, reg, KEYP_LK_PTV); + ret = twl4030_kpwrite_u8(kp, TWL4030_MODULE_KEYPAD, reg, KEYP_LK_PTV); if (ret < 0) goto err3; /* Set key debounce time to 20 ms */ i = KEYP_PERIOD_US(20000, PTV_PRESCALER); - ret = twl4030_kpwrite_u8(TWL4030_MODULE_KEYPAD, i, KEYP_DEB); + ret = twl4030_kpwrite_u8(kp, TWL4030_MODULE_KEYPAD, i, KEYP_DEB); if (ret < 0) goto err3; /* Set timeout period to 100 ms */ i = KEYP_PERIOD_US(200000, PTV_PRESCALER); - ret = twl4030_kpwrite_u8(TWL4030_MODULE_KEYPAD, + ret = twl4030_kpwrite_u8(kp, TWL4030_MODULE_KEYPAD, (i & 0xFF), KEYP_TIMEOUT_L); if (ret < 0) goto err3; - ret = twl4030_kpwrite_u8(TWL4030_MODULE_KEYPAD, + ret = twl4030_kpwrite_u8(kp, TWL4030_MODULE_KEYPAD, (i >> 8), KEYP_TIMEOUT_H); if (ret < 0) goto err3; /* Enable Clear-on-Read */ reg = KEYP_SIH_CTRL_COR | KEYP_SIH_CTRL_PEND_DIS; - ret = twl4030_kpwrite_u8(TWL4030_MODULE_KEYPAD, + ret = twl4030_kpwrite_u8(kp, TWL4030_MODULE_KEYPAD, reg, KEYP_SIH_CTRL); if (ret < 0) goto err3; @@ -304,50 +340,54 @@ static int __init omap_kp_probe(struct platform_device *pdev) * This ISR will always execute in kernel thread context because of * the need to access the TWL4030 over the I2C bus. */ - ret = request_irq(TWL4030_MODIRQ_KEYPAD, do_kp_irq, - IRQF_DISABLED, "TWL4030 Keypad", omap_twl4030kp); + ret = request_irq(kp->irq, do_kp_irq, IRQF_DISABLED, + "TWL4030 Keypad", kp); if (ret < 0) { - dev_info(dbg_dev, "request_irq failed for irq no=%d\n", - TWL4030_MODIRQ_KEYPAD); + dev_info(kp->dbg_dev, "request_irq failed for irq no=%d\n", + kp->irq); goto err3; } else { /* Enable KP and TO interrupts now. */ reg = ~(KEYP_IMR1_KP | KEYP_IMR1_TO); - ret = twl4030_kpwrite_u8(TWL4030_MODULE_KEYPAD, + ret = twl4030_kpwrite_u8(kp, TWL4030_MODULE_KEYPAD, reg, KEYP_IMR1); if (ret < 0) goto err5; } - ret = omap_kp_read_kp_matrix_state(kp_state); + ret = omap_kp_read_kp_matrix_state(kp, kp->kp_state); if (ret < 0) goto err4; return ret; err5: /* mask all events - we don't care about the result */ - (void) twl4030_kpwrite_u8(TWL4030_MODULE_KEYPAD, 0xff, KEYP_IMR1); + (void) twl4030_kpwrite_u8(kp, TWL4030_MODULE_KEYPAD, 0xff, KEYP_IMR1); err4: - free_irq(TWL4030_MODIRQ_KEYPAD, NULL); + free_irq(kp->irq, NULL); err3: - input_unregister_device(omap_twl4030kp); + input_unregister_device(kp->omap_twl4030kp); err2: - input_free_device(omap_twl4030kp); + input_free_device(kp->omap_twl4030kp); + return -ENODEV; } static int omap_kp_remove(struct platform_device *pdev) { - free_irq(TWL4030_MODIRQ_KEYPAD, NULL); + struct omap_keypad *kp = dev_get_drvdata(&pdev->dev); + + free_irq(kp->irq, kp); + input_unregister_device(kp->omap_twl4030kp); + kfree(kp); - input_unregister_device(omap_twl4030kp); return 0; } static struct platform_driver omap_kp_driver = { .probe = omap_kp_probe, - .remove = omap_kp_remove, + .remove = __devexit_p(omap_kp_remove), .driver = { .name = "omap_twl4030keypad", .owner = THIS_MODULE, diff --git a/include/asm-arm/arch-omap/keypad.h b/include/asm-arm/arch-omap/keypad.h index b7f8307..ac87f37 100644 --- a/include/asm-arm/arch-omap/keypad.h +++ b/include/asm-arm/arch-omap/keypad.h @@ -14,6 +14,7 @@ struct omap_kp_platform_data { int rows; int cols; int *keymap; + int irq; unsigned int keymapsize; unsigned int rep:1; unsigned long delay; -- 1.6.0.rc1.71.gfba5 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 2/3] usb: musb: pass configuration specifics via pdata 2008-08-13 17:36 ` [PATCH 1/3] input: keypad: General fixes to omap-twl4030keypad.c Felipe Balbi @ 2008-08-13 17:36 ` Felipe Balbi 2008-08-13 17:36 ` [PATCH 3/3] input: add more error checks to lm8323 driver Felipe Balbi 2008-08-18 23:43 ` [PATCH 2/3] usb: musb: pass configuration specifics via pdata Steve Sakoman 0 siblings, 2 replies; 10+ messages in thread From: Felipe Balbi @ 2008-08-13 17:36 UTC (permalink / raw) To: linux-omap; +Cc: Felipe Balbi Use platform_data to pass musb configuration-specific details to musb driver. It's easier to maintain in the sense that neither board will affect the other Signed-off-by: Felipe Balbi <felipe.balbi@nokia.com> --- arch/arm/mach-omap2/board-n800-usb.c | 46 +++++++++- arch/arm/mach-omap2/usb-musb.c | 51 +++++++++- arch/arm/mach-omap2/usb-tusb6010.c | 1 - drivers/usb/musb/musb_core.c | 37 +++---- drivers/usb/musb/musb_core.h | 14 +--- drivers/usb/musb/tusb6010.h | 169 -------------------------------- include/asm-arm/arch-omap/hdrc_cnf.h | 177 ---------------------------------- include/linux/usb/musb.h | 38 +++++++- 8 files changed, 144 insertions(+), 389 deletions(-) delete mode 100644 include/asm-arm/arch-omap/hdrc_cnf.h diff --git a/arch/arm/mach-omap2/board-n800-usb.c b/arch/arm/mach-omap2/board-n800-usb.c index 7599f64..16ea8fc 100644 --- a/arch/arm/mach-omap2/board-n800-usb.c +++ b/arch/arm/mach-omap2/board-n800-usb.c @@ -35,14 +35,58 @@ static int tusb_set_clock(struct clk *osc_ck, int state); # define BOARD_MODE MUSB_HOST #endif +static struct musb_hdrc_eps_bits musb_eps[] = { + { "ep1_tx", 5, }, + { "ep1_rx", 5, }, + { "ep2_tx", 5, }, + { "ep2_rx", 5, }, + { "ep3_tx", 3, }, + { "ep3_rx", 3, }, + { "ep4_tx", 3, }, + { "ep4_rx", 3, }, + { "ep5_tx", 2, }, + { "ep5_rx", 2, }, + { "ep6_tx", 2, }, + { "ep6_rx", 2, }, + { "ep7_tx", 2, }, + { "ep7_rx", 2, }, + { "ep8_tx", 2, }, + { "ep8_rx", 2, }, + { "ep9_tx", 2, }, + { "ep9_rx", 2, }, + { "ep10_tx", 2, }, + { "ep10_rx", 2, }, + { "ep11_tx", 2, }, + { "ep11_rx", 2, }, + { "ep12_tx", 2, }, + { "ep12_rx", 2, }, + { "ep13_tx", 2, }, + { "ep13_rx", 2, }, + { "ep14_tx", 2, }, + { "ep14_rx", 2, }, + { "ep15_tx", 2, }, + { "ep15_rx", 2, }, +}; + +static struct musb_hdrc_config musb_config = { + .multipoint = 1, + .dyn_fifo = 1, + .soft_con = 1, + .dma = 1, + .num_eps = 32, + .dma_channels = 7, + .ram_bits = 12, + .eps_bits = musb_eps, +}; + static struct musb_hdrc_platform_data tusb_data = { .mode = BOARD_MODE, - .multipoint = 1, .set_power = tusb_set_power, .set_clock = tusb_set_clock, .min_power = 25, /* x2 = 50 mA drawn from VBUS as peripheral */ .power = 100, /* Max 100 mA VBUS for host mode */ .clock = "osc_ck", + .config = &musb_config, }; /* diff --git a/arch/arm/mach-omap2/usb-musb.c b/arch/arm/mach-omap2/usb-musb.c index cbd59f8..842edb3 100644 --- a/arch/arm/mach-omap2/usb-musb.c +++ b/arch/arm/mach-omap2/usb-musb.c @@ -37,7 +37,7 @@ static struct resource musb_resources[] = { : OMAP243X_HS_BASE, .end = cpu_is_omap34xx() ? OMAP34XX_HSUSB_OTG_BASE + SZ_8K - 1 - : OMAP243X_HS_BASE + SZ_8K -1, + : OMAP243X_HS_BASE + SZ_8K - 1, .flags = IORESOURCE_MEM, }, [1] = { /* general IRQ */ @@ -73,6 +73,51 @@ static int musb_set_clock(struct clk *clk, int state) return 0; } +static struct musb_hdrc_eps_bits musb_eps[] = { + { "ep1_tx", 10, }, + { "ep1_rx", 10, }, + { "ep2_tx", 9, }, + { "ep2_rx", 9, }, + { "ep3_tx", 3, }, + { "ep3_rx", 3, }, + { "ep4_tx", 3, }, + { "ep4_rx", 3, }, + { "ep5_tx", 3, }, + { "ep5_rx", 3, }, + { "ep6_tx", 3, }, + { "ep6_rx", 3, }, + { "ep7_tx", 3, }, + { "ep7_rx", 3, }, + { "ep8_tx", 2, }, + { "ep8_rx", 2, }, + { "ep9_tx", 2, }, + { "ep9_rx", 2, }, + { "ep10_tx", 2, }, + { "ep10_rx", 2, }, + { "ep11_tx", 2, }, + { "ep11_rx", 2, }, + { "ep12_tx", 2, }, + { "ep12_rx", 2, }, + { "ep13_tx", 2, }, + { "ep13_rx", 2, }, + { "ep14_tx", 2, }, + { "ep14_rx", 2, }, + { "ep15_tx", 2, }, + { "ep15_rx", 2, }, +}; + +static struct musb_hdrc_config musb_config = { + .multipoint = 1, + .dyn_fifo = 1, + .soft_con = 1, + .dma = 1, + .num_eps = 32, + .dma_channels = 7, + .dma_req_chan = (1 << 0) | (1 << 1) | (1 << 2) | (1 << 3), + .ram_bits = 12, + .eps_bits = musb_eps, +}; + static struct musb_hdrc_platform_data musb_plat = { #ifdef CONFIG_USB_MUSB_OTG .mode = MUSB_OTG, @@ -81,18 +126,18 @@ static struct musb_hdrc_platform_data musb_plat = { #elif defined(CONFIG_USB_GADGET_MUSB_HDRC) .mode = MUSB_PERIPHERAL, #endif - .multipoint = 1, .clock = cpu_is_omap34xx() ? "hsotgusb_ick" : "usbhs_ick", .set_clock = musb_set_clock, + .config = &musb_config, }; static u64 musb_dmamask = ~(u32)0; static struct platform_device musb_device = { .name = "musb_hdrc", - .id = 0, + .id = -1, .dev = { .dma_mask = &musb_dmamask, .coherent_dma_mask = 0xffffffff, diff --git a/arch/arm/mach-omap2/usb-tusb6010.c b/arch/arm/mach-omap2/usb-tusb6010.c index 9924300..75e2669 100644 --- a/arch/arm/mach-omap2/usb-tusb6010.c +++ b/arch/arm/mach-omap2/usb-tusb6010.c @@ -315,7 +315,6 @@ tusb6010_setup_interface(struct musb_hdrc_platform_data *data, printk(error, 6, status); return -ENODEV; } - data->multipoint = 1; tusb_device.dev.platform_data = data; /* REVISIT let the driver know what DMA channels work */ diff --git a/drivers/usb/musb/musb_core.c b/drivers/usb/musb/musb_core.c index 1404a10..4f8aa1d 100644 --- a/drivers/usb/musb/musb_core.c +++ b/drivers/usb/musb/musb_core.c @@ -990,12 +990,6 @@ static void musb_shutdown(struct platform_device *pdev) * We don't currently use dynamic fifo setup capability to do anything * more than selecting one of a bunch of predefined configurations. */ -#ifdef MUSB_C_DYNFIFO_DEF -#define can_dynfifo() 1 -#else -#define can_dynfifo() 0 -#endif - #if defined(CONFIG_USB_TUSB6010) || \ defined(CONFIG_ARCH_OMAP2430) || defined(CONFIG_ARCH_OMAP34XX) static ushort __initdata fifo_mode = 4; @@ -1008,8 +1002,6 @@ module_param(fifo_mode, ushort, 0); MODULE_PARM_DESC(fifo_mode, "initial endpoint configuration"); -#define DYN_FIFO_SIZE (1<<(MUSB_C_RAM_BITS+2)) - enum fifo_style { FIFO_RXTX, FIFO_TX, FIFO_RX } __attribute__ ((packed)); enum buf_mode { BUF_SINGLE, BUF_DOUBLE } __attribute__ ((packed)); @@ -1119,11 +1111,12 @@ fifo_setup(struct musb *musb, struct musb_hw_ep *hw_ep, c_size = size - 3; if (cfg->mode == BUF_DOUBLE) { - if ((offset + (maxpacket << 1)) > DYN_FIFO_SIZE) + if ((offset + (maxpacket << 1)) > + (1 << (musb->config->ram_bits + 2))) return -EMSGSIZE; c_size |= MUSB_FIFOSZ_DPB; } else { - if ((offset + maxpacket) > DYN_FIFO_SIZE) + if ((offset + maxpacket) > (1 << (musb->config->ram_bits + 2))) return -EMSGSIZE; } @@ -1219,13 +1212,13 @@ static int __init ep_config_from_table(struct musb *musb) /* assert(offset > 0) */ /* NOTE: for RTL versions >= 1.400 EPINFO and RAMINFO would - * be better than static MUSB_C_NUM_EPS and DYN_FIFO_SIZE... + * be better than static musb->config->num_eps and DYN_FIFO_SIZE... */ for (i = 0; i < n; i++) { u8 epn = cfg->hw_ep_num; - if (epn >= MUSB_C_NUM_EPS) { + if (epn >= musb->config->num_eps) { pr_debug("%s: invalid ep %d\n", musb_driver_name, epn); continue; @@ -1242,8 +1235,8 @@ static int __init ep_config_from_table(struct musb *musb) printk(KERN_DEBUG "%s: %d/%d max ep, %d/%d memory\n", musb_driver_name, - n + 1, MUSB_C_NUM_EPS * 2 - 1, - offset, DYN_FIFO_SIZE); + n + 1, musb->config->num_eps * 2 - 1, + offset, (1 << (musb->config->ram_bits + 2))); #ifdef CONFIG_USB_MUSB_HDRC_HCD if (!musb->bulk_ep) { @@ -1270,7 +1263,7 @@ static int __init ep_config_from_hw(struct musb *musb) /* FIXME pick up ep0 maxpacket size */ - for (epnum = 1; epnum < MUSB_C_NUM_EPS; epnum++) { + for (epnum = 1; epnum < musb->config->num_eps; epnum++) { musb_ep_select(mbase, epnum); hw_ep = musb->endpoints + epnum; @@ -1424,14 +1417,14 @@ static int __init musb_core_init(u16 musb_type, struct musb *musb) musb->epmask = 1; if (reg & MUSB_CONFIGDATA_DYNFIFO) { - if (can_dynfifo()) + if (musb->config->dyn_fifo) status = ep_config_from_table(musb); else { ERR("reconfigure software for Dynamic FIFOs\n"); status = -ENODEV; } } else { - if (!can_dynfifo()) + if (!musb->config->dyn_fifo) status = ep_config_from_hw(musb); else { ERR("reconfigure software for static FIFOs\n"); @@ -1788,7 +1781,8 @@ static void musb_irq_work(struct work_struct *data) */ static struct musb *__init -allocate_instance(struct device *dev, void __iomem *mbase) +allocate_instance(struct device *dev, + struct musb_hdrc_config *config, void __iomem *mbase) { struct musb *musb; struct musb_hw_ep *ep; @@ -1820,8 +1814,9 @@ allocate_instance(struct device *dev, void __iomem *mbase) musb->mregs = mbase; musb->ctrl_base = mbase; musb->nIrq = -ENODEV; + musb->config = config; for (epnum = 0, ep = musb->endpoints; - epnum < MUSB_C_NUM_EPS; + epnum < musb->config->num_eps; epnum++, ep++) { ep->musb = musb; @@ -1932,7 +1927,7 @@ bad_config: } /* allocate */ - musb = allocate_instance(dev, ctrl); + musb = allocate_instance(dev, plat->config, ctrl); if (!musb) return -ENOMEM; @@ -1990,7 +1985,7 @@ bad_config: musb_generic_disable(musb); /* setup musb parts of the core (especially endpoints) */ - status = musb_core_init(plat->multipoint + status = musb_core_init(plat->config->multipoint ? MUSB_CONTROLLER_MHDRC : MUSB_CONTROLLER_HDRC, musb); if (status < 0) diff --git a/drivers/usb/musb/musb_core.h b/drivers/usb/musb/musb_core.h index c2cd5a9..73a794a 100644 --- a/drivers/usb/musb/musb_core.h +++ b/drivers/usb/musb/musb_core.h @@ -56,18 +56,6 @@ struct musb_ep; #include "musb_debug.h" #include "musb_dma.h" -#ifdef CONFIG_USB_MUSB_SOC -/* - * Get core configuration from a header converted (by cfg_conv) - * from the Verilog config file generated by the core config utility - * - * For now we assume that header is provided along with other - * arch-specific files. Discrete chips will need a build tweak. - * So will using AHB IDs from silicon that provides them. - */ -#include <asm/arch/hdrc_cnf.h> -#endif - #include "musb_io.h" #include "musb_regs.h" @@ -439,6 +427,8 @@ struct musb { struct usb_gadget_driver *gadget_driver; /* its driver */ #endif + struct musb_hdrc_config *config; + #ifdef MUSB_CONFIG_PROC_FS struct proc_dir_entry *proc_entry; #endif diff --git a/drivers/usb/musb/tusb6010.h b/drivers/usb/musb/tusb6010.h index db6dad0..ab8c962 100644 --- a/drivers/usb/musb/tusb6010.h +++ b/drivers/usb/musb/tusb6010.h @@ -230,173 +230,4 @@ extern u8 tusb_get_revision(struct musb *musb); #define TUSB_REV_30 0x30 #define TUSB_REV_31 0x31 -/*----------------------------------------------------------------------------*/ - -#ifdef CONFIG_USB_TUSB6010 - -/* configuration parameters specific to this silicon */ - -/* Number of Tx endpoints. Legal values are 1 - 16 (this value includes EP0) */ -#define MUSB_C_NUM_EPT 16 - -/* Number of Rx endpoints. Legal values are 1 - 16 (this value includes EP0) */ -#define MUSB_C_NUM_EPR 16 - -/* Endpoint 1 to 15 direction types. C_EP1_DEF is defined if either Tx endpoint - * 1 or Rx endpoint 1 are used. - */ -#define MUSB_C_EP1_DEF - -/* C_EP1_TX_DEF is defined if Tx endpoint 1 is used */ -#define MUSB_C_EP1_TX_DEF - -/* C_EP1_RX_DEF is defined if Rx endpoint 1 is used */ -#define MUSB_C_EP1_RX_DEF - -/* C_EP1_TOR_DEF is defined if Tx endpoint 1 and Rx endpoint 1 share a FIFO */ -/* #define C_EP1_TOR_DEF */ - -/* C_EP1_TAR_DEF is defined if both Tx endpoint 1 and Rx endpoint 1 are used - * and do not share a FIFO. - */ -#define MUSB_C_EP1_TAR_DEF - -/* Similarly for all other used endpoints */ -#define MUSB_C_EP2_DEF -#define MUSB_C_EP2_TX_DEF -#define MUSB_C_EP2_RX_DEF -#define MUSB_C_EP2_TAR_DEF -#define MUSB_C_EP3_DEF -#define MUSB_C_EP3_TX_DEF -#define MUSB_C_EP3_RX_DEF -#define MUSB_C_EP3_TAR_DEF -#define MUSB_C_EP4_DEF -#define MUSB_C_EP4_TX_DEF -#define MUSB_C_EP4_RX_DEF -#define MUSB_C_EP4_TAR_DEF - -/* Endpoint 1 to 15 FIFO address bits. Legal values are 3 to 13 - corresponding - * to FIFO sizes of 8 to 8192 bytes. If an Tx endpoint shares a FIFO with an Rx - * endpoint then the Rx FIFO size must be the same as the Tx FIFO size. All - * endpoints 1 to 15 must be defined, unused endpoints should be set to 2. - */ -#define MUSB_C_EP1T_BITS 5 -#define MUSB_C_EP1R_BITS 5 -#define MUSB_C_EP2T_BITS 5 -#define MUSB_C_EP2R_BITS 5 -#define MUSB_C_EP3T_BITS 3 -#define MUSB_C_EP3R_BITS 3 -#define MUSB_C_EP4T_BITS 3 -#define MUSB_C_EP4R_BITS 3 - -#define MUSB_C_EP5T_BITS 2 -#define MUSB_C_EP5R_BITS 2 -#define MUSB_C_EP6T_BITS 2 -#define MUSB_C_EP6R_BITS 2 -#define MUSB_C_EP7T_BITS 2 -#define MUSB_C_EP7R_BITS 2 -#define MUSB_C_EP8T_BITS 2 -#define MUSB_C_EP8R_BITS 2 -#define MUSB_C_EP9T_BITS 2 -#define MUSB_C_EP9R_BITS 2 -#define MUSB_C_EP10T_BITS 2 -#define MUSB_C_EP10R_BITS 2 -#define MUSB_C_EP11T_BITS 2 -#define MUSB_C_EP11R_BITS 2 -#define MUSB_C_EP12T_BITS 2 -#define MUSB_C_EP12R_BITS 2 -#define MUSB_C_EP13T_BITS 2 -#define MUSB_C_EP13R_BITS 2 -#define MUSB_C_EP14T_BITS 2 -#define MUSB_C_EP14R_BITS 2 -#define MUSB_C_EP15T_BITS 2 -#define MUSB_C_EP15R_BITS 2 - -/* Define the following constant if the USB2.0 Transceiver Macrocell data width - * is 16-bits. - */ -/* #define C_UTM_16 */ - -/* Define this constant if the CPU uses big-endian byte ordering. */ -/* #define C_BIGEND */ - -/* Define the following constant if any Tx endpoint is required to support - * multiple bulk packets. - */ -/* #define C_MP_TX */ - -/* Define the following constant if any Rx endpoint is required to support - * multiple bulk packets. - */ -/* #define C_MP_RX */ - -/* Define the following constant if any Tx endpoint is required to support high - * bandwidth ISO. - */ -/* #define C_HB_TX */ - -/* Define the following constant if any Rx endpoint is required to support high - * bandwidth ISO. - */ -/* #define C_HB_RX */ - -/* Define the following constant if software connect/disconnect control is - * required. - */ -#define MUSB_C_SOFT_CON - -/* Define the following constant if Vendor Control Registers are required. */ -/* #define C_VEND_REG */ - -/* Vendor control register widths. */ -#define MUSB_C_VCTL_BITS 4 -#define MUSB_C_VSTAT_BITS 8 - -/* Define the following constant to include a DMA controller. */ -/* #define C_DMA */ - -/* Define the following constant if 2 or more DMA channels are required. */ -/* #define C_DMA2 */ - -/* Define the following constant if 3 or more DMA channels are required. */ -/* #define C_DMA3 */ - -/* Define the following constant if 4 or more DMA channels are required. */ -/* #define C_DMA4 */ - -/* Define the following constant if 5 or more DMA channels are required. */ -/* #define C_DMA5 */ - -/* Define the following constant if 6 or more DMA channels are required. */ -/* #define C_DMA6 */ - -/* Define the following constant if 7 or more DMA channels are required. */ -/* #define C_DMA7 */ - -/* Define the following constant if 8 or more DMA channels are required. */ -/* #define C_DMA8 */ - -/* Enable Dynamic FIFO Sizing */ -#define MUSB_C_DYNFIFO_DEF - -/* Derived constants. The following constants are derived from the previous - * configuration constants - */ - -/* Total number of endpoints. Legal values are 2 - 16. This must be equal to - * the larger of C_NUM_EPT, C_NUM_EPR - */ -/* #define MUSB_C_NUM_EPS 5 */ - -/* C_EPMAX_BITS is equal to the largest endpoint FIFO word address bits */ -#define MUSB_C_EPMAX_BITS 11 - -/* C_RAM_BITS is the number of address bits required to address the RAM (32-bit - * addresses). It is defined as log2 of the sum of 2** of all the endpoint FIFO - * dword address bits (rounded up). - */ -#define MUSB_C_RAM_BITS 12 - -#endif /* CONFIG_USB_TUSB6010 */ - #endif /* __TUSB6010_H__ */ diff --git a/include/asm-arm/arch-omap/hdrc_cnf.h b/include/asm-arm/arch-omap/hdrc_cnf.h deleted file mode 100644 index 74c8432..0000000 --- a/include/asm-arm/arch-omap/hdrc_cnf.h +++ /dev/null @@ -1,177 +0,0 @@ -/* - * Copyright 2005 Mentor Graphics Corporation - * USB High-Speed Multi-Point Dual-Role Controller Configuration - * - * Copyright Mentor Graphics Corporation and Licensors 2004 - * Copyright (C) 2005 by Texas Instruments - * - * This file contains configuration constants for the (m)hdrc - * silicon as integrated into DaVinci CPUs. - */ - -#ifndef __ARCH_MUSB_HDRC_CNF -#define __ARCH_MUSB_HDRC_CNF - -/* ** Number of Tx endpoints ** */ -/* Legal values are 1 - 16 (this value includes EP0) */ -#define MUSB_C_NUM_EPT 8 - -/* ** Number of Rx endpoints ** */ -/* Legal values are 1 - 16 (this value includes EP0) */ -#define MUSB_C_NUM_EPR 8 - -/* ** Endpoint 1 to 15 direction types ** */ -/* C_EP1_DEF is defined if either Tx endpoint 1 or Rx endpoint 1 are used */ -#define MUSB_C_EP1_DEF - -/* C_EP1_TX_DEF is defined if Tx endpoint 1 is used */ -#define MUSB_C_EP1_TX_DEF - -/* C_EP1_RX_DEF is defined if Rx endpoint 1 is used */ -#define MUSB_C_EP1_RX_DEF - -/* C_EP1_TOR_DEF is defined if Tx endpoint 1 and Rx endpoint 1 share a FIFO */ -/*`define C_EP1_TOR_DEF */ - -/* C_EP1_TAR_DEF is defined if both Tx endpoint 1 and Rx endpoint 1 are used */ -/* and do not share a FIFO */ -#define MUSB_C_EP1_TAR_DEF - -/* Similarly for all other used endpoints */ -#define MUSB_C_EP2_DEF -#define MUSB_C_EP2_TX_DEF -#define MUSB_C_EP2_RX_DEF -#define MUSB_C_EP2_TAR_DEF -#define MUSB_C_EP3_DEF -#define MUSB_C_EP3_TX_DEF -#define MUSB_C_EP3_RX_DEF -#define MUSB_C_EP3_TAR_DEF -#define MUSB_C_EP4_DEF -#define MUSB_C_EP4_TX_DEF -#define MUSB_C_EP4_RX_DEF -#define MUSB_C_EP4_TAR_DEF -#define MUSB_C_EP5_DEF -#define MUSB_C_EP5_TX_DEF -#define MUSB_C_EP5_RX_DEF -#define MUSB_C_EP5_TAR_DEF -#define MUSB_C_EP6_DEF -#define MUSB_C_EP6_TX_DEF -#define MUSB_C_EP6_RX_DEF -#define MUSB_C_EP6_TAR_DEF -#define MUSB_C_EP7_DEF -#define MUSB_C_EP7_TX_DEF -#define MUSB_C_EP7_RX_DEF -#define MUSB_C_EP7_TAR_DEF - -/* ** Endpoint 1 to 15 FIFO address bits ** */ -/* Legal values are 3 to 13 - corresponding to FIFO sizes of 8 to 8192 bytes. */ -/* If an Tx endpoint shares a FIFO with an Rx endpoint then the Rx FIFO size */ -/* must be the same as the Tx FIFO size. */ -/* All endpoints 1 to 15 must be defined, unused endpoints should be set to 2. */ -#define MUSB_C_EP1T_BITS 10 -#define MUSB_C_EP1R_BITS 10 -#define MUSB_C_EP2T_BITS 9 -#define MUSB_C_EP2R_BITS 9 -#define MUSB_C_EP3T_BITS 3 -#define MUSB_C_EP3R_BITS 3 -#define MUSB_C_EP4T_BITS 3 -#define MUSB_C_EP4R_BITS 3 -#define MUSB_C_EP5T_BITS 3 -#define MUSB_C_EP5R_BITS 3 -#define MUSB_C_EP6T_BITS 3 -#define MUSB_C_EP6R_BITS 3 -#define MUSB_C_EP7T_BITS 3 -#define MUSB_C_EP7R_BITS 3 -#define MUSB_C_EP8T_BITS 2 -#define MUSB_C_EP8R_BITS 2 -#define MUSB_C_EP9T_BITS 2 -#define MUSB_C_EP9R_BITS 2 -#define MUSB_C_EP10T_BITS 2 -#define MUSB_C_EP10R_BITS 2 -#define MUSB_C_EP11T_BITS 2 -#define MUSB_C_EP11R_BITS 2 -#define MUSB_C_EP12T_BITS 2 -#define MUSB_C_EP12R_BITS 2 -#define MUSB_C_EP13T_BITS 2 -#define MUSB_C_EP13R_BITS 2 -#define MUSB_C_EP14T_BITS 2 -#define MUSB_C_EP14R_BITS 2 -#define MUSB_C_EP15T_BITS 2 -#define MUSB_C_EP15R_BITS 2 - -/* Define the following constant if the USB2.0 Transceiver Macrocell data width is 16-bits. */ -/* `define C_UTM_16 */ - -/* Define this constant if the CPU uses big-endian byte ordering. */ -/*`define C_BIGEND */ - -/* Define the following constant if any Tx endpoint is required to support multiple bulk packets. */ -/* `define C_MP_TX */ - -/* Define the following constant if any Rx endpoint is required to support multiple bulk packets. */ -/* `define C_MP_RX */ - -/* Define the following constant if any Tx endpoint is required to support high bandwidth ISO. */ -/* `define C_HB_TX */ - -/* Define the following constant if any Rx endpoint is required to support high bandwidth ISO. */ -/* `define C_HB_RX */ - -/* Define the following constant if software connect/disconnect control is required. */ -#define MUSB_C_SOFT_CON - -/* Define the following constant if Vendor Control Registers are required. */ -/* `define C_VEND_REG */ - -/* Vendor control register widths. */ -#define MUSB_C_VCTL_BITS 4 -#define MUSB_C_VSTAT_BITS 8 - - -/* Define the following constant to include a DMA controller. */ -#define MUSB_C_DMA - -/* Define the following constant if 2 or more DMA channels are required. */ -#define MUSB_C_DMA2 - -/* Define the following constant if 3 or more DMA channels are required. */ -#define MUSB_C_DMA3 - -/* Define the following constant if 4 or more DMA channels are required. */ -#define MUSB_C_DMA4 - -/* Define the following constant if 5 or more DMA channels are required. */ -/*`define C_DMA5 */ - -/* Define the following constant if 6 or more DMA channels are required. */ -/*`define C_DMA6 */ - -/* Define the following constant if 7 or more DMA channels are required. */ -/*`define C_DMA7 */ - -/* Define the following constant if 8 or more DMA channels are required. */ -/*`define C_DMA8 */ - - -/* ** Enable Dynamic FIFO Sizing ** */ -#define MUSB_C_DYNFIFO_DEF - -/* ** Derived constants ** */ -/* The following constants are derived from the previous configuration constants */ - -/* Total number of endpoints - * Legal values are 2 - 16 - * This must be equal to the larger of C_NUM_EPT, C_NUM_EPR - */ -#define MUSB_C_NUM_EPS 8 - -/* C_EPMAX_BITS is equal to the largest endpoint FIFO word address bits */ -#define MUSB_C_EPMAX_BITS 12 - -/* C_RAM_BITS is the number of address bits required to address the RAM (32-bit - * addresses). It is defined as log2 of the sum of 2** of all the endpoint FIFO - * dword address bits (rounded up). - */ -#define MUSB_C_RAM_BITS 12 - -#endif /* __ARCH_MUSB_HDRC_CNF */ diff --git a/include/linux/usb/musb.h b/include/linux/usb/musb.h index d325a0d..630962c 100644 --- a/include/linux/usb/musb.h +++ b/include/linux/usb/musb.h @@ -19,6 +19,36 @@ enum musb_mode { struct clk; +struct musb_hdrc_eps_bits { + const char name[16]; + u8 bits; +}; + +struct musb_hdrc_config { + /* MUSB configuration-specific details */ + unsigned multipoint:1; /* multipoint device */ + unsigned dyn_fifo:1; /* supports dynamic fifo sizing */ + unsigned soft_con:1; /* soft connect required */ + unsigned utm_16:1; /* utm data witdh is 16 bits */ + unsigned big_endian:1; /* true if CPU uses big-endian */ + unsigned mult_bulk_tx:1; /* Tx ep required for multbulk pkts */ + unsigned mult_bulk_rx:1; /* Rx ep required for multbulk pkts */ + unsigned high_iso_tx:1; /* Tx ep required for HB iso */ + unsigned high_iso_rx:1; /* Rx ep required for HD iso */ + unsigned dma:1; /* supports DMA */ + unsigned vendor_req:1; /* vendor registers required */ + + u8 num_eps; /* number of endpoints _with_ ep0 */ + u8 dma_channels; /* number of dma channels */ + u8 dyn_fifo_size; /* dynamic size in bytes */ + u8 vendor_ctrl; /* vendor control reg width */ + u8 vendor_stat; /* vendor status reg witdh */ + u8 dma_req_chan; /* bitmask for required dma channels */ + u8 ram_bits; /* ram address size */ + + struct musb_hdrc_eps_bits *eps_bits; +}; + struct musb_hdrc_platform_data { /* MUSB_HOST, MUSB_PERIPHERAL, or MUSB_OTG */ u8 mode; @@ -38,16 +68,14 @@ struct musb_hdrc_platform_data { /* (HOST or OTG) msec/2 after VBUS on till power good */ u8 potpgt; - /* TBD: chip defaults should probably go someplace else, - * e.g. number of tx/rx endpoints, etc - */ - unsigned multipoint:1; - /* Power the device on or off */ int (*set_power)(int state); /* Turn device clock on or off */ int (*set_clock)(struct clk *clock, int is_on); + + /* MUSB configuration-specific details */ + struct musb_hdrc_config *config; }; -- 1.6.0.rc1.71.gfba5 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH 3/3] input: add more error checks to lm8323 driver 2008-08-13 17:36 ` [PATCH 2/3] usb: musb: pass configuration specifics via pdata Felipe Balbi @ 2008-08-13 17:36 ` Felipe Balbi 2008-08-14 13:34 ` Tony Lindgren 2008-08-18 23:43 ` [PATCH 2/3] usb: musb: pass configuration specifics via pdata Steve Sakoman 1 sibling, 1 reply; 10+ messages in thread From: Felipe Balbi @ 2008-08-13 17:36 UTC (permalink / raw) To: linux-omap; +Cc: Felipe Balbi If we can't reach the driver, we stop trying to probe it. Useful when building kernel for n800 and n810. n800 doesn't have lm8323, so that driver shouldn't probe there. Signed-off-by: Felipe Balbi <felipe.balbi@nokia.com> --- drivers/input/keyboard/lm8323.c | 42 +++++++++++++++++++++++++++++--------- 1 files changed, 32 insertions(+), 10 deletions(-) diff --git a/drivers/input/keyboard/lm8323.c b/drivers/input/keyboard/lm8323.c index 72bb587..3d10a0f 100644 --- a/drivers/input/keyboard/lm8323.c +++ b/drivers/input/keyboard/lm8323.c @@ -348,10 +348,10 @@ static void lm8323_process_error(struct lm8323_chip *lm) } } -static void lm8323_reset(struct lm8323_chip *lm) +static int lm8323_reset(struct lm8323_chip *lm) { /* The docs say we must pass 0xAA as the data byte. */ - lm8323_write(lm, 2, LM8323_CMD_RESET, 0xAA); + return lm8323_write(lm, 2, LM8323_CMD_RESET, 0xAA); } static int lm8323_configure(struct lm8323_chip *lm) @@ -360,6 +360,7 @@ static int lm8323_configure(struct lm8323_chip *lm) int clock = (CLK_SLOWCLKEN | CLK_RCPWM_EXTERNAL); int debounce = lm->debounce_time >> 2; int active = lm->active_time >> 2; + int ret; /* * Active time must be greater than the debounce time: if it's @@ -368,13 +369,25 @@ static int lm8323_configure(struct lm8323_chip *lm) if (debounce >= active) active = debounce + 3; - lm8323_write(lm, 2, LM8323_CMD_WRITE_CFG, 0); - lm8323_write(lm, 2, LM8323_CMD_WRITE_CLOCK, clock); - lm8323_write(lm, 2, LM8323_CMD_SET_KEY_SIZE, keysize); + ret = lm8323_write(lm, 2, LM8323_CMD_WRITE_CFG, 0); + if (ret) + goto err; + ret = lm8323_write(lm, 2, LM8323_CMD_WRITE_CLOCK, clock); + if (ret) + goto err; + ret = lm8323_write(lm, 2, LM8323_CMD_SET_KEY_SIZE, keysize); + if (ret) + goto err; lm8323_set_active_time(lm, lm->active_time); - lm8323_write(lm, 2, LM8323_CMD_SET_DEBOUNCE, debounce); - lm8323_write(lm, 3, LM8323_CMD_WRITE_PORT_STATE, 0xff, 0xff); - lm8323_write(lm, 3, LM8323_CMD_WRITE_PORT_SEL, 0, 0); + ret = lm8323_write(lm, 2, LM8323_CMD_SET_DEBOUNCE, debounce); + if (ret) + goto err; + ret = lm8323_write(lm, 3, LM8323_CMD_WRITE_PORT_STATE, 0xff, 0xff); + if (ret) + goto err; + ret = lm8323_write(lm, 3, LM8323_CMD_WRITE_PORT_SEL, 0, 0); + if (ret) + goto err; /* * Not much we can do about errors at this point, so just hope @@ -382,6 +395,11 @@ static int lm8323_configure(struct lm8323_chip *lm) */ return 0; + +err: + dev_err(&lm->client->dev, "failed to configure lm8323\n"); + + return ret; } /* @@ -721,7 +739,9 @@ static int lm8323_probe(struct i2c_client *client, else if (lm->active_time == -1) /* Disable sleep. */ lm->active_time = 0; - lm8323_reset(lm); + err = lm8323_reset(lm); + if (err) + goto fail2; /* Nothing's set up to service the IRQ yet, so just spin for max. * 100ms until we can configure. */ @@ -738,7 +758,9 @@ static int lm8323_probe(struct i2c_client *client, msleep(1); } - lm8323_configure(lm); + err = lm8323_configure(lm); + if (err) + goto fail2; /* If a true probe check the device */ if (lm8323_read_id(lm, data) != 0) { -- 1.6.0.rc1.71.gfba5 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 3/3] input: add more error checks to lm8323 driver 2008-08-13 17:36 ` [PATCH 3/3] input: add more error checks to lm8323 driver Felipe Balbi @ 2008-08-14 13:34 ` Tony Lindgren 2008-08-14 14:36 ` Tony Lindgren 0 siblings, 1 reply; 10+ messages in thread From: Tony Lindgren @ 2008-08-14 13:34 UTC (permalink / raw) To: Felipe Balbi; +Cc: linux-omap * Felipe Balbi <felipe.balbi@nokia.com> [080813 20:37]: > If we can't reach the driver, we stop trying to probe > it. Useful when building kernel for n800 and n810. > > n800 doesn't have lm8323, so that driver shouldn't probe > there. Pushing today. Tony > Signed-off-by: Felipe Balbi <felipe.balbi@nokia.com> > --- > drivers/input/keyboard/lm8323.c | 42 +++++++++++++++++++++++++++++--------- > 1 files changed, 32 insertions(+), 10 deletions(-) > > diff --git a/drivers/input/keyboard/lm8323.c b/drivers/input/keyboard/lm8323.c > index 72bb587..3d10a0f 100644 > --- a/drivers/input/keyboard/lm8323.c > +++ b/drivers/input/keyboard/lm8323.c > @@ -348,10 +348,10 @@ static void lm8323_process_error(struct lm8323_chip *lm) > } > } > > -static void lm8323_reset(struct lm8323_chip *lm) > +static int lm8323_reset(struct lm8323_chip *lm) > { > /* The docs say we must pass 0xAA as the data byte. */ > - lm8323_write(lm, 2, LM8323_CMD_RESET, 0xAA); > + return lm8323_write(lm, 2, LM8323_CMD_RESET, 0xAA); > } > > static int lm8323_configure(struct lm8323_chip *lm) > @@ -360,6 +360,7 @@ static int lm8323_configure(struct lm8323_chip *lm) > int clock = (CLK_SLOWCLKEN | CLK_RCPWM_EXTERNAL); > int debounce = lm->debounce_time >> 2; > int active = lm->active_time >> 2; > + int ret; > > /* > * Active time must be greater than the debounce time: if it's > @@ -368,13 +369,25 @@ static int lm8323_configure(struct lm8323_chip *lm) > if (debounce >= active) > active = debounce + 3; > > - lm8323_write(lm, 2, LM8323_CMD_WRITE_CFG, 0); > - lm8323_write(lm, 2, LM8323_CMD_WRITE_CLOCK, clock); > - lm8323_write(lm, 2, LM8323_CMD_SET_KEY_SIZE, keysize); > + ret = lm8323_write(lm, 2, LM8323_CMD_WRITE_CFG, 0); > + if (ret) > + goto err; > + ret = lm8323_write(lm, 2, LM8323_CMD_WRITE_CLOCK, clock); > + if (ret) > + goto err; > + ret = lm8323_write(lm, 2, LM8323_CMD_SET_KEY_SIZE, keysize); > + if (ret) > + goto err; > lm8323_set_active_time(lm, lm->active_time); > - lm8323_write(lm, 2, LM8323_CMD_SET_DEBOUNCE, debounce); > - lm8323_write(lm, 3, LM8323_CMD_WRITE_PORT_STATE, 0xff, 0xff); > - lm8323_write(lm, 3, LM8323_CMD_WRITE_PORT_SEL, 0, 0); > + ret = lm8323_write(lm, 2, LM8323_CMD_SET_DEBOUNCE, debounce); > + if (ret) > + goto err; > + ret = lm8323_write(lm, 3, LM8323_CMD_WRITE_PORT_STATE, 0xff, 0xff); > + if (ret) > + goto err; > + ret = lm8323_write(lm, 3, LM8323_CMD_WRITE_PORT_SEL, 0, 0); > + if (ret) > + goto err; > > /* > * Not much we can do about errors at this point, so just hope > @@ -382,6 +395,11 @@ static int lm8323_configure(struct lm8323_chip *lm) > */ > > return 0; > + > +err: > + dev_err(&lm->client->dev, "failed to configure lm8323\n"); > + > + return ret; > } > > /* > @@ -721,7 +739,9 @@ static int lm8323_probe(struct i2c_client *client, > else if (lm->active_time == -1) /* Disable sleep. */ > lm->active_time = 0; > > - lm8323_reset(lm); > + err = lm8323_reset(lm); > + if (err) > + goto fail2; > > /* Nothing's set up to service the IRQ yet, so just spin for max. > * 100ms until we can configure. */ > @@ -738,7 +758,9 @@ static int lm8323_probe(struct i2c_client *client, > > msleep(1); > } > - lm8323_configure(lm); > + err = lm8323_configure(lm); > + if (err) > + goto fail2; > > /* If a true probe check the device */ > if (lm8323_read_id(lm, data) != 0) { > -- > 1.6.0.rc1.71.gfba5 > > -- > 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 ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 3/3] input: add more error checks to lm8323 driver 2008-08-14 13:34 ` Tony Lindgren @ 2008-08-14 14:36 ` Tony Lindgren 0 siblings, 0 replies; 10+ messages in thread From: Tony Lindgren @ 2008-08-14 14:36 UTC (permalink / raw) To: Felipe Balbi; +Cc: linux-omap * Tony Lindgren <tony@atomide.com> [080814 16:32]: > * Felipe Balbi <felipe.balbi@nokia.com> [080813 20:37]: > > If we can't reach the driver, we stop trying to probe > > it. Useful when building kernel for n800 and n810. > > > > n800 doesn't have lm8323, so that driver shouldn't probe > > there. > > Pushing today. Actually this breaks the keyboard on n810 as lm8323_reset() fails during probe. And I guess n800 should not call probe now at all as the I2C device is not registered. So I'll revert this for now. Tony > > Tony > > > Signed-off-by: Felipe Balbi <felipe.balbi@nokia.com> > > --- > > drivers/input/keyboard/lm8323.c | 42 +++++++++++++++++++++++++++++--------- > > 1 files changed, 32 insertions(+), 10 deletions(-) > > > > diff --git a/drivers/input/keyboard/lm8323.c b/drivers/input/keyboard/lm8323.c > > index 72bb587..3d10a0f 100644 > > --- a/drivers/input/keyboard/lm8323.c > > +++ b/drivers/input/keyboard/lm8323.c > > @@ -348,10 +348,10 @@ static void lm8323_process_error(struct lm8323_chip *lm) > > } > > } > > > > -static void lm8323_reset(struct lm8323_chip *lm) > > +static int lm8323_reset(struct lm8323_chip *lm) > > { > > /* The docs say we must pass 0xAA as the data byte. */ > > - lm8323_write(lm, 2, LM8323_CMD_RESET, 0xAA); > > + return lm8323_write(lm, 2, LM8323_CMD_RESET, 0xAA); > > } > > > > static int lm8323_configure(struct lm8323_chip *lm) > > @@ -360,6 +360,7 @@ static int lm8323_configure(struct lm8323_chip *lm) > > int clock = (CLK_SLOWCLKEN | CLK_RCPWM_EXTERNAL); > > int debounce = lm->debounce_time >> 2; > > int active = lm->active_time >> 2; > > + int ret; > > > > /* > > * Active time must be greater than the debounce time: if it's > > @@ -368,13 +369,25 @@ static int lm8323_configure(struct lm8323_chip *lm) > > if (debounce >= active) > > active = debounce + 3; > > > > - lm8323_write(lm, 2, LM8323_CMD_WRITE_CFG, 0); > > - lm8323_write(lm, 2, LM8323_CMD_WRITE_CLOCK, clock); > > - lm8323_write(lm, 2, LM8323_CMD_SET_KEY_SIZE, keysize); > > + ret = lm8323_write(lm, 2, LM8323_CMD_WRITE_CFG, 0); > > + if (ret) > > + goto err; > > + ret = lm8323_write(lm, 2, LM8323_CMD_WRITE_CLOCK, clock); > > + if (ret) > > + goto err; > > + ret = lm8323_write(lm, 2, LM8323_CMD_SET_KEY_SIZE, keysize); > > + if (ret) > > + goto err; > > lm8323_set_active_time(lm, lm->active_time); > > - lm8323_write(lm, 2, LM8323_CMD_SET_DEBOUNCE, debounce); > > - lm8323_write(lm, 3, LM8323_CMD_WRITE_PORT_STATE, 0xff, 0xff); > > - lm8323_write(lm, 3, LM8323_CMD_WRITE_PORT_SEL, 0, 0); > > + ret = lm8323_write(lm, 2, LM8323_CMD_SET_DEBOUNCE, debounce); > > + if (ret) > > + goto err; > > + ret = lm8323_write(lm, 3, LM8323_CMD_WRITE_PORT_STATE, 0xff, 0xff); > > + if (ret) > > + goto err; > > + ret = lm8323_write(lm, 3, LM8323_CMD_WRITE_PORT_SEL, 0, 0); > > + if (ret) > > + goto err; > > > > /* > > * Not much we can do about errors at this point, so just hope > > @@ -382,6 +395,11 @@ static int lm8323_configure(struct lm8323_chip *lm) > > */ > > > > return 0; > > + > > +err: > > + dev_err(&lm->client->dev, "failed to configure lm8323\n"); > > + > > + return ret; > > } > > > > /* > > @@ -721,7 +739,9 @@ static int lm8323_probe(struct i2c_client *client, > > else if (lm->active_time == -1) /* Disable sleep. */ > > lm->active_time = 0; > > > > - lm8323_reset(lm); > > + err = lm8323_reset(lm); > > + if (err) > > + goto fail2; > > > > /* Nothing's set up to service the IRQ yet, so just spin for max. > > * 100ms until we can configure. */ > > @@ -738,7 +758,9 @@ static int lm8323_probe(struct i2c_client *client, > > > > msleep(1); > > } > > - lm8323_configure(lm); > > + err = lm8323_configure(lm); > > + if (err) > > + goto fail2; > > > > /* If a true probe check the device */ > > if (lm8323_read_id(lm, data) != 0) { > > -- > > 1.6.0.rc1.71.gfba5 > > > > -- > > 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 ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/3] usb: musb: pass configuration specifics via pdata 2008-08-13 17:36 ` [PATCH 2/3] usb: musb: pass configuration specifics via pdata Felipe Balbi 2008-08-13 17:36 ` [PATCH 3/3] input: add more error checks to lm8323 driver Felipe Balbi @ 2008-08-18 23:43 ` Steve Sakoman 2008-08-19 9:39 ` Felipe Balbi 1 sibling, 1 reply; 10+ messages in thread From: Steve Sakoman @ 2008-08-18 23:43 UTC (permalink / raw) To: Felipe Balbi, linux-omap@vger.kernel.org Does this patch require all board files to pass in platform_data or are there reasonable defaults for boards that don't? There are reports that this patch has broken musb on beagle. Regards, Steve On Wed, Aug 13, 2008 at 10:36 AM, Felipe Balbi <felipe.balbi@nokia.com> wrote: > Use platform_data to pass musb configuration-specific > details to musb driver. > > It's easier to maintain in the sense that neither board > will affect the other > > Signed-off-by: Felipe Balbi <felipe.balbi@nokia.com> > --- > arch/arm/mach-omap2/board-n800-usb.c | 46 +++++++++- > arch/arm/mach-omap2/usb-musb.c | 51 +++++++++- > arch/arm/mach-omap2/usb-tusb6010.c | 1 - > drivers/usb/musb/musb_core.c | 37 +++---- > drivers/usb/musb/musb_core.h | 14 +--- > drivers/usb/musb/tusb6010.h | 169 -------------------------------- > include/asm-arm/arch-omap/hdrc_cnf.h | 177 ---------------------------------- > include/linux/usb/musb.h | 38 +++++++- > 8 files changed, 144 insertions(+), 389 deletions(-) > delete mode 100644 include/asm-arm/arch-omap/hdrc_cnf.h > > diff --git a/arch/arm/mach-omap2/board-n800-usb.c b/arch/arm/mach-omap2/board-n800-usb.c > index 7599f64..16ea8fc 100644 > --- a/arch/arm/mach-omap2/board-n800-usb.c > +++ b/arch/arm/mach-omap2/board-n800-usb.c > @@ -35,14 +35,58 @@ static int tusb_set_clock(struct clk *osc_ck, int state); > # define BOARD_MODE MUSB_HOST > #endif > > +static struct musb_hdrc_eps_bits musb_eps[] = { > + { "ep1_tx", 5, }, > + { "ep1_rx", 5, }, > + { "ep2_tx", 5, }, > + { "ep2_rx", 5, }, > + { "ep3_tx", 3, }, > + { "ep3_rx", 3, }, > + { "ep4_tx", 3, }, > + { "ep4_rx", 3, }, > + { "ep5_tx", 2, }, > + { "ep5_rx", 2, }, > + { "ep6_tx", 2, }, > + { "ep6_rx", 2, }, > + { "ep7_tx", 2, }, > + { "ep7_rx", 2, }, > + { "ep8_tx", 2, }, > + { "ep8_rx", 2, }, > + { "ep9_tx", 2, }, > + { "ep9_rx", 2, }, > + { "ep10_tx", 2, }, > + { "ep10_rx", 2, }, > + { "ep11_tx", 2, }, > + { "ep11_rx", 2, }, > + { "ep12_tx", 2, }, > + { "ep12_rx", 2, }, > + { "ep13_tx", 2, }, > + { "ep13_rx", 2, }, > + { "ep14_tx", 2, }, > + { "ep14_rx", 2, }, > + { "ep15_tx", 2, }, > + { "ep15_rx", 2, }, > +}; > + > +static struct musb_hdrc_config musb_config = { > + .multipoint = 1, > + .dyn_fifo = 1, > + .soft_con = 1, > + .dma = 1, > + .num_eps = 32, > + .dma_channels = 7, > + .ram_bits = 12, > + .eps_bits = musb_eps, > +}; > + > static struct musb_hdrc_platform_data tusb_data = { > .mode = BOARD_MODE, > - .multipoint = 1, > .set_power = tusb_set_power, > .set_clock = tusb_set_clock, > .min_power = 25, /* x2 = 50 mA drawn from VBUS as peripheral */ > .power = 100, /* Max 100 mA VBUS for host mode */ > .clock = "osc_ck", > + .config = &musb_config, > }; > > /* > diff --git a/arch/arm/mach-omap2/usb-musb.c b/arch/arm/mach-omap2/usb-musb.c > index cbd59f8..842edb3 100644 > --- a/arch/arm/mach-omap2/usb-musb.c > +++ b/arch/arm/mach-omap2/usb-musb.c > @@ -37,7 +37,7 @@ static struct resource musb_resources[] = { > : OMAP243X_HS_BASE, > .end = cpu_is_omap34xx() > ? OMAP34XX_HSUSB_OTG_BASE + SZ_8K - 1 > - : OMAP243X_HS_BASE + SZ_8K -1, > + : OMAP243X_HS_BASE + SZ_8K - 1, > .flags = IORESOURCE_MEM, > }, > [1] = { /* general IRQ */ > @@ -73,6 +73,51 @@ static int musb_set_clock(struct clk *clk, int state) > return 0; > } > > +static struct musb_hdrc_eps_bits musb_eps[] = { > + { "ep1_tx", 10, }, > + { "ep1_rx", 10, }, > + { "ep2_tx", 9, }, > + { "ep2_rx", 9, }, > + { "ep3_tx", 3, }, > + { "ep3_rx", 3, }, > + { "ep4_tx", 3, }, > + { "ep4_rx", 3, }, > + { "ep5_tx", 3, }, > + { "ep5_rx", 3, }, > + { "ep6_tx", 3, }, > + { "ep6_rx", 3, }, > + { "ep7_tx", 3, }, > + { "ep7_rx", 3, }, > + { "ep8_tx", 2, }, > + { "ep8_rx", 2, }, > + { "ep9_tx", 2, }, > + { "ep9_rx", 2, }, > + { "ep10_tx", 2, }, > + { "ep10_rx", 2, }, > + { "ep11_tx", 2, }, > + { "ep11_rx", 2, }, > + { "ep12_tx", 2, }, > + { "ep12_rx", 2, }, > + { "ep13_tx", 2, }, > + { "ep13_rx", 2, }, > + { "ep14_tx", 2, }, > + { "ep14_rx", 2, }, > + { "ep15_tx", 2, }, > + { "ep15_rx", 2, }, > +}; > + > +static struct musb_hdrc_config musb_config = { > + .multipoint = 1, > + .dyn_fifo = 1, > + .soft_con = 1, > + .dma = 1, > + .num_eps = 32, > + .dma_channels = 7, > + .dma_req_chan = (1 << 0) | (1 << 1) | (1 << 2) | (1 << 3), > + .ram_bits = 12, > + .eps_bits = musb_eps, > +}; > + > static struct musb_hdrc_platform_data musb_plat = { > #ifdef CONFIG_USB_MUSB_OTG > .mode = MUSB_OTG, > @@ -81,18 +126,18 @@ static struct musb_hdrc_platform_data musb_plat = { > #elif defined(CONFIG_USB_GADGET_MUSB_HDRC) > .mode = MUSB_PERIPHERAL, > #endif > - .multipoint = 1, > .clock = cpu_is_omap34xx() > ? "hsotgusb_ick" > : "usbhs_ick", > .set_clock = musb_set_clock, > + .config = &musb_config, > }; > > static u64 musb_dmamask = ~(u32)0; > > static struct platform_device musb_device = { > .name = "musb_hdrc", > - .id = 0, > + .id = -1, > .dev = { > .dma_mask = &musb_dmamask, > .coherent_dma_mask = 0xffffffff, > diff --git a/arch/arm/mach-omap2/usb-tusb6010.c b/arch/arm/mach-omap2/usb-tusb6010.c > index 9924300..75e2669 100644 > --- a/arch/arm/mach-omap2/usb-tusb6010.c > +++ b/arch/arm/mach-omap2/usb-tusb6010.c > @@ -315,7 +315,6 @@ tusb6010_setup_interface(struct musb_hdrc_platform_data *data, > printk(error, 6, status); > return -ENODEV; > } > - data->multipoint = 1; > tusb_device.dev.platform_data = data; > > /* REVISIT let the driver know what DMA channels work */ > diff --git a/drivers/usb/musb/musb_core.c b/drivers/usb/musb/musb_core.c > index 1404a10..4f8aa1d 100644 > --- a/drivers/usb/musb/musb_core.c > +++ b/drivers/usb/musb/musb_core.c > @@ -990,12 +990,6 @@ static void musb_shutdown(struct platform_device *pdev) > * We don't currently use dynamic fifo setup capability to do anything > * more than selecting one of a bunch of predefined configurations. > */ > -#ifdef MUSB_C_DYNFIFO_DEF > -#define can_dynfifo() 1 > -#else > -#define can_dynfifo() 0 > -#endif > - > #if defined(CONFIG_USB_TUSB6010) || \ > defined(CONFIG_ARCH_OMAP2430) || defined(CONFIG_ARCH_OMAP34XX) > static ushort __initdata fifo_mode = 4; > @@ -1008,8 +1002,6 @@ module_param(fifo_mode, ushort, 0); > MODULE_PARM_DESC(fifo_mode, "initial endpoint configuration"); > > > -#define DYN_FIFO_SIZE (1<<(MUSB_C_RAM_BITS+2)) > - > enum fifo_style { FIFO_RXTX, FIFO_TX, FIFO_RX } __attribute__ ((packed)); > enum buf_mode { BUF_SINGLE, BUF_DOUBLE } __attribute__ ((packed)); > > @@ -1119,11 +1111,12 @@ fifo_setup(struct musb *musb, struct musb_hw_ep *hw_ep, > > c_size = size - 3; > if (cfg->mode == BUF_DOUBLE) { > - if ((offset + (maxpacket << 1)) > DYN_FIFO_SIZE) > + if ((offset + (maxpacket << 1)) > > + (1 << (musb->config->ram_bits + 2))) > return -EMSGSIZE; > c_size |= MUSB_FIFOSZ_DPB; > } else { > - if ((offset + maxpacket) > DYN_FIFO_SIZE) > + if ((offset + maxpacket) > (1 << (musb->config->ram_bits + 2))) > return -EMSGSIZE; > } > > @@ -1219,13 +1212,13 @@ static int __init ep_config_from_table(struct musb *musb) > /* assert(offset > 0) */ > > /* NOTE: for RTL versions >= 1.400 EPINFO and RAMINFO would > - * be better than static MUSB_C_NUM_EPS and DYN_FIFO_SIZE... > + * be better than static musb->config->num_eps and DYN_FIFO_SIZE... > */ > > for (i = 0; i < n; i++) { > u8 epn = cfg->hw_ep_num; > > - if (epn >= MUSB_C_NUM_EPS) { > + if (epn >= musb->config->num_eps) { > pr_debug("%s: invalid ep %d\n", > musb_driver_name, epn); > continue; > @@ -1242,8 +1235,8 @@ static int __init ep_config_from_table(struct musb *musb) > > printk(KERN_DEBUG "%s: %d/%d max ep, %d/%d memory\n", > musb_driver_name, > - n + 1, MUSB_C_NUM_EPS * 2 - 1, > - offset, DYN_FIFO_SIZE); > + n + 1, musb->config->num_eps * 2 - 1, > + offset, (1 << (musb->config->ram_bits + 2))); > > #ifdef CONFIG_USB_MUSB_HDRC_HCD > if (!musb->bulk_ep) { > @@ -1270,7 +1263,7 @@ static int __init ep_config_from_hw(struct musb *musb) > > /* FIXME pick up ep0 maxpacket size */ > > - for (epnum = 1; epnum < MUSB_C_NUM_EPS; epnum++) { > + for (epnum = 1; epnum < musb->config->num_eps; epnum++) { > musb_ep_select(mbase, epnum); > hw_ep = musb->endpoints + epnum; > > @@ -1424,14 +1417,14 @@ static int __init musb_core_init(u16 musb_type, struct musb *musb) > musb->epmask = 1; > > if (reg & MUSB_CONFIGDATA_DYNFIFO) { > - if (can_dynfifo()) > + if (musb->config->dyn_fifo) > status = ep_config_from_table(musb); > else { > ERR("reconfigure software for Dynamic FIFOs\n"); > status = -ENODEV; > } > } else { > - if (!can_dynfifo()) > + if (!musb->config->dyn_fifo) > status = ep_config_from_hw(musb); > else { > ERR("reconfigure software for static FIFOs\n"); > @@ -1788,7 +1781,8 @@ static void musb_irq_work(struct work_struct *data) > */ > > static struct musb *__init > -allocate_instance(struct device *dev, void __iomem *mbase) > +allocate_instance(struct device *dev, > + struct musb_hdrc_config *config, void __iomem *mbase) > { > struct musb *musb; > struct musb_hw_ep *ep; > @@ -1820,8 +1814,9 @@ allocate_instance(struct device *dev, void __iomem *mbase) > musb->mregs = mbase; > musb->ctrl_base = mbase; > musb->nIrq = -ENODEV; > + musb->config = config; > for (epnum = 0, ep = musb->endpoints; > - epnum < MUSB_C_NUM_EPS; > + epnum < musb->config->num_eps; > epnum++, ep++) { > > ep->musb = musb; > @@ -1932,7 +1927,7 @@ bad_config: > } > > /* allocate */ > - musb = allocate_instance(dev, ctrl); > + musb = allocate_instance(dev, plat->config, ctrl); > if (!musb) > return -ENOMEM; > > @@ -1990,7 +1985,7 @@ bad_config: > musb_generic_disable(musb); > > /* setup musb parts of the core (especially endpoints) */ > - status = musb_core_init(plat->multipoint > + status = musb_core_init(plat->config->multipoint > ? MUSB_CONTROLLER_MHDRC > : MUSB_CONTROLLER_HDRC, musb); > if (status < 0) > diff --git a/drivers/usb/musb/musb_core.h b/drivers/usb/musb/musb_core.h > index c2cd5a9..73a794a 100644 > --- a/drivers/usb/musb/musb_core.h > +++ b/drivers/usb/musb/musb_core.h > @@ -56,18 +56,6 @@ struct musb_ep; > #include "musb_debug.h" > #include "musb_dma.h" > > -#ifdef CONFIG_USB_MUSB_SOC > -/* > - * Get core configuration from a header converted (by cfg_conv) > - * from the Verilog config file generated by the core config utility > - * > - * For now we assume that header is provided along with other > - * arch-specific files. Discrete chips will need a build tweak. > - * So will using AHB IDs from silicon that provides them. > - */ > -#include <asm/arch/hdrc_cnf.h> > -#endif > - > #include "musb_io.h" > #include "musb_regs.h" > > @@ -439,6 +427,8 @@ struct musb { > struct usb_gadget_driver *gadget_driver; /* its driver */ > #endif > > + struct musb_hdrc_config *config; > + > #ifdef MUSB_CONFIG_PROC_FS > struct proc_dir_entry *proc_entry; > #endif > diff --git a/drivers/usb/musb/tusb6010.h b/drivers/usb/musb/tusb6010.h > index db6dad0..ab8c962 100644 > --- a/drivers/usb/musb/tusb6010.h > +++ b/drivers/usb/musb/tusb6010.h > @@ -230,173 +230,4 @@ extern u8 tusb_get_revision(struct musb *musb); > #define TUSB_REV_30 0x30 > #define TUSB_REV_31 0x31 > > -/*----------------------------------------------------------------------------*/ > - > -#ifdef CONFIG_USB_TUSB6010 > - > -/* configuration parameters specific to this silicon */ > - > -/* Number of Tx endpoints. Legal values are 1 - 16 (this value includes EP0) */ > -#define MUSB_C_NUM_EPT 16 > - > -/* Number of Rx endpoints. Legal values are 1 - 16 (this value includes EP0) */ > -#define MUSB_C_NUM_EPR 16 > - > -/* Endpoint 1 to 15 direction types. C_EP1_DEF is defined if either Tx endpoint > - * 1 or Rx endpoint 1 are used. > - */ > -#define MUSB_C_EP1_DEF > - > -/* C_EP1_TX_DEF is defined if Tx endpoint 1 is used */ > -#define MUSB_C_EP1_TX_DEF > - > -/* C_EP1_RX_DEF is defined if Rx endpoint 1 is used */ > -#define MUSB_C_EP1_RX_DEF > - > -/* C_EP1_TOR_DEF is defined if Tx endpoint 1 and Rx endpoint 1 share a FIFO */ > -/* #define C_EP1_TOR_DEF */ > - > -/* C_EP1_TAR_DEF is defined if both Tx endpoint 1 and Rx endpoint 1 are used > - * and do not share a FIFO. > - */ > -#define MUSB_C_EP1_TAR_DEF > - > -/* Similarly for all other used endpoints */ > -#define MUSB_C_EP2_DEF > -#define MUSB_C_EP2_TX_DEF > -#define MUSB_C_EP2_RX_DEF > -#define MUSB_C_EP2_TAR_DEF > -#define MUSB_C_EP3_DEF > -#define MUSB_C_EP3_TX_DEF > -#define MUSB_C_EP3_RX_DEF > -#define MUSB_C_EP3_TAR_DEF > -#define MUSB_C_EP4_DEF > -#define MUSB_C_EP4_TX_DEF > -#define MUSB_C_EP4_RX_DEF > -#define MUSB_C_EP4_TAR_DEF > - > -/* Endpoint 1 to 15 FIFO address bits. Legal values are 3 to 13 - corresponding > - * to FIFO sizes of 8 to 8192 bytes. If an Tx endpoint shares a FIFO with an Rx > - * endpoint then the Rx FIFO size must be the same as the Tx FIFO size. All > - * endpoints 1 to 15 must be defined, unused endpoints should be set to 2. > - */ > -#define MUSB_C_EP1T_BITS 5 > -#define MUSB_C_EP1R_BITS 5 > -#define MUSB_C_EP2T_BITS 5 > -#define MUSB_C_EP2R_BITS 5 > -#define MUSB_C_EP3T_BITS 3 > -#define MUSB_C_EP3R_BITS 3 > -#define MUSB_C_EP4T_BITS 3 > -#define MUSB_C_EP4R_BITS 3 > - > -#define MUSB_C_EP5T_BITS 2 > -#define MUSB_C_EP5R_BITS 2 > -#define MUSB_C_EP6T_BITS 2 > -#define MUSB_C_EP6R_BITS 2 > -#define MUSB_C_EP7T_BITS 2 > -#define MUSB_C_EP7R_BITS 2 > -#define MUSB_C_EP8T_BITS 2 > -#define MUSB_C_EP8R_BITS 2 > -#define MUSB_C_EP9T_BITS 2 > -#define MUSB_C_EP9R_BITS 2 > -#define MUSB_C_EP10T_BITS 2 > -#define MUSB_C_EP10R_BITS 2 > -#define MUSB_C_EP11T_BITS 2 > -#define MUSB_C_EP11R_BITS 2 > -#define MUSB_C_EP12T_BITS 2 > -#define MUSB_C_EP12R_BITS 2 > -#define MUSB_C_EP13T_BITS 2 > -#define MUSB_C_EP13R_BITS 2 > -#define MUSB_C_EP14T_BITS 2 > -#define MUSB_C_EP14R_BITS 2 > -#define MUSB_C_EP15T_BITS 2 > -#define MUSB_C_EP15R_BITS 2 > - > -/* Define the following constant if the USB2.0 Transceiver Macrocell data width > - * is 16-bits. > - */ > -/* #define C_UTM_16 */ > - > -/* Define this constant if the CPU uses big-endian byte ordering. */ > -/* #define C_BIGEND */ > - > -/* Define the following constant if any Tx endpoint is required to support > - * multiple bulk packets. > - */ > -/* #define C_MP_TX */ > - > -/* Define the following constant if any Rx endpoint is required to support > - * multiple bulk packets. > - */ > -/* #define C_MP_RX */ > - > -/* Define the following constant if any Tx endpoint is required to support high > - * bandwidth ISO. > - */ > -/* #define C_HB_TX */ > - > -/* Define the following constant if any Rx endpoint is required to support high > - * bandwidth ISO. > - */ > -/* #define C_HB_RX */ > - > -/* Define the following constant if software connect/disconnect control is > - * required. > - */ > -#define MUSB_C_SOFT_CON > - > -/* Define the following constant if Vendor Control Registers are required. */ > -/* #define C_VEND_REG */ > - > -/* Vendor control register widths. */ > -#define MUSB_C_VCTL_BITS 4 > -#define MUSB_C_VSTAT_BITS 8 > - > -/* Define the following constant to include a DMA controller. */ > -/* #define C_DMA */ > - > -/* Define the following constant if 2 or more DMA channels are required. */ > -/* #define C_DMA2 */ > - > -/* Define the following constant if 3 or more DMA channels are required. */ > -/* #define C_DMA3 */ > - > -/* Define the following constant if 4 or more DMA channels are required. */ > -/* #define C_DMA4 */ > - > -/* Define the following constant if 5 or more DMA channels are required. */ > -/* #define C_DMA5 */ > - > -/* Define the following constant if 6 or more DMA channels are required. */ > -/* #define C_DMA6 */ > - > -/* Define the following constant if 7 or more DMA channels are required. */ > -/* #define C_DMA7 */ > - > -/* Define the following constant if 8 or more DMA channels are required. */ > -/* #define C_DMA8 */ > - > -/* Enable Dynamic FIFO Sizing */ > -#define MUSB_C_DYNFIFO_DEF > - > -/* Derived constants. The following constants are derived from the previous > - * configuration constants > - */ > - > -/* Total number of endpoints. Legal values are 2 - 16. This must be equal to > - * the larger of C_NUM_EPT, C_NUM_EPR > - */ > -/* #define MUSB_C_NUM_EPS 5 */ > - > -/* C_EPMAX_BITS is equal to the largest endpoint FIFO word address bits */ > -#define MUSB_C_EPMAX_BITS 11 > - > -/* C_RAM_BITS is the number of address bits required to address the RAM (32-bit > - * addresses). It is defined as log2 of the sum of 2** of all the endpoint FIFO > - * dword address bits (rounded up). > - */ > -#define MUSB_C_RAM_BITS 12 > - > -#endif /* CONFIG_USB_TUSB6010 */ > - > #endif /* __TUSB6010_H__ */ > diff --git a/include/asm-arm/arch-omap/hdrc_cnf.h b/include/asm-arm/arch-omap/hdrc_cnf.h > deleted file mode 100644 > index 74c8432..0000000 > --- a/include/asm-arm/arch-omap/hdrc_cnf.h > +++ /dev/null > @@ -1,177 +0,0 @@ > -/* > - * Copyright 2005 Mentor Graphics Corporation > - * USB High-Speed Multi-Point Dual-Role Controller Configuration > - * > - * Copyright Mentor Graphics Corporation and Licensors 2004 > - * Copyright (C) 2005 by Texas Instruments > - * > - * This file contains configuration constants for the (m)hdrc > - * silicon as integrated into DaVinci CPUs. > - */ > - > -#ifndef __ARCH_MUSB_HDRC_CNF > -#define __ARCH_MUSB_HDRC_CNF > - > -/* ** Number of Tx endpoints ** */ > -/* Legal values are 1 - 16 (this value includes EP0) */ > -#define MUSB_C_NUM_EPT 8 > - > -/* ** Number of Rx endpoints ** */ > -/* Legal values are 1 - 16 (this value includes EP0) */ > -#define MUSB_C_NUM_EPR 8 > - > -/* ** Endpoint 1 to 15 direction types ** */ > -/* C_EP1_DEF is defined if either Tx endpoint 1 or Rx endpoint 1 are used */ > -#define MUSB_C_EP1_DEF > - > -/* C_EP1_TX_DEF is defined if Tx endpoint 1 is used */ > -#define MUSB_C_EP1_TX_DEF > - > -/* C_EP1_RX_DEF is defined if Rx endpoint 1 is used */ > -#define MUSB_C_EP1_RX_DEF > - > -/* C_EP1_TOR_DEF is defined if Tx endpoint 1 and Rx endpoint 1 share a FIFO */ > -/*`define C_EP1_TOR_DEF */ > - > -/* C_EP1_TAR_DEF is defined if both Tx endpoint 1 and Rx endpoint 1 are used */ > -/* and do not share a FIFO */ > -#define MUSB_C_EP1_TAR_DEF > - > -/* Similarly for all other used endpoints */ > -#define MUSB_C_EP2_DEF > -#define MUSB_C_EP2_TX_DEF > -#define MUSB_C_EP2_RX_DEF > -#define MUSB_C_EP2_TAR_DEF > -#define MUSB_C_EP3_DEF > -#define MUSB_C_EP3_TX_DEF > -#define MUSB_C_EP3_RX_DEF > -#define MUSB_C_EP3_TAR_DEF > -#define MUSB_C_EP4_DEF > -#define MUSB_C_EP4_TX_DEF > -#define MUSB_C_EP4_RX_DEF > -#define MUSB_C_EP4_TAR_DEF > -#define MUSB_C_EP5_DEF > -#define MUSB_C_EP5_TX_DEF > -#define MUSB_C_EP5_RX_DEF > -#define MUSB_C_EP5_TAR_DEF > -#define MUSB_C_EP6_DEF > -#define MUSB_C_EP6_TX_DEF > -#define MUSB_C_EP6_RX_DEF > -#define MUSB_C_EP6_TAR_DEF > -#define MUSB_C_EP7_DEF > -#define MUSB_C_EP7_TX_DEF > -#define MUSB_C_EP7_RX_DEF > -#define MUSB_C_EP7_TAR_DEF > - > -/* ** Endpoint 1 to 15 FIFO address bits ** */ > -/* Legal values are 3 to 13 - corresponding to FIFO sizes of 8 to 8192 bytes. */ > -/* If an Tx endpoint shares a FIFO with an Rx endpoint then the Rx FIFO size */ > -/* must be the same as the Tx FIFO size. */ > -/* All endpoints 1 to 15 must be defined, unused endpoints should be set to 2. */ > -#define MUSB_C_EP1T_BITS 10 > -#define MUSB_C_EP1R_BITS 10 > -#define MUSB_C_EP2T_BITS 9 > -#define MUSB_C_EP2R_BITS 9 > -#define MUSB_C_EP3T_BITS 3 > -#define MUSB_C_EP3R_BITS 3 > -#define MUSB_C_EP4T_BITS 3 > -#define MUSB_C_EP4R_BITS 3 > -#define MUSB_C_EP5T_BITS 3 > -#define MUSB_C_EP5R_BITS 3 > -#define MUSB_C_EP6T_BITS 3 > -#define MUSB_C_EP6R_BITS 3 > -#define MUSB_C_EP7T_BITS 3 > -#define MUSB_C_EP7R_BITS 3 > -#define MUSB_C_EP8T_BITS 2 > -#define MUSB_C_EP8R_BITS 2 > -#define MUSB_C_EP9T_BITS 2 > -#define MUSB_C_EP9R_BITS 2 > -#define MUSB_C_EP10T_BITS 2 > -#define MUSB_C_EP10R_BITS 2 > -#define MUSB_C_EP11T_BITS 2 > -#define MUSB_C_EP11R_BITS 2 > -#define MUSB_C_EP12T_BITS 2 > -#define MUSB_C_EP12R_BITS 2 > -#define MUSB_C_EP13T_BITS 2 > -#define MUSB_C_EP13R_BITS 2 > -#define MUSB_C_EP14T_BITS 2 > -#define MUSB_C_EP14R_BITS 2 > -#define MUSB_C_EP15T_BITS 2 > -#define MUSB_C_EP15R_BITS 2 > - > -/* Define the following constant if the USB2.0 Transceiver Macrocell data width is 16-bits. */ > -/* `define C_UTM_16 */ > - > -/* Define this constant if the CPU uses big-endian byte ordering. */ > -/*`define C_BIGEND */ > - > -/* Define the following constant if any Tx endpoint is required to support multiple bulk packets. */ > -/* `define C_MP_TX */ > - > -/* Define the following constant if any Rx endpoint is required to support multiple bulk packets. */ > -/* `define C_MP_RX */ > - > -/* Define the following constant if any Tx endpoint is required to support high bandwidth ISO. */ > -/* `define C_HB_TX */ > - > -/* Define the following constant if any Rx endpoint is required to support high bandwidth ISO. */ > -/* `define C_HB_RX */ > - > -/* Define the following constant if software connect/disconnect control is required. */ > -#define MUSB_C_SOFT_CON > - > -/* Define the following constant if Vendor Control Registers are required. */ > -/* `define C_VEND_REG */ > - > -/* Vendor control register widths. */ > -#define MUSB_C_VCTL_BITS 4 > -#define MUSB_C_VSTAT_BITS 8 > - > - > -/* Define the following constant to include a DMA controller. */ > -#define MUSB_C_DMA > - > -/* Define the following constant if 2 or more DMA channels are required. */ > -#define MUSB_C_DMA2 > - > -/* Define the following constant if 3 or more DMA channels are required. */ > -#define MUSB_C_DMA3 > - > -/* Define the following constant if 4 or more DMA channels are required. */ > -#define MUSB_C_DMA4 > - > -/* Define the following constant if 5 or more DMA channels are required. */ > -/*`define C_DMA5 */ > - > -/* Define the following constant if 6 or more DMA channels are required. */ > -/*`define C_DMA6 */ > - > -/* Define the following constant if 7 or more DMA channels are required. */ > -/*`define C_DMA7 */ > - > -/* Define the following constant if 8 or more DMA channels are required. */ > -/*`define C_DMA8 */ > - > - > -/* ** Enable Dynamic FIFO Sizing ** */ > -#define MUSB_C_DYNFIFO_DEF > - > -/* ** Derived constants ** */ > -/* The following constants are derived from the previous configuration constants */ > - > -/* Total number of endpoints > - * Legal values are 2 - 16 > - * This must be equal to the larger of C_NUM_EPT, C_NUM_EPR > - */ > -#define MUSB_C_NUM_EPS 8 > - > -/* C_EPMAX_BITS is equal to the largest endpoint FIFO word address bits */ > -#define MUSB_C_EPMAX_BITS 12 > - > -/* C_RAM_BITS is the number of address bits required to address the RAM (32-bit > - * addresses). It is defined as log2 of the sum of 2** of all the endpoint FIFO > - * dword address bits (rounded up). > - */ > -#define MUSB_C_RAM_BITS 12 > - > -#endif /* __ARCH_MUSB_HDRC_CNF */ > diff --git a/include/linux/usb/musb.h b/include/linux/usb/musb.h > index d325a0d..630962c 100644 > --- a/include/linux/usb/musb.h > +++ b/include/linux/usb/musb.h > @@ -19,6 +19,36 @@ enum musb_mode { > > struct clk; > > +struct musb_hdrc_eps_bits { > + const char name[16]; > + u8 bits; > +}; > + > +struct musb_hdrc_config { > + /* MUSB configuration-specific details */ > + unsigned multipoint:1; /* multipoint device */ > + unsigned dyn_fifo:1; /* supports dynamic fifo sizing */ > + unsigned soft_con:1; /* soft connect required */ > + unsigned utm_16:1; /* utm data witdh is 16 bits */ > + unsigned big_endian:1; /* true if CPU uses big-endian */ > + unsigned mult_bulk_tx:1; /* Tx ep required for multbulk pkts */ > + unsigned mult_bulk_rx:1; /* Rx ep required for multbulk pkts */ > + unsigned high_iso_tx:1; /* Tx ep required for HB iso */ > + unsigned high_iso_rx:1; /* Rx ep required for HD iso */ > + unsigned dma:1; /* supports DMA */ > + unsigned vendor_req:1; /* vendor registers required */ > + > + u8 num_eps; /* number of endpoints _with_ ep0 */ > + u8 dma_channels; /* number of dma channels */ > + u8 dyn_fifo_size; /* dynamic size in bytes */ > + u8 vendor_ctrl; /* vendor control reg width */ > + u8 vendor_stat; /* vendor status reg witdh */ > + u8 dma_req_chan; /* bitmask for required dma channels */ > + u8 ram_bits; /* ram address size */ > + > + struct musb_hdrc_eps_bits *eps_bits; > +}; > + > struct musb_hdrc_platform_data { > /* MUSB_HOST, MUSB_PERIPHERAL, or MUSB_OTG */ > u8 mode; > @@ -38,16 +68,14 @@ struct musb_hdrc_platform_data { > /* (HOST or OTG) msec/2 after VBUS on till power good */ > u8 potpgt; > > - /* TBD: chip defaults should probably go someplace else, > - * e.g. number of tx/rx endpoints, etc > - */ > - unsigned multipoint:1; > - > /* Power the device on or off */ > int (*set_power)(int state); > > /* Turn device clock on or off */ > int (*set_clock)(struct clk *clock, int is_on); > + > + /* MUSB configuration-specific details */ > + struct musb_hdrc_config *config; > }; > > > -- > 1.6.0.rc1.71.gfba5 > > -- > 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 > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/3] usb: musb: pass configuration specifics via pdata 2008-08-18 23:43 ` [PATCH 2/3] usb: musb: pass configuration specifics via pdata Steve Sakoman @ 2008-08-19 9:39 ` Felipe Balbi 2008-08-19 14:21 ` Steve Sakoman 0 siblings, 1 reply; 10+ messages in thread From: Felipe Balbi @ 2008-08-19 9:39 UTC (permalink / raw) To: ext Steve Sakoman; +Cc: Felipe Balbi, linux-omap@vger.kernel.org On Mon, Aug 18, 2008 at 04:43:43PM -0700, ext Steve Sakoman wrote: > Does this patch require all board files to pass in platform_data or > are there reasonable defaults for boards that don't? > > There are reports that this patch has broken musb on beagle. It's default for SOC- and for TUSB-based musb boards. Last time i sent this patch as RFC I asked people to test and everybody (including beagleboard people) had reported it was ok. I've been testing this patch for over a month now on an omap3 board. If you find any issues, please report the problem. -- balbi ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/3] usb: musb: pass configuration specifics via pdata 2008-08-19 9:39 ` Felipe Balbi @ 2008-08-19 14:21 ` Steve Sakoman 2008-08-19 14:40 ` Felipe Balbi 0 siblings, 1 reply; 10+ messages in thread From: Steve Sakoman @ 2008-08-19 14:21 UTC (permalink / raw) To: felipe.balbi, linux-omap@vger.kernel.org > It's default for SOC- and for TUSB-based musb boards. Last time i sent > this patch as RFC I asked people to test and everybody (including > beagleboard people) had reported it was ok. I've been testing it on the Gumstix Overo board (with no board specific platform-data) and have also seen no issues. I asked because I had noticed that a few people reported USB issues on the #beagle irc channel. One specifically mentioned that this patch caused his issue. So I just wanted to verify that board specific platform data was not required for all omap3 boards. Thanks, Steve ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/3] usb: musb: pass configuration specifics via pdata 2008-08-19 14:21 ` Steve Sakoman @ 2008-08-19 14:40 ` Felipe Balbi 0 siblings, 0 replies; 10+ messages in thread From: Felipe Balbi @ 2008-08-19 14:40 UTC (permalink / raw) To: ext Steve Sakoman; +Cc: felipe.balbi, linux-omap@vger.kernel.org On Tue, Aug 19, 2008 at 07:21:21AM -0700, ext Steve Sakoman wrote: > I've been testing it on the Gumstix Overo board (with no board > specific platform-data) and have also seen no issues. > > I asked because I had noticed that a few people reported USB issues on > the #beagle irc channel. One specifically mentioned that this patch > caused his issue. So I just wanted to verify that board specific > platform data was not required for all omap3 boards. point him to linux-usb mailing list. It's the best place for us to discuss usb related problems :-) also, was him using linux-omap or the beagle provided kernel ? Did he bisect to point out exactly this patch been the root cause ? I'm saying that cuz people had more than one month for trying out this patch and now that it's applied on both linux-omap and mainline the problems start to appear. Quite awkward but ok, not everybody follows the patches. Anyways, I didn't have any problems with this patch since I started using it. It passes testusb (left running for two days in a row using test.sh). If there really is an issue, let's narrow it down and fix it. -- balbi ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2008-08-19 14:39 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2008-08-13 17:36 [PATCH 0/3] my pending patches Felipe Balbi 2008-08-13 17:36 ` [PATCH 1/3] input: keypad: General fixes to omap-twl4030keypad.c Felipe Balbi 2008-08-13 17:36 ` [PATCH 2/3] usb: musb: pass configuration specifics via pdata Felipe Balbi 2008-08-13 17:36 ` [PATCH 3/3] input: add more error checks to lm8323 driver Felipe Balbi 2008-08-14 13:34 ` Tony Lindgren 2008-08-14 14:36 ` Tony Lindgren 2008-08-18 23:43 ` [PATCH 2/3] usb: musb: pass configuration specifics via pdata Steve Sakoman 2008-08-19 9:39 ` Felipe Balbi 2008-08-19 14:21 ` Steve Sakoman 2008-08-19 14:40 ` Felipe Balbi
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox