Replace macro VNSvInPortB with ioread8. Avoid cast of the return value is possible with one exception. The name of macro and the arguments use CamelCase which is not accepted by checkpatch.pl The one casting exception could be removed by changing bb_read_embedded. Instead of having a unused return value bool the return value could be an unsigned char. Since there are more than one checkpatch issue per line, more steps are rquired to fix all issues. This patch series is new but the second patch is already in V4. Tested with vt6655 on mini PCI Module Transferred this patch over wlan connection of vt6655 Philipp Hortmann (2): staging: vt6655: Replace MACvGPIOIn with VNSvInPortB staging: vt6655: Replace VNSvInPortB with ioread8 drivers/staging/vt6655/baseband.c | 6 +++--- drivers/staging/vt6655/card.c | 2 +- drivers/staging/vt6655/device_main.c | 6 +++--- drivers/staging/vt6655/mac.h | 27 +++++++++------------------ drivers/staging/vt6655/srom.c | 6 +++--- drivers/staging/vt6655/upc.h | 3 --- 6 files changed, 19 insertions(+), 31 deletions(-) -- 2.25.1