On Tue, Sep 25, 2012 at 11:35:14AM +0900, Jaehoon Chung wrote: > On 09/24/2012 04:22 PM, Sascha Hauer wrote: > > /* > > * If your platform has 8-bit width support but is not a v3 controller, > > * or if it requires special setup code, you should implement that in > > - * platform_8bit_width(). > > + * platform_bus_width(). > > */ > > - if (host->ops->platform_8bit_width) > > - host->ops->platform_8bit_width(host, ios->bus_width); > > - else { > > + if (host->ops->platform_bus_width) { > > + host->ops->platform_bus_width(host, ios->bus_width); > > + } else { > Why add the brackets? Took the chance to cleanup the codingstyle when changing these lines anyway. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html