"BUG: unable to handle kernel NULL pointer dereference" in em28xx driver

2018-04-29 Thread Alexandre-Xavier Labonté-Lamoureux
Hi, This is another report for a crash that happened right after I disconnected a device. It froze the computer with the call trace displayed on the screen and the message "BUG: unable to handle kernel NULL pointer dereference at 0139". The call trace shows that it happened inside

"BUG: unable to handle kernel paging request" with em28xx driver

2018-04-29 Thread Alexandre-Xavier Labonté-Lamoureux
Hi, I believe this happened right after I disconnected the device. It froze the computer with the call trace on the screen. All the relevant information from /var/log/kern.log can be found below: Apr 29 23:02:32 limus kernel: [0.00] Linux version 4.17.0-rc1+ (ax@limus) (gcc version

cron job: media_tree daily build: OK

2018-04-29 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: Mon Apr 30 05:00:20 CEST 2018 media-tree git hash:a2b2eff6ac2716f499defa590a6ec4ba379d765e media_build

Re: [RFC PATCH] media: i2c: add SCCB helpers

2018-04-29 Thread Sakari Ailus
Hi Mita-san, On Fri, Apr 27, 2018 at 01:13:32AM +0900, Akinobu Mita wrote: > (This patch is in prototype stage) > > This adds SCCB helper functions (sccb_read_byte and sccb_write_byte) that > are intended to be used by some of Omnivision sensor drivers. > > The ov772x driver is going to use

[PATCH v4 14/14] media: ov772x: create subdevice device node

2018-04-29 Thread Akinobu Mita
Set the V4L2_SUBDEV_FL_HAS_DEVNODE flag for the subdevice so that the subdevice device node is created. Cc: Jacopo Mondi Cc: Laurent Pinchart Cc: Hans Verkuil Cc: Sakari Ailus

[PATCH v4 06/14] media: ov772x: use generic names for reset and powerdown gpios

2018-04-29 Thread Akinobu Mita
The ov772x driver uses "rstb-gpios" and "pwdn-gpios" for reset and powerdown pins. However, using generic names for these gpios is preferred. ("reset-gpios" and "powerdown-gpios" respectively) There is only one mainline user for these gpios, so rename to generic names. Cc: Jacopo Mondi

[PATCH v4 12/14] media: ov772x: avoid accessing registers under power saving mode

2018-04-29 Thread Akinobu Mita
The set_fmt() in subdev pad ops, the s_ctrl() for subdev control handler, and the s_frame_interval() in subdev video ops could be called when the device is under power saving mode. These callbacks for ov772x driver cause updating H/W registers that will fail under power saving mode. This avoids

[PATCH v4 11/14] media: ov772x: use v4l2_ctrl to get current control value

2018-04-29 Thread Akinobu Mita
The ov772x driver provides three V4L2 controls and the current value of each control is saved as a variable in the private data structure. We don't need to keep track of the current value by ourself, if we use v4l2_ctrl returned from v4l2_ctrl_new_std() instead. This is a preparatory change to

[PATCH v4 05/14] media: ov772x: add media controller support

2018-04-29 Thread Akinobu Mita
Create a source pad and set the media controller type to the sensor. Cc: Jacopo Mondi Cc: Laurent Pinchart Cc: Hans Verkuil Cc: Sakari Ailus Cc: Mauro Carvalho Chehab

[PATCH v4 09/14] media: ov772x: handle nested s_power() calls

2018-04-29 Thread Akinobu Mita
Depending on the v4l2 driver, calling s_power() could be nested. So the actual transitions between power saving mode and normal operation mode should only happen at the first power on and the last power off. This adds an s_power() nesting counter and updates the power state if the counter is

[PATCH v4 08/14] media: ov772x: support device tree probing

2018-04-29 Thread Akinobu Mita
The ov772x driver currently only supports legacy platform data probe. This change enables device tree probing. Note that the platform data probe can select auto or manual edge control mode, but the device tree probling can only select auto edge control mode for now. Cc: Jacopo Mondi

[PATCH v4 07/14] media: ov772x: omit consumer ID when getting clock reference

2018-04-29 Thread Akinobu Mita
Currently the ov772x driver obtains a clock with a specific consumer ID. As there's a single clock for this driver, we could omit clock-names property in device tree by passing NULL as a consumer ID to clk_get(). Cc: Jacopo Mondi Cc: Laurent Pinchart

[PATCH v4 03/14] media: ov772x: allow i2c controllers without I2C_FUNC_PROTOCOL_MANGLING

2018-04-29 Thread Akinobu Mita
The ov772x driver only works when the i2c controller have I2C_FUNC_PROTOCOL_MANGLING. However, many i2c controller drivers don't support it. The reason that the ov772x requires I2C_FUNC_PROTOCOL_MANGLING is that it doesn't support repeated starts. This changes the reading ov772x register method

[PATCH v4 13/14] media: ov772x: make set_fmt() return -EBUSY while streaming

2018-04-29 Thread Akinobu Mita
The ov772x driver is going to offer a V4L2 sub-device interface, so changing the output data format on this sub-device can be made anytime. However, the request is preferred to fail while the video stream on the device is active. Cc: Jacopo Mondi Cc: Laurent Pinchart

[PATCH v4 02/14] media: ov772x: correct setting of banding filter

2018-04-29 Thread Akinobu Mita
The banding filter ON/OFF is controlled via bit 5 of COM8 register. It is attempted to be enabled in ov772x_set_params() by the following line. ret = ov772x_mask_set(client, COM8, BNDF_ON_OFF, 1); But this unexpectedly results disabling the banding filter, because the mask and set bits

[PATCH v4 04/14] media: ov772x: add checks for register read errors

2018-04-29 Thread Akinobu Mita
This change adds checks for register read errors and returns correct error code. Cc: Jacopo Mondi Cc: Laurent Pinchart Cc: Hans Verkuil Cc: Sakari Ailus Cc: Mauro Carvalho

[PATCH v4 10/14] media: ov772x: reconstruct s_frame_interval()

2018-04-29 Thread Akinobu Mita
This splits the s_frame_interval() in subdev video ops into selecting the frame interval and setting up the registers. This is a preparatory change to avoid accessing registers under power saving mode. Cc: Jacopo Mondi Cc: Laurent Pinchart

[PATCH v4 01/14] media: dt-bindings: ov772x: add device tree binding

2018-04-29 Thread Akinobu Mita
This adds a device tree binding documentation for OV7720/OV7725 sensor. Cc: Jacopo Mondi Cc: Laurent Pinchart Cc: Hans Verkuil Cc: Sakari Ailus Cc: Mauro Carvalho Chehab

[PATCH v4 00/14] media: ov772x: support media controller, device tree probing, etc.

2018-04-29 Thread Akinobu Mita
This patchset includes support media controller, sub-device interface, device tree probing and other miscellanuous changes for ov772x driver. * v4 (thanks to Laurent Pinchart) - Add Reviewed-by: lines - Correct setting of banding filter (New) - Omit consumer ID when getting clock reference (New)

[PATCH] media: atomisp: fix spelling mistake: "diregard" -> "disregard"

2018-04-29 Thread Colin King
From: Colin Ian King Trivial fix to spelling mistake in ia_css_print message text Signed-off-by: Colin Ian King --- .../css2400/css_2401_csi2p_system/host/csi_rx_private.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [PATCH 04/17] dma-fence: Allow wait_any_timeout for all fences

2018-04-29 Thread Christian König
Am 27.04.2018 um 08:17 schrieb Daniel Vetter: When this was introduced in commit a519435a96597d8cd96123246fea4ae5a6c90b02 Author: Christian König Date: Tue Oct 20 16:34:16 2015 +0200 dma-buf/fence: add fence_wait_any_timeout function v2 there was a