)
[] (v4l2_device_unregister_subdev) from []
(v4l2_device_unregister+0x64/0x94)
[] (v4l2_device_unregister) from []
(fimc_md_probe+0x4ec/0xaf8 [s5p_fimc])
[...]
Signed-off-by: Seung-Woo Kim
---
drivers/media/platform/exynos4-is/media-dev.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a
ssing lock held on
vb2_fop_release")
Signed-off-by: Seung-Woo Kim
---
drivers/media/platform/exynos4-is/fimc-isp-video.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/media/platform/exynos4-is/fimc-isp-video.c
b/drivers/media/platform/exynos4-is/fimc-is
from queue_setup.
Signed-off-by: Seung-Woo Kim
---
drivers/media/platform/s5p-mfc/s5p_mfc_enc.c |9 +
1 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c
b/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c
index e65993f..2e57e9f
ck);
>>>> INIT_LIST_HEAD(&dmabuf->attachments);
>>>> + INIT_LIST_HEAD(&dmabuf->metas);
>>>>
>>> I am not sure I understand the relationship of 'meta data' with
>>> 'dma-buf', or 'attachments
This patch removes meaningless assignment of memory bank to itself.
Signed-off-by: Seung-Woo Kim
---
change from v2
- simplify description and change typo in subject
change from v1
- fixes subject and adds proper description
---
drivers/media/platform/s5p-mfc/s5p_mfc_ctrl.c |2 --
1 files
Hello Sachin,
On 2014년 03월 06일 12:28, Sachin Kamat wrote:
> Hi Seung-Woo,
>
> On 6 March 2014 07:43, Seung-Woo Kim wrote:
>> Hello Sachin,
>>
>> On 2014년 03월 05일 20:42, Sachin Kamat wrote:
>>> On 5 March 2014 16:38, Seung-Woo Kim wrote:
>>
>&g
Hello Sachin,
On 2014년 03월 05일 20:42, Sachin Kamat wrote:
> On 5 March 2014 16:38, Seung-Woo Kim wrote:
(...)
>> - dev->bank1 = dev->bank1;
>
> Are you sure this isn't some kind of typo? If not then your commit
> description is too verbose
> to actuall
There was assignment of memory bank with dma address converted
from physical address. But allocation has been changed with dma
function, so the assignment is not necessary.
Signed-off-by: Seung-Woo Kim
---
change from v1
- fixes subject and adds proper description
---
drivers/media/platform/s5p
Signed-off-by: Seung-Woo Kim
---
drivers/media/platform/s5p-mfc/s5p_mfc_ctrl.c |2 --
1 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc_ctrl.c
b/drivers/media/platform/s5p-mfc/s5p_mfc_ctrl.c
index 2475a3c..ee05f2d 100644
--- a/drivers
rmat, and usually *it is not aligned with
PAGE_SIZE* even though internal allocation is aligned. So the check
routine of size in __qbuf_dmabuf() returns failure.
I am not sure about queue_setup of UVC driver, but it seems same issue.
Maybe we can add PAGE_ALIGN for size from queue_setup in the
__
__qbuf_dmabuf checks whether size of provided dmabuf is large
enough, and it returns error without any log. So this patch adds
error log in the case.
Signed-off-by: Seung-Woo Kim
---
drivers/media/v4l2-core/videobuf2-core.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git
zed in this function
drivers/media/platform/s5p-jpeg/jpeg-core.c:433: warning: 'height' may be used
uninitialized in this function
drivers/media/platform/s5p-jpeg/jpeg-core.c:433: warning: 'width' may be used
uninitialized in this function
Signed-off-by: Seung-Woo Kim
---
chan
Hi Sylwester,
Thanks for you comment.
On 2013년 10월 12일 18:39, Sylwester Nawrocki wrote:
> Hi Seung-Woo,
>
> On 10/10/2013 09:06 AM, Seung-Woo Kim wrote:
>> For hdr parse error, it can return false without any assignments
>> which cause build warning.
>>
>
It is hard to distinguish between decoder and encoder video device
because their names are same. So this patch fixes the names.
Signed-off-by: Seung-Woo Kim
---
drivers/media/platform/s5p-jpeg/jpeg-core.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers
For hdr parse error, it can return false without any assignments
which cause build warning.
Signed-off-by: Seung-Woo Kim
---
drivers/media/platform/s5p-jpeg/jpeg-core.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/platform/s5p-jpeg/jpeg-core.c
b
__qbuf_userptr checks whether provided buffer is large enough, and
it returns error without any log.
Signed-off-by: Seung-Woo Kim
Signed-off-by: Heejin Woo
---
drivers/media/v4l2-core/videobuf2-core.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/drivers/media
used at reimport time if it is assigned
with drm gem object at first import.
This can also remove remapping dma address for the hardware having
its own iommu.
Signed-off-by: Seung-Woo Kim
---
drivers/gpu/drm/drm_prime.c| 19 ++-
drivers/gpu/drm/exynos
-by: Seung-Woo Kim
---
drivers/base/dma-buf.c | 31 +++
include/linux/dma-buf.h |4
2 files changed, 35 insertions(+), 0 deletions(-)
diff --git a/drivers/base/dma-buf.c b/drivers/base/dma-buf.c
index 08fe897..a1eaaf2 100644
--- a/drivers/base/dma-buf.c
importer private data in dma-buf attachment can be used by importer to
reimport same dma-buf.
Seung-Woo Kim (2):
dma-buf: add importer private data to attachment
drm/prime: find gem object from the reimported dma-buf
drivers/base/dma-buf.c | 31
The v4l2_m2m_poll() does not need to wait if there is already a buffer in
done_list of source and destination queues, but current v4l2_m2m_poll() always
waits. So done_list of each queue is checked before calling poll_wait().
Signed-off-by: Seung-Woo Kim
Acked-by: Marek Szyprowski
---
drivers
The vb2_poll() does not need to wait next vb_buffer_done() if there is already
a buffer in done_list of queue, but current vb2_poll() always waits.
So done_list is checked before calling poll_wait().
Signed-off-by: Seung-Woo Kim
Acked-by: Marek Szyprowski
---
drivers/media/v4l2-core/videobuf2
() or queue is cancelled.
So I add check routine for done_list before calling poll_wait().
But I'm not sure that locking for done_lock of queue is also needed in this
case or not because done_list of queue is checked without locking in some
other parts of vb2.
Seung-Woo Kim (2):
media: vb2: r
The length of mmap() can be bigger than length of vb2 buffer, so
it should be checked.
Signed-off-by: Seung-Woo Kim
---
drivers/media/v4l2-core/videobuf2-core.c |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/drivers/media/v4l2-core/videobuf2-core.c
b/drivers/media
The vb2_poll() does not need to wait next vb_buffer_done() if there is already
a buffer in done_list of queue, but current vb2_poll() always waits.
So done_list is checked before calling poll_wait().
Signed-off-by: Seung-Woo Kim
---
drivers/media/v4l2-core/videobuf2-core.c |3 ++-
1 files
The v4l2_m2m_poll() does not need to wait if there is already a buffer in
done_list of source and destination queues, but current v4l2_m2m_poll() always
waits. So done_list of each queue is checked before calling poll_wait().
Signed-off-by: Seung-Woo Kim
---
drivers/media/v4l2-core/v4l2
() or queue is cancelled.
So I add check routine for done_list before calling poll_wait().
But I'm not sure that locking for done_lock of queue is also needed in this
case or not because done_list of queue is checked without locking in some
other parts of vb2.
Seung-Woo Kim (2):
media: vb2: r
26 matches
Mail list logo