Andrew, On Thu, Apr 11, 2013 at 11:36 AM, Andrew Bresticker <abrestic@xxxxxxxxxxxx> wrote: > + if (pdata->probe) { > + err = pdata->probe(pdata); > + if (err) { > + dev_err(dev, "platform probe failed: %d\n", err); It would probably be good not to print in the case of -EPROBE_DEFER? Otherwise this looks good to me. -Doug -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html