On Tue, Nov 28, 2006 at 03:08:30PM +0900, Akinobu Mita wrote: > On Mon, Nov 27, 2006 at 12:34:52PM -0800, Mark Gross wrote: > > > > > > tlclk_device = platform_device_register_simple("telco_clock", > > > -1, NULL, 0); > > > - if (!tlclk_device) { > > > + if (IS_ERR(tlclk_device)) { > > ok > > > > > printk(KERN_ERR "tlclk: platform_device_register failed.\n"); > > > - ret = -EBUSY; > > > + ret = PTR_ERR(tlclk_device); > > > > I don't know about this but I could be wrong. Please convince me. > > We expect platform_device_register_simple() returns proper errno as pointer > when it fails.
What's wrong with EBUSY? --mgross - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/