On Sat, Jan 10, 2009 at 11:49:42PM +0100, Ladislav Michl wrote: > On Sun, Dec 07, 2008 at 01:47:47PM -0800, Tony Lindgren wrote: > > diff --git a/arch/arm/mach-omap1/board-h2-mmc.c b/arch/arm/mach-omap1/board-h2-mmc.c > > index 504ae88..409fa56 100644 > > --- a/arch/arm/mach-omap1/board-h2-mmc.c > > +++ b/arch/arm/mach-omap1/board-h2-mmc.c > [snip] > > +static void mmc_shutdown(struct device *dev) > > +{ > > + gpio_free(H2_TPS_GPIO_MMC_PWR_EN); > > +} > > + > > +/* > > + * H2 could use the following functions tested: > > + * - mmc_get_cover_state that uses OMAP_MPUIO(1) > > + * - mmc_get_wp that uses OMAP_MPUIO(3) > > + */ > > +static struct omap_mmc_platform_data mmc1_data = { > > + .nr_slots = 1, > > + .init = mmc_late_init, > > + .shutdown = mmc_shutdown, > It seems that shutdown is no-op, so what about patch below? After all, lets remove some unused fields. Signed-off-by: Ladislav Michl <ladis@xxxxxxxxxxxxxx> diff --git a/arch/arm/plat-omap/include/mach/mmc.h b/arch/arm/plat-omap/include/mach/mmc.h index 031250f..1129e97 100644 --- a/arch/arm/plat-omap/include/mach/mmc.h +++ b/arch/arm/plat-omap/include/mach/mmc.h @@ -51,7 +51,6 @@ struct omap_mmc_platform_data { * not supported */ int (* init)(struct device *dev); void (* cleanup)(struct device *dev); - void (* shutdown)(struct device *dev); /* To handle board related suspend/resume functionality for MMC */ int (*suspend)(struct device *dev, int slot); @@ -77,10 +76,6 @@ struct omap_mmc_platform_data { /* use the internal clock */ unsigned internal_clock:1; - s16 power_pin; - - int switch_pin; /* gpio (card detect) */ - int gpio_wp; /* gpio (write protect) */ int (* set_bus_mode)(struct device *dev, int slot, int bus_mode); int (* set_power)(struct device *dev, int slot, int power_on, int vdd); diff --git a/drivers/mmc/host/omap.c b/drivers/mmc/host/omap.c index 67d7b7f..84de289 100644 --- a/drivers/mmc/host/omap.c +++ b/drivers/mmc/host/omap.c @@ -157,8 +157,6 @@ struct mmc_omap_host { struct timer_list dma_timer; unsigned dma_len; - short power_pin; - struct mmc_omap_slot *slots[OMAP_MMC_MAX_SLOTS]; struct mmc_omap_slot *current_slot; spinlock_t slot_lock; -- 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