On Mon, Mar 19, 2012 at 4:39 PM, Sergei Shtylyov <[email protected]> wrote: > Hello. > > > On 19-03-2012 14:06, Felipe Balbi wrote: > >>> + ver = usbtll_read(base, OMAP_USBTLL_REVISION); >>> + if (ver == OMAP_USBTLL_REV1) >>> + count = OMAP_TLL_CHANNEL_COUNT; >>> + else if (ver == OMAP_USBTLL_REV2) >>> + count = OMAP_REV2_TLL_CHANNEL_COUNT; >>> + else { >>> + dev_err(dev, "TLL version failed\n"); >>> + ret = -ENODEV; >>> + goto err_ioremap; >>> + } > > >> wrong coding style. > > > And *switch* seems more fitting here. > > WBR, Sergei
Thanks , I will do this, -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
