[{"id":27757,"web_url":"https://patchwork.libcamera.org/comment/27757/","msgid":"<169452148260.2427060.9712252227388588802@ping.linuxembedded.co.uk>","date":"2023-09-12T12:24:42","subject":"Re: [libcamera-devel] [PATCH 0/4] android: switch over to modern\n\tgralloc API via libui","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/people/4/","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"content":"Hi Mattijs,\n\nThis looks like a very interesting development.\nI've just tried to run it through my integration tests and get the\nfollowing:\n\nclang++-11 -Isrc/android/libcamera-hal.so.p -Isrc/android -I../../../src/libcamera/src/android -I../../../src/libcamera/include/android/frameworks/native/libs/ui/include -I../../../src/libcamera/include/android/hardware/libhardware/include -I../../../src/libcamera/include/android/metadata -I../../../src/libcamera/include/android/system/core/include -I../../../src/libcamera/include/android/system/core/libutils/include -Iinclude -I../../../src/libcamera/include -I../../../src/libcamera/subprojects/libyuv/include -Isubprojects/libyuv/__CMake_build -I../../../src/libcamera/subprojects/libyuv/__CMake_build -Isubprojects/libyuv -I../../../src/libcamera/subprojects/libyuv -Iinclude/libcamera/ipa -Iinclude/libcamera -fcolor-diagnostics -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wnon-virtual-dtor -Wextra -Werror -std=c++17 -O0 -g -Wextra-semi -Wthread-safety -Wshadow -include /home/kbingham/iob/libcamera/ci/integrator/builds/build-matrix/clang-11/config.h -Wno-c99-designator -fPIC -DLIBCAMERA_BASE_PRIVATE -MD -MQ src/android/libcamera-hal.so.p/mm_graphic_buffer_allocator_stub.cpp.o -MF src/android/libcamera-hal.so.p/mm_graphic_buffer_allocator_stub.cpp.o.d -o src/android/libcamera-hal.so.p/mm_graphic_buffer_allocator_stub.cpp.o -c ../../../src/libcamera/src/android/mm/graphic_buffer_allocator_stub.cpp\nIn file included from ../../../src/libcamera/src/android/mm/graphic_buffer_allocator_stub.cpp:10:\nIn file included from ../../../src/libcamera/include/android/frameworks/native/libs/ui/include/ui/GraphicBufferAllocator.h:29:\n../../../src/libcamera/include/android/frameworks/native/libs/ui/include/ui/PixelFormat.h:73:2: error: extra ';' outside of a function is incompatible with C++98 [-Werror,-Wc++98-compat-extra-semi]\n}; // namespace android\n ^\nIn file included from ../../../src/libcamera/src/android/mm/graphic_buffer_allocator_stub.cpp:10:\n../../../src/libcamera/include/android/frameworks/native/libs/ui/include/ui/GraphicBufferAllocator.h:79:2: error: extra ';' outside of a function is incompatible with C++98 [-Werror,-Wc++98-compat-extra-semi]\n}; // namespace android\n ^\n../../../src/libcamera/src/android/mm/graphic_buffer_allocator_stub.cpp:50:2: error: extra ';' outside of a function is incompatible with C++98 [-Werror,-Wc++98-compat-extra-semi]\n}; // namespace android\n ^\n\n\nI guess you haven't hit these warnings ?\n\n\nThis is building with clang-11.\n\nFor files we import we should probably disable the extra warning level.\nI don't think we should modify external headers.\n\nBut for files in libcamera\n(src/libcamera/src/android/mm/graphic_buffer_allocator_stub.cpp) we\nshould probably keep consistent with the rest of the coding style.\n\nCan you investigate this please?\n\n--\nKieran\n\n\n\nQuoting Mattijs Korpershoek via libcamera-devel (2023-09-12 10:36:09)\n> gralloc.h is a very old API that has been deprecated at least since\n> Android P (9).\n> \n> Devices are encouraged to switch over to HIDL interface named\n> android.hardware.graphics.allocator@, where can be\n> 2.0 ,3.0 or 4.0.\n> \n> This is mandatory since Android Q (10) [1]\n> \n> Fortunately, Android provides an abstraction on top of\n> android.hardware.graphics.allocator which is compatible with each\n> version.\n> This abstraction is implemented in libui, which is available in the\n> VNDK.\n> \n> Import all necessary headers from AOSP and switch over the\n> generic_frame_buffer_allocator to use GraphicBufferAllocator.\n> \n> This series has been build-tested on a linux host and functionally\n> tested on an AM62x SK EVM with Android 13. (preview and capture).\n> \n> [1] https://android.googlesource.com/platform/hardware/interfaces/+/refs/heads/main/compatibility_matrices/compatibility_matrix.4.xml#195\n> \n> Signed-off-by: Mattijs Korpershoek \n> ---\n> Mattijs Korpershoek (4):\n> android: Import libutils/libui headers from vndk v33\n> android: Import GraphicBufferAllocator header from vndk v33\n> android: Stub GraphicBufferAllocator for build tests\n> android: mm: generic: Use GraphicBufferAllocator instead of gralloc.h\n> \n> .../libs/ui/include/ui/GraphicBufferAllocator.h | 81 ++++++++\n> .../native/libs/ui/include/ui/PixelFormat.h | 75 +++++++\n> include/android/meson.build | 2 +\n> .../system/core/libutils/include/utils/Compat.h | 94 +++++++++\n> .../system/core/libutils/include/utils/Errors.h | 78 ++++++++\n> .../system/core/libutils/include/utils/Mutex.h | 219 +++++++++++++++++++++\n> .../system/core/libutils/include/utils/Singleton.h | 102 ++++++++++\n> .../system/core/libutils/include/utils/Timers.h | 103 ++++++++++\n> src/android/mm/generic_frame_buffer_allocator.cpp | 58 ++----\n> src/android/mm/graphic_buffer_allocator_stub.cpp | 50 +++++\n> src/android/mm/libhardware_stub.c | 17 --\n> src/android/mm/meson.build | 9 +-\n> 12 files changed, 825 insertions(+), 63 deletions(-)\n> ---\n> base-commit: 58e501c71c47e57f02afde1bd296a037038cd6d5\n> change-id: 20230824-gralloc-api-v4-e3388fd364c6\n> \n> Best regards,\n> -- \n> Mattijs Korpershoek \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 2BC7DBE080\n\tfor ;\n\tTue, 12 Sep 2023 12:24:47 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 71F68628EC;\n\tTue, 12 Sep 2023 14:24:46 +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 3DA6061DEF\n\tfor ;\n\tTue, 12 Sep 2023 14:24:45 +0200 (CEST)","from pendragon.ideasonboard.com\n\t(aztw-30-b2-v4wan-166917-cust845.vm26.cable.virginm.net\n\t[82.37.23.78])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id B8FE512F;\n\tTue, 12 Sep 2023 14:23:13 +0200 (CEST)"],"DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1694521486;\n\tbh=+Hxk6ARqCw5HiHFcQLWjbnhtMZC/DTszGUihxqJkxCo=;\n\th=In-Reply-To:References:To:Date:Subject:List-Id:List-Unsubscribe:\n\tList-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:\n\tFrom;\n\tb=X19lSWim6oXji+jVLvoZk2AwaJGnCIStxvMPMdTlds0kiC+P4pFtKD8Gy6w4WFXlV\n\tA0MoXbPAzp+mDzc0qWWc3NC1TnDUBRObcF5E1s0ou14ZUBxUD8mgPrcWMF+XIHsIUB\n\t0jV+93rim17iiSIW7A44cOLUOsJHlHQ7i0R15tT2/a/2MUeTePVFnSnjcjeS8l/WVM\n\tih1e+H8Zc5RTESMF1DOHjBLhB9eRJctl9c08+MSsZNs6CkIvdJc3qbSiUfWvkhY7BX\n\tioYxf7IVK/RnAEBDWIb8Zi/qZd4s2xtdfxyC29v7AEPja+zX8IOKpbiQa6kBMnNJUh\n\t9aQzJV0CpAkkw==","v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1694521393;\n\tbh=+Hxk6ARqCw5HiHFcQLWjbnhtMZC/DTszGUihxqJkxCo=;\n\th=In-Reply-To:References:Subject:From:To:Date:From;\n\tb=EbBGowGdt+GqmXNOnGxQsBbtVCdq4oGDXFwP6050ksEtJU/LPwNQFgP2RW0Ng4XSY\n\tTtiEh3rCX/PxLnFFvvl58IOXJkPiHej3VCItEtz1rlVzk2T1R06UKKzvEbOdey9hc0\n\tl2CZsYVnJqbgZslxQDpHemyDtaYRSo+gxeoiFucQ="],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key; \n\tunprotected) header.d=ideasonboard.com\n\theader.i=@ideasonboard.com\n\theader.b=\"EbBGowGd\"; dkim-atps=neutral","Content-Type":"text/plain; charset=\"utf-8\"","MIME-Version":"1.0","Content-Transfer-Encoding":"quoted-printable","In-Reply-To":"<20230912-gralloc-api-v4-v1-0-0f80402d8e7a@baylibre.com>","References":"<20230912-gralloc-api-v4-v1-0-0f80402d8e7a@baylibre.com>","To":"Mattijs Korpershoek ,\n\tlibcamera-devel@lists.libcamera.org","Date":"Tue, 12 Sep 2023 13:24:42 +0100","Message-ID":"<169452148260.2427060.9712252227388588802@ping.linuxembedded.co.uk>","User-Agent":"alot/0.10","Subject":"Re: [libcamera-devel] [PATCH 0/4] android: switch over to modern\n\tgralloc API via libui","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":"Kieran Bingham via libcamera-devel\n\t","Reply-To":"Kieran Bingham ","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}},{"id":27758,"web_url":"https://patchwork.libcamera.org/comment/27758/","msgid":"<87ledblgka.fsf@baylibre.com>","date":"2023-09-12T12:30:29","subject":"Re: [libcamera-devel] [PATCH 0/4] android: switch over to modern\n\tgralloc API via libui","submitter":{"id":153,"url":"https://patchwork.libcamera.org/api/people/153/","name":"Mattijs Korpershoek","email":"mkorpershoek@baylibre.com"},"content":"Hi Kieran,\n\nOn mar., sept. 12, 2023 at 13:24, Kieran Bingham wrote:\n\n> Hi Mattijs,\n>\n> This looks like a very interesting development.\n> I've just tried to run it through my integration tests and get the\n> following:\n>\n> clang++-11 -Isrc/android/libcamera-hal.so.p -Isrc/android -I../../../src/libcamera/src/android -I../../../src/libcamera/include/android/frameworks/native/libs/ui/include -I../../../src/libcamera/include/android/hardware/libhardware/include -I../../../src/libcamera/include/android/metadata -I../../../src/libcamera/include/android/system/core/include -I../../../src/libcamera/include/android/system/core/libutils/include -Iinclude -I../../../src/libcamera/include -I../../../src/libcamera/subprojects/libyuv/include -Isubprojects/libyuv/__CMake_build -I../../../src/libcamera/subprojects/libyuv/__CMake_build -Isubprojects/libyuv -I../../../src/libcamera/subprojects/libyuv -Iinclude/libcamera/ipa -Iinclude/libcamera -fcolor-diagnostics -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wnon-virtual-dtor -Wextra -Werror -std=c++17 -O0 -g -Wextra-semi -Wthread-safety -Wshadow -include /home/kbingham/iob/libcamera/ci/integrator/builds/build-matrix/clang-11/config.h -Wno-c99-designator -fPIC -DLIBCAMERA_BASE_PRIVATE -MD -MQ src/android/libcamera-hal.so.p/mm_graphic_buffer_allocator_stub.cpp.o -MF src/android/libcamera-hal.so.p/mm_graphic_buffer_allocator_stub.cpp.o.d -o src/android/libcamera-hal.so.p/mm_graphic_buffer_allocator_stub.cpp.o -c ../../../src/libcamera/src/android/mm/graphic_buffer_allocator_stub.cpp\n> In file included from ../../../src/libcamera/src/android/mm/graphic_buffer_allocator_stub.cpp:10:\n> In file included from ../../../src/libcamera/include/android/frameworks/native/libs/ui/include/ui/GraphicBufferAllocator.h:29:\n> ../../../src/libcamera/include/android/frameworks/native/libs/ui/include/ui/PixelFormat.h:73:2: error: extra ';' outside of a function is incompatible with C++98 [-Werror,-Wc++98-compat-extra-semi]\n> }; // namespace android\n> ^\n> In file included from ../../../src/libcamera/src/android/mm/graphic_buffer_allocator_stub.cpp:10:\n> ../../../src/libcamera/include/android/frameworks/native/libs/ui/include/ui/GraphicBufferAllocator.h:79:2: error: extra ';' outside of a function is incompatible with C++98 [-Werror,-Wc++98-compat-extra-semi]\n> }; // namespace android\n> ^\n> ../../../src/libcamera/src/android/mm/graphic_buffer_allocator_stub.cpp:50:2: error: extra ';' outside of a function is incompatible with C++98 [-Werror,-Wc++98-compat-extra-semi]\n> }; // namespace android\n> ^\n\nThank you for the quick feedback and sorry I missed this.\n\nI was aware that including libui.h requires us to ignore extra-semi\ncolumns because GraphicBufferAllocator.h has some.\n\n>\n>\n> I guess you haven't hit these warnings ?\n\nI did in the Android build system, but not when building with meson.\n\n>\n>\n> This is building with clang-11.\n\nAck.\n\n>\n> For files we import we should probably disable the extra warning level.\n> I don't think we should modify external headers.\n>\n> But for files in libcamera\n> (src/libcamera/src/android/mm/graphic_buffer_allocator_stub.cpp) we\n> should probably keep consistent with the rest of the coding style.\n>\n> Can you investigate this please?\n\nYes, will do and send a v2.\n\n>\n> --\n> Kieran\n>\n>\n>\n> Quoting Mattijs Korpershoek via libcamera-devel (2023-09-12 10:36:09)\n>> gralloc.h is a very old API that has been deprecated at least since\n>> Android P (9).\n>> \n>> Devices are encouraged to switch over to HIDL interface named\n>> android.hardware.graphics.allocator@, where can be\n>> 2.0 ,3.0 or 4.0.\n>> \n>> This is mandatory since Android Q (10) [1]\n>> \n>> Fortunately, Android provides an abstraction on top of\n>> android.hardware.graphics.allocator which is compatible with each\n>> version.\n>> This abstraction is implemented in libui, which is available in the\n>> VNDK.\n>> \n>> Import all necessary headers from AOSP and switch over the\n>> generic_frame_buffer_allocator to use GraphicBufferAllocator.\n>> \n>> This series has been build-tested on a linux host and functionally\n>> tested on an AM62x SK EVM with Android 13. (preview and capture).\n>> \n>> [1] https://android.googlesource.com/platform/hardware/interfaces/+/refs/heads/main/compatibility_matrices/compatibility_matrix.4.xml#195\n>> \n>> Signed-off-by: Mattijs Korpershoek \n>> ---\n>> Mattijs Korpershoek (4):\n>> android: Import libutils/libui headers from vndk v33\n>> android: Import GraphicBufferAllocator header from vndk v33\n>> android: Stub GraphicBufferAllocator for build tests\n>> android: mm: generic: Use GraphicBufferAllocator instead of gralloc.h\n>> \n>> .../libs/ui/include/ui/GraphicBufferAllocator.h | 81 ++++++++\n>> .../native/libs/ui/include/ui/PixelFormat.h | 75 +++++++\n>> include/android/meson.build | 2 +\n>> .../system/core/libutils/include/utils/Compat.h | 94 +++++++++\n>> .../system/core/libutils/include/utils/Errors.h | 78 ++++++++\n>> .../system/core/libutils/include/utils/Mutex.h | 219 +++++++++++++++++++++\n>> .../system/core/libutils/include/utils/Singleton.h | 102 ++++++++++\n>> .../system/core/libutils/include/utils/Timers.h | 103 ++++++++++\n>> src/android/mm/generic_frame_buffer_allocator.cpp | 58 ++----\n>> src/android/mm/graphic_buffer_allocator_stub.cpp | 50 +++++\n>> src/android/mm/libhardware_stub.c | 17 --\n>> src/android/mm/meson.build | 9 +-\n>> 12 files changed, 825 insertions(+), 63 deletions(-)\n>> ---\n>> base-commit: 58e501c71c47e57f02afde1bd296a037038cd6d5\n>> change-id: 20230824-gralloc-api-v4-e3388fd364c6\n>> \n>> Best regards,\n>> -- \n>> Mattijs Korpershoek \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 1A00EBD160\n\tfor ;\n\tTue, 12 Sep 2023 12:30:33 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 674FD61DF5;\n\tTue, 12 Sep 2023 14:30:32 +0200 (CEST)","from mail-wm1-x334.google.com (mail-wm1-x334.google.com\n\t[IPv6:2a00:1450:4864:20::334])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 0A85861DEF\n\tfor ;\n\tTue, 12 Sep 2023 14:30:31 +0200 (CEST)","by mail-wm1-x334.google.com with SMTP id\n\t5b1f17b1804b1-400a087b0bfso59363565e9.2\n\tfor ;\n\tTue, 12 Sep 2023 05:30:30 -0700 (PDT)","from localhost ([82.66.159.240]) by smtp.gmail.com with ESMTPSA id\n\tn21-20020a1c7215000000b003fc16ee2864sm12677572wmc.48.2023.09.12.05.30.29\n\t(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n\tTue, 12 Sep 2023 05:30:30 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1694521832;\n\tbh=+vq7+tuyZjybBkW6QJVk2nG/6+b5LlCFj7ysAUq31WQ=;\n\th=To:In-Reply-To:References:Date:Subject:List-Id:List-Unsubscribe:\n\tList-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:\n\tFrom;\n\tb=EzZu73ceKjPlAvok4mIAM7juGQeAmT4AiNnY/RD/Ve1ATa7FdH0so0+Qp+P2OI4eh\n\tAsZwL0C1qsepb3RhYFmafCslBKVzVcDMAefFlGZuMcbpxgsI/RW30C6WWcqs18GtOa\n\tX4P7ep8PRuv5TBe5LmwDuIedRVskIk1volLCvHcJcFxNQJOjhD/sBlyVryt3fNrnU0\n\tOpXKKkX8DBwYc7Wh5TPXy88cY+0aH4sw9d1o+BHFIMTgbPo5mJ95L9hM3sIbXLjtW5\n\t2MuzinzFLtFYE0mHQhvVBxEARbzGcXw3e3C0xIMPYKOd78SmFJPJbb1mz/LnAIsMdL\n\t+SnH1HzIyNMRw==","v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1694521830;\n\tx=1695126630; darn=lists.libcamera.org; \n\th=content-transfer-encoding:mime-version:message-id:date:references\n\t:in-reply-to:subject:to:from:from:to:cc:subject:date:message-id\n\t:reply-to; bh=Xf/j+38R9q+5blKnLEI8XUevk8e3lNZKFR/Mv7tiGg4=;\n\tb=k/RG+IbGQVU+RKLhGuy3ioMTnY7aFuRgCq/GB8T34Lbw2MxvXwnDLHV9/TJ7/UjyDz\n\teFATHPg2NRjNA/L3HJW9TrsrsqS4B2GGVZW+LZt/MLseubd3y4JiFlPUAEJy2ktgZG11\n\td8li2Ven3JdOmOZJ4rcPUw1XhKiWEWIRFvWpydOP/PKvki45EBYVbdFDhL8Qf1AQqnCK\n\tfV9F4mmL117UvZTAjyjfC8qwPtQmqwYuUkKTUL9DpIIpS3REL4zRz1NqdcfiWXbtJHet\n\tY2AfILKIAM7hnNB38ImmZFnFgs1h2orHOUhl5Fn0JLCT+zgzNdr0OW9IB8ONhKBNn1bP\n\tcW/Q=="],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (2048-bit key; \n\tunprotected)\n\theader.d=baylibre-com.20230601.gappssmtp.com\n\theader.i=@baylibre-com.20230601.gappssmtp.com header.b=\"k/RG+IbG\"; \n\tdkim-atps=neutral","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20230601; t=1694521830; x=1695126630;\n\th=content-transfer-encoding:mime-version:message-id:date:references\n\t:in-reply-to:subject:to:from:x-gm-message-state:from:to:cc:subject\n\t:date:message-id:reply-to;\n\tbh=Xf/j+38R9q+5blKnLEI8XUevk8e3lNZKFR/Mv7tiGg4=;\n\tb=vm1SnVHtAOMJ+NgyXMcWuMBRQfuzAR6r39kgzd+J/HJfS+fyhtnxo9vrOmDeok6Y3s\n\tZzPGL4OXY8womLtpktDmo9wJfqqCSD9xaq4wkRlGpCmsKfhIXQUtCMnrtKiGMWbDC3un\n\t2BDn6O4pru/kkX9ciqRYiJ8UmW5910qenrJgsDfumc0JJit32RA0ZczE6Vnx8FUVq1Ig\n\t8h7XbAmsb8ZN5m5r2OJqt3y4A8BcmZp9kxcLu0e9NITpxIXB350Iy1j2MCvqLFo0/hJf\n\td/5kGM3CNEpbvvHAwjI8Ln5EMVu+VK5mAdCORBzl52uOjoRUMFsJtsL17jOPCCmoaZyO\n\tAQeg==","X-Gm-Message-State":"AOJu0YzWDw0HmuyQ7XvnLYuyW7GffNZAx5pdo+Mdoa4kBajlPhMrNr2A\n\tlic8xk5YKIT3VqphzRglVRnsuAhz+NP9R1b09vs=","X-Google-Smtp-Source":"AGHT+IE58+OHLBHGzldjvYkLyR4gngcvMZYAMGo7nhqYbEGGWYI3vKPKhYZqD6mQo96PuvzW9I1IDA==","X-Received":"by 2002:a1c:7412:0:b0:402:ee4b:c96b with SMTP id\n\tp18-20020a1c7412000000b00402ee4bc96bmr10917451wmc.27.1694521830468; \n\tTue, 12 Sep 2023 05:30:30 -0700 (PDT)","To":"Kieran Bingham ,\n\tlibcamera-devel@lists.libcamera.org","In-Reply-To":"<169452148260.2427060.9712252227388588802@ping.linuxembedded.co.uk>","References":"<20230912-gralloc-api-v4-v1-0-0f80402d8e7a@baylibre.com>\n\t<169452148260.2427060.9712252227388588802@ping.linuxembedded.co.uk>","Date":"Tue, 12 Sep 2023 14:30:29 +0200","Message-ID":"<87ledblgka.fsf@baylibre.com>","MIME-Version":"1.0","Content-Type":"text/plain","Content-Transfer-Encoding":"quoted-printable","Subject":"Re: [libcamera-devel] [PATCH 0/4] android: switch over to modern\n\tgralloc API via libui","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":"Mattijs Korpershoek via libcamera-devel\n\t","Reply-To":"Mattijs Korpershoek ","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "}}]