* Ajay Kumar Gupta <[email protected]> [091215 05:31]:
> We can get dynamic FIFO information from CONFIGDATA register and
> thus there is no need for any user defiend dyn_fifo.

Have you tested this?

$ grep dyn_fifo arch/arm/mach-omap2/*.c
arch/arm/mach-omap2/usb-musb.c: .dyn_fifo       = 1,

Tony
 
> Signed-off-by: Ajay Kumar Gupta <[email protected]>
> ---
>  drivers/usb/musb/musb_core.c |   19 ++++---------------
>  include/linux/usb/musb.h     |    1 -
>  2 files changed, 4 insertions(+), 16 deletions(-)
> 
> diff --git a/drivers/usb/musb/musb_core.c b/drivers/usb/musb/musb_core.c
> index 49f2346..2858940 100644
> --- a/drivers/usb/musb/musb_core.c
> +++ b/drivers/usb/musb/musb_core.c
> @@ -1404,21 +1404,10 @@ static int __init musb_core_init(u16 musb_type, 
> struct musb *musb)
>       musb->nr_endpoints = 1;
>       musb->epmask = 1;
>  
> -     if (reg & MUSB_CONFIGDATA_DYNFIFO) {
> -             if (musb->config->dyn_fifo)
> -                     status = ep_config_from_table(musb);
> -             else {
> -                     ERR("reconfigure software for Dynamic FIFOs\n");
> -                     status = -ENODEV;
> -             }
> -     } else {
> -             if (!musb->config->dyn_fifo)
> -                     status = ep_config_from_hw(musb);
> -             else {
> -                     ERR("reconfigure software for static FIFOs\n");
> -                     return -ENODEV;
> -             }
> -     }
> +     if (reg & MUSB_CONFIGDATA_DYNFIFO)
> +             status = ep_config_from_table(musb);
> +     else
> +             status = ep_config_from_hw(musb);
>  
>       if (status < 0)
>               return status;
> diff --git a/include/linux/usb/musb.h b/include/linux/usb/musb.h
> index d437556..c5f006e 100644
> --- a/include/linux/usb/musb.h
> +++ b/include/linux/usb/musb.h
> @@ -30,7 +30,6 @@ struct musb_hdrc_eps_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 */
> -- 
> 1.6.2.4
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to [email protected]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to