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]

 



On Tue, Dec 15, 2009 at 02:34:32PM +0100, Balbi Felipe (Nokia-D/Helsinki) wrote:
Hi,

On Tue, Dec 15, 2009 at 02:31:46PM +0100, ext Ajay Kumar Gupta wrote:
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 */

same problem as the other patch I sent:

$ git grep -w dyn_fifo arch/
arch/arm/mach-davinci/usb.c:    .dyn_fifo       = true,
arch/arm/mach-omap2/usb-musb.c: .dyn_fifo       = 1,
arch/blackfin/mach-bf527/boards/cm_bf527.c:     .dyn_fifo       = 0,
arch/blackfin/mach-bf527/boards/ezbrd.c:        .dyn_fifo       = 0,
arch/blackfin/mach-bf527/boards/ezkit.c:        .dyn_fifo       = 0,
arch/blackfin/mach-bf548/boards/cm_bf548.c:     .dyn_fifo       = 0,
arch/blackfin/mach-bf548/boards/ezkit.c:        .dyn_fifo       = 0,

we have to agree who will carry the arch changes.

what we can do is mark that field as deprecated and give one major release for people to remove that from their boards, then we send a patch finally removing it.

I'm talking about:

diff --git a/include/linux/usb/musb.h b/include/linux/usb/musb.h
index d437556..c3ca147 100644
--- a/include/linux/usb/musb.h
+++ b/include/linux/usb/musb.h
@@ -30,7 +30,7 @@ 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        dyn_fifo:1 __deprecated;        /* 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 */

how do you feel about that ?

--
balbi
--
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

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux