media_device_init() before registering the V4L2 device.
So, move the call for media_device_init() to happen earlier on
exynos4-is and s3c-camif.
This is just a cleanup patch.
Signed-off-by: Mauro Carvalho Chehab <mche...@osg.samsung.com>
---
drivers/media/platform/exynos4-is/media-dev.c | 4 ++--
d
Em Mon, 28 Dec 2015 08:26:33 -0200
Mauro Carvalho Chehab <mche...@osg.samsung.com> escreveu:
> Em Mon, 28 Dec 2015 03:14:53 +0200
> Sakari Ailus <sakari.ai...@iki.fi> escreveu:
>
> > Hi Mauro,
> >
> > On Tue, Dec 15, 2015 at 09:13:42AM -0200, Mauro Carv
Em Thu, 10 Sep 2015 20:14:04 +0300
Sakari Ailus escreveu:
> Hi Javier,
>
> Thanks for the set! A few comments below.
>
> Javier Martinez Canillas wrote:
> > The media device node is registered and so made visible to user-space
> > before entities are registered
t's revert to WARN_ON() and fix
the init code in a way that, if something goes wrong during
device init, driver probe will fail without causing the Kernel
to BUG.
Signed-off-by: Mauro Carvalho Chehab <mche...@osg.samsung.com>
---
Javier is in vacations. So, instead of waiting for him to fixup tho
t's revert to WARN_ON() and fix
the init code in a way that, if something goes wrong during
device init, driver probe will fail without causing the Kernel
to BUG.
Signed-off-by: Mauro Carvalho Chehab <mche...@osg.samsung.com>
---
drivers/media/media-device.c | 8 ---
driver
Carvalho Chehab <mche...@osg.samsung.com>
---
Documentation/media-framework.txt | 18 +++---
Documentation/video4linux/v4l2-framework.txt | 8
Documentation/zh_CN/video4linux/v4l2-framework.txt | 8
drivers/media/dvb-core/dv
Em Sun, 06 Dec 2015 04:16:15 +0200
Laurent Pinchart <laurent.pinch...@ideasonboard.com> escreveu:
> Hi Mauro,
>
> Thank you for the patch.
>
> On Sunday 30 August 2015 00:06:43 Mauro Carvalho Chehab wrote:
> > Instead of relying on media subtype, use the new macros
Em Mon, 16 Nov 2015 13:28:10 +0100
Arnd Bergmann escreveu:
> I think we can also move some of the existing platform data headers to the
> same
> place, but that could be a separate patch:
>
> $ git grep linux/platform_data drivers/media/
>
cd include/$MAIN_DIR;
for j in $DIRS; do
for i in $(ls $j); do
echo " perl -ne
's,include/$PREV_DIR($i)\b,include/$MAIN_DIR$j\1,; print \$_' |\\";
done;
done;
Em Wed, 11 Nov 2015 21:26:31 +0100
Arnd Bergmann <a...@arndb.de> escreveu:
> On Wednesday 11 November 2015 15:14:48 Mauro Carvalho Chehab wrote:
> > rename include/media/{ => platform}/exynos-fimc.h (100%)
> > rename include/media/{ => platform}/mmp-camera.h (100%
Em Fri, 13 Nov 2015 22:31:15 +0100
Arnd Bergmann <a...@arndb.de> escreveu:
> On Friday 13 November 2015 17:13:41 Mauro Carvalho Chehab wrote:
> > Em Wed, 11 Nov 2015 21:26:31 +0100
> > Arnd Bergmann <a...@arndb.de> escreveu:
> >
>
> &g
in
$(ls $j); do echo "perl -ne 's,(include
[\\\"\\<]media/)($i)([\\\"\\>]),\1$j\2\3,; print \$_' |\\"; done; done; echo
"cat > a && mv a \$i; done") >script && . ./script
Signed-off-by: Mauro Carvalho Chehab <mche...@osg.samsung.com>
ren
7 @@ static inline bool is_media_entity_v4l2_subdev(struct
> media_entity *entity)
> case MEDIA_ENT_F_LENS:
> case MEDIA_ENT_F_ATV_DECODER:
> case MEDIA_ENT_F_TUNER:
> + case MEDIA_ENT_F_V4L2_SUBDEV_UNKNOWN:
> return true;
OK.
Reviewed-by:
Em Mon, 12 Oct 2015 00:07:52 +0300
Sakari Ailus <sakari.ai...@iki.fi> escreveu:
> Hi Mauro,
>
> On Sun, Aug 30, 2015 at 12:06:43AM -0300, Mauro Carvalho Chehab wrote:
> > Instead of relying on media subtype, use the new macros to detect
> > if an entity is a
:709:17:got void [noderef]
*
Signed-off-by: Mauro Carvalho Chehab <mche...@osg.samsung.com>
diff --git a/drivers/media/platform/exynos4-is/mipi-csis.c
b/drivers/media/platform/exynos4-is/mipi-csis.c
index d74e1bec3d86..4b85105dc159 100644
--- a/drivers/media/platform/exynos4-is/mipi-
nez Canillas <jav...@osg.samsung.com>
Tested-by: Mauro Carvalho Chehab <mche...@osg.samsung.com>
Tested on my Chromebook snow revision 5, with the following its file:
/dts-v1/;
/ {
description = "Chrome OS kernel image with one or more FDT blobs";
#address-cells = &
ing thing is to add the dependency of EXYNOS_IOMMU
to the MFC driver.
Regards,
Mauro
>
> On 29/09/15 16:01, Mauro Carvalho Chehab wrote:
> > Em Tue, 29 Sep 2015 15:37:51 +0200
> > Sylwester Nawrocki <s.nawro...@samsung.com> escreveu:
> >
> >> Hi Mauro,
&g
Em Mon, 21 Sep 2015 11:59:27 +0200
Andrzej Pietrasiewicz escreveu:
> Hi Hans,
>
> W dniu 21.09.2015 o 11:50, Hans Verkuil pisze:
> > On 18-09-15 16:21, Andrzej Pietrasiewicz wrote:
> >> From: Marek Szyprowski
> >>
> >> Add Exynos 5433 jpeg h/w
Em Fri, 11 Sep 2015 09:31:36 +0200
Javier Martinez Canillas escreveu:
> Hello Sakari,
>
> On 09/11/2015 07:51 AM, Sakari Ailus wrote:
> > Hi Javier,
> >
> > Javier Martinez Canillas wrote:
> >> Hello Sakari,
> >>
> >> On 09/10/2015 07:14 PM, Sakari Ailus wrote:
> >>> Hi
Em Sun, 30 Aug 2015 00:06:11 -0300
Mauro Carvalho Chehab mche...@osg.samsung.com escreveu:
That's the 8th version of the MC next generation patches.
Differences from version 7:
- Patches reworked to make the reviewers happy;
- Bug fixes;
- ALSA changes got their own separate patches
and FB/DRM
too.
Ok, on the current pipelines supported by those drivers, just V4L
stuff are there, but, assuming that some day a pipeline that also
works with other subsystems will ever added, it is better to add
explicit checks for the AV_DMA stuff.
Signed-off-by: Mauro Carvalho Chehab mche
this shell script:
for i in $(find drivers/media -name '*.[ch]' -type f) $(find
drivers/staging/media -name '*.[ch]' -type f) $(find include/ -name '*.h' -type
f) ; do sed s,media_entity_create_link,media_create_pad_link,g $i a mv a
$i; done
Signed-off-by: Mauro Carvalho Chehab mche
:
@@
struct media_entity *me;
@@
- me-parent
+ me-graph_obj.mdev
@@
struct media_entity *link;
@@
- link-source-entity-parent
+ link-source-entity-graph_obj.mdev
@@
struct exynos_video_entity *ve;
@@
- ve-vdev.entity.parent
+ ve-vdev.entity.graph_obj.mdev
Suggested-by: Mauro Carvalho Chehab mche
[media] omap3isp: separate links creation from entities init
[media] omap3isp: create links after all subdevs have been bound
Mauro Carvalho Chehab (49):
[media] media: create a macro to get entity ID
[media] media: add a common struct to be embed on media graph objects
[media] media: use
Em Wed, 26 Aug 2015 11:54:03 -0300
Mauro Carvalho Chehab mche...@osg.samsung.com escreveu:
Em Tue, 25 Aug 2015 12:55:41 -0600
Shuah Khan shua...@osg.samsung.com escreveu:
On 08/23/2015 02:17 PM, Mauro Carvalho Chehab wrote:
Now that a link can be either between two different graph
Em Tue, 25 Aug 2015 13:25:15 -0600
Shuah Khan shuahk...@gmail.com escreveu:
On Tue, Aug 25, 2015 at 12:36 AM, Hans Verkuil hverk...@xs4all.nl wrote:
On 08/23/2015 10:17 PM, Mauro Carvalho Chehab wrote:
From: Javier Martinez Canillas jav...@osg.samsung.com
The struct media_entity has
Em Tue, 25 Aug 2015 12:55:41 -0600
Shuah Khan shua...@osg.samsung.com escreveu:
On 08/23/2015 02:17 PM, Mauro Carvalho Chehab wrote:
Now that a link can be either between two different graph
objects, we'll need to add more functions to create links.
Is this an incomplete sentence. Should
and FB/DRM
too.
Ok, on the current pipelines supported by those drivers, just V4L
stuff are there, but, assuming that some day a pipeline that also
works with other subsystems will ever added, it is better to add
explicit checks for the AV_DMA stuff.
Signed-off-by: Mauro Carvalho Chehab mche
in $(find drivers/media -name '*.[ch]' -type f) $(find
drivers/staging/media -name '*.[ch]' -type f) $(find include/ -name '*.h' -type
f) ; do sed s,media_entity_create_link,media_create_pad_link,g $i a mv a
$i; done
Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com
Acked-by: Hans
:
@@
struct media_entity *me;
@@
- me-parent
+ me-graph_obj.mdev
@@
struct media_entity *link;
@@
- link-source-entity-parent
+ link-source-entity-graph_obj.mdev
@@
struct exynos_video_entity *ve;
@@
- ve-vdev.entity.parent
+ ve-vdev.entity.graph_obj.mdev
Suggested-by: Mauro Carvalho Chehab mche
Hi Javier,
Em Wed, 19 Aug 2015 17:35:18 +0200
Javier Martinez Canillas jav...@osg.samsung.com escreveu:
Hello,
This series contains a couple of build fixes and cleanups for the
Media Controller framework. The goal of the series is to get rid of
the struct media_entity .parent member since
in $(find drivers/media -name '*.[ch]' -type f) $(find
drivers/staging/media -name '*.[ch]' -type f) $(find include/ -name '*.h' -type
f) ; do sed s,media_entity_create_link,media_create_pad_link,g $i a mv a
$i; done
Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com
Acked-by: Hans
in $(find drivers/media -name '*.[ch]' -type f) $(find
drivers/staging/media -name '*.[ch]' -type f) $(find include/ -name '*.h' -type
f) ; do sed s,media_entity_create_link,media_create_pad_link,g $i a mv a
$i; done
Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com
diff --git
Em Fri, 14 Aug 2015 13:33:48 +0300
Sakari Ailus sakari.ai...@iki.fi escreveu:
Hi Mauro,
On Wed, Aug 12, 2015 at 05:14:51PM -0300, Mauro Carvalho Chehab wrote:
Currently, media_entity_init() creates an array with the links,
allocated at init time. It provides a parameter (extra_links
to change the implementation of the links. So,
before doing that, let's first remove that extra unused
parameter, in order to cleanup the interface first.
Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com
Acked-by: Sakari Ailus sakari.ai...@linux.intel.com
diff --git a/Documentation/media
to change the implementation of the links. So,
before doing that, let's first remove that extra unused
parameter, in order to cleanup the interface first.
Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com
diff --git a/Documentation/media-framework.txt
b/Documentation/media-framework.txt
index
in $(find drivers/media -name '*.[ch]' -type f) $(find
drivers/staging/media -name '*.[ch]' -type f) $(find include/ -name '*.h' -type
f) ; do sed s,media_entity_create_link,media_create_pad_link,g $i a mv a
$i; done
Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com
diff --git
f) ; do sed s,link-sink,link-pad1_sink,g;
s,link-source,link-pad0_source,g; $i a mv a $i; done
Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com
diff --git a/drivers/media/dvb-core/dvb_frontend.c
b/drivers/media/dvb-core/dvb_frontend.c
index 3c97ebdf9f2a..e673f6f7c398 100644
Em Tue, 11 Aug 2015 14:25:18 +0200
Hans Verkuil hansv...@cisco.com escreveu:
Hi Mauro,
Thanks for posting the missing patches.
Thanks for reviewing this patch series!
On 08/11/15 14:09, Mauro Carvalho Chehab wrote:
As we'll need to create links between entities and interfaces,
we
introduce links between entities and interfaces,
we may need to change some existing code to work with links
that aren't pad to pad.
Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com
diff --git a/drivers/media/dvb-core/dvb_frontend.c
b/drivers/media/dvb-core/dvb_frontend.c
index
in $(find drivers/media -name '*.[ch]' -type f) $(find
drivers/staging/media -name '*.[ch]' -type f) $(find include/ -name '*.h' -type
f) ; do sed s,media_entity_create_link,media_create_pad_link,g $i a mv a
$i; done
Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com
diff --git
to change the implementation of the links. So,
before doing that, let's first remove that extra unused
parameter, in order to cleanup the interface first.
Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com
diff --git a/Documentation/media-framework.txt
b/Documentation/media-framework.txt
index
-off-by: Jan Kara j...@suse.cz
Signed-off-by: Hans Verkuil hans.verk...@cisco.com
Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com
diff --git a/drivers/gpu/drm/exynos/exynos_drm_g2d.c
b/drivers/gpu/drm/exynos/exynos_drm_g2d.c
index 81a250830808..810e1ee7c07d 100644
--- a/drivers/gpu/drm
Signed-off-by: Hans Verkuil hans.verk...@cisco.com
Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com
create mode 100644 mm/frame_vector.c
diff --git a/drivers/gpu/drm/exynos/Kconfig b/drivers/gpu/drm/exynos/Kconfig
index 0a6780367d28..fc678289cf79 100644
--- a/drivers/gpu/drm/exynos
As reported by smatch:
drivers/media/platform/s3c-camif/camif-capture.c:463 queue_setup()
warn: variable dereferenced before check 'fmt' (see line 460)
Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com
diff --git a/drivers/media/platform/s3c-camif/camif-capture.c
b/drivers
on CONFIG_PM_RUNTIME may now be changed to depend on
CONFIG_PM.
The alternative of CONFIG_PM_SLEEP and CONFIG_PM_RUNTIME may be
replaced with CONFIG_PM too.
Make these changes everywhere under drivers/media/.
Signed-off-by: Rafael J. Wysocki rafael.j.wyso...@intel.com
Acked-by: Mauro Carvalho Chehab
be used uninitialized
Reported-by: Prabhakar Lad prabhakar.cse...@gmail.com
Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com
diff --git a/drivers/media/platform/exynos-gsc/gsc-core.c
b/drivers/media/platform/exynos-gsc/gsc-core.c
index 91d226b8fe5c..3062e9fac6da 100644
--- a/drivers
-by: Mauro Carvalho Chehab mche...@osg.samsung.com
diff --git a/drivers/media/platform/s3c-camif/camif-capture.c
b/drivers/media/platform/s3c-camif/camif-capture.c
index f33641384e15..4f81b4c9d113 100644
--- a/drivers/media/platform/s3c-camif/camif-capture.c
+++ b/drivers/media/platform/s3c-camif/camif
of macro
‘pr_debug’
pr_debug(shared region: %#x, parameter region: %#x\n,
^
Signed-off-by: Mauro Carvalho Chehab mche...@osg.samsung.com
diff --git a/drivers/media/platform/exynos4-is/fimc-is.c
b/drivers/media/platform/exynos4-is/fimc-is.c
index 5476dce3ad29..22162b2567da 100644
--- a/drivers
ERROR: __bad_ndelay [drivers/media/platform/s5p-jpeg/s5p-jpeg.ko] undefined!
Yet, it sounds a bad idea to use ndelay to wait for 100 us
for the device to reset.
Reported-by: Stephen Rothwell s...@canb.auug.org.au
Signed-off-by: Mauro Carvalho Chehab m.che...@samsung.com
diff --git a/drivers
Em Tue, 09 Sep 2014 16:58:58 +0200
Sylwester Nawrocki s.nawro...@samsung.com escreveu:
On 09/09/14 16:38, Mauro Carvalho Chehab wrote:
ERROR: __bad_ndelay [drivers/media/platform/s5p-jpeg/s5p-jpeg.ko]
undefined!
Yet, it sounds a bad idea to use ndelay to wait for 100 us
() instead.
Reported-by: Stephen Rothwell s...@canb.auug.org.au
Signed-off-by: Mauro Carvalho Chehab m.che...@samsung.com
diff --git a/drivers/media/platform/s5p-jpeg/jpeg-hw-exynos4.c
b/drivers/media/platform/s5p-jpeg/jpeg-hw-exynos4.c
index e51c078360f5..ab6d6f43c96f 100644
--- a/drivers/media
Em Tue, 09 Sep 2014 19:54:19 +0200
Arnd Bergmann a...@arndb.de escreveu:
On Tuesday 09 September 2014 12:09:36 Mauro Carvalho Chehab wrote:
-exynos4.c
index e51c078360f5..01eeacf28843 100644
--- a/drivers/media/platform/s5p-jpeg/jpeg-hw-exynos4.c
+++ b/drivers/media/platform/s5p
() instead.
Reported-by: Stephen Rothwell s...@canb.auug.org.au
Acked-by: Sylwester Nawrocki s.nawro...@samsung.com
Signed-off-by: Mauro Carvalho Chehab m.che...@samsung.com
diff --git a/drivers/media/platform/s5p-jpeg/jpeg-hw-exynos4.c
b/drivers/media/platform/s5p-jpeg/jpeg-hw-exynos4.c
index
Instead of allocating a var to store 0 and just return it,
change the code to return 0 directly.
Signed-off-by: Mauro Carvalho Chehab m.che...@samsung.com
diff --git a/drivers/media/platform/exynos4-is/fimc-isp-video.c
b/drivers/media/platform/exynos4-is/fimc-isp-video.c
index 93f9cf2ebcd6
Em Thu, 17 Oct 2013 20:06:49 +0200
Sylwester Nawrocki s.nawro...@samsung.com escreveu:
This patch adds clock provider to expose the sclk_cam0/1 clocks
for external image sensor devices.
Signed-off-by: Sylwester Nawrocki s.nawro...@samsung.com
Signed-off-by: Kyungmin Park
Em Fri, 4 Oct 2013 17:56:31 +0530
Shaik Ameer Basha shaik.am...@samsung.com escreveu:
This patch adds support for SCALER device which is a new device
for scaling, blending, color fill and color space conversion
on EXYNOS5410 and EXYNOS5420 SoCs.
This device supports the followings as key
Em Fri, 4 Oct 2013 17:56:32 +0530
Shaik Ameer Basha shaik.am...@samsung.com escreveu:
This patch adds the core functionality for the SCALER driver.
Signed-off-by: Shaik Ameer Basha shaik.am...@samsung.com
---
drivers/media/platform/exynos-scaler/scaler.c | 1238
+
Em Fri, 18 Oct 2013 11:19:24 +0530
Arun Kumar K arun...@samsung.com escreveu:
From: Shaik Ameer Basha shaik.am...@samsung.com
This patch adds the Makefile and memory to memory (m2m) interface
functionality for the SCALER driver.
Signed-off-by: Shaik Ameer Basha shaik.am...@samsung.com
) seems ok,
as, if this function is called, the caller would be expecting it to not
fail.
Either way:
Reviewed-by: Mauro Carvalho Chehab m.che...@samsung.com
+ ptr = NULL;
+ if (!ptr)
goto out_unlock;
dmabuf-vmap_ptr = ptr;
diff --git
, right?
If so:
Acked-by: Mauro Carvalho Chehab m.che...@samsung.com
Regards,
Hans
---
drivers/staging/media/dt3155v4l/dt3155v4l.c |5 +
1 files changed, 1 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/media/dt3155v4l/dt3155v4l.c
b/drivers/staging
Em Sat, 19 Oct 2013 18:07:57 +0200
Ricardo Ribalda ricardo.riba...@gmail.com escreveu:
vb2_fop_relase does not held the lock although it is modifying the
queue-owner field.
This could lead to race conditions on the vb2_perform_io function
when multiple applications are accessing the video
Nawrocki s.nawro...@samsung.com
Signed-off-by: Wolfram Sang w...@the-dreams.de
Acked-by: Mauro Carvalho Chehab m.che...@samsung.com
---
V2-V3: Was trying to be too smart by only fixing includes needed.
Took a more general approach this time, converting of_i2c.h
to i2c.h in case i2c.h
.
Signed-off-by: Arnd Bergmann a...@arndb.de
Cc: linux-me...@vger.kernel.org
Cc: Mauro Carvalho Chehab mche...@redhat.com
Acked-by: Mauro Carvalho Chehab mche...@redhat.com
---
drivers/media/platform/exynos-gsc/gsc-regs.c | 1 -
drivers/media/platform/s5p-tv/sii9234_drv.c | 3 ---
2 files
Hi all,
Today, I finally merged the DMABUF V4L2 patches from Tomasz.
The DMABUF allows replacing the old V4L2 Overlay method by something more
robust and safer.
It was a long road to get them ready for their upstream inclusion, and to
be able to test on both embedded and personal computers.
Em 08-09-2011 03:48, Sylwester Nawrocki escreveu:
On 09/06/2011 10:52 PM, Mauro Carvalho Chehab wrote:
Em 03-09-2011 13:32, Sylwester Nawrocki escreveu:
On 09/01/2011 05:30 PM, Sylwester Nawrocki wrote:
Hello,
following is a fourth version of the patchset converting s5p-fimc driver
Em 03-09-2011 13:32, Sylwester Nawrocki escreveu:
On 09/01/2011 05:30 PM, Sylwester Nawrocki wrote:
Hello,
following is a fourth version of the patchset converting s5p-fimc driver
to the media controller API and the new control framework.
Mauro, could you please have a look at the patches
-schedule.txt
$
Mauro.
Cc: Sylwester Nawrocki s.nawro...@samsung.com
Cc: Kyungmin Park kyungmin.p...@samsung.com
Cc: Mauro Carvalho Chehab mche...@infradead.org
Signed-off-by: Kukjin Kim kgene@samsung.com
---
drivers/media/video/s5p-fimc/fimc-core.c | 14 --
1 files changed
Em 02-08-2010 07:52, Russell King - ARM Linux escreveu:
On Mon, Aug 02, 2010 at 12:32:20PM +0200, Pawel Osciak wrote:
Well, some of them are indeed unused, but it's not an uncommon practice in
kernel and might help future developers.
On the other hand, arch/arm is getting soo big that we
Hiremath, Vaibhav wrote:
-Original Message-
From: Pawel Osciak [mailto:p.osc...@samsung.com]
Sent: Monday, December 28, 2009 8:19 PM
To: 'Hans Verkuil'
Cc: linux-me...@vger.kernel.org; linux-samsung-soc@vger.kernel.org;
linux-arm-ker...@lists.infradead.org; Marek Szyprowski;
70 matches
Mail list logo