[ 30.628929] mmap_region+0xe85/0x1cd0
Lock the queue before we start finding the matching plane and buffer so
that there is no chance of the memory being freed while we are about
to use it.
Reported-by: syzbot+be93025dd45dccd89...@syzkaller.appspotmail.com
Signed-off-by: Sudip Mukherjee
From: Sudip Mukherjee
Modify lirc_parallel driver to use the new parallel port device model.
Signed-off-by: Sudip Mukherjee
---
Resending after more than one year.
Prevoius patch is at https://patchwork.kernel.org/patch/7883591/
drivers/staging/media/lirc/lirc_parallel.c | 93
If dvb_attach() fails then we were just printing an error message and
exiting but the memory allocated to state was not released.
Signed-off-by: Sudip Mukherjee
---
drivers/media/pci/bt8xx/dvb-bt8xx.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/media/pci/bt8xx/dvb-bt8xx.c
b
On Sunday 03 July 2016 09:12 AM, kbuild test robot wrote:
Hi,
It's probably a bug fix that unveils the link errors.
Yes, and more is yet to come. :)
Regards
Sudip
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@
io/imu/inv_mpu6050/inv-mpu6050.ko] undefined!
same reply as above.
ERROR: "bad_dma_ops" [drivers/hwtracing/intel_th/intel_th_msu.ko] undefined!
ERROR: "bad_dma_ops" [drivers/hwtracing/intel_th/intel_th.ko] undefined!
ERROR: "bad_dma_ops" [drivers/fpga/zynq-f
On Monday 06 June 2016 09:37 AM, Sudip Mukherjee wrote:
On Thu, Jan 10, 2002 at 12:50:58AM +0800, kbuild test robot wrote:
Hi,
It's probably a bug fix that unveils the link errors.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
We have assigned memory while requesting the firmware but if the sanity
check fails then we are not releasing the firmware.
Signed-off-by: Sudip Mukherjee
---
drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/media/platform/sti
On Monday 25 January 2016 10:42 PM, Mauro Carvalho Chehab wrote:
Em Mon, 25 Jan 2016 22:32:31 +0530
Sudip Mukherjee escreveu:
On Mon, Jan 25, 2016 at 02:29:06PM -0200, Mauro Carvalho Chehab wrote:
Em Fri, 18 Dec 2015 18:35:29 +0530
Sudip Mukherjee escreveu:
Modify lirc_parallel driver to
On the particular case when the product id is 0x2101 we have requested
for a firmware but after processing it we missed releasing it.
Signed-off-by: Sudip Mukherjee
---
drivers/media/usb/dvb-usb/dw2102.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/media/usb/dvb-usb/dw2102.c
When we returned on error we missed freeing p_current_fw and p_buffer.
Signed-off-by: Sudip Mukherjee
---
drivers/media/usb/cx231xx/cx231xx-417.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers/media/usb/cx231xx/cx231xx-417.c
b/drivers/media/usb/cx231xx/cx231xx-417.c
index
utines")
Signed-off-by: Sudip Mukherjee
---
build logs at:
https://travis-ci.org/sudipm-mukherjee/parport/jobs/113601228
and
https://travis-ci.org/sudipm-mukherjee/parport/jobs/113601203
include/media/v4l2-mc.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/inclu
On Mon, Jan 25, 2016 at 03:12:57PM -0200, Mauro Carvalho Chehab wrote:
> Em Mon, 25 Jan 2016 22:32:31 +0530
> Sudip Mukherjee escreveu:
>
> > On Mon, Jan 25, 2016 at 02:29:06PM -0200, Mauro Carvalho Chehab wrote:
> > > Em Fri, 18 Dec 2015 18:35:29 +0530
> &g
the symbol VIDEO_STI_BDISP also
selects VIDEOBUF2_DMA_CONTIG, so it is trying to compile
videobuf2-dma-contig.c even though HAS_DMA is not defined.
Signed-off-by: Sudip Mukherjee
---
drivers/media/platform/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/media/platform/Kconfig
On Fri, Dec 18, 2015 at 06:35:25PM +0530, Sudip Mukherjee wrote:
> A NULL comparison can be written as if (var) or if (!var).
> Reported by checkpatch.
>
> Signed-off-by: Sudip Mukherjee
> ---
Hi Mauro,
A gentle ping.
Can this series be considered for 4.5?
regards
sudip
--
A NULL comparison can be written as if (var) or if (!var).
Reported by checkpatch.
Signed-off-by: Sudip Mukherjee
---
drivers/staging/media/lirc/lirc_parallel.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/media/lirc/lirc_parallel.c
b/drivers/staging
checkpatch complains about missing space around operators.
Signed-off-by: Sudip Mukherjee
---
drivers/staging/media/lirc/lirc_parallel.c | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/media/lirc/lirc_parallel.c
b/drivers/staging
checkpatch complains about space after type cast.
Signed-off-by: Sudip Mukherjee
---
drivers/staging/media/lirc/lirc_parallel.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/media/lirc/lirc_parallel.c
b/drivers/staging/media/lirc/lirc_parallel.c
checkpatch complains about the logical operator, which should be on the
previous line.
Signed-off-by: Sudip Mukherjee
---
drivers/staging/media/lirc/lirc_parallel.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/media/lirc/lirc_parallel.c
b
Modify lirc_parallel driver to use the new parallel port device model.
Signed-off-by: Sudip Mukherjee
---
drivers/staging/media/lirc/lirc_parallel.c | 100 +++--
1 file changed, 65 insertions(+), 35 deletions(-)
diff --git a/drivers/staging/media/lirc/lirc_parallel.c
b
| 10 +++---
> drivers/staging/android/sync_debug.c |2 ++
> include/linux/fence.h|1 +
> 4 files changed, 23 insertions(+), 3 deletions(-)
Usually staging patches can not touch anything outside staging.
regards
sudip
--
To unsubscribe from this list: sen
loop also as it is initially 0 and whenever that value
changes we have done a break from the loop.
Signed-off-by: Sudip Mukherjee
---
drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/media/platform/sti/c8sectpfe
On Sat, Aug 08, 2015 at 01:55:02AM -0500, Junsu Shin wrote:
>
> On 08/06/2015 11:45 PM, Sudip Mukherjee wrote:
> > On Thu, Aug 06, 2015 at 09:55:54PM -0500, Junsu Shin wrote:
> >
>
> Thanks for pointing it out.
> Again, this is a patch to the dm365_ipipe.c th
ruct v4l2_subdev_frame_size_enum *fse)
since you are modifying this line, please fix up the indention also.
regards
sudip
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
ok.
regards
sudip
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
On Sat, Jul 18, 2015 at 07:57:44AM +0300, Adi Ratiu wrote:
> Signed-off-by: Adi Ratiu
> ---
Your patch seems to be corrupted (line wrapped?). And you have not
mentioned any commit message.
regards
sudip
--
To unsubscribe from this list: send the line "unsubscribe linux-media"
On Fri, Jun 19, 2015 at 02:22:02PM +0530, Sunil Shahu wrote:
> Fix code indentation error by replacing tab in place of spaces.
>
> Signed-off-by: Sunil Shahu
when you are sending a modified patch, please mark it as [PATCH v2]
otherwise it becomes confusing.
regards
sudip
--
To unsubsc
This variable was not being used anywhere.
Signed-off-by: Sudip Mukherjee
---
drivers/media/v4l2-core/videobuf2-core.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/media/v4l2-core/videobuf2-core.c
b/drivers/media/v4l2-core/videobuf2-core.c
index c11aee7..d3f7bf0 100644
--- a
Thanks for your feedback. I will improve my patch submission.
BR,
Sudip
From: Jeremiah Mahler [jmmah...@gmail.com]
Sent: Thursday, February 26, 2015 10:39 PM
To: Sudip JAIN
Cc: linux-media@vger.kernel.org; linux-ker...@vger.kernel.org
Subject: Re: 0001
Hello Jeremiah,
Please find the patch "inline"
commit 3390900680e5182998916c8fa231bc79cd84046b
Author: Sudip Jain
Date: Thu Feb 26 10:40:34 2015 +0530
media: vb2: Fill vb2_buffer with bytesused from user
In vb2_qbuf for dmabuf memory type, userside bytesused is n
Dear Maintainer,
PFA attached patch that prevents user from being returned garbage bytesused
value from vb2 framework.
Regards,
Sudip Jain
From 804e76f89d73d10b9fd7b25a48a6edc31faa40a9 Mon Sep 17 00:00:00 2001
From: Sudip Jain
Date: Wed, 25 Feb 2015 12:24:53 +0530
Subject: [PATCH] media: vb2
fail_mutex_unlock.
Signed-off-by: Sudip Mukherjee
---
drivers/media/platform/davinci/vpbe.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/media/platform/davinci/vpbe.c
b/drivers/media/platform/davinci/vpbe.c
index 49d2de0..e5df991 100644
--- a/drivers/media/platform/davinci/vpbe.c
+++ b
ter if you add version number in the subject.
like this is v2.
thanks
sudip
>
> Signed-off-by: Aya Mahfouz
> ---
> drivers/staging/media/lirc/lirc_zilog.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/media/lirc/lirc_zilog.c
On Tue, Nov 04, 2014 at 11:48:26PM +0200, Aya Mahfouz wrote:
> On Tue, Nov 04, 2014 at 03:06:53PM +0530, Sudip Mukherjee wrote:
> > On Tue, Nov 04, 2014 at 02:13:19AM +0200, Aya Mahfouz wrote:
> > > This patch replaces dev_err by pr_err since the value
> > > of ir
please try dev_err like this :
dev_err(&client->dev, "%s: probing IR %s on %s (i2c-%d) failed with %d\n",
__func__, tx_probe ? "Tx" : "Rx", adap->name, adap->nr,
ret);
thanks
sudip
> __func__, tx_probe
On Mon, Sep 08, 2014 at 09:55:58AM -0500, Morgan Phillips wrote:
> Change array initialization format to fix style error.
> from:
> u8 foo[] =
> {1, 2, 3};
> to:
checkpatch is giving a warning here , but looks like a false positive
On Mon, Sep 08, 2014 at 08:51:18AM -0500, Morgan Reece wrote:
> Hi Sudip,
>
> I searched through the logs for examples of messages where people had just
> fixed checkpatch errors. I found lots like this, so went the format, ex:
>
> commit 588a12d789e1a9b8193465c09f32024
On Mon, Sep 08, 2014 at 07:32:22AM -0500, Morgan Phillips wrote:
> Signed-off-by: Morgan Phillips
no commit message ?
thanks
sudip
> ---
> drivers/media/usb/gspca/sn9c20x.c | 10 ++
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/media/u
On Mon, Sep 08, 2014 at 10:07:45AM +0200, Hans Verkuil wrote:
> On 09/06/2014 01:24 PM, Sudip Mukherjee wrote:
> > On Fri, Aug 29, 2014 at 01:38:01PM +0530, Sudip Mukherjee wrote:
> >> with -Werror=implicit-function-declaration build failed with error :
> >> error: impli
On Fri, Aug 29, 2014 at 01:38:01PM +0530, Sudip Mukherjee wrote:
> with -Werror=implicit-function-declaration build failed with error :
> error: implicit declaration of function 'inb'
> error: implicit declaration of function 'outb'
>
> Reported-by: Jim Davi
with -Werror=implicit-function-declaration build failed with error :
error: implicit declaration of function 'inb'
error: implicit declaration of function 'outb'
Reported-by: Jim Davis
Signed-off-by: Sudip Mukherjee
---
Jim reported for next-20140828 , but the error st
e[1]: *** [drivers/media] Error 2
Hi,
Can you please try the attached patch , for me it solved the error/
thanks
sudip
diff --git a/drivers/media/radio/radio-miropcm20.c b/drivers/media/radio/radio-miropcm20.c
index 998919e..3309f7c 100644
--- a/drivers/media/radio/radio-miropcm20.c
+++ b/drivers/
On Wed, Aug 27, 2014 at 10:33:46AM -0700, Jim Davis wrote:
> On Wed, Aug 27, 2014 at 3:58 AM, Sudip Mukherjee
> wrote:
>
> > Hi,
> > I tried to build next-20140826 with your given config file . But for me
> > everything was fine.
>
> Well, you should be able
linux/v4l2-pci-skeleton.ko] undefined!
> ERROR: "v4l2_ctrl_handler_free"
> [Documentation/video4linux/v4l2-pci-skeleton.ko] undefined!
> ERROR: "video_unregister_device"
> [Documentation/video4linux/v4l2-pci-skeleton.ko] undefined!
> make[1]: *** [__modpost] Error 1
> make: *** [modules] Error 2
>
> Similar to a build error from January 7th:
> https://lists.01.org/pipermail/kbuild-all/2014-January/002566.html
Hi,
I tried to build next-20140826 with your given config file . But for me
everything was fine.
And I was just wondering why the skeleton code in the Documentation is building
?
thanks
sudip
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
43 matches
Mail list logo
