From: "Niklas Söderlund" <niklas.soderlund+renesas@ragnatech.se>
To: 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 v4 0/4] rcar-vin: Add support for RGB formats with alpha component
Date: Thu, 4 Jul 2019 19:15:58 +0200 [thread overview]
Message-ID: <20190704171602.29541-1-niklas.soderlund+renesas@ragnatech.se> (raw)
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-tree and is tested on both
Renesas Gen2 and Gen3 hardware without any regressions found.
Patch 1/4 fixes a badly named register name, 2/4 adds the new control
while 3/4 adds the two new pixel formats. Patch 4/4 is a clean up now
that Gen2 and Gen3 always have controls and v4l2_ctrl_handler_setup()
should be called for both cases.
* Changes since v3
- Fixed up commit message.
- Collected review tags.
* Changes since v2
- Protect the writing of the alpha value when streaming with the spin
lock to make sure the streaming state is stable.
- Add patch 4/4 to call v4l2_ctrl_handler_setup() for the media
controller centric Gen3 mode of the driver.
Niklas Söderlund (4):
rcar-vin: Rename VNDMR_DTMD_ARGB1555 to VNDMR_DTMD_ARGB
rcar-vin: Add control for alpha component
rcar-vin: Add support for RGB formats with alpha component
rcar-vin: Always setup controls when opening video device
drivers/media/platform/rcar-vin/rcar-core.c | 53 ++++++++++++++++++++-
drivers/media/platform/rcar-vin/rcar-dma.c | 44 ++++++++++++++++-
drivers/media/platform/rcar-vin/rcar-v4l2.c | 38 +++++++++------
drivers/media/platform/rcar-vin/rcar-vin.h | 5 ++
4 files changed, 121 insertions(+), 19 deletions(-)
--
2.21.0
next reply other threads:[~2019-07-04 17:16 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-07-04 17:15 Niklas Söderlund [this message]
2019-07-04 17:15 ` [PATCH v4 1/4] rcar-vin: Rename VNDMR_DTMD_ARGB1555 to VNDMR_DTMD_ARGB Niklas Söderlund
2019-07-04 17:16 ` [PATCH v4 2/4] rcar-vin: Add control for alpha component Niklas Söderlund
2019-07-04 17:16 ` [PATCH v4 3/4] rcar-vin: Add support for RGB formats with " Niklas Söderlund
2019-07-04 17:16 ` [PATCH v4 4/4] rcar-vin: Always setup controls when opening video device 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=20190704171602.29541-1-niklas.soderlund+renesas@ragnatech.se \
--to=niklas.soderlund+renesas@ragnatech.se \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).