Re: [PATCH] dvb_usb_dvbsky: Mygica T230C2 add support for T230C hw version 2

2019-06-13 Thread JP
On 6/13/19 3:45 AM, JP wrote: On 6/13/19 3:23 AM, Antti Palosaari wrote: On 6/13/19 4:15 AM, JP wrote: And according to old usb sniffs from pctv 292e [Si2168B] default manual ts clock is set to 7.2MHz, which means 57.6Mbit/s datarate, it should be quite optimal for DVB-T2 max. In theory i

Re: [RFC v2 1/4] dt-binding: mt8183: Add Mediatek MDP3 dt-bindings

2019-06-13 Thread Rob Herring
On Thu, May 16, 2019 at 11:23:29AM +0800, Daoyuan Huang wrote: > From: daoyuan huang > > This patch adds DT binding document for Media Data Path 3 (MDP3) > a unit in multimedia system used for scaling and color format convert. > > Signed-off-by: Ping-Hsun Wu > Signed-off-by: daoyuan huang > --

[GIT PULL FOR v5.3] Minor fixes

2019-06-13 Thread Sean Young
Hi Mauro, Just some minor fixes. Thanks, Sean -- The following changes since commit 3dad39e67cca68b321073cecabf2d7d1103d97d5: media: imx214: Fix typo in module description of imx214 (2019-06-12 10:47:15 -0400) are available in the Git repository at: git://linuxtv.org/syoung/media_tree.g

Re: [GIT PULL FOR v5.3] Add Hauppauge DVB devices

2019-06-13 Thread Brad Love
Hi Mauro, On 13/06/2019 13.35, Mauro Carvalho Chehab wrote: > Em Thu, 13 Jun 2019 13:14:47 -0500 > Brad Love escreveu: > >> Hi Mauro, >> >> Here is my first pull request. Please be gentle :) > :-) > > First problem: > > error: FETCH_HEAD: cannot verify a non-tag object of type commit. > >

Re: [GIT PULL FOR v5.3] Add Hauppauge DVB devices

2019-06-13 Thread Mauro Carvalho Chehab
Em Thu, 13 Jun 2019 13:14:47 -0500 Brad Love escreveu: > Hi Mauro, > > Here is my first pull request. Please be gentle :) :-) First problem: error: FETCH_HEAD: cannot verify a non-tag object of type commit. You should use branches and sign them with your gpg signature. Patches appli

[GIT PULL FOR v5.3] Add Hauppauge DVB devices

2019-06-13 Thread Brad Love
Hi Mauro, Here is my first pull request. Please be gentle :) This adds support for Hauppauge HVR1955 and HVR1975 devices, along with adding a vid:pid for a missing OEM Hauppauge 955Q variant. Cheers, Brad --- The following changes since commit 513dbd35b5d93c45fa7291147f21fc0227a9f999: media:

Re: [PATCH 1/8] media: dt-bindings: imx7-csi: Document a single CSI clock

2019-06-13 Thread Rob Herring
On Sat, 4 May 2019 11:40:25 -0300, Fabio Estevam wrote: > As per the i.MX7D Reference Manual only the MCLK is used for > the CSI block, so only document this single clock. > > Signed-off-by: Fabio Estevam > --- > Documentation/devicetree/bindings/media/imx7-csi.txt | 9 +++-- > 1 file chang

Re: [PATCH] dvb_usb_dvbsky: Mygica T230C2 add support for T230C hw version 2

2019-06-13 Thread Antti Palosaari
On 6/12/19 11:07 PM, Frantisek Rysanek wrote: On 12 Jun 2019 at 1:28, Antti Palosaari wrote: [...] What is that T230C2 stick? JP has already explained the details, how that name was arrived at. As previously suggested, I can call it T230C v2 in the descriptive texts. I'd suggest keeping T230C2

[linux-next:master 5296/6049] ERROR: "rc_unregister_device" [drivers/media/pci/ttpci/dvb-ttpci.ko] undefined!

2019-06-13 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: 81a72c79224f83af18e26dc0c599f917312e6c78 commit: 71f49a8bf5c592413edb5c8839ec0e6d754db3e1 [5296/6049] media: ttpci: use rc-core for the IR receiver config: x86_64-randconfig-s4-05260805 (attached as .config

[PATCH v3 1/8] rcar-vin: Do not call pm_runtime_{resume,suspend}()

2019-06-13 Thread Niklas Söderlund
The driver does not implement runtime resume and suspend function so there is little point in trying to call them. This is a leftover from the drivers soc_camera beginnings. Signed-off-by: Niklas Söderlund Reviewed-by: Ulrich Hecht --- drivers/media/platform/rcar-vin/rcar-v4l2.c | 5 - 1 fi

[PATCH v3 2/8] rcar-vin: Remove unneeded calls to pm_runtime_{enable,disable}

2019-06-13 Thread Niklas Söderlund
Runtime PM is already enabled unconditionally when the driver is probed and disabled when it's removed. There is no point in doing it again for Gen2 when opening and closing the video device. Signed-off-by: Niklas Söderlund Reviewed-by: Ulrich Hecht --- drivers/media/platform/rcar-vin/rcar-v4l2

Re: [PATCH] dvb_usb_dvbsky: Mygica T230C2 add support for T230C hw version 2

2019-06-13 Thread JP
On 6/13/19 12:31 AM, Antti Palosaari wrote: On 6/12/19 11:07 PM, Frantisek Rysanek wrote: On 12 Jun 2019 at 1:28, Antti Palosaari wrote: [...] What is that T230C2 stick? JP has already explained the details, how that name was arrived at. As previously suggested, I can call it T230C v2 in t

[PATCH v3 5/8] rcar-vin: Move pm_runtime_{get,put} out of helpers

2019-06-13 Thread Niklas Söderlund
The helpers rvin_power_{on,off} deal with both VIN and the parallel subdevice power. This makes it hard to merge the Gen2 and Gen3 open/release functions. Move the VIN power handling directly to the open/release functions to prepare for the merge. Signed-off-by: Niklas Söderlund Reviewed-by: Ulri

[PATCH v3 4/8] rcar-vin: Do not sync subdevice format when opening the video device

2019-06-13 Thread Niklas Söderlund
The format is already synced when the subdevice is bound, there is no need to do do it every time the video device is opened. Signed-off-by: Niklas Söderlund Reviewed-by: Ulrich Hecht Reviewed-by: Laurent Pinchart --- drivers/media/platform/rcar-vin/rcar-v4l2.c | 25 - 1 fi

[PATCH v3 0/8] rcar-vin: Merge Gen2 and Gen3 file operations

2019-06-13 Thread Niklas Söderlund
Hi, This series aims to merge the two different set of file operations used in the rcar-vin driver. One set was used on Renesas Gen2 boards while the other set was used on Gen3. Main difference between the two is that Gen2 uses a device node centric world view while Gen3 subscribes to the media co

[PATCH v3 8/8] rcar-vin: Merge Gen2 and Gen3 file operations

2019-06-13 Thread Niklas Söderlund
After the rework of the Gen2 file operations it's now trivial to merge the Gen2 and Gen3 versions. Signed-off-by: Niklas Söderlund Reviewed-by: Ulrich Hecht --- drivers/media/platform/rcar-vin/rcar-v4l2.c | 100 1 file changed, 18 insertions(+), 82 deletions(-) diff --git

[PATCH v3 7/8] rcar-vin: Fold rvin_initialize_device() into rvin_open()

2019-06-13 Thread Niklas Söderlund
The function no longer serve a purpose as most tasks it performed have been refactored, fold what remains of it into the only caller. While at it add error checking for v4l2_ctrl_handler_setup(). Signed-off-by: Niklas Söderlund Reviewed-by: Ulrich Hecht --- drivers/media/platform/rcar-vin/rcar-

[PATCH v3 6/8] rcar-vin: Merge helpers dealing with powering the parallel subdevice

2019-06-13 Thread Niklas Söderlund
The two power helpers are now only dealing with the parallel subdevice, merge them into a single rvin_power_parallel() helper to reduce code duplication. Signed-off-by: Niklas Söderlund Reviewed-by: Ulrich Hecht Reviewed-by: Laurent Pinchart --- drivers/media/platform/rcar-vin/rcar-v4l2.c | 20

[PATCH v3 3/8] rcar-vin: Allow interrupting lock when trying to open the video device

2019-06-13 Thread Niklas Söderlund
The user should be allowed to break waiting for the lock when opening the video device. Signed-off-by: Niklas Söderlund Reviewed-by: Ulrich Hecht Reviewed-by: Laurent Pinchart --- drivers/media/platform/rcar-vin/rcar-v4l2.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a

[PATCH v2 1/3] rcar-vin: Rename VNDMR_DTMD_ARGB1555 to VNDMR_DTMD_ARGB

2019-06-13 Thread Niklas Söderlund
The value have nothing to do with ARGB1555, it controls if the alpha component should be filled in for ARGB1555 or ARGB888. Rename it to reflect this. Signed-off-by: Niklas Söderlund Reviewed-by: Ulrich Hecht Reviewed-by: Laurent Pinchart --- drivers/media/platform/rcar-vin/rcar-dma.c | 4 ++--

[PATCH v2 3/3] rcar-vin: Add support for RGB formats with alpha component

2019-06-13 Thread Niklas Söderlund
The R-Car VIN module supports V4L2_PIX_FMT_ARGB555 and V4L2_PIX_FMT_ABGR32 pixel formats. Add the hardware register setup and allow the alpha component to be changed while streaming using the V4L2_CID_ALPHA_COMPONENT control. Signed-off-by: Niklas Söderlund --- drivers/media/platform/rcar-vin/rc

[PATCH v2 2/3] rcar-vin: Add control for alpha component

2019-06-13 Thread Niklas Söderlund
In preparation to adding support for RGB pixel formats with an alpha component add a control to allow the user to control which alpha value should be used. Signed-off-by: Niklas Söderlund Reviewed-by: Laurent Pinchart --- drivers/media/platform/rcar-vin/rcar-core.c | 53 - d

[PATCH v2 0/3] rcar-vin: Add support for RGB formats with alpha component

2019-06-13 Thread Niklas Söderlund
Hi, This small series adds support for two new pixel formats for the rcar-vin driver; V4L2_PIX_FMT_ARGB555 and V4L2_PIX_FMT_ABGR32. Both formats have an alpha component so a new standard control is also added to control its value, V4L2_CID_ALPHA_COMPONENT. The series is based on the latest media-

Re: [PATCH for v5.2] videobuf2-core.c: always reacquire USERPTR memory

2019-06-13 Thread Nicolas Dufresne
Le mercredi 12 juin 2019 à 11:17 +0300, Laurent Pinchart a écrit : > Hi Nicolas, > > On Tue, Jun 11, 2019 at 08:09:13PM -0400, Nicolas Dufresne wrote: > > Le mardi 11 juin 2019 à 13:24 +0300, Laurent Pinchart a écrit : > > > On Fri, Jun 07, 2019 at 03:38:39PM -0400, Nicolas Dufresne wrote: > > > >

Dear Friend,

2019-06-13 Thread Mrs Alice Johnson
-- Dear Friend, I am Mrs Alice Johnson.am sending you this brief letter to solicit your partnership to transfer $18.5 million US Dollars.I shall send you more information and procedures when I receive positive response from you. please send me a message in my Email box (mrsalicejohns...@gmail.c

[PATCH] media: media.h: Fix shifting signed 32-bit value by 31 bits problem

2019-06-13 Thread Shuah Khan
Fix MEDIA_ENT_ID_FLAG_NEXT to use "U" cast to avoid shifting signed 32-bit value by 31 bits problem. This isn't a problem for kernel builds with gcc. This could be problem since this header is part of public API which could be included for builds using compilers that don't handle this condition sa

Re: [PATCH 2/9] v4l2-async: Use endpoint node, not device node, for fwnode match

2019-06-13 Thread Niklas Söderlund
Hi Sakari, Thanks for your work. On 2019-04-05 12:41:20 +0300, Sakari Ailus wrote: > V4L2 async framework can use both device's fwnode and endpoints's fwnode > for matching the async sub-device with the sub-device. In order to proceed > moving towards endpoint matching assign the endpoint to the

Re: [PATCH] dvb_usb_dvbsky: Mygica T230C2 add support for T230C hw version 2

2019-06-13 Thread JP
On 6/13/19 1:46 AM, JP wrote: On 6/13/19 12:31 AM, Antti Palosaari wrote: On 6/12/19 11:07 PM, Frantisek Rysanek wrote: On 12 Jun 2019 at 1:28, Antti Palosaari wrote: [...] What is that T230C2 stick? JP has already explained the details, how that name was arrived at. As previously suggeste

Re: [PATCH] dvb_usb_dvbsky: Mygica T230C2 add support for T230C hw version 2

2019-06-13 Thread Antti Palosaari
On 6/13/19 4:15 AM, JP wrote: And according to old usb sniffs from pctv 292e [Si2168B] default manual ts clock is set to 7.2MHz, which means 57.6Mbit/s datarate, it should be quite optimal for DVB-T2 max. In theory it could be a little higher only when 10MHz channel bandwidth and most less erro

Re: [PATCH] dvb_usb_dvbsky: Mygica T230C2 add support for T230C hw version 2

2019-06-13 Thread JP
On 6/13/19 3:23 AM, Antti Palosaari wrote: On 6/13/19 4:15 AM, JP wrote: And according to old usb sniffs from pctv 292e [Si2168B] default manual ts clock is set to 7.2MHz, which means 57.6Mbit/s datarate, it should be quite optimal for DVB-T2 max. In theory it could be a little higher only

Re: [PATCH] dvb_usb_dvbsky: Mygica T230C2 add support for T230C hw version 2

2019-06-13 Thread JP
On 6/13/19 3:23 AM, Antti Palosaari wrote: On 6/13/19 4:15 AM, JP wrote: And according to old usb sniffs from pctv 292e [Si2168B] default manual ts clock is set to 7.2MHz, which means 57.6Mbit/s datarate, it should be quite optimal for DVB-T2 max. In theory it could be a little higher only

Re: [PATCH 1/3] media: i2c: ov02a10: Add ov02a10 camera sensor driver

2019-06-13 Thread Dongchun Zhu
Hi Nicolas, On Sun, 2019-05-26 at 16:53 +0800, Nicolas Boichat wrote: > On Thu, May 23, 2019 at 6:22 PM wrote: > > > > From: Dongchun Zhu > > > > Add a V4L2 sub-device driver for the OmniVision image sensor. > > This is a camera sensor using the I2C bus for control and the > > CSI-2 bus for data

Re: [PATCH 3/3] media: dt-bindings: media: i2c: Add bindings for ov02a10

2019-06-13 Thread Dongchun Zhu
Hi Sakari, On Fri, 2019-05-31 at 19:16 +0300, Sakari Ailus wrote: > Hi Dongchun, > > Thanks for the patch. > > On Thu, May 23, 2019 at 06:22:04PM +0800, dongchun@mediatek.com wrote: > > From: Dongchun Zhu > > > > Add device tree binding documentation for the OV02A10 CMOS image sensor. > >

Re: [PATCH 1/3] media: i2c: ov02a10: Add ov02a10 camera sensor driver

2019-06-13 Thread Dongchun Zhu
Hi Bingbu, On Tue, 2019-05-28 at 14:20 +0800, Bingbu Cao wrote: > > On 5/23/19 6:22 PM, dongchun@mediatek.com wrote: > > From: Dongchun Zhu > > > > Add a V4L2 sub-device driver for the OmniVision image sensor. > > This is a camera sensor using the I2C bus for control and the > > CSI-2 bus f

cron job: media_tree daily build: WARNINGS

2019-06-13 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 13 05:00:11 CEST 2019 media-tree git hash:513dbd35b5d93c45fa7291147f21fc0227a9f999 media_build gi

Re: The restricted number (LIRCBUF_SIZE) of pulse-spaces in IR sequences is too small

2019-06-13 Thread Takashi Kanamaru
Dear Sean Young, > I don't see any reason why it can't be increased. For the record, it would > be interesting to know what your air conditioner model is and what the > IR signal looks like. I have not seen such a signal before. I observed the IR signals of my air conditioner (Panasoic ACXA75C006

[PATCH] media: rc: IR signal for Panasonic air conditioner too long sequences is too small

2019-06-13 Thread Sean Young
The IR signal to control the Panasonic ACXA75C00600 air conditioner has 439 pulse/spaces. Increase limit to make it possible to transmit signal. Reported-by: Takashi Kanamaru Signed-off-by: Sean Young --- drivers/media/rc/lirc_dev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --

Re: [v8] media: imx: add mem2mem device

2019-06-13 Thread Robin Gong
Thank Sven, please have a try with the below patch. I'll send it to review later. The root cause is that channel0 done interrupt may come later after after sdma clock disabled(sdma_load_firmware()), which means clearing channel0 interrupt status in ISR never work, thus infinite interrupt comes out.

Re: [PATCH][next] staging: media: meson: remove redundant initialization of mpeg12

2019-06-13 Thread Maxime Jourdan
Hi Colin, On Wed, Jun 12, 2019 at 4:42 PM Colin King wrote: > > From: Colin Ian King > > The pointer mpeg12 is being initialized however that value is never > read and mpeg12 is being re-assigned almost immediately afterwards. > Remove the redundant initialization. > > Addresses-Coverity: ("Unuse

Re: [PATCH v2] dt-bindings: rcar-{csi2,vin}: Rename bindings documentation files

2019-06-13 Thread Simon Horman
On Wed, Jun 12, 2019 at 11:12:41PM +0200, Niklas Söderlund wrote: > Renesas media binding documentation files uses a naming schema of > 'renesas,.txt'. Rename VIN and CSI-2 files to match this > pattern. > > Signed-off-by: Niklas Söderlund > Reviewed-by: Geert Uytterhoeven > Reviewed-by: Ulrich

[PATCH] media: s5p-mfc: Make additional clocks optional

2019-06-13 Thread Marek Szyprowski
Since the beginning the second clock ('special', 'sclk') was optional and it is not available on some variants of Exynos SoCs (i.e. Exynos5420 with v7 of MFC hardware). However commit 1bce6fb3edf1 made handling of all specified clocks mandatory. This patch restores original behavior of the driver a

[PATCH v2] media: s5p-mfc: Make additional clocks optional

2019-06-13 Thread Marek Szyprowski
Since the beginning the second clock ('special', 'sclk') was optional and it is not available on some variants of Exynos SoCs (i.e. Exynos5420 with v7 of MFC hardware). However commit 1bce6fb3edf1 made handling of all specified clocks mandatory. This patch restores original behavior of the driver a

[PATCH 0/2] Fix a few smatch warnings

2019-06-13 Thread Sakari Ailus
Hi, This small set fixes a few trivial smatch warnings. They're not really bugs either but changing the code cleans it up as well. Sakari Ailus (2): v4l2-fwnode: Avoid using PTR_ERR(NULL) ov9640: Don't check for NULL on devm_gpiod_get return values drivers/media/i2c/ov9640.c| 4

[PATCH 2/2] ov9640: Don't check for NULL on devm_gpiod_get return values

2019-06-13 Thread Sakari Ailus
devm_gpiod_get never returns NULL; therefore it's not necessary to check for that. PTR_ERR(NULL) also yields zero, which is confusing to smatch. Reported-by: Hans Verkuil Signed-off-by: Sakari Ailus --- drivers/media/i2c/ov9640.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff -

[PATCH 1/2] v4l2-fwnode: Avoid using PTR_ERR(NULL)

2019-06-13 Thread Sakari Ailus
PTR_ERR(NULL) yields 0 which is commonly used to denote success. This is the case here, and PTR_ERR(NULL) is apparently shunned upon. Fix this by explicitly returning 0 if fwnode == NULL. Reported-by: Hans Verkuil Signed-off-by: Sakari Ailus --- drivers/media/v4l2-core/v4l2-fwnode.c | 6 +-

[PATCH v1.1 1/2] v4l2-fwnode: Avoid using PTR_ERR(NULL)

2019-06-13 Thread Sakari Ailus
PTR_ERR(NULL) yields 0 which is commonly used to denote success. This is the case here, and PTR_ERR(NULL) is apparently shunned upon. Fix this by explicitly returning 0 if fwnode == NULL. Reported-by: Hans Verkuil Signed-off-by: Sakari Ailus --- since v1: - Fix rebase mess. drivers/media/v4l2

[PATCH 2/2] media: hantro: Add support for VP8 decoding on rk3288

2019-06-13 Thread Ezequiel Garcia
From: ZhiChao Yu Introduce VP8 decoding support in RK3288. Signed-off-by: ZhiChao Yu Signed-off-by: Tomasz Figa Signed-off-by: Boris Brezillon Signed-off-by: Ezequiel Garcia --- drivers/staging/media/hantro/Makefile | 4 +- drivers/staging/media/hantro/hantro.h | 5 + dr

[PATCH 1/2] media: uapi: Add VP8 stateless decoder API

2019-06-13 Thread Ezequiel Garcia
From: Pawel Osciak Add the parsed VP8 frame pixel format and controls, to be used with the new stateless decoder API for VP8 to provide parameters for accelerator (aka stateless) codecs. Signed-off-by: Pawel Osciak Signed-off-by: Ezequiel Garcia -- Changes from RFC: * Make sure the uAPI has th

[PATCH 0/2] RK3288 VP8 decoding support

2019-06-13 Thread Ezequiel Garcia
This patchset adds support for Hantro G1 VP8 stateless decoding, as available on RK3288 SoC. In order to support VP8 stateless decoding, a new pixel format is introduced V4L2_PIX_FMT_VP8_FRAME, to be used with a new control V4L2_CID_MPEG_VIDEO_VP8_FRAME_HDR. The VP8 stateless uAPI was submitted a

Re: [v8] media: imx: add mem2mem device

2019-06-13 Thread Sven Van Asbroeck
On Thu, Jun 13, 2019 at 4:52 AM Robin Gong wrote: > > Thank Sven, please have a try with the below patch. I'll send it to > review later. That's awesome, we are cooking with gas :) Actually I patched only the sdma_load_script() function (not _context()) and the crash no longer happens. Cc me on