From: Steve Longerbeam <slongerbeam@gmail.com>
To: Tim Harvey <tharvey@gateworks.com>
Cc: linux-media <linux-media@vger.kernel.org>,
Philipp Zabel <p.zabel@pengutronix.de>,
Stephan Bauroth <der_steffi@gmx.de>
Subject: Re: IMX6 ADV7180 no /dev/media
Date: Fri, 15 Sep 2017 16:39:14 -0700 [thread overview]
Message-ID: <421dbf8f-6574-7d3b-8842-ffe4c0c6da78@gmail.com> (raw)
In-Reply-To: <CAJ+vNU3srz1u4x2wku4JKAOWGH8Gc8Wh0eo5aTEhACqoNeE1ow@mail.gmail.com>
On 09/15/2017 04:28 PM, Tim Harvey wrote:
> On Fri, Sep 15, 2017 at 3:26 PM, Steve Longerbeam <slongerbeam@gmail.com> wrote:
>> Hi Tim,
>>
>>
>> On 09/15/2017 02:26 PM, Tim Harvey wrote:
>>> Greetings,
>>>
>>> I'm testing Linux master built with imx_v6_v7_defconfig on a GW51xx which
>>> has an ADV7180 analog video decoder and am not seeing the imx6 /dev/media
>>> node get created:
>>>
>>> [ 0.000000] OF: fdt: Machine model: Gateworks Ventana i.MX6 Dual/Quad
>>> GW51XX
>>> ...
>>> [ 6.089039] imx-media: Registered subdev ipu1_vdic
>>> [ 6.094505] imx-media: Registered subdev ipu2_vdic
>>> [ 6.099851] imx-media: Registered subdev ipu1_ic_prp
>>> [ 6.105074] imx-media: Registered subdev ipu1_ic_prpenc
>>> [ 6.111346] ipu1_ic_prpenc: Registered ipu1_ic_prpenc capture as
>>> /dev/video0
>>> [ 6.119007] imx-media: Registered subdev ipu1_ic_prpvf
>>> [ 6.124733] ipu1_ic_prpvf: Registered ipu1_ic_prpvf capture as
>>> /dev/video1
>>> [ 6.131867] imx-media: Registered subdev ipu2_ic_prp
>>> [ 6.137125] imx-media: Registered subdev ipu2_ic_prpenc
>>> [ 6.142921] ipu2_ic_prpenc: Registered ipu2_ic_prpenc capture as
>>> /dev/video2
>>> [ 6.150226] imx-media: Registered subdev ipu2_ic_prpvf
>>> [ 6.155934] ipu2_ic_prpvf: Registered ipu2_ic_prpvf capture as
>>> /dev/video3
>>> [ 6.164011] imx-media: Registered subdev ipu1_csi0
>>> [ 6.169768] ipu1_csi0: Registered ipu1_csi0 capture as /dev/video4
>>> [ 6.176281] imx-media: Registered subdev ipu1_csi1
>>> [ 6.181681] ipu1_csi1: Registered ipu1_csi1 capture as /dev/video5
>>> [ 6.188189] imx-media: Registered subdev ipu2_csi0
>>> [ 6.193680] ipu2_csi0: Registered ipu2_csi0 capture as /dev/video6
>>> [ 6.200108] imx-media: Registered subdev ipu2_csi1
>>> [ 6.205577] ipu2_csi1: Registered ipu2_csi1 capture as /dev/video7
>>> ...
>>> [ 96.981117] adv7180 2-0020: chip found @ 0x20 (21a8000.i2c)
>>> [ 97.019674] imx-media: Registered subdev adv7180 2-0020
>>> [ 97.019712] imx-media capture-subsystem: Entity type for entity adv7180
>>> 2-0020 was not initialized!
>>>
>>> I suspect the failure of the adv7180 is causing the issue. Steve mentioned
>>> some time ago that this was an error that needed to be fixed upstream but
>>> I'm not clear if that is still the case.
>>>
>> That does need fixing but is not the cause.
>>
>>> I haven't looked at IMX media drivers since they were accepted to mainline
>>> a few months back. Perhaps I'm simply forgetting to enable something in the
>>> kernel that imx_v6_v7_defconfig doesn't turn on?
>>
>> Yes, it looks like you are missing the video-mux. Enable CONFIG_VIDEO_MUX
>> and CONFIG_MUX_MMIO.
>>
> Steve,
>
> Indeed that was it! What clued you into that being the missing component?
It was easy :) There needs to be the following message from imx-media:
imx-media: Registered subdev ipu1_csi0_mux
imx-media: Registered subdev ipu2_csi1_mux
for quad, and
imx-media: Registered subdev ipu1_csi0_mux
imx-media: Registered subdev ipu1_csi1_mux
for D/L.
All imx6 platforms require the video-mux for video capture.
Steve
>
> Sounds like we need to enable that for imx_v6_v7_defconfig.
Agreed, but I notice now that CONFIG_MEDIA_CONTROLLER and
CONFIG_VIDEO_V4L2_SUBDEV_API are not enabled there anymore.
I guess there was some config conflict that necessitated disabling
them, but without them there is no media platform device support at
all anyway. Those must be enabled first before getting CONFIG_VIDEO_MUX.
Steve
next prev parent reply other threads:[~2017-09-15 23:39 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CAJ+vNU3DPFEc6YnEfcYAv1=beJ96W5PSt=eBfoxCXqKnbNqfMg@mail.gmail.com>
2017-09-15 22:26 ` IMX6 ADV7180 no /dev/media Steve Longerbeam
2017-09-15 23:28 ` Tim Harvey
2017-09-15 23:39 ` Steve Longerbeam [this message]
2017-09-15 23:46 ` Fabio Estevam
2017-09-16 0:08 ` Steve Longerbeam
2017-09-17 0:48 ` Fabio Estevam
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=421dbf8f-6574-7d3b-8842-ffe4c0c6da78@gmail.com \
--to=slongerbeam@gmail.com \
--cc=der_steffi@gmx.de \
--cc=linux-media@vger.kernel.org \
--cc=p.zabel@pengutronix.de \
--cc=tharvey@gateworks.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