From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754192AbbLFDn5 (ORCPT ); Sat, 5 Dec 2015 22:43:57 -0500 Received: from kvm5.telegraphics.com.au ([98.124.60.144]:53096 "EHLO kvm5.telegraphics.com.au" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753706AbbLFDnx (ORCPT ); Sat, 5 Dec 2015 22:43:53 -0500 Message-Id: <20151206013126.995379403@telegraphics.com.au> User-Agent: quilt/0.50-1 Date: Sun, 06 Dec 2015 12:31:26 +1100 From: Finn Thain To: "James E.J. Bottomley" , Michael Schmitz , , , Subject: [PATCH v2 00/72] More fixes, cleanup and modernization for NCR5380 drivers Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 about 750 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. --- drivers/scsi/Kconfig | 17 drivers/scsi/NCR5380.c | 2867 +++++++++++++++++++------------------------ drivers/scsi/NCR5380.h | 74 - drivers/scsi/arm/cumana_1.c | 30 drivers/scsi/arm/oak.c | 26 drivers/scsi/atari_NCR5380.c | 2289 +++++++++++++++------------------- drivers/scsi/atari_scsi.c | 102 - drivers/scsi/dmx3191d.c | 32 drivers/scsi/dtc.c | 114 - drivers/scsi/dtc.h | 45 drivers/scsi/g_NCR5380.c | 183 +- drivers/scsi/g_NCR5380.h | 56 drivers/scsi/mac_scsi.c | 116 - drivers/scsi/pas16.c | 115 - drivers/scsi/pas16.h | 40 drivers/scsi/sun3_scsi.c | 141 -- drivers/scsi/t128.c | 101 - drivers/scsi/t128.h | 39 18 files changed, 2801 insertions(+), 3586 deletions(-) From mboxrd@z Thu Jan 1 00:00:00 1970 From: Finn Thain Subject: [PATCH v2 00/72] More fixes, cleanup and modernization for NCR5380 drivers Date: Sun, 06 Dec 2015 12:31:26 +1100 Message-ID: <20151206013126.995379403@telegraphics.com.au> Return-path: Received: from kvm5.telegraphics.com.au ([98.124.60.144]:53096 "EHLO kvm5.telegraphics.com.au" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753706AbbLFDnx (ORCPT ); Sat, 5 Dec 2015 22:43:53 -0500 Sender: linux-m68k-owner@vger.kernel.org List-Id: linux-m68k@vger.kernel.org To: "James E.J. Bottomley" , Michael Schmitz , linux-m68k@vger.kernel.org, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org 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 about 750 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. --- drivers/scsi/Kconfig | 17 drivers/scsi/NCR5380.c | 2867 +++++++++++++++++++------------------------ drivers/scsi/NCR5380.h | 74 - drivers/scsi/arm/cumana_1.c | 30 drivers/scsi/arm/oak.c | 26 drivers/scsi/atari_NCR5380.c | 2289 +++++++++++++++------------------- drivers/scsi/atari_scsi.c | 102 - drivers/scsi/dmx3191d.c | 32 drivers/scsi/dtc.c | 114 - drivers/scsi/dtc.h | 45 drivers/scsi/g_NCR5380.c | 183 +- drivers/scsi/g_NCR5380.h | 56 drivers/scsi/mac_scsi.c | 116 - drivers/scsi/pas16.c | 115 - drivers/scsi/pas16.h | 40 drivers/scsi/sun3_scsi.c | 141 -- drivers/scsi/t128.c | 101 - drivers/scsi/t128.h | 39 18 files changed, 2801 insertions(+), 3586 deletions(-)