{"id":17146,"url":"https://patchwork.libcamera.org/api/1.1/covers/17146/?format=json","web_url":"https://patchwork.libcamera.org/cover/17146/","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":"<20220818064923.2573060-1-paul.elder@ideasonboard.com>","date":"2022-08-18T06:49:15","name":"[libcamera-devel,v2,0/7] utils: ipc: Add support for enums and Flags","submitter":{"id":17,"url":"https://patchwork.libcamera.org/api/1.1/people/17/?format=json","name":"Paul Elder","email":"paul.elder@ideasonboard.com"},"mbox":"https://patchwork.libcamera.org/cover/17146/mbox/","series":[{"id":3424,"url":"https://patchwork.libcamera.org/api/1.1/series/3424/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=3424","date":"2022-08-18T06:49:15","name":"utils: ipc: Add support for enums and Flags","version":2,"mbox":"https://patchwork.libcamera.org/series/3424/mbox/"}],"comments":"https://patchwork.libcamera.org/api/covers/17146/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 2FC8FC3272\n\tfor ;\n\tThu, 18 Aug 2022 06:49:37 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 74C0B61FC0;\n\tThu, 18 Aug 2022 08:49:36 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id AD12661FA3\n\tfor ;\n\tThu, 18 Aug 2022 08:49:35 +0200 (CEST)","from pyrite.rasen.tech (KD027085204050.au-net.ne.jp [27.85.204.50])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 03EAA4A8;\n\tThu, 18 Aug 2022 08:49:33 +0200 (CEST)"],"DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1660805376;\n\tbh=nyfpckKaYnFmo9U8pl/F+eoE1uK3wQyi9pz1FfoENPM=;\n\th=To:Date:Subject:List-Id:List-Unsubscribe:List-Archive:List-Post:\n\tList-Help:List-Subscribe:From:Reply-To:From;\n\tb=gOIww7e+rGYKrxhb956YKZi810D+lufVpwLGltQUzcOBGH07XT5mLHyxVABLvKcg7\n\ttrGAxhnvsEx8p7uXNjJxrTrnM1FULB4bkUJAi2dAfVy82oY3lVfmqHfMCsaPB6Am6r\n\tVh2ZERQWFMqMCQcJbk/UYWoGHTuw4yBhLf2OyBixpgCjZtbQYWIQnH0Eot4BbfEF0+\n\to49M3ZVSbE4BUJ5YKued6VGj/zvj+O4JAUV2KUoK7IdV3NuGvaqp8d5njy64nA/3d3\n\taKJtjvz81QET4A+qPeMqUFec6PO13Yr6A46gDZ19+raF7C5nM5Ch9TCKOMTOW3qz6S\n\tizHQ5mhRso80w==","v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1660805375;\n\tbh=nyfpckKaYnFmo9U8pl/F+eoE1uK3wQyi9pz1FfoENPM=;\n\th=From:To:Cc:Subject:Date:From;\n\tb=dsJC0jvGaM8HLNHOmWfPlrQxn41+fcXSl7S8R4Zlb4lzk17WdjYj7dM81rFcsgC7G\n\tXT1V/h5A2hN3MXL3ahw0+1iE+5FEAGSC8Se4RCy0IrJpYa55FDHmfDn+ikwe7EI9j9\n\t8+3mahIKxTdzBYptatZ454n5EwWUV2GLyLQ8VCxc="],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key; \n\tunprotected) header.d=ideasonboard.com\n\theader.i=@ideasonboard.com\n\theader.b=\"dsJC0jvG\"; dkim-atps=neutral","To":"libcamera-devel@lists.libcamera.org","Date":"Thu, 18 Aug 2022 15:49:15 +0900","Message-Id":"<20220818064923.2573060-1-paul.elder@ideasonboard.com>","X-Mailer":"git-send-email 2.30.2","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","Subject":"[libcamera-devel] [PATCH v2 0/7] utils: ipc: Add support for enums\n\tand Flags","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","From":"Paul Elder via libcamera-devel ","Reply-To":"Paul Elder ","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "},"content":"This patch series adds support for enums in function parameters, and\nFlags all-around.\n\n\"all-around\" does happen to exclude direction function return values,\nthough.\n\nPatches 1~2 touch enums, where the first patch adds a test for current\nenum support that hasn't been tested before, and the second actually\nadds support for enums in function parameters.\n\nPatches 3~4 add support for Flags, and 5 adds a test for it.\n\nPatches 6 and 7 extend vimc to have dummy function parameters that are\nenums and Flags to test enums and Flags as function parameters. On one\nhand I want these to be included so that we can continue to have tests\nfor function parameters, but I'm not sure that those patches are the\nbest implementation.\n\nPaul Elder (7):\n test: generated_serializer: Test enum that is struct member\n utils: ipc: Add support for enums in function parameters\n libcamera: ipa_data_serializer: Add serializer for Flags\n utils: ipc: Add support for Flags\n test: generated_serializer: Test Flags that is struct member\n [TEST] ipa: vimc: Add IPAOperationCode to init() parameter list\n [TEST] ipa: vimc: Add Flags to parameters\n\n .../libcamera/internal/ipa_data_serializer.h | 46 +++++++++++++++++++\n include/libcamera/ipa/ipa_interface.h | 1 +\n include/libcamera/ipa/vimc.mojom | 14 +++++-\n src/ipa/vimc/vimc.cpp | 23 ++++++++--\n src/libcamera/pipeline/vimc/vimc.cpp | 16 +++++--\n test/ipa/ipa_interface_test.cpp | 6 ++-\n .../generated_serializer_test.cpp | 23 ++++++++++\n .../include/libcamera/ipa/test.mojom | 9 ++++\n .../definition_functions.tmpl | 2 +-\n .../libcamera_templates/proxy_functions.tmpl | 20 +++++++-\n .../libcamera_templates/serializer.tmpl | 4 ++\n .../generators/mojom_libcamera_generator.py | 23 +++++++++-\n 12 files changed, 173 insertions(+), 14 deletions(-)"}