[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <05f38c1c-bf6a-3be0-9aa9-c23a4ca24cdd@gmail.com>
Date: Wed, 25 Jan 2017 17:22:34 -0800
From: Steve Longerbeam <slongerbeam@...il.com>
To: Javier Martinez Canillas <javier@...hile0.org>
Cc: Rob Herring <robh+dt@...nel.org>,
Mark Rutland <mark.rutland@....com>,
Shawn Guo <shawnguo@...nel.org>,
Sascha Hauer <kernel@...gutronix.de>,
Fabio Estevam <fabio.estevam@....com>,
Russell King <linux@...linux.org.uk>,
Mauro Carvalho Chehab <mchehab@...nel.org>,
Hans Verkuil <hverkuil@...all.nl>, nick@...anahar.org,
Markus Heiser <markus.heiser@...marit.de>,
Philipp Zabel <p.zabel@...gutronix.de>,
Laurent Pinchart <laurent.pinchart+renesas@...asonboard.com>,
Benoit Parrot <bparrot@...com>,
Geert Uytterhoeven <geert@...ux-m68k.org>,
Arnd Bergmann <arnd@...db.de>, sudipm.mukherjee@...il.com,
minghsiu.tsai@...iatek.com, tiffany.lin@...iatek.com,
Jean-Christophe Trotin <jean-christophe.trotin@...com>,
horms+renesas@...ge.net.au, niklas.soderlund+renesas@...natech.se,
robert.jarzmik@...e.fr, songjun.wu@...rochip.com,
andrew-ct.chen@...iatek.com,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
devel@...verdev.osuosl.org,
"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
Steve Longerbeam <steve_longerbeam@...tor.com>,
Sascha Hauer <s.hauer@...gutronix.de>,
Linux Kernel <linux-kernel@...r.kernel.org>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>,
Linux Media Mailing List <linux-media@...r.kernel.org>
Subject: Re: [PATCH v3 13/24] platform: add video-multiplexer subdevice driver
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@...il.com> wrote:
>> From: Philipp Zabel <p.zabel@...gutronix.de>
> [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
Powered by blists - more mailing lists