On 03/14/2016 05:27 AM, Finn Thain wrote: > The only chip that needs the workarounds enabled is an early NMOS > device. That means that the common case is to disable them. > > Unfortunately the sense of the flag is such that it has to be set > for the common case. > > Rename the flag so that zero can be used to mean "no errata workarounds > needed". This simplifies the code. > > Signed-off-by: Finn Thain <fthain@xxxxxxxxxxxxxxxxxxx> > > --- > drivers/scsi/NCR5380.c | 14 +++++++------- > drivers/scsi/NCR5380.h | 2 +- > drivers/scsi/arm/cumana_1.c | 2 +- > drivers/scsi/arm/oak.c | 2 +- > drivers/scsi/dtc.c | 2 +- > drivers/scsi/g_NCR5380.c | 8 +------- > drivers/scsi/pas16.c | 2 +- > drivers/scsi/t128.c | 2 +- > 8 files changed, 14 insertions(+), 20 deletions(-) > Reviewed-by: Hannes Reinecke <hare@xxxxxxxx> Cheers, Hannes -- Dr. Hannes Reinecke Teamlead Storage & Networking hare@xxxxxxx +49 911 74053 688 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton HRB 21284 (AG Nürnberg) -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html