[SCSI] handle scsi_add_host failure for aic7xxx and fix compiler warning
Add scsi_add_host() failure handling for aic7xxx Also silence a compiler warning : drivers/scsi/aic7xxx/aic7xxx_osm.c: In function `ahc_linux_register_host': drivers/scsi/aic7xxx/aic7xxx_osm.c:1100: warning: ignoring return value of `scsi_add_host', declared with attribute warn_unused_result Signed-off-by: Jesper Juhl <jesper.juhl@gmail.com> Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
This commit is contained in:
committed by
James Bottomley
parent
e7a1ca1d27
commit
9e1fe9314c
@@ -1065,6 +1065,7 @@ ahc_linux_register_host(struct ahc_softc *ahc, struct scsi_host_template *templa
|
|||||||
struct Scsi_Host *host;
|
struct Scsi_Host *host;
|
||||||
char *new_name;
|
char *new_name;
|
||||||
u_long s;
|
u_long s;
|
||||||
|
int retval;
|
||||||
|
|
||||||
template->name = ahc->description;
|
template->name = ahc->description;
|
||||||
host = scsi_host_alloc(template, sizeof(struct ahc_softc *));
|
host = scsi_host_alloc(template, sizeof(struct ahc_softc *));
|
||||||
@@ -1097,9 +1098,16 @@ ahc_linux_register_host(struct ahc_softc *ahc, struct scsi_host_template *templa
|
|||||||
|
|
||||||
host->transportt = ahc_linux_transport_template;
|
host->transportt = ahc_linux_transport_template;
|
||||||
|
|
||||||
scsi_add_host(host, (ahc->dev_softc ? &ahc->dev_softc->dev : NULL)); /* XXX handle failure */
|
retval = scsi_add_host(host,
|
||||||
|
(ahc->dev_softc ? &ahc->dev_softc->dev : NULL));
|
||||||
|
if (retval) {
|
||||||
|
printk(KERN_WARNING "aic7xxx: scsi_add_host failed\n");
|
||||||
|
scsi_host_put(host);
|
||||||
|
return retval;
|
||||||
|
}
|
||||||
|
|
||||||
scsi_scan_host(host);
|
scsi_scan_host(host);
|
||||||
return (0);
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
|
Reference in New Issue
Block a user