From: Philipp Zabel <p.zabel@pengutronix.de>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: Mark Rutland <mark.rutland@arm.com>,
Stephen Boyd <sboyd@codeaurora.org>,
Michael Turquette <mturquette@baylibre.com>,
dri-devel@lists.freedesktop.org, Jie Qiu <jie.qiu@mediatek.com>,
Cawa Cheng <cawa.cheng@mediatek.com>,
Mao Huang <littlecvr@chromium.org>,
YT Shen <yt.shen@mediatek.com>,
Yingjoe Chen <yingjoe.chen@mediatek.com>,
devicetree@vger.kernel.org, Jitao Shi <jitao.shi@mediatek.com>,
Pawel Moll <pawel.moll@arm.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Rob Herring <robh+dt@kernel.org>,
linux-mediatek@lists.infradead.org,
Matthias Brugger <matthias.bgg@gmail.com>,
Paul Bolle <pebolle@tiscali.nl>,
Emil Velikov <emil.l.velikov@gmail.com>,
Tomasz Figa <tfiga@chromium.org>,
kernel@pengutronix.de, Kumar Gala <galak@codeaurora.org>
Subject: Re: [PATCH v16 3/8] drm/mediatek: Add DRM Driver for Mediatek SoC MT8173.
Date: Fri, 06 May 2016 16:40:00 +0200 [thread overview]
Message-ID: <1462545600.5058.59.camel@pengutronix.de> (raw)
In-Reply-To: <20160506131648.GB27098@phenom.ffwll.local>
Hi Daniel,
Am Freitag, den 06.05.2016, 15:16 +0200 schrieb Daniel Vetter:
> On Fri, May 06, 2016 at 03:05:49PM +0200, Philipp Zabel wrote:
> > From: CK Hu <ck.hu@mediatek.com>
> >
> > This patch adds an initial DRM driver for the Mediatek MT8173 DISP
> > subsystem. It currently supports two fixed output streams from the
> > OVL0/OVL1 sources to the DSI0/DPI0 sinks, respectively.
> >
> > Signed-off-by: CK Hu <ck.hu@mediatek.com>
> > Signed-off-by: YT Shen <yt.shen@mediatek.com>
> > Signed-off-by: Daniel Kurtz <djkurtz@chromium.org>
> > Signed-off-by: Bibby Hsieh <bibby.hsieh@mediatek.com>
> > Signed-off-by: Mao Huang <littlecvr@chromium.org>
> > Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
>
> Spotted two tiny things below.
[...]
> > +static int mtk_drm_bind(struct device *dev)
> > +{
> > + struct mtk_drm_private *private = dev_get_drvdata(dev);
> > + struct drm_device *drm;
> > + int ret;
> > +
> > + drm = drm_dev_alloc(&mtk_drm_driver, dev);
> > + if (!drm)
> > + return -ENOMEM;
> > +
> > + drm_dev_set_unique(drm, dev_name(dev));
> > +
> > + ret = drm_dev_register(drm, 0);
> > + if (ret < 0)
> > + goto err_free;
> > +
> > + drm->dev_private = private;
> > + private->drm = drm;
> > +
> > + ret = mtk_drm_kms_init(drm);
> > + if (ret < 0)
> > + goto err_unregister;
>
> Wrong way round, drm_dev_register should be last. And you're missing
> drm_connector_register_all here.
> -Daniel
Thanks, that also means the mtk_dsi and mtk_hdmi drivers must not call
drm_connector_(un)register anymore.
What about the connectors registered by external bridge drivers? Should
those check for (dev->driver->load) before calling
drm_connector_unregister?
regards
Philipp
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2016-05-06 14:40 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-06 13:05 [PATCH v16 0/8] MT8173 DRM support Philipp Zabel
2016-05-06 13:05 ` [PATCH v16 1/8] memory: mtk-smi: export mtk_smi_larb_get/put Philipp Zabel
2016-05-06 13:05 ` [PATCH v16 2/8] dt-bindings: drm/mediatek: Add Mediatek display subsystem dts binding Philipp Zabel
2016-05-06 13:05 ` [PATCH v16 3/8] drm/mediatek: Add DRM Driver for Mediatek SoC MT8173 Philipp Zabel
2016-05-06 13:16 ` Daniel Vetter
2016-05-06 14:40 ` Philipp Zabel [this message]
2016-05-06 13:05 ` [PATCH v16 4/8] drm/mediatek: Add DSI sub driver Philipp Zabel
2016-05-06 13:05 ` [PATCH v16 5/8] drm/mediatek: Add DPI " Philipp Zabel
2016-05-06 13:05 ` [PATCH v16 6/8] clk: mediatek: make dpi0_sel propagate rate changes Philipp Zabel
2016-05-06 13:05 ` [PATCH v16 7/8] clk: mediatek: Add hdmi_ref HDMI PHY PLL reference clock output Philipp Zabel
2016-05-06 13:05 ` [PATCH v16 8/8] clk: mediatek: remove hdmitx_dig_cts from TOP clocks Philipp Zabel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1462545600.5058.59.camel@pengutronix.de \
--to=p.zabel@pengutronix.de \
--cc=cawa.cheng@mediatek.com \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=emil.l.velikov@gmail.com \
--cc=galak@codeaurora.org \
--cc=ijc+devicetree@hellion.org.uk \
--cc=jie.qiu@mediatek.com \
--cc=jitao.shi@mediatek.com \
--cc=kernel@pengutronix.de \
--cc=linux-mediatek@lists.infradead.org \
--cc=littlecvr@chromium.org \
--cc=mark.rutland@arm.com \
--cc=matthias.bgg@gmail.com \
--cc=mturquette@baylibre.com \
--cc=pawel.moll@arm.com \
--cc=pebolle@tiscali.nl \
--cc=robh+dt@kernel.org \
--cc=sboyd@codeaurora.org \
--cc=tfiga@chromium.org \
--cc=yingjoe.chen@mediatek.com \
--cc=yt.shen@mediatek.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).