> > > > > + if (!known_i2c_address) { > > > > > + dev_warn(&priv->pci_dev->dev, > > > > > + "Accelerometer lis3lv02d i2c device is present " > > > > > + "but its i2c address is unknown, skipping ...\n"); > > > > > > > > You are probably well aware of this, but checkpatch prefers > > > > keeping long log messages in one line. I am pointing it out > > > > just in case. > > > > > > Yes, but I do not know how to fix it. Splitting message into two > > > lines generates warning. Having long line generates warning too. > > > > Weird, checkpatch does not protest on my machine when the log message > > is written on a single line... > > I hope that i2c maintainers decide how to format that line. Make it one line saying "accelerometer address unknown"?