[PATCH 1/1] smiapp: Implement power-on and power-off sequences without runtime PM

2016-11-18 Thread Sakari Ailus
Power on the sensor when the module is loaded and power it off when it is removed. Signed-off-by: Sakari Ailus --- Hi Arnd and others, The patch is tested with CONFIG_PM set, as the system does I was testing on did not boot with CONFIG_PM disabled. I'm not really

patch "media: usb: uvc: remove unnecessary & operation" added to usb-next

2016-11-18 Thread gregkh
This is a note to let you know that I've just added the patch titled media: usb: uvc: remove unnecessary & operation to my usb git tree which can be found at git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git in the usb-next branch. The patch will show up in the next release

Re: [PATCH 1/1] smiapp: Implement power-on and power-off sequences without runtime PM

2016-11-18 Thread Arnd Bergmann
On Friday, November 18, 2016 3:50:16 PM CET Sakari Ailus wrote: > Power on the sensor when the module is loaded and power it off when it is > removed. > > Signed-off-by: Sakari Ailus > --- > Hi Arnd and others, > > The patch is tested with CONFIG_PM set, as the

patch "media: usbtv: core: make use of new usb_endpoint_maxp_mult()" added to usb-testing

2016-11-18 Thread gregkh
This is a note to let you know that I've just added the patch titled media: usbtv: core: make use of new usb_endpoint_maxp_mult() to my usb git tree which can be found at git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git in the usb-testing branch. The patch will show up in

Re: [PATCH] [media] rc/keymaps: Add helper macro for rc_map_list boilerplate

2016-11-18 Thread Mauro Carvalho Chehab
Em Sat, 11 Jul 2015 08:47:37 +0530 Vaishali Thakkar escreveu: > For simple modules that contain a single rc_map_list without any > additional setup code then ends up being a block of duplicated > boilerplate. This patch adds a new macro, module_rc_map_list(), > which

patch "media: usb: uvc: remove unnecessary & operation" added to usb-testing

2016-11-18 Thread gregkh
This is a note to let you know that I've just added the patch titled media: usb: uvc: remove unnecessary & operation to my usb git tree which can be found at git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git in the usb-testing branch. The patch will show up in the next

patch "media: usbtv: core: make use of new usb_endpoint_maxp_mult()" added to usb-next

2016-11-18 Thread gregkh
This is a note to let you know that I've just added the patch titled media: usbtv: core: make use of new usb_endpoint_maxp_mult() to my usb git tree which can be found at git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git in the usb-next branch. The patch will show up in the

patch "media: usb: uvc: make use of new usb_endpoint_maxp_mult()" added to usb-next

2016-11-18 Thread gregkh
This is a note to let you know that I've just added the patch titled media: usb: uvc: make use of new usb_endpoint_maxp_mult() to my usb git tree which can be found at git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git in the usb-next branch. The patch will show up in the

patch "media: usb: uvc: make use of new usb_endpoint_maxp_mult()" added to usb-testing

2016-11-18 Thread gregkh
This is a note to let you know that I've just added the patch titled media: usb: uvc: make use of new usb_endpoint_maxp_mult() to my usb git tree which can be found at git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git in the usb-testing branch. The patch will show up in the

Re: ir-keytable: infinite loops, segfaults

2016-11-18 Thread Vincent McIntyre
On Fri, Nov 18, 2016 at 05:40:34PM +, Sean Young wrote: > > > > # ir-keytable > > Found /sys/class/rc/rce0/ (/dev/input/event5) with: > > Driver imon, table rc-imon-mce > > Supported protocols: rc-6 > > Enabled protocols: rc-6 > > Name: iMON Remote (15c2:ffdc) > > bus:

[linuxtv-media:master 879/885] htmldocs: drivers/media/dvb-core/dvb_frontend.h:684: warning: No description found for parameter 'refcount'

2016-11-18 Thread kbuild test robot
tree: git://linuxtv.org/media_tree.git master head: c044170fcfca3783f7dd8eb69ff8b06d66fad5d8 commit: 1f862a68df2449bc7b1cf78dce616891697b4bdf [879/885] [media] dvb_frontend: move kref to struct dvb_frontend reproduce: make htmldocs; make DOCBOOKS='' pdfdocs All warnings (new ones prefixed by

[linuxtv-media:master] BUILD REGRESSION c044170fcfca3783f7dd8eb69ff8b06d66fad5d8

2016-11-18 Thread kbuild test robot
git://linuxtv.org/media_tree.git master c044170fcfca3783f7dd8eb69ff8b06d66fad5d8 [media] media: rc: nuvoton: replace usage of spin_lock_irqsave in ISR drivers/built-in.o:(.rodata+0x117278): undefined reference to `dvb_tuner_simple_release' drivers/built-in.o:(.rodata+0x27638): undefined

[PATCH v5 1/3] media: Media Device Allocator API

2016-11-18 Thread Shuah Khan
Media Device Allocator API to allows multiple drivers share a media device. Using this API, drivers can allocate a media device with the shared struct device as the key. Once the media device is allocated by a driver, other drivers can get a reference to it. The media device is released when all

Re: [PATCH 08/35] [media] cx88: convert it to use pr_foo() macros

2016-11-18 Thread Andrey Utkin
On Wed, Nov 16, 2016 at 02:42:40PM -0200, Mauro Carvalho Chehab wrote: > From: Mauro Carvalho Chehab > > Instead of calling printk() directly, use pr_foo() > macros, as suggested at the Kernel's coding style. > > Please notice that a conversion to dev_foo() is not

Re: ir-keytable: infinite loops, segfaults

2016-11-18 Thread Vincent McIntyre
On Fri, Nov 18, 2016 at 05:40:34PM +, Sean Young wrote: > > > > So are you saying that the hex codes in the rc_map_dvico_mce_table > > struct are invalid (at least in some cases)? > > Most likely the remote produces IR in a standard protocol (e.g. rc5, rc6). > If we first get the keymap

Re: [PATCH dtv-scan-tables] Rename pl-Krosno_Sucha_Gora with only ASCII characters

2016-11-18 Thread Thomas Petazzoni
Hello, On Fri, 18 Nov 2016 22:05:02 +0100, Olliver Schinagl wrote: > I agree for consistency sake and ease of use, to use plain ascii for > pl-Krosno_Sucha_Gora as well. If someone feels that we should follow > proper spelling using UTF-8, someone should fix up and correct all names > in 1

[linuxtv-media:master 874/885] ERROR: "dvb_tuner_simple_release" [drivers/media/tuners/tea5767.ko] undefined!

2016-11-18 Thread kbuild test robot
tree: git://linuxtv.org/media_tree.git master head: c044170fcfca3783f7dd8eb69ff8b06d66fad5d8 commit: 22a613e89825ea7a3984a968463cc6d425bd8856 [874/885] [media] dvb_frontend: merge duplicate dvb_tuner_ops.release implementations config: i386-randconfig-r0-201646 (attached as .config) compiler:

Re: [PATCH dtv-scan-tables] Rename pl-Krosno_Sucha_Gora with only ASCII characters

2016-11-18 Thread Olliver Schinagl
Hey Thomas, On 14-11-16 22:05, Thomas Petazzoni wrote: The pl-Krosno_Sucha_Gora file, added in commit 4cb113fd15e562f0629000fcad9f41405595198d, is the only file that contains non-ASCII characters in the tree. This causes a number of build issues with other packages that don't necessarily handle

Re: [PATCH 09/35] [media] cx88: make checkpatch happier

2016-11-18 Thread Andrey Utkin
On Wed, Nov 16, 2016 at 02:42:41PM -0200, Mauro Carvalho Chehab wrote: > This driver is old, and have lots of checkpatch violations. > As we're touching a lot on this driver due to the printk > conversions, let's run checkpatch --fix on it, in order to > solve some of those issues. Also, let's

Re: [PATCH 2/3] [media] v4l: rcar_fdp1: add FCP dependency

2016-11-18 Thread Geert Uytterhoeven
Hi Arnd, On Fri, Nov 18, 2016 at 5:16 PM, Arnd Bergmann wrote: > drivers/media/platform/rcar_fdp1.o: In function `fdp1_pm_runtime_resume': > rcar_fdp1.c:(.text.fdp1_pm_runtime_resume+0x78): undefined reference to > `rcar_fcp_enable' > drivers/media/platform/rcar_fdp1.o: In

[PATCH 1/3] [media] v4l: rcar_fdp1: mark PM functions as __maybe_unused

2016-11-18 Thread Arnd Bergmann
The new driver produces a warning when CONFIG_PM is disabled: platform/rcar_fdp1.c:2408:12: error: 'fdp1_pm_runtime_resume' defined but not used [-Werror=unused-function] platform/rcar_fdp1.c:2399:12: error: 'fdp1_pm_runtime_suspend' defined but not used [-Werror=unused-function] This marks

[PATCH 2/3] [media] v4l: rcar_fdp1: add FCP dependency

2016-11-18 Thread Arnd Bergmann
drivers/media/platform/rcar_fdp1.o: In function `fdp1_pm_runtime_resume': rcar_fdp1.c:(.text.fdp1_pm_runtime_resume+0x78): undefined reference to `rcar_fcp_enable' drivers/media/platform/rcar_fdp1.o: In function `fdp1_pm_runtime_suspend': rcar_fdp1.c:(.text.fdp1_pm_runtime_suspend+0x14):

Re: ir-keytable: infinite loops, segfaults

2016-11-18 Thread Sean Young
On Fri, Nov 18, 2016 at 11:14:25PM +1100, Vincent McIntyre wrote: > On Thu, Nov 17, 2016 at 01:45:26PM +, Sean Young wrote: > > On Wed, Nov 16, 2016 at 09:52:58PM +1100, Vincent McIntyre wrote: > > > I have a fairly old dvico dual digital 4 tuner and remote. > > > There seem to be some issues

[PATCH] [media] soc-camera: Fix a return value in case of error

2016-11-18 Thread Christophe JAILLET
If 'ov9640_reg_read()' does not return 0, then 'val' is left unmodified. As it is not initialized either, the return value can be anything. It is likely that returning the error code was expected here. Signed-off-by: Christophe JAILLET ---

[PATCH 3/3] [media] mtk-mdp: mark PM functions as __maybe_unused

2016-11-18 Thread Arnd Bergmann
The driver still produces a warning when CONFIG_PM is disabled, an earlier fix only partially solved this: media/platform/mtk-mdp/mtk_mdp_core.c:72:13: error: 'mtk_mdp_clock_off' defined but not used [-Werror=unused-function] media/platform/mtk-mdp/mtk_mdp_core.c:63:13: error: 'mtk_mdp_clock_on'

Re: [PATCH 1/5] media: rc: nuvoton: remove nvt_open and nvt_close

2016-11-18 Thread Mauro Carvalho Chehab
Em Tue, 25 Oct 2016 21:23:16 +0200 Heiner Kallweit escreveu: > What is done in nvt_probe was done in nvt_probe already > (in nvt_cir_ldev_init and nvt_cir_regs_init, both called from > nvt_probe). It's the same with nvt_close, it's covered by nvt_remove. > Therefore I don't

[linuxtv-media:master 874/881] undefined reference to `dvb_tuner_simple_release'

2016-11-18 Thread kbuild test robot
tree: git://linuxtv.org/media_tree.git master head: e7cd17a29d72868aa5cfbbd4de7c54a28ed30ff0 commit: 22a613e89825ea7a3984a968463cc6d425bd8856 [874/881] [media] dvb_frontend: merge duplicate dvb_tuner_ops.release implementations config: x86_64-randconfig-i0-201646 (attached as .config)

Re: [PATCH 2/3] [media] v4l: rcar_fdp1: add FCP dependency

2016-11-18 Thread Arnd Bergmann
On Friday, November 18, 2016 6:08:01 PM CET Geert Uytterhoeven wrote: > > diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig > > index 3c5a0b6b23a9..cd0cab6e0e31 100644 > > --- a/drivers/media/platform/Kconfig > > +++ b/drivers/media/platform/Kconfig > > @@ -311,6 +311,7

Re: [PATCH v5] [media] vb2: Add support for capture_dma_bidirectional queue flag

2016-11-18 Thread Mauro Carvalho Chehab
Em Tue, 25 Oct 2016 10:22:29 +0200 Thierry Escande escreveu: > From: Pawel Osciak > > When this flag is set for CAPTURE queues by the driver on calling > vb2_queue_init(), it forces the buffers on the queue to be > allocated/mapped with

[PATCH] [media] Kconfig: fix breakages when DVB_CORE is not selected

2016-11-18 Thread Mauro Carvalho Chehab
On some weird randconfigs, it is possible to select DVB drivers, without having the DVB_CORE: CONFIG_DVB_AU8522=m CONFIG_DVB_AU8522_V4L=m CONFIG_DVB_TUNER_DIB0090=m This was never supposed to work, but changeset 22a613e89825 ("[media] dvb_frontend: merge duplicate dvb_tuner_ops.release

[PATCH] Revert "[media] dvb_frontend: merge duplicate dvb_tuner_ops.release implementations"

2016-11-18 Thread Mauro Carvalho Chehab
While this patch sounded a good idea, unfortunately, it causes bad dependencies, as drivers that would otherwise work without the DVB core will now break: ERROR: "dvb_tuner_simple_release" [drivers/media/tuners/tea5767.ko] undefined! ERROR: "dvb_tuner_simple_release"

[Patch v2 24/35] media: ti-vpe: vpe: Fix vb2 buffer cleanup

2016-11-18 Thread Benoit Parrot
When stop_streaming is called we need to cleanup the queued vb2 buffers properly. This was not previously being done which caused kernel warning when the application using the resources was killed. Kernel warnings were also generated on successful completion of a de-interlacing case as well as

[Patch v2 11/35] media: ti-vpe: vpdma: Add support for setting max width height

2016-11-18 Thread Benoit Parrot
From: Nikhil Devshatwar Add a helper function to be able to set the maximum VPDMA transfer size to limit potential buffer overrun. Added enums for max_width and max_height fields of the outbound data descriptor. Changed vpdma_add_out_dtd to accept two more arguments for max

[Patch v2 19/35] media: ti-vpe: vpdma: allocate and maintain hwlist

2016-11-18 Thread Benoit Parrot
From: Nikhil Devshatwar VPDMA block used in ti-vip and ti-vpe modules have support for up to 8 hardware descriptor lists. A descriptor list can be submitted to any of the 8 lists (as long as it's not busy). When multiple clients want to transfer data in parallel, its easier to

[Patch v2 18/35] media: ti-vpe: vpe: Add RGB565 and RGB5551 support

2016-11-18 Thread Benoit Parrot
From: Nikhil Devshatwar VPE hardware can generate output in RGB565 or in RGB5551 format. Add these formats in the supported format list for CAPTURE stream. Also, for RGB5551 format, the alpha component is not processed, so the alpha value is taken from the default color. Set

[Patch v2 21/35] media: ti-vpe: vpdma: Corrected YUV422 data type label.

2016-11-18 Thread Benoit Parrot
The YUV data type definition below are taken from both the TRM and i839 Errata information. Use the correct data type considering byte reordering of components. Added the 2 missing YUV422 variant. Also since the single use of "C" in the 422 case to mean "Cr" (i.e. V component). It was decided to

[Patch v2 22/35] media: ti-vpe: vpdma: RGB data type yield inverted data

2016-11-18 Thread Benoit Parrot
The VPDMA RGB data type definition have been updated to match with Errata i839. But some of the ARGB definition appeared to be wrong in the document also. As they would yield RGBA instead. They have been corrected based on experimentation. Signed-off-by: Benoit Parrot Acked-by:

[Patch v2 17/35] media: ti-vpe: vpe: Post next descriptor only for list complete IRQ

2016-11-18 Thread Benoit Parrot
From: Nikhil Devshatwar vpe_irq checks for the possible interrupt sources and prints the errors for the DEI_ERROR and DS_UV interrupts. But it also post the next descriptor list irrespective of whichever interrupt has occurred. Because of this, driver may release the buffers

[Patch v2 20/35] media: ti-vpe: vpe: Added MODULE_DEVICE_TABLE hint

2016-11-18 Thread Benoit Parrot
ti_vpe module currently does not get loaded automatically. Added MODULE_DEVICE_TABLE hint to the driver to assist. Signed-off-by: Benoit Parrot Acked-by: Hans Verkuil Reviewed-by: Javier Martinez Canillas ---

[Patch v2 06/35] media: ti-vpe: vpe: Do not perform job transaction atomically

2016-11-18 Thread Benoit Parrot
From: Nikhil Devshatwar Current VPE driver does not start the job until all the buffers for a transaction are queued. When running in multiple context, this might increase the processing latency. Alternate solution would be to try to continue the same context as long as

[Patch v2 16/35] media: ti-vpe: vpe: Setup srcdst parameters in start_streaming

2016-11-18 Thread Benoit Parrot
From: Nikhil Devshatwar For deinterlacing operation, each operation needs 2 fields in the history. This is achieved by holding three buffers in ctx->src_vbs[0,1,2] (f,f-1,f-2) This is achieved by using the ctx->sequence which gets reset via the s_fmt ioctl. These buffers are

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

2016-11-18 Thread Kevin Hilman
Cc: Rob Herring Signed-off-by: Kevin Hilman --- .../devicetree/bindings/media/ti,vpif-capture.txt | 65 ++ .../devicetree/bindings/media/ti,vpif.txt | 8 +++ 2 files changed, 73 insertions(+) create mode 100644

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

2016-11-18 Thread Kevin Hilman
Allow getting of subdevs from DT ports and endpoints. The _get_pdata() function was larely inspired by (i.e. stolen from) am437x-vpfe.c Signed-off-by: Kevin Hilman --- drivers/media/platform/davinci/vpif_capture.c | 130 +-

[Patch v2 01/35] media: ti-vpe: vpdma: Make vpdma library into its own module

2016-11-18 Thread Benoit Parrot
The VPDMA (Video Port DMA) as found in devices such as DRA7xx is used for both the Video Processing Engine (VPE) and the Video Input Port (VIP). In preparation for this we need to turn vpdma into its own kernel module. Signed-off-by: Benoit Parrot ---

[Patch v2 14/35] media: ti-vpe: vpdma: Clear IRQs for individual lists

2016-11-18 Thread Benoit Parrot
From: Nikhil Devshatwar VPDMA IRQs are registered for multiple lists When clearing an IRQ for a list interrupt, all the IRQs for the individual lists are to be cleared separately. Signed-off-by: Nikhil Devshatwar Signed-off-by: Benoit Parrot

[Patch v2 07/35] media: ti-vpe: Add support for SEQ_TB buffers

2016-11-18 Thread Benoit Parrot
From: Nikhil Devshatwar The video source can generate the data in the SEQ_TB buffer format. In the case of TI SoC, the IVA_HD can generate the interlaced content in the SEQ_TB buffer format. This is the format where the top and bottom field data can be contained in a single

[Patch v2 02/35] media: ti-vpe: vpdma: Add multi-instance and multi-client support

2016-11-18 Thread Benoit Parrot
The VPDMA (Video Port DMA) as found in devices such as DRA7xx is used for both the Video Processing Engine (VPE) and the Video Input Port (VIP). Some devices may have multiple VIP instances each with its own VPDMA engine. Within VIP two slices can use a single VPDMA engine simultaneously. So

[Patch v2 08/35] media: ti-vpe: Increasing max buffer height and width

2016-11-18 Thread Benoit Parrot
From: Harinarayan Bhatta Increasing max buffer height and width to allow for padded buffers. Signed-off-by: Harinarayan Bhatta Signed-off-by: Nikhil Devshatwar Signed-off-by: Benoit Parrot Acked-by: Hans Verkuil

[Patch v2 15/35] media: ti-vpe: vpe: configure line mode separately

2016-11-18 Thread Benoit Parrot
From: Nikhil Devshatwar Current driver configures the line mode of the DEI clients from the open function directly. Even if the newly created context is not yet scheduled, it updates some of the VPDMA registers. This causes a problem in multi instance use case where just

[Patch v2 13/35] media: ti-vpe: vpdma: Make list post atomic operation

2016-11-18 Thread Benoit Parrot
From: Nikhil Devshatwar Writing to the "VPDMA list attribute" register is considered as a list post. This informs the VPDMA firmware to load the list from the address which should be taken from the "VPDMA list address" register. As these two register writes are dependent, it

[Patch v2 04/35] media: ti-vpe: vpdma: Fix bus error when vpdma is writing a descriptor

2016-11-18 Thread Benoit Parrot
On DRA7 since l3_noc event are being reported it was found that when the write descriptor was being written it was consistently causing bus error events. The write address was improperly programmed. Signed-off-by: Benoit Parrot Acked-by: Hans Verkuil ---

[Patch v2 00/35] media: ti-vpe: fixes and enhancements

2016-11-18 Thread Benoit Parrot
This patch series is to publish a number of enhancements we have been carrying for a while. A number of bug fixes and feature enhancements have been included. We also need to prepare the way for the introduction of the VIP (Video Input Port) driver (coming soon) which has internal IP module in

[Patch v2 03/35] media: ti-vpe: vpdma: Add helper to set a background color

2016-11-18 Thread Benoit Parrot
Add a helper to set the background color during vpdma transfer. This is needed when VPDMA is generating 32 bits RGB format to have the Alpha channel set to an appropriate value. Signed-off-by: Benoit Parrot Acked-by: Hans Verkuil ---

[Patch v2 05/35] media: ti-vpe: Use line average de-interlacing for first 2 frames

2016-11-18 Thread Benoit Parrot
From: Archit Taneja The motion detection block requires 3 fields to create the motion vector data. This means that using the default method the first progressive frame is only generated after 3rd field is consumed. Hence by default for N input field we would generate N - 2

[Patch v2 10/35] media: ti-vpe: Free vpdma buffers in vpe_release

2016-11-18 Thread Benoit Parrot
From: Harinarayan Bhatta Free vpdma buffers in vpe_release. Otherwise it was generating random backtrace. Signed-off-by: Harinarayan Bhatta Signed-off-by: Somnath Mukherjee Signed-off-by: Nikhil Devshatwar

[Patch v2 09/35] media: ti-vpe: vpe: Return NULL for invalid buffer type

2016-11-18 Thread Benoit Parrot
From: Nikhil Devshatwar get_q_data can be called with different values for type e.g. vpe_try_crop calls it with the buffer type which gets passed from user space Framework doesn't check wheather its correct type or not If user space passes wrong type, kernel should not crash.

Re: ir-keytable: infinite loops, segfaults

2016-11-18 Thread Vincent McIntyre
On Fri, Nov 18, 2016 at 05:40:34PM +, Sean Young wrote: > > At the moment it's not easy to determine what protocol an remote uses; > I would like to change that but for now, the following is probably > the easiest way. > > cd /sys/class/rc/rc1 # replace rc1 with your receiver > for i in

[Patch v2 33/35] media: ti-vpe: Make colorspace converter library into its own module

2016-11-18 Thread Benoit Parrot
In preparation to add colorspace conversion support to VIP, we need to turn csc.c into its own kernel module. Signed-off-by: Benoit Parrot Acked-by: Hans Verkuil --- drivers/media/platform/Kconfig | 4 drivers/media/platform/ti-vpe/Makefile

[Patch v2 32/35] media: ti-vpe: vpdma: Add RAW8 and RAW16 data types

2016-11-18 Thread Benoit Parrot
Add RAW8 and RAW16 data type to VPDMA. To handle RAW format we are re-using the YUV CBY422 vpdma data type so that we use the vpdma to re-order the incoming bytes, as the VIP parser assumes that the first byte presented on the bus is the MSB of a 2 bytes value. RAW8 handles from 1 to 8 bits.

[Patch v2 35/35] media: ti-vpe: vpe: Add proper support single and multi-plane buffer

2016-11-18 Thread Benoit Parrot
The VPE was restricting the number of plane per buffer based on the fact that if a particular format had color separation it was meant to need 2 planes. However NV12/NV16 are color separate format which are meant to be presented in a single contiguous buffer/plane. It could also be presented in a

[Patch v2 31/35] media: ti-vpe: vpe: Make sure frame size dont exceed scaler capacity

2016-11-18 Thread Benoit Parrot
When scaler is to be used we need to make sure that the input and output frame size do not exceed the maximum frame sizes that the scaler h/w can handle otherwise streaming stall as the scaler cannot proceed. The scaler buffer is limited to 2047 pixels (i.e. 11 bits) when attempting anything

[Patch v2 34/35] media: ti-vpe: csc: Add debug support for multi-instance

2016-11-18 Thread Benoit Parrot
Since there might be more then one instance it is better to show the base address when dumping registers to help with debugging. Signed-off-by: Benoit Parrot Acked-by: Hans Verkuil --- drivers/media/platform/ti-vpe/csc.c | 2 ++ 1 file changed, 2

[Patch v2 28/35] media: ti-vpe: vpe: Enable DMABUF export

2016-11-18 Thread Benoit Parrot
Allow VPE to be able to export DMA buffer. Signed-off-by: Benoit Parrot Acked-by: Hans Verkuil --- drivers/media/platform/ti-vpe/vpe.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/platform/ti-vpe/vpe.c

[Patch v2 27/35] media: ti-vpe: vpe: Fix line stride for output motion vector

2016-11-18 Thread Benoit Parrot
From: Nikhil Devshatwar For deinterlacing operation, VPE hardware uses motion vectors. MV calculated in the previous iteration are used for next interation. Therefore driver allocates two motion vectors in ping-pong fashion. For every transaction, one MV is DMAed in and one is

[Patch v2 29/35] media: ti-vpe: Make scaler library into its own module

2016-11-18 Thread Benoit Parrot
In preparation to add scaler support into VIP we need to turn sc.c into its own kernel module. Add support for multiple SC memory block as VIP contains 2 scaler instances. This is done by passing the resource name to sc_create() and modify the vpe invocation accordingly. Signed-off-by: Benoit

[Patch v2 23/35] media: ti-vpe: sc: Fix incorrect optimization

2016-11-18 Thread Benoit Parrot
From: Nikhil Devshatwar Current scaler library implementation of sc_set_hs_coeffs and sc_set_vs_coeffs tries to return immediately if the calculated coefficient index is already being used. As the same scaler block is going to be used for all the VPE contexts, even if the

[Patch v2 25/35] media: ti-vpe: vpdma: Fix race condition for firmware loading

2016-11-18 Thread Benoit Parrot
From: Nikhil Devshatwar vpdma_create API is supposed to allocated the struct vpdma_data and return it to the driver. Also, it would call the callback function when the VPDMA firmware is loaded. Typically, VPE driver have following function call: dev->vpdma =

[Patch v2 12/35] media: ti-vpe: vpdma: Add abort channel desc and cleanup APIs

2016-11-18 Thread Benoit Parrot
From: Nikhil Devshatwar Whenever VPDMA processes a data descriptor of a list, it processes it and sets up the channel for the DMA transaction. List manager holds the descriptor in the list until the DMA is complete. If sync_on_channel descriptor, or another descriptor for the

[Patch v2 30/35] media: ti-vpe: scaler: Add debug support for multi-instance

2016-11-18 Thread Benoit Parrot
Since there might be more then one instance it is better to show the base address when dumping registers to help with debugging. Signed-off-by: Benoit Parrot Acked-by: Hans Verkuil --- drivers/media/platform/ti-vpe/sc.c | 2 ++ 1 file changed, 2

[Patch v2 26/35] media: ti-vpe: vpdma: Use bidirectional cached buffers

2016-11-18 Thread Benoit Parrot
From: Nikhil Devshatwar VPDMA buffer will be used by CPU as well as by the VPDMA. CPU will write/update the VPDMA descriptors containing data about the video buffer DMA addresses. VPDMA will write the "write descriptor" containing the data about the DMA operation. When

[PATCH 1/4] [media] davinci: add support for DT init

2016-11-18 Thread Kevin Hilman
Add basic support for initialization via DT. Signed-off-by: Kevin Hilman --- drivers/media/platform/davinci/vpif.c | 9 + drivers/media/platform/davinci/vpif_capture.c | 14 ++ 2 files changed, 23 insertions(+) diff --git

[PATCH 2/4] [media] davinci: vpif_capture: don't lock over s_stream

2016-11-18 Thread Kevin Hilman
Video capture subdevs may be over I2C and may sleep during xfer, so we cannot do IRQ-disabled locking when calling the subdev. Signed-off-by: Kevin Hilman --- drivers/media/platform/davinci/vpif_capture.c | 4 1 file changed, 4 insertions(+) diff --git

cron job: media_tree daily build: ERRORS

2016-11-18 Thread Hans Verkuil
This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Sat Nov 19 05:00:17 CET 2016 media-tree git hash:f2709c206d8a3e11729e68d80c57e7470bbe8e5e media_build

Re: [PATCH 14/18] [media] RedRat3: Rename a jump label in redrat3_init_rc_dev()

2016-11-18 Thread Mauro Carvalho Chehab
Em Fri, 18 Nov 2016 10:52:40 -0200 Mauro Carvalho Chehab escreveu: > Em Thu, 13 Oct 2016 18:42:16 +0200 > SF Markus Elfring escreveu: > > > From: Markus Elfring > > Date: Thu, 13 Oct 2016 15:00:12 +0200 >

[PATCH v2 03/10] ARM: multi_v7_defconfig: enable STMicroelectronics DELTA Support

2016-11-18 Thread Hugues Fruchet
Signed-off-by: Hugues Fruchet --- arch/arm/configs/multi_v7_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig index 11f37ed..8500f75 100644 --- a/arch/arm/configs/multi_v7_defconfig +++

[PATCH v2 00/10] Add support for DELTA video decoder of STMicroelectronics STiH4xx SoC series

2016-11-18 Thread Hugues Fruchet
This patchset introduces a basic support for DELTA multi-format video decoder of STMicroelectronics STiH4xx SoC series. DELTA hardware IP is controlled by a remote firmware loaded in a ST231 coprocessor. Communication with firmware is done within an IPC layer using rpmsg kernel framework and a

[PATCH v2 04/10] [media] MAINTAINERS: add st-delta driver

2016-11-18 Thread Hugues Fruchet
Add entry for the STMicroelectronics DELTA driver. Signed-off-by: Hugues Fruchet --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 7db3f7a..a96dd22 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -2394,6 +2394,14 @@

[PATCH v2 02/10] ARM: dts: STiH410: add DELTA dt node

2016-11-18 Thread Hugues Fruchet
This patch adds DT node for STMicroelectronics DELTA V4L2 video decoder Signed-off-by: Hugues Fruchet --- arch/arm/boot/dts/stih410.dtsi | 10 ++ 1 file changed, 10 insertions(+) diff --git a/arch/arm/boot/dts/stih410.dtsi b/arch/arm/boot/dts/stih410.dtsi index

[PATCH v2 01/10] Documentation: DT: add bindings for ST DELTA

2016-11-18 Thread Hugues Fruchet
This patch adds DT binding documentation for STMicroelectronics DELTA V4L2 video decoder. Signed-off-by: Hugues Fruchet --- Documentation/devicetree/bindings/media/st,st-delta.txt | 17 + 1 file changed, 17 insertions(+) create mode 100644

[PATCH v2 08/10] [media] st-delta: EOS (End Of Stream) support

2016-11-18 Thread Hugues Fruchet
Signed-off-by: Hugues Fruchet --- drivers/media/platform/sti/delta/delta-v4l2.c | 143 +- drivers/media/platform/sti/delta/delta.h | 23 + 2 files changed, 165 insertions(+), 1 deletion(-) diff --git

[PATCH v2 05/10] [media] st-delta: STiH4xx multi-format video decoder v4l2 driver

2016-11-18 Thread Hugues Fruchet
This V4L2 driver enables DELTA multi-format video decoder of STMicroelectronics STiH4xx SoC series. Signed-off-by: Hugues Fruchet --- drivers/media/platform/Kconfig| 20 + drivers/media/platform/Makefile |2 +

[PATCH v2 09/10] [media] st-delta: add mjpeg support

2016-11-18 Thread Hugues Fruchet
Signed-off-by: Hugues Fruchet --- drivers/media/platform/Kconfig | 6 + drivers/media/platform/sti/delta/Makefile | 4 + drivers/media/platform/sti/delta/delta-cfg.h | 3 + drivers/media/platform/sti/delta/delta-mjpeg-dec.c | 454

[PATCH v2 06/10] [media] st-delta: add contiguous memory allocator

2016-11-18 Thread Hugues Fruchet
Signed-off-by: Hugues Fruchet --- drivers/media/platform/sti/delta/Makefile| 2 +- drivers/media/platform/sti/delta/delta-mem.c | 51 drivers/media/platform/sti/delta/delta-mem.h | 14 drivers/media/platform/sti/delta/delta.h

[PATCH v2 10/10] [media] st-delta: debug: trace stream/frame information & summary

2016-11-18 Thread Hugues Fruchet
Signed-off-by: Hugues Fruchet --- drivers/media/platform/sti/delta/Makefile | 2 +- drivers/media/platform/sti/delta/delta-debug.c | 72 ++ drivers/media/platform/sti/delta/delta-debug.h | 18 +++

[PATCH v2 07/10] [media] st-delta: rpmsg ipc support

2016-11-18 Thread Hugues Fruchet
IPC (Inter Process Communication) support for communication with DELTA coprocessor firmware using rpmsg kernel framework. Based on 4 services open/set_stream/decode/close and their associated rpmsg messages. The messages structures are duplicated on both host and firmware side and are packed (use

[patch] Staging: media: davinci_vpfe: unlock on error in vpfe_reqbufs()

2016-11-18 Thread Dan Carpenter
We should unlock before returning this error code in vpfe_reqbufs(). Fixes: 622897da67b3 ("[media] davinci: vpfe: add v4l2 video driver support") Signed-off-by: Dan Carpenter diff --git a/drivers/staging/media/davinci_vpfe/vpfe_video.c

Re: [Ksummit-discuss] Including images on Sphinx documents

2016-11-18 Thread Daniel Vetter
On Fri, Nov 18, 2016 at 11:15:09AM +0200, Jani Nikula wrote: > On Thu, 17 Nov 2016, Linus Torvalds wrote: > > We have makefiles, but more importantly, few enough people actually > > *generate* the documentation, that I think if it's an option to just > > fix sphinx,

Re: [Ksummit-discuss] Including images on Sphinx documents

2016-11-18 Thread Jani Nikula
On Thu, 17 Nov 2016, Linus Torvalds wrote: > We have makefiles, but more importantly, few enough people actually > *generate* the documentation, that I think if it's an option to just > fix sphinx, we should do that instead. If it means that you have to > have some

Re: [PATCH] [media] VPU: mediatek: fix dereference of pdev before checking it is null

2016-11-18 Thread Matthias Brugger
On 16/11/16 20:16, Colin King wrote: From: Colin Ian King pdev is dereferenced using platform_get_drvdata before a check to see if it is null, hence there could be a potential null pointer dereference issue. Instead, first check if pdev is null and only then

Re: [PATCH 14/18] [media] RedRat3: Rename a jump label in redrat3_init_rc_dev()

2016-11-18 Thread Mauro Carvalho Chehab
Em Thu, 13 Oct 2016 18:42:16 +0200 SF Markus Elfring escreveu: > From: Markus Elfring > Date: Thu, 13 Oct 2016 15:00:12 +0200 > > Adjust a jump label according to the Linux coding style convention. > > Signed-off-by: Markus Elfring

Re: ir-keytable: infinite loops, segfaults

2016-11-18 Thread Vincent McIntyre
On Thu, Nov 17, 2016 at 01:45:26PM +, Sean Young wrote: > On Wed, Nov 16, 2016 at 09:52:58PM +1100, Vincent McIntyre wrote: > > I have a fairly old dvico dual digital 4 tuner and remote. > > There seem to be some issues with support for it, can I help fix them? > > > > I am using ir-keytable

Re: [PATCH v3 4/9] media: venus: vdec: add video decoder files

2016-11-18 Thread Stanimir Varbanov
Hi Hans, >>> + >>> +static int >>> +vdec_reqbufs(struct file *file, void *fh, struct v4l2_requestbuffers *b) >>> +{ >>> + struct vb2_queue *queue = to_vb2q(file, b->type); >>> + >>> + if (!queue) >>> + return -EINVAL; >>> + >>> + return vb2_reqbufs(queue, b); >>> +} >> >> Is there