Cover Letter Detail
Show a cover letter.
GET /api/1.1/covers/13168/?format=api
{ "id": 13168, "url": "https://patchwork.libcamera.org/api/1.1/covers/13168/?format=api", "web_url": "https://patchwork.libcamera.org/cover/13168/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/1.1/projects/1/?format=api", "name": "libcamera", "link_name": "libcamera", "list_id": "libcamera_core", "list_email": "libcamera-devel@lists.libcamera.org", "web_url": "", "scm_url": "", "webscm_url": "" }, "msgid": "<20210730110154.181370-1-umang.jain@ideasonboard.com>", "date": "2021-07-30T11:01:50", "name": "[libcamera-devel,v3,0/4] android: Handle internal UVC cameras", "submitter": { "id": 86, "url": "https://patchwork.libcamera.org/api/1.1/people/86/?format=api", "name": "Umang Jain", "email": "umang.jain@ideasonboard.com" }, "mbox": "https://patchwork.libcamera.org/cover/13168/mbox/", "series": [ { "id": 2292, "url": "https://patchwork.libcamera.org/api/1.1/series/2292/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=2292", "date": "2021-07-30T11:01:50", "name": "android: Handle internal UVC cameras", "version": 3, "mbox": "https://patchwork.libcamera.org/series/2292/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/covers/13168/comments/", "headers": { "Return-Path": "<libcamera-devel-bounces@lists.libcamera.org>", "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 DAE39C3230\n\tfor <parsemail@patchwork.libcamera.org>;\n\tFri, 30 Jul 2021 11:02:09 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 4F67A687C5;\n\tFri, 30 Jul 2021 13:02:09 +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 85311687BF\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tFri, 30 Jul 2021 13:02:08 +0200 (CEST)", "from perceval.ideasonboard.com (unknown [103.251.226.16])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 5F9E189B;\n\tFri, 30 Jul 2021 13:02:07 +0200 (CEST)" ], "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=\"Rkp4aVzv\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1627642928;\n\tbh=c+34qa8aMZBTvDHQXVehLuouzjpJuViqAwK3oPMEZDM=;\n\th=From:To:Cc:Subject:Date:From;\n\tb=Rkp4aVzvM1vY00wmkV/UJN05ZxgYp2xjt8nyi4kYQrtDlvdLQnjbnoydt0XiyP2WK\n\tZInyl7pHOc2XWA/KBENKA6LnLsri8GMv1HnykU9b52oyvR3M2TICaFUPXEAs/zdCok\n\tOKeIJZFnMN69DBRbDWuxAdqiJVe1AlJ3ltBL1WfE=", "From": "Umang Jain <umang.jain@ideasonboard.com>", "To": "libcamera-devel@lists.libcamera.org", "Date": "Fri, 30 Jul 2021 16:31:50 +0530", "Message-Id": "<20210730110154.181370-1-umang.jain@ideasonboard.com>", "X-Mailer": "git-send-email 2.31.1", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[libcamera-devel] [PATCH v3 0/4] android: Handle internal UVC\n\tcameras", "X-BeenThere": "libcamera-devel@lists.libcamera.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "<libcamera-devel.lists.libcamera.org>", "List-Unsubscribe": "<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>", "List-Archive": "<https://lists.libcamera.org/pipermail/libcamera-devel/>", "List-Post": "<mailto:libcamera-devel@lists.libcamera.org>", "List-Help": "<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>", "List-Subscribe": "<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>", "Errors-To": "libcamera-devel-bounces@lists.libcamera.org", "Sender": "\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>" }, "content": "libcamera-core all UVC cameras as 'external' (which makes sense\nas there is not much information to accurately determine location).\nHowever, in the HAL layer, we can probably determine accurately\nby looking at HAL config file camera entries.\n\nThis series addresses the handling of such cameras, where the libcamera\nreports properties::CameraLocationExternal, but we know one of the UVC\nis internal/integrated to the system. It should be treated as internal.\nAlso, as a point raised while reviews, we should disallow\n'location: external' in HAL config file.\n\nTesting:\nWith nautilus, it has one UVC and one IPU3 cameras. With these patches\napplied, both these cameras have id as '0' & '1' (i.e. internal).\n\nWith camera service still running, I hotplugged a external UVC camera,\nand it got the camera numerical id as '1000'.\n\nChanges in v3:\n- Introduce a new patch 2/4 to disallow 'external' location in HAL\n config\n- In 3/4, Reflect and adapt `if` checks accordingly in context of 2/4.\n (No tags have been collected intentionally due to change in `if`\n conditional checks)\n- Few minor clarifications in comments.\n\nChange in v2:\n- Introduce a re-factor patch 1/3\n- Change comments in 2/3\n\nUmang Jain (4):\n android: Instantiate CameraDevice after checking HAL config validity\n android: Disallow external location in HAL config\n android: Override camera as \"Internal\" provided found in HAL config\n android: nautilus: Add camera HAL configuration\n\n src/android/camera_device.cpp | 13 ++++++++++++-\n src/android/camera_hal_config.cpp | 2 --\n src/android/camera_hal_manager.cpp | 23 ++++++++++++++++++++---\n src/android/data/nautilus/camera_hal.yaml | 8 ++++++++\n 4 files changed, 40 insertions(+), 6 deletions(-)\n create mode 100644 src/android/data/nautilus/camera_hal.yaml" }