[{"id":15135,"web_url":"https://patchwork.libcamera.org/comment/15135/","msgid":"<20210212094234.bdxmyqby5fuimdfv@uno.localdomain>","date":"2021-02-12T09:42:34","subject":"Re: [libcamera-devel] [PATCH v2 4/4] cam: Default camera location\n\tto external","submitter":{"id":3,"url":"https://patchwork.libcamera.org/api/people/3/","name":"Jacopo Mondi","email":"jacopo@jmondi.org"},"content":"Hi Paul,\n I think the patch subject is wrong :/\n\nOn Fri, Feb 12, 2021 at 02:48:16PM +0900, Paul Elder wrote:\n> Now that we default the camera sensor location to Unknown, handle it in\n> cam the same way external is handled.\n\nI would not call an unknown camera \"External\".\nThis is also the second different default we assign to \"Unknown\" in\nthis series. For the HAL it defaults to Front, here it defaults to\n\"External\". Isn't this a bit confusing ?\n\nI would rather name it just \"Camera\" without reporting\nexternal/front/back or anything else.\n\nThanks\n j\n\n>\n> Signed-off-by: Paul Elder \n>\n> ---\n> New in v2\n> ---\n> src/cam/main.cpp | 1 +\n> 1 file changed, 1 insertion(+)\n>\n> diff --git a/src/cam/main.cpp b/src/cam/main.cpp\n> index e01be63a..261ed0c8 100644\n> --- a/src/cam/main.cpp\n> +++ b/src/cam/main.cpp\n> @@ -387,6 +387,7 @@ std::string const CamApp::cameraName(const Camera *camera)\n> \t\tname = \"Internal back camera\";\n> \t\tbreak;\n> \tcase properties::CameraLocationExternal:\n> +\tcase properties::CameraLocationUnknown:\n> \t\tname = \"External camera\";\n> \t\tif (props.contains(properties::Model))\n> \t\t\tname += \" '\" + props.get(properties::Model) + \"'\";\n> --\n> 2.27.0\n>\n> _______________________________________________\n> libcamera-devel mailing list\n> libcamera-devel@lists.libcamera.org\n> https://lists.libcamera.org/listinfo/libcamera-devel","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 548EBBD160\n\tfor ;\n\tFri, 12 Feb 2021 09:42:12 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id CC9716376F;\n\tFri, 12 Feb 2021 10:42:11 +0100 (CET)","from relay7-d.mail.gandi.net (relay7-d.mail.gandi.net\n\t[217.70.183.200])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id A63FC63762\n\tfor ;\n\tFri, 12 Feb 2021 10:42:10 +0100 (CET)","from uno.localdomain (93-34-118-233.ip49.fastwebnet.it\n\t[93.34.118.233]) (Authenticated sender: jacopo@jmondi.org)\n\tby relay7-d.mail.gandi.net (Postfix) with ESMTPSA id 371E62000C;\n\tFri, 12 Feb 2021 09:42:09 +0000 (UTC)"],"X-Originating-IP":"93.34.118.233","Date":"Fri, 12 Feb 2021 10:42:34 +0100","From":"Jacopo Mondi ","To":"Paul Elder ","Message-ID":"<20210212094234.bdxmyqby5fuimdfv@uno.localdomain>","References":"<20210212054816.53561-1-paul.elder@ideasonboard.com>\n\t<20210212054816.53561-5-paul.elder@ideasonboard.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20210212054816.53561-5-paul.elder@ideasonboard.com>","Subject":"Re: [libcamera-devel] [PATCH v2 4/4] cam: Default camera location\n\tto external","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","Cc":"libcamera-devel@lists.libcamera.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":15160,"web_url":"https://patchwork.libcamera.org/comment/15160/","msgid":"","date":"2021-02-12T19:22:37","subject":"Re: [libcamera-devel] [PATCH v2 4/4] cam: Default camera location\n\tto external","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Hi Paul,\n\nOn 12/02/2021 09:42, Jacopo Mondi wrote:\n> Hi Paul,\n> I think the patch subject is wrong :/\n> \n> On Fri, Feb 12, 2021 at 02:48:16PM +0900, Paul Elder wrote:\n>> Now that we default the camera sensor location to Unknown, handle it in\n>> cam the same way external is handled.\n> \n> I would not call an unknown camera \"External\".\n> This is also the second different default we assign to \"Unknown\" in\n> this series. For the HAL it defaults to Front, here it defaults to\n> \"External\". Isn't this a bit confusing ?\n> \n> I would rather name it just \"Camera\" without reporting\n> external/front/back or anything else.\n> \n\nI wouldn't even use Camera, just print the model, and the rest of the\nstring:\n\nI.e.:\n> 1: Camera 'HP Wide Vision FHD Camera: HP W' (\\_SB_.PCI0.XHC_.RHUB.HS05-5:1.0-0408:5251)\n\n\nvs\n> 2: 'HP Wide Vision FHD Camera: HP I' (\\_SB_.PCI0.XHC_.RHUB.HS05-5:1.2-0408:5251)\n\nOtherwise, as this is cam/test tool - we should possibly report that the\nlocation is unknown:\n\n>> 1: Unknown Location 'HP Wide Vision FHD Camera: HP W' (\\_SB_.PCI0.XHC_.RHUB.HS05-5:1.0-0408:5251)\n\n\n--\nKieran\n\n> Thanks\n> j\n> \n>>\n>> Signed-off-by: Paul Elder \n>>\n>> ---\n>> New in v2\n>> ---\n>> src/cam/main.cpp | 1 +\n>> 1 file changed, 1 insertion(+)\n>>\n>> diff --git a/src/cam/main.cpp b/src/cam/main.cpp\n>> index e01be63a..261ed0c8 100644\n>> --- a/src/cam/main.cpp\n>> +++ b/src/cam/main.cpp\n>> @@ -387,6 +387,7 @@ std::string const CamApp::cameraName(const Camera *camera)\n>> \t\tname = \"Internal back camera\";\n>> \t\tbreak;\n>> \tcase properties::CameraLocationExternal:\n>> +\tcase properties::CameraLocationUnknown:\n>> \t\tname = \"External camera\";\n>> \t\tif (props.contains(properties::Model))\n>> \t\t\tname += \" '\" + props.get(properties::Model) + \"'\";\n>> --\n>> 2.27.0\n>>\n>> _______________________________________________\n>> libcamera-devel mailing list\n>> libcamera-devel@lists.libcamera.org\n>> https://lists.libcamera.org/listinfo/libcamera-devel\n> _______________________________________________\n> libcamera-devel mailing list\n> libcamera-devel@lists.libcamera.org\n> https://lists.libcamera.org/listinfo/libcamera-devel\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 26BCABD160\n\tfor ;\n\tFri, 12 Feb 2021 19:22:43 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id AD25A6379A;\n\tFri, 12 Feb 2021 20:22:42 +0100 (CET)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id EE04263794\n\tfor ;\n\tFri, 12 Feb 2021 20:22:40 +0100 (CET)","from [192.168.0.20]\n\t(cpc89244-aztw30-2-0-cust3082.18-1.cable.virginm.net [86.31.172.11])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 504F071;\n\tFri, 12 Feb 2021 20:22:40 +0100 (CET)"],"Authentication-Results":"lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"QUnnY5lh\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1613157760;\n\tbh=jjsa/uOUa6Dpcx+rN8hRFoy74F+73PmK9JL7leivXQg=;\n\th=Reply-To:Subject:To:Cc:References:From:Date:In-Reply-To:From;\n\tb=QUnnY5lhpkCJ6KF9GrDOhRZW2lB2s1c1/AM0OTUORB/pURmNtB5Q2jmteTZZ6P1D8\n\tc8hNA15tYfwE28ggFOZYAn+eAxRcYLpzdOUa7h5vg/2QThZjfh9+4gO+VMn8hfI1L2\n\tu/fZq4ouFLqNeb/wFM/F0xV8Hm1Fq0sNtpEDBC/A=","To":"Jacopo Mondi ,\n\tPaul Elder ","References":"<20210212054816.53561-1-paul.elder@ideasonboard.com>\n\t<20210212054816.53561-5-paul.elder@ideasonboard.com>\n\t<20210212094234.bdxmyqby5fuimdfv@uno.localdomain>","From":"Kieran Bingham ","Autocrypt":"addr=kieran.bingham@ideasonboard.com; keydata=\n\tmQINBFYE/WYBEACs1PwjMD9rgCu1hlIiUA1AXR4rv2v+BCLUq//vrX5S5bjzxKAryRf0uHat\n\tV/zwz6hiDrZuHUACDB7X8OaQcwhLaVlq6byfoBr25+hbZG7G3+5EUl9cQ7dQEdvNj6V6y/SC\n\trRanWfelwQThCHckbobWiQJfK9n7rYNcPMq9B8e9F020LFH7Kj6YmO95ewJGgLm+idg1Kb3C\n\tpotzWkXc1xmPzcQ1fvQMOfMwdS+4SNw4rY9f07Xb2K99rjMwZVDgESKIzhsDB5GY465sCsiQ\n\tcSAZRxqE49RTBq2+EQsbrQpIc8XiffAB8qexh5/QPzCmR4kJgCGeHIXBtgRj+nIkCJPZvZtf\n\tKr2EAbc6tgg6DkAEHJb+1okosV09+0+TXywYvtEop/WUOWQ+zo+Y/OBd+8Ptgt1pDRyOBzL8\n\tRXa8ZqRf0Mwg75D+dKntZeJHzPRJyrlfQokngAAs4PaFt6UfS+ypMAF37T6CeDArQC41V3ko\n\tlPn1yMsVD0p+6i3DPvA/GPIksDC4owjnzVX9kM8Zc5Cx+XoAN0w5Eqo4t6qEVbuettxx55gq\n\t8K8FieAjgjMSxngo/HST8TpFeqI5nVeq0/lqtBRQKumuIqDg+Bkr4L1V/PSB6XgQcOdhtd36\n\tOe9X9dXB8YSNt7VjOcO7BTmFn/Z8r92mSAfHXpb07YJWJosQOQARAQABtDBLaWVyYW4gQmlu\n\tZ2hhbSA8a2llcmFuLmJpbmdoYW1AaWRlYXNvbmJvYXJkLmNvbT6JAlcEEwEKAEECGwMFCwkI\n\tBwIGFQgJCgsCBBYCAwECHgECF4ACGQEWIQSQLdeYP70o/eNy1HqhHkZyEKRh/QUCXWTtygUJ\n\tCyJXZAAKCRChHkZyEKRh/f8dEACTDsbLN2nioNZMwyLuQRUAFcXNolDX48xcUXsWS2QjxaPm\n\tVsJx8Uy8aYkS85mdPBh0C83OovQR/OVbr8AxhGvYqBs3nQvbWuTl/+4od7DfK2VZOoKBAu5S\n\tQK2FYuUcikDqYcFWJ8DQnubxfE8dvzojHEkXw0sA4igINHDDFX3HJGZtLio+WpEFQtCbfTAG\n\tYZslasz1YZRbwEdSsmO3/kqy5eMnczlm8a21A3fKUo3g8oAZEFM+f4DUNzqIltg31OAB/kZS\n\tenKZQ/SWC8PmLg/ZXBrReYakxXtkP6w3FwMlzOlhGxqhIRNiAJfXJBaRhuUWzPOpEDE9q5YJ\n\tBmqQL2WJm1VSNNVxbXJHpaWMH1sA2R00vmvRrPXGwyIO0IPYeUYQa3gsy6k+En/aMQJd27dp\n\taScf9am9PFICPY5T4ppneeJLif2lyLojo0mcHOV+uyrds9XkLpp14GfTkeKPdPMrLLTsHRfH\n\tfA4I4OBpRrEPiGIZB/0im98MkGY/Mu6qxeZmYLCcgD6qz4idOvfgVOrNh+aA8HzIVR+RMW8H\n\tQGBN9f0E3kfwxuhl3omo6V7lDw8XOdmuWZNC9zPq1UfryVHANYbLGz9KJ4Aw6M+OgBC2JpkD\n\thXMdHUkC+d20dwXrwHTlrJi1YNp6rBc+xald3wsUPOZ5z8moTHUX/uPA/qhGsbkCDQRWBP1m\n\tARAAzijkb+Sau4hAncr1JjOY+KyFEdUNxRy+hqTJdJfaYihxyaj0Ee0P0zEi35CbE6lgU0Uz\n\ttih9fiUbSV3wfsWqg1Ut3/5rTKu7kLFp15kF7eqvV4uezXRD3Qu4yjv/rMmEJbbD4cTvGCYI\n\td6MDC417f7vK3hCbCVIZSp3GXxyC1LU+UQr3fFcOyCwmP9vDUR9JV0BSqHHxRDdpUXE26Dk6\n\tmhf0V1YkspE5St814ETXpEus2urZE5yJIUROlWPIL+hm3NEWfAP06vsQUyLvr/GtbOT79vXl\n\tEn1aulcYyu20dRRxhkQ6iILaURcxIAVJJKPi8dsoMnS8pB0QW12AHWuirPF0g6DiuUfPmrA5\n\tPKe56IGlpkjc8cO51lIxHkWTpCMWigRdPDexKX+Sb+W9QWK/0JjIc4t3KBaiG8O4yRX8ml2R\n\t+rxfAVKM6V769P/hWoRGdgUMgYHFpHGSgEt80OKK5HeUPy2cngDUXzwrqiM5Sz6Od0qw5pCk\n\tNlXqI0W/who0iSVM+8+RmyY0OEkxEcci7rRLsGnM15B5PjLJjh1f2ULYkv8s4SnDwMZ/kE04\n\t/UqCMK/KnX8pwXEMCjz0h6qWNpGwJ0/tYIgQJZh6bqkvBrDogAvuhf60Sogw+mH8b+PBlx1L\n\toeTK396wc+4c3BfiC6pNtUS5GpsPMMjYMk7kVvEAEQEAAYkCPAQYAQoAJgIbDBYhBJAt15g/\n\tvSj943LUeqEeRnIQpGH9BQJdizzIBQkLSKZiAAoJEKEeRnIQpGH9eYgQAJpjaWNgqNOnMTmD\n\tMJggbwjIotypzIXfhHNCeTkG7+qCDlSaBPclcPGYrTwCt0YWPU2TgGgJrVhYT20ierN8LUvj\n\t6qOPTd+Uk7NFzL65qkh80ZKNBFddx1AabQpSVQKbdcLb8OFs85kuSvFdgqZwgxA1vl4TFhNz\n\tPZ79NAmXLackAx3sOVFhk4WQaKRshCB7cSl+RIng5S/ThOBlwNlcKG7j7W2MC06BlTbdEkUp\n\tECzuuRBv8wX4OQl+hbWbB/VKIx5HKlLu1eypen/5lNVzSqMMIYkkZcjV2SWQyUGxSwq0O/sx\n\tS0A8/atCHUXOboUsn54qdxrVDaK+6jIAuo8JiRWctP16KjzUM7MO0/+4zllM8EY57rXrj48j\n\tsbEYX0YQnzaj+jO6kJtoZsIaYR7rMMq9aUAjyiaEZpmP1qF/2sYenDx0Fg2BSlLvLvXM0vU8\n\tpQk3kgDu7kb/7PRYrZvBsr21EIQoIjXbZxDz/o7z95frkP71EaICttZ6k9q5oxxA5WC6sTXc\n\tMW8zs8avFNuA9VpXt0YupJd2ijtZy2mpZNG02fFVXhIn4G807G7+9mhuC4XG5rKlBBUXTvPU\n\tAfYnB4JBDLmLzBFavQfvonSfbitgXwCG3vS+9HEwAjU30Bar1PEOmIbiAoMzuKeRm2LVpmq4\n\tWZw01QYHU/GUV/zHJSFk","Organization":"Ideas on Board","Message-ID":"","Date":"Fri, 12 Feb 2021 19:22:37 +0000","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101\n\tThunderbird/68.10.0","MIME-Version":"1.0","In-Reply-To":"<20210212094234.bdxmyqby5fuimdfv@uno.localdomain>","Content-Language":"en-GB","Subject":"Re: [libcamera-devel] [PATCH v2 4/4] cam: Default camera location\n\tto external","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","Reply-To":"kieran.bingham@ideasonboard.com","Cc":"libcamera-devel@lists.libcamera.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":15161,"web_url":"https://patchwork.libcamera.org/comment/15161/","msgid":"<20210213034958.GA66270@pyrite.rasen.tech>","date":"2021-02-13T03:49:58","subject":"Re: [libcamera-devel] [PATCH v2 4/4] cam: Default camera location\n\tto external","submitter":{"id":17,"url":"https://patchwork.libcamera.org/api/people/17/","name":"Paul Elder","email":"paul.elder@ideasonboard.com"},"content":"Hi Jacopo and Kieran,\n\nOn Fri, Feb 12, 2021 at 07:22:37PM +0000, Kieran Bingham wrote:\n> Hi Paul,\n> \n> On 12/02/2021 09:42, Jacopo Mondi wrote:\n> > Hi Paul,\n> > I think the patch subject is wrong :/\n> > \n> > On Fri, Feb 12, 2021 at 02:48:16PM +0900, Paul Elder wrote:\n> >> Now that we default the camera sensor location to Unknown, handle it in\n> >> cam the same way external is handled.\n> > \n> > I would not call an unknown camera \"External\".\n> > This is also the second different default we assign to \"Unknown\" in\n> > this series. For the HAL it defaults to Front, here it defaults to\n> > \"External\". Isn't this a bit confusing ?\n\ntbh I think it's fine for different applications to handle the unknown\nlocation differently.\n\n> > I would rather name it just \"Camera\" without reporting\n> > external/front/back or anything else.\n> > \n> \n> I wouldn't even use Camera, just print the model, and the rest of the\n> string:\n> \n> I.e.:\n> > 1: Camera 'HP Wide Vision FHD Camera: HP W' (\\_SB_.PCI0.XHC_.RHUB.HS05-5:1.0-0408:5251)\n> \n> \n> vs\n> > 2: 'HP Wide Vision FHD Camera: HP I' (\\_SB_.PCI0.XHC_.RHUB.HS05-5:1.2-0408:5251)\n> \n> Otherwise, as this is cam/test tool - we should possibly report that the\n> location is unknown:\n> \n> >> 1: Unknown Location 'HP Wide Vision FHD Camera: HP W' (\\_SB_.PCI0.XHC_.RHUB.HS05-5:1.0-0408:5251)\n\nBut for cam I think this would be the better way.\n\n\nThanks,\n\nPaul\n\n> >>\n> >> Signed-off-by: Paul Elder \n> >>\n> >> ---\n> >> New in v2\n> >> ---\n> >> src/cam/main.cpp | 1 +\n> >> 1 file changed, 1 insertion(+)\n> >>\n> >> diff --git a/src/cam/main.cpp b/src/cam/main.cpp\n> >> index e01be63a..261ed0c8 100644\n> >> --- a/src/cam/main.cpp\n> >> +++ b/src/cam/main.cpp\n> >> @@ -387,6 +387,7 @@ std::string const CamApp::cameraName(const Camera *camera)\n> >> \t\tname = \"Internal back camera\";\n> >> \t\tbreak;\n> >> \tcase properties::CameraLocationExternal:\n> >> +\tcase properties::CameraLocationUnknown:\n> >> \t\tname = \"External camera\";\n> >> \t\tif (props.contains(properties::Model))\n> >> \t\t\tname += \" '\" + props.get(properties::Model) + \"'\";\n> >> --\n> >> 2.27.0\n> >>\n> >> _______________________________________________\n> >> libcamera-devel mailing list\n> >> libcamera-devel@lists.libcamera.org\n> >> https://lists.libcamera.org/listinfo/libcamera-devel\n> > _______________________________________________\n> > libcamera-devel mailing list\n> > libcamera-devel@lists.libcamera.org\n> > https://lists.libcamera.org/listinfo/libcamera-devel\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 65260BD160\n\tfor ;\n\tSat, 13 Feb 2021 03:50:09 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id DE89663799;\n\tSat, 13 Feb 2021 04:50:08 +0100 (CET)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id E5054601B5\n\tfor ;\n\tSat, 13 Feb 2021 04:50:06 +0100 (CET)","from pyrite.rasen.tech (unknown\n\t[IPv6:2400:4051:61:600:2c71:1b79:d06d:5032])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id E97BF564;\n\tSat, 13 Feb 2021 04:50:04 +0100 (CET)"],"Authentication-Results":"lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"dYUEPrNB\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1613188206;\n\tbh=wNpEkbTgquU9ricGdMpE4LoJvwCN0cRdCssRrf0v7m4=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=dYUEPrNBydVPUEqbDyVYyW+pUjIPis2W9BHdzZ1sH8AHkWrc1SbgybfN4KjAm16A+\n\tn/FJ/jnKMD5ze8560kOhTxwp/0HT/E/cFDbtTemb3xHFgIUTxvGY/O5yjIPTEjFf8w\n\tUDKfQ/UWZ5yPe+Ng8wlQltdLDwH4m8na/MLQNdqU=","Date":"Sat, 13 Feb 2021 12:49:58 +0900","From":"paul.elder@ideasonboard.com","To":"Kieran Bingham ","Message-ID":"<20210213034958.GA66270@pyrite.rasen.tech>","References":"<20210212054816.53561-1-paul.elder@ideasonboard.com>\n\t<20210212054816.53561-5-paul.elder@ideasonboard.com>\n\t<20210212094234.bdxmyqby5fuimdfv@uno.localdomain>\n\t","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"","Subject":"Re: [libcamera-devel] [PATCH v2 4/4] cam: Default camera location\n\tto external","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","Cc":"libcamera-devel@lists.libcamera.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}}]