All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hans Verkuil <hverkuil@xs4all.nl>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	sadegh abbasi <sadegh612000@yahoo.co.uk>
Cc: "linux-media@vger.kernel.org" <linux-media@vger.kernel.org>
Subject: Re: [PATCH v2 1/6] v4l2-ctrls: Add new S8, S16 and S32 compound control types
Date: Mon, 13 Jul 2015 10:12:30 +0200	[thread overview]
Message-ID: <55A372EE.4040103@xs4all.nl> (raw)
In-Reply-To: <1738054.4vofqz5FHv@avalon>

On 07/13/2015 12:18 AM, Laurent Pinchart wrote:
> Hi Sadegh,
> 
> On Friday 10 July 2015 15:45:23 sadegh abbasi wrote:
>> Hi Hans / Laurent,
>> Just wondering what has happened to these patches. I used them in my driver
>> and can not find them in 4.1 release. Have they been rejected?
> 
> Not exactly. The changes to v4l2-ctrls were considered to be fine, but we have 
> a policy not to merge core changes without at least one driver using them. As 
> the OMAP4 ISS part of the series still needs work, nothing got merged.
> 
> Hans, you mentioned you wanted to look at the RGB2RGB controls in a wider 
> context (including the adv drivers for instance). Do you have anything to 
> report ?

Yes and no :-)

It's part of my work to support colorspace conversion hardware. Now for 4.2 I merged
a lot of patches that pave the way for the remaining code to go in. But that remaining
code still needs to be cleaned up. My code can be found here:

http://git.linuxtv.org/cgit.cgi/hverkuil/media_tree.git/log/?h=csc

Unfortunately it's on hold because the CEC framework has priority for me (and Cisco).

In addition, for drm Intel is working on color manager code as well (see the patch
series titled "Color Manager Implementation" at dri-devel). I would like to be able
to share the low-level matrix/vector calculation code with them. So I am waiting to
see how that turns out.

So quite some work has been done, but it's not ready for merging.

Regards,

	Hans

  reply	other threads:[~2015-07-13  8:13 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-28  9:17 [PATCH v2 0/6] omap4iss: Add RGB2RGB blending matrix support Laurent Pinchart
2015-01-28  9:17 ` [PATCH v2 1/6] v4l2-ctrls: Add new S8, S16 and S32 compound control types Laurent Pinchart
2015-01-28 10:19   ` Hans Verkuil
     [not found]     ` <532636346.4083310.1436543123211.JavaMail.yahoo@mail.yahoo.com>
2015-07-12 22:18       ` Laurent Pinchart
2015-07-13  8:12         ` Hans Verkuil [this message]
2015-01-28  9:17 ` [PATCH v2 2/6] v4l2-ctrls: Don't initialize array tail when setting a control Laurent Pinchart
2015-01-28  9:17 ` [PATCH v2 3/6] v4l2-ctrls: Make the control type init op initialize the whole control Laurent Pinchart
2015-01-28 10:20   ` Hans Verkuil
2015-01-28  9:17 ` [PATCH v2 4/6] v4l2-ctrls: Export the standard control type operations Laurent Pinchart
2015-01-28  9:17 ` [PATCH v2 5/6] staging: media: omap4iss: Cleanup media entities after unregistration Laurent Pinchart
2015-01-28  9:17 ` [PATCH v2 6/6] staging: media: omap4iss: ipipe: Expose the RGB2RGB blending matrix Laurent Pinchart
2015-01-28 10:27   ` Hans Verkuil
2015-01-28 12:18     ` Laurent Pinchart
2015-06-14 10:30       ` Laurent Pinchart

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=55A372EE.4040103@xs4all.nl \
    --to=hverkuil@xs4all.nl \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-media@vger.kernel.org \
    --cc=sadegh612000@yahoo.co.uk \
    /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.