WARNING: Missing commit description - Add an appropriate one CHECK: Please don't use multiple blank lines #119: FILE: drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c:555: + + WARNING: Prefer [subsystem eg: netdev]_crit([subsystem]dev, ... then dev_crit(dev, ... then pr_crit(... to printk(KERN_CRIT ... #134: FILE: drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c:570: + printk(KERN_CRIT "edev_fwctl is NULL %s\n", __func__); CHECK: Please don't use multiple blank lines #162: FILE: drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c:598: + + WARNING: Prefer [subsystem eg: netdev]_crit([subsystem]dev, ... then dev_crit(dev, ... then pr_crit(... to printk(KERN_CRIT ... #215: FILE: drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c:651: + printk(KERN_CRIT "Made it %s\n", __func__); total: 0 errors, 3 warnings, 2 checks, 194 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 5ddd85fc7e9c ("bnxt: Create an auxiliary device for fwctl_bnxt") 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, 3 warnings, 2 checks, 194 lines checked