All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Niklas Söderlund" <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,
	"Niklas Söderlund" <niklas.soderlund+renesas@ragnatech.se>
Subject: [PATCH 0/3] v4l2-dev/ioctl: Add V4L2_CAP_IO_MC
Date: Tue, 15 Oct 2019 16:35:49 +0200	[thread overview]
Message-ID: <20191015143552.317669-1-niklas.soderlund+renesas@ragnatech.se> (raw)

Hi,

First of this series is based on work done by Helen in 2017 [1], I have 
synced with her that it's OK for me to takeover the work.

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/3 adds the core parts of this change by adding a new video 
device capability flag V4L2_CAP_IO_MC which if set provides helper 
implementations for the get, set and enum inputs and outputs ioctrls 
freeing the video device driver from the need to implement them.

Patch 2/3 and 3/3 converts the R-Car VIN and Intel IPU3 drivers to use 
this new 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.

1. https://patchwork.linuxtv.org/patch/41857/

Niklas Söderlund (3):
  v4l2-dev/ioctl: Add V4L2_CAP_IO_MC
  rcar-vin: Make use of V4L2_CAP_IO_MC
  staging/intel-ipu3: Make use of V4L2_CAP_IO_MC

 .../media/uapi/v4l/vidioc-querycap.rst        |  3 +
 .../media/videodev2.h.rst.exceptions          |  1 +
 drivers/media/platform/rcar-vin/rcar-v4l2.c   | 17 +---
 drivers/media/v4l2-core/v4l2-dev.c            | 24 +++--
 drivers/media/v4l2-core/v4l2-ioctl.c          | 87 ++++++++++++++++++-
 drivers/staging/media/ipu3/ipu3-v4l2.c        | 60 +------------
 include/uapi/linux/videodev2.h                |  2 +
 7 files changed, 110 insertions(+), 84 deletions(-)

-- 
2.23.0


             reply	other threads:[~2019-10-15 14:37 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-15 14:35 Niklas Söderlund [this message]
2019-10-15 14:35 ` [PATCH 1/3] v4l2-dev/ioctl: Add V4L2_CAP_IO_MC Niklas Söderlund
2019-10-16 13:56   ` Hans Verkuil
2019-11-15 23:47     ` Niklas Söderlund
2019-10-15 14:35 ` [PATCH 2/3] rcar-vin: Make use of V4L2_CAP_IO_MC Niklas Söderlund
2019-10-15 14:35 ` [PATCH 3/3] staging/intel-ipu3: " Niklas Söderlund

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20191015143552.317669-1-niklas.soderlund+renesas@ragnatech.se \
    --to=niklas.soderlund+renesas@ragnatech.se \
    --cc=helen.koike@collabora.com \
    --cc=hverkuil@xs4all.nl \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=sakari.ailus@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.