On Mon, 4 Jan 2016 01:56:13 +0800 Huawei Xie <huawei.xie at intel.com> wrote:
> + if (pci_dev->kdrv != RTE_KDRV_NONE) { > + PMD_INIT_LOG(INFO, > + "kernel driver is manipulating this device." \ > + " Please unbind the kernel driver."); Splitting strings in general is a bad idea since it makes it harder to find log messages. Also the first clause is lower case and the second is captialized. Lastly, the backslash continuation is unnecessary here and will cause checkpatch warning.