]> git.baikalelectronics.ru Git - kernel.git/commit
scsi: esp_scsi: Correct ordering of PCSCSI definition in esp_rev enum
authorKars de Jong <jongk@linux-m68k.org>
Tue, 19 Nov 2019 20:20:20 +0000 (21:20 +0100)
committerMartin K. Petersen <martin.petersen@oracle.com>
Fri, 20 Dec 2019 03:08:51 +0000 (22:08 -0500)
commitf175da4640fdb41b8a79975414a8423538ce72ac
treebbc6ed5af03fd46aab247c36e3253ea143ad3121
parent18ca123f6ebe35a17ec03d9d706c2e230bcf298a
scsi: esp_scsi: Correct ordering of PCSCSI definition in esp_rev enum

The order of the definitions in the esp_rev enum is important. The values
are used in comparisons for chip features.

Add a comment to the enum explaining this.

Also, the actual values for the enum fields are irrelevant, so remove the
explicit values (suggested by Geert Uytterhoeven). This makes adding a new
field in the middle of the enum easier.

Finally, move the PCSCSI definition to the right place in the enum. In its
previous location, at the end of the enum, the wrong values are written to
the CONFIG3 register when used with FAST-SCSI targets.

Link: https://lore.kernel.org/r/20191119202021.28720-2-jongk@linux-m68k.org
Signed-off-by: Kars de Jong <jongk@linux-m68k.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/esp_scsi.c
drivers/scsi/esp_scsi.h