redonkable
/
alistair23-linux
1
0
Fork 0
Code Releases Activity

[SCSI] ncr53c8xx: Call scsi_host_put in release 

Since ncr53c8xx_attach() calls scsi_host_put(), make ncr53c8xx_release()
call scsi_host_put() too, for symmetry.  Both callers already expect
it to put the host for them, so that works out nicely.  While the zalon
driver does 'use' the host pointer afterwards, it only compares it for
equality and doesn't dereference it, so that's safe.

While I'm at it, get rid of pointless checks for NULL, use shost_priv()
and change ncr53c8xx_release to return void.

Signed-off-by: Matthew Wilcox <matthew@wil.cx>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
hifive-unleashed-5.1
Matthew Wilcox 2007-08-15 12:56:56 -06:00 committed by James Bottomley
parent 6fdea8dbbe
commit dffe807cdb
2 changed files with 5 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
drivers/scsi/ncr53c8xx.c
View File

@ -8528,18 +8528,15 @@ struct Scsi_Host * __init ncr_attach(struct scsi_host_template *tpnt,
}
int ncr53c8xx_release(struct Scsi_Host *host)
void ncr53c8xx_release(struct Scsi_Host *host)
{
struct host_data *host_data;
struct host_data *host_data = shost_priv(host);
#ifdef DEBUG_NCR53C8XX
printk("ncr53c8xx: release\n");
#endif
if (!host)
return 1;
host_data = (struct host_data *)host->hostdata;
if (host_data && host_data->ncb)
if (host_data->ncb)
ncr_detach(host_data->ncb);
return 1;
scsi_host_put(host);
}
static void ncr53c8xx_set_period(struct scsi_target *starget, int period)

2
drivers/scsi/ncr53c8xx.h
View File

@ -1321,7 +1321,7 @@ struct ncr_device {
};
extern struct Scsi_Host *ncr_attach(struct scsi_host_template *tpnt, int unit, struct ncr_device *device);
extern int ncr53c8xx_release(struct Scsi_Host *host);
extern void ncr53c8xx_release(struct Scsi_Host *host);
irqreturn_t ncr53c8xx_intr(int irq, void *dev_id);
extern int ncr53c8xx_init(void);
extern void ncr53c8xx_exit(void);