From patchwork Fri Jul 1 15:46:58 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jacopo Mondi X-Patchwork-Id: 16516 Return-Path: X-Original-To: parsemail@patchwork.libcamera.org Delivered-To: parsemail@patchwork.libcamera.org Received: from lancelot.ideasonboard.com (lancelot.ideasonboard.com [92.243.16.209]) by patchwork.libcamera.org (Postfix) with ESMTPS id 00141BD808 for ; Fri, 1 Jul 2022 15:47:15 +0000 (UTC) Received: from lancelot.ideasonboard.com (localhost [IPv6:::1]) by lancelot.ideasonboard.com (Postfix) with ESMTP id BB8DE6565C; Fri, 1 Jul 2022 17:47:15 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org; s=mail; t=1656690435; bh=cOryWfdVASI7VdmRVSGK8PW3OalV4m+56k8h8v5fo2g=; h=To:Date:In-Reply-To:References:Subject:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To: From; b=TyVlD3c4JAWLO56DH+PGYd6AQIfGMv3GezZnabemys9kcgI7s9akTp9HxijOqRSAG V06bPoKV83fPT5qtvJ4N8Enmeg7w5U6EAkd12m8UeHdElBCW7LOpEbvSRL8HYYaksg IXFlvImBjJ264PfSgHExh4XX3W7+OoUHgoEqGaykHn9Ju2YT29Fwbe5qzSDbYJxj6w cT5UG7IrYQVi5b61BQOd1KUq6qFQIn1sjjy9Hce2hCx0LaQzk/Vl74FqDPLmmwh4kB dEeAfkrdJFsjHVyTUyXK/M8DLfFH1fu2l3ZJHHtyTwiu9JbkeK0RoU3JhBfl+bN//h 7lOFFbKZQdsyw== Received: from relay4-d.mail.gandi.net (relay4-d.mail.gandi.net [217.70.183.196]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 85F0065655 for ; Fri, 1 Jul 2022 17:47:11 +0200 (CEST) Received: (Authenticated sender: jacopo@jmondi.org) by mail.gandi.net (Postfix) with ESMTPSA id E59C0E0006; Fri, 1 Jul 2022 15:47:10 +0000 (UTC) To: libcamera-devel@lists.libcamera.org Date: Fri, 1 Jul 2022 17:46:58 +0200 Message-Id: <20220701154701.354052-6-jacopo@jmondi.org> X-Mailer: git-send-email 2.36.1 In-Reply-To: <20220701154701.354052-1-jacopo@jmondi.org> References: <20220518134728.777709-2-paul.elder@ideasonboard.com> <20220701154701.354052-1-jacopo@jmondi.org> MIME-Version: 1.0 Subject: [libcamera-devel] [PATCH 6/9] fixup: Reword ExposureTimeMode X-BeenThere: libcamera-devel@lists.libcamera.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-Patchwork-Original-From: Jacopo Mondi via libcamera-devel From: Jacopo Mondi Reply-To: Jacopo Mondi Errors-To: libcamera-devel-bounces@lists.libcamera.org Sender: "libcamera-devel" Slightly reword the first part of the control definition. Signed-off-by: Jacopo Mondi Acked-by: Paul Elder --- src/libcamera/control_ids.yaml | 16 +++++----------- 1 file changed, 5 insertions(+), 11 deletions(-) diff --git a/src/libcamera/control_ids.yaml b/src/libcamera/control_ids.yaml index b062b7e234a6..e6676a149795 100644 --- a/src/libcamera/control_ids.yaml +++ b/src/libcamera/control_ids.yaml @@ -203,21 +203,15 @@ controls: Controls the source of the exposure time that is applied to the image sensor. When set to Auto, the AE algorithm computes the exposure time and configures the image sensor accordingly. When set to Manual, - exposure time specified in ExposureTime is applied to the image sensor. - If ExposureTime is not set, then the value last computed by the AE - algorithm when the mode was Auto will be used. + the value of the ExposureTime control is used. - If ExposureTime is not set and the mode is ExposureTimeModeManual and - AE was never Auto (either because the camera started in Manual mode, + When transitioning from Auto to Manual mode and no ExposureTime control + is provided by the application, the last value computed by the AE + algorithm when the mode was Auto will be used. If the ExposureTimeMode + was never set to Auto (either because the camera started in Manual mode, or Auto is not supported by the camera), the camera should use a best-effort default value. - When ExposureTimeMode is set Auto, the value set in ExposureTime is - ignored and is not retained. This means that if ExposureTimeMode is set - to Manual and ExposureTime is not also set, the exposure time that - was last computed by the AE algorithm while the mode was Auto will be - applied to the sensor. - If ExposureTimeModeManual is supported, the ExposureTime control must also be supported.