Re: [PATCH v5 2/2] Add test for new features in cec-follower

On 10/3/19 5:18 AM, Jiunn Chang wrote: > Analog tuner control test tuner_ctl_test(): > - give analog tuner status > - select tuner analog service > - analog tuner step features > > Signed-off-by: Jiunn Chang > --- > utils/cec-compliance/cec-test.cpp | 74 +++ >

cron job: media_tree daily build: OK

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 Oct 3 05:00:10 CEST 2019 media-tree git hash:503e59365dd134b2c63864f14e2de0476284b003 media_build gi

[PATCH v5 2/2] Add test for new features in cec-follower

Analog tuner control test tuner_ctl_test(): - give analog tuner status - select tuner analog service - analog tuner step features Signed-off-by: Jiunn Chang --- utils/cec-compliance/cec-test.cpp | 74 +++ 1 file changed, 74 insertions(+) diff --git a/utils/cec-

[PATCH v5 0/2] cec-compliance: tuner control

This patch series implements analog tuner step features and cec-compliance tests. Added tuner_ctl_test() to test new tuner features implemented in the cec-follower. --- Changes made since v4: - Add TOT_ANALOG_FREQS macro: hverkuil - Rename function to analog_update_tuner_dev_info(): hverkui

[PATCH v5 1/2] cec-follower: add tuner step increment/decrement

Tuner step feature will select the next highest or lowest service frequency. There are a total of three possible frequencies given a broadcast type and system for a total of 81 analog channels. Opcodes implemented: - - Signed-off-by: Jiunn Chang --- utils/cec-follower/cec-follower.cpp |

Re: [GIT PULL FOR v5.5] DVB & RC (I)

From: buil...@linuxtv.org Pull request: https://patchwork.linuxtv.org/patch/59133/ Build log: https://builder.linuxtv.org/job/patchwork/18716/ Build time: 00:07:49 Link: https://lore.kernel.org/linux-media/20191002213433.ga12...@gofer.mess.org gpg: Signature made Wed 02 Oct 2019 09:26:41 PM UTC g

[GIT PULL FOR v5.5] DVB & RC (I)

Hi Mauro, Here we have a new i2c for gl861, minor cleanups for rc-core & dvb and banishing procfs from the media tree and few usb ids. Thanks, Sean The following changes since commit 503e59365dd134b2c63864f14e2de0476284b003: media: i2c: ov2659: Switch to SPDX Licensing (2019-10-01 17:39:16 -0

Re: PureThermal2 UVC video camera: Failed to submit URB 0 (-28)

On Wed, Oct 2, 2019 at 10:58 AM Alan Stern wrote: > > On Wed, 2 Oct 2019, Tim Harvey wrote: > > > On Tue, Oct 1, 2019 at 12:19 PM Alan Stern > > wrote: > > > > > > On Tue, 1 Oct 2019, Tim Harvey wrote: > > > > > > > On Thu, Sep 26, 2019 at 3:47 PM Tim Harvey > > > > wrote: > > > > > > > > > >

Re: [PATCH v2] media: uvcvideo: Add a quirk to force GEO GC6500 Camera bits-per-pixel value

Laurent, > Would it make sense to split the calculation from v4l2_fill_pixfmt() to > a helper function that the UVC driver could call ? It would of course be possible, and would benefit v4l2-common.c as it would be common between v4l2_fill_pixfmt() and v4l2_fill_pixfmt_mp(), but as I noted before

Re: [PATCH] dt-bindings: media: rc: Fix redundant string

On Wed, 2 Oct 2019 16:44:00 +0200, Maxime Ripard wrote: > The linux,rc-map-name property is described using an enum, yet a value has > been put in that enum twice, resulting in a warning. Let's fix that. > > Fixes: 7c31b9d67342 ("media: dt-bindings: media: Add YAML schemas for the > generic RC b

Re: [PATCH v2] media: uvcvideo: Add a quirk to force GEO GC6500 Camera bits-per-pixel value

Hi Sergey, On Wed, Oct 02, 2019 at 09:15:45PM +0400, Sergey Zakharchenko wrote: > Sergey Zakharchenko : > > Laurent Pinchart : > > > Do you think it would make sense to do this by default, without > > > requiring a quirk ? Or are there cases where this calculation would lead > > > to incorrect res

Re: PureThermal2 UVC video camera: Failed to submit URB 0 (-28)

On Wed, 2 Oct 2019, Tim Harvey wrote: > On Tue, Oct 1, 2019 at 12:19 PM Alan Stern wrote: > > > > On Tue, 1 Oct 2019, Tim Harvey wrote: > > > > > On Thu, Sep 26, 2019 at 3:47 PM Tim Harvey wrote: > > > > > > > > Greetings, > > > > > > > > I'm running into an issue with a USB UVC Full speed camer

[PATCH v2] media: dvbsky: add support for eyeTV Geniatech T2 lite

Adds USB ID for the eyeTV Geniatech T2 lite to the dvbsky driver. This is a Geniatech T230C based stick without IR and a different USB ID. Signed-off-by: Thomas Voegtle Tested-by: Jan Pieter van Woerkom --- Changes in v2: - fixed a typo, missed a "C" - fixed sort order of IDs in dvb-usb-ids.h

Re: [PATCH v2] media: uvcvideo: Add a quirk to force GEO GC6500 Camera bits-per-pixel value

Sergey Zakharchenko : > Laurent Pinchart : > > Do you think it would make sense to do this by default, without > > requiring a quirk ? Or are there cases where this calculation would lead > > to incorrect results while the bpp reported by the camera would be right > > ? > > The loop is a simplified

Re: PureThermal2 UVC video camera: Failed to submit URB 0 (-28)

On Tue, Oct 1, 2019 at 12:19 PM Alan Stern wrote: > > On Tue, 1 Oct 2019, Tim Harvey wrote: > > > On Thu, Sep 26, 2019 at 3:47 PM Tim Harvey wrote: > > > > > > Greetings, > > > > > > I'm running into an issue with a USB UVC Full speed camera, the > > > PureThermal2 [1] on an IMX6 based ARM board.

Re: [PATCH v2] media: uvcvideo: Add a quirk to force GEO GC6500 Camera bits-per-pixel value

Hi Laurent, Laurent Pinchart : > Do you think it would make sense to do this by default, without > requiring a quirk ? Or are there cases where this calculation would lead > to incorrect results while the bpp reported by the camera would be right > ? I don't nearly have the experience with a broa

[PATCH] dt-bindings: media: rc: Fix redundant string

The linux,rc-map-name property is described using an enum, yet a value has been put in that enum twice, resulting in a warning. Let's fix that. Fixes: 7c31b9d67342 ("media: dt-bindings: media: Add YAML schemas for the generic RC bindings") Signed-off-by: Maxime Ripard --- Documentation/devicetr

Re: [PATCH 7/7] dt-bindings: smia: Remove documentation of nokia,nvm-size

On Tue, 24 Sep 2019 14:23:38 +0300, Sakari Ailus wrote: > The nokia,nvm-size property was used to tell the size of the NVM memory > accessible through the sensor's register interface. However, while the > size isn't directly readable through the sensor's register interface, it > can be detected rea

Re: [GIT PULL for v5.5] Venus updates, take 2

Hi Mauro, On 10/2/19 5:01 PM, Mauro Carvalho Chehab wrote: > Em Wed, 2 Oct 2019 14:29:53 +0300 > Stanimir Varbanov escreveu: > >> Hi Mauro, >> >> The Venus driver updates include: >> >> * three fixes: fail to suspend, enum frameinterval issue with encoder >> and frequency table modifications fo

Re: venus: venc: Fix enum frameintervals - was: [GIT PULL for v5.5] Venus updates, take 2

Em Wed, 2 Oct 2019 14:29:53 +0300 Stanimir Varbanov escreveu: > Hi Mauro, > > The Venus driver updates include: > > * three fixes: fail to suspend, enum frameinterval issue with encoder > and frequency table modifications for v3 to handle performance issues. > * two new features: interconnect

Re: [PATCH v2] media: uvcvideo: Add a quirk to force GEO GC6500 Camera bits-per-pixel value

Hi Sergey, Thank you for the patch. On Wed, Oct 02, 2019 at 01:01:02PM +, Sergey Zakharchenko wrote: > This device does not function correctly in raw mode in kernel > versions validating buffer sizes in bulk mode. It erroneously > announces 16 bits per pixel instead of 12 for NV12 format, so

Re: [GIT PULL for v5.5] Venus updates, take 2

Em Wed, 2 Oct 2019 14:29:53 +0300 Stanimir Varbanov escreveu: > Hi Mauro, > > The Venus driver updates include: > > * three fixes: fail to suspend, enum frameinterval issue with encoder > and frequency table modifications for v3 to handle performance issues. > * two new features: interconnect

[PATCH v2] media: uvcvideo: Add a quirk to force GEO GC6500 Camera bits-per-pixel value

This device does not function correctly in raw mode in kernel versions validating buffer sizes in bulk mode. It erroneously announces 16 bits per pixel instead of 12 for NV12 format, so it needs this quirk to fix computed frame size and avoid legitimate frames getting discarded. Signed-off-by: Ser

Re: [PATCHv2 2/2] v4l2-ioctl.c: use new v4l2_fourcc_conv/args macros

On Mon, 2019-09-16 at 12:04 +0200, Hans Verkuil wrote: > Use these new standard macros to log the fourcc value in a > human readable format. > > Signed-off-by: Hans Verkuil > Acked-by: Sakari Ailus > --- > drivers/media/v4l2-core/v4l2-ioctl.c | 58 +--- > 1 file changed,

Re: [PATCHv2 1/2] v4l: Add macros for printing V4L fourcc values

On Mon, 2019-09-16 at 12:04 +0200, Hans Verkuil wrote: > From: Sakari Ailus > > Add two macros that facilitate printing V4L fourcc values with printf > family of functions. v4l2_fourcc_conv provides the printf conversion > specifier for printing formats and v4l2_fourcc_args provides the actual >

Re: [PATCH] media: uvcvideo: Add a quirk to force GEO GC6500 Camera bits-per-pixel value

Hi Sergey, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on linuxtv-media/master] [cannot apply to v5.4-rc1 next-20191002] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system. BTW, we also suggest to use '-

Re: [GIT PULL for v5.5] Venus updates, take 2

From: buil...@linuxtv.org Pull request: https://patchwork.linuxtv.org/patch/59114/ Build log: https://builder.linuxtv.org/job/patchwork/18656/ Build time: 00:02:36 Link: https://lore.kernel.org/linux-media/20191002112953.16642-1-stanimir.varba...@linaro.org gpg: Signature made Wed 02 Oct 2019 11

[GIT PULL for v5.5] Venus updates, take 2

Hi Mauro, The Venus driver updates include: * three fixes: fail to suspend, enum frameinterval issue with encoder and frequency table modifications for v3 to handle performance issues. * two new features: interconnect bandwidth support on v4 and more precise clock-scaling on v4. Please pull. Ch

Re: [ANN] v4l-utils: remove utils/cec-follower/cec-log.h after a 'git pull'

On 10/2/19 10:52 AM, Hans Verkuil wrote: > Just a heads up: I synced v4l-utils with the latest media_tree master branch, > and that required some reorganization for the CEC utilities. > > If you do a 'git pull' in an existing v4l-utils repo, then it will fail to > build due to an orphaned utils/ce

Re: [RFC,V2,07/11] media: platform: Add Mediatek ISP P1 private control

On Wed, Oct 02, 2019 at 01:55:59PM +0300, Sakari Ailus wrote: > Hi Jungo, Hans, > > On Mon, May 13, 2019 at 10:46:46AM +0200, Hans Verkuil wrote: > > On 5/10/19 3:58 AM, Jungo Lin wrote: > ... > > > +struct v4l2_ctrl_config mtk_cam_controls[] = { > > > + { > > > + .ops = &mtk_cam_dev_ctrl_ops, > >

Re: Build failed in Jenkins: v4l-utils #43

On 10/2/19 12:41 PM, Mauro Carvalho Chehab wrote: > Em Wed, 2 Oct 2019 11:49:20 +0200 > Hans Verkuil escreveu: > >> On 10/2/19 11:36 AM, Mauro Carvalho Chehab wrote: >>> Em Wed, 2 Oct 2019 10:25:02 +0200 >>> Hans Verkuil escreveu: >>> Hi Mauro, On 10/2/19 10:16 AM, Jenkins Buil

Re: [RFC,V2,07/11] media: platform: Add Mediatek ISP P1 private control

Hi Jungo, Hans, On Mon, May 13, 2019 at 10:46:46AM +0200, Hans Verkuil wrote: > On 5/10/19 3:58 AM, Jungo Lin wrote: ... > > +struct v4l2_ctrl_config mtk_cam_controls[] = { > > + { > > + .ops = &mtk_cam_dev_ctrl_ops, > > + .id = V4L2_CID_PRIVATE_GET_BIN_INFO, > > Don't use "PRIVATE" in the

Re: Build failed in Jenkins: v4l-utils #43

Em Wed, 2 Oct 2019 11:49:20 +0200 Hans Verkuil escreveu: > On 10/2/19 11:36 AM, Mauro Carvalho Chehab wrote: > > Em Wed, 2 Oct 2019 10:25:02 +0200 > > Hans Verkuil escreveu: > > > >> Hi Mauro, > >> > >> On 10/2/19 10:16 AM, Jenkins Builder Robot wrote: > >>> See > >>>

Re: [GIT PULL for v5.5] Venus updates

From: buil...@linuxtv.org Pull request: https://patchwork.linuxtv.org/patch/59112/ Build log: https://builder.linuxtv.org/job/patchwork/18645/ Build time: 00:06:36 Link: https://lore.kernel.org/linux-media/20191002094743.28589-1-stanimir.varba...@linaro.org gpg: Signature made Wed 02 Oct 2019 08

Re: Build failed in Jenkins: v4l-utils #43

On 10/2/19 11:36 AM, Mauro Carvalho Chehab wrote: > Em Wed, 2 Oct 2019 10:25:02 +0200 > Hans Verkuil escreveu: > >> Hi Mauro, >> >> On 10/2/19 10:16 AM, Jenkins Builder Robot wrote: >>> See >>> >>> >>> Changes: >>> >>>

[GIT PULL for v5.5] Venus updates

Hi Mauro, The Venus driver updates include: * three fixes: fail to suspend, enum frameinterval issue with encoder and frequency table modifications for v3 to handle performance issues. * two new features: interconnect bandwidth support on v4 and more precise clock-scaling on v4. Please pull. re

Re: Build failed in Jenkins: v4l-utils #43

Em Wed, 2 Oct 2019 10:25:02 +0200 Hans Verkuil escreveu: > Hi Mauro, > > On 10/2/19 10:16 AM, Jenkins Builder Robot wrote: > > See > > > > > > Changes: > > > > [hverkuil-cisco] keytable: add new generated keymaps > >

Jenkins build is back to normal : v4l-utils #45

See

Re: [RFC PATCH V3 1/5] dt-bindings: mt8183: Added DIP dt-bindings

Hi Frederic, On Tue, Sep 10, 2019 at 03:22:40AM +0800, frederic.c...@mediatek.com wrote: > From: Frederic Chen > > This patch adds DT binding documentation for the Digital Image > Processing (DIP) unit of camera ISP system on Mediatek's SoCs. > > It depends on the SCP and MDP 3 patch as followi

Re: [RFC PATCH V3 5/5] media: platform: mtk-mdp3: Add struct tuning_addr and img_sw_buffer

On Tue, Sep 10, 2019 at 03:22:44AM +0800, frederic.c...@mediatek.com wrote: > From: Frederic Chen > > We added a struct tuning_addr which contains a field "present" > so that the driver can tell the firmware if we have user tuning > dataor not. > > The strcut img_sw_buffer is also added. This st

Re: [RFC PATCH V3 3/5] media: platform: Add Mediatek DIP driver KConfig

Hi Frederic, On Tue, Sep 10, 2019 at 03:22:42AM +0800, frederic.c...@mediatek.com wrote: > From: Frederic Chen > > This patch adds KConfig for Mediatek Digital Image Processing > driver(DIP). DIP is embedded in Mediatek SoCs. It provides > image format conversion, resizing, and rotation function

Build failed in Jenkins: v4l-utils #44

See Changes: [hverkuil-cisco] cec utils: support CEC_ADAP_G_CONNECTOR_INFO -- [...truncated 1.09 MB...] ^~~ log_msg In file included from cec-follower.h:25,

[PATCH] media: uvcvideo: Add a quirk to force GEO GC6500 Camera bits-per-pixel value

This device does not function correctly in raw mode in kernel versions validating buffer sizes in bulk mode. It erroneously announces 16 bits per pixel instead of 12 for NV12 format, so it needs this quirk to fix computed frame size and avoid legitimate frames getting discarded. Signed-off-by: Ser

[ANN] v4l-utils: remove utils/cec-follower/cec-log.h after a 'git pull'

Just a heads up: I synced v4l-utils with the latest media_tree master branch, and that required some reorganization for the CEC utilities. If you do a 'git pull' in an existing v4l-utils repo, then it will fail to build due to an orphaned utils/cec-follower/cec-log.h file. Just remove it and it wi

Re: Build failed in Jenkins: v4l-utils #43

Hi Mauro, On 10/2/19 10:16 AM, Jenkins Builder Robot wrote: > See > > > Changes: > > [hverkuil-cisco] keytable: add new generated keymaps > > [hverkuil-cisco] msg2ctl.pl: add newline after log_msg > > [hverkuil-cisco

Build failed in Jenkins: v4l-utils #43

See Changes: [hverkuil-cisco] keytable: add new generated keymaps [hverkuil-cisco] msg2ctl.pl: add newline after log_msg [hverkuil-cisco] cec-follower: drop the hardcoded UI commands list [hverkuil-cisco] cec-ctl/cec-

Re: [PATCH v10 03/14] media: v4l2-fwnode: add initial connector parsing support

Hi Sakari, On 19-10-02 10:03, Sakari Ailus wrote: > Hi Marco, > > On Fri, Aug 30, 2019 at 12:16:35PM +0200, Marco Felsch wrote: > > The patch adds the initial connector parsing code, so we can move from a > > driver specific parsing code to a generic one. Currently only the > > generic fields and

[PATCH] cec-funcs.h: use new CEC_OP_UI_CMD defines

When the new CEC_OP_UI_CMD defines were added I forgot to update this header to use these new defines. This is now fixed. Signed-off-by: Hans Verkuil --- diff --git a/include/uapi/linux/cec-funcs.h b/include/uapi/linux/cec-funcs.h index 8997d5068c08..a5e579f7e86b 100644 --- a/include/uapi/linux/c

Re: [PATCH v10 03/14] media: v4l2-fwnode: add initial connector parsing support

Hi Marco, On Fri, Aug 30, 2019 at 12:16:35PM +0200, Marco Felsch wrote: > The patch adds the initial connector parsing code, so we can move from a > driver specific parsing code to a generic one. Currently only the > generic fields and the analog-connector specific fields are parsed. Parsing > the