From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sascha Hauer Date: Tue, 17 Dec 2013 07:05:52 +0000 Subject: Re: [PATCHv2 23/27] OMAPDSS: connector-dvi: Add DT support Message-Id: <20131217070551.GR24559@pengutronix.de> List-Id: References: <1387205794-32246-1-git-send-email-tomi.valkeinen@ti.com> <1387205794-32246-24-git-send-email-tomi.valkeinen@ti.com> In-Reply-To: <1387205794-32246-24-git-send-email-tomi.valkeinen@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Tomi Valkeinen Cc: linux-omap@vger.kernel.org, linux-fbdev@vger.kernel.org, devicetree@vger.kernel.org Hi Tomi, On Mon, Dec 16, 2013 at 04:56:30PM +0200, Tomi Valkeinen wrote: > Signed-off-by: Tomi Valkeinen > --- > drivers/video/omap2/displays-new/connector-dvi.c | 43 ++++++++++++++++++++++++ > 1 file changed, 43 insertions(+) > > diff --git a/drivers/video/omap2/displays-new/connector-dvi.c b/drivers/video/omap2/displays-new/connector-dvi.c > index b6c50904038e..d1204b1c5182 100644 > > +static const struct of_device_id dvic_of_match[] = { > + { .compatible = "dvi-connector", }, Either the driver is too specific or the binding is too generic, but having such a generic name for an omap specific driver seems wrong. Same for panel-dpi, svideo-connector, composite-video-connector and hdmi-connector, Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |