diff options
author | Finn Thain <fthain@telegraphics.com.au> | 2014-11-12 00:12:00 -0500 |
---|---|---|
committer | Christoph Hellwig <hch@lst.de> | 2014-11-20 03:11:09 -0500 |
commit | d572f65fdf78a6dcb55b86ad8684f88830bf2e08 (patch) | |
tree | c2a690e481eabd36133589b0bd26d16e640dcf3a /drivers/scsi/mac_scsi.c | |
parent | a9c2dc43c14cc9e9333d451bc4db8a827a695332 (diff) |
ncr5380: Remove pointless compiler command line override macros
Compile-time override of scsi host defaults is pointless for drivers that
provide module parameters and __setup options for that. Too many macros make
the code hard to read so remove them.
Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Tested-by: Michael Schmitz <schmitzmic@gmail.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'drivers/scsi/mac_scsi.c')
-rw-r--r-- | drivers/scsi/mac_scsi.c | 19 |
1 files changed, 9 insertions, 10 deletions
diff --git a/drivers/scsi/mac_scsi.c b/drivers/scsi/mac_scsi.c index a27216d534ed..f00c987478ec 100644 --- a/drivers/scsi/mac_scsi.c +++ b/drivers/scsi/mac_scsi.c | |||
@@ -177,13 +177,12 @@ int __init macscsi_detect(struct scsi_host_template * tpnt) | |||
177 | if (macintosh_config->scsi_type != MAC_SCSI_OLD) | 177 | if (macintosh_config->scsi_type != MAC_SCSI_OLD) |
178 | return( 0 ); | 178 | return( 0 ); |
179 | 179 | ||
180 | /* setup variables */ | 180 | if (setup_can_queue > 0) |
181 | tpnt->can_queue = | 181 | tpnt->can_queue = setup_can_queue; |
182 | (setup_can_queue > 0) ? setup_can_queue : CAN_QUEUE; | 182 | if (setup_cmd_per_lun > 0) |
183 | tpnt->cmd_per_lun = | 183 | tpnt->cmd_per_lun = setup_cmd_per_lun; |
184 | (setup_cmd_per_lun > 0) ? setup_cmd_per_lun : CMD_PER_LUN; | 184 | if (setup_sg_tablesize >= 0) |
185 | tpnt->sg_tablesize = | 185 | tpnt->sg_tablesize = setup_sg_tablesize; |
186 | (setup_sg_tablesize >= 0) ? setup_sg_tablesize : SG_TABLESIZE; | ||
187 | 186 | ||
188 | if (setup_hostid >= 0) | 187 | if (setup_hostid >= 0) |
189 | tpnt->this_id = setup_hostid; | 188 | tpnt->this_id = setup_hostid; |
@@ -194,7 +193,7 @@ int __init macscsi_detect(struct scsi_host_template * tpnt) | |||
194 | 193 | ||
195 | #ifdef SUPPORT_TAGS | 194 | #ifdef SUPPORT_TAGS |
196 | if (setup_use_tagged_queuing < 0) | 195 | if (setup_use_tagged_queuing < 0) |
197 | setup_use_tagged_queuing = USE_TAGGED_QUEUING; | 196 | setup_use_tagged_queuing = 0; |
198 | #endif | 197 | #endif |
199 | 198 | ||
200 | /* Once we support multiple 5380s (e.g. DuoDock) we'll do | 199 | /* Once we support multiple 5380s (e.g. DuoDock) we'll do |
@@ -496,10 +495,10 @@ static struct scsi_host_template driver_template = { | |||
496 | .queuecommand = macscsi_queue_command, | 495 | .queuecommand = macscsi_queue_command, |
497 | .eh_abort_handler = macscsi_abort, | 496 | .eh_abort_handler = macscsi_abort, |
498 | .eh_bus_reset_handler = macscsi_bus_reset, | 497 | .eh_bus_reset_handler = macscsi_bus_reset, |
499 | .can_queue = CAN_QUEUE, | 498 | .can_queue = 16, |
500 | .this_id = 7, | 499 | .this_id = 7, |
501 | .sg_tablesize = SG_ALL, | 500 | .sg_tablesize = SG_ALL, |
502 | .cmd_per_lun = CMD_PER_LUN, | 501 | .cmd_per_lun = 2, |
503 | .use_clustering = DISABLE_CLUSTERING | 502 | .use_clustering = DISABLE_CLUSTERING |
504 | }; | 503 | }; |
505 | 504 | ||