Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c

2017-02-27 Thread Ingo Molnar
* Linus Torvalds wrote: > In other words: what will happen is that distros start getting bootup problem > reports six months or a year after we've done it, and *if* they figure out > it's > the irq enabling, they'll disable it, because they have no way to solve

cron job: media_tree daily build: ERRORS

2017-02-27 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: Tue Feb 28 05:00:21 CET 2017 media-tree git hash:e6b377dbbb944d5e3ceef4e5d429fc5c841e3692 media_build

Re: Kaffeine commit b510bff2 won't compile

2017-02-27 Thread bill murphy
Hi Mauro, Thanks for looking in to it. All is well now. On a sidenote, given 700 MHz is used for LTE, and not broadcasting anymore, would you folks consider removing ch 52 thru 69 in the us-atsc-frequencies if I posted a simple patch to dtv-scan-tables? Bill On 02/27/2017 05:11 AM, Mauro

Re: [PATCH] [media] tw5864: handle unknown video std gracefully

2017-02-27 Thread Andrey Utkin
Hi Arnd, Thanks for sending this patch. On Mon, Feb 27, 2017 at 09:32:34PM +0100, Arnd Bergmann wrote: > tw5864_frameinterval_get() only initializes its output when it successfully > identifies the video standard in tw5864_input. We get a warning here because > gcc can't always track the state

Re: [PATCH] bcm2048: Fix checkpatch checks

2017-02-27 Thread Man Choy
On Mon, Feb 27, 2017 at 4:21 PM, Greg Kroah-Hartman wrote: > On Sat, Feb 18, 2017 at 11:52:37AM +0800, Man Choy wrote: >> Fix following checks: >> >> CHECK: Avoid crashing the kernel - try using WARN_ON & recovery code rather >> than BUG() or BUG_ON() >> +

Re: [PATCH 14/15] media: s5p-mfc: Use preallocated block allocator always for MFC v6+

2017-02-27 Thread Shuah Khan
On 02/27/2017 05:50 AM, Marek Szyprowski wrote: > Hi Shuah, > > On 2017-02-24 15:23, Shuah Khan wrote: >> On Thu, Feb 23, 2017 at 11:26 PM, Marek Szyprowski >> wrote: >>> On 2017-02-23 22:43, Shuah Khan wrote: On Tue, Feb 14, 2017 at 12:52 AM, Marek Szyprowski

Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c

2017-02-27 Thread Linus Torvalds
On Mon, Feb 27, 2017 at 7:41 AM, Ingo Molnar wrote: > > BTW., instead of trying to avoid the scenario, wow about moving in the other > direction: making CONFIG_DEBUG_SHIRQ=y unconditional property in the IRQ core > code > starting from v4.12 or so The problem is that it's

em28xx: new board id [1d19:6901]

2017-02-27 Thread Łukasz Strzeszkowski
Hi, I’ve found a new device which is not listed model: LogiLink VG0011 vendor/product: [1d19:6901] Dexatek Technology Ltd. mode: analog I am unable to load a driver, because there is no such vendor in driver list. dmesg output: [ 1232.506295] usb 2-4: new high-speed USB device number

Re: [PATCH 1/4] v4l2: device_register_subdev_nodes: allow calling multiple times

2017-02-27 Thread Sakari Ailus
Hi Pavel, Please find my comments below. On Sat, Feb 25, 2017 at 11:12:55PM +0100, Pavel Machek wrote: > Hi! > > > > On Mon 2017-02-20 15:56:36, Sakari Ailus wrote: > > > > On Mon, Feb 20, 2017 at 03:09:13PM +0200, Sakari Ailus wrote: > > > > > I've tested ACPI, will test DT soon... > > > > >

[PATCH] [media] tw5864: handle unknown video std gracefully

2017-02-27 Thread Arnd Bergmann
tw5864_frameinterval_get() only initializes its output when it successfully identifies the video standard in tw5864_input. We get a warning here because gcc can't always track the state if initialized warnings across a WARN() macro, and thinks it might get used incorrectly in tw5864_s_parm:

Re: [PATCH 1/4] v4l2: device_register_subdev_nodes: allow calling multiple times

2017-02-27 Thread Pavel Machek
Hi! > > > On Mon 2017-02-20 15:56:36, Sakari Ailus wrote: > > > > On Mon, Feb 20, 2017 at 03:09:13PM +0200, Sakari Ailus wrote: > > > > > I've tested ACPI, will test DT soon... > > > > > > > > DT case works, too (Nokia N9). > > > > > > Hmm. Good to know. Now to figure out how to get N900 case

Re: [PATCHv4 1/9] video: add hotplug detect notifier support

2017-02-27 Thread Russell King - ARM Linux
On Mon, Feb 27, 2017 at 06:21:05PM +0100, Hans Verkuil wrote: > On 02/27/2017 06:04 PM, Russell King - ARM Linux wrote: > > I'm afraid that I walked away from this after it became clear that there > > was little hope for any forward progress being made in a timely manner > > for multiple reasons

Re: [PATCHv4 1/9] video: add hotplug detect notifier support

2017-02-27 Thread Hans Verkuil
On 02/27/2017 06:04 PM, Russell King - ARM Linux wrote: > On Mon, Feb 27, 2017 at 05:08:41PM +0100, Daniel Vetter wrote: >> On Mon, Feb 06, 2017 at 11:29:43AM +0100, Hans Verkuil wrote: >>> From: Hans Verkuil >>> >>> Add support for video hotplug detect and EDID/ELD

Re: [PATCHv4 1/9] video: add hotplug detect notifier support

2017-02-27 Thread Russell King - ARM Linux
On Mon, Feb 27, 2017 at 05:08:41PM +0100, Daniel Vetter wrote: > On Mon, Feb 06, 2017 at 11:29:43AM +0100, Hans Verkuil wrote: > > From: Hans Verkuil > > > > Add support for video hotplug detect and EDID/ELD notifiers, which is used > > to convey information from video

Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c

2017-02-27 Thread Thomas Gleixner
On Mon, 27 Feb 2017, Tony Lindgren wrote: > * Ingo Molnar [170227 07:44]: > > Because it's not the requirement that hurts primarily, but the resulting > > non-determinism and the sporadic crashes. Which can be solved by making the > > race > > deterministic via the debug

Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c

2017-02-27 Thread Thomas Gleixner
On Mon, 27 Feb 2017, Ingo Molnar wrote: > * Thomas Gleixner wrote: > > > The pending interrupt issue happens, at least on my test boxen, mostly on > > the 'legacy' interrupts (0 - 15). But even the IOAPIC interrupts >=16 > > happen occasionally. > > > > > > - Spurious

Re: [PATCH v3 2/3] stih-cec: add HPD notifier support

2017-02-27 Thread Rob Herring
On Fri, Feb 17, 2017 at 11:46:51AM +0100, Benjamin Gaignard wrote: > By using the HPD notifier framework there is no longer any reason > to manually set the physical address. This was the one blocking > issue that prevented this driver from going out of staging, so do > this move as well. > >

Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c

2017-02-27 Thread Tony Lindgren
* Thomas Gleixner [170227 08:20]: > On Mon, 27 Feb 2017, Tony Lindgren wrote: > > * Ingo Molnar [170227 07:44]: > > > Because it's not the requirement that hurts primarily, but the resulting > > > non-determinism and the sporadic crashes. Which can be

Re: [PATCHv4 1/9] video: add hotplug detect notifier support

2017-02-27 Thread Daniel Vetter
On Mon, Feb 06, 2017 at 11:29:43AM +0100, Hans Verkuil wrote: > From: Hans Verkuil > > Add support for video hotplug detect and EDID/ELD notifiers, which is used > to convey information from video drivers to their CEC and audio counterparts. > > Based on an earlier

Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c

2017-02-27 Thread Tony Lindgren
* Ingo Molnar [170227 07:44]: > > * Thomas Gleixner wrote: > > > The pending interrupt issue happens, at least on my test boxen, mostly on > > the 'legacy' interrupts (0 - 15). But even the IOAPIC interrupts >=16 > > happen occasionally. > > > > > > -

Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c

2017-02-27 Thread Ingo Molnar
* Thomas Gleixner wrote: > The pending interrupt issue happens, at least on my test boxen, mostly on > the 'legacy' interrupts (0 - 15). But even the IOAPIC interrupts >=16 > happen occasionally. > > > - Spurious interrupts on IRQ7, which are triggered by IRQ 0 (PIT/HPET).

Re: [PATCH v4 24/36] [media] add Omnivision OV5640 sensor driver

2017-02-27 Thread Rob Herring
On Wed, Feb 15, 2017 at 06:19:26PM -0800, Steve Longerbeam wrote: > This driver is based on ov5640_mipi.c from Freescale imx_3.10.17_1.0.0_beta > branch, modified heavily to bring forward to latest interfaces and code > cleanup. > > Signed-off-by: Steve Longerbeam >

Re: [PATCH v4 15/36] platform: add video-multiplexer subdevice driver

2017-02-27 Thread Rob Herring
On Wed, Feb 15, 2017 at 06:19:17PM -0800, Steve Longerbeam wrote: > From: Philipp Zabel > > This driver can handle SoC internal and external video bus multiplexers, > controlled either by register bit fields or by a GPIO. The subdevice > passes through frame interval and

Re: [PATCH v4 01/36] [media] dt-bindings: Add bindings for i.MX media driver

2017-02-27 Thread Rob Herring
On Wed, Feb 15, 2017 at 06:19:03PM -0800, Steve Longerbeam wrote: > Add bindings documentation for the i.MX media driver. > > Signed-off-by: Steve Longerbeam > --- > Documentation/devicetree/bindings/media/imx.txt | 66 > + > 1 file changed,

[PATCH 4/9] cec: return -EPERM when no LAs are configured

2017-02-27 Thread Hans Verkuil
From: Hans Verkuil The CEC_TRANSMIT ioctl now returns -EPERM if an attempt is made to transmit a message for an unconfigured adapter (i.e. userspace never called CEC_ADAP_S_LOG_ADDRS). This differentiates this case from when LAs are configured, but no physical address is

[PATCH 3/9] cec: allow specific messages even when unconfigured

2017-02-27 Thread Hans Verkuil
From: Hans Verkuil The CEC specifications explicitly allows you to send poll messages and Image/Text View On messages to a TV, even when unconfigured (i.e. there is no hotplug signal detected). Some TVs will pull the HPD low when switching to another input, or when going

[PATCH 8/9] cec: improve cec_transmit_msg_fh logging

2017-02-27 Thread Hans Verkuil
From: Hans Verkuil Several error paths didn't log why an error was returned. Add this. Also handle the corner case of "adapter is unconfigured AND the message is from Unregistered to TV AND reply is non-zero" separately and return EINVAL in that case, since it really is

[PATCH 1/9] cec: documentation fixes

2017-02-27 Thread Hans Verkuil
From: Hans Verkuil Fixed a few spelling mistakes, but mostly incorrect rst syntax that caused wrong references or font style. No actual documentation changes, just fixes. Signed-off-by: Hans Verkuil ---

[PATCH 0/9] cec: code and doc fixes/improvements

2017-02-27 Thread Hans Verkuil
From: Hans Verkuil Besides various documentation and logging improvements, the main addition to CEC is support for a special corner case: When the physical address is invalid, it is still allowed by the CEC specification to send messages from 0xf ('Unregistered') to 0

[PATCH 9/9] cec: log reason for returning -EINVAL

2017-02-27 Thread Hans Verkuil
When validating the struct cec_s_log_addrs input a debug message is printed for all except two of the 'return -EINVAL' paths. Also log the reason for the missing two paths. Signed-off-by: Hans Verkuil --- drivers/media/cec/cec-adap.c | 8 ++-- 1 file changed, 6

[PATCH 5/9] cec: document the error codes

2017-02-27 Thread Hans Verkuil
Document all the various error codes returned by the CEC ioctls. These were never documented, instead the documentation relied on a reference to the generic error codes, but that's not sufficient. Signed-off-by: Hans Verkuil ---

[PATCH 2/9] cec: improve flushing queue

2017-02-27 Thread Hans Verkuil
From: Hans Verkuil When the adapter is unloaded or unconfigured, then all transmits and pending waits should be flushed. Move this code into its own function and improve the code that cancels delayed work to avoid having to unlock adap->lock. Signed-off-by: Hans Verkuil

[PATCH 7/9] cec: use __func__ in log messages.

2017-02-27 Thread Hans Verkuil
From: Hans Verkuil The hardcoded function name is actually wrong. Use __func__ instead. Signed-off-by: Hans Verkuil --- drivers/media/cec/cec-adap.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git

[PATCH 6/9] cec: document the special unconfigured case

2017-02-27 Thread Hans Verkuil
From: Hans Verkuil Even when the CEC device is unconfigured due to an invalid physical address it is still allowed to send a message from 0xf (Unregistered) to 0 (TV). This is a corner case explicitly allowed by the CEC specification. Document this corner case.

Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c

2017-02-27 Thread Thomas Gleixner
On Mon, 27 Feb 2017, Thomas Gleixner wrote: > On Sat, 25 Feb 2017, Linus Torvalds wrote: > > There are several things that set IRQS_PENDING, ranging from "try to > > test mis-routed interrupts while irqd was working", to "prepare for > > suspend losing the irq for us", to "irq auto-probing uses it

Re: [PATCH 14/15] media: s5p-mfc: Use preallocated block allocator always for MFC v6+

2017-02-27 Thread Marek Szyprowski
Hi Shuah, On 2017-02-24 15:23, Shuah Khan wrote: On Thu, Feb 23, 2017 at 11:26 PM, Marek Szyprowski wrote: On 2017-02-23 22:43, Shuah Khan wrote: On Tue, Feb 14, 2017 at 12:52 AM, Marek Szyprowski wrote: It turned out that all versions of

[PATCH v2] soc-camera: fix rectangle adjustment in cropping

2017-02-27 Thread Guennadi Liakhovetski
From: Koji Matsuoka update_subrect() adjusts the sub-rectangle to be inside a base area. It checks width and height to not exceed those of the area, then it checks the low border (left or top) to lie within the area, then the high border (right or bottom) to lie

Re: Kaffeine commit b510bff2 won't compile

2017-02-27 Thread Mauro Carvalho Chehab
Em Sun, 26 Feb 2017 20:57:20 -0500 bill murphy escreveu: > Hi, > Can someone double check me on this? > > It seems there might be a missing header, > in the src directory, preventing the last commit from > compiling. The commit prior compiles fine. So not that big a deal,

Re: [PATCH] soc-camera: fix rectangle adjustment in cropping

2017-02-27 Thread Guennadi Liakhovetski
On Mon, 27 Feb 2017, Laurent Pinchart wrote: > Hi Guennadi, > > On Monday 27 Feb 2017 09:54:19 Guennadi Liakhovetski wrote: > > On Mon, 27 Feb 2017, Laurent Pinchart wrote: > > > On Sunday 26 Feb 2017 21:58:16 Guennadi Liakhovetski wrote: > > >> From: Koji Matsuoka

Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c

2017-02-27 Thread Thomas Gleixner
On Sat, 25 Feb 2017, Linus Torvalds wrote: > On Sat, Feb 25, 2017 at 1:07 AM, Ingo Molnar wrote: > > > > So, should we revert the hw-retrigger change: > > > > a9b4f08770b4 x86/ioapic: Restore IO-APIC irq_chip retrigger callback > > > > ... until we managed to fix

Re: [PATCH] soc-camera: fix rectangle adjustment in cropping

2017-02-27 Thread Laurent Pinchart
Hi Guennadi, Thank you for the patch. On Sunday 26 Feb 2017 21:58:16 Guennadi Liakhovetski wrote: > From: Koji Matsuoka > > update_subrect() adjusts the sub-rectangle to be inside a base area. > It checks width and height to not exceed those of the area, then it >

Re: [PATCH] soc-camera: fix rectangle adjustment in cropping

2017-02-27 Thread Guennadi Liakhovetski
On Mon, 27 Feb 2017, Hans Verkuil wrote: > On 02/27/2017 10:02 AM, Laurent Pinchart wrote: > > Hi Guennadi, > > > > On Monday 27 Feb 2017 09:54:19 Guennadi Liakhovetski wrote: > >> On Mon, 27 Feb 2017, Laurent Pinchart wrote: > >>> On Sunday 26 Feb 2017 21:58:16 Guennadi Liakhovetski wrote: >

Re: [PATCH] soc-camera: fix rectangle adjustment in cropping

2017-02-27 Thread Laurent Pinchart
Hi Guennadi, On Monday 27 Feb 2017 10:13:53 Guennadi Liakhovetski wrote: > On Mon, 27 Feb 2017, Laurent Pinchart wrote: > > On Monday 27 Feb 2017 09:54:19 Guennadi Liakhovetski wrote: > >> On Mon, 27 Feb 2017, Laurent Pinchart wrote: > >>> On Sunday 26 Feb 2017 21:58:16 Guennadi Liakhovetski

Re: [PATCH] soc-camera: fix rectangle adjustment in cropping

2017-02-27 Thread Hans Verkuil
On 02/27/2017 10:02 AM, Laurent Pinchart wrote: > Hi Guennadi, > > On Monday 27 Feb 2017 09:54:19 Guennadi Liakhovetski wrote: >> On Mon, 27 Feb 2017, Laurent Pinchart wrote: >>> On Sunday 26 Feb 2017 21:58:16 Guennadi Liakhovetski wrote: From: Koji Matsuoka

Re: [PATCH] soc-camera: fix rectangle adjustment in cropping

2017-02-27 Thread Laurent Pinchart
Hi Guennadi, On Monday 27 Feb 2017 09:54:19 Guennadi Liakhovetski wrote: > On Mon, 27 Feb 2017, Laurent Pinchart wrote: > > On Sunday 26 Feb 2017 21:58:16 Guennadi Liakhovetski wrote: > >> From: Koji Matsuoka > >> > >> update_subrect() adjusts the sub-rectangle to

Re: [PATCH] soc-camera: fix rectangle adjustment in cropping

2017-02-27 Thread Guennadi Liakhovetski
Hi Laurent, On Mon, 27 Feb 2017, Laurent Pinchart wrote: > Hi Guennadi, > > Thank you for the patch. > > On Sunday 26 Feb 2017 21:58:16 Guennadi Liakhovetski wrote: > > From: Koji Matsuoka > > > > update_subrect() adjusts the sub-rectangle to be inside a base

Re: [PATCH] bcm2048: Fix checkpatch checks

2017-02-27 Thread Greg Kroah-Hartman
On Sat, Feb 18, 2017 at 11:52:37AM +0800, Man Choy wrote: > Fix following checks: > > CHECK: Avoid crashing the kernel - try using WARN_ON & recovery code rather > than BUG() or BUG_ON() > + BUG_ON((index+2) >= BCM2048_MAX_RDS_RT); > > CHECK: spaces preferred around that '+' (ctx:VxV) > +