now
> it deals with multiple - so the naming is a bit confusing.
Hm, I too think that the name is confusing and doesn't show the correct
context..is vimc_streamer_referance a better name? What name do you
suggest?
Thanks,
Kaaira
>
> --
> Kieran
>
> On 19/08/2020 19:0
On Wed, Sep 02, 2020 at 11:46:50AM +0100, Kieran Bingham wrote:
> Hi Kaaira,
>
> On 19/08/2020 19:04, Kaaira Gupta wrote:
> > If multiple captures try to enable stream, start their stream but do not
> > initialise the pipeline again. Also, don't start the thread sepa
On Wed, Sep 02, 2020 at 11:29:31AM +0100, Kieran Bingham wrote:
> Hi Kaaira,
>
> On 19/08/2020 19:04, Kaaira Gupta wrote:
> > Multiple streams will share same stream struct if we want them to run on
> > same thread. So remove it from vimc_cap struct so that it doesn't
Hi,
On Wed, Sep 02, 2020 at 11:13:09AM +0100, Kieran Bingham wrote:
> Hi Kaaira,
>
> On 19/08/2020 19:04, Kaaira Gupta wrote:
> > Make separate functions for stopping streaming of entities in path of a
> > particular stream and stopping thread. This is needed to ensure th
On Wed, Sep 02, 2020 at 10:56:46AM +0100, Kieran Bingham wrote:
> Hi Kaaira, Dafna,
>
> On 28/08/2020 21:37, Dafna Hirschfeld wrote:
> > Hi,
> >
> > Am 21.08.20 um 23:01 schrieb Kaaira Gupta:
> >> Hi,
> >>
> >> On Fri, Au
Hi,
Sorry for the late reply
On Fri, Aug 28, 2020 at 10:37:14PM +0200, Dafna Hirschfeld wrote:
> Hi,
>
> Am 21.08.20 um 23:01 schrieb Kaaira Gupta:
> > Hi,
> >
> > On Fri, Aug 21, 2020 at 05:11:23PM +0200, Dafna Hirschfeld wrote:
> > >
> > >
&g
On Thu, Aug 20, 2020 at 04:36:56PM +0100, Kieran Bingham wrote:
> Hi Kaaira,
>
> On 19/08/2020 19:04, Kaaira Gupta wrote:
> > In the process of making vimc compatible for multiple streams, we need
> > to create a frame passing process such that two different entities can
>
Hi,
On Fri, Aug 21, 2020 at 05:11:23PM +0200, Dafna Hirschfeld wrote:
>
>
> Am 19.08.20 um 20:04 schrieb Kaaira Gupta:
> > Separate the process of initialising pipeline array from starting
> > streaming for all entities in path of a stream. This is needed because
> >
subdevices could go
out of sync as they are being started and stopped at the same time,
prevent this by serializing starting and stopping of the vimc streamer.
[Kaaira: only rebased the patch on current HEAD of media-tree]
Signed-off-by: Niklas Söderlund
Signed-off-by: Kaaira Gupta
---
.../media/test
through their use counts.
Signed-off-by: Kaaira Gupta
---
.../media/test-drivers/vimc/vimc-streamer.c | 95 ---
1 file changed, 83 insertions(+), 12 deletions(-)
diff --git a/drivers/media/test-drivers/vimc/vimc-streamer.c
b/drivers/media/test-drivers/vimc/vimc-streamer.c
index
Start another capture, if one is already running, by checking for
existing pipe. If it exists already, don't fail to start second capture,
instead join it to the already running pipeline.
Use the same stream struct used by already running capture.
Signed-off-by: Kaaira Gupta
---
drivers/
If multiple captures try to enable stream, start their stream but do not
initialise the pipeline again. Also, don't start the thread separately.
Starting their streams will update the use count and their frames would
be processed by the already running thread.
Signed-off-by: Kaaira
ff-by: Kaaira Gupta
---
drivers/media/test-drivers/vimc/vimc-capture.c | 15 +++
drivers/media/test-drivers/vimc/vimc-streamer.c | 17 ++---
drivers/media/test-drivers/vimc/vimc-streamer.h | 2 ++
3 files changed, 19 insertions(+), 15 deletions(-)
diff --git a/drivers/media
: Kaaira Gupta
---
drivers/media/test-drivers/vimc/vimc-capture.c | 3 +++
drivers/media/test-drivers/vimc/vimc-common.h | 2 ++
drivers/media/test-drivers/vimc/vimc-debayer.c | 7 +++
drivers/media/test-drivers/vimc/vimc-scaler.c | 7 +++
drivers/media/test-drivers/vimc/vimc-sensor.c | 6
Move the function vimc_get_source_entity() to vimc-common.c to make it
reusable.
Signed-off-by: Kaaira Gupta
---
drivers/media/test-drivers/vimc/vimc-common.c | 14 +++
drivers/media/test-drivers/vimc/vimc-common.h | 12 ++
.../media/test-drivers/vimc/vimc-streamer.c | 24
s the frame which gets passed on.
So, to take care of this, add a get_frame callback to vimc device and
use it to get the frames for an entity from previous entity instead of
returning and passing the frames as an argument in process_frame.
Signed-off-by: Kaaira Gupta
---
.../media/test-drivers
Make separate functions for stopping streaming of entities in path of a
particular stream and stopping thread. This is needed to ensure that
thread doesn't stop when one device stops streaming in case of multiple
streams.
Signed-off-by: Kaaira Gupta
---
.../media/test-drivers/vimc
.
Kaaira Gupta (7):
media: vimc: Move get_source_entity to vimc-common
media: vimc: Add get_frame callback
media: vimc: Separate starting stream from pipeline initialisation
media: vimc: Separate closing of stream and thread
media: vimc: Dynamically allocate stream struct
media: vimc: Join
On Wed, Aug 05, 2020 at 12:19:52AM +0530, Kaaira Gupta wrote:
> Hi
>
> On Tue, Aug 04, 2020 at 11:24:56AM +0100, Kieran Bingham wrote:
> > Hi Kaaira,
> >
> > On 31/07/2020 18:22, Kaaira Gupta wrote:
> > > Hi everyone,
> > >
> > > On Wed, Ju
Hi
On Tue, Aug 04, 2020 at 11:24:56AM +0100, Kieran Bingham wrote:
> Hi Kaaira,
>
> On 31/07/2020 18:22, Kaaira Gupta wrote:
> > Hi everyone,
> >
> > On Wed, Jul 29, 2020 at 05:24:25PM +0200, Dafna Hirschfeld wrote:
> >>
> >>
> >> On 29.07.
29.07.20 15:05, Kieran Bingham wrote:
> > > > Hi Dafna,
> > > >
> > > > On 28/07/2020 15:00, Dafna Hirschfeld wrote:
> > > > >
> > > > >
> > > > > On 28.07.20 14:07, Dafna Hirschfeld wrote:
> > > > > > Hi
>
On Thu, Jul 30, 2020 at 01:51:12PM +0300, Laurent Pinchart wrote:
Hi,
> Hi Kaaira,
>
> Thank you for the patches.
>
> On Fri, Jul 24, 2020 at 05:32:10PM +0530, Kaaira Gupta wrote:
> > This is version 2 of the patch series posted by Niklas for allowing
> > multip
On Tue, Jul 28, 2020 at 04:00:46PM +0200, Dafna Hirschfeld wrote:
>
>
> On 28.07.20 14:07, Dafna Hirschfeld wrote:
> > Hi
> >
> > On 28.07.20 13:39, Kaaira Gupta wrote:
> > > On Mon, Jul 27, 2020 at 02:54:30PM -0300, Helen Koike wrote:
> > > >
On Tue, Jul 28, 2020 at 02:24:37PM +0200, Dafna Hirschfeld wrote:
>
>
> On 24.07.20 14:02, Kaaira Gupta wrote:
> > An output which is running is already part of a pipeline and trying to
> > start a new pipeline is not possible. This prevents two capture devices
> >
On Mon, Jul 27, 2020 at 02:54:30PM -0300, Helen Koike wrote:
> Hi,
>
> On 7/27/20 11:31 AM, Kieran Bingham wrote:
> > Hi all,
> >
> > +Dafna for the thread discussion, as she's missed from the to/cc list.
> >
> >
> > On 24/07/2020 13:21, Kaai
On Fri, Jul 24, 2020 at 02:15:21PM +0200, Niklas Söderlund wrote:
Hi,
> Hi Kaaira,
>
> Thanks for your work.
Thanks for yours :D
>
> On 2020-07-24 17:32:10 +0530, Kaaira Gupta wrote:
> > This is version 2 of the patch series posted by Niklas for allowing
> > multip
they are being started and stopped at the same time,
prevent this by serializing starting and stopping of the vimc streamer.
[Kaaira: rebased the patch on current HEAD of media-tree
(8f2a4a9d5ff5202d0b3e3a144ebb9b67aabadd29)]
Signed-off-by: Niklas Söderlund
Signed-off-by: Kaaira Gupta
Söderlund
Signed-off-by: Kaaira Gupta
---
drivers/media/test-drivers/vimc/vimc-debayer.c | 8
drivers/media/test-drivers/vimc/vimc-scaler.c | 8
drivers/media/test-drivers/vimc/vimc-sensor.c | 9 -
3 files changed, 24 insertions(+), 1 deletion(-)
diff --git a/drivers
simultaneously use.
Changes since v1:
- All three patches rebased on latest media-tree.
Patch 3:
- Search for an entity with a non-NULL pipe instead of searching
for sensor. This terminates the search at output itself.
Kaaira Gupta (3):
media: vimc: Add usage count to
conditions accordingly]
Signed-off-by: Niklas Söderlund
Signed-off-by: Kaaira Gupta
---
.../media/test-drivers/vimc/vimc-capture.c| 35 ++-
1 file changed, 34 insertions(+), 1 deletion(-)
diff --git a/drivers/media/test-drivers/vimc/vimc-capture.c
b/drivers/media/test-drivers
this function.
Signed-off-by: Kaaira Gupta
Reviewed-by: Helen Koike
Reviewed-by: Kieran Bingham
---
drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 10 +-
include/media/tpg/v4l2-tpg.h | 2 +-
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/media
check for text in tpg_gen_text().
Signed-off-by: Kaaira Gupta
Reviewed-by: Kieran Bingham
Reviewed-by: Helen Koike
---
drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 30 +--
include/media/tpg/v4l2-tpg.h | 1 +
2 files changed, 29 insertions(+), 2 deletion
ses for separate test patterns
(Reported-by: kernel test robot )
- Renamed variables from camelcase to use '_'
- prefixed 'media' to the patches.
Kaaira Gupta (3):
media: tpg: change char argument to const char
media: tpg: Add function to return colors&#x
Add a control in VIMC to display information such as the correct order of
colors for a given test pattern, counter, brightness, hue, saturation,
contrast, width and height at sensor over test image.
Signed-off-by: Kaaira Gupta
Acked-by: Helen Koike
Reviewed-by: Kieran Bingham
---
drivers
Add a control in VIMC to display information such as the correct order of
colors for a given test pattern, counter, brightness, hue, saturation,
contrast, width and height at sensor over test image.
Signed-off-by: Kaaira Gupta
Acked-by: Helen Koike
---
drivers/media/test-drivers/vimc/Kconfig
check for text in tpg_gen_text().
Signed-off-by: Kaaira Gupta
Reviewed-by: Kieran Bingham
Reviewed-by: Helen Koike
---
drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 30 +--
include/media/tpg/v4l2-tpg.h | 1 +
2 files changed, 29 insertions(+), 2 deletion
this function.
Signed-off-by: Kaaira Gupta
Reviewed-by: Helen Koike
---
drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 10 +-
include/media/tpg/v4l2-tpg.h | 2 +-
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/drivers/media/common/v4l2-tpg/v4l2-tpg-core.c
refixed 'media' to the patches.
Kaaira Gupta (3):
media: tpg: change char argument to const char
media: tpg: Add function to return colors' order of test image
media: vimc: Add a control to display info on test image
drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 40 +++
check for text in tpg_gen_text().
Signed-off-by: Kaaira Gupta
Reviewed-by: Kieran Bingham
---
drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 30 +--
include/media/tpg/v4l2-tpg.h | 1 +
2 files changed, 29 insertions(+), 2 deletions(-)
diff --git a/drivers/me
ate switch cases for separate test patterns
(Reported-by: kernel test robot )
- Renamed variables from camelcase to use '_'
- prefixed 'media' to the patches.
Kaaira Gupta (3):
media: tpg: change char argument to const char
media: tpg: Add functi
Add a control in VIMC to display information such as the correct order of
colors for a given test pattern, counter, brightness, hue, saturation,
contrast, width and height at sensor over test image.
Signed-off-by: Kaaira Gupta
---
drivers/media/test-drivers/vimc/Kconfig | 2 +
drivers
Change the argument of type char * to const char * for function
tpg_gen_text().
Signed-off-by: Kaaira Gupta
---
drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 10 +-
include/media/tpg/v4l2-tpg.h | 2 +-
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a
On Fri, Jun 26, 2020 at 01:01:03PM -0300, Helen Koike wrote:
> Hi Kaaira,
>
> On 6/26/20 10:07 AM, Kaaira Gupta wrote:
> > Add a control in VIMC to display information such as the correct order of
> > colors for a given test pattern, brightness, hue, saturation, contrast,
&g
Add a control in VIMC to display information such as the correct order of
colors for a given test pattern, brightness, hue, saturation, contrast,
width and height at sensor over test image.
Signed-off-by: Kaaira Gupta
---
drivers/media/test-drivers/vimc/Kconfig | 2 +
drivers/media/test
y: kernel test robot )
- Made separate switch cases for separate test patterns
(Reported-by: kernel test robot )
- Renamed variables from camelcase to use '_'
- prefixed 'media' to the patches.
Kaaira Gupta (3):
media: tpg: change char argument to const c
check for text in tpg_gen_text().
Signed-off-by: Kaaira Gupta
Reviewed-by: Kieran Bingham
---
drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 30 +--
include/media/tpg/v4l2-tpg.h | 1 +
2 files changed, 29 insertions(+), 2 deletions(-)
diff --git a/drivers/me
Change the argument of type char * to const char * for function
tpg_gen_text().
Signed-off-by: Kaaira Gupta
---
drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 10 +-
include/media/tpg/v4l2-tpg.h | 2 +-
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a
On Fri, Jun 26, 2020 at 08:59:19AM -0300, Helen Koike wrote:
> Hi Kaaira,
>
> Thanks for the patch,
>
> On 6/26/20 8:36 AM, Kaaira Gupta wrote:
> > Add a control in VIMC to display information such as the correct order of
> > colors for a given test pattern, brightness
On Fri, Jun 26, 2020 at 08:59:19AM -0300, Helen Koike wrote:
> Hi Kaaira,
>
> Thanks for the patch,
>
> On 6/26/20 8:36 AM, Kaaira Gupta wrote:
> > Add a control in VIMC to display information such as the correct order of
> > colors for a given test pattern, brightness
)
- Renamed variables from camelcase to use '_'
- prefixed 'media' to the patches.
Kaaira Gupta (3):
media: tpg: change char argument to const char
media: tpg: Add function to return colors' order of test image
media: vimc: Add a control to display i
check for text in tpg_gen_text().
Signed-off-by: Kaaira Gupta
Reviewed-by: Kieran Bingham
---
drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 30 +--
include/media/tpg/v4l2-tpg.h | 1 +
2 files changed, 29 insertions(+), 2 deletions(-)
diff --git a/drivers/me
Add a control in VIMC to display information such as the correct order of
colors for a given test pattern, brightness, hue, saturation, contrast,
width and height at sensor, and time since streaming started over test image.
Signed-off-by: Kaaira Gupta
---
drivers/media/test-drivers/vimc/Kconfig
Change the argument of type char * to const char * for function
tpg_gen_text().
Signed-off-by: Kaaira Gupta
---
drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 10 +-
include/media/tpg/v4l2-tpg.h | 2 +-
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a
check for text in tpg_gen_text().
Signed-off-by: Kaaira Gupta
Reviewed-by: Kieran Bingham
---
drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 29 +--
include/media/tpg/v4l2-tpg.h | 1 +
2 files changed, 28 insertions(+), 2 deletions(-)
diff --git a/drivers/me
Add a control in VIMC to display information such as the correct order of
colors for a given test pattern, brightness, hue, saturation, contrast,
width and height at sensor over test image.
Signed-off-by: Kaaira Gupta
---
drivers/media/test-drivers/vimc/Kconfig | 2 +
drivers/media/test
Change the argument of type char * to const char * for function
tpg_gen_text().
Signed-off-by: Kaaira Gupta
---
drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 10 +-
include/media/tpg/v4l2-tpg.h | 2 +-
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a
be
defined. (Reported-by: kernel test robot )
- Made separate switch cases for separate test patterns
(Reported-by: kernel test robot )
- Renamed variables from camelcase to use '_'
- prefixed 'media' to the patches.
Kaaira Gupta (3):
media:
On Sat, Jun 20, 2020 at 12:05:28PM +0200, Dafna Hirschfeld wrote:
> Hi, thanks for the patch
>
> On 18.06.20 21:05, Kaaira Gupta wrote:
> > Add a control in VIMC to display information such as the correct oder of
> > colors for a given test pattern, brightness, hue, saturat
check for text in tpg_gen_text().
Signed-off-by: Kaaira Gupta
Reviewed-by: Kieran Bingham
---
drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 32 +--
include/media/tpg/v4l2-tpg.h | 1 +
2 files changed, 31 insertions(+), 2 deletions(-)
diff --git a/drivers/me
rted-by: kernel test robot )
- Made separate switch cases for separate test patterns
(Reported-by: kernel test robot )
- Renamed variables from camelcase to use '_'
- prefixed 'media' to the patches.
Kaaira Gupta (2):
media: tpg: Add functi
Add a control in VIMC to display information such as the correct oder of
colors for a given test pattern, brightness, hue, saturation, contrast
and, width and height at sensor over test image; and display that
information.
Signed-off-by: Kaaira Gupta
---
drivers/media/test-drivers/vimc/Kconfig
check for text in tpg_gen_text().
Signed-off-by: Kaaira Gupta
Reviewed-by: Kieran Bingham
---
Changes since v2:
- Create a 'define' to prevent repetition of the common color
sequence string.
- Use 'fallthrough' on case statement to prevent repetiti
On Mon, Jun 15, 2020 at 12:48:20PM +0100, Kieran Bingham wrote:
> Hi Kaaira,
>
> On 14/06/2020 21:02, Kaaira Gupta wrote:
> > Add a control in VIMC to show the correct order of the colors for a
> > given test pattern.
> > The control can be accessed by using sh
Add a control in VIMC to show the correct order of the colors for a
given test pattern.
The control can be accessed by using show_colors_order in v4l2-ctl
Signed-off-by: Kaaira Gupta
---
drivers/media/test-drivers/vimc/Kconfig | 2 ++
drivers/media/test-drivers/vimc/vimc-common.h | 1
check for text in tpg_gen_text().
Signed-off-by: Kaaira Gupta
---
drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 32 +--
include/media/tpg/v4l2-tpg.h | 1 +
2 files changed, 31 insertions(+), 2 deletions(-)
diff --git a/drivers/media/common/v4l2-tpg/v4l2-tpg
camelcase to use '_'
- prefixed 'media' to the patches.
Kaaira Gupta (2):
media: tpg: Add function to return colors' order of test image
media: vimc: Add a control to show test pattern colors' order
drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 32 +++
g_gen_text() for patterns for which color order
does not exist/make sense to print.
Make a separate control for the same in vimc to make displaying the
text optional.
Signed-off-by: Kaaira Gupta
---
drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 24 -
drivers/media/test-drivers/vi
ntrol, and maybe we can print some other useful information as well
(like vivid does).
Signed-off-by: Kaaira Gupta
---
drivers/media/test-drivers/vimc/Kconfig | 2 ++
drivers/media/test-drivers/vimc/vimc-core.c | 9 +
drivers/media/test-drivers/vimc/vimc-sensor.c | 8
3 fil
On Mon, Jun 01, 2020 at 01:36:59PM -0300, Helen Koike wrote:
>
>
> On 6/1/20 10:53 AM, Kaaira Gupta wrote:
> > Hi!
> >
> > Currently there is no method to know if the test image generated by vimc
> > is correct (except for comparing it with a known
Hi!
Currently there is no method to know if the test image generated by vimc
is correct (except for comparing it with a known 'correct' image). So, I
wanted to investigate about a possibility to add text to each color bar
of the generated pattern. I think currently vivid supports this
functionalit
sts as its supported formats wih --list-formats. Shouldn't BA24 be
possible to capture with vimc?
If yes, which entity should support it, if not debayer? Should there be
a separate conversion entity, or should we keep the support in debayer
itself for efficiency issues?
>
>
> On
Running qcam for pixelformat 0x34324142 showed that vimc debayer does
not support it. Hence, add the support for Alpha (255).
Signed-off-by: Kaaira Gupta
---
.../media/test-drivers/vimc/vimc-debayer.c| 27 ---
1 file changed, 18 insertions(+), 9 deletions(-)
diff --git a
72 matches
Mail list logo
