| Message ID | 20200413202351.1359754-1-niklas.soderlund+renesas@ragnatech.se (mailing list archive) |
|---|---|
| Headers | show
Return-Path: <SRS0=YRPh=55=vger.kernel.org=linux-media-owner@kernel.org>
Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org
[172.30.200.123])
by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id E552B17D4
for <patchwork-linux-media@patchwork.kernel.org>;
Mon, 13 Apr 2020 20:26:03 +0000 (UTC)
Received: from vger.kernel.org (vger.kernel.org [23.128.96.18])
by mail.kernel.org (Postfix) with ESMTP id CB82720663
for <patchwork-linux-media@patchwork.kernel.org>;
Mon, 13 Apr 2020 20:26:03 +0000 (UTC)
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
id S2388364AbgDMU0C (ORCPT
<rfc822;patchwork-linux-media@patchwork.kernel.org>);
Mon, 13 Apr 2020 16:26:02 -0400
Received: from bin-mail-out-05.binero.net ([195.74.38.228]:21022 "EHLO
bin-mail-out-05.binero.net" rhost-flags-OK-OK-OK-OK)
by vger.kernel.org with ESMTP id S2388320AbgDMU0B (ORCPT
<rfc822;linux-media@vger.kernel.org>);
Mon, 13 Apr 2020 16:26:01 -0400
X-Halon-ID: f0cd14a2-7dc4-11ea-aeed-005056917f90
Authorized-sender: niklas@soderlund.pp.se
Received: from bismarck.berto.se (p4fca2392.dip0.t-ipconnect.de
[79.202.35.146])
by bin-vsp-out-02.atm.binero.net (Halon) with ESMTPA
id f0cd14a2-7dc4-11ea-aeed-005056917f90;
Mon, 13 Apr 2020 22:25:51 +0200 (CEST)
From: =?utf-8?q?Niklas_S=C3=B6derlund?=
<niklas.soderlund+renesas@ragnatech.se>
To: Helen Koike <helen.koike@collabora.com>,
Hans Verkuil <hverkuil@xs4all.nl>,
Sakari Ailus <sakari.ailus@linux.intel.com>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
linux-media@vger.kernel.org
Cc: linux-renesas-soc@vger.kernel.org,
=?utf-8?q?Niklas_S=C3=B6derlund?= <niklas.soderlund+renesas@ragnatech.se>
Subject: [PATCH v7 0/6] v4l2-dev/ioctl: Add V4L2_CAP_IO_MC
Date: Mon, 13 Apr 2020 22:23:45 +0200
Message-Id: <20200413202351.1359754-1-niklas.soderlund+renesas@ragnatech.se>
X-Mailer: git-send-email 2.26.0
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Sender: linux-media-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-media.vger.kernel.org>
X-Mailing-List: linux-media@vger.kernel.org
|
| Series |
v4l2-dev/ioctl: Add V4L2_CAP_IO_MC
|
expand
|
Hi, This series aims to reduce the amount of boiler plate code in video device drivers who's inputs and/or outputs are controlled by the Media Controller instead of the V4L2 API. Patch 1/6 adds a new video device capability flag V4L2_CAP_IO_MC which if set provides helper implementations for the enum inputs and outputs ioctls freeing the video device driver from the need to implement them. Patch 2/6 fix initialization of reserved fields in the ivtv driver which becomes a problem in 3/6 where Laurent adds mbus filters to VIDIOC_ENUM_FMT. Patch 4/6, 5/6 and 6/6 converts the R-Car VIN, Intel IPU3 and VIMC drivers to use the new default handlers and capability flag and delete the now redundant boiler plate code. I'm sure more video device drivers can make use of this new flag but as I can only test on these two platforms I have limited my changes to those. A separate patch to v4l-utils have been posted as [1] to add a test for this feature in v4l2-compliance. 1. [PATCH 0/2] v4l2-compliance: add tests for V4L2_CAP_IO_MC Laurent Pinchart (1): media: v4l2: Extend VIDIOC_ENUM_FMT to support MC-centric devices Niklas Söderlund (5): v4l2-dev/ioctl: Add V4L2_CAP_IO_MC ivtv-ioctl.c: Do not initialize the reserved field of struct v4l2_fmtdesc rcar-vin: Make use of V4L2_CAP_IO_MC staging/intel-ipu3: Make use of V4L2_CAP_IO_MC vimc: Make use of V4L2_CAP_IO_MC .../media/uapi/v4l/vidioc-enum-fmt.rst | 17 ++++- .../media/uapi/v4l/vidioc-querycap.rst | 6 ++ .../media/videodev2.h.rst.exceptions | 1 + drivers/media/pci/ivtv/ivtv-ioctl.c | 4 -- drivers/media/platform/rcar-vin/rcar-v4l2.c | 40 ++++++----- drivers/media/platform/vimc/vimc-capture.c | 10 ++- drivers/media/v4l2-core/v4l2-dev.c | 25 +++++-- drivers/media/v4l2-core/v4l2-ioctl.c | 70 +++++++++++++++++-- drivers/staging/media/ipu3/ipu3-v4l2.c | 64 ++--------------- include/uapi/linux/videodev2.h | 5 +- 10 files changed, 147 insertions(+), 95 deletions(-)