RE: [PATCH 04/10 v2] v4l: Add Samsung FIMC (video postprocessor) driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----Original Message-----
> From: Kukjin Kim [mailto:kgene.kim@xxxxxxxxxxx]
> Sent: Friday, July 16, 2010 1:39 PM
> To: 'Sylwester Nawrocki'; linux-samsung-soc@xxxxxxxxxxxxxxx; linux-arm-
> kernel@xxxxxxxxxxxxxxxxxxx
> Cc: p.osciak@xxxxxxxxxxx; m.szyprowski@xxxxxxxxxxx;
> kyungmin.park@xxxxxxxxxxx; linux-media@xxxxxxxxxxxxxxx
> Subject: RE: [PATCH 04/10 v2] v4l: Add Samsung FIMC (video
> postprocessor) driver
> 
> Sylwester Nawrocki wrote:
> >
> Hi,
> 
> Samsung S.LSI hardly recommends DMA mode to implement the feature not
> FIFO
> mode because of H/W limitation.

Does the hardware issue occur in all SoCs or only some specific versions 
are affected?
So far we were successful with S5PC110 EVT0, however we could not make 
it work on S5PV210 (EVT1). That said I have created an option in the FIMC
variant definition (samsung_fimc_variant::fifo_capable) so that 
the V4L2 video node for FIMC FIFO operation mode is registered only 
for selected SoC variants.

> 
> > This driver exports two video device nodes per each FIMC device,
> > one for for memory to memory (color conversion, image resizing,
> > flipping and rotation) operations and one for direct
> > V4L2 output interface to framebuffer.
> >
> > Signed-off-by: Sylwester Nawrocki <s.nawrocki@xxxxxxxxxxx>
> > Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx>
> > Reviewed-by: Pawel Osciak <p.osciak@xxxxxxxxxxx>
> > Reviewed-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>
> > ---
> >  drivers/media/video/Kconfig                  |   20 +
> >  drivers/media/video/Makefile                 |    1 +
> >  drivers/media/video/samsung/fimc/Makefile    |    3 +
> >  drivers/media/video/samsung/fimc/fimc-core.c | 1664
> > ++++++++++++++++++++++++++
> >  drivers/media/video/samsung/fimc/fimc-core.h |  541 +++++++++
> >  drivers/media/video/samsung/fimc/fimc-fifo.c |  814 +++++++++++++
> >  drivers/media/video/samsung/fimc/fimc-reg.c  |  585 +++++++++
> >  include/linux/videodev2.h                    |    1 +
> >  8 files changed, 3629 insertions(+), 0 deletions(-)
> >  create mode 100644 drivers/media/video/samsung/fimc/Makefile
> >  create mode 100644 drivers/media/video/samsung/fimc/fimc-core.c
> >  create mode 100644 drivers/media/video/samsung/fimc/fimc-core.h
> >  create mode 100644 drivers/media/video/samsung/fimc/fimc-fifo.c
> >  create mode 100644 drivers/media/video/samsung/fimc/fimc-reg.c
> >
> 
> (snip)
> 
> Thanks.
> 
> Best regards,
> Kgene.
> --
> Kukjin Kim <kgene.kim@xxxxxxxxxxx>, Senior Engineer,
> SW Solution Development Team, Samsung Electronics Co., Ltd.

Best regards,
--
Sylwester Nawrocki
Samsung Poland R&D Center


--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux