Re: [PATCH v3 10/10] media: aspeed: add a workaround to fix a silicon bug

2019-06-12 Thread Hans Verkuil
Eddie: ping! Regards, Hans On 6/6/19 2:53 AM, Jae Hyun Yoo wrote: > Hi Eddie, > > All patches in this series were queued to linux/media tree except this > one. Can you please review this patch? > > Thanks, > Jae > > On 5/31/2019 3:15 PM, Jae Hyun Yoo wrote: >> AST2500 silicon revision

Re: [PATCH 5/7 RESEND] touchscreen/sur40: set device_caps in struct video_device

2019-06-12 Thread Hans Verkuil
Florian: ping! Regards, Hans On 6/4/19 2:36 PM, Hans Verkuil wrote: > Instead of filling in the struct v4l2_capability device_caps > field, fill in the struct video_device device_caps field. > > That way the V4L2 core knows what the capabilities of the > video device are. > > But this

Re: [PATCH] media: imx7-media-csi: get csi upstream endpoint

2019-06-12 Thread Dan Carpenter
On Tue, Jun 11, 2019 at 04:09:55PM +0100, Rui Miguel Silva wrote: > When the upstream endpoint is neither a mux nor a CSI2 module, just get > the source pad directly upstream from the CSI. > > Fixes: 05f634040c0d ("media: staging/imx7: add imx7 CSI subdev driver") > Reported-by: Sebastien Szymansk

Re: [PATCH v4 00/10] Rename Rockchip VPU driver to Hantro, add initial i.MX8M support

2019-06-12 Thread Hans Verkuil
On 6/11/19 2:50 PM, Philipp Zabel wrote: > There are several other SoCs that contain Hantro IP based VPUs, such as > NXP i.MX8MQ (Hantro G1 and G2) and i.MX8MM (Hantro G1, G2, and H1). To > maximize code sharing, add initial support for these SoCs to the > Rockchip VPU driver, after renaming it to

Re: [PATCH v4 00/10] Rename Rockchip VPU driver to Hantro, add initial i.MX8M support

2019-06-12 Thread Hans Verkuil
On 6/12/19 9:55 AM, Hans Verkuil wrote: > On 6/11/19 2:50 PM, Philipp Zabel wrote: >> There are several other SoCs that contain Hantro IP based VPUs, such as >> NXP i.MX8MQ (Hantro G1 and G2) and i.MX8MM (Hantro G1, G2, and H1). To >> maximize code sharing, add initial support for these SoCs to the

Re: [PATCH v4 01/10] rockchip/vpu: rename from rockchip to hantro

2019-06-12 Thread Jonas Karlman
On 2019-06-11 14:50, Philipp Zabel wrote: > Rename the driver and all relevant identifiers from Rockchip to Hantro, > as other Hantro IP based VPU implementations can be supported by the > same driver. > The RK3288 decoder is Hantro G1 based, the encoder is Hantro H1. The RK3288 has two VPU blocks

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

2019-06-12 Thread Laurent Pinchart
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: > >> Le vendredi 07 juin 2019 à 16:58 +0300, Laurent Pinchart a écrit : > >>> On Fri

Re: [PATCH v4 01/10] rockchip/vpu: rename from rockchip to hantro

2019-06-12 Thread Boris Brezillon
On Wed, 12 Jun 2019 08:14:35 + Jonas Karlman wrote: > On 2019-06-11 14:50, Philipp Zabel wrote: > > Rename the driver and all relevant identifiers from Rockchip to Hantro, > > as other Hantro IP based VPU implementations can be supported by the > > same driver. > > The RK3288 decoder is Hantr

Re: [PATCH v4 00/10] Rename Rockchip VPU driver to Hantro, add initial i.MX8M support

2019-06-12 Thread Boris Brezillon
On Wed, 12 Jun 2019 10:00:45 +0200 Hans Verkuil wrote: > On 6/12/19 9:55 AM, Hans Verkuil wrote: > > On 6/11/19 2:50 PM, Philipp Zabel wrote: > >> There are several other SoCs that contain Hantro IP based VPUs, such as > >> NXP i.MX8MQ (Hantro G1 and G2) and i.MX8MM (Hantro G1, G2, and H1). To

Re: [PATCH 5/7 RESEND] touchscreen/sur40: set device_caps in struct video_device

2019-06-12 Thread Florian Echtler
Sorry, didn't realize you'd also need my feedback. No complaints. Acked-by: Florian Echtler Best, Florian On 04.06.19 18:06, Dmitry Torokhov wrote: > Hi Hans, > > On Tue, Jun 04, 2019 at 02:36:27PM +0200, Hans Verkuil wrote: >> Instead of filling in the struct v4l2_capability device_caps >> fi

[PATCH v5 00/10] Rename Rockchip VPU driver to Hantro, add initial i.MX8M support

2019-06-12 Thread Philipp Zabel
There are several other SoCs that contain Hantro IP based VPUs, such as NXP i.MX8MQ (Hantro G1 and G2) and i.MX8MM (Hantro G1, G2, and H1). To maximize code sharing, add initial support for these SoCs to the Rockchip VPU driver, after renaming it to Hantro VPU. This series is based on the br-v5.3g

[PATCH v5 08/10] media: dt-bindings: Document i.MX8MQ and i.MX8MM VPU bindings

2019-06-12 Thread Philipp Zabel
Add devicetree binding documentation for the Hantro G1/G2 VPU on i.MX8MQ and for the Hantro G1/G2/H1 VPU on i.MX8MM. Signed-off-by: Philipp Zabel --- .../devicetree/bindings/media/imx8m-vpu.txt | 56 +++ 1 file changed, 56 insertions(+) create mode 100644 Documentation/devicet

[PATCH v5 10/10] media: hantro: add initial i.MX8MM support (untested)

2019-06-12 Thread Philipp Zabel
This should enable MPEG-2 decoding on the Hantro G1 and JPEG encoding on the Hantro H1 on i.MX8MM. Signed-off-by: Philipp Zabel --- drivers/staging/media/hantro/hantro_drv.c | 1 + drivers/staging/media/hantro/hantro_hw.h| 1 + drivers/staging/media/hantro/imx8m_vpu_hw.c | 139

[PATCH v5 02/10] media: hantro: print video device name in addition to device node

2019-06-12 Thread Philipp Zabel
It can be helpful to know which video device was registered at which device node. Signed-off-by: Philipp Zabel Reviewed-by: Boris Brezillon --- drivers/staging/media/hantro/hantro_drv.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/media/hantro/hantro_drv

[PATCH v5 06/10] media: hantro: add support for separate control block

2019-06-12 Thread Philipp Zabel
On i.MX8MQ/MM a separate control block contains registers for per-core resets, clock gating, and fuse register control. Signed-off-by: Philipp Zabel Reviewed-by: Boris Brezillon --- drivers/staging/media/hantro/hantro.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/staging/media

[PATCH v5 04/10] media: hantro: make irq names configurable

2019-06-12 Thread Philipp Zabel
The i.MX8MQ bindings will use different IRQ names ("g1" instead of "vdpu", and "g2"), so make them configurable. This also allows to register more than two IRQs, which will be required for i.MX8MM support later (it will add "h1" instead of "vepu"). Signed-off-by: Philipp Zabel Reviewed-by: Boris

[PATCH v5 03/10] media: hantro: add PM runtime resume callback

2019-06-12 Thread Philipp Zabel
It seems that on i.MX8MQ the power domain controller does not propagate resets to the VPU cores on resume. Add a callback to allow implementing manual reset of the VPU cores after ungating the power domain. Signed-off-by: Philipp Zabel Reviewed-by: Boris Brezillon --- drivers/staging/media/hant

[PATCH v5 07/10] media: hantro: allow arbitrary number of clocks

2019-06-12 Thread Philipp Zabel
Dynamically allocate clocks and move clock names out of struct hantro_variant. This lifts the four clock limit and allows to use ARRAY_SIZE() to fill .num_clocks to reduce the risk of mismatches. Signed-off-by: Philipp Zabel Reviewed-by: Boris Brezillon --- Changes since v4 [1]: - Rebased onto

[PATCH v5 05/10] media: hantro: add support for named register ranges

2019-06-12 Thread Philipp Zabel
Add support for multiple register ranges with SoC specific names. Signed-off-by: Philipp Zabel Reviewed-by: Boris Brezillon --- drivers/staging/media/hantro/hantro.h | 8 ++-- drivers/staging/media/hantro/hantro_drv.c | 24 +-- 2 files changed, 24 insertions(+), 8 d

[PATCH v5 09/10] media: hantro: add initial i.MX8MQ support

2019-06-12 Thread Philipp Zabel
For now this just enables MPEG-2 decoding on the Hantro G1 on i.MX8MQ. Signed-off-by: Philipp Zabel -- Changes since v4 [1]: - Fix duplicated num_irqs initializer [1] https://patchwork.linuxtv.org/patch/56802/ --- drivers/staging/media/hantro/Kconfig| 16 +- drivers/staging/media/hant

[PATCH v7] media: hantro: Add support for MPEG-2 decoding on RK3328

2019-06-12 Thread Philipp Zabel
From: Jonas Karlman Add necessary bits to support MPEG2 decoding on RK3328. Signed-off-by: Jonas Karlman Signed-off-by: Ezequiel Garcia Signed-off-by: Philipp Zabel --- Changes since v6 [1]: - Rebased onto Hantro rename series [2] [1] https://patchwork.linuxtv.org/patch/56402/ [2] https://pa

Re: [PATCH v4 00/10] Rename Rockchip VPU driver to Hantro, add initial i.MX8M support

2019-06-12 Thread Philipp Zabel
On Wed, 2019-06-12 at 10:00 +0200, Hans Verkuil wrote: > On 6/12/19 9:55 AM, Hans Verkuil wrote: > > On 6/11/19 2:50 PM, Philipp Zabel wrote: > > > There are several other SoCs that contain Hantro IP based VPUs, such as > > > NXP i.MX8MQ (Hantro G1 and G2) and i.MX8MM (Hantro G1, G2, and H1). To >

[PATCH v2] media: imx7-media-csi: get csi upstream endpoint

2019-06-12 Thread Rui Miguel Silva
When the upstream endpoint is neither a mux nor a CSI2 module, just get the source pad directly upstream from the CSI. Fixes: 05f634040c0d ("media: staging/imx7: add imx7 CSI subdev driver") Reported-by: Sebastien Szymanski Signed-off-by: Rui Miguel Silva --- v1->v2: Dan Carpenter: - s/in/is/

[GIT PULL FOR v5.3] Rename Rockchip VPU driver to Hantro, prepare for initial i.MX8M support

2019-06-12 Thread Hans Verkuil
This PR contains patches 1-7 of this series: https://www.mail-archive.com/linux-media@vger.kernel.org/msg147710.html Patch 8 is still waiting for an Acked-by so I postpone that and the following patches for a future PR. Let's do the rename now to get that out of the way. Regards, Hans

Re: [PATCH v3 2/2] media: v4l2-ctrl: Move compound control initialization

2019-06-12 Thread Hans Verkuil
On 6/6/19 6:12 PM, Ezequiel Garcia wrote: > Rework std_init adding an explicit initialization for > compound controls. > > While here, make sure the control is initialized to zero, > before providing default values for all its fields. > > Signed-off-by: Ezequiel Garcia > --- > Changes from v2: >

Re: Unsupported Anysee version

2019-06-12 Thread JP
On 6/8/19 4:03 PM, W. Talsma wrote: Dear, Today decided to dust off my Anysee E30 S2 Plus, and hope to get it to work with the Debian stretch tvheadend server. Once I plug it in however, I see the following: [27054.699829] usb 1-1.6: new high-speed USB device number 7 using ehci-pci [27055.2

[PATCH] media: s5p-mfc: fix reading min scratch buffer size on MFC v6/v7

2019-06-12 Thread Marek Szyprowski
pgd = f25837f9 [] *pgd=bd93d835 Internal error: Oops: 17 [#1] PREEMPT SMP ARM Modules linked in: btmrvl_sdio btmrvl bluetooth mwifiex_sdio mwifiex ecdh_generic ecc CPU: 0 PID: 1430 Comm: v4l2_decode Not tainted 5.2.0-rc4-next-20190612-6-gf077fba72e95-dirty #6167 Hardware name: SAMS

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

2019-06-12 Thread Takashi Kanamaru
Dear all, When using kernel 4.19.X and sending IR commands with LIRC, the number of pulse-spaces of IR sequence is restricted to be smaller than 256. In kernel 4.19, this restriction is caused by the following line in media/rc/lirc_dev.c, which did not exist kernel 4.14. https://github.com/torva

[GIT PULL FOR v5.3] Various fixes

2019-06-12 Thread Hans Verkuil (hansverk)
The following changes since commit 4e8c120de9268fc26f583268b9d22e7d37c4595f: media: fdp1: Support M3N and E3 platforms (2019-06-11 12:29:54 -0400) are available in the Git repository at: git://linuxtv.org/hverkuil/media_tree.git for-v5.3p for you to fetch changes up to 4ca498ad26603d78d76de

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

2019-06-12 Thread Sean Young
Dear Takashi Kanamaru, On Wed, Jun 12, 2019 at 11:08:34PM +0900, Takashi Kanamaru wrote: > Dear all, > > When using kernel 4.19.X and sending IR commands with LIRC, > the number of pulse-spaces of IR sequence is restricted to be smaller than > 256. > > In kernel 4.19, this restriction is cause

Re: [PATCH v3 10/10] media: aspeed: add a workaround to fix a silicon bug

2019-06-12 Thread Eddie James
On 6/12/19 2:17 AM, Hans Verkuil wrote: Eddie: ping! Regards, Hans On 6/6/19 2:53 AM, Jae Hyun Yoo wrote: Hi Eddie, All patches in this series were queued to linux/media tree except this one. Can you please review this patch? Thanks, Jae On 5/31/2019 3:15 PM, Jae Hyun Yoo wrote:

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

2019-06-12 Thread JP
It is the default in all the si2168 drivers, but the signal is lost somehow in this hw version of the t230c v2. The clock is 0 (null) Mhz! So, no data *at all* without setting it manually. IMy best guess is that it was a design flaw. I meant it is the default value for the SI2168D. I'll g

Re: [PATCH v3 10/10] media: aspeed: add a workaround to fix a silicon bug

2019-06-12 Thread Jae Hyun Yoo
On 6/12/2019 8:03 AM, Eddie James wrote: On 6/12/19 2:17 AM, Hans Verkuil wrote: Eddie: ping! Regards, Hans On 6/6/19 2:53 AM, Jae Hyun Yoo wrote: Hi Eddie, All patches in this series were queued to linux/media tree except this one. Can you please review this patch? Thanks, Jae On 5/

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

2019-06-12 Thread Frantisek Rysanek
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 in the USB ID macro (or suggest a more appr

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

2019-06-12 Thread Niklas Söderlund
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 Hecht --- .../media/{renesas,rcar-csi2.txt => renesas,csi2.txt} | 0 .