Commit 56a1c5ee authored by Finn Thain's avatar Finn Thain Committed by Jens Axboe

block/swim: Rename macros to avoid inconsistent inverted logic

The Sony drive status bits use active-low logic. The swim_readbit()
function converts that to 'C' logic for readability. Hence, the
sense of the names of the status bit macros should not be inverted.

Mostly they are correct. However, the TWOMEG_DRIVE, MFM_MODE and
TWOMEG_MEDIA macros have inverted sense (like MkLinux). Fix this
inconsistency and make the following patches less confusing.

The same problem affects swim3.c so fix that too.

No functional change.

The FDHD drive status bits are documented in sonydriv.cpp from MAME
and in swimiii.h from MkLinux.

Cc: Laurent Vivier <lvivier@redhat.com>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: linuxppc-dev@lists.ozlabs.org
Cc: Jens Axboe <axboe@kernel.dk>
Cc: stable@vger.kernel.org # v4.14+
Tested-by: default avatarStan Johnson <userm57@yahoo.com>
Signed-off-by: default avatarFinn Thain <fthain@telegraphics.com.au>
Acked-by: default avatarLaurent Vivier <lvivier@redhat.com>
Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent 8e2ab5a4
...@@ -110,7 +110,7 @@ struct iwm { ...@@ -110,7 +110,7 @@ struct iwm {
/* Select values for swim_select and swim_readbit */ /* Select values for swim_select and swim_readbit */
#define READ_DATA_0 0x074 #define READ_DATA_0 0x074
#define TWOMEG_DRIVE 0x075 #define ONEMEG_DRIVE 0x075
#define SINGLE_SIDED 0x076 #define SINGLE_SIDED 0x076
#define DRIVE_PRESENT 0x077 #define DRIVE_PRESENT 0x077
#define DISK_IN 0x170 #define DISK_IN 0x170
...@@ -118,9 +118,9 @@ struct iwm { ...@@ -118,9 +118,9 @@ struct iwm {
#define TRACK_ZERO 0x172 #define TRACK_ZERO 0x172
#define TACHO 0x173 #define TACHO 0x173
#define READ_DATA_1 0x174 #define READ_DATA_1 0x174
#define MFM_MODE 0x175 #define GCR_MODE 0x175
#define SEEK_COMPLETE 0x176 #define SEEK_COMPLETE 0x176
#define ONEMEG_MEDIA 0x177 #define TWOMEG_MEDIA 0x177
/* Bits in handshake register */ /* Bits in handshake register */
...@@ -612,7 +612,7 @@ static void setup_medium(struct floppy_state *fs) ...@@ -612,7 +612,7 @@ static void setup_medium(struct floppy_state *fs)
struct floppy_struct *g; struct floppy_struct *g;
fs->disk_in = 1; fs->disk_in = 1;
fs->write_protected = swim_readbit(base, WRITE_PROT); fs->write_protected = swim_readbit(base, WRITE_PROT);
fs->type = swim_readbit(base, ONEMEG_MEDIA); fs->type = swim_readbit(base, TWOMEG_MEDIA);
if (swim_track00(base)) if (swim_track00(base))
printk(KERN_ERR printk(KERN_ERR
......
...@@ -148,7 +148,7 @@ struct swim3 { ...@@ -148,7 +148,7 @@ struct swim3 {
#define MOTOR_ON 2 #define MOTOR_ON 2
#define RELAX 3 /* also eject in progress */ #define RELAX 3 /* also eject in progress */
#define READ_DATA_0 4 #define READ_DATA_0 4
#define TWOMEG_DRIVE 5 #define ONEMEG_DRIVE 5
#define SINGLE_SIDED 6 /* drive or diskette is 4MB type? */ #define SINGLE_SIDED 6 /* drive or diskette is 4MB type? */
#define DRIVE_PRESENT 7 #define DRIVE_PRESENT 7
#define DISK_IN 8 #define DISK_IN 8
...@@ -156,9 +156,9 @@ struct swim3 { ...@@ -156,9 +156,9 @@ struct swim3 {
#define TRACK_ZERO 10 #define TRACK_ZERO 10
#define TACHO 11 #define TACHO 11
#define READ_DATA_1 12 #define READ_DATA_1 12
#define MFM_MODE 13 #define GCR_MODE 13
#define SEEK_COMPLETE 14 #define SEEK_COMPLETE 14
#define ONEMEG_MEDIA 15 #define TWOMEG_MEDIA 15
/* Definitions of values used in writing and formatting */ /* Definitions of values used in writing and formatting */
#define DATA_ESCAPE 0x99 #define DATA_ESCAPE 0x99
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment