On 2017-01-01 16:38, Andy Shevchenko wrote: > On Wed, Nov 30, 2016 at 10:17 AM, Peter Rosin <peda@xxxxxxxxxx> wrote: >> This is a generic simple i2c mux that uses the generic multiplexer >> subsystem to do the muxing. > >> +static const struct of_device_id i2c_mux_of_match[] = { >> + { .compatible = "i2c-mux-simple,parent-locked", >> + .data = (void *)0, }, >> + { .compatible = "i2c-mux-simple,mux-locked", >> + .data = (void *)1, }, >> + {}, >> +}; > > Perhaps > > #define I2C_MUX_LOCKED_PARENT 0 > #define I2C_MUX_LOCKED 1 I2C_MUX_LOCKED is already "taken" in include/linux/i2c-mux.h but I'll use SIMPLE_PARENT_LOCKED and SIMPLE_MUX_LOCKED instead. Cheers, peda -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html