On Sun, 2015-11-01 at 17:32 +0100, Christian Fetzer wrote: > This patch adds support for port names for the SB800 chipset. > Since the chipset supports a multiplexed main SMBus controller, > adding > the channel name to the adapter name is necessary to differentiate > the > ports better (for example in sensors output). > +static const char *piix4_main_port_names_sb800[4] = { Would it be constant you defined instead of magic number? > + "SDA0", "SDA2", "SDA3", "SDA4" > +}; > +static const char *piix4_aux_port_name_sb800 = "SDA1"; -- Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Intel Finland Oy -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html