From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from nblzone-211-213.nblnetworks.fi ([83.145.211.213]:57860 "EHLO hillosipuli.retiisi.org.uk" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751473AbdEDWrS (ORCPT ); Thu, 4 May 2017 18:47:18 -0400 Date: Fri, 5 May 2017 01:47:14 +0300 From: Sakari Ailus To: Philipp Zabel Cc: linux-media@vger.kernel.org, devicetree@vger.kernel.org, Steve Longerbeam , Peter Rosin , Pavel Machek , Rob Herring , Mark Rutland , Vladimir Zapolskiy , kernel@pengutronix.de, Sascha Hauer , Steve Longerbeam Subject: Re: [PATCH v3 2/2] [media] platform: add video-multiplexer subdevice driver Message-ID: <20170504224714.GF7456@valkosipuli.retiisi.org.uk> References: <1493905137-27051-1-git-send-email-p.zabel@pengutronix.de> <1493905137-27051-2-git-send-email-p.zabel@pengutronix.de> <20170504203208.GC7456@valkosipuli.retiisi.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170504203208.GC7456@valkosipuli.retiisi.org.uk> Sender: linux-media-owner@vger.kernel.org List-ID: 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@iki.fi XMPP: sailus@retiisi.org.uk