>> @@ -258,7 +256,6 @@ static int usbtll_omap_probe(struct platform_device *pdev) >> GFP_KERNEL); >> if (!tll->ch_clk) { >> ret = -ENOMEM; >> - dev_err(dev, "Couldn't allocate memory for channel clocks\n"); > > I'd either leave this one, just to know which allocation failed or better use > something like this … Are you aware on the structure for a Linux allocation failure report? Regards, Markus -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html