[RFC PATCH V0 7/7] [media] platform: mtk-isp: Add Mediatek FD driver

2019-02-19 Thread Jerry-ch Chen
This patch adds the driver of Face Detection (FD) unit in Mediatek camera system, providing face detection function. The mtk-isp directory will contain drivers for multiple IP blocks found in Mediatek ISP system. It will include ISP Pass 1 driver (CAM), sensor interface driver, DIP driver and face

[RFC PATCH V0 3/7] [media] dt-bindings: mt8183: Added FD-SMEM dt-bindings

2019-02-19 Thread Jerry-ch Chen
This patch adds the DT binding documentation for the shared memory between Face Detection unit of the camera system and the co-processor in Mediatek SoCs. Signed-off-by: Jerry-ch Chen --- .../devicetree/bindings/media/mediatek,fd_smem.txt | 28 ++ 1 file changed, 28 insertion

[RFC PATCH V0 4/7] [media] dt-bindings: mt8183: Added FD dt-bindings

2019-02-19 Thread Jerry-ch Chen
This patch adds DT binding documentation for the Face Detection (FD) unit of the camera system on Mediatek's SoCs. Signed-off-by: Jerry-ch Chen --- .../bindings/media/mediatek,mt8183-fd.txt | 30 ++ 1 file changed, 30 insertions(+) create mode 100644 Documentation/

[RFC PATCH V0 5/7] dts: arm64: mt8183: Add FD nodes

2019-02-19 Thread Jerry-ch Chen
This patch adds nodes for Face Detection (FD) unit. FD is embedded in Mediatek SoCs and works with the co-processor to perform face detection on the input data and image and output detected face result. Signed-off-by: Jerry-ch Chen --- arch/arm64/boot/dts/mediatek/mt8183.dtsi | 20 ++

[RFC PATCH V0 6/7] media: platform: Add Mediatek FD driver KConfig

2019-02-19 Thread Jerry-ch Chen
This patch adds KConfig for Mediatek Face Detection driver (FD). FD is embedded in Mediatek SoCs. It can provide hardware accelerated face detection function. Signed-off-by: Jerry-ch Chen --- drivers/media/platform/Kconfig | 2 ++ drivers/media/platform/mtk-isp/Kconfig | 10 ++

[RFC PATCH V0 0/7] media: platform: Add support for Face Detection (FD) on mt8183 SoC

2019-02-19 Thread Jerry-ch Chen
Hello, This is the first version of the RFC patch series adding Face Detection (FD) driver on Mediatek mt8183 SoC, which will be used in camera features on CrOS application. It belongs to the first Mediatek's camera driver series based on V4L2 and media controller framework. I posted the main part

[RFC PATCH V0 2/7] dts: arm64: mt8183: Add FD shared memory node

2019-02-19 Thread Jerry-ch Chen
This patch adds a shared memory region used on mt8183 for exchanging meta data between co-processor and Face Detection (FD) unit. Signed-off-by: Jerry-ch Chen --- arch/arm64/boot/dts/mediatek/mt8183.dtsi | 8 1 file changed, 8 insertions(+) diff --git a/arch/arm64/boot/dts/mediatek/mt8

[RFC PATCH V0 1/7] dt-bindings: mt8183: Add binding for FD shared memory

2019-02-19 Thread Jerry-ch Chen
This patch adds the binding for describing the shared memory used to exchange meta data between the co-processor and Face Detection (FD) unit of the camera system on Mediatek SoCs. Signed-off-by: Jerry-ch Chen --- .../mediatek,reserve-memory-fd_smem.txt| 44 ++ 1

Re: [RFC PATCH V0 7/7] [media] platform: mtk-isp: Add Mediatek ISP Pass 1 driver

2019-02-19 Thread Jungo Lin
On Tue, 2019-02-19 at 17:51 +0900, Tomasz Figa wrote: > Hi Jungo, > > On Sun, Feb 17, 2019 at 11:56 AM Jungo Lin wrote: > > > > On Wed, 2019-02-13 at 18:50 +0900, Tomasz Figa wrote: > > > (() . ( strHi Frederic, Jungo, > > > > > > On Tue, Feb 5, 2019 at 3:43 PM Frederic Chen > > > wrote: > > >

Re: [PATCH 01/10] media: Introduce helpers to fill pixel format structs

2019-02-19 Thread Tomasz Figa
On Thu, Feb 7, 2019 at 1:36 AM Hans Verkuil wrote: > > On 2/6/19 5:22 PM, Ezequiel Garcia wrote: > > On Wed, 2019-02-06 at 11:43 +0100, Hans Verkuil wrote: > >> Hi Ezequiel, > >> > >> A quick review below. This looks really useful, BTW. > >> > >> On 2/5/19 9:24 PM, Ezequiel Garcia wrote: [snip] >

[RESEND PATCH 5/7] IB/hfi1: Use the new FOLL_LONGTERM flag to get_user_pages_fast()

2019-02-19 Thread ira . weiny
From: Ira Weiny Use the new FOLL_LONGTERM to get_user_pages_fast() to protect against FS DAX pages being mapped. Signed-off-by: Ira Weiny --- drivers/infiniband/hw/hfi1/user_pages.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/infiniband/hw/hfi1/user_pages.

[RESEND PATCH 0/7] Add FOLL_LONGTERM to GUP fast and use it

2019-02-19 Thread ira . weiny
From: Ira Weiny Resending these as I had only 1 minor comment which I believe we have covered in this series. I was anticipating these going through the mm tree as they depend on a cleanup patch there and the IB changes are very minor. But they could just as well go through the IB tree. NOTE:

[RESEND PATCH 7/7] IB/mthca: Use the new FOLL_LONGTERM flag to get_user_pages_fast()

2019-02-19 Thread ira . weiny
From: Ira Weiny Use the new FOLL_LONGTERM to get_user_pages_fast() to protect against FS DAX pages being mapped. Signed-off-by: Ira Weiny --- drivers/infiniband/hw/mthca/mthca_memfree.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/infiniband/hw/mthca/mthca_memf

[RESEND PATCH 1/7] mm/gup: Replace get_user_pages_longterm() with FOLL_LONGTERM

2019-02-19 Thread ira . weiny
From: Ira Weiny Rather than have a separate get_user_pages_longterm() call, introduce FOLL_LONGTERM and change the longterm callers to use it. This patch does not change any functionality. FOLL_LONGTERM can only be supported with get_user_pages() as it requires vmas to determine if DAX is in us

[RESEND PATCH 6/7] IB/qib: Use the new FOLL_LONGTERM flag to get_user_pages_fast()

2019-02-19 Thread ira . weiny
From: Ira Weiny Use the new FOLL_LONGTERM to get_user_pages_fast() to protect against FS DAX pages being mapped. Signed-off-by: Ira Weiny --- drivers/infiniband/hw/qib/qib_user_sdma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/infiniband/hw/qib/qib_user_sdma.c

[RESEND PATCH 2/7] mm/gup: Change write parameter to flags in fast walk

2019-02-19 Thread ira . weiny
From: Ira Weiny In order to support more options in the GUP fast walk, change the write parameter to flags throughout the call stack. This patch does not change functionality and passes FOLL_WRITE where write was previously used. Signed-off-by: Ira Weiny --- mm/gup.c | 52

[RESEND PATCH 3/7] mm/gup: Change GUP fast to use flags rather than a write 'bool'

2019-02-19 Thread ira . weiny
From: Ira Weiny To facilitate additional options to get_user_pages_fast() change the singular write parameter to be gup_flags. This patch does not change any functionality. New functionality will follow in subsequent patches. Some of the get_user_pages_fast() call sites were unchanged because

[RESEND PATCH 4/7] mm/gup: Add FOLL_LONGTERM capability to GUP fast

2019-02-19 Thread ira . weiny
From: Ira Weiny DAX pages were previously unprotected from longterm pins when users called get_user_pages_fast(). Use the new FOLL_LONGTERM flag to check for DEVMAP pages and fall back to regular GUP processing if a DEVMAP page is encountered. Signed-off-by: Ira Weiny --- mm/gup.c | 24 ++

cron job: media_tree daily build: ERRORS

2019-02-19 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: Wed Feb 20 05:00:11 CET 2019 media-tree git hash:9fabe1d108ca4755a880de43f751f1c054f8894d media_build git

[PATCH] media: imx: vdic: Restore default case to prepare_vdi_in_buffers()

2019-02-19 Thread Steve Longerbeam
Restore a default case to prepare_vdi_in_buffers() to fix the following smatch errors: drivers/staging/media/imx/imx-media-vdic.c:236 prepare_vdi_in_buffers() error: uninitialized symbol 'prev_phys'. drivers/staging/media/imx/imx-media-vdic.c:237 prepare_vdi_in_buffers() error: uninitialized sym

[PATCH v5 5/7] gpu: ipu-v3: ipu-ic: Add support for limited range encoding

2019-02-19 Thread Steve Longerbeam
Add support for the following conversions: - YUV full-range to YUV limited-range - YUV limited-range to YUV full-range - YUV limited-range to RGB full-range - RGB full-range to YUV limited-range The last two conversions require operating on the YUV full-range encoding and inverse encoding coeffic

[PATCH v5 6/7] media: imx: Try colorimetry at both sink and source pads

2019-02-19 Thread Steve Longerbeam
The colorimetry parameters need to be tested at both sink and source pads. Specifically, for prpencvf, the IC only supports RGB full-range quantization at input and output. Fix this by cleaning up imx_media_fill_default_mbus_fields(), renaming to imx_media_try_colorimetry(), and call it at both si

[PATCH v5 7/7] media: imx: Allow BT.709 encoding for IC routes

2019-02-19 Thread Steve Longerbeam
The IC now supports BT.709 Y'CbCr encoding, in addition to existing BT.601 encoding, so allow both, for pipelines that route through the IC. Reported-by: Tim Harvey Signed-off-by: Steve Longerbeam --- Changes in v5: - rebased this patch on top of repurposing the function to imx_media_try_color

[PATCH v5 1/7] gpu: ipu-v3: ipu-ic: Fix saturation bit offset in TPMEM

2019-02-19 Thread Steve Longerbeam
The saturation bit was being set at bit 9 in the second 32-bit word of the TPMEM CSC. This isn't correct, the saturation bit is bit 42, which is bit 10 of the second word. Fixes: 1aa8ea0d2bd5d ("gpu: ipu-v3: Add Image Converter unit") Signed-off-by: Steve Longerbeam Cc: sta...@vger.kernel.org --

[PATCH v5 4/7] gpu: ipu-v3: ipu-ic: Add support for Rec.709 encoding

2019-02-19 Thread Steve Longerbeam
Add support for Rec.709 encoding and inverse encoding. The determination of the CSC coefficients based on the input/output colorspace parameters are moved to a new function calc_csc_coeffs(). Reported-by: Tim Harvey Signed-off-by: Steve Longerbeam --- Changes in v5: - moved API changes to a pre

[PATCH v5 2/7] gpu: ipu-v3: ipu-ic: Fix BT.601 coefficients

2019-02-19 Thread Steve Longerbeam
The ycbcr2rgb and inverse rgb2ycbcr tables define the BT.601 Y'CbCr encoding coefficients. The rgb2ycbcr table specifically describes the BT.601 encoding from full range RGB to full range YUV. Add table comments to make this more clear. The ycbcr2rgb inverse table describes encoding YUV limited r

[PATCH v5 3/7] gpu: ipu-v3: ipu-ic: Fully describe colorspace conversions

2019-02-19 Thread Steve Longerbeam
Only providing the input and output RGB/YUV space to the IC task init functions is not sufficient. To fully characterize a colorspace conversion, the colorspace (chromaticities), Y'CbCr encoding standard, and quantization also need to be specified. Define a 'struct ipu_ic_colorspace' that includes

[PATCH v5 0/7] media: imx: Add support for BT.709 encoding

2019-02-19 Thread Steve Longerbeam
This patchset adds support for the BT.709 encoding and inverse encoding matrices to the ipu_ic task init functions. The imx-media driver can now support both BT.601 and BT.709 encoding. History: v5: - the hard-coded encode coefficients now convert only between full-range quantization. A new func

Re: [PATCH] Add suport for Avermedia TD310

2019-02-19 Thread Jose Alberto Reguero
El 19 de febrero de 2019 22:57:31 CET, Sean Young escribió: >On Mon, Feb 11, 2019 at 11:13:25PM +0100, Jose Alberto Reguero wrote: >> This patch add support for Avermedia TD310 usb stik. >> This patch must be aplied after this one: >> https://patchwork.linuxtv.org/patch/40087/ > >That patch is mar

Re: [PATCH] Add suport for Avermedia TD310

2019-02-19 Thread Sean Young
On Mon, Feb 11, 2019 at 11:13:25PM +0100, Jose Alberto Reguero wrote: > This patch add support for Avermedia TD310 usb stik. > This patch must be aplied after this one: > https://patchwork.linuxtv.org/patch/40087/ That patch is marked Changes Requested. Please fix that patch first, and then re-sub

Re: DVB-T2 Stick

2019-02-19 Thread Sean Young
Hi, On Wed, Jan 30, 2019 at 11:32:12AM +0100, Gonsolo wrote: > Hi! > > The following patch adds support for the Logilink VG0022A DVB-T2 stick. > After patching and building the kernel it shows up with lsusb and I > used w_scan to scan for channels and vlc for watching. > The original patches were

Re: [PATCH 2/3] media: vicodec: avoic clang frame size warning

2019-02-19 Thread Hans Verkuil
On 2/19/19 6:01 PM, Arnd Bergmann wrote: > Clang-9 makes some different inlining decisions compared to gcc, which > leads to a warning about a possible stack overflow problem when building > with CONFIG_KASAN, including when setting asan-stack=0, which avoids > most other frame overflow warnings: >

Re: [PATCH 2/3] media: vicodec: avoic clang frame size warning

2019-02-19 Thread Arnd Bergmann
On Tue, Feb 19, 2019 at 8:02 PM Nick Desaulniers wrote: > On Tue, Feb 19, 2019 at 9:02 AM Arnd Bergmann wrote: > > > > Clang-9 makes some different inlining decisions compared to gcc, which > > leads to a warning about a possible stack overflow problem when building > > with CONFIG_KASAN, includi

Re: [PATCH 2/3] media: vicodec: avoic clang frame size warning

2019-02-19 Thread Nick Desaulniers
On Tue, Feb 19, 2019 at 9:02 AM Arnd Bergmann wrote: > > Clang-9 makes some different inlining decisions compared to gcc, which > leads to a warning about a possible stack overflow problem when building > with CONFIG_KASAN, including when setting asan-stack=0, which avoids > most other frame overf

Re: [PATCH 1/3] media: saa7146: avoid high stack usage with clang

2019-02-19 Thread Nick Desaulniers
On Tue, Feb 19, 2019 at 9:02 AM Arnd Bergmann wrote: > > Two saa7146/hexium files contain a construct that causes a warning > when built with clang: > > drivers/media/pci/saa7146/hexium_orion.c:210:12: error: stack frame size of > 2272 bytes in function 'hexium_probe' > [-Werror,-Wframe-lar

[PATCH 3/3] media: go7007: avoid clang frame overflow warning with KASAN

2019-02-19 Thread Arnd Bergmann
clang-8 warns about one function here when KASAN is enabled, even without the 'asan-stack' option: drivers/media/usb/go7007/go7007-fw.c:1551:5: warning: stack frame size of 2656 bytes in function I have reported this issue in the llvm bugzilla, but to make it work with the clang-8 release, a sma

[PATCH 2/3] media: vicodec: avoic clang frame size warning

2019-02-19 Thread Arnd Bergmann
Clang-9 makes some different inlining decisions compared to gcc, which leads to a warning about a possible stack overflow problem when building with CONFIG_KASAN, including when setting asan-stack=0, which avoids most other frame overflow warnings: drivers/media/platform/vicodec/codec-fwht.c:673:1

[PATCH 1/3] media: saa7146: avoid high stack usage with clang

2019-02-19 Thread Arnd Bergmann
Two saa7146/hexium files contain a construct that causes a warning when built with clang: drivers/media/pci/saa7146/hexium_orion.c:210:12: error: stack frame size of 2272 bytes in function 'hexium_probe' [-Werror,-Wframe-larger-than=] static int hexium_probe(struct saa7146_dev *dev)

Re: [PATCH 13/14] media: Documentation: fix several typos

2019-02-19 Thread Philipp Zabel
On Mon, Feb 18, 2019 at 02:29:07PM -0500, Mauro Carvalho Chehab wrote: [...] > diff --git a/Documentation/media/v4l-drivers/imx.rst > b/Documentation/media/v4l-drivers/imx.rst > index 9314af00d067..1d7eb8c7bd5c 100644 > --- a/Documentation/media/v4l-drivers/imx.rst > +++ b/Documentation/media/v4l-

Re: [PATCH 12/14] media: include: fix several typos

2019-02-19 Thread Lad, Prabhakar
Hi Mauro, On Mon, Feb 18, 2019 at 7:29 PM Mauro Carvalho Chehab wrote: > > Use codespell to fix lots of typos over frontends. > > Manually verified to avoid false-positives. > > Signed-off-by: Mauro Carvalho Chehab > --- > include/media/davinci/dm355_ccdc.h | 4 ++-- > include/media/dav

Re: [PATCH] Add support to reset device controls

2019-02-19 Thread Laurent Pinchart
Hi Kieran, Thank you for the patch. On Mon, Dec 03, 2018 at 12:03:31PM +, Kieran Bingham wrote: > Provide a new option '--reset-controls' which will enumerate the > available controls on a device or sub-device, and re-initialise them to > defaults. > > Signed-off-by: Kieran Bingham > > ---

[PATCH 1/2] yavta: Refactor video_list_controls()

2019-02-19 Thread Laurent Pinchart
Separate iteration over controls from printing, in order to reuse the iteration to implement control reset. Signed-off-by: Laurent Pinchart --- yavta.c | 134 +++- 1 file changed, 83 insertions(+), 51 deletions(-) diff --git a/yavta.c b/yavta.

[PATCH 2/2] Add support to reset device controls

2019-02-19 Thread Laurent Pinchart
From: Kieran Bingham Provide a new option '--reset-controls' which will enumerate the available controls on a device or sub-device, and re-initialise them to defaults. Signed-off-by: Kieran Bingham Signed-off-by: Laurent Pinchart --- yavta.c | 45 +

sysfs: cannot create duplicate filename '/devices/pci0000:00/0000:00:14.0/usb1/1-3/dvb/dvb0.frontend0'

2019-02-19 Thread Brüns , Stefan
Hi Frank, this is likely the same problem as the one covered here: https://patchwork.linuxtv.org/patch/54044/ The CT2-4400 uses the dvbsky_t330_props. Kind regards, Stefan

Re: [BUG] Regression caused by "media: gspca: convert to vb2"

2019-02-19 Thread Matti Hämäläinen
On Tue, 19 Feb 2019, Hans Verkuil wrote: Hi Matti, On 2/19/19 6:30 AM, Matti Hämäläinen wrote: Hello! Last week while testing some webcams that use gspca-based v4l2 drivers, I noticed that the driver was spewing some errors in klog whenever the program using them issued VIDIOC_STREAMOFF ioct

Re: [PATCH 1/2] media: ipu3: shut up warnings produced with W=1

2019-02-19 Thread Mauro Carvalho Chehab
Em Tue, 19 Feb 2019 16:03:17 +0200 Sakari Ailus escreveu: > Hi Mauro, > > On Tue, Feb 19, 2019 at 09:00:29AM -0500, Mauro Carvalho Chehab wrote: > > There are lots of warnings produced by this driver. It is not > > as much as atomisp, but it is still a lot. > > > > So, use the same solution to

[ragnatech:media-tree] BUILD SUCCESS b3c786566d8f3f69b9f4144c2707db74158caf9a

2019-02-19 Thread kbuild test robot
tree/branch: git://git.ragnatech.se/linux media-tree branch HEAD: b3c786566d8f3f69b9f4144c2707db74158caf9a media: MAINTAINERS: add entry for Freescale i.MX7 media driver elapsed time: 836m configs tested: 143 The following configs have been built successfully. More configs may be tested in th

[ragnatech:media-tree 306/312] drivers/staging/media/imx/imx7-media-csi.c:1085:38: sparse: error: not enough arguments for function imx_media_mbus_fmt_to_pix_fmt

2019-02-19 Thread kbuild test robot
tree: git://git.ragnatech.se/linux media-tree head: b3c786566d8f3f69b9f4144c2707db74158caf9a commit: 05f634040c0d05f59f2dcd39722157cb3b57c85b [306/312] media: staging/imx7: add imx7 CSI subdev driver reproduce: # apt-get install sparse git checkout 05f634040c0d05f59f2dcd3972215

Re: [PATCH 2/2] ipu3-mmu: fix some kernel-doc macros

2019-02-19 Thread Sakari Ailus
On Tue, Feb 19, 2019 at 09:00:30AM -0500, Mauro Carvalho Chehab wrote: > Some kernel-doc markups are wrong. fix them. > > Signed-off-by: Mauro Carvalho Chehab Acked-by: Sakari Ailus -- Sakari Ailus sakari.ai...@linux.intel.com

Re: [PATCH 1/2] media: ipu3: shut up warnings produced with W=1

2019-02-19 Thread Sakari Ailus
Hi Mauro, On Tue, Feb 19, 2019 at 09:00:29AM -0500, Mauro Carvalho Chehab wrote: > There are lots of warnings produced by this driver. It is not > as much as atomisp, but it is still a lot. > > So, use the same solution to hide most of them. > Those need to be fixed before promoting it out of sta

[PATCH 2/2] ipu3-mmu: fix some kernel-doc macros

2019-02-19 Thread Mauro Carvalho Chehab
Some kernel-doc markups are wrong. fix them. Signed-off-by: Mauro Carvalho Chehab --- drivers/staging/media/ipu3/ipu3-mmu.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/media/ipu3/ipu3-mmu.c b/drivers/staging/media/ipu3/ipu3-mmu.c index cd2038b22b55..

[PATCH 1/2] media: ipu3: shut up warnings produced with W=1

2019-02-19 Thread Mauro Carvalho Chehab
There are lots of warnings produced by this driver. It is not as much as atomisp, but it is still a lot. So, use the same solution to hide most of them. Those need to be fixed before promoting it out of staging, so add it at the TODO list. Signed-off-by: Mauro Carvalho Chehab --- drivers/stagin

[PATCH 1/2] media: imx7-media-csi: don't store a floating pointer

2019-02-19 Thread Mauro Carvalho Chehab
if imx7_csi_try_fmt() fails, outcc variable won't be initialized and csi->cc[IMX7_CSI_PAD_SRC] would be pointing to a random location. Signed-off-by: Mauro Carvalho Chehab --- drivers/staging/media/imx/imx7-media-csi.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) di

[PATCH 2/2] media: imx7-media-csi: get rid of unused var

2019-02-19 Thread Mauro Carvalho Chehab
drivers/staging/media/imx/imx7-media-csi.c: In function 'imx7_csi_enum_mbus_code': drivers/staging/media/imx/imx7-media-csi.c:926:33: warning: variable 'in_cc' set but not used [-Wunused-but-set-variable] const struct imx_media_pixfmt *in_cc;

Re: v4l2 mem2mem compose support?

2019-02-19 Thread Jean-Michel Hautbois
Hi Tim, Nicolas, all, Le sam. 16 févr. 2019 à 22:14, Nicolas Dufresne a écrit : > > Le sam. 16 févr. 2019 à 13:40, Hans Verkuil a écrit : > > > > On 2/16/19 4:42 PM, Nicolas Dufresne wrote: > > > Le sam. 16 févr. 2019 à 04:48, Hans Verkuil a écrit : > > >> > > >> On 2/16/19 10:42 AM, Hans Verku

overdue payment

2019-02-19 Thread First Commercial Bank London

Re: [BUG] Regression caused by "media: gspca: convert to vb2"

2019-02-19 Thread Hans Verkuil
Hi Matti, On 2/19/19 6:30 AM, Matti Hämäläinen wrote: > > Hello! > > Last week while testing some webcams that use gspca-based v4l2 drivers, I > noticed that the driver was spewing some errors in klog whenever the > program using them issued VIDIOC_STREAMOFF ioctl. This seems to be a > regress

Re: [PATCH v2 10/10] media: vicodec: Add support for stateless decoder.

2019-02-19 Thread Hans Verkuil
On 2/15/19 2:09 PM, Dafna Hirschfeld wrote: > Implement a stateless decoder for the new node. > > Signed-off-by: Dafna Hirschfeld > --- > .../media/platform/vicodec/codec-v4l2-fwht.h | 1 + > drivers/media/platform/vicodec/vicodec-core.c | 325 -- > include/uapi/linux/v4l2-con

Re: [PATCH v2 06/10] media: vicodec: Introducing stateless fwht defs and structs

2019-02-19 Thread Hans Verkuil
On 2/15/19 2:05 PM, Dafna Hirschfeld wrote: > Add structs and definitions needed to implement stateless > decoder for fwht. > > Signed-off-by: Dafna Hirschfeld > --- > drivers/media/platform/vicodec/vicodec-core.c | 12 > drivers/media/v4l2-core/v4l2-ctrls.c | 6 ++ >

Re: [PATCH v2 04/10] media: vicodec: keep the ref frame according to the format in decoder

2019-02-19 Thread Hans Verkuil
On 2/19/19 11:30 AM, Hans Verkuil wrote: > On 2/15/19 2:05 PM, Dafna Hirschfeld wrote: >> In the decoder, save the inner reference frame in the same >> format as the capture buffer. >> The decoder writes directly to the capture buffer and then >> the capture buffer is copied to the reference buffer

Re: [PATCH v2 04/10] media: vicodec: keep the ref frame according to the format in decoder

2019-02-19 Thread Hans Verkuil
On 2/15/19 2:05 PM, Dafna Hirschfeld wrote: > In the decoder, save the inner reference frame in the same > format as the capture buffer. > The decoder writes directly to the capture buffer and then > the capture buffer is copied to the reference buffer. > This will simplify the stateless decoder. >

Re: [PATCH v2 07/10] media: vicodec: Register another node for stateless decoder

2019-02-19 Thread Hans Verkuil
On 2/15/19 2:05 PM, Dafna Hirschfeld wrote: > Add stateless decoder instance field to the dev struct and > register another node for the statelsess decoder. > The stateless API for the node will be implemented in further patches. > > Signed-off-by: Dafna Hirschfeld > --- > drivers/media/platform

Re: [PATCH] imx7-media-csi.c: fix merge breakage

2019-02-19 Thread Rui Miguel Silva
Hi Hans, On Tue 19 Feb 2019 at 07:38, Hans Verkuil wrote: Commit 5964cbd86922 ("imx: Set capture compose rectangle in capture_device_set_format") broke the compilation of commit 05f634040c0d ("staging/imx7: add imx7 CSI subdev driver"). These patches came in through different pull requests and n

Re: [PATCH v2 08/10] media: vicodec: call v4l2_m2m_buf_copy_metadata also upon error

2019-02-19 Thread Hans Verkuil
On 2/15/19 2:05 PM, Dafna Hirschfeld wrote: > call v4l2_m2m_buf_copy_metadata also if decoding/encoding > ends with status VB2_BUF_STATE_ERROR. Is this a bug fix? Why is this needed? The commit log can use a bit more work :-) Also, I don't think this has anything to do with the stateless codec,

Re: [PATCH v2 06/10] media: vicodec: Introducing stateless fwht defs and structs

2019-02-19 Thread Hans Verkuil
On 2/15/19 2:05 PM, Dafna Hirschfeld wrote: > Add structs and definitions needed to implement stateless > decoder for fwht. > > Signed-off-by: Dafna Hirschfeld > --- > drivers/media/platform/vicodec/vicodec-core.c | 12 > drivers/media/v4l2-core/v4l2-ctrls.c | 6 ++ >

[PATCH v4l-utils 1/2] ir-ctl/keytable: add see also to reference to man pages

2019-02-19 Thread Sean Young
Add hint to how you can start to figure out what protocol a remote uses. Signed-off-by: Sean Young --- utils/ir-ctl/ir-ctl.1.in| 3 +++ utils/keytable/ir-keytable.1.in | 9 + 2 files changed, 12 insertions(+) diff --git a/utils/ir-ctl/ir-ctl.1.in b/utils/ir-ctl/ir-ctl.1.in index

[PATCH v4l-utils 2/2] rc-mm protocol support

2019-02-19 Thread Sean Young
Signed-off-by: Sean Young --- utils/common/ir-encode.c | 3 +++ utils/keytable/keytable.c | 2 ++ 2 files changed, 5 insertions(+) diff --git a/utils/common/ir-encode.c b/utils/common/ir-encode.c index ccc75032..4bd1b694 100644 --- a/utils/common/ir-encode.c +++ b/utils/common/ir-encode.c @@ -3

Re: [PATCH 07/14] media: rc: fix several typos

2019-02-19 Thread Sean Young
On Mon, Feb 18, 2019 at 02:29:01PM -0500, Mauro Carvalho Chehab wrote: > Use codespell to fix lots of typos over frontends. > > Manually verified to avoid false-positives. > > Signed-off-by: Mauro Carvalho Chehab Acked-by: Sean Young > --- > drivers/media/rc/Kconfig

Re: [PATCH] media: vicodec: selection api should not care about signal/multiplanar

2019-02-19 Thread Hans Verkuil
On 2/19/19 10:43 AM, Dafna Hirschfeld wrote: > Change the selection api to always accept both signal and > multiplanar buffer types. > > Signed-off-by: Dafna Hirschfeld > --- > drivers/media/platform/vicodec/vicodec-core.c | 25 +++ > 1 file changed, 9 insertions(+), 16 deletions

[PATCH] media: vicodec: selection api should not care about signal/multiplanar

2019-02-19 Thread Dafna Hirschfeld
Change the selection api to always accept both signal and multiplanar buffer types. Signed-off-by: Dafna Hirschfeld --- drivers/media/platform/vicodec/vicodec-core.c | 25 +++ 1 file changed, 9 insertions(+), 16 deletions(-) diff --git a/drivers/media/platform/vicodec/vicodec-co

[PATCH] media: staging/imx: Allow driver to build if COMPILE_TEST is enabled

2019-02-19 Thread Javier Martinez Canillas
The driver has runtime but no build time dependency with IMX_IPUV3_CORE, so can be built for testing purposes if COMPILE_TEST option is enabled. This is useful to have more build coverage and make sure that the driver is not affected by changes that could cause build regressions. Signed-off-by: J

Re: [RFC PATCH V0 7/7] [media] platform: mtk-isp: Add Mediatek ISP Pass 1 driver

2019-02-19 Thread Tomasz Figa
Hi Jungo, On Sun, Feb 17, 2019 at 11:56 AM Jungo Lin wrote: > > On Wed, 2019-02-13 at 18:50 +0900, Tomasz Figa wrote: > > (() . ( strHi Frederic, Jungo, > > > > On Tue, Feb 5, 2019 at 3:43 PM Frederic Chen > > wrote: > > > > > > From: Jungo Lin > > > > > > This patch adds the driver for Pass u