Re: [PATCH v2 2/2] x86/mm/pat, drivers/media/ivtv: move pat warn and replace WARN() with pr_warn()

2015-06-24 Thread Ingo Molnar
* Luis R. Rodriguez wrote: > From: "Luis R. Rodriguez" > > On built-in kernels this warning will always splat as this is part > of the module init. Fix that by shifting the PAT requirement check > out under the code that does the "quasi-probe" for the device. This > device driver relies on an

Re: [PATCH v2 1/2] x86/mm/pat, drivers/infiniband/ipath: replace WARN() with pr_warn()

2015-06-24 Thread Ingo Molnar
* Luis R. Rodriguez wrote: > From: "Luis R. Rodriguez" > > WARN() may confuse users, fix that. ipath_init_one() is part the > device's probe so this would only be triggered if a corresponding > device was found. > > Signed-off-by: Luis R. Rodriguez > --- > drivers/infiniband/hw/ipath/ipath_

cron job: media_tree daily build: OK

2015-06-24 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: Thu Jun 25 04:00:30 CEST 2015 git branch: test git hash: faebbd8f134f0c054f372982c8ddd1bbcc41b440 gcc versi

Re: [PATCH 02/12] [media] dvb-pll: Add support for THOMSON DTT7546X tuner.

2015-06-24 Thread Joe Perches
On Wed, 2015-06-24 at 16:11 +0100, Peter Griffin wrote: > This is used in conjunction with the STV0367 demodulator on > the STV0367-NIM-V1.0 NIM card which can be used with the STi > STB SoC's. Barely associated to this specific patch, but for dvb-pll.c, another thing that seems possible is to con

[PATCH v2 2/2] x86/mm/pat, drivers/media/ivtv: move pat warn and replace WARN() with pr_warn()

2015-06-24 Thread Luis R. Rodriguez
From: "Luis R. Rodriguez" On built-in kernels this warning will always splat as this is part of the module init. Fix that by shifting the PAT requirement check out under the code that does the "quasi-probe" for the device. This device driver relies on an existing driver to find its own devices, i

[PATCH v2 1/2] x86/mm/pat, drivers/infiniband/ipath: replace WARN() with pr_warn()

2015-06-24 Thread Luis R. Rodriguez
From: "Luis R. Rodriguez" WARN() may confuse users, fix that. ipath_init_one() is part the device's probe so this would only be triggered if a corresponding device was found. Signed-off-by: Luis R. Rodriguez --- drivers/infiniband/hw/ipath/ipath_driver.c | 6 -- 1 file changed, 4 insertion

[PATCH v2 0/2] x86/mm/pat: modify nopat requirement warning

2015-06-24 Thread Luis R. Rodriguez
From: "Luis R. Rodriguez" The 0-day robot found that the notpat requirement warning was being triggered on the ivtv driver on the module init path, that will always trigger on built-in devices. We want that warning to trigger only if real hardware is found so this moves the ivtv warning out under

[PATCH 5/5] [media] adv7842: Deliver resolution change events to userspace

2015-06-24 Thread Lars-Peter Clausen
Use the new v4l2_subdev_notify_event() helper function to deliver the resolution change event to userspace via the v4l2 subdev event queue as well as to the bridge driver using the callback notify mechanism. This allows userspace applications to react to changes in resolution. This is useful and o

[PATCH 2/5] [media] adv7842: Add support for control event notifications

2015-06-24 Thread Lars-Peter Clausen
Allow userspace applications to subscribe to control change events. This can e.g. be used to monitor the 5V detect control to be notified when a source is connected or disconnected. Signed-off-by: Lars-Peter Clausen --- drivers/media/i2c/adv7842.c | 5 - 1 file changed, 4 insertions(+), 1 de

[PATCH 4/5] [media] adv7604: Deliver resolution change events to userspace

2015-06-24 Thread Lars-Peter Clausen
Use the new v4l2_subdev_notify_event() helper function to deliver the resolution change event to userspace via the v4l2 subdev event queue as well as to the bridge driver using the callback notify mechanism. This allows userspace applications to react to changes in resolution. This is useful and o

[PATCH 3/5] [media] Add helper function for subdev event notifications

2015-06-24 Thread Lars-Peter Clausen
Add a new helper function called v4l2_subdev_notify_event() which will deliver the specified event to both the v4l2 subdev event queue as well as to the notify callback. The former is typically used by userspace applications to listen to notification events while the later is used by bridge drivers

[PATCH 1/5] [media] adv7604: Add support for control event notifications

2015-06-24 Thread Lars-Peter Clausen
Allow userspace applications to subscribe to control change events. This can e.g. be used to monitor the 5V detect control to be notified when a source is connected or disconnected. Signed-off-by: Lars-Peter Clausen --- drivers/media/i2c/adv7604.c | 5 - 1 file changed, 4 insertions(+), 1 de

RE: [patch] [media] v4l: xilinx: missing error code

2015-06-24 Thread Hyun Kwon
Hi Dan, Thanks for the patch. > -Original Message- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Wednesday, June 24, 2015 7:29 AM > To: Hyun Kwon > Cc: Laurent Pinchart; Mauro Carvalho Chehab; Michal Simek; Soren Brinkmann; > linux-media@vger.kernel.org; kernel-janit...@

Re: [PATCH 02/13] dmaengine: Introduce dma_request_slave_channel_compat_reason()

2015-06-24 Thread Vinod Koul
On Mon, Jun 22, 2015 at 02:31:00PM +0300, Peter Ujfalusi wrote: > On 06/12/2015 03:58 PM, Vinod Koul wrote: > > Sorry this slipped thru > > I was away for a week anyways ;) > > > Thinking about it again, I think we should coverge to two APIs and mark the > > legacy depracuated and look to convert

[PATCH 11/12] [media] tsin: c8sectpfe: Add Kconfig and Makefile for the driver.

2015-06-24 Thread Peter Griffin
This patch adds the Kconfig and Makefile for the c8sectpfe driver so it will be built. It also selects additional demodulator and tuners which are required by the supported NIM cards. Signed-off-by: Peter Griffin --- drivers/media/Kconfig | 1 + drivers/media/Makefile

[PATCH 00/12] Add c8sectpfe LinuxDVB demux driver

2015-06-24 Thread Peter Griffin
Hi Maruro, This patchset adds support for a LinuxDVB demux driver for the ST STB stih407 family SoC's. It is what I spoke to you about when we met at ELC-E in Dusseldorf last year. One advantage of having a upstream demux driver implementation for ST SoC's is that it will be easier to add support

[PATCH 06/12] ARM: DT: STi: STiH407: Add c8sectpfe LinuxDVB DT node.

2015-06-24 Thread Peter Griffin
This patch adds in the required DT node for the c8sectpfe Linux DVB demux driver which allows the tsin channels to be used on an upstream kernel. Signed-off-by: Peter Griffin --- arch/arm/boot/dts/stihxxx-b2120.dtsi | 50 1 file changed, 50 insertions(+) dif

[PATCH 08/12] [media] tsin: c8sectpfe: Add LDVB helper functions.

2015-06-24 Thread Peter Griffin
These functions are used by the core code for creating the LDVB devices and adapter. Addtionally some older SoC's (and potentially newer ones) have different frontend HW which would allow those devices to be easily supported in the future by keeping the code specific to the IP separate from the mo

[PATCH 10/12] [media] tsin: c8sectpfe: Add c8sectpfe debugfs support.

2015-06-24 Thread Peter Griffin
Some basic debugfs support to dump the IP registers. Further statistics could easily be added in the future for example for each enabled tsin channel we could expose number of corrupt packets received etc. Signed-off-by: Peter Griffin --- drivers/media/tsin/c8sectpfe/c8sectpfe-debugfs.c | 271 ++

[PATCH 09/12] [media] tsin: c8sectpfe: Add support for various ST NIM cards.

2015-06-24 Thread Peter Griffin
This patch adds support for the following 3 NIM cards: - 1) STV0367-NIM (stv0367 demod with Thompson PLL) 2) B2100A (2x stv0367 demods & 2x NXP tda18212 tuners) 3) STV0903-6110NIM (stv0903 demod + 6110 tuner, lnb24) Signed-off-by: Peter Griffin --- drivers/media/tsin/c8sectpfe/c8sectpfe-dvb.c |

[PATCH 07/12] [media] tsin: c8sectpfe: STiH407/10 Linux DVB demux support

2015-06-24 Thread Peter Griffin
This patch adds support for the c8sectpfe input HW found on STiH407/410 SoC's. It currently supports the TS input block, memdma engine and hw PID filtering blocks of the C8SECTPFE subsystem. The driver creates one LinuxDVB adapter, and a demux/dvr/frontend set of devices for each tsin channel whi

[PATCH 02/12] [media] dvb-pll: Add support for THOMSON DTT7546X tuner.

2015-06-24 Thread Peter Griffin
This is used in conjunction with the STV0367 demodulator on the STV0367-NIM-V1.0 NIM card which can be used with the STi STB SoC's. This tuner has a fifth register, so some changes have been made to accommodate this. Signed-off-by: Peter Griffin --- drivers/media/dvb-frontends/dvb-pll.c | 74 ++

[PATCH 12/12] MAINTAINERS: Add c8sectpfe driver directory to STi section

2015-06-24 Thread Peter Griffin
Add the new c8sectpfe demux driver to the STi section of the MAINTAINERS file. Signed-off-by: Peter Griffin --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index d8afd29..49c8963 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1481,6 +1481,7 @@ F:

[PATCH 04/12] [media] stv0367: Add support for 16Mhz reference clock

2015-06-24 Thread Peter Griffin
The B2100A dvb NIM card from ST has 2x stv0367 demodulators and 2x TDA18212 silicon tuners, with a 16Mhz crystal. To get this working properly with the upstream driver we need to add support for the 16Mhz reference clock. Signed-off-by: Peter Griffin --- drivers/media/dvb-frontends/stv0367.c | 5

[PATCH 03/12] [media] stv0367: Refine i2c error trace to include i2c address

2015-06-24 Thread Peter Griffin
When using stv0367 demodulator with STi STB platforms, we can have easily have four or more stv0367 demods running in the system at one time. As typically the b2120 reference design ships with a b2004a daughter board, which can accept two dvb NIM cards, and each b2100A NIM has 2x stv0367 demods an

[PATCH 05/12] [media] tsin: c8sectpfe: Add DT bindings documentation for c8sectpfe driver.

2015-06-24 Thread Peter Griffin
This patch adds the DT bindings documentation for the c8sectpfe LinuxDVB demux driver whose IP is in the STiH407 family silicon SoC's. Signed-off-by: Peter Griffin --- .../bindings/media/stih407-c8sectpfe.txt | 90 ++ include/dt-bindings/media/c8sectpfe.h

[PATCH 01/12] ARM: DT: STi: stihxxx-b2120: Add pulse-width properties to ssc2 & ssc3

2015-06-24 Thread Peter Griffin
Adding these properties makes the I2C bus to the demodulators much more reliable, and we no longer suffer from I2C errors when tuning. Signed-off-by: Peter Griffin --- arch/arm/boot/dts/stihxxx-b2120.dtsi | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/arch/arm/boo

[patch] [media] v4l: xilinx: missing error code

2015-06-24 Thread Dan Carpenter
We should set "ret" on this error path instead of returning success. Fixes: df3305156f98 ('[media] v4l: xilinx: Add Xilinx Video IP core') Signed-off-by: Dan Carpenter diff --git a/drivers/media/platform/xilinx/xilinx-dma.c b/drivers/media/platform/xilinx/xilinx-dma.c index 98e50e4..e779c93 100

[RFC PATCH v2 0/2] utils/v4l2-ctl/v4l2-ctl-modes.cpp

2015-06-24 Thread Prashant Laddha
Posting v2 again. Change compared to v1: Updated function description that was missed in v1 Prashant Laddha (2): v4l2-utils: add support for reduced fps in cvt modeline v4l2-utils: extend set-dv-timings to support reduced fps utils/v4l2-ctl/v4l2-ctl-modes.cpp | 7 ++- utils/v4l2-ctl/v4

[RFC PATCH v2 2/2] v4l2-utils: extend set-dv-timings to support reduced fps

2015-06-24 Thread Prashant Laddha
Extended command line option for set-dv-timings to support timings calculations for reduced fps. This will allow supporting NTSC frame rates like 29.97 or 59.94. Cc: Hans Verkuil Signed-off-by: Prashant Laddha --- utils/v4l2-ctl/v4l2-ctl-stds.cpp | 14 -- 1 file changed, 12 insertio

[RFC PATCH v2 1/2] v4l2-utils: add support for reduced fps in cvt modeline

2015-06-24 Thread Prashant Laddha
Added reduced fps option in cvt timings calculation. In this case, pixel clock is slowed down by a factor of 1000 / 1001 and all other timing parameters are unchanged. With reduced fps option one could generate timings for refresh rates like 29.97 or 59.94. Pixel clock in this case needs better pre

Re: [RFC PATCH v2 0/2] Support for reduced fps in v4l2-utils

2015-06-24 Thread Prashant Laddha (prladdha)
Please ignore v2 patches. By mistake I posted v1 again. On 24/06/15 7:22 pm, "Prashant Laddha (prladdha)" wrote: >Change compared to v1: >Updated function description that was missed in v1. > >Prashant Laddha (2): > v4l2-utils: add support for reduced fps in cvt modeline > v4l2-utils: extend s

[RFC PATCH v2 0/2] Support for reduced fps in v4l2-utils

2015-06-24 Thread Prashant Laddha
Change compared to v1: Updated function description that was missed in v1. Prashant Laddha (2): v4l2-utils: add support for reduced fps in cvt modeline v4l2-utils: extend set-dv-timings to support reduced fps utils/v4l2-ctl/v4l2-ctl-modes.cpp | 6 +- utils/v4l2-ctl/v4l2-ctl-stds.cpp |

[RFC PATCH v2 2/2] v4l2-utils: extend set-dv-timings to support reduced fps

2015-06-24 Thread Prashant Laddha
Extended command line option for set-dv-timings to support timings calculations for reduced fps. This will allow supporting NTSC frame rates like 29.97 or 59.94. Cc: Hans Verkuil Signed-off-by: Prashant Laddha --- utils/v4l2-ctl/v4l2-ctl-stds.cpp | 14 -- 1 file changed, 12 insertio

[RFC PATCH v2 1/2] v4l2-utils: add support for reduced fps in cvt modeline

2015-06-24 Thread Prashant Laddha
Added reduced fps option in cvt timings calculation. In this case, pixel clock is slowed down by a factor of 1000 / 1001 and all other timing parameters are unchanged. With reduced fps option one could generate timings for refresh rates like 29.97 or 59.94. Pixel clock in this case needs better pre

Re: [PATCH v10.1] media: Add registration helpers for V4L2 flash sub-devices

2015-06-24 Thread Sakari Ailus
On Fri, Jun 19, 2015 at 09:31:47AM +0200, Jacek Anaszewski wrote: > This patch adds helper functions for registering/unregistering > LED Flash class devices as V4L2 sub-devices. The functions should > be called from the LED subsystem device driver. In case the > support for V4L2 Flash sub-devices i

Re: [PATCH v10.1] leds: aat1290: add support for V4L2 Flash sub-device

2015-06-24 Thread Sakari Ailus
On Fri, Jun 19, 2015 at 09:33:22AM +0200, Jacek Anaszewski wrote: > Add support for V4L2 Flash sub-device to the aat1290 LED Flash class > driver. The support allows for V4L2 Flash sub-device to take the control > of the LED Flash class device. > > Signed-off-by: Jacek Anaszewski > Acked-by: Kyun

Re: [PATCH] leds: aat1290: Add 'static' modifier to init_mm_current_scale

2015-06-24 Thread Sakari Ailus
On Fri, Jun 19, 2015 at 09:32:44AM +0200, Jacek Anaszewski wrote: > Fix sparse warning by adding static modifier to the function > init_mm_current_scale. > > Signed-off-by: Jacek Anaszewski Acked-by: Sakari Ailus -- Sakari Ailus e-mail: sakari.ai...@iki.fi XMPP: sai...@retiisi.org.uk -- T

Re: [PATCH 3/7] [media] use CONFIG_PM_SLEEP for suspend/resume

2015-06-24 Thread Maxime Coquelin
On 06/24/2015 12:49 PM, Mauro Carvalho Chehab wrote: Using CONFIG_PM_SLEEP suppress the warnings when the driver is compiled without PM sleep functions: drivers/media/rc/st_rc.c:338:12: warning: ‘st_rc_suspend’ defined but not used [-Wunused-function] drivers/media/rc/st_rc.c:359:12: warning:

[PATCH v2] [media] lmedm04: fix the range for relative measurements

2015-06-24 Thread Mauro Carvalho Chehab
Relative measurements are typically between 0 and 0x. However, for some tuners (TUNER_S7395 and TUNER_S0194), the range were from 0 to 0xff00, with means that 100% is never archived. Also, TUNER_RS2000 uses a more complex math. So, create a macro that does the conversion using bit operations a

[PATCH 4/7] [media] saa7134: fix page size on some archs

2015-06-24 Thread Mauro Carvalho Chehab
On some archs, like tile, the PAGE_SIZE is not 4K. In the case of tile arch, it can be either 16KB or 64KB. Due to that, a warning is produced: drivers/media/pci/saa7134/saa7134.h:678:43: warning: large integer implicitly truncated to unsigned type [-Woverflow] This is actually an error,

[PATCH 1/7] [media] si470x: cleanup define namespace

2015-06-24 Thread Mauro Carvalho Chehab
Some architectures already use CHIPID defines: drivers/media/radio/si470x/radio-si470x.h:57:0: warning: "CHIPID" redefined [enabled by default] drivers/media/radio/si470x/radio-si470x.h:57:0: warning: "CHIPID" redefined [enabled by default] drivers/media/radio/si470x/radi

[PATCH 3/7] [media] use CONFIG_PM_SLEEP for suspend/resume

2015-06-24 Thread Mauro Carvalho Chehab
Using CONFIG_PM_SLEEP suppress the warnings when the driver is compiled without PM sleep functions: drivers/media/rc/st_rc.c:338:12: warning: ‘st_rc_suspend’ defined but not used [-Wunused-function] drivers/media/rc/st_rc.c:359:12: warning: ‘st_rc_resume’ defined but not used [-Wunused-function]

[PATCH 5/7] [media] omap3isp: remove unused var

2015-06-24 Thread Mauro Carvalho Chehab
drivers/media/platform/omap3isp/isppreview.c:932:6: warning: variable ‘features’ set but not used [-Wunused-but-set-variable] Signed-off-by: Mauro Carvalho Chehab diff --git a/drivers/media/platform/omap3isp/isppreview.c b/drivers/media/platform/omap3isp/isppreview.c index 15cb254ccc39..138032

[PATCH 6/7] [media] lmedm04: use u32 instead of u64 for relative stats

2015-06-24 Thread Mauro Carvalho Chehab
Cleanup this sparse warning: drivers/media/usb/dvb-usb-v2/lmedm04.c:302 lme2510_update_stats() warn: should '((255 - st->signal_sn - 161) * 3) << 8' be a 64 bit type? Both c_tmp and s_tmp actually stores a u16 stat. Using a u64 data there is a waste, specially on u32 archs, as 64 ints the

[PATCH 7/7] [media] lmedm04: fix the range for relative measurements

2015-06-24 Thread Mauro Carvalho Chehab
Relative measurements are typically between 0 and 0x. However, for some tuners (TUNER_S7395 and TUNER_S0194), the range were from 0 to 0xff00, with means that 100% is never archived. Also, TUNER_RS2000 uses a more complex math. So, create a macro that does the conversion using bit operations a

[PATCH 2/7] [media] tuner-i2c: be consistent with I2C declaration

2015-06-24 Thread Mauro Carvalho Chehab
On alpha, gcc warns a log about signed/unsigned ballance, with produces 3185 warnings. Ok, this is bogus, but it indicates that the declaration at V4L2 side is not consistent with the one at I2C. With this trivial patch, the number of errors reduce to 2959 warnings. Still too much, but it is 7.1%

Re: userspace media util repository

2015-06-24 Thread Mauro Carvalho Chehab
Em Wed, 24 Jun 2015 14:58:12 +0900 "Kim, Heung Jun" escreveu: > Thank you for reply, Guennadi. > > I'm not sure of it. In my memory it was capable of making "ps" file shows > the relationship between entities of media devices. And, in my quick glance > I can not find that codes in it (yet). > >

[PATCH] v4l2-utils: use boolean for interlaced flag

2015-06-24 Thread Prashant Laddha
This change does not affect functionality. A minor change so that the options that flags are captured as booleans and look consistent with other flags. Cc: Hans Verkuil Signed-off-by: Prashant Laddha --- utils/v4l2-ctl/v4l2-ctl-stds.cpp | 13 ++--- 1 file changed, 6 insertions(+), 7 del