Messages in this thread | | | From | Andi Kleen <> | Subject | Re: [PATCH][5/5][resend] floppy.c: Fix device_create_file() warning | Date | Mon, 19 Mar 2007 16:20:00 +0100 |
| |
On Monday 19 March 2007 16:16, Jesper Juhl wrote: > On 19/03/07, Andi Kleen <ak@suse.de> wrote: > > > > > > > > - device_create_file(&floppy_device[drive].dev,&dev_attr_cmos); > > > + err = device_create_file(&floppy_device[drive].dev, &dev_attr_cmos); > > > + if (err) > > > + printk(KERN_WARNING "Unable to create sysfs attribute " > > > + "file for floppy device: %s\n", > > > + floppy_device[drive].name); > > > > That change looks pretty useless. Either the error should be handled correctly > > by bailing out or the warn_unused_results should be dropped. > > > At least letting the user know that something failed is better than > the current situation of just failing silently I'd say.
I don't think so. That's just bloat.
-Andi - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |