linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* IMX frame interval monitor (FIM) available on ipu{1,2}_ic_prp{enc,vf} capture nodes?
@ 2020-12-31 16:48 Tim Harvey
  2021-01-02 17:55 ` Steve Longerbeam
  0 siblings, 1 reply; 4+ messages in thread
From: Tim Harvey @ 2020-12-31 16:48 UTC (permalink / raw)
  To: Steve Longerbeam; +Cc: linux-media, Hans Verkuil, Philipp Zabel, Fabio Estevam

Steve,

Some time ago you mentioned that adding FIM capability
(imx-media-fim.c) on the ipu{1,2}_ic_prp{enc,vf} capture nodes was on
your plate to fix as currently the controls are only available on the
ipu{1,2}_csi{0,1} capture nodes. Do you have any patches for that or
can you explain what needs to be done?

I have a user that wishes to use FIM to deal with a lossy analog
signal coming out of sync but they also wish to use the VDIC to
de-interlace.

Best regards,

Tim

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

* Re: IMX frame interval monitor (FIM) available on ipu{1,2}_ic_prp{enc,vf} capture nodes?
  2020-12-31 16:48 IMX frame interval monitor (FIM) available on ipu{1,2}_ic_prp{enc,vf} capture nodes? Tim Harvey
@ 2021-01-02 17:55 ` Steve Longerbeam
  2021-01-05 23:57   ` Tim Harvey
  0 siblings, 1 reply; 4+ messages in thread
From: Steve Longerbeam @ 2021-01-02 17:55 UTC (permalink / raw)
  To: Tim Harvey; +Cc: linux-media, Hans Verkuil, Philipp Zabel, Fabio Estevam

Hi Tim,

I gave this a shot a while ago, but I recall struggling with some 
locking problems. I haven't been able to return to it. I've updated the 
imx/add-fim-to-prpencvf branch to latest media-tree master branch 
(5.11.0-rc1), in my github fork [1]. The patches against master are:

0001-media-imx.rst-update-doc-to-reflect-FIM-changes.patch
0002-media-imx-prpencvf-attach-FIM.patch
0003-WIP.patch

The last patch was my first attempts to debug the locking problems.

I don't have access to imx6 hardware any longer, so I can't progress any 
further on it at this point.

Steve

[1] git@github.com:slongerbeam/mediatree.git


On 12/31/20 8:48 AM, Tim Harvey wrote:
> Steve,
>
> Some time ago you mentioned that adding FIM capability
> (imx-media-fim.c) on the ipu{1,2}_ic_prp{enc,vf} capture nodes was on
> your plate to fix as currently the controls are only available on the
> ipu{1,2}_csi{0,1} capture nodes. Do you have any patches for that or
> can you explain what needs to be done?
>
> I have a user that wishes to use FIM to deal with a lossy analog
> signal coming out of sync but they also wish to use the VDIC to
> de-interlace.
>
> Best regards,
>
> Tim


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

* Re: IMX frame interval monitor (FIM) available on ipu{1,2}_ic_prp{enc,vf} capture nodes?
  2021-01-02 17:55 ` Steve Longerbeam
@ 2021-01-05 23:57   ` Tim Harvey
  2021-01-07 23:51     ` Steve Longerbeam
  0 siblings, 1 reply; 4+ messages in thread
From: Tim Harvey @ 2021-01-05 23:57 UTC (permalink / raw)
  To: Steve Longerbeam; +Cc: linux-media, Hans Verkuil, Philipp Zabel, Fabio Estevam

On Sat, Jan 2, 2021 at 9:55 AM Steve Longerbeam <slongerbeam@gmail.com> wrote:
>
> Hi Tim,
>
> I gave this a shot a while ago, but I recall struggling with some
> locking problems. I haven't been able to return to it. I've updated the
> imx/add-fim-to-prpencvf branch to latest media-tree master branch
> (5.11.0-rc1), in my github fork [1]. The patches against master are:
>
> 0001-media-imx.rst-update-doc-to-reflect-FIM-changes.patch
> 0002-media-imx-prpencvf-attach-FIM.patch
> 0003-WIP.patch
>
> The last patch was my first attempts to debug the locking problems.
>
> I don't have access to imx6 hardware any longer, so I can't progress any
> further on it at this point.
>
> Steve
>
> [1] git@github.com:slongerbeam/mediatree.git

Steve,

Thanks for the patches!

I have found via experimentation that if you are using the VDIC to
de-interlace and you loose/regain the signal to the adv7180 it will
roll endlesslessly. However if you are doing raw capture the video
will sync up within a few seconds (240 lines * 16.6ms-per-frame =
3.8secs max I suspect). Thus the simplest solution for users who
suffer from this may just be to capture raw and upscale.

Best regards,

Tim

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

* Re: IMX frame interval monitor (FIM) available on ipu{1,2}_ic_prp{enc,vf} capture nodes?
  2021-01-05 23:57   ` Tim Harvey
@ 2021-01-07 23:51     ` Steve Longerbeam
  0 siblings, 0 replies; 4+ messages in thread
From: Steve Longerbeam @ 2021-01-07 23:51 UTC (permalink / raw)
  To: Tim Harvey; +Cc: linux-media, Hans Verkuil, Philipp Zabel, Fabio Estevam

Hi Tim,

On 1/5/21 3:57 PM, Tim Harvey wrote:
> On Sat, Jan 2, 2021 at 9:55 AM Steve Longerbeam <slongerbeam@gmail.com> wrote:
>> Hi Tim,
>>
>> I gave this a shot a while ago, but I recall struggling with some
>> locking problems. I haven't been able to return to it. I've updated the
>> imx/add-fim-to-prpencvf branch to latest media-tree master branch
>> (5.11.0-rc1), in my github fork [1]. The patches against master are:
>>
>> 0001-media-imx.rst-update-doc-to-reflect-FIM-changes.patch
>> 0002-media-imx-prpencvf-attach-FIM.patch
>> 0003-WIP.patch
>>
>> The last patch was my first attempts to debug the locking problems.
>>
>> I don't have access to imx6 hardware any longer, so I can't progress any
>> further on it at this point.
>>
>> Steve
>>
>> [1] git@github.com:slongerbeam/mediatree.git
> Steve,
>
> Thanks for the patches!
>
> I have found via experimentation that if you are using the VDIC to
> de-interlace and you loose/regain the signal to the adv7180 it will
> roll endlesslessly. However if you are doing raw capture the video
> will sync up within a few seconds (240 lines * 16.6ms-per-frame =
> 3.8secs max I suspect). Thus the simplest solution for users who
> suffer from this may just be to capture raw and upscale.

Yes, but for our users, a 3.8 second delay is unacceptable, for example 
in a rear-view-camera application, it's not acceptable to see rolling 
video for that long. Hence the FIM which will restore sync very quickly 
on loss/regain of the analog signal to the adv718x.

Steve

> Best regards,
>
> Tim


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

end of thread, other threads:[~2021-01-07 23:52 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-31 16:48 IMX frame interval monitor (FIM) available on ipu{1,2}_ic_prp{enc,vf} capture nodes? Tim Harvey
2021-01-02 17:55 ` Steve Longerbeam
2021-01-05 23:57   ` Tim Harvey
2021-01-07 23:51     ` Steve Longerbeam

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