RE: [PATCH 1/4] musb: get rid of unneeded musb->config->dyn_fifo

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

 



> -----Original Message-----
> From: Tony Lindgren [mailto:tony@xxxxxxxxxxx]
> Sent: Tuesday, December 15, 2009 11:10 PM
> To: Gupta, Ajay Kumar
> Cc: linux-usb@xxxxxxxxxxxxxxx; linux-omap@xxxxxxxxxxxxxxx;
> felipe.balbi@xxxxxxxxx
> Subject: Re: [PATCH 1/4] musb: get rid of unneeded musb->config->dyn_fifo
> 
> * Ajay Kumar Gupta <ajay.gupta@xxxxxx> [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,

I did test but with this arch specific compilation fixes. 

We are still waiting a response from Greg if he will accept these arch specific changes with USB patch set.

-Ajay
> 
> Tony
> 
> > Signed-off-by: Ajay Kumar Gupta <ajay.gupta@xxxxxx>
> > ---
> >  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 majordomo@xxxxxxxxxxxxxxx
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
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