[{"id":30590,"web_url":"https://patchwork.libcamera.org/comment/30590/","msgid":"<172286527865.1687952.10775777185922124829@ping.linuxembedded.co.uk>","date":"2024-08-05T13:41:18","subject":"Re: [PATCH v1 1/6] libcamera: controls: Update the ColourTemperature\n\tcontrol to be writable","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Hi Stefan,\n\nThanks for updating this\n\nTechnically, I think this patch might be at v3, given:\n\n https://patchwork.libcamera.org/patch/19231/\n[libcamera-devel,v2,1/2] libcamera: controls: Update the ColourTemperature control to be writable\n\nBut lets continue ;-)\n\nQuoting Stefan Klug (2024-08-05 13:05:02)\n> For manual control it is helpful to be able to specify a fixed colour\n> temperature. It also provides a easy way to apply the temperature\n\ns/a/an/\n\n> specific CCMs and colour gains that are contained in the tuning files.\n> \n> Document this and update the control dependencies.\n> \n> Signed-off-by: Stefan Klug \n> ---\n> src/libcamera/control_ids_core.yaml | 24 ++++++++++++++++++++----\n> 1 file changed, 20 insertions(+), 4 deletions(-)\n> \n> diff --git a/src/libcamera/control_ids_core.yaml b/src/libcamera/control_ids_core.yaml\n> index 9d413a94e0ee..90a52bccaa98 100644\n> --- a/src/libcamera/control_ids_core.yaml\n> +++ b/src/libcamera/control_ids_core.yaml\n> @@ -252,9 +252,12 @@ controls:\n> - AwbEnable:\n> type: bool\n> description: |\n> - Enable or disable the AWB.\n> + Enable or disable the AWB. Disabling AWB stops updates to the\n> + ColourGains and to the ColourCorrectionMatrix.\n> \n> + \\sa ColourCorrectionMatrix\n> \\sa ColourGains\n> + \\sa ColourTemperature\n> \n> # AwbMode needs further attention:\n> # - Auto-generate max enum value.\n> @@ -309,13 +312,24 @@ controls:\n> disabled.\n> \n> \\sa AwbEnable\n> + \\sa ColourTemperature\n> size: [2]\n> \n> - ColourTemperature:\n> type: int32_t\n> - description: Report the current estimate of the colour temperature, in\n> - kelvin, for this frame. The ColourTemperature control can only be\n> - returned in metadata.\n> + description: |\n> + Report the current estimate of the colour temperature, in kelvin, for\n> + this frame. An implementation may also allow this control to be set when\n> + AWB is disabled. In that case ColourGains and the ColourCorrectionMatrix\n> + get set accordingly. If either ColourGains or ColourCorrectionMatrix are\n> + specified at the same time, they takes precedence.\n\nI presume both ColourGains and ColourCorrectionMatrix can operate\nindependently of each other though.\n\n> +\n> + Note that the metadata always contains the measured value, even when the\n> + value was set manually.\n\nWill this always be true? Or is it possible to always be true? I see\nthat David stated that in the Raspberry Pi implementation\n\nhttps://patchwork.libcamera.org/patch/19231/#28242 :\n\"\n> The AWB is \"disabled\" at that point, and\n> you don't get any feedback as to what it thinks the colour temperature\n> really is (as AWB is not doing a search any more). Setting a fixed\n> colour temperature overrides any fixed colour gains, and also vice\n> versa, because they basically do the same thing (i.e. set fixed colour\n> gains).\n\"\n\nPresumably if no colour temperature is measured though - it simply\ndoesn't get returned in the metadata ... so perhaps this isn't an issue.\n\nInstead of 'always' which implies it will be guaranteed to be set .. I\nmight reword slightly to:\n\n\"The metadata will only report measured colour temperature values when\navailable even if set manually.\"\n\nMaybe that's closer to what would be implemented?\n\n\n> +\n> + \\sa AwbEnable\n> + \\sa ColourCorrectionMatrix\n> + \\sa ColourGains\n> \n> - Saturation:\n> type: float\n> @@ -365,6 +379,8 @@ controls:\n> transformation. The 3x3 matrix is stored in conventional reading\n> order in an array of 9 floating point values.\n> \n> + \\sa AwbEnable\n> + \\sa ColourTemperature\n> size: [3,3]\n> \n> - ScalerCrop:\n> -- \n> 2.43.0\n>","headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 91ED4BE173\n\tfor ;\n\tMon, 5 Aug 2024 13:41:24 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id A1F0163381;\n\tMon, 5 Aug 2024 15:41:23 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 9A8F26195D\n\tfor ;\n\tMon, 5 Aug 2024 15:41:21 +0200 (CEST)","from pendragon.ideasonboard.com\n\t(cpc89244-aztw30-2-0-cust6594.18-1.cable.virginm.net [86.31.185.195])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 049AE18D;\n\tMon, 5 Aug 2024 15:40:29 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"oG8IrRAZ\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1722865230;\n\tbh=aCNmiyAky+mdrEZiFbLlXAVzvPVTQiNAAoP+O4p0uo8=;\n\th=In-Reply-To:References:Subject:From:Cc:To:Date:From;\n\tb=oG8IrRAZ76rUnIlduU+x6VMrZ2YY+8zM15APBGY1stCZ15vjmBdXQoGENIyhbxd5G\n\tjJYUFA7MyeDCHm5g5JzExKXR4jeBKbE11gTh1lSZAn7rAx/yLmbEvqjpUKvoKTEqip\n\t3oh2a2ZL2wVuoOMzkcykKXkPd1zSMn5zhvYM+12I=","Content-Type":"text/plain; charset=\"utf-8\"","MIME-Version":"1.0","Content-Transfer-Encoding":"quoted-printable","In-Reply-To":"<20240805120522.1613342-2-stefan.klug@ideasonboard.com>","References":"<20240805120522.1613342-1-stefan.klug@ideasonboard.com>\n\t<20240805120522.1613342-2-stefan.klug@ideasonboard.com>","Subject":"Re: [PATCH v1 1/6] libcamera: controls: Update the ColourTemperature\n\tcontrol to be writable","From":"Kieran Bingham ","Cc":"David Plowman ,\n\tStefan Klug ","To":"Stefan Klug ,\n\tlibcamera-devel@lists.libcamera.org","Date":"Mon, 05 Aug 2024 14:41:18 +0100","Message-ID":"<172286527865.1687952.10775777185922124829@ping.linuxembedded.co.uk>","User-Agent":"alot/0.10","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":30605,"web_url":"https://patchwork.libcamera.org/comment/30605/","msgid":"","date":"2024-08-06T06:07:56","subject":"Re: [PATCH v1 1/6] libcamera: controls: Update the ColourTemperature\n\tcontrol to be writable","submitter":{"id":184,"url":"https://patchwork.libcamera.org/api/people/184/","name":"Stefan Klug","email":"stefan.klug@ideasonboard.com"},"content":"Hi Kieran,\n\nThanks for the review.\n\nOn Mon, Aug 05, 2024 at 02:41:18PM +0100, Kieran Bingham wrote:\n> Hi Stefan,\n> \n> Thanks for updating this\n> \n> Technically, I think this patch might be at v3, given:\n> \n> https://patchwork.libcamera.org/patch/19231/\n> [libcamera-devel,v2,1/2] libcamera: controls: Update the ColourTemperature control to be writable\n> \n\nYes you're right. This commit was the last one I added to the series. So\nit felt like a v1 :-)\n\n> But lets continue ;-)\n> \n> Quoting Stefan Klug (2024-08-05 13:05:02)\n> > For manual control it is helpful to be able to specify a fixed colour\n> > temperature. It also provides a easy way to apply the temperature\n> \n> s/a/an/\n> \n> > specific CCMs and colour gains that are contained in the tuning files.\n> > \n> > Document this and update the control dependencies.\n> > \n> > Signed-off-by: Stefan Klug \n> > ---\n> > src/libcamera/control_ids_core.yaml | 24 ++++++++++++++++++++----\n> > 1 file changed, 20 insertions(+), 4 deletions(-)\n> > \n> > diff --git a/src/libcamera/control_ids_core.yaml b/src/libcamera/control_ids_core.yaml\n> > index 9d413a94e0ee..90a52bccaa98 100644\n> > --- a/src/libcamera/control_ids_core.yaml\n> > +++ b/src/libcamera/control_ids_core.yaml\n> > @@ -252,9 +252,12 @@ controls:\n> > - AwbEnable:\n> > type: bool\n> > description: |\n> > - Enable or disable the AWB.\n> > + Enable or disable the AWB. Disabling AWB stops updates to the\n> > + ColourGains and to the ColourCorrectionMatrix.\n> > \n> > + \\sa ColourCorrectionMatrix\n> > \\sa ColourGains\n> > + \\sa ColourTemperature\n> > \n> > # AwbMode needs further attention:\n> > # - Auto-generate max enum value.\n> > @@ -309,13 +312,24 @@ controls:\n> > disabled.\n> > \n> > \\sa AwbEnable\n> > + \\sa ColourTemperature\n> > size: [2]\n> > \n> > - ColourTemperature:\n> > type: int32_t\n> > - description: Report the current estimate of the colour temperature, in\n> > - kelvin, for this frame. The ColourTemperature control can only be\n> > - returned in metadata.\n> > + description: |\n> > + Report the current estimate of the colour temperature, in kelvin, for\n> > + this frame. An implementation may also allow this control to be set when\n> > + AWB is disabled. In that case ColourGains and the ColourCorrectionMatrix\n> > + get set accordingly. If either ColourGains or ColourCorrectionMatrix are\n> > + specified at the same time, they takes precedence.\n> \n> I presume both ColourGains and ColourCorrectionMatrix can operate\n> independently of each other though.\n\nYes they can, but if you set ColorTemperature, both get set.\n\n> \n> > +\n> > + Note that the metadata always contains the measured value, even when the\n> > + value was set manually.\n> \n> Will this always be true? Or is it possible to always be true? I see\n> that David stated that in the Raspberry Pi implementation\n> \n> https://patchwork.libcamera.org/patch/19231/#28242 :\n> \"\n> > The AWB is \"disabled\" at that point, and\n> > you don't get any feedback as to what it thinks the colour temperature\n> > really is (as AWB is not doing a search any more). Setting a fixed\n> > colour temperature overrides any fixed colour gains, and also vice\n> > versa, because they basically do the same thing (i.e. set fixed colour\n> > gains).\n> \"\n> \n> Presumably if no colour temperature is measured though - it simply\n> doesn't get returned in the metadata ... so perhaps this isn't an issue.\n> \n> Instead of 'always' which implies it will be guaranteed to be set .. I\n> might reword slightly to:\n> \n> \"The metadata will only report measured colour temperature values when\n> available even if set manually.\"\n> \n> Maybe that's closer to what would be implemented?\n> \n\nSure. I updated it for v2 (or v4 :-) ).\n\nCheers,\nStefan\n\n> \n> > +\n> > + \\sa AwbEnable\n> > + \\sa ColourCorrectionMatrix\n> > + \\sa ColourGains\n> > \n> > - Saturation:\n> > type: float\n> > @@ -365,6 +379,8 @@ controls:\n> > transformation. The 3x3 matrix is stored in conventional reading\n> > order in an array of 9 floating point values.\n> > \n> > + \\sa AwbEnable\n> > + \\sa ColourTemperature\n> > size: [3,3]\n> > \n> > - ScalerCrop:\n> > -- \n> > 2.43.0\n> >","headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 48803C323E\n\tfor ;\n\tTue, 6 Aug 2024 06:08:02 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 752DA6337E;\n\tTue, 6 Aug 2024 08:08:01 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id EB94763369\n\tfor ;\n\tTue, 6 Aug 2024 08:07:58 +0200 (CEST)","from ideasonboard.com (unknown\n\t[IPv6:2a00:6020:448c:6c00:7a68:2b6f:8265:aa72])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id F014E2C5;\n\tTue, 6 Aug 2024 08:07:06 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"PhJhgbBG\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1722924427;\n\tbh=6b/RXz6KGDeLZlwYpsVFtPxHD3qeWs3TKZf65nwya74=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=PhJhgbBG38c6qoqO3XTWiJ/mxBRVC2J+E62CJbtuVh8I/M0VwTlwV6NLPz7MPXwVT\n\tcEaVIp8tpQUi4D3lVe4elB9Ltp6lUUC9Zce7WOxnrvIry9e0nSYaxLcUXhv9yzxqrg\n\t7FYqLo0e/22Vomrh0hBCQOEXaNesB+WjKp7NFGiQ=","Date":"Tue, 6 Aug 2024 08:07:56 +0200","From":"Stefan Klug ","To":"Kieran Bingham ","Cc":"libcamera-devel@lists.libcamera.org, \n\tDavid Plowman ","Subject":"Re: [PATCH v1 1/6] libcamera: controls: Update the ColourTemperature\n\tcontrol to be writable","Message-ID":"","References":"<20240805120522.1613342-1-stefan.klug@ideasonboard.com>\n\t<20240805120522.1613342-2-stefan.klug@ideasonboard.com>\n\t<172286527865.1687952.10775777185922124829@ping.linuxembedded.co.uk>","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Disposition":"inline","In-Reply-To":"<172286527865.1687952.10775777185922124829@ping.linuxembedded.co.uk>","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":30616,"web_url":"https://patchwork.libcamera.org/comment/30616/","msgid":"<87wmkunjhs.fsf@redhat.com>","date":"2024-08-06T08:27:59","subject":"Re: [PATCH v1 1/6] libcamera: controls: Update the\n\tColourTemperature control to be writable","submitter":{"id":177,"url":"https://patchwork.libcamera.org/api/people/177/","name":"Milan Zamazal","email":"mzamazal@redhat.com"},"content":"Stefan Klug writes:\n\n> For manual control it is helpful to be able to specify a fixed colour\n> temperature. It also provides a easy way to apply the temperature\n> specific CCMs and colour gains that are contained in the tuning files.\n>\n> Document this and update the control dependencies.\n>\n> Signed-off-by: Stefan Klug \n> ---\n> src/libcamera/control_ids_core.yaml | 24 ++++++++++++++++++++----\n> 1 file changed, 20 insertions(+), 4 deletions(-)\n>\n> diff --git a/src/libcamera/control_ids_core.yaml b/src/libcamera/control_ids_core.yaml\n> index 9d413a94e0ee..90a52bccaa98 100644\n> --- a/src/libcamera/control_ids_core.yaml\n> +++ b/src/libcamera/control_ids_core.yaml\n> @@ -252,9 +252,12 @@ controls:\n> - AwbEnable:\n> type: bool\n> description: |\n> - Enable or disable the AWB.\n> + Enable or disable the AWB. Disabling AWB stops updates to the\n> + ColourGains and to the ColourCorrectionMatrix.\n> \n> + \\sa ColourCorrectionMatrix\n> \\sa ColourGains\n> + \\sa ColourTemperature\n> \n> # AwbMode needs further attention:\n> # - Auto-generate max enum value.\n> @@ -309,13 +312,24 @@ controls:\n> disabled.\n> \n> \\sa AwbEnable\n> + \\sa ColourTemperature\n> size: [2]\n> \n> - ColourTemperature:\n> type: int32_t\n> - description: Report the current estimate of the colour temperature, in\n> - kelvin, for this frame. The ColourTemperature control can only be\n> - returned in metadata.\n> + description: |\n> + Report the current estimate of the colour temperature, in kelvin, for\n> + this frame. An implementation may also allow this control to be set when\n> + AWB is disabled. In that case ColourGains and the ColourCorrectionMatrix\n> + get set accordingly. If either ColourGains or ColourCorrectionMatrix are\n> + specified at the same time, they takes precedence.\n\ns/takes/take/\n\n> + Note that the metadata always contains the measured value, even when the\n> + value was set manually.\n> +\n> + \\sa AwbEnable\n> + \\sa ColourCorrectionMatrix\n> + \\sa ColourGains\n> \n> - Saturation:\n> type: float\n> @@ -365,6 +379,8 @@ controls:\n> transformation. The 3x3 matrix is stored in conventional reading\n> order in an array of 9 floating point values.\n> \n> + \\sa AwbEnable\n> + \\sa ColourTemperature\n> size: [3,3]\n> \n> - ScalerCrop:","headers":{"Return-Path":"","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id D63CABE173\n\tfor ;\n\tTue, 6 Aug 2024 08:28:10 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 05DE263394;\n\tTue, 6 Aug 2024 10:28:10 +0200 (CEST)","from us-smtp-delivery-124.mimecast.com\n\t(us-smtp-delivery-124.mimecast.com [170.10.133.124])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 458FF61955\n\tfor ;\n\tTue, 6 Aug 2024 10:28:08 +0200 (CEST)","from mail-wm1-f69.google.com (mail-wm1-f69.google.com\n\t[209.85.128.69]) by relay.mimecast.com with ESMTP with STARTTLS\n\t(version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id\n\tus-mta-60-h0QSRbZgNjSnDERDjj6X2Q-1; Tue, 06 Aug 2024 04:28:02 -0400","by mail-wm1-f69.google.com with SMTP id\n\t5b1f17b1804b1-4280f233115so2795635e9.2\n\tfor ;\n\tTue, 06 Aug 2024 01:28:02 -0700 (PDT)","from nuthatch (nat-pool-brq-t.redhat.com. [213.175.37.10])\n\tby smtp.gmail.com with ESMTPSA id\n\tffacd0b85a97d-36bbcf0cc8esm12163208f8f.19.2024.08.06.01.27.59\n\t(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n\tTue, 06 Aug 2024 01:27:59 -0700 (PDT)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=redhat.com header.i=@redhat.com\n\theader.b=\"al22cA64\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n\ts=mimecast20190719; t=1722932887;\n\th=from:from:reply-to:subject:subject:date:date:message-id:message-id:\n\tto:to:cc:cc:mime-version:mime-version:content-type:content-type:\n\tin-reply-to:in-reply-to:references:references;\n\tbh=UYwMqTkHQI1Y9pWJOgJBsv8Idky7td5ujcAg/NQGN2I=;\n\tb=al22cA64Wm8uJGZ9LXMGeNGx6w/gW8zKuStOl5roq6Qn81kecnI1+lTVRvTIr0v1K8ccby\n\tEMSpn4bzZByiJRpOyWR6/IKtSx4L8wutBAITnYGxScOizEl1GIlCq7Azm3UbvgYV95mBB+\n\t5n1jpRp/AERY/k07mszoD/fZj+xVetw=","X-MC-Unique":"h0QSRbZgNjSnDERDjj6X2Q-1","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20230601; t=1722932881; x=1723537681;\n\th=mime-version:user-agent:message-id:date:references:in-reply-to\n\t:subject:cc:to:from:x-gm-message-state:from:to:cc:subject:date\n\t:message-id:reply-to;\n\tbh=UYwMqTkHQI1Y9pWJOgJBsv8Idky7td5ujcAg/NQGN2I=;\n\tb=dJ6tWTS3a1B9H6Eq/ofiPyS/IlffRr6RGrgPN0HyAKuxE7VoZiVEi9fhsB5zAGouQe\n\t+y8Je1zLcW9X7QMDVigErcYzeQGj9xFn6U8EXO83FKOgEruYApqIqEi9wlH+jkXIcmbg\n\tyL7h/tl1YxYSzeoY6doJWp0imeyQJ+yi2qLEHAg9IS3A1khP6y83bWKOsQnNn8OINRUI\n\t42ZNBw3A7/7xvpVYeMD/9AHNztr0fIvZnkQvTmgLKePehQFm3XmoXmdEI/FfmuhfcKak\n\tDTMYgafEayPMc00xVlQMgw8Ey9xKFQQYonAWU42qO91HXTc1Csc7/lU8mIzB2HP0af54\n\twUaA==","X-Gm-Message-State":"AOJu0YzTWeDfAgIEiD9+m2EtO6NCGammU0iDxljws14rRpWEYTXlKd8z\n\tH4VfpFauJSrdqPaOIUgMd9QW3qEr08Tgqfa27N1QS30xVuZbbTip2rWSWdUXUNz9MGHS3VyrHt5\n\tt5uV7kVA4tbyChri4INZz77Xk3WxnmDhk512qgJ8V1RLkT8k8F2VroY/E+4Hdv3Y0I6XAtkc=","X-Received":["by 2002:a05:600c:3ca2:b0:426:6171:6083 with SMTP id\n\t5b1f17b1804b1-428e6b07aacmr94527175e9.13.1722932881302; \n\tTue, 06 Aug 2024 01:28:01 -0700 (PDT)","by 2002:a05:600c:3ca2:b0:426:6171:6083 with SMTP id\n\t5b1f17b1804b1-428e6b07aacmr94527045e9.13.1722932880785; \n\tTue, 06 Aug 2024 01:28:00 -0700 (PDT)"],"X-Google-Smtp-Source":"AGHT+IHQrhxV715rz1lZcqbN3pPj4rrKbKGvFMhqYG7V2OXa3wcxGwqvql5vcCMOOyNFg2/NYmDdVw==","From":"Milan Zamazal ","To":"Stefan Klug ","Cc":"libcamera-devel@lists.libcamera.org, David Plowman\n\t","Subject":"Re: [PATCH v1 1/6] libcamera: controls: Update the\n\tColourTemperature control to be writable","In-Reply-To":"<20240805120522.1613342-2-stefan.klug@ideasonboard.com> (Stefan\n\tKlug's message of \"Mon, 5 Aug 2024 14:05:02 +0200\")","References":"<20240805120522.1613342-1-stefan.klug@ideasonboard.com>\n\t<20240805120522.1613342-2-stefan.klug@ideasonboard.com>","Date":"Tue, 06 Aug 2024 10:27:59 +0200","Message-ID":"<87wmkunjhs.fsf@redhat.com>","User-Agent":"Gnus/5.13 (Gnus v5.13)","MIME-Version":"1.0","X-Mimecast-Spam-Score":"0","X-Mimecast-Originator":"redhat.com","Content-Type":"text/plain","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}}]