[{"id":35576,"web_url":"https://patchwork.libcamera.org/comment/35576/","msgid":"<85ms7ml179.fsf@mzamazal-thinkpadp1gen7.tpbc.csb>","date":"2025-08-26T12:25:14","subject":"Re: [PATCH v2 23/37] libcamera: utils: Move ALIGN_TO from\n\tcamera_metadata.c to utils.h","submitter":{"id":177,"url":"https://patchwork.libcamera.org/api/people/177/","name":"Milan Zamazal","email":"mzamazal@redhat.com"},"content":"Bryan O'Donoghue <bryan.odonoghue@linaro.org> writes:\n\n> ALIGN_TO() is useful to the GPU debayer logic, move to a shared header to\n> facilitate its reuse.\n>\n> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>\n> ---\n>  include/libcamera/base/utils.h         | 3 +++\n>  src/android/metadata/camera_metadata.c | 4 +---\n>  2 files changed, 4 insertions(+), 3 deletions(-)\n>\n> diff --git a/include/libcamera/base/utils.h b/include/libcamera/base/utils.h\n> index f21c6dc016ece00e9842ad17ae4dd036fb5683c2..a0d3ffc27804d27c6b225d7da8c74feb3c261f3d 100644\n> --- a/include/libcamera/base/utils.h\n> +++ b/include/libcamera/base/utils.h\n> @@ -33,6 +33,9 @@\n>  \n>  #endif\n>  \n> +#define ALIGN_TO(val, alignment) \\\n> +    (((uintptr_t)(val) + ((alignment) - 1)) & ~((alignment) - 1))\n\nIs uintptr_t the best type in the extended use?\n\nSince we use the &~(alignment-1) pattern at more places in software ISP,\nwould it be useful to introduce something like ALIGN_UP and ALIGN_DOWN?\n\n> +\n>  namespace libcamera {\n>  \n>  namespace utils {\n> diff --git a/src/android/metadata/camera_metadata.c b/src/android/metadata/camera_metadata.c\n> index b86586a7e6857a0920234ea312eb5181b27c3ac0..4fafb54076a4a4be41a31f5ce3bd8c74bf649f59 100644\n> --- a/src/android/metadata/camera_metadata.c\n> +++ b/src/android/metadata/camera_metadata.c\n> @@ -24,6 +24,7 @@\n>   */\n>  #define ALOGE(...) fprintf(stderr, LOG_TAG __VA_ARGS__)\n>  \n> +#include <libcamera/base/utils.h>\n>  #include <system/camera_metadata.h>\n>  #include <camera_metadata_hidden.h>\n>  \n> @@ -39,9 +40,6 @@\n>  #define NOT_FOUND       (-ENOENT)\n>  #define SN_EVENT_LOG_ID 0x534e4554\n>  \n> -#define ALIGN_TO(val, alignment) \\\n> -    (((uintptr_t)(val) + ((alignment) - 1)) & ~((alignment) - 1))\n> -\n>  /**\n>   * A single metadata entry, storing an array of values of a given type. If the\n>   * array is no larger than 4 bytes in size, it is stored in the data.value[]","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 9378CBD87C\n\tfor <parsemail@patchwork.libcamera.org>;\n\tTue, 26 Aug 2025 12:25:24 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 48BDD692EF;\n\tTue, 26 Aug 2025 14:25:23 +0200 (CEST)","from us-smtp-delivery-124.mimecast.com\n\t(us-smtp-delivery-124.mimecast.com [170.10.133.124])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id CE313692D5\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue, 26 Aug 2025 14:25:20 +0200 (CEST)","from mail-qv1-f71.google.com (mail-qv1-f71.google.com\n\t[209.85.219.71]) by relay.mimecast.com with ESMTP with STARTTLS\n\t(version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id\n\tus-mta-554-ktFpZwEEMtqis1mXe6t2rw-1; Tue, 26 Aug 2025 08:25:17 -0400","by mail-qv1-f71.google.com with SMTP id\n\t6a1803df08f44-70d9a65c355so89735066d6.1\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue, 26 Aug 2025 05:25:17 -0700 (PDT)","from mzamazal-thinkpadp1gen7.tpbc.csb ([85.93.96.130])\n\tby smtp.gmail.com with ESMTPSA id\n\t6a1803df08f44-70da72872bbsm64257746d6.49.2025.08.26.05.25.15\n\t(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n\tTue, 26 Aug 2025 05:25:15 -0700 (PDT)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=redhat.com header.i=@redhat.com\n\theader.b=\"NuPv2A5g\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n\ts=mimecast20190719; t=1756211119;\n\th=from:from:reply-to:subject:subject:date:date:message-id:message-id:\n\tto:to:cc:cc:mime-version:mime-version:content-type:content-type:\n\tin-reply-to:in-reply-to:references:references;\n\tbh=b0+Woyo9TSIWwfYBLubU9QvMDITPBEKCV5PmBJah8tM=;\n\tb=NuPv2A5gWwri7vYEtPQIqsJ+/piFjAxMn1bNeLTFkncDcU6u3woYh/I14qWlThZJ7p42SZ\n\th6E594DJuCEnXBK+ycf55KzdpbZ1eOgYp3XpgjoYO16+J8ZJriQ2mU+zrB2QeXZFYH3Q+Z\n\tNGiGifPqeZJ9p/VLjCNNpOaz6nSyedM=","X-MC-Unique":"ktFpZwEEMtqis1mXe6t2rw-1","X-Mimecast-MFC-AGG-ID":"ktFpZwEEMtqis1mXe6t2rw_1756211117","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20230601; t=1756211117; x=1756815917;\n\th=mime-version:user-agent:message-id:date:references:in-reply-to\n\t:subject:cc:to:from:x-gm-message-state:from:to:cc:subject:date\n\t:message-id:reply-to;\n\tbh=b0+Woyo9TSIWwfYBLubU9QvMDITPBEKCV5PmBJah8tM=;\n\tb=FVhnc4ZALlJPMtZ6TGe6V83mGxnwEtgp4lf1M/Qq9OzKT8KdymIIDQzOucx/BScJPw\n\tk3rWso3Tw5OAECovl5rTPVh82fDYRa1tiy8nsq5i7XfqgtipsES/UrRHK9fYLh822s0v\n\ttaG11kESR2w1+QgcMrCaW4+chVmgwlAD86s2puR1+b3cDo5LhEnyoAxFHxSsWvRvAAku\n\tfpFRnQDS65zv9kgQZ4sbw/6k6nF+aHOFE40rSv0vqObxImd4W5vkNz1D00qbdoLSmG+e\n\tZGd2/B6XG41PtdIW8Y2HOvEfg3U9sJxwD+UFE5PY5HpiARP5cYhwF8ORolkOeJ0Gl/pq\n\tvYsA==","X-Gm-Message-State":"AOJu0YzUf4Zr4H+zi+dyD0Jp20XEkZckkl7Tg50Tpb7NehzoY4KlXlsY\n\toRyLNgMQ+y+HwhUIMNdQ8NaqjrnovlTuElhhCzcUi7og3ajs6Es3+MBa/PVyZ2vDoXjVmHIOGhh\n\tpTGVzvf+misHxqxUbRq67Da3FZj/XwkpNz7ZnqVnjLGe2BR7EsYMEcyb8x3Q0T6YPTn3RY+DtNk\n\tXQwdIX46dCf1mGeRzrNGtvyl++djVF7jp9o4Pjc35xqj71uDY1mf0uxY8ZzM0=","X-Gm-Gg":"ASbGnctUw6PtIKIicFEEOkhuVqOTRpOOd1hYP/yBpwfFwBwIwhlgWG6kVwAurZrfF6t\n\tgHLDhzorxeBf5HOxl06RQ2DhDBlAfEtKhndwJcIBNGbsRetcrlQ4FCcNrJnB0JOqfxuAnM1fq9T\n\tEtoxuJVIZDBaOoVc7BTY+S2AJ1ah/yfftJdUQRI998N0qnMPSWMa7kFVu76gyWzyu4n0v/6AHx6\n\tw01Ie20Fit1JVmlwQtUBZjXfR7SST7vflcXLdwoJpDmbOioRU2C6lUkWkszAFd9OIhaVEmv7Ky0\n\t0v0mIXbDny2Amo+ITBdplMEy7RvCycnMjO1dp3UtMVrExcesR+7H/6tZ//LhKv8K","X-Received":["by 2002:ad4:4ea7:0:b0:707:6cf8:5963 with SMTP id\n\t6a1803df08f44-70dd59c102amr9841096d6.9.1756211116857; \n\tTue, 26 Aug 2025 05:25:16 -0700 (PDT)","by 2002:ad4:4ea7:0:b0:707:6cf8:5963 with SMTP id\n\t6a1803df08f44-70dd59c102amr9840826d6.9.1756211116332; \n\tTue, 26 Aug 2025 05:25:16 -0700 (PDT)"],"X-Google-Smtp-Source":"AGHT+IE4I/XosLIdr8/Bl8SAoSZuEqvY2CqBsg3yleDHDpj7rmVgZgN4q3Oa04wzGcjHWKIM5aqkkg==","From":"Milan Zamazal <mzamazal@redhat.com>","To":"Bryan O'Donoghue <bryan.odonoghue@linaro.org>","Cc":"libcamera-devel@lists.libcamera.org","Subject":"Re: [PATCH v2 23/37] libcamera: utils: Move ALIGN_TO from\n\tcamera_metadata.c to utils.h","In-Reply-To":"<20250824-b4-v0-5-2-gpuisp-v2-a-v2-23-96f4576c814e@linaro.org>\n\t(Bryan O'Donoghue's message of \"Sun, 24 Aug 2025 01:48:35 +0100\")","References":"<20250824-b4-v0-5-2-gpuisp-v2-a-v2-0-96f4576c814e@linaro.org>\n\t<20250824-b4-v0-5-2-gpuisp-v2-a-v2-23-96f4576c814e@linaro.org>","Date":"Tue, 26 Aug 2025 14:25:14 +0200","Message-ID":"<85ms7ml179.fsf@mzamazal-thinkpadp1gen7.tpbc.csb>","User-Agent":"Gnus/5.13 (Gnus v5.13)","MIME-Version":"1.0","X-Mimecast-Spam-Score":"0","X-Mimecast-MFC-PROC-ID":"1SSvDItO8VSO47qJYFcm2F-SWRswE86GUrO_pAo45U0_1756211117","X-Mimecast-Originator":"redhat.com","Content-Type":"text/plain","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>"}},{"id":35990,"web_url":"https://patchwork.libcamera.org/comment/35990/","msgid":"<466ff38e-c529-43cb-b9ef-b1cd696c111b@ideasonboard.com>","date":"2025-09-25T15:16:54","subject":"Re: [PATCH v2 23/37] libcamera: utils: Move ALIGN_TO from\n\tcamera_metadata.c to utils.h","submitter":{"id":216,"url":"https://patchwork.libcamera.org/api/people/216/","name":"Barnabás Pőcze","email":"barnabas.pocze@ideasonboard.com"},"content":"Hi\n\n2025. 08. 24. 2:48 keltezéssel, Bryan O'Donoghue írta:\n> ALIGN_TO() is useful to the GPU debayer logic, move to a shared header to\n> facilitate its reuse.\n> \n> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>\n> ---\n>   include/libcamera/base/utils.h         | 3 +++\n>   src/android/metadata/camera_metadata.c | 4 +---\n>   2 files changed, 4 insertions(+), 3 deletions(-)\n> \n> diff --git a/include/libcamera/base/utils.h b/include/libcamera/base/utils.h\n> index f21c6dc016ece00e9842ad17ae4dd036fb5683c2..a0d3ffc27804d27c6b225d7da8c74feb3c261f3d 100644\n> --- a/include/libcamera/base/utils.h\n> +++ b/include/libcamera/base/utils.h\n> @@ -33,6 +33,9 @@\n>   \n>   #endif\n>   \n> +#define ALIGN_TO(val, alignment) \\\n> +    (((uintptr_t)(val) + ((alignment) - 1)) & ~((alignment) - 1))\n> +\n>   namespace libcamera {\n>   \n>   namespace utils {\n> diff --git a/src/android/metadata/camera_metadata.c b/src/android/metadata/camera_metadata.c\n> index b86586a7e6857a0920234ea312eb5181b27c3ac0..4fafb54076a4a4be41a31f5ce3bd8c74bf649f59 100644\n> --- a/src/android/metadata/camera_metadata.c\n> +++ b/src/android/metadata/camera_metadata.c\n\nThis file is imported from android. I think it would be best to avoid\nmodifications, even at the cost of duplicating the same thing.\n\nThere is already `alignUp` and `alignDown` in `utils.h`. If `unsigned int` is not\nsufficient, I would just make them templates.\n\n\nRegards,\nBarnabás Pőcze\n\n\n> @@ -24,6 +24,7 @@\n>    */\n>   #define ALOGE(...) fprintf(stderr, LOG_TAG __VA_ARGS__)\n>   \n> +#include <libcamera/base/utils.h>\n>   #include <system/camera_metadata.h>\n>   #include <camera_metadata_hidden.h>\n>   \n> @@ -39,9 +40,6 @@\n>   #define NOT_FOUND       (-ENOENT)\n>   #define SN_EVENT_LOG_ID 0x534e4554\n>   \n> -#define ALIGN_TO(val, alignment) \\\n> -    (((uintptr_t)(val) + ((alignment) - 1)) & ~((alignment) - 1))\n> -\n>   /**\n>    * A single metadata entry, storing an array of values of a given type. If the\n>    * array is no larger than 4 bytes in size, it is stored in the data.value[]\n>","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 CBB09C328C\n\tfor <parsemail@patchwork.libcamera.org>;\n\tThu, 25 Sep 2025 15:17:00 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id F291869318;\n\tThu, 25 Sep 2025 17:16:59 +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 580E069318\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tThu, 25 Sep 2025 17:16:58 +0200 (CEST)","from [192.168.33.22] (185.221.140.70.nat.pool.zt.hu\n\t[185.221.140.70])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id AA4041129;\n\tThu, 25 Sep 2025 17:15:32 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"AEnyzRpZ\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1758813333;\n\tbh=RYCmdwa5ZLSdv+ZdadP7sWf7t0fpsKqvhCY9ulAcbqA=;\n\th=Date:Subject:To:References:From:In-Reply-To:From;\n\tb=AEnyzRpZMdxN/F0bqvYrVC8EjYY0akCe7msBND2LP2YGjIEVgmI3WTJxA4NdEkTrH\n\twEJVReVpVI+6R/UJmRHVQU43IgIQEQv27Nv2banPDyShJWYj2GeZB7cHI0cqwpfyqZ\n\tnnGUAuD6gBvCPord1lJEaO/0oMafhJKWlmUiwrjo=","Message-ID":"<466ff38e-c529-43cb-b9ef-b1cd696c111b@ideasonboard.com>","Date":"Thu, 25 Sep 2025 17:16:54 +0200","MIME-Version":"1.0","User-Agent":"Mozilla Thunderbird","Subject":"Re: [PATCH v2 23/37] libcamera: utils: Move ALIGN_TO from\n\tcamera_metadata.c to utils.h","To":"Bryan O'Donoghue <bryan.odonoghue@linaro.org>,\n\tlibcamera-devel@lists.libcamera.org","References":"<20250824-b4-v0-5-2-gpuisp-v2-a-v2-0-96f4576c814e@linaro.org>\n\t<20250824-b4-v0-5-2-gpuisp-v2-a-v2-23-96f4576c814e@linaro.org>","From":"=?utf-8?q?Barnab=C3=A1s_P=C5=91cze?= <barnabas.pocze@ideasonboard.com>","Content-Language":"en-US, hu-HU","In-Reply-To":"<20250824-b4-v0-5-2-gpuisp-v2-a-v2-23-96f4576c814e@linaro.org>","Content-Type":"text/plain; charset=UTF-8; format=flowed","Content-Transfer-Encoding":"8bit","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>"}},{"id":36153,"web_url":"https://patchwork.libcamera.org/comment/36153/","msgid":"<db256ddb-92c4-4f3d-8109-5b187a034d0a@nxsw.ie>","date":"2025-10-06T21:03:00","subject":"Re: [PATCH v2 23/37] libcamera: utils: Move ALIGN_TO from\n\tcamera_metadata.c to utils.h","submitter":{"id":226,"url":"https://patchwork.libcamera.org/api/people/226/","name":"Bryan O'Donoghue","email":"bod.linux@nxsw.ie"},"content":"On 25/09/2025 16:16, Barnabás Pőcze wrote:\n> Hi\n> \n> 2025. 08. 24. 2:48 keltezéssel, Bryan O'Donoghue írta:\n>> ALIGN_TO() is useful to the GPU debayer logic, move to a shared header to\n>> facilitate its reuse.\n>>\n>> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>\n>> ---\n>>    include/libcamera/base/utils.h         | 3 +++\n>>    src/android/metadata/camera_metadata.c | 4 +---\n>>    2 files changed, 4 insertions(+), 3 deletions(-)\n>>\n>> diff --git a/include/libcamera/base/utils.h b/include/libcamera/base/utils.h\n>> index f21c6dc016ece00e9842ad17ae4dd036fb5683c2..a0d3ffc27804d27c6b225d7da8c74feb3c261f3d 100644\n>> --- a/include/libcamera/base/utils.h\n>> +++ b/include/libcamera/base/utils.h\n>> @@ -33,6 +33,9 @@\n>>\n>>    #endif\n>>\n>> +#define ALIGN_TO(val, alignment) \\\n>> +    (((uintptr_t)(val) + ((alignment) - 1)) & ~((alignment) - 1))\n>> +\n>>    namespace libcamera {\n>>\n>>    namespace utils {\n>> diff --git a/src/android/metadata/camera_metadata.c b/src/android/metadata/camera_metadata.c\n>> index b86586a7e6857a0920234ea312eb5181b27c3ac0..4fafb54076a4a4be41a31f5ce3bd8c74bf649f59 100644\n>> --- a/src/android/metadata/camera_metadata.c\n>> +++ b/src/android/metadata/camera_metadata.c\n> \n> This file is imported from android. I think it would be best to avoid\n> modifications, even at the cost of duplicating the same thing.\n> \n> There is already `alignUp` and `alignDown` in `utils.h`. If `unsigned int` is not\n> sufficient, I would just make them templates.\n> \n> \n> Regards,\n> Barnabás Pőcze\n> \n> \n>> @@ -24,6 +24,7 @@\n>>     */\n>>    #define ALOGE(...) fprintf(stderr, LOG_TAG __VA_ARGS__)\n>>\n>> +#include <libcamera/base/utils.h>\n>>    #include <system/camera_metadata.h>\n>>    #include <camera_metadata_hidden.h>\n>>\n>> @@ -39,9 +40,6 @@\n>>    #define NOT_FOUND       (-ENOENT)\n>>    #define SN_EVENT_LOG_ID 0x534e4554\n>>\n>> -#define ALIGN_TO(val, alignment) \\\n>> -    (((uintptr_t)(val) + ((alignment) - 1)) & ~((alignment) - 1))\n>> -\n>>    /**\n>>     * A single metadata entry, storing an array of values of a given type. If the\n>>     * array is no larger than 4 bytes in size, it is stored in the data.value[]\n>>\n> \n\nDropped this patch in favour of your suggested change.\n\n---\nbod","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 51F78C324C\n\tfor <parsemail@patchwork.libcamera.org>;\n\tMon,  6 Oct 2025 21:03:09 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 680C06B5F3;\n\tMon,  6 Oct 2025 23:03:08 +0200 (CEST)","from mail-4323.protonmail.ch (mail-4323.protonmail.ch\n\t[185.70.43.23])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 9DC586936E\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon,  6 Oct 2025 23:03:07 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (2048-bit key;\n\tunprotected) header.d=nxsw.ie header.i=@nxsw.ie header.b=\"ikITckkS\";\n\tdkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxsw.ie;\n\ts=protonmail3; t=1759784586; x=1760043786;\n\tbh=ZbJwsZFlnMx4hCozw+NUKqEeW2zDRfb5OBUacPmgtrA=;\n\th=Date:To:From:Subject:Message-ID:In-Reply-To:References:\n\tFeedback-ID:From:To:Cc:Date:Subject:Reply-To:Feedback-ID:\n\tMessage-ID:BIMI-Selector;\n\tb=ikITckkS/hzAhQwJU7/XUDPzA3wWnOhCaLQlN+PG2JpL+2RxipxOYJ7CBePJf4L1j\n\t2+0WXSz1VKfPkS3ELODEcY2Zgg8L82mU7L7Xo8NDpjrwHP6Y0eS/IBOEzszJ3UnsoA\n\t80vATGui88hZs3q3kqjoSE1Jf6L8XNeNuHOxbxwdeI3rvi9E/aKF1BuPZNHm+veYy7\n\t3H6/KqXb9APNeUPhI7BY8rxC1vXadzEFJaqEoIT4/yyOg/OC6cKWeZTs9uIv2nKax1\n\tXUATD9VU5jOfzPpiD7dD2vYU9fUik6wAsq2dryhMRF1iDw4cVtt3NVXvQQBOSV4Evy\n\toA3NkngFlmdRg==","Date":"Mon, 06 Oct 2025 21:03:00 +0000","To":"=?utf-8?q?Barnab=C3=A1s_P=C5=91cze?= <barnabas.pocze@ideasonboard.com>,\n\tBryan O'Donoghue <bryan.odonoghue@linaro.org>, \n\tlibcamera-devel@lists.libcamera.org","From":"Bryan O'Donoghue <bod.linux@nxsw.ie>","Subject":"Re: [PATCH v2 23/37] libcamera: utils: Move ALIGN_TO from\n\tcamera_metadata.c to utils.h","Message-ID":"<db256ddb-92c4-4f3d-8109-5b187a034d0a@nxsw.ie>","In-Reply-To":"<466ff38e-c529-43cb-b9ef-b1cd696c111b@ideasonboard.com>","References":"<20250824-b4-v0-5-2-gpuisp-v2-a-v2-0-96f4576c814e@linaro.org>\n\t<20250824-b4-v0-5-2-gpuisp-v2-a-v2-23-96f4576c814e@linaro.org>\n\t<466ff38e-c529-43cb-b9ef-b1cd696c111b@ideasonboard.com>","Feedback-ID":"136405006:user:proton","X-Pm-Message-ID":"e1c5c96cb1db530c8d16613183d4e073de72de45","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Transfer-Encoding":"quoted-printable","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>"}}]