cron job: media_tree daily build: ERRORS

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: Tue Nov 1 05:00:15 CET 2016 media-tree git hash:bd676c0c04ec94bd830b9192e2c33f2c4532278d media_build

Notes on V4L2 async discussion

Hi folks, Here are my notes on the V4L2 async development discussion we had a couple of days ago. Philipp Zabel, Niklas Söderlund, Laurent Pinchart and myself were present. Background -- The V4L2 async framework provides all-important support for delaying the registration of the

Re: [PATCH v4] media: Driver for Toshiba et8ek8 5MP sensor

Hi Pavel, (Cc Laurent.) On Sun, Oct 23, 2016 at 10:40:01PM +0200, Pavel Machek wrote: > Hi! > > > Thanks, this answered half of my questions already. ;-) > > > > Do all the modes work for you currently btw.? > > Aha, went through my notes. This is what it does in 5MP mode, even on > v4.9: >

Re: [PATCH v4] media: Driver for Toshiba et8ek8 5MP sensor

Hi Pavel, On Sun, Oct 23, 2016 at 10:33:15PM +0200, Pavel Machek wrote: > Hi! > > > Thanks, this answered half of my questions already. ;-) > > :-). > > I'll have to go through the patches, et8ek8 driver is probably not > enough to get useful video. platform/video-bus-switch.c is needed for >

Re: [PATCH v4] media: et8ek8: add device tree binding documentation

Hi Rob and Pavel, On Sun, Oct 30, 2016 at 03:41:34PM -0500, Rob Herring wrote: > On Sun, Oct 23, 2016 at 09:17:06PM +0200, Pavel Machek wrote: > > > > Add device tree binding documentation for toshiba et8ek8 sensor. > > > > Signed-off-by: Ivaylo Dimitrov > >

Re: [PATCH 5/9] [media] redrat3: enable carrier reports using wideband receiver

The wideband receiver is a little awkward on the redrat3. Data arrives on a different endpoint, and the learning command must be reissued every time data is learned. Signed-off-by: Sean Young --- drivers/media/rc/redrat3.c | 186 ++--- 1

Re: [PATCH 14/82] media: usb: uvc: make use of new usb_endpoint_maxp_mult()

Hi Felipe, Thank you for the patch. On Monday 31 Oct 2016 12:48:06 Felipe Balbi wrote: > We have introduced a helper to calculate multiplier > value from wMaxPacketSize. Start using it. > > Cc: Laurent Pinchart > Cc: Mauro Carvalho Chehab

Re: [PATCH 2/2] mm: remove get_user_pages_locked()

On Mon, Oct 31, 2016 at 06:55:33PM +0100, Paolo Bonzini wrote: > > 2. There is currently only one caller of get_user_pages_locked() in > >mm/frame_vector.c which seems to suggest this function isn't widely > >used/known. > > Or not widely necessary. :) Well, quite :) > > > 3. This change

Re: [PATCH 5/9] [media] redrat3: enable carrier reports using wideband receiver

Hi Sean, [auto build test WARNING on linuxtv-media/master] [also build test WARNING on next-20161028] [cannot apply to v4.9-rc3] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto

Re: [PATCH 2/2] mm: remove get_user_pages_locked()

On 31/10/2016 14:48, Lorenzo Stoakes wrote: > On Mon, Oct 31, 2016 at 12:45:36PM +0100, Paolo Bonzini wrote: >> >> >> On 31/10/2016 11:02, Lorenzo Stoakes wrote: >>> - * >>> - * get_user_pages should be phased out in favor of >>> - * get_user_pages_locked|unlocked or get_user_pages_fast. Nothing

[PATCH 5/9] [media] redrat3: enable carrier reports using wideband receiver

The wideband receiver is a little awkward on the redrat3. Data arrives on a different endpoint, and the learning command must be reissued every time data is learned. Signed-off-by: Sean Young --- drivers/media/rc/redrat3.c | 184 ++--- 1

[PATCH 9/9] [media] lirc: use-after free while reading from device and unplugging

Many lirc drivers have their own receive buffers which are freed on unplug (e.g. ir_lirc_unregister). This means that ir->buf->wait_poll will be freed directly after unplug so do not remove yourself from the wait queue. Signed-off-by: Sean Young --- drivers/media/rc/lirc_dev.c |

[PATCH 4/9] [media] redrat3: fix error paths in probe

If redrat3_delete() is called, ensure ep_in and udev members are set up so we don't dereference null in the error path. Also ensure that rc dev device exists before we enable the receiver and that the led urb exists before we create the led device. Signed-off-by: Sean Young ---

[PATCH 3/9] [media] redrat3: remove dead code and pointless messages

No need to log kmalloc failures. Signed-off-by: Sean Young --- drivers/media/rc/redrat3.c | 42 ++ 1 file changed, 6 insertions(+), 36 deletions(-) diff --git a/drivers/media/rc/redrat3.c b/drivers/media/rc/redrat3.c index de40e58..23180ec

[PATCH 1/9] [media] winbond-cir: use name without space for pnp driver

Rename the pnp driver in sysfs from /sys/bus/pnp/drivers/Winbond CIR to /sys/bus/pnp/drivers/winbond-cir Signed-off-by: Sean Young --- drivers/media/rc/winbond-cir.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/rc/winbond-cir.c

[PATCH 6/9] [media] redrat3: increase set size for lengths to maximum

In learning mode, you can get much longer messages which can run out of lengths. The usb message will slightly larger. Signed-off-by: Sean Young --- drivers/media/rc/redrat3.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/rc/redrat3.c

[PATCH 7/9] [media] lirc: might sleep error in lirc_dev_fop_read

[ 101.457944] [ cut here ] [ 101.457954] WARNING: CPU: 3 PID: 1819 at kernel/sched/core.c:7708 __might_sleep+0x7e/0x80 [ 101.457960] do not call blocking ops when !TASK_RUNNING; state=1 set at [] lirc_dev_fop_read+0x292/0x4e0 [lirc_dev] Signed-off-by: Sean Young

[PATCH 8/9] [media] lirc: prevent use-after free

If you unplug an lirc device while reading from it, you will get an use after free as the cdev is freed while still in use. Signed-off-by: Sean Young --- drivers/media/rc/lirc_dev.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git

[PATCH 0/9] Various IR fixes

Teach the redrat3 driver how to use the wideband receiver, also fix some very nasty crashes if you disconnect a lirc device while reading from it. Sean Young (9): [media] winbond-cir: use name without space for pnp driver [media] redrat3: don't include vendor/product id in name [media]

[PATCH 2/9] [media] redrat3: don't include vendor/product id in name

No need to duplicate these in the rc name. Signed-off-by: Sean Young --- drivers/media/rc/redrat3.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/media/rc/redrat3.c b/drivers/media/rc/redrat3.c index 3b0ed1c..de40e58 100644 ---

Re: [PATCH v2 5/7] [media] ir-lirc-codec: don't wait any transmitting time for tx only devices

Hi David, Andi, On Mon, Oct 31, 2016 at 02:31:52PM +, David Härdeman wrote: > October 27, 2016 4:36 PM, "Sean Young" wrote: > > Since we have to be able to switch between waiting and not waiting, > > we need some sort of ABI for this. I think this warrants a new ioctl; > > I'm

Re: [PATCH v2 5/7] [media] ir-lirc-codec: don't wait any transmitting time for tx only devices

October 27, 2016 4:36 PM, "Sean Young" wrote: > Since we have to be able to switch between waiting and not waiting, > we need some sort of ABI for this. I think this warrants a new ioctl; > I'm not sure how else it can be done. I'll be sending out a patch > shortly. Hi Sean, have

Re: [PATCH 2/2] mm: remove get_user_pages_locked()

On Mon, Oct 31, 2016 at 12:45:36PM +0100, Paolo Bonzini wrote: > > > On 31/10/2016 11:02, Lorenzo Stoakes wrote: > > - * > > - * get_user_pages should be phased out in favor of > > - * get_user_pages_locked|unlocked or get_user_pages_fast. Nothing > > - * should use get_user_pages because it

Problem with uvcvideo timestamps

Hi, I'm tracking down a problem in Chrome, where video streams captured from a Logitech c930e camera get bogus timestamps. Chrome started using camera timestamps on linux a few months ago. I've noted commit

Re: [PATCH 2/2] mm: remove get_user_pages_locked()

On 31/10/2016 11:02, Lorenzo Stoakes wrote: > - * > - * get_user_pages should be phased out in favor of > - * get_user_pages_locked|unlocked or get_user_pages_fast. Nothing > - * should use get_user_pages because it cannot pass > - * FAULT_FLAG_ALLOW_RETRY to handle_mm_fault. This comment

[PATCH 14/82] media: usb: uvc: make use of new usb_endpoint_maxp_mult()

We have introduced a helper to calculate multiplier value from wMaxPacketSize. Start using it. Cc: Laurent Pinchart Cc: Mauro Carvalho Chehab Cc: Signed-off-by: Felipe Balbi ---

[PATCH 13/82] media: usbtv: core: make use of new usb_endpoint_maxp_mult()

We have introduced a helper to calculate multiplier value from wMaxPacketSize. Start using it. Cc: Mauro Carvalho Chehab Cc: Signed-off-by: Felipe Balbi --- drivers/media/usb/usbtv/usbtv-core.c | 7 +-- 1 file

[PATCH 30/82] media: usb: uvc: remove unnecessary & operation

Now that usb_endpoint_maxp() only returns the lowest 11 bits from wMaxPacketSize, we can remove the & operation from this driver. Cc: Laurent Pinchart Cc: Mauro Carvalho Chehab Cc: Acked-by: Laurent Pinchart

Lower exposure times than 100 us?

Hello, V4L2_CID_EXPOSURE_ABSOLUTE uses a 100 us resolution. Quite a few sensors offer lower exposures and more fine resolutions than that. Can the resolution be switched to e.g. 1 us, similar to how the tuner frequency may be specified in smaller steps if the driver implements the

StarTech SVID2USB2 suport

Hello, I bought a usb2 video capture card which is not supported yet : https://www.startech.com/AV/Converters/Video/USB-S-Video-and-Composite-Video-Capture-Device-Cable-with-Audio~SVID2USB2 I posted the pictures here : https://linuxtv.org/wiki/index.php/File:Startech_svid2usb2_board_back.jpg

[PATCH 2/2] mm: remove get_user_pages_locked()

get_user_pages() now has an int *locked parameter which renders get_user_pages_locked() redundant, so remove it. This patch should not introduce any functional changes. Signed-off-by: Lorenzo Stoakes --- include/linux/mm.h | 2 -- mm/frame_vector.c | 4 ++-- mm/gup.c

[PATCH 0/2] mm: remove get_user_pages_locked()

by adding an int *locked parameter to get_user_pages() callers to this function can now utilise VM_FAULT_RETRY functionality. Taken in conjunction with the patch series adding the same parameter to get_user_pages_remote() this means all slow-path get_user_pages*() functions will now have the

[PATCH 1/2] mm: add locked parameter to get_user_pages()

This patch adds an int *locked parameter to get_user_pages() to allow VM_FAULT_RETRY faulting behaviour similar to get_user_pages_[un]locked(). It additionally clears the way for get_user_pages_locked() to be removed as its sole remaining useful characteristic was to allow for VM_FAULT_RETRY

[PATCH v2 1/3] dt-bindings: mediatek: Add a binding for Mediatek JPEG Decoder

Add a DT binding documentation for Mediatek JPEG Decoder of MT2701 SoC. Signed-off-by: Rick Chang Signed-off-by: Minghsiu Tsai --- .../bindings/media/mediatek-jpeg-codec.txt | 35 ++ 1 file changed, 35

[PATCH v2 3/3] arm: dts: mt2701: Add node for Mediatek JPEG Decoder

Signed-off-by: Rick Chang Signed-off-by: Minghsiu Tsai --- This patch depends on: CCF "Add clock support for Mediatek MT2701"[1] iommu and smi "Add the dtsi node of iommu and smi for mt2701"[2] [1]

[PATCH v2 2/3] vcodec: mediatek: Add Mediatek JPEG Decoder Driver

Add v4l2 driver for Mediatek JPEG Decoder Signed-off-by: Rick Chang Signed-off-by: Minghsiu Tsai --- drivers/media/platform/Kconfig | 15 + drivers/media/platform/Makefile |2 +

[PATCH v2 0/3] Add Mediatek JPEG Decoder

This series of patches provide a v4l2 driver to control Mediatek JPEG hw for decoding JPEG image and Motion JPEG bitstream. changes since v1: - Rebase for v4.9-rc1. - Update Compliance test version and result - Remove redundant path in Makefile - Fix potential build error without

Re: [PATCH] [media] lirc: introduce LIRC_SET_TRANSMITTER_WAIT ioctl

Hi Sean. > > > ret *= sizeof(unsigned int); > > > > > > - /* > > > - * The lircd gap calculation expects the write function to > > > - * wait for the actual IR signal to be transmitted before > > > - * returning. > > > - */ > > > - towait = ktime_us_delta(ktime_add_us(start, duration),