Hi, > -----Original Message----- > From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap- > owner@xxxxxxxxxxxxxxx] On Behalf Of Mankad, Maulik Ojas > Sent: Wednesday, December 30, 2009 4:15 PM > To: linux-omap@xxxxxxxxxxxxxxx > Cc: linux-usb@xxxxxxxxxxxxxxx; Mankad, Maulik Ojas; Felipe Balbi; Tony > Lindgren; Greg Kroah-Hartman > Subject: [PATCH 1/2] ARM : OMAP: MUSB : Pass board specific data using a > structure > > ARM : OMAP: MUSB :Pass board specific data from board file > > This patch proposes to pass board specific data for MUSB > (like interface_type, mode etc) from board file by defining > board specific structure. > > Each board file can define this structure based on > its requirement and pass this information to the > driver. > > It addresses a comment from Felipe and thus help > by preventing addition of too many fields in > musb_platform_data. > > Signed-off-by: Maulik Mankad <x0082077@xxxxxx> > Cc: Felipe Balbi <felipe.balbi@xxxxxxxxx> > Cc: Tony Lindgren <tony@xxxxxxxxxxx> > Cc: Greg Kroah-Hartman <gregkh@xxxxxxx> > > Index: felipe_musb/arch/arm/mach-omap2/board-2430sdp.c > =================================================================== > --- felipe_musb.orig/arch/arm/mach-omap2/board-2430sdp.c > +++ felipe_musb/arch/arm/mach-omap2/board-2430sdp.c > @@ -195,6 +195,12 @@ static struct twl4030_hsmmc_info mmc[] _ > {} /* Terminator */ > }; > > +static struct musb_omap_data omap2430sdp_data = { > +.interface_type = MUSB_INTERFACE_ULPI, > +.mode = MUSB_OTG, > +.power = 100, > +} > + > static void __init omap_2430sdp_init(void) > { > int ret; > @@ -204,7 +210,7 @@ static void __init omap_2430sdp_init(voi > platform_add_devices(sdp2430_devices, ARRAY_SIZE(sdp2430_devices)); > omap_serial_init(); > twl4030_mmc_init(mmc); > - usb_musb_init(MUSB_OTG, 100); > + usb_musb_init(&omap2430sdp_data); > board_smc91x_init(); > > /* Turn off secondary LCD backlight */ > Index: felipe_musb/arch/arm/mach-omap2/board-3430sdp.c > =================================================================== > --- felipe_musb.orig/arch/arm/mach-omap2/board-3430sdp.c > +++ felipe_musb/arch/arm/mach-omap2/board-3430sdp.c > @@ -651,6 +651,12 @@ static struct omap_board_mux board_mux[] > #define board_mux NULL > #endif > > +static struct musb_omap_data omap3430sdp_data = { > +.interface_type = MUSB_INTERFACE_ULPI, > +.mode = MUSB_OTG, > +.power = 100, > +}; > + > static void __init omap_3430sdp_init(void) > { > omap3_mux_init(board_mux, OMAP_PACKAGE_CBB); > @@ -665,7 +671,7 @@ static void __init omap_3430sdp_init(voi > ARRAY_SIZE(sdp3430_spi_board_info)); > ads7846_dev_init(); > omap_serial_init(); > - usb_musb_init(MUSB_OTG, 100); > + usb_musb_init(&omap3430sdp_data); Isn't it simpler to use same name across the boards like, 'musb_board_data' Instead of 'omap3430sdp_data'. > board_smc91x_init(); > sdp3430_display_init(); > enable_board_wakeup_source(); > Index: felipe_musb/arch/arm/mach-omap2/board-cm-t35.c > =================================================================== > --- felipe_musb.orig/arch/arm/mach-omap2/board-cm-t35.c > +++ felipe_musb/arch/arm/mach-omap2/board-cm-t35.c > @@ -576,6 +576,12 @@ static struct omap_board_mux board_mux[] > { .reg_offset = OMAP_MUX_TERMINATOR }, > }; > > +static struct musb_omap_data omapcm_t35_data = { > +.interface_type = MUSB_INTERFACE_ULPI, > +.mode = MUSB_OTG, > +.power = 100, > +}; > + > static void __init cm_t35_init(void) > { > omap3_mux_init(board_mux, OMAP_PACKAGE_CUS); > @@ -586,7 +592,7 @@ static void __init cm_t35_init(void) > cm_t35_init_ethernet(); > cm_t35_init_led(); > > - usb_musb_init(MUSB_OTG, 100); > + usb_musb_init(&omapcm_t35_data); > } > > MACHINE_START(CM_T35, "Compulab CM-T35") > Index: felipe_musb/arch/arm/mach-omap2/board-igep0020.c > =================================================================== > --- felipe_musb.orig/arch/arm/mach-omap2/board-igep0020.c > +++ felipe_musb/arch/arm/mach-omap2/board-igep0020.c > @@ -212,12 +212,18 @@ static struct omap_board_mux board_mux[] > #define board_mux NULL > #endif > > +static struct musb_omap_data omapigep2_data = { > +.interface_type = MUSB_INTERFACE_ULPI, > +.mode = MUSB_OTG, > +.power = 100, > +}; > + > static void __init igep2_init(void) > { > omap3_mux_init(board_mux, OMAP_PACKAGE_CBB); > igep2_i2c_init(); > omap_serial_init(); > - usb_musb_init(MUSB_OTG, 100); > + usb_musb_init(&omapigep2_data); > > igep2_init_smsc911x(); > > Index: felipe_musb/arch/arm/mach-omap2/board-ldp.c > =================================================================== > --- felipe_musb.orig/arch/arm/mach-omap2/board-ldp.c > +++ felipe_musb/arch/arm/mach-omap2/board-ldp.c > @@ -384,6 +384,12 @@ static struct omap_board_mux board_mux[] > #define board_mux NULL > #endif > > +static struct musb_omap_data omapldp_data = { > +.interface_type = MUSB_INTERFACE_ULPI, > +.mode = MUSB_OTG, > +.power = 100, > +}; > + > static void __init omap_ldp_init(void) > { > omap3_mux_init(board_mux, OMAP_PACKAGE_CBB); > @@ -395,7 +401,7 @@ static void __init omap_ldp_init(void) > ARRAY_SIZE(ldp_spi_board_info)); > ads7846_dev_init(); > omap_serial_init(); > - usb_musb_init(MUSB_OTG, 100); > + usb_musb_init(&omapldp_data); > > twl4030_mmc_init(mmc); > /* link regulators to MMC adapters */ > Index: felipe_musb/arch/arm/mach-omap2/board-omap3beagle.c > =================================================================== > --- felipe_musb.orig/arch/arm/mach-omap2/board-omap3beagle.c > +++ felipe_musb/arch/arm/mach-omap2/board-omap3beagle.c > @@ -431,6 +431,12 @@ static struct omap_board_mux board_mux[] > #define board_mux NULL > #endif > > +static struct musb_omap_data omap3beagle_data = { > +.interface_type = MUSB_INTERFACE_ULPI, > +.mode = MUSB_OTG, > +.power = 100, > +}; > + > static void __init omap3_beagle_init(void) > { > omap3_mux_init(board_mux, OMAP_PACKAGE_CBB); > @@ -444,7 +450,7 @@ static void __init omap3_beagle_init(voi > /* REVISIT leave DVI powered down until it's needed ... */ > gpio_direction_output(170, true); > > - usb_musb_init(MUSB_OTG, 100); > + usb_musb_init(&omap3beagle_data); > usb_ehci_init(&ehci_pdata); > omap3beagle_flash_init(); > > Index: felipe_musb/arch/arm/mach-omap2/board-omap3evm.c > =================================================================== > --- felipe_musb.orig/arch/arm/mach-omap2/board-omap3evm.c > +++ felipe_musb/arch/arm/mach-omap2/board-omap3evm.c > @@ -431,6 +431,12 @@ static struct omap_board_mux board_mux[] > #define board_mux NULL > #endif > > +static struct musb_omap_data omap3evm_data = { > +.interface_type = MUSB_INTERFACE_ULPI, > +.mode = MUSB_OTG, > +.power = 100, > +}; > + > static void __init omap3_evm_init(void) > { > omap3_evm_get_revision(); > @@ -470,7 +476,7 @@ static void __init omap3_evm_init(void) > omap_mux_init_gpio(135, OMAP_PIN_OUTPUT); > ehci_pdata.reset_gpio_port[1] = 135; > } > - usb_musb_init(MUSB_OTG, 100); > + usb_musb_init(&omap3evm_data); > usb_ehci_init(&ehci_pdata); > ads7846_dev_init(); > omap3evm_init_smsc911x(); > Index: felipe_musb/arch/arm/mach-omap2/board-omap3pandora.c > =================================================================== > --- felipe_musb.orig/arch/arm/mach-omap2/board-omap3pandora.c > +++ felipe_musb/arch/arm/mach-omap2/board-omap3pandora.c > @@ -402,6 +402,12 @@ static struct omap_board_mux board_mux[] > #define board_mux NULL > #endif > > +static struct musb_omap_data omap3pandora_data = { > +.interface_type = MUSB_INTERFACE_ULPI, > +.mode = MUSB_OTG, > +.power = 100, > +}; > + > static void __init omap3pandora_init(void) > { > omap3_mux_init(board_mux, OMAP_PACKAGE_CBB); > @@ -414,7 +420,7 @@ static void __init omap3pandora_init(voi > omap3pandora_ads7846_init(); > usb_ehci_init(&ehci_pdata); > pandora_keys_gpio_init(); > - usb_musb_init(MUSB_OTG, 100); > + usb_musb_init(&omap3pandora_data); > > /* Ensure SDRC pins are mux'd for self-refresh */ > omap_mux_init_signal("sdrc_cke0", OMAP_PIN_OUTPUT); > Index: felipe_musb/arch/arm/mach-omap2/board-omap3touchbook.c > =================================================================== > --- felipe_musb.orig/arch/arm/mach-omap2/board-omap3touchbook.c > +++ felipe_musb/arch/arm/mach-omap2/board-omap3touchbook.c > @@ -528,6 +528,12 @@ static void __init early_touchbook_revis > } > __early_param("tbr=", early_touchbook_revision); > > +static struct musb_omap_data omap3touchbook_data = { > +.interface_type = MUSB_INTERFACE_ULPI, > +.mode = MUSB_OTG, > +.power = 100, > +}; > + > static void __init omap3_touchbook_init(void) > { > pm_power_off = omap3_touchbook_poweroff; > @@ -546,7 +552,7 @@ static void __init omap3_touchbook_init( > spi_register_board_info(omap3_ads7846_spi_board_info, > ARRAY_SIZE(omap3_ads7846_spi_board_info)); > omap3_ads7846_init(); > - usb_musb_init(MUSB_OTG, 100); > + usb_musb_init(&omap3touchbook_data); > usb_ehci_init(&ehci_pdata); > omap3touchbook_flash_init(); > > Index: felipe_musb/arch/arm/mach-omap2/board-overo.c > =================================================================== > --- felipe_musb.orig/arch/arm/mach-omap2/board-overo.c > +++ felipe_musb/arch/arm/mach-omap2/board-overo.c > @@ -414,6 +414,12 @@ static struct omap_board_mux board_mux[] > #define board_mux NULL > #endif > > +static struct musb_omap_data omapovero_data = { > +.interface_type = MUSB_INTERFACE_ULPI, > +.mode = MUSB_OTG, > +.power = 100, > +}; > + > static void __init overo_init(void) > { > omap3_mux_init(board_mux, OMAP_PACKAGE_CBB); > @@ -421,7 +427,7 @@ static void __init overo_init(void) > platform_add_devices(overo_devices, ARRAY_SIZE(overo_devices)); > omap_serial_init(); > overo_flash_init(); > - usb_musb_init(MUSB_OTG, 100); > + usb_musb_init(&omapovero_data); > usb_ehci_init(&ehci_pdata); > overo_ads7846_init(); > overo_init_smsc911x(); > Index: felipe_musb/arch/arm/mach-omap2/board-rx51.c > =================================================================== > --- felipe_musb.orig/arch/arm/mach-omap2/board-rx51.c > +++ felipe_musb/arch/arm/mach-omap2/board-rx51.c > @@ -79,11 +79,16 @@ static struct omap_board_mux board_mux[] > #define board_mux NULL > #endif > > +static struct musb_omap_data omaprx51_data = { > +.interface_type = MUSB_INTERFACE_ULPI, > +.mode = MUSB_PERIPHERAL, > +.power = 0, > +}; > static void __init rx51_init(void) > { > omap3_mux_init(board_mux, OMAP_PACKAGE_CBB); > omap_serial_init(); > - usb_musb_init(MUSB_PERIPHERAL, 0); > + usb_musb_init(&omaprx51_data); > rx51_peripherals_init(); > > /* Ensure SDRC pins are mux'd for self-refresh */ > Index: felipe_musb/arch/arm/mach-omap2/board-zoom-peripherals.c > =================================================================== > --- felipe_musb.orig/arch/arm/mach-omap2/board-zoom-peripherals.c > +++ felipe_musb/arch/arm/mach-omap2/board-zoom-peripherals.c > @@ -264,9 +264,15 @@ static int __init omap_i2c_init(void) > return 0; > } > > +static struct musb_omap_data omapzoom_data = { > +.interface_type = MUSB_INTERFACE_ULPI, > +.mode = MUSB_OTG, > +.power = 100, > +}; > + > void __init zoom_peripherals_init(void) > { > omap_i2c_init(); > omap_serial_init(); > - usb_musb_init(MUSB_OTG, 100); > + usb_musb_init(&omapzoom_data); > } > Index: felipe_musb/arch/arm/plat-omap/include/plat/usb.h > =================================================================== > --- felipe_musb.orig/arch/arm/plat-omap/include/plat/usb.h > +++ felipe_musb/arch/arm/plat-omap/include/plat/usb.h > @@ -43,7 +43,15 @@ struct ehci_hcd_omap_platform_data { > #define UDC_BASE OMAP2_UDC_BASE > #define OMAP_OHCI_BASE OMAP2_OHCI_BASE > > -extern void usb_musb_init(enum musb_mode mode, unsigned power); > +struct musb_omap_data { > +u8 interface_type; > +u8 mode; > +u8 power; > +}; It's board data so how about 'struct omap_musb_board_data'. > + > +enum musb_interface {MUSB_INTERFACE_ULPI, MUSB_INTERFACE_UTMI}; > + > +extern void usb_musb_init(void *); Can be modified to: +extern void usb_musb_init(omap_musb_board_data *); > > extern void usb_ehci_init(struct ehci_hcd_omap_platform_data *pdata); > > Index: felipe_musb/arch/arm/mach-omap2/usb-musb.c > =================================================================== > --- felipe_musb.orig/arch/arm/mach-omap2/usb-musb.c > +++ felipe_musb/arch/arm/mach-omap2/usb-musb.c > @@ -75,8 +75,10 @@ static struct platform_device musb_devic > .resource = musb_resources, > }; > > -void __init usb_musb_init(enum musb_mode mode, unsigned power) > +void __init usb_musb_init(void *arch_data) It's actually board data so *board_data is clearer. > { > + struct musb_omap_data *data = arch_data; > + Required ? we can anyway use 'arch_data' (or board_data). > if (cpu_is_omap243x()) > musb_resources[0].start = OMAP243X_HS_BASE; > else > @@ -88,15 +90,16 @@ void __init usb_musb_init(enum musb_mode > * musb_core.c have been converted to use use clkdev. > */ > musb_plat.clock = "ick"; > - musb_plat.power = power >> 1; > - musb_plat.mode = mode; > + musb_plat.arch_data = arch_data; Not needed. > + musb_plat.power = data->power >> 1; > + musb_plat.mode = data->mode; > > if (platform_device_register(&musb_device) < 0) > printk(KERN_ERR "Unable to register HS-USB (MUSB) device\n"); > } > > #else > -void __init usb_musb_init(enum musb_mode mode, unsigned power) > +void __init usb_musb_init(void *arch_data) > { > } > #endif /* CONFIG_USB_MUSB_SOC */ > Index: felipe_musb/include/linux/usb/musb.h > =================================================================== > --- felipe_musb.orig/include/linux/usb/musb.h > +++ felipe_musb/include/linux/usb/musb.h > @@ -84,6 +84,9 @@ struct musb_hdrc_platform_data { > > /* MUSB configuration-specific details */ > struct musb_hdrc_config *config; > + > + /* Board specific data */ > + void *arch_data; We don't need it as we will continue using 'power', 'mode', 'extvbus' >From musb_plat only. > }; > > > -- > 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-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html