From: Marc Dietrich <marvin24-Mmb7MZpHnFY@public.gmane.org>
To: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
Cc: linux-tegra <linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
Lucas Stach <dev-8ppwABl0HbeELgA04lAiVw@public.gmane.org>,
Thomas Meyer <thomas-VsYtu1Qij5c@public.gmane.org>,
Laxman Dewangan
<ldewangan-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Subject: Re: No HDMI output on AC100
Date: Sun, 09 Jun 2013 14:02:16 +0200 [thread overview]
Message-ID: <1963558.n4FXxA4Dl5@ax5200p> (raw)
In-Reply-To: <5714076.saxNrNAXNW@ax5200p>
On Sunday 09 June 2013 13:42:46 Marc Dietrich wrote:
> On Saturday 08 June 2013 20:45:17 Stephen Warren wrote:
> > On 06/08/2013 02:07 PM, Marc Dietrich wrote:
> > > On Saturday 08 June 2013 18:05:52 Lucas Stach wrote:
> > >> Am Samstag, den 08.06.2013, 16:21 +0200 schrieb Thomas Meyer:
> > >>>> On Saturday 08 June 2013 10:33:55 Thomas Meyer wrote:
> > >>>>> Kernel version: 3.10.0-0.rc4.git0.1.fc20.armv7hl
> > >>>>>
> > >>>>> This is the latest Fedora ARM kernel. Config is available here:
> > >>>>> http://pkgs.fedoraproject.org/cgit/kernel.git/tree/config-armv7
> > >>>>>
> > >>>>> This is what I see in the kernel log:
> > >>>>>
> > >>>>> $ dmesg |grep hdmi
> > >>>>> [ 33.403417] tegra-hdmi 54280000.hdmi: failed to get VDD regulator
> > >>>>> [ 33.403441] platform 54280000.hdmi: Driver tegra-hdmi requests
> > >>>>> probe
> > >>>>>
> > >>>>> any ideas?
> > >>>>
> > >>>> is the regulator (tps6586x) build-in?
> > >>>
> > >>> No, it's a module.
> > >>>
> > >>>> The make everything a module path isn't
> > >>>> well tested I guess.
> > >>
> > >> It's not the problem here. The driver requests to be probed again and
> > >> comes up once the regulator is there.
> > >>
> > >> Though the pixel clock looks odd for a HD HDMI monitor. Try looking at
> > >> the Xorg log to see what mode of your monitor get's chosen and why it
> > >> fails.
> > >
> > > I just checked myself. Found that tps6586x-regulator module can't be
> > > autoloaded, so I connected my TV, booted and modprobed the module by
> > > hand.
> > > All works more or less fine, so probe deferral is no problem.
> >
> > It could well just be that the tps6586x driver is missing the module
> > correct MODULE_ALIAS that makes the module auto-load when the relevant
> > entry is in the DT. The top-level MFD driver does have a
> > MODULE_DEVICE_TABLE which should set this up automatically, so perhaps
> > it's because there's nothing causing the regulator MFD component to load
> > when the top-level MFD component loads?
>
> the core driver adds sub devices, but in case of DT, the number of
> subdevices is zero AFAICS (tps6586x_parse_dt). However, these subdevs do
> not include the regulators.
ah, the driver is named tps6586x-pmic (and the module alias is tps6586x-
regulator), so in case of platform_device type init, it is probed.
> Also, the regulator nodes don't have a compatibly property, so direct
> probing won't work.
>
> I have no idea how the regulators are probed at all...
just found subsys_initcall(tps6586x_regulator_init), so the regulator is
forced probed if build-in.
I guess replacing this by module_init (or better the short hand macro) should
work now that deferral probing is working. Correct? Will try out...
Marc
next prev parent reply other threads:[~2013-06-09 12:02 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-08 8:33 No HDMI output on AC100 Thomas Meyer
[not found] ` <1370680435.24531.12.camel-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2013-06-08 13:00 ` Marc Dietrich
2013-06-08 14:21 ` Thomas Meyer
[not found] ` <68C7110D-C493-4A1F-8011-5F9259C2F4E7-VsYtu1Qij5c@public.gmane.org>
2013-06-08 15:12 ` Marc Dietrich
2013-06-08 16:05 ` Lucas Stach
[not found] ` <1370707552.1829.4.camel-6fNNZcyRN12a6lf8Wg2v7Z5kstrrjoWp@public.gmane.org>
2013-06-08 20:07 ` Marc Dietrich
2013-06-09 2:45 ` Stephen Warren
[not found] ` <51B3EC3D.3080306-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-06-09 11:42 ` Marc Dietrich
2013-06-09 12:02 ` Marc Dietrich [this message]
2013-06-09 12:22 ` Marc Dietrich
2013-06-10 16:13 ` Stephen Warren
2013-06-09 8:31 ` Thomas Meyer
2013-06-09 8:52 ` Thomas Meyer
[not found] ` <1370767921.2283.8.camel-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2013-06-12 10:50 ` Thierry Reding
2013-06-14 18:27 ` Thomas Meyer
[not found] ` <EF2DBD80-3DDD-416F-935C-E48261BF6CF5-VsYtu1Qij5c@public.gmane.org>
2013-06-14 20:37 ` Thierry Reding
2013-06-14 20:59 ` Thomas Meyer
[not found] ` <C2E8F821-5848-4FC0-992C-385D48611324-VsYtu1Qij5c@public.gmane.org>
2013-06-14 21:26 ` Thierry Reding
2013-06-15 9:37 ` Thomas Meyer
[not found] ` <1371289066.3792.2.camel-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2013-06-15 9:43 ` Thierry Reding
2013-06-15 15:02 ` Thomas Meyer
[not found] ` <1371308577.2480.6.camel-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2013-07-17 4:39 ` Thierry Reding
2013-07-17 19:24 ` Marc Dietrich
2013-06-09 8:33 ` Thomas Meyer
2013-06-09 8:38 ` Thomas Meyer
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=1963558.n4FXxA4Dl5@ax5200p \
--to=marvin24-mmb7mzphnfy@public.gmane.org \
--cc=dev-8ppwABl0HbeELgA04lAiVw@public.gmane.org \
--cc=ldewangan-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
--cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org \
--cc=thomas-VsYtu1Qij5c@public.gmane.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.