floppy: rename the global "fdc" variable to "current_fdc"
authorWilly Tarreau <w@1wt.eu>
Sun, 1 Mar 2020 19:55:55 +0000 (20:55 +0100)
committerJens Axboe <axboe@kernel.dk>
Mon, 16 Mar 2020 14:26:58 +0000 (08:26 -0600)
commite83995c9f84161900b80d337d6df358a7803870a
treed62f4d1663049593491b8988072aba7ff71f4b39
parente2032464fe189c33005a6296fa67c47d2fea4d05
floppy: rename the global "fdc" variable to "current_fdc"

This is done in order to remove the confusion that arises at some places
in the code where local variables or arguments shadow the global variable.
It is already visible that some places are a bit awkward and iterate over
the global variable, for the sole reason that they used to rely on it being
named "fdc" in order to get the correct address when using FD_DOR. These
ones are easy to spot by searching for "for (current_fdc...".

Some more cleanup is definitely possible. For example
"fdc_state[current_fdc].somefield" is used all over the code and would
probably be better with "fdc_state->somefield" with fdc_state being set
when current_fdc is assigned. This would require to pass the pointer to
the current state instead of the current_fdc to the I/O functions.

Link: https://lore.kernel.org/r/20200301195555.11154-7-w@1wt.eu
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Willy Tarreau <w@1wt.eu>
Signed-off-by: Denis Efremov <efremov@linux.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/block/floppy.c