On 20 May 2016 at 18:21, Charles Keepax <ckeepax@xxxxxxxxxxxxxxxxxxxxxxxxxxx> wrote: > On Tue, May 17, 2016 at 02:49:49PM +0200, Petr Kulhavy wrote: >> +static enum wm8985_type wm8985_data = WM8985; >> +static enum wm8985_type wm8758_data = WM8758; >> + >> +static const struct of_device_id wm8985_of_match[] = { >> + { .compatible = "wlf,wm8985", .data = &wm8985_data}, >> + { .compatible = "wlf,wm8758", .data = &wm8758_data}, > > You can probably just use (void *)WM8985 instead here. I know this pretty safe with integers but can you do this with enums? Doesn't it lead to undefined behaviour? Regards Petr -- 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