Re: [PATCH 08/28] usb: musb: pass fifo_mode from hw glue driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Thu, Dec 16, 2010 at 06:06:25PM +0800, Ming Lei wrote:
2010/12/16 Felipe Balbi <balbi@xxxxxx>:
On Thu, Dec 16, 2010 at 02:14:35PM +0800, Ming Lei wrote:

 struct musb_hdrc_config {
+       struct musb_fifo_cfg    *fifo_cfg;      /* board fifo configuration
*/
+       unsigned                fifo_cfg_size;  /* size of the fifo
configuration */
+
       /* MUSB configuration-specific details */
       unsigned        multipoint:1;   /* multipoint device */
       unsigned        dyn_fifo:1 __deprecated; /* supports dynamic fifo
sizing */

the thing is that we don't want to "bloat" musb_core.c with several
different fifo_modes and have each glue layer change that, instead we
can let board initialization data pass in an optmized version of the
fifo_mode the usecases the board will be used for.

But seems the commit can't meet my requirement, I just want to remove
the config options below to support one set of musb binary drivers for
multiple musb hw controller:

hmmm, I see. Ideally we wouldn't have that, those are only sane defaults
anyway.

/*musb_core.c*/
#if defined(CONFIG_USB_MUSB_TUSB6010) || defined(CONFIG_USB_MUSB_OMAP2PLUS) \
   || defined(CONFIG_USB_MUSB_AM35X)
static ushort __initdata fifo_mode = 4;
#else
static ushort __initdata fifo_mode = 2;
#endif

Any suggestions?

none whatsoever :-( But we need to drop those, for sure. Although I
think the way you proposed is kinda duplication, I think it's fine. At
least drops the ifdeferry.

--
balbi
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux