Re: [PATCH v3 13/24] platform: add video-multiplexer subdevice driver

[Steve Longerbeam]

On 01/24/2017 04:44 AM, Javier Martinez Canillas wrote:
> Hello Steve,
>
> On Fri, Jan 6, 2017 at 11:11 PM, Steve Longerbeam <slongerbeam@xxxxxxxxx> wrote:
> > From: Philipp Zabel <p.zabel@xxxxxxxxxxxxxx>
> [snip]
>
> > +config VIDEO_MULTIPLEXER
> > +       tristate "Video Multiplexer"
> > +       depends on VIDEO_V4L2_SUBDEV_API && MEDIA_CONTROLLER
> The driver can be build as a module...
>
> > +
> > +static const struct of_device_id vidsw_dt_ids[] = {
> > +       { .compatible = "video-multiplexer", },
> > +       { /* sentinel */ }
> > +};
> > +
> ... so you need a MODULE_DEVICE_TABLE(of, vidsw_dt_ids) here or
> otherwise module autoloading won't work.

Hi Javier, thanks for catching, done.

Steve