Message ID | 20210816113909.234872-1-david.plowman@raspberrypi.com (mailing list archive) |
---|---|
Headers |
Received: from vger.kernel.org ([23.128.96.18]) by www.linuxtv.org with esmtp (Exim 4.92) (envelope-from <linux-media-owner@vger.kernel.org>) id 1mFaxh-001EHJ-VA; Mon, 16 Aug 2021 11:39:29 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236140AbhHPLjy (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Mon, 16 Aug 2021 07:39:54 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53172 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236026AbhHPLjy (ORCPT <rfc822;linux-media@vger.kernel.org>); Mon, 16 Aug 2021 07:39:54 -0400 Received: from mail-wm1-x334.google.com (mail-wm1-x334.google.com [IPv6:2a00:1450:4864:20::334]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CCAB3C061764 for <linux-media@vger.kernel.org>; Mon, 16 Aug 2021 04:39:22 -0700 (PDT) Received: by mail-wm1-x334.google.com with SMTP id u1so11411350wmm.0 for <linux-media@vger.kernel.org>; Mon, 16 Aug 2021 04:39:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=raspberrypi.com; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=dhLr6dNru65fF6f/Z2bmJvlD/DzsoJ8cTCEXBqOz4bw=; b=Gy0HWWeyafmUG9ycFVnQIFzhljb1M9jUWviRZlS6zkZz3f4rgR/8B/ZcE1SGNLr/FN Y2pGHsBloXZPvNoCAjudCr95QIitFBFhLRVUNtZ0kFu8Bt7QhxnMcDfdpvGPnu/YkJP+ xiCLUZFHRszcaVW3hTOBYhV4+BM4xGFD5wuL1y0hGPPHN71hlBVTyc1XjYs9JJImFYiH XFjIDnc+61Y0OnO/PtA6klfxsh0/PTnXg0KJavXWtPLK7a7VcRGZTQBYurjRRrQrc3iZ NoC7eKTrcUvvY6+i3kwV6D+BskKvWQ8QTVOa8W5gJoe3XA/dbEyF6MTBpVGi1oeVipKM L+8w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=dhLr6dNru65fF6f/Z2bmJvlD/DzsoJ8cTCEXBqOz4bw=; b=d8oAX3aDB/weCpStwqxtfMwKb972C+Sv70NurfACRkCJM8j4ZA8hWVcVdozjK4SsvZ 7Dpq1EUFfUi0sOxsnfd05XgD/eaPIpBgFQo+2H7jjwVxAPLvwkvaJHYIXNYOsAcIPutb cbbXRff5cJwOyPVLl0ET4R2YTACBn2ylg/wwK1XkqETVuO5rIs+EqsJkxj6tPbjonnC8 8BtmiXDHXj4it/QXQGJw4s0FL6I0FWJ3iMuPVQp+7Lbih+aJxionff+ppfLXwROkDSJd yDqxykhdBhOh81TgJhnOKU9YjoMhO3LhbR0WR/Qk8yi1j+A1Ml34JIjxOnCHiwl0EfGD 0byA== X-Gm-Message-State: AOAM533+lOXmq9q+ZJAbaoQIjhcw5ju6lDp3a7yUDwajjPicBJH34t3b 3zGUT96g0NgqF1NKpQSCtgGzMwsuNUiilw== X-Google-Smtp-Source: ABdhPJwHt7EhZGUXv+RiXFHix8XARDXx4lUPeQctOjGmElDfjDo5BJzrtfpjxp6D/oWkEBIm2ABHDA== X-Received: by 2002:a05:600c:3590:: with SMTP id p16mr14476445wmq.33.1629113961030; Mon, 16 Aug 2021 04:39:21 -0700 (PDT) Received: from davidp-xps-13.lan (plowpeople3.plus.com. [80.229.223.72]) by smtp.gmail.com with ESMTPSA id g9sm10356548wmk.34.2021.08.16.04.39.20 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 16 Aug 2021 04:39:20 -0700 (PDT) From: David Plowman <david.plowman@raspberrypi.com> To: linux-media@vger.kernel.org, sakari.ailus@iki.fi, Laurent Pinchart <laurent.pinchart@ideasonboard.com>, Kieran Bingham <kieran.bingham@ideasonboard.com>, Hans Verkuil <hverkuil-cisco@xs4all.nl>, Mauro Carvalho Chehab <mchehab@kernel.org> Cc: David Plowman <david.plowman@raspberrypi.com> Subject: [PATCH v6 0/2] New V4L2 control V4L2_CID_NOTIFY_GAINS Date: Mon, 16 Aug 2021 12:39:07 +0100 Message-Id: <20210816113909.234872-1-david.plowman@raspberrypi.com> X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-LSpam-Score: -3.2 (---) X-LSpam-Report: No, score=-3.2 required=5.0 tests=BAYES_00=-1.9,DKIM_SIGNED=0.1,DKIM_VALID=-0.1,DKIM_VALID_AU=-0.1,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1,RCVD_IN_DNSWL_LOW=-0.7 autolearn=ham autolearn_force=no |
Series |
New V4L2 control V4L2_CID_NOTIFY_GAINS
|
|
Message
David Plowman
Aug. 16, 2021, 11:39 a.m. UTC
Hi again everyone Thanks for last round of feedback, leading to this v6 pair of patches. I've made the documentation improvements that were suggested, and accordingly added Laurent's "Reviewed-by" tags. Thanks and best regards David David Plowman (2): media: v4l2-ctrls: Add V4L2_CID_NOTIFY_GAINS control media: v4l2-ctrls: Document V4L2_CID_NOTIFY_GAINS control .../media/v4l/ext-ctrls-image-source.rst | 20 +++++++++++++++++++ drivers/media/v4l2-core/v4l2-ctrls-defs.c | 1 + include/uapi/linux/v4l2-controls.h | 1 + 3 files changed, 22 insertions(+)
Comments
Hello, I've reviewed this series and it looks good to me. Sakari, would you like to have a look, or should I send a pull request ? On Mon, Aug 16, 2021 at 12:39:07PM +0100, David Plowman wrote: > Hi again everyone > > Thanks for last round of feedback, leading to this v6 pair of > patches. I've made the documentation improvements that were suggested, > and accordingly added Laurent's "Reviewed-by" tags. > > Thanks and best regards > > David > > David Plowman (2): > media: v4l2-ctrls: Add V4L2_CID_NOTIFY_GAINS control > media: v4l2-ctrls: Document V4L2_CID_NOTIFY_GAINS control > > .../media/v4l/ext-ctrls-image-source.rst | 20 +++++++++++++++++++ > drivers/media/v4l2-core/v4l2-ctrls-defs.c | 1 + > include/uapi/linux/v4l2-controls.h | 1 + > 3 files changed, 22 insertions(+)
Hi Laurent, Sakari, everyone I was wondering whether I might give this another little nudge just so that it doesn't get lost. Thanks again and best regards David On Tue, 7 Sept 2021 at 14:32, Laurent Pinchart <laurent.pinchart@ideasonboard.com> wrote: > > Hello, > > I've reviewed this series and it looks good to me. Sakari, would you > like to have a look, or should I send a pull request ? > > On Mon, Aug 16, 2021 at 12:39:07PM +0100, David Plowman wrote: > > Hi again everyone > > > > Thanks for last round of feedback, leading to this v6 pair of > > patches. I've made the documentation improvements that were suggested, > > and accordingly added Laurent's "Reviewed-by" tags. > > > > Thanks and best regards > > > > David > > > > David Plowman (2): > > media: v4l2-ctrls: Add V4L2_CID_NOTIFY_GAINS control > > media: v4l2-ctrls: Document V4L2_CID_NOTIFY_GAINS control > > > > .../media/v4l/ext-ctrls-image-source.rst | 20 +++++++++++++++++++ > > drivers/media/v4l2-core/v4l2-ctrls-defs.c | 1 + > > include/uapi/linux/v4l2-controls.h | 1 + > > 3 files changed, 22 insertions(+) > > -- > Regards, > > Laurent Pinchart
Hi David, On Tue, Oct 05, 2021 at 10:37:54AM +0100, David Plowman wrote: > Hi Laurent, Sakari, everyone > > I was wondering whether I might give this another little nudge just so > that it doesn't get lost. I've already reviewed it, I can only join you to nicely ask Sakari if he could have a look :-) I know he has lots of patches on his plate at the moment though. Hans, would you like to share your opinion on this series ? > On Tue, 7 Sept 2021 at 14:32, Laurent Pinchart wrote: > > > > Hello, > > > > I've reviewed this series and it looks good to me. Sakari, would you > > like to have a look, or should I send a pull request ? > > > > On Mon, Aug 16, 2021 at 12:39:07PM +0100, David Plowman wrote: > > > Hi again everyone > > > > > > Thanks for last round of feedback, leading to this v6 pair of > > > patches. I've made the documentation improvements that were suggested, > > > and accordingly added Laurent's "Reviewed-by" tags. > > > > > > Thanks and best regards > > > > > > David > > > > > > David Plowman (2): > > > media: v4l2-ctrls: Add V4L2_CID_NOTIFY_GAINS control > > > media: v4l2-ctrls: Document V4L2_CID_NOTIFY_GAINS control > > > > > > .../media/v4l/ext-ctrls-image-source.rst | 20 +++++++++++++++++++ > > > drivers/media/v4l2-core/v4l2-ctrls-defs.c | 1 + > > > include/uapi/linux/v4l2-controls.h | 1 + > > > 3 files changed, 22 insertions(+)
Hi David, Laurent, On Wed, Oct 06, 2021 at 04:10:26AM +0300, Laurent Pinchart wrote: > Hi David, > > On Tue, Oct 05, 2021 at 10:37:54AM +0100, David Plowman wrote: > > Hi Laurent, Sakari, everyone > > > > I was wondering whether I might give this another little nudge just so > > that it doesn't get lost. > > I've already reviewed it, I can only join you to nicely ask Sakari if he > could have a look :-) I know he has lots of patches on his plate at the > moment though. > > Hans, would you like to share your opinion on this series ? The patches have been merged some time ago to the media tree master branch: commit 311a839a1ad255ebcb7291fb4e0d2ec2f32312a7 Author: David Plowman <david.plowman@raspberrypi.com> Date: Mon Aug 16 13:39:09 2021 +0200 media: v4l2-ctrls: Document V4L2_CID_NOTIFY_GAINS control Add documentation for the V4L2_CID_NOTIFY_GAINS control. This control is required by sensors that need to know what colour gains will be applied to pixels by downstream processing (such as by an ISP), though the sensor does not apply these gains itself. Signed-off-by: David Plowman <david.plowman@raspberrypi.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org> commit a9c80593ff80ddb7c6496624e5384e1ea3460a72 Author: David Plowman <david.plowman@raspberrypi.com> Date: Mon Aug 16 13:39:08 2021 +0200 media: v4l2-ctrls: Add V4L2_CID_NOTIFY_GAINS control We add a new control V4L2_CID_NOTIFY_GAINS which allows the sensor to be notified what gains will be applied to the different colour channels by subsequent processing (such as by an ISP), even though the sensor will not apply any of these gains itself. For Bayer sensors this will be an array control taking 4 values which are the 4 gains arranged in the fixed order B, Gb, Gr and R, irrespective of the exact Bayer order of the sensor itself. The use of an array makes it straightforward to extend this control to non-Bayer sensors (for example, sensors with an RGBW pattern) in future. The units are in all cases linear with the default value indicating a gain of exactly 1.0. For example, if the default value were reported as 128 then the value 192 would represent a gain of exactly 1.5. Signed-off-by: David Plowman <david.plowman@raspberrypi.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>