<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Oct 23, 2019 at 12:24 PM Niklas Söderlund <<a href="mailto:niklas.soderlund%2Brenesas@ragnatech.se">niklas.soderlund+renesas@ragnatech.se</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Hans,<br>
<br>
On 2019-10-23 09:01:40 +0200, Hans Verkuil wrote:<br>
> Hi Niklas,<br>
> <br>
> For one reason or another this series was never reviewed/picked up and<br>
> it now no longer applies.<br>
> <br>
> Combined with the big switch to a monolithic driver I am sure that this<br>
> series needs to be redone. So I am marking it as "Changes Requested" and<br>
> it is up to you to decide whether to rebase/rework this series.<br>
<br>
I talked to Helen about this series and she thought it could be a nice <br>
candidate for a student workshop/class so I'm not planing to respin this <br>
myself.<br>
<br></blockquote><div>Hi, </div><div>Thanks for the patchset,</div><div>I just reviewed it in case it will ever be continued, </div><div><br></div><div>Dafna </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> <br>
> Regards,<br>
> <br>
>       Hans<br>
> <br>
> On 5/18/19 3:07 AM, Niklas Söderlund wrote:<br>
> > Hi,<br>
> > <br>
> > This series adds support for two (or more) capture devices to be <br>
> > connected to the same senors and run simultaneously. Each capture device <br>
> > can be started and stopped independent of each other.<br>
> > <br>
> > Patch 1/3 and 2/3 deals with solving the issues that arises once two <br>
> > capture devices can be part of the same pipeline. While 3/3 allows for <br>
> > two capture devices to be part of the same pipeline and thus allows for <br>
> > simultaneously use.<br>
> > <br>
> > The series is based on the latest media-tree and it functionality can be <br>
> > demonstrated with the following test.<br>
> > <br>
> >>>> begin test <<<<br>
> > mdev=/dev/media0<br>
> > <br>
> > media-ctl -d $mdev -l "'Debayer A':1 -> 'Scaler':0 [1]"<br>
> > media-ctl -d $mdev -l "'Debayer B':1 -> 'Scaler':0 [0]"<br>
> > <br>
> > media-ctl -d $mdev -V "'Debayer A':0 [fmt:RGB888_1X24/640x480 field:none]"<br>
> > media-ctl -d /dev/media0 -V "'Sensor A':0 [fmt:SRGGB8_1X8/640x480 field:none]"<br>
> > <br>
> > yavta -f RGB24 -s 1920x1440 --field none /dev/video2<br>
> > yavta -f SRGGB8 -s 640x480 --field none /dev/video0<br>
> > <br>
> > yavta -f RGB24 -s 1920x1440 --field none /dev/video2 --capture=100 &<br>
> > yavta -f SRGGB8 -s 640x480 --field none /dev/video0 --capture=100<br>
> > wait<br>
> >>>> end test <<<<br>
> > <br>
> > In addition to testing with this test the series is tested with multiple <br>
> > qv4l2 instances controlling different capture devices connected to the <br>
> > same sensor.<br>
> > <br>
> > Niklas Söderlund (3):<br>
> >   vimc: Add usage count to subdevices<br>
> >   vimc: Serialize vimc_streamer_s_stream()<br>
> >   vimc: Join pipeline if one already exists<br>
> > <br>
> >  drivers/media/platform/vimc/vimc-capture.c  | 35 ++++++++++++++++++++-<br>
> >  drivers/media/platform/vimc/vimc-debayer.c  |  8 +++++<br>
> >  drivers/media/platform/vimc/vimc-scaler.c   |  8 +++++<br>
> >  drivers/media/platform/vimc/vimc-sensor.c   |  7 +++++<br>
> >  drivers/media/platform/vimc/vimc-streamer.c | 23 +++++++++-----<br>
> >  5 files changed, 73 insertions(+), 8 deletions(-)<br>
> > <br>
> <br>
<br>
-- <br>
Regards,<br>
Niklas Söderlund<br>
</blockquote></div></div>