{"id":355,"url":"https://patchwork.libcamera.org/api/1.1/covers/355/?format=json","web_url":"https://patchwork.libcamera.org/cover/355/","project":{"id":1,"url":"https://patchwork.libcamera.org/api/1.1/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":"<20190124101651.9993-1-laurent.pinchart@ideasonboard.com>","date":"2019-01-24T10:16:41","name":"[libcamera-devel,00/10] Hotplug support and object lifetime management","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/1.1/people/2/?format=json","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"mbox":"https://patchwork.libcamera.org/cover/355/mbox/","series":[{"id":125,"url":"https://patchwork.libcamera.org/api/1.1/series/125/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=125","date":"2019-01-24T10:16:41","name":"Hotplug support and object lifetime management","version":1,"mbox":"https://patchwork.libcamera.org/series/125/mbox/"}],"comments":"https://patchwork.libcamera.org/api/covers/355/comments/","headers":{"Return-Path":"<laurent.pinchart@ideasonboard.com>","Received":["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 21AF060C78\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tThu, 24 Jan 2019 11:16:57 +0100 (CET)","from pendragon.bb.dnainternet.fi\n\t(dfj612yhrgyx302h3jwwy-3.rev.dnainternet.fi\n\t[IPv6:2001:14ba:21f5:5b00:ce28:277f:58d7:3ca4])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 776F123A;\n\tThu, 24 Jan 2019 11:16:56 +0100 (CET)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1548325016;\n\tbh=/2DeNMKzSiE1WCoJiV9h6bSZWGu1kxiAP+nNqD/JeFA=;\n\th=From:To:Cc:Subject:Date:From;\n\tb=jX+YZJfKEI834nGWz1YOSsQzxCkQ//+9BsY4QHWX08JASGBZvWjkgGOtDnlh///ji\n\t5stQ6NQaDT5b9CiqGuBUQSZf1o7Es+n2eePQXjBgikZy6zqm5SlD/P+s7zMiScMLP1\n\tJLsiEI6qpBUADmtxM8TuC+3zHMuLE3sTjSc0zglc=","From":"Laurent Pinchart <laurent.pinchart@ideasonboard.com>","To":"libcamera-devel@lists.libcamera.org","Date":"Thu, 24 Jan 2019 12:16:41 +0200","Message-Id":"<20190124101651.9993-1-laurent.pinchart@ideasonboard.com>","X-Mailer":"git-send-email 2.19.2","MIME-Version":"1.0","Content-Type":"text/plain; charset=UTF-8","Content-Transfer-Encoding":"8bit","Subject":"[libcamera-devel] [PATCH 00/10] Hotplug support and object lifetime\n\tmanagement","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.23","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>","X-List-Received-Date":"Thu, 24 Jan 2019 10:16:57 -0000"},"content":"Hello everybody,\n\nThis patch series implements hotplug support in libcamera, and handles\nall the lifetime management-related fallouts.\n\nThe first two patches, 01/10 and 02/10, are simple cleanups. The real\nfun begins with pPatch 03/10 that stores a reference to the pipeline\nhandler in the camera, turning PipelineHandler into shared pointers.\nPatch 04/10 turns MediaDevice into shared pointers as well, due to the\nneed to support hot-unplug when references to the media device can still\nexist. This results in all three core objects related to hotplug,\nMediaDevice, PipelineHandler and Camera, being managed as shared\npointers.\n\nThe next four patches, 05/10 to 08/10, add disconnection support to\nMediaDevice, Camera, CameraManager and PipelineHandler through signals\nand slots. Patch 09/10 marks the media device used by the UVC pipeline\nhandler as hot-pluggable, and patch 10/10 implements unplug detection in\nthe udev enumerator for the last piece of the puzzle.\n\nWhile I hope the architecture is fine, I'm sure there will be lots of\nvarious issues that will be discussed, and I expect a few more versions\nof this series before merging it in the master branch. If the\narchitecture is approved, I would however recommend already depending on\nthis series as it addresses lots of lifetime management problems.\n\nLaurent Pinchart (7):\n  libcamera: pipeline_handler: Store the camera manager pointer\n  libcamera: pipeline_handler: Declare factory children classes as final\n  libcamera: device_enumerator: Reference-count MediaDevice instances\n  libcamera: media_device: Add disconnected signal\n  libcamera: camera_manager: Add method to unregister a camera\n  libcamera: pipeline: uvc: Mark the media device as hotpluggable\n  libcamera: device_enumerator: Add hotplug support\n\nNiklas Söderlund (3):\n  libcamera: camera: Associate cameras with their pipeline handler\n  libcamera: camera: Add disconnection notification\n  libcamera: pipeline_handler: Add camera disconnection support\n\n include/libcamera/camera.h                   |  15 ++-\n include/libcamera/camera_manager.h           |   3 +-\n src/libcamera/camera.cpp                     |  47 +++++++-\n src/libcamera/camera_manager.cpp             |  38 ++++--\n src/libcamera/device_enumerator.cpp          | 106 ++++++++++++++---\n src/libcamera/include/device_enumerator.h    |  10 +-\n src/libcamera/include/media_device.h         |   4 +\n src/libcamera/include/pipeline_handler.h     |  33 ++++--\n src/libcamera/media_device.cpp               |  10 ++\n src/libcamera/pipeline/ipu3/ipu3.cpp         |  32 +++--\n src/libcamera/pipeline/uvcvideo.cpp          |  20 ++--\n src/libcamera/pipeline/vimc.cpp              |  26 ++---\n src/libcamera/pipeline_handler.cpp           | 117 ++++++++++++++++++-\n test/media_device/media_device_link_test.cpp |   2 +-\n test/pipeline/ipu3/ipu3_pipeline_test.cpp    |   2 +-\n 15 files changed, 372 insertions(+), 93 deletions(-)"}