On Mon, Nov 04, 2013 at 11:21:42PM +0100, Alexander Aring wrote: > Hi Sascha, > > On Mon, Nov 04, 2013 at 03:04:26PM +0100, Sascha Hauer wrote: > .... > > > > - entry->me.display = asprintf("%-20s %-20s %s", devname, hwdevname, > > + entry->me.display = asprintf("%-20s %-20s %s", > > + devname ? devname : "", > > + hwdevname ? hwdevname : "", > > blspec_entry_var_get(entry, "title")); > > + > > free(devname); > > free(hwdevname); > > > > @@ -277,7 +281,7 @@ static int blspec_scan_cdev(struct blspec *blspec, struct cdev *cdev) > > if (IS_ERR(rootpath)) > > return PTR_ERR(rootpath); > > > > - ret = blspec_scan_directory(blspec, rootpath, cdev); > > + ret = blspec_scan_directory(blspec, rootpath); > > > > return ret; > unnecessary assign of ret here, but it's not important... Changed anyway. Thanks Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox