On Tue, Feb 04, 2014 at 04:31:19PM +0200, Mika Westerberg wrote: > Intel Baytrail I2C controllers can be enumerated from PCI as well as from > ACPI. In order to support this add the Baytrail PCI IDs to the driver. > > Signed-off-by: Mika Westerberg <mika.westerb...@linux.intel.com>
Basically fine, one question: > - snprintf(adap->name, sizeof(adap->name), "i2c-designware-pci-%d", > - adap->nr); > + > + if (adap->nr < 0) > + snprintf(adap->name, sizeof(adap->name), "i2c-designware-pci"); > + else > + snprintf(adap->name, sizeof(adap->name), > + "i2c-designware-pci-%d", adap->nr); Maybe we just drop the "-%d" suffix entirely?
signature.asc
Description: Digital signature
