[PATCH] bdisp: Clean up file handle in open() error path.

2016-12-01 Thread Shailendra Verma
The File handle is not yet added in the vdev list.So no need to call v4l2_fh_del(>fh)if it fails to create control. Signed-off-by: Shailendra Verma --- drivers/media/platform/sti/bdisp/bdisp-v4l2.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] exynos-gsc: Clean up file handle in open() error path.

2016-12-01 Thread Shailendra Verma
The File handle is not yet added in the vfd list.So no need to call v4l2_fh_del(>fh) if it fails to create control. Signed-off-by: Shailendra Verma --- drivers/media/platform/exynos-gsc/gsc-m2m.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] exynos4-is: Clean up file handle in open() error path.

2016-12-01 Thread Shailendra Verma
The File handle is not yet added in the vfd list.So no need to call v4l2_fh_del(>fh) if it fails to create control. Signed-off-by: Shailendra Verma --- drivers/media/platform/exynos4-is/fimc-m2m.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v1] mtk-vcodec: use V4L2_DEC_CMD_STOP to implement flush

2016-12-01 Thread Wu-Cheng Li
From: Tiffany Lin Also remove the code using size-0 OUTPUT buffer to flush. Singed-off-by: Tiffany Lin Signed-off-by: Wu-Cheng Li Reviewed-by: Kuang-che Wu ---

[PATCH v1] mtk-vcodec: use V4L2_DEC_CMD_STOP to implement flush

2016-12-01 Thread Wu-Cheng Li
From: Wu-Cheng Li This patch uses V4L2_DEC_CMD_STOP to implement flush -- requesting the remaining images to be returned to userspace. The old unofficial way was to use a size-0 input buffer and the code is removed. Tiffany Lin (1): mtk-vcodec: use V4L2_DEC_CMD_STOP to

Re: [PATCH v3 0/4] stk1160: Let the driver setup the device's internal AC97 codec

2016-12-01 Thread Ezequiel Garcia
On 27 November 2016 at 08:07, Marcel Hasler wrote: > This patchset is a result of my attempt to fix a bug > (https://bugzilla.kernel.org/show_bug.cgi?id=180071) that eventually turned > out to be caused by a missing quirk in snd-usb-audio. My idea was to remove > the AC97

Re: [PATCH] Doc: Correct typo, "Introdution" => "Introduction"

2016-12-01 Thread Jonathan Corbet
On Thu, 1 Dec 2016 23:36:00 +0800 Sanjeev wrote: > This corrects a set of spelling mistakes, probably from an > automated conversion. That does indeed seem worth fixing. Applied to the docs tree, thanks. jon -- To unsubscribe from this list: send the line "unsubscribe

Re: [PATCH 2/2] media: protect enable and disable source handler checks and calls

2016-12-01 Thread Shuah Khan
Hi Sakari, On 12/01/2016 06:51 AM, Sakari Ailus wrote: > Hi Shuah, > > On Tue, Nov 29, 2016 at 10:41:51AM -0700, Shuah Khan wrote: >> On 11/29/2016 02:22 AM, Sakari Ailus wrote: >>> Hi Shuah, >>> >>> On Mon, Nov 28, 2016 at 07:15:14PM -0700, Shuah Khan wrote: Protect enable and disable

[PATCH] Doc: Correct typo, "Introdution" => "Introduction"

2016-12-01 Thread Sanjeev
This corrects a set of spelling mistakes, probably from an automated conversion. Signed-off-by: Sanjeev Gupta --- Documentation/admin-guide/unicode.rst | 4 ++-- Documentation/media/dvb-drivers/intro.rst | 4 ++-- Documentation/media/v4l-drivers/cafe_ccic.rst | 4

Re: [PATCH 29/39] Annotate hardware config module parameters in drivers/staging/media/

2016-12-01 Thread Mauro Carvalho Chehab
Em Thu, 01 Dec 2016 14:59:56 + David Howells escreveu: > Mauro Carvalho Chehab wrote: > > > drivers/staging/media/lirc/lirc_parallel.c:728:19: error: Expected ) in > > function declarator > > Did you apply patch 1 first? That defines

Re: [PATCH 29/39] Annotate hardware config module parameters in drivers/staging/media/

2016-12-01 Thread David Howells
Mauro Carvalho Chehab wrote: > drivers/staging/media/lirc/lirc_parallel.c:728:19: error: Expected ) in > function declarator Did you apply patch 1 first? That defines module_param_hw*. David -- To unsubscribe from this list: send the line "unsubscribe linux-media"

Re: [PATCH 29/39] Annotate hardware config module parameters in drivers/staging/media/

2016-12-01 Thread Mauro Carvalho Chehab
Em Thu, 01 Dec 2016 12:33:30 + David Howells escreveu: > When the kernel is running in secure boot mode, we lock down the kernel to > prevent userspace from modifying the running kernel image. Whilst this > includes prohibiting access to things like /dev/mem, it must

Re: [PATCH 2/2] media: protect enable and disable source handler checks and calls

2016-12-01 Thread Sakari Ailus
Hi Shuah, On Tue, Nov 29, 2016 at 10:41:51AM -0700, Shuah Khan wrote: > On 11/29/2016 02:22 AM, Sakari Ailus wrote: > > Hi Shuah, > > > > On Mon, Nov 28, 2016 at 07:15:14PM -0700, Shuah Khan wrote: > >> Protect enable and disable source handler checks and calls from dvb-core > >> and v4l2-core.

[PATCH 29/39] Annotate hardware config module parameters in drivers/staging/media/

2016-12-01 Thread David Howells
When the kernel is running in secure boot mode, we lock down the kernel to prevent userspace from modifying the running kernel image. Whilst this includes prohibiting access to things like /dev/mem, it must also prevent access by means of configuring driver modules in such a way as to cause a

[PATCH 13/39] Annotate hardware config module parameters in drivers/media/

2016-12-01 Thread David Howells
When the kernel is running in secure boot mode, we lock down the kernel to prevent userspace from modifying the running kernel image. Whilst this includes prohibiting access to things like /dev/mem, it must also prevent access by means of configuring driver modules in such a way as to cause a

Re: [PATCH 1/2] mn88473: fix chip id check on probe

2016-12-01 Thread Benjamin Larsson
On 2016-12-01 01:29, Antti Palosaari wrote: A register used to identify chip during probe was overwritten during firmware download and due to that later probe's for warm chip were failing. Detect chip from the another register, which is located on different register bank 2. Fixes: 7908fad99a6c

Re: [PATCH] Platform: vsp1: Clean up file handle in open() error path.

2016-12-01 Thread Laurent Pinchart
Hi Shailendra, Thank you for the patch. On Thursday 01 Dec 2016 10:20:18 Shailendra Verma wrote: > v4l2_fh_init is already done.So call the v4l2_fh_exit in error condition > before returing from the function. > > Signed-off-by: Shailendra Verma Reviewed-by: Laurent

Re: [PATCH] V4l: omap4iss: Clean up file handle in open() and release().

2016-12-01 Thread Laurent Pinchart
Hi Shailendra, Thank you for the patch. On Thursday 01 Dec 2016 10:22:52 Shailendra Verma wrote: > Both functions initialize the file handle with v4l2_fh_init() > and thus need to call clean up with v4l2_fh_exit() as appropriate. > > Signed-off-by: Shailendra Verma

Re: [PATCH] V4l: omap3isp: Clean up file handle in open() and release().

2016-12-01 Thread Laurent Pinchart
Hi Shailendra, Thank you for the patch. On Thursday 01 Dec 2016 10:15:40 Shailendra Verma wrote: > Both functions initialize the file handle with v4l2_fh_init() > and thus need to call clean up with v4l2_fh_exit() as appropriate. > > Signed-off-by: Shailendra Verma

Re: [RFCv2 PATCH 1/5] video: add HDMI state notifier support

2016-12-01 Thread Hans Verkuil
On 11/14/16 16:22, Hans Verkuil wrote: From: Hans Verkuil Add support for HDMI hotplug and EDID notifiers, which is used to convey information from HDMI drivers to their CEC and audio counterparts. I realized that the name 'HDMI notifier' isn't the best: the same

Re: [PATCH] Platform: Sti: Bdisp: Clean up file handle in open() error path.

2016-12-01 Thread Fabien DESSENNE
Hi Shailendra Thank you for the patch. Could you please update the subject line (try to have it as short as possible): - there is no need for the 'platform' keyword. This remark applies to your other patches. - write all keywords in lower case (Bdisp -> bdisp). This remark applies also to

Re: [PATCH v3 3/4] [media] davinci: vpif_capture: get subdevs from DT

2016-12-01 Thread Laurent Pinchart
Hello, On Thursday 01 Dec 2016 09:57:31 Sakari Ailus wrote: > On Wed, Nov 30, 2016 at 04:14:11PM -0800, Kevin Hilman wrote: > > Sakari Ailus writes: > >> On Wed, Nov 23, 2016 at 03:25:32PM -0800, Kevin Hilman wrote: > >>> Sakari Ailus writes: > On

[PATCH v4 5/9] media: venus: vdec: add video decoder files

2016-12-01 Thread Stanimir Varbanov
This consists of video decoder implementation plus decoder controls. Signed-off-by: Stanimir Varbanov --- drivers/media/platform/qcom/venus/vdec.c | 976 + drivers/media/platform/qcom/venus/vdec.h | 32 +

[PATCH v4 2/9] doc: DT: venus: binding document for Qualcomm video driver

2016-12-01 Thread Stanimir Varbanov
Add binding document for Venus video encoder/decoder driver Cc: Rob Herring Cc: Mark Rutland Cc: devicet...@vger.kernel.org Signed-off-by: Stanimir Varbanov --- Rob, I have removed vmem clocks, interrupts and reg

[PATCH v4 3/9] MAINTAINERS: Add Qualcomm Venus video accelerator driver

2016-12-01 Thread Stanimir Varbanov
Add an entry for Venus video encoder/decoder accelerator driver. Signed-off-by: Stanimir Varbanov --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 52cc0775a799..9528488f538e 100644 --- a/MAINTAINERS +++

[PATCH v4 6/9] media: venus: venc: add video encoder files

2016-12-01 Thread Stanimir Varbanov
This adds encoder part of the driver plus encoder controls. Signed-off-by: Stanimir Varbanov --- drivers/media/platform/qcom/venus/venc.c | 1099 drivers/media/platform/qcom/venus/venc.h | 32 +

[PATCH v4 7/9] media: venus: hfi: add Host Firmware Interface (HFI)

2016-12-01 Thread Stanimir Varbanov
This is the implementation of HFI. It is charged with the responsibility to comunicate with the firmware through an interface commands and messages. - hfi.c has interface functions used by the core, decoder and encoder parts to comunicate with the firmware. For example there are functions for

[PATCH v4 0/9] Qualcomm video decoder/encoder driver

2016-12-01 Thread Stanimir Varbanov
Hi, Here is version 4 of the Venus video driver. The changes since v3 are: * the driver now use m2m APIs. In order to make possible to use m2m I had to extend m2m APIs with few more helper functions for buffer list manipulations. * addressed comments from Hans about buffer done on error

[PATCH v4 8/9] media: venus: hfi: add Venus HFI files

2016-12-01 Thread Stanimir Varbanov
Here is the implementation of Venus video accelerator low-level functionality. It contanins code which setup the registers and startup uthe processor, allocate and manipulates with the shared memory used for sending commands and receiving messages. Signed-off-by: Stanimir Varbanov

[PATCH v4 4/9] media: venus: adding core part and helper functions

2016-12-01 Thread Stanimir Varbanov
* core.c has implemented the platform dirver methods, file operations and v4l2 registration. * helpers.c has implemented common helper functions for: - buffer management - vb2_ops and functions for format propagation, - functions for allocating and freeing buffers for internal

[PATCH v4 1/9] media: v4l2-mem2mem: extend m2m APIs for more accurate buffer management

2016-12-01 Thread Stanimir Varbanov
this add functions for: - remove buffers from src/dst queue by index - remove exact buffer from src/dst queue also extends m2m API to iterate over a list of src/dst buffers in safely and non-safely manner. Signed-off-by: Stanimir Varbanov ---

[PATCH v4 9/9] media: venus: enable building of Venus video driver

2016-12-01 Thread Stanimir Varbanov
This adds Venus driver Makefile and changes v4l2 platform Makefile/Kconfig in order to enable building of the driver. Signed-off-by: Stanimir Varbanov --- drivers/media/platform/Kconfig | 13 + drivers/media/platform/Makefile| 2

Re: [PATCH v3 4/4] [media] dt-bindings: add TI VPIF documentation

2016-12-01 Thread Sakari Ailus
Hi Kevin, On Wed, Nov 30, 2016 at 03:48:51PM -0800, Kevin Hilman wrote: > Sakari Ailus writes: > > > Hi Rob and Kevin, > > > > On Tue, Nov 29, 2016 at 08:41:44AM -0600, Rob Herring wrote: > >> On Mon, Nov 28, 2016 at 4:30 PM, Kevin Hilman wrote: > >>