{"id":12791,"url":"https://patchwork.libcamera.org/api/covers/12791/?format=json","web_url":"https://patchwork.libcamera.org/cover/12791/","project":{"id":1,"url":"https://patchwork.libcamera.org/api/projects/1/?format=json","name":"libcamera","link_name":"libcamera","list_id":"libcamera_core","list_email":"libcamera-devel@lists.libcamera.org","web_url":"","scm_url":"","webscm_url":""},"msgid":"<20210702150940.226941-1-naush@raspberrypi.com>","date":"2021-07-02T15:09:32","name":"[libcamera-devel,v3,0/8] Raspberry Pi: Enable imx477 long exposure modes","submitter":{"id":34,"url":"https://patchwork.libcamera.org/api/people/34/?format=json","name":"Naushir Patuck","email":"naush@raspberrypi.com"},"mbox":"https://patchwork.libcamera.org/cover/12791/mbox/","series":[{"id":2203,"url":"https://patchwork.libcamera.org/api/series/2203/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=2203","date":"2021-07-02T15:09:32","name":"Raspberry Pi: Enable imx477 long exposure modes","version":3,"mbox":"https://patchwork.libcamera.org/series/2203/mbox/"}],"comments":"https://patchwork.libcamera.org/api/covers/12791/comments/","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 015B7BD794\n\tfor ;\n\tFri, 2 Jul 2021 15:09:52 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 447F568503;\n\tFri, 2 Jul 2021 17:09:52 +0200 (CEST)","from mail-wm1-x336.google.com (mail-wm1-x336.google.com\n\t[IPv6:2a00:1450:4864:20::336])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id D4956684E6\n\tfor ;\n\tFri, 2 Jul 2021 17:09:50 +0200 (CEST)","by mail-wm1-x336.google.com with SMTP id\n\tg8-20020a1c9d080000b02901f13dd1672aso6047169wme.0\n\tfor ;\n\tFri, 02 Jul 2021 08:09:50 -0700 (PDT)","from naush-laptop.patuck.local ([88.97.76.4])\n\tby smtp.gmail.com with ESMTPSA id\n\tn4sm3619335wrw.21.2021.07.02.08.09.49\n\t(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n\tFri, 02 Jul 2021 08:09:49 -0700 (PDT)"],"Authentication-Results":"lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=raspberrypi.com header.i=@raspberrypi.com\n\theader.b=\"ZkBKZM/O\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=raspberrypi.com; s=google;\n\th=from:to:cc:subject:date:message-id:mime-version\n\t:content-transfer-encoding;\n\tbh=zZEO52OrzRwYwR67G+f8JSK5k40RHvpeUXk6kAoy4zU=;\n\tb=ZkBKZM/O1Zcq7NnSUK663LDCoE8lxste3sfzUNq7IKi9w6dLF8o+AcSE7JSbhSQN/l\n\tz3qEuVFjGV3jvop0V1KHhDcpXxbJFKga3f6bppEH0LDaaI2xjaMBHJvOeEkanr57iEyq\n\tJ+69p4J+h2wkGYNXC5aY359nS3zj4vwq9qPm41lvIjpww3zebFyWXP/VuCr0pbyTp+fI\n\tXHLKzsGLsQLTHLKtPdUEhV3bJay6Gj5fdJ8H7+M8r+yF6JV6xe5fPmCJ0SW8G0a+Wrnf\n\tAm9ALNLLFKiFtiXHmzqSmJQuWM3USyBp8CnpGXPBc+pPdzBUOOtb42jfGDmxbaZrKqoy\n\tUMvQ==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version\n\t:content-transfer-encoding;\n\tbh=zZEO52OrzRwYwR67G+f8JSK5k40RHvpeUXk6kAoy4zU=;\n\tb=k/6QmJTNiTzl+mNW9YAWL7KDtmnqmFdbNIxtoR/4cCbDDwjIn2IHsiv3IhJlfVFkAX\n\tAWikGupl2VWdvgduC4OgoxY58AZNHKnADeEnXsrrgXhidgRKsCAfwyjmXlkS4ua7qxoL\n\tF5ntaUmVwQEGAM4PzXzfpBhgTHuwfVpvM86mf4KYWR216l4h2ZzojenjcA5EDBfj9imR\n\t6nyXgTOPYOQWJSbq+rqXK1RpyfP5VtLZxVPKlaXevF6bcm8vEjzKYp0xo7k5cLSWTHKf\n\t7WkqyU9IACs6IaB2yH4hNDdMwvmHF04OXq3L+aZZ1pMaY7S54bSak9zjAkAd8Py0whyT\n\teqBA==","X-Gm-Message-State":"AOAM531SLySWAaL+sObaQoQQq76Lu5UZTdeLuiR7G7anun+Seh1IzBbR\n\ttbjqORhOrPKB2zN8k+9HiT270TwTrOptNQ==","X-Google-Smtp-Source":"ABdhPJxD02kr1njc4t1vpYi0jdI/+ZVkc4/2ZjkUIc1ZOZs4Nk6CMSd+Cl17dUyWYYZxDXa/LAalKA==","X-Received":"by 2002:a05:600c:4f4e:: with SMTP id\n\tm14mr229726wmq.151.1625238590041; \n\tFri, 02 Jul 2021 08:09:50 -0700 (PDT)","From":"Naushir Patuck ","To":"libcamera-devel@lists.libcamera.org","Date":"Fri, 2 Jul 2021 16:09:32 +0100","Message-Id":"<20210702150940.226941-1-naush@raspberrypi.com>","X-Mailer":"git-send-email 2.25.1","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","Subject":"[libcamera-devel] [PATCH v3 0/8] Raspberry Pi: Enable imx477 long\n\texposure modes","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\" "},"content":"Hi,\n\nThere has been some substantial rework on this patch series in version 3:\n\n- Added patches 1/8 - 3/8 to C++ify our DeviceStatus structure.\n- Added patch 4/8 to store the frame length in DeviceStatus.\n- Updated patch 5/8 to use frame length instead of exposure time to determine\nif we are in long exposure mode. The previous way of using exposure time would\nbe incorrect in cases where we have a short exposure time, and a long frame\nlength. Also change the CamHelperImx477::Prepare() method to not touch the metadata\nbuffers so many times.\n- Patches 6/8 - 8/8 remain unchanged.\n\nRegards,\nNaush\n\nNaushir Patuck (8):\n ipa: raspberrypi: Make device_status.h C++ only header, and update\n comments\n ipa: raspberrypi: Add a constructor struct DeviceStatus\n ipa: raspberrypi: Add an operator<< to struct DeviceStatus\n ipa: raspberrypi: Add frame_length to DeviceStatus\n ipa: raspberrypi: Allow long exposure modes for imx477.\n pipeline: raspberrypi: Use priority write for vblank when writing\n sensor ctrls\n ipa: raspberrypi: Increase the default max frame duration to 250s\n ipa: raspberrypi: Make variable names consistent\n\n src/ipa/raspberrypi/cam_helper.cpp | 5 +-\n src/ipa/raspberrypi/cam_helper_imx219.cpp | 6 +-\n src/ipa/raspberrypi/cam_helper_imx477.cpp | 101 +++++++++++++++++-\n .../raspberrypi/controller/device_status.h | 48 ++++++---\n src/ipa/raspberrypi/controller/rpi/agc.cpp | 2 +-\n src/ipa/raspberrypi/controller/rpi/lux.cpp | 8 +-\n src/ipa/raspberrypi/raspberrypi.cpp | 17 ++-\n .../pipeline/raspberrypi/raspberrypi.cpp | 37 +++++--\n 8 files changed, 176 insertions(+), 48 deletions(-)"}