> error = scsi_dh_add_device(sdev); > - if (error) > - /* > - * device_handler is optional, so any error can be ignored > - */ > - sdev_printk(KERN_INFO, sdev, > - "failed to add device handler: %d\n", error); > + /* device_handler is optional, so any error can be ignored */ In that case it should return void.