> > Boards using the TWL4030 regulator may not all use the LDOs the same way
> > (e.g. MMC2 power can be controlled by another LDO than VMMC2).
> > This delegates TWL4030 MMC power initializations to board-specific
> > functions,
> > that may still call twl4030_power_mmc_init for the default behavior.
> [snip]
> > +int board_mmc_power_init(void)
> > +{
> > +#if defined(CONFIG_TWL4030_POWER)
> > + twl4030_power_mmc_init();
> > + mdelay(100); /* ramp-up delay from Linux code */
> > +#endif
> > + return 0;
> > +}
> > #endif
>
> I think we can do away with the #if defined(CONFIG_TWL4030_POWER) part
> here since all of these boards will fail pretty spectacularly without
> that being set and if someone does remove that a link error is better
> than a crazy runtime problem.That makes sense, I'll do that in v5. > [snip] > > diff --git a/drivers/mmc/omap_hsmmc.c b/drivers/mmc/omap_hsmmc.c > > index 5b0c302..6e93a07 100644 > > --- a/drivers/mmc/omap_hsmmc.c > > +++ b/drivers/mmc/omap_hsmmc.c > > @@ -130,12 +130,7 @@ static unsigned char mmc_board_init(struct mmc *mmc) > > pbias_lite = readl(&t2_base->pbias_lite); > > pbias_lite &= ~(PBIASLITEPWRDNZ1 | PBIASLITEPWRDNZ0); > > writel(pbias_lite, &t2_base->pbias_lite); > > -#endif > > -#if defined(CONFIG_TWL4030_POWER) > > - twl4030_power_mmc_init(); > > - mdelay(100); /* ramp-up delay from Linux code */ > > -#endif > > -#if defined(CONFIG_OMAP34XX) > > + > > writel(pbias_lite | PBIASLITEPWRDNZ1 | > > PBIASSPEEDCTRL0 | PBIASLITEPWRDNZ0, > > &t2_base->pbias_lite); > > Since it's not clear from the context here, it's OK to drop the check on > CONFIG_OMAP34XX here since there's already a test on it above where diff > cut us off? Thanks! Indeed, that's the case (not sure this was actually more of a question than a statement). Thanks for the review! -- Paul Kocialkowski, Replicant developer Replicant is a fully free Android distribution Website: http://www.replicant.us/ Redmine: http://redmine.replicant.us/
signature.asc
Description: This is a digitally signed message part
_______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
