> like this? generally.... > buf = wdperror(wd->drvp, wd->sc_wdc_bio.r_error); ... > diskerr(bp, "wd", buf ? buf : "nomem" , LOG_PRINTF, > wd->sc_wdc_bio.blkdone, wd->sc_dk.dk_label); Would simplify things a bit. David -- David Laight: david@l8s.co.uk