linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Setting up the links for imx7-mipi-csis
@ 2019-06-28 22:03 Fabio Estevam
  2019-06-29  1:12 ` Fabio Estevam
  0 siblings, 1 reply; 5+ messages in thread
From: Fabio Estevam @ 2019-06-28 22:03 UTC (permalink / raw)
  To: Rui Miguel Silva
  Cc: Steve Longerbeam, Philipp Zabel, Hans Verkuil, linux-media, Sakari Ailus

Hi Rui,

I am trying to get a OV5645 camera to work with a imx7d-pico board on
linux-next 20190628.

Here is the log:

[    0.285361] videodev: Linux video capture interface: v2.00
[    3.684025] csi: Registered csi capture as /dev/video0
..
[    3.689618] imx-media: subdev csi bound
[    4.055676] imx-media: imx7-mipi-csis.0:1 -> csi-mux:1
[    4.061032] imx-media: csi-mux:2 -> csi:0
[    4.065339] imx-media: ov5645 3-003c:0 -> imx7-mipi-csis.0:0

I try to follow the instructions from
Documentation/media/v4l-drivers/imx7.rst to setup the links, which
would be the following for imx7d-pico:

   media-ctl -l "'ov5645 3-003c':0 -> 'imx7-mipi-csis.0':0[1]"
   media-ctl -l "'imx7-mipi-csis.0':1 -> 'csi_mux':1[1]"
   media-ctl -l "'csi_mux':2 -> 'csi':0[1]"
   media-ctl -l "'csi':1 -> 'csi capture':0[1]"

The first command succeeds, but the second one fails:

# media-ctl -l "'imx7-mipi-csis.0':1 -> 'csi_mux':1[1]"
Unable to parse link: Invalid argument (22)

Any ideas, please?

Thanks,

Fabio Estevam

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Setting up the links for imx7-mipi-csis
  2019-06-28 22:03 Setting up the links for imx7-mipi-csis Fabio Estevam
@ 2019-06-29  1:12 ` Fabio Estevam
  2019-06-29  9:12   ` Sakari Ailus
  0 siblings, 1 reply; 5+ messages in thread
From: Fabio Estevam @ 2019-06-29  1:12 UTC (permalink / raw)
  To: Rui Miguel Silva
  Cc: Steve Longerbeam, Philipp Zabel, Hans Verkuil, linux-media, Sakari Ailus

Hi Rui,

On Fri, Jun 28, 2019 at 7:03 PM Fabio Estevam <festevam@gmail.com> wrote:

> The first command succeeds, but the second one fails:
>
> # media-ctl -l "'imx7-mipi-csis.0':1 -> 'csi_mux':1[1]"
> Unable to parse link: Invalid argument (22)

I have also tested it on a imx7s-warp and I got the same error.

Please let me know if you have any ideas on how to solve this problem.

Thanks

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Setting up the links for imx7-mipi-csis
  2019-06-29  1:12 ` Fabio Estevam
@ 2019-06-29  9:12   ` Sakari Ailus
  2019-06-29 11:42     ` Fabio Estevam
  0 siblings, 1 reply; 5+ messages in thread
From: Sakari Ailus @ 2019-06-29  9:12 UTC (permalink / raw)
  To: Fabio Estevam
  Cc: Rui Miguel Silva, Steve Longerbeam, Philipp Zabel, Hans Verkuil,
	linux-media

Hi Fabio,

On Fri, Jun 28, 2019 at 10:12:21PM -0300, Fabio Estevam wrote:
> Hi Rui,
> 
> On Fri, Jun 28, 2019 at 7:03 PM Fabio Estevam <festevam@gmail.com> wrote:
> 
> > The first command succeeds, but the second one fails:
> >
> > # media-ctl -l "'imx7-mipi-csis.0':1 -> 'csi_mux':1[1]"
> > Unable to parse link: Invalid argument (22)
> 
> I have also tested it on a imx7s-warp and I got the same error.
> 
> Please let me know if you have any ideas on how to solve this problem.

What does media-ctp -p say?

-- 
Sakari Ailus

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Setting up the links for imx7-mipi-csis
  2019-06-29  9:12   ` Sakari Ailus
@ 2019-06-29 11:42     ` Fabio Estevam
  2019-07-01  9:20       ` Rui Miguel Silva
  0 siblings, 1 reply; 5+ messages in thread
From: Fabio Estevam @ 2019-06-29 11:42 UTC (permalink / raw)
  To: Sakari Ailus
  Cc: Rui Miguel Silva, Steve Longerbeam, Philipp Zabel, Hans Verkuil,
	linux-media

Hi Sakari,

On Sat, Jun 29, 2019 at 6:13 AM Sakari Ailus <sakari.ailus@iki.fi> wrote:

> What does media-ctp -p say?

Thanks for this hint!

I managed to fix the media-ctl failure by looking at media-ctl -p output.

Documentation/media/v4l-drivers/imx7.rst mentions "csi_mux", but
media-ctl says "csi-mux", so I needed to change it to:

media-ctl -l "'imx7-mipi-csis.0':1 -> 'csi-mux':1[1]"

and then I see no errors after the media-ctl command.

I will send a patch fixing Documentation/media/v4l-drivers/imx7.rst .

Thanks,

Fabio Estevam

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Setting up the links for imx7-mipi-csis
  2019-06-29 11:42     ` Fabio Estevam
@ 2019-07-01  9:20       ` Rui Miguel Silva
  0 siblings, 0 replies; 5+ messages in thread
From: Rui Miguel Silva @ 2019-07-01  9:20 UTC (permalink / raw)
  To: Fabio Estevam
  Cc: Sakari Ailus, Steve Longerbeam, Philipp Zabel, Hans Verkuil, linux-media

Hi Guys,
On Sat 29 Jun 2019 at 12:42, Fabio Estevam wrote:
> Hi Sakari,
>
> On Sat, Jun 29, 2019 at 6:13 AM Sakari Ailus <sakari.ailus@iki.fi> wrote:
>
>> What does media-ctp -p say?
>
> Thanks for this hint!
>
> I managed to fix the media-ctl failure by looking at media-ctl -p output.
>
> Documentation/media/v4l-drivers/imx7.rst mentions "csi_mux", but
> media-ctl says "csi-mux", so I needed to change it to:
>
> media-ctl -l "'imx7-mipi-csis.0':1 -> 'csi-mux':1[1]"
>
> and then I see no errors after the media-ctl command.
>
> I will send a patch fixing Documentation/media/v4l-drivers/imx7.rst .
>

arriving late to the party. Thanks for the fix. yup, there is a
typo in there. Thanks Sakari.

---
Cheers,
	Rui


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2019-07-01  9:20 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-28 22:03 Setting up the links for imx7-mipi-csis Fabio Estevam
2019-06-29  1:12 ` Fabio Estevam
2019-06-29  9:12   ` Sakari Ailus
2019-06-29 11:42     ` Fabio Estevam
2019-07-01  9:20       ` Rui Miguel Silva

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).