CHECK: spaces preferred around that '<<' (ctx:VxV) #32: FILE: drivers/net/phy/marvell.c:199: +#define MII_88E1510_FSCR2_BYPASS_ENABLE (1<<6) ^ CHECK: Prefer using the BIT macro #32: FILE: drivers/net/phy/marvell.c:199: +#define MII_88E1510_FSCR2_BYPASS_ENABLE (1<<6) CHECK: spaces preferred around that '<<' (ctx:VxV) #33: FILE: drivers/net/phy/marvell.c:200: +#define MII_88E1510_FSCR2_BYPASS_STATUS (1<<5) ^ CHECK: Prefer using the BIT macro #33: FILE: drivers/net/phy/marvell.c:200: +#define MII_88E1510_FSCR2_BYPASS_STATUS (1<<5) WARNING: line length of 81 exceeds 80 columns #53: FILE: drivers/net/phy/marvell.c:1642: + if (genphy_read_status_fixed(phydev) < 0) WARNING: From:/Signed-off-by: email address mismatch: 'From: Qingtao Cao ' != 'Signed-off-by: Qingtao Cao ' total: 0 errors, 2 warnings, 4 checks, 37 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. Commit da1c08d4007d ("net: phy: marvell: avoid bringing down fibre link when autoneg is bypassed") has style problems, please review. NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT BAD_REPORTED_BY_LINK CAMELCASE COMMIT_LOG_LONG_LINE GIT_COMMIT_ID MACRO_ARG_REUSE NO_AUTHOR_SIGN_OFF NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. total: 0 errors, 2 warnings, 4 checks, 37 lines checked