Commit 9c3f0e2b authored by Finn Thain's avatar Finn Thain Committed by Martin K. Petersen
Browse files

atari_NCR5380: Remove RESET_BOOT, CONFIG_ATARI_SCSI_TOSHIBA_DELAY and CONFIG_ATARI_SCSI_RESET_BOOT 



The atari_NCR5380.c core driver now takes care of bus reset upon driver
initialization if required (same as NCR5380.c). Move the Toshiba CD-ROM
support into the core driver, enabled with a host flag, so that all
NCR5380 drivers can make use of it.

Drop the RESET_BOOT macros and the ATARI_SCSI_RESET_BOOT and
ATARI_SCSI_TOSHIBA_DELAY Kconfig symbols, which are now redundant.

Remove the atari_scsi_reset_boot(), mac_scsi_reset_boot() and
sun3_scsi_reset_boot() routines. None of this duplicated code is needed
now that all drivers can use NCR5380_maybe_reset_bus().

This brings atari_scsi, mac_scsi and sun3_scsi into line with all of the
other NCR5380 drivers.

The bus reset may raise an interrupt. That would be new behaviour for
atari_scsi only when CONFIG_ATARI_SCSI_RESET_BOOT=n. The ST DMA interrupt
is not assigned to atari_scsi at this stage, so
CONFIG_ATARI_SCSI_RESET_BOOT=y may well be problematic already.
Regardless, do_reset() now raises and clears the interrupt within
local_irq_save/restore which should avoid problems.

Signed-off-by: default avatarFinn Thain <fthain@telegraphics.com.au>
Reviewed-by: default avatarHannes Reinecke <hare@suse.com>
Tested-by: default avatarOndrej Zary <linux@rainbow-software.org>
Tested-by: default avatarMichael Schmitz <schmitzmic@gmail.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 636b1ec8

drivers/scsi/Kconfig

+0 −17
Original line number Diff line number Diff line
@@ -1617,23 +1617,6 @@ config ATARI_SCSI 
	  ST-DMA, replacing ACSI).  It does NOT support other schemes, like

	  in the Hades (without DMA).



config ATARI_SCSI_TOSHIBA_DELAY

	bool "Long delays for Toshiba CD-ROMs"

	depends on ATARI_SCSI

	help

	  This option increases the delay after a SCSI arbitration to

	  accommodate some flaky Toshiba CD-ROM drives. Say Y if you intend to

	  use a Toshiba CD-ROM drive; otherwise, the option is not needed and

	  would impact performance a bit, so say N.



config ATARI_SCSI_RESET_BOOT

	bool "Reset SCSI-devices at boottime"

	depends on ATARI_SCSI

	help

	  Reset the devices on your Atari whenever it boots.  This makes the

	  boot process fractionally longer but may assist recovery from errors

	  that leave the devices with SCSI operations partway completed.



config MAC_SCSI

	tristate "Macintosh NCR5380 SCSI"

	depends on MAC && SCSI=y

drivers/scsi/NCR5380.c

+15 −2
Original line number Diff line number Diff line
@@ -618,7 +618,7 @@ static void prepare_info(struct Scsi_Host *instance) 
	         "base 0x%lx, irq %d, "

	         "can_queue %d, cmd_per_lun %d, "

	         "sg_tablesize %d, this_id %d, "

	         "flags { %s%s%s}, "

	         "flags { %s%s%s%s}, "

#if defined(USLEEP_POLL) && defined(USLEEP_WAITLONG)

		 "USLEEP_POLL %lu, USLEEP_WAITLONG %lu, "

#endif
@@ -630,6 +630,7 @@ static void prepare_info(struct Scsi_Host *instance) 
	         hostdata->flags & FLAG_NCR53C400     ? "NCR53C400 "     : "",

	         hostdata->flags & FLAG_DTC3181E      ? "DTC3181E "      : "",

	         hostdata->flags & FLAG_NO_PSEUDO_DMA ? "NO_PSEUDO_DMA " : "",

	         hostdata->flags & FLAG_TOSHIBA_DELAY ? "TOSHIBA_DELAY "  : "",

#if defined(USLEEP_POLL) && defined(USLEEP_WAITLONG)

	         USLEEP_POLL, USLEEP_WAITLONG,

#endif
@@ -831,6 +832,7 @@ static int NCR5380_init(struct Scsi_Host *instance, int flags) 


static int NCR5380_maybe_reset_bus(struct Scsi_Host *instance)

{

	struct NCR5380_hostdata *hostdata = shost_priv(instance);

	int pass;



	for (pass = 1; (NCR5380_read(STATUS_REG) & SR_BSY) && pass <= 6; ++pass) {
@@ -849,6 +851,14 @@ static int NCR5380_maybe_reset_bus(struct Scsi_Host *instance) 
		case 4:

			shost_printk(KERN_ERR, instance, "bus busy, attempting reset\n");

			do_reset(instance);

			/* Wait after a reset; the SCSI standard calls for

			 * 250ms, we wait 500ms to be on the safe side.

			 * But some Toshiba CD-ROMs need ten times that.

			 */

			if (hostdata->flags & FLAG_TOSHIBA_DELAY)

				msleep(2500);

			else

				msleep(500);

			break;

		case 6:

			shost_printk(KERN_ERR, instance, "bus locked solid\n");
@@ -1253,6 +1263,9 @@ static int NCR5380_select(struct Scsi_Host *instance, struct scsi_cmnd *cmd) 
	 * a minimum so we'll udelay ceil(1.2)

	 */



	if (hostdata->flags & FLAG_TOSHIBA_DELAY)

		udelay(15);

	else

		udelay(2);



	dprintk(NDEBUG_ARBITRATION, "scsi%d : won arbitration\n", instance->host_no);

drivers/scsi/NCR5380.h

+1 −0
Original line number Diff line number Diff line
@@ -243,6 +243,7 @@ 
#define FLAG_DTC3181E			16	/* DTC3181E */

#define FLAG_LATE_DMA_SETUP		32	/* Setup NCR before DMA H/W */

#define FLAG_TAGGED_QUEUING		64	/* as X3T9.2 spelled it */

#define FLAG_TOSHIBA_DELAY		128	/* Allow for borken CD-ROMs */



#ifdef SUPPORT_TAGS

struct tag_alloc {

drivers/scsi/atari_NCR5380.c

+15 −7
Original line number Diff line number Diff line
@@ -674,13 +674,14 @@ static void prepare_info(struct Scsi_Host *instance) 
	         "base 0x%lx, irq %d, "

	         "can_queue %d, cmd_per_lun %d, "

	         "sg_tablesize %d, this_id %d, "

	         "flags { %s}, "

	         "flags { %s%s}, "

	         "options { %s} ",

	         instance->hostt->name, instance->io_port, instance->n_io_port,

	         instance->base, instance->irq,

	         instance->can_queue, instance->cmd_per_lun,

	         instance->sg_tablesize, instance->this_id,

	         hostdata->flags & FLAG_TAGGED_QUEUING ? "TAGGED_QUEUING " : "",

	         hostdata->flags & FLAG_TOSHIBA_DELAY  ? "TOSHIBA_DELAY "  : "",

#ifdef DIFFERENTIAL

	         "DIFFERENTIAL "

#endif
@@ -860,6 +861,7 @@ static int __init NCR5380_init(struct Scsi_Host *instance, int flags) 


static int NCR5380_maybe_reset_bus(struct Scsi_Host *instance)

{

	struct NCR5380_hostdata *hostdata = shost_priv(instance);

	int pass;



	for (pass = 1; (NCR5380_read(STATUS_REG) & SR_BSY) && pass <= 6; ++pass) {
@@ -878,6 +880,14 @@ static int NCR5380_maybe_reset_bus(struct Scsi_Host *instance) 
		case 4:

			shost_printk(KERN_ERR, instance, "bus busy, attempting reset\n");

			do_reset(instance);

			/* Wait after a reset; the SCSI standard calls for

			 * 250ms, we wait 500ms to be on the safe side.

			 * But some Toshiba CD-ROMs need ten times that.

			 */

			if (hostdata->flags & FLAG_TOSHIBA_DELAY)

				msleep(2500);

			else

				msleep(500);

			break;

		case 6:

			shost_printk(KERN_ERR, instance, "bus locked solid\n");
@@ -1493,12 +1503,10 @@ static int NCR5380_select(struct Scsi_Host *instance, struct scsi_cmnd *cmd) 
	 * a minimum so we'll udelay ceil(1.2)

	 */



#ifdef CONFIG_ATARI_SCSI_TOSHIBA_DELAY

	/* ++roman: But some targets (see above :-) seem to need a bit more... */

	if (hostdata->flags & FLAG_TOSHIBA_DELAY)

		udelay(15);

#else

	else

		udelay(2);

#endif



	if (hostdata->connected) {

		NCR5380_write(MODE_REG, MR_BASE);

drivers/scsi/atari_scsi.c

+9 −51
Original line number Diff line number Diff line
@@ -164,15 +164,6 @@ static inline unsigned long SCSI_DMA_GETADR(void) 
#define HOSTDATA_DMALEN		(((struct NCR5380_hostdata *) \

				(atari_scsi_host->hostdata))->dma_len)



/* Time (in jiffies) to wait after a reset; the SCSI standard calls for 250ms,

 * we usually do 0.5s to be on the safe side. But Toshiba CD-ROMs once more

 * need ten times the standard value... */

#ifndef CONFIG_ATARI_SCSI_TOSHIBA_DELAY

#define	AFTER_RESET_DELAY	(HZ/2)

#else

#define	AFTER_RESET_DELAY	(5*HZ/2)

#endif



#ifdef REAL_DMA

static void atari_scsi_fetch_restbytes(void);

#endif
@@ -208,12 +199,12 @@ static int setup_cmd_per_lun = -1; 
module_param(setup_cmd_per_lun, int, 0);

static int setup_sg_tablesize = -1;

module_param(setup_sg_tablesize, int, 0);

#ifdef SUPPORT_TAGS

static int setup_use_tagged_queuing = -1;

module_param(setup_use_tagged_queuing, int, 0);

#endif

static int setup_hostid = -1;

module_param(setup_hostid, int, 0);

static int setup_toshiba_delay = -1;

module_param(setup_toshiba_delay, int, 0);





#if defined(REAL_DMA)
@@ -488,7 +479,7 @@ static int __init atari_scsi_setup(char *str) 
	 * Defaults depend on TT or Falcon, determined at run time.

	 * Negative values mean don't change.

	 */

	int ints[6];

	int ints[8];



	get_options(str, ARRAY_SIZE(ints), ints);
@@ -504,10 +495,11 @@ static int __init atari_scsi_setup(char *str) 
		setup_sg_tablesize = ints[3];

	if (ints[0] >= 4)

		setup_hostid = ints[4];

#ifdef SUPPORT_TAGS

	if (ints[0] >= 5)

		setup_use_tagged_queuing = ints[5];

#endif

	/* ints[6] (use_pdma) is ignored */

	if (ints[0] >= 7)

		setup_toshiba_delay = ints[7];



	return 1;

}
@@ -516,38 +508,6 @@ __setup("atascsi=", atari_scsi_setup); 
#endif /* !MODULE */





#ifdef CONFIG_ATARI_SCSI_RESET_BOOT

static void __init atari_scsi_reset_boot(void)

{

	unsigned long end;



	/*

	 * Do a SCSI reset to clean up the bus during initialization. No messing

	 * with the queues, interrupts, or locks necessary here.

	 */



	printk("Atari SCSI: resetting the SCSI bus...");



	/* get in phase */

	NCR5380_write(TARGET_COMMAND_REG,

		      PHASE_SR_TO_TCR(NCR5380_read(STATUS_REG)));



	/* assert RST */

	NCR5380_write(INITIATOR_COMMAND_REG, ICR_BASE | ICR_ASSERT_RST);

	/* The min. reset hold time is 25us, so 40us should be enough */

	udelay(50);

	/* reset RST and interrupt */

	NCR5380_write(INITIATOR_COMMAND_REG, ICR_BASE);

	NCR5380_read(RESET_PARITY_INTERRUPT_REG);



	end = jiffies + AFTER_RESET_DELAY;

	while (time_before(jiffies, end))

		barrier();



	printk(" done\n");

}

#endif



#if defined(REAL_DMA)



static unsigned long atari_scsi_dma_setup(struct Scsi_Host *instance,
@@ -917,17 +877,13 @@ static int __init atari_scsi_probe(struct platform_device *pdev) 
	}

	atari_scsi_host = instance;



#ifdef CONFIG_ATARI_SCSI_RESET_BOOT

	atari_scsi_reset_boot();

#endif



	instance->irq = irq->start;



	host_flags |= IS_A_TT() ? 0 : FLAG_LATE_DMA_SETUP;



#ifdef SUPPORT_TAGS

	host_flags |= setup_use_tagged_queuing > 0 ? FLAG_TAGGED_QUEUING : 0;

#endif

	host_flags |= setup_toshiba_delay > 0 ? FLAG_TOSHIBA_DELAY : 0;



	NCR5380_init(instance, host_flags);
@@ -975,6 +931,8 @@ static int __init atari_scsi_probe(struct platform_device *pdev) 
#endif

	}



	NCR5380_maybe_reset_bus(instance);



	error = scsi_add_host(instance, NULL);

	if (error)

		goto fail_host;

CRA Git | Maintained and supported by SUSTech CRA and CCSE