All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3 00/77] More fixes, cleanup and modernization for NCR5380 drivers
@ 2015-12-22  1:17 ` Finn Thain
  0 siblings, 0 replies; 264+ messages in thread
From: Finn Thain @ 2015-12-22  1:17 UTC (permalink / raw)
  To: James E.J. Bottomley, Michael Schmitz, linux-m68k, linux-scsi,
	linux-kernel


Like my previous work on the NCR5380 drivers, this patch series has bug
fixes, code cleanup and modernization. These drivers suffer from mistakes,
poor style and neglect and this long series addresses the worst of it,
covering all ten wrapper drivers and both of the core driver forks. The
combined size of the drivers is reduced by over 700 LoC.

This series continues to reduce divergence between the two core driver
forks, often by copying a bug fix from one to the other. Most patches are
larger for having to keep the two forks in sync. Making the same change to
both is churn if one of them is to be removed but neither can be as yet.
By the end of this series the diff between the two forks is minimal, so it
becomes clear what caused the fork and what can be done about it.

This patch series did benefit from scripts/checkpatch.pl but not too much.
Decades ago, these drivers started out with 4-space tabs and if the 80
column limit were to be strictly enforced now, it would require adding new
functions and shortening identifiers. I would defer this sort of activity
until after the fork has been resolved.

All patches to all NCR5380 drivers (x86, ARM, m68k) have been compile-
tested. The mac_scsi, dmx3191d, g_NCR5380 and atari_scsi modules were
regression tested on suitable hardware.

Changes since v1:
- Patch 8 omits a pointless assignment.
- Patch 10 gets a better error message.
- Patch 20 drops the WQ_CPU_INTENSIVE flag.
- Patch 21 adds timing calibration for the register polling loop.
- Patch 49 is replaced by a new one that removes FLAG_DTC3181E.
- Patch 72 by Ondrej Zary was added to fix pseudo DMA on 53C400.

Changes since v2:
- Patch 21 has better calibration with low HZ values.
- Patch 57 no longer attempts to dereference a NULL pointer on Atari.
- Patch 66 initializes max_sectors in the host templates.
- Patches 73 thru 77 by Ondrej Zary were added with additional fixes for
  53C400-compatible cards.

---
 drivers/scsi/Kconfig         |   17 
 drivers/scsi/NCR5380.c       | 2868 +++++++++++++++++++------------------------
 drivers/scsi/NCR5380.h       |   87 -
 drivers/scsi/arm/cumana_1.c  |   31 
 drivers/scsi/arm/oak.c       |   27 
 drivers/scsi/atari_NCR5380.c | 2292 +++++++++++++++-------------------
 drivers/scsi/atari_scsi.c    |  102 -
 drivers/scsi/dmx3191d.c      |   33 
 drivers/scsi/dtc.c           |  115 -
 drivers/scsi/dtc.h           |   45 
 drivers/scsi/g_NCR5380.c     |  408 +++---
 drivers/scsi/g_NCR5380.h     |   66 
 drivers/scsi/mac_scsi.c      |  117 -
 drivers/scsi/pas16.c         |  116 -
 drivers/scsi/pas16.h         |   40 
 drivers/scsi/sun3_scsi.c     |  141 --
 drivers/scsi/t128.c          |  102 -
 drivers/scsi/t128.h          |   39 
 18 files changed, 2957 insertions(+), 3689 deletions(-)





^ permalink raw reply	[flat|nested] 264+ messages in thread

end of thread, other threads:[~2016-01-01  1:28 UTC | newest]

Thread overview: 264+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-12-22  1:17 [PATCH v3 00/77] More fixes, cleanup and modernization for NCR5380 drivers Finn Thain
2015-12-22  1:17 ` Finn Thain
2015-12-22  1:17 ` [PATCH v3 01/77] atari_scsi: Fix SCSI host ID setting Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17 ` [PATCH v3 02/77] ncr5380: Remove redundant static variable initializers Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17 ` [PATCH v3 03/77] ncr5380: Eliminate PDEBUG*, TDEBUG* and DTCDEBUG* macros Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17 ` [PATCH v3 04/77] ncr5380: Remove more pointless macros Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17 ` [PATCH v3 05/77] ncr5380: Remove NCR5380_local_declare and NCR5380_setup macros Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17 ` [PATCH v3 06/77] ncr5380: Remove NCR5380_instance_name macro Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17 ` [PATCH v3 07/77] ncr5380: Split NCR5380_init() into two functions Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17 ` [PATCH v3 08/77] ncr5380: Move NCR53C400-specific code Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17 ` [PATCH v3 09/77] atari_NCR5380: Reset bus on driver initialization if required Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17 ` [PATCH v3 10/77] atari_NCR5380: Remove RESET_BOOT, CONFIG_ATARI_SCSI_TOSHIBA_DELAY and CONFIG_ATARI_SCSI_RESET_BOOT Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17 ` [PATCH v3 11/77] ncr5380: Simplify bus reset handlers Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17 ` [PATCH v3 12/77] ncr5380: Remove unused hostdata->aborted flag Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17 ` [PATCH v3 13/77] ncr5380: Remove redundant register writes Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17 ` [PATCH v3 14/77] ncr5380: Use return instead of goto in NCR5380_select() Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17 ` [PATCH v3 15/77] ncr5380: Always escalate bad target time-out " Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17 ` [PATCH v3 16/77] ncr5380: Proceed with next command after NCR5380_select() calls scsi_done Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17 ` [PATCH v3 17/77] ncr5380: Keep BSY asserted when entering SELECTION phase Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  7:01   ` Hannes Reinecke
2015-12-22  1:17 ` [PATCH v3 18/77] ncr5380: Eliminate USLEEP_WAITLONG delay Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  7:05   ` Hannes Reinecke
2015-12-22 12:38     ` Finn Thain
2015-12-22 13:37       ` Hannes Reinecke
2015-12-22  1:17 ` [PATCH v3 19/77] ncr5380: Cleanup bogus {request,release}_region() calls Finn Thain
2015-12-22  1:17   ` [PATCH v3 19/77] ncr5380: Cleanup bogus {request, release}_region() calls Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  7:05   ` [PATCH v3 19/77] ncr5380: Cleanup bogus {request,release}_region() calls Hannes Reinecke
2015-12-22  7:05     ` Hannes Reinecke
2015-12-22  1:17 ` [PATCH v3 20/77] ncr5380: Introduce unbound workqueue Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  7:10   ` Hannes Reinecke
2015-12-22  7:10     ` Hannes Reinecke
2015-12-22 12:44     ` Finn Thain
2015-12-22 12:44       ` Finn Thain
2015-12-22 14:48       ` Hannes Reinecke
2015-12-22 14:48         ` Hannes Reinecke
2015-12-22  1:17 ` [PATCH v3 21/77] ncr5380: Sleep when polling, if possible Finn Thain
2015-12-22  1:17   ` Finn Thain
2015-12-22  7:12   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 22/77] ncr5380: Eliminate selecting state Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:14   ` Hannes Reinecke
2015-12-22  7:14     ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 23/77] ncr5380: Always retry arbitration and selection Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:14   ` Hannes Reinecke
2015-12-22  7:14     ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 24/77] ncr5380: Implement NCR5380_dma_xfer_len and remove LIMIT_TRANSFERSIZE macro Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:17   ` Hannes Reinecke
2015-12-22  7:17     ` Hannes Reinecke
2015-12-22  7:17     ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 25/77] ncr5380: Rework disconnect versus poll logic Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:21   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 26/77] ncr5380: Fix NCR5380_transfer_pio() result Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:22   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 27/77] ncr5380: Add missing lock in eh_abort_handler Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:23   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 28/77] ncr5380: Drop DEF_SCSI_QCMD macro Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:24   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 29/77] ncr5380: Remove references to linked commands Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:25   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 30/77] ncr5380: Add missing break after case MESSAGE_REJECT Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:26   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 31/77] ncr5380: Fix !REQ timeout in do_abort() Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:26   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 32/77] ncr5380: Fix bus phase " Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:27   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 33/77] atari_NCR5380: Set do_abort() timeouts Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:29   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 34/77] atari_NCR5380: Use arbitration timeout Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:30   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 35/77] ncr5380: Dont wait for BUS FREE after disconnect Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:31   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 36/77] ncr5380: Use work_struct instead of delayed_work Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:32   ` Hannes Reinecke
2015-12-22  7:32     ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 37/77] ncr5380: Standardize work queueing algorithm Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:33   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 38/77] ncr5380: Remove UNSAFE macro Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:34   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 39/77] ncr5380: Standardize interrupt handling Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:36   ` Hannes Reinecke
2015-12-22  7:36     ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 40/77] ncr5380: Introduce NCR5380_poll_politely2 Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:37   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 41/77] ncr5380: Replace redundant flags with FLAG_NO_DMA_FIXUP Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:39   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 42/77] ncr5380: Replace READ_OVERRUNS macro with FLAG_NO_DMA_FIXUPS Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:39   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 43/77] ncr5380: Standardize reselection handling Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:41   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 44/77] ncr5380: Fix off-by-one bug in extended_msg[] bounds check Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:41   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 45/77] ncr5380: Cleanup #include directives Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:42   ` Hannes Reinecke
2015-12-22  7:42     ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 46/77] ncr5380: Fix NDEBUG_NO_DATAOUT flag Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:42   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 47/77] ncr5380: Fix and cleanup scsi_host_template initializers Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:43   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 48/77] atari_NCR5380: Fix queue_size limit Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:44   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 49/77] ncr5380: Remove redundant ICR_ARBITRATION_LOST test and eliminate FLAG_DTC3181E Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:45   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 50/77] ncr5380: Change instance->host_lock to hostdata->lock Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:46   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 51/77] ncr5380: Remove command list debug code Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:47   ` Hannes Reinecke
2015-12-22  7:47     ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 52/77] ncr5380: Remove H_NO macro and introduce dsprintk Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:48   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 53/77] ncr5380: Use shost_priv helper Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:48   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 54/77] ncr5380: Use dsprintk() for queue debugging Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:50   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 55/77] ncr5380: Remove LIST and REMOVE macros Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:50   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 56/77] ncr5380: Remove redundant volatile qualifiers Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:51   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 57/77] ncr5380: Use standard list data structure Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:55   ` Hannes Reinecke
2015-12-22  7:55     ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 58/77] ncr5380: Refactor command completion Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:56   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 59/77] ncr5380: Fix autosense bugs Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:57   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 60/77] ncr5380: Implement new eh_abort_handler Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  7:59   ` Hannes Reinecke
2015-12-22  7:59     ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 61/77] ncr5380: Fix EH during arbitration and selection Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  8:00   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 62/77] ncr5380: Implement new eh_bus_reset_handler Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  8:01   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 63/77] atari_NCR5380: Remove HOSTNO macro from printk() and seq_printf() calls Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  8:02   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 64/77] atari_NCR5380: Eliminate HOSTNO macro Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  8:02   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 65/77] atari_scsi, sun3_scsi: Remove global Scsi_Host pointer Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  8:03   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 66/77] ncr5380: Fix soft lockups Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  8:03   ` Hannes Reinecke
2015-12-22  8:03     ` Hannes Reinecke
2015-12-22 11:39   ` One Thousand Gnomes
2015-12-22 11:39     ` One Thousand Gnomes
2015-12-22 11:39     ` One Thousand Gnomes
2015-12-22 13:47     ` Finn Thain
2015-12-22 13:47       ` Finn Thain
2015-12-23  0:42       ` Michael Schmitz
2015-12-23  0:42         ` Michael Schmitz
2015-12-22  1:18 ` [PATCH v3 67/77] ncr5380: Cleanup comments Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  8:04   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 68/77] ncr5380: Fix whitespace issues using regexp Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  8:04   ` Hannes Reinecke
2015-12-22 14:46   ` Joe Perches
2015-12-23  0:56     ` Finn Thain
2015-12-23  1:13       ` Joe Perches
2015-12-23  2:03         ` Finn Thain
2015-12-23  2:18           ` Joe Perches
2015-12-23  4:14             ` Finn Thain
2015-12-23  1:22       ` James Bottomley
2015-12-23  2:31         ` Finn Thain
2015-12-22  1:18 ` [PATCH v3 69/77] ncr5380: Merge changes from atari_NCR5380.c Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  8:05   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 70/77] atari_NCR5380: Merge changes from NCR5380.c Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  8:05   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 71/77] ncr5380: Cleanup whitespace and parentheses Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  8:06   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 72/77] ncr5380: Fix pseudo DMA transfers on 53C400 Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  8:06   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 73/77] ncr5380: Use runtime register mapping Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  8:07   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 74/77] ncr5380: Enable PDMA for NCR53C400A Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  8:07   ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 75/77] ncr5380: Enable PDMA for DTC chips Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  8:08   ` Hannes Reinecke
2015-12-22  8:08     ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 76/77] ncr5380: Fix wait for 53C80 registers registers after PDMA Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  8:08   ` Hannes Reinecke
2015-12-22  8:08     ` Hannes Reinecke
2015-12-22  1:18 ` [PATCH v3 77/77] ncr5380: Add support for HP C2502 Finn Thain
2015-12-22  1:18   ` Finn Thain
2015-12-22  8:09   ` Hannes Reinecke
2015-12-22 20:14 ` [PATCH v3 00/77] More fixes, cleanup and modernization for NCR5380 drivers Ondrej Zary
2016-01-01  1:28 ` Michael Schmitz

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.