On Thu, May 04, 2017 at 11:32:08PM +0300, Sakari Ailus wrote: > Hi Philipp, > > On Thu, May 04, 2017 at 03:38:57PM +0200, Philipp Zabel wrote: > ... > > +static const struct of_device_id video_mux_dt_ids[] = { > > + { .compatible = "video-mux", }, > > + { /* sentinel */ } > > +}; > > +MODULE_DEVICE_TABLE(of, video_mux_dt_ids); > > + > > +static struct platform_driver video_mux_driver = { > > Shouldn't this be const? To reply myself --- no. Driver registration requires that, which makes perfect sense. Please ignore the comment. -- Sakari Ailus e-mail: sakari.ailus@xxxxxx XMPP: sailus@xxxxxxxxxxxxxx