From: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
To: linux-samsung-soc <linux-samsung-soc@vger.kernel.org>,
Maling list - DRI developers <dri-devel@lists.freedesktop.org>
Subject: drm/exynos: probe deferral for sub drivers
Date: Wed, 21 Sep 2016 00:07:23 +0200 [thread overview]
Message-ID: <57E1B31B.4000501@math.uni-bielefeld.de> (raw)
Hello,
I'm currently facing the following problem.
I want to use a devfreq device in one of the Exynos sub drivers, namely
the G2D sub driver. My current approach is to use
devfreq_get_devfreq_by_phandle() in g2d_probe(). But at the G2D probing
time the devfreq subsystem isn't initialized yet, and I get EPROBE_DEFER
back.
Now I simply let g2d_probe() fail with EPROBE_DEFER in that case.
Eventually devfreq is online and g2d_probe() runs successfully.
So far, so good. However some debugging shows that g2d_subdrv_probe(),
which does IOMMU setup and allocates the cmdlist, never happens in that
case.
I've looked at exynos_drm_load() which issues
exynos_drm_device_subdrv_probe() at some point, which apparantly only
probes those devices which are registered at that time. Hence, if the
G2D probe hasn't happened until then, the Exynos DRM won't ever see it.
That made me wonder how the driver makes sure that the sub drivers are
all readily probed in that situation. I see that a component_bind_all()
is issued prior to subdrv probing, but both the G2D and the IPP subdrv
don't have bind/unbind calls. I guess subdrv != subdrv, eh?
At present it appears that it's merely luck that the G2D is probed when
sub drivers are probed?
Anyway, probe deferral seems to work correctly with Exynos HDMI, so I
was wondering if there is some way to also make it work with the G2D?
Thanks in advance!
With best wishes,
Tobias
next reply other threads:[~2016-09-20 22:07 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20160920220749eucas1p1b78ce2aea95e1f6fdd31b28091ba9faa@eucas1p1.samsung.com>
2016-09-20 22:07 ` Tobias Jakobi [this message]
2016-09-22 13:51 ` drm/exynos: probe deferral for sub drivers Andrzej Hajda
2016-09-22 14:08 ` Tobias Jakobi
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=57E1B31B.4000501@math.uni-bielefeld.de \
--to=tjakobi@math.uni-bielefeld.de \
--cc=dri-devel@lists.freedesktop.org \
--cc=linux-samsung-soc@vger.kernel.org \
/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