[{"id":4853,"web_url":"https://patchwork.libcamera.org/comment/4853/","msgid":"<20200519140236.GI470768@oden.dyn.berto.se>","date":"2020-05-19T14:02:36","subject":"Re: [libcamera-devel] [PATCH 1/2] libcamera: pipeline: simple:\n\tconverter: Add scaling support","submitter":{"id":5,"url":"https://patchwork.libcamera.org/api/people/5/","name":"Niklas Söderlund","email":"niklas.soderlund@ragnatech.se"},"content":"Hi Laurent,\n\nThanks for your work.\n\nOn 2020-05-19 06:06:23 +0300, Laurent Pinchart wrote:\n> Extend the SimpleConverter to support scaling, with reporting of the\n> minimum and maximum output sizes supported for a given input size.\n> \n> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>\n> ---\n>  src/libcamera/pipeline/simple/converter.cpp | 67 ++++++++++++++++++---\n>  src/libcamera/pipeline/simple/converter.h   |  7 ++-\n>  src/libcamera/pipeline/simple/simple.cpp    |  2 +-\n>  3 files changed, 63 insertions(+), 13 deletions(-)\n> \n> diff --git a/src/libcamera/pipeline/simple/converter.cpp b/src/libcamera/pipeline/simple/converter.cpp\n> index 02443e7f78ed..e5e2f0fddb62 100644\n> --- a/src/libcamera/pipeline/simple/converter.cpp\n> +++ b/src/libcamera/pipeline/simple/converter.cpp\n> @@ -8,10 +8,12 @@\n>  #include \"converter.h\"\n>  \n>  #include <algorithm>\n> +#include <limits.h>\n>  \n>  #include <libcamera/buffer.h>\n>  #include <libcamera/geometry.h>\n>  #include <libcamera/signal.h>\n> +#include <libcamera/stream.h>\n>  \n>  #include \"libcamera/internal/log.h\"\n>  #include \"libcamera/internal/media_device.h\"\n> @@ -92,15 +94,60 @@ std::vector<PixelFormat> SimpleConverter::formats(PixelFormat input)\n>  \treturn pixelFormats;\n>  }\n>  \n> -int SimpleConverter::configure(PixelFormat inputFormat,\n> -\t\t\t       PixelFormat outputFormat, const Size &size)\n> +SizeRange SimpleConverter::sizes(const Size &input)\n> +{\n> +\tif (!m2m_)\n> +\t\treturn {};\n> +\n> +\t/*\n> +\t * Set the size on the input side (V4L2 output) of the converter to\n> +\t * enumerate the scaling capabilities on its output (V4L2 capture).\n> +\t */\n> +\tV4L2DeviceFormat format;\n> +\tformat.fourcc = V4L2PixelFormat();\n> +\tformat.size = input;\n> +\n> +\tint ret = m2m_->output()->setFormat(&format);\n> +\tif (ret < 0) {\n> +\t\tLOG(SimplePipeline, Error)\n> +\t\t\t<< \"Failed to set format: \" << strerror(-ret);\n> +\t\treturn {};\n> +\t}\n> +\n> +\tSizeRange sizes;\n> +\n> +\tformat.size = { 1, 1 };\n> +\tret = m2m_->capture()->setFormat(&format);\n> +\tif (ret < 0) {\n> +\t\tLOG(SimplePipeline, Error)\n> +\t\t\t<< \"Failed to set format: \" << strerror(-ret);\n> +\t\treturn {};\n> +\t}\n> +\n> +\tsizes.min = format.size;\n> +\n> +\tformat.size = { UINT_MAX, UINT_MAX };\n> +\tret = m2m_->capture()->setFormat(&format);\n> +\tif (ret < 0) {\n> +\t\tLOG(SimplePipeline, Error)\n> +\t\t\t<< \"Failed to set format: \" << strerror(-ret);\n> +\t\treturn {};\n> +\t}\n> +\n> +\tsizes.max = format.size;\n> +\n> +\treturn sizes;\n> +}\n> +\n> +int SimpleConverter::configure(PixelFormat inputFormat, const Size &inputSize,\n> +\t\t\t       StreamConfiguration *cfg)\n>  {\n>  \tV4L2DeviceFormat format;\n>  \tint ret;\n>  \n>  \tV4L2PixelFormat videoFormat = m2m_->output()->toV4L2PixelFormat(inputFormat);\n>  \tformat.fourcc = videoFormat;\n> -\tformat.size = size;\n> +\tformat.size = inputSize;\n>  \n>  \tret = m2m_->output()->setFormat(&format);\n>  \tif (ret < 0) {\n> @@ -109,18 +156,16 @@ int SimpleConverter::configure(PixelFormat inputFormat,\n>  \t\treturn ret;\n>  \t}\n>  \n> -\tif (format.fourcc != videoFormat || format.size != size) {\n> +\tif (format.fourcc != videoFormat || format.size != inputSize) {\n>  \t\tLOG(SimplePipeline, Error)\n>  \t\t\t<< \"Input format not supported\";\n>  \t\treturn -EINVAL;\n>  \t}\n>  \n> -\t/*\n> -\t * Set the pixel format on the output, the size is identical to the\n> -\t * input as we don't support scaling.\n> -\t */\n> -\tvideoFormat = m2m_->capture()->toV4L2PixelFormat(outputFormat);\n> +\t/* Set the pixel format and size on the output. */\n> +\tvideoFormat = m2m_->capture()->toV4L2PixelFormat(cfg->pixelFormat);\n>  \tformat.fourcc = videoFormat;\n> +\tformat.size = cfg->size;\n>  \n>  \tret = m2m_->capture()->setFormat(&format);\n>  \tif (ret < 0) {\n> @@ -129,12 +174,14 @@ int SimpleConverter::configure(PixelFormat inputFormat,\n>  \t\treturn ret;\n>  \t}\n>  \n> -\tif (format.fourcc != videoFormat || format.size != size) {\n> +\tif (format.fourcc != videoFormat || format.size != cfg->size) {\n>  \t\tLOG(SimplePipeline, Error)\n>  \t\t\t<< \"Output format not supported\";\n>  \t\treturn -EINVAL;\n>  \t}\n>  \n> +\tcfg->stride = format.planes[0].bpl;\n\nThis seems to be an unrelated change, but nice that it's fixed ;-)\n\nReviewed-by: Niklas Söderlund <niklas.soderlund@ragnatech.se>\n\n> +\n>  \treturn 0;\n>  }\n>  \n> diff --git a/src/libcamera/pipeline/simple/converter.h b/src/libcamera/pipeline/simple/converter.h\n> index a33071fa8578..1f770eb844b5 100644\n> --- a/src/libcamera/pipeline/simple/converter.h\n> +++ b/src/libcamera/pipeline/simple/converter.h\n> @@ -20,6 +20,8 @@ namespace libcamera {\n>  class FrameBuffer;\n>  class MediaDevice;\n>  struct Size;\n> +class SizeRange;\n> +struct StreamConfiguration;\n>  class V4L2M2MDevice;\n>  \n>  class SimpleConverter\n> @@ -32,9 +34,10 @@ public:\n>  \tvoid close();\n>  \n>  \tstd::vector<PixelFormat> formats(PixelFormat input);\n> +\tSizeRange sizes(const Size &input);\n>  \n> -\tint configure(PixelFormat inputFormat, PixelFormat outputFormat,\n> -\t\t      const Size &size);\n> +\tint configure(PixelFormat inputFormat, const Size &inputSize,\n> +\t\t      StreamConfiguration *cfg);\n>  \tint exportBuffers(unsigned int count,\n>  \t\t\t  std::vector<std::unique_ptr<FrameBuffer>> *buffers);\n>  \n> diff --git a/src/libcamera/pipeline/simple/simple.cpp b/src/libcamera/pipeline/simple/simple.cpp\n> index 2565190082c8..b21ad82e1a05 100644\n> --- a/src/libcamera/pipeline/simple/simple.cpp\n> +++ b/src/libcamera/pipeline/simple/simple.cpp\n> @@ -554,7 +554,7 @@ int SimplePipelineHandler::configure(Camera *camera, CameraConfiguration *c)\n>  \n>  \tif (useConverter_) {\n>  \t\tint ret = converter_->configure(pipeConfig.pixelFormat,\n> -\t\t\t\t\t\tcfg.pixelFormat, cfg.size);\n> +\t\t\t\t\t\tcfg.size, &cfg);\n>  \t\tif (ret < 0) {\n>  \t\t\tLOG(SimplePipeline, Error)\n>  \t\t\t\t<< \"Unable to configure converter\";\n> -- \n> Regards,\n> \n> Laurent Pinchart\n> \n> _______________________________________________\n> libcamera-devel mailing list\n> libcamera-devel@lists.libcamera.org\n> https://lists.libcamera.org/listinfo/libcamera-devel","headers":{"Return-Path":"<niklas.soderlund@ragnatech.se>","Received":["from mail-lf1-x144.google.com (mail-lf1-x144.google.com\n\t[IPv6:2a00:1450:4864:20::144])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id E6D54603D9\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue, 19 May 2020 16:02:38 +0200 (CEST)","by mail-lf1-x144.google.com with SMTP id w15so3979490lfe.11\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue, 19 May 2020 07:02:38 -0700 (PDT)","from localhost (h-209-203.A463.priv.bahnhof.se. [155.4.209.203])\n\tby smtp.gmail.com with ESMTPSA id\n\tu16sm3252235ljk.37.2020.05.19.07.02.36\n\t(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n\tTue, 19 May 2020 07:02:37 -0700 (PDT)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (2048-bit key; \n\tunprotected)\n\theader.d=ragnatech-se.20150623.gappssmtp.com\n\theader.i=@ragnatech-se.20150623.gappssmtp.com header.b=\"hBVLG9CU\"; \n\tdkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=ragnatech-se.20150623.gappssmtp.com; s=20150623;\n\th=date:from:to:cc:subject:message-id:references:mime-version\n\t:content-disposition:content-transfer-encoding:in-reply-to;\n\tbh=bb6pmP/q+6k+MW8lW36Q9A6nRKuu3u1uPDd93j9QsCs=;\n\tb=hBVLG9CUsusnlsiwAolpa/NegBklmCwN8IvS1bqrpAr7NkYZr6GLZcJ1K2i4qX0uDh\n\tIBC+nqB/ltVLBGyLbGnGVtcaYbqeBuASgR+aJVBc9K4kJrWh6w3xoGZPziCCjBsjxwYL\n\tVFEk77ZLctp4ALUKCVxwB+zp/bWH0R+z3B6MOPvxIOtRzIBdhDnkUQ4jRHJhMckJz6H6\n\tVJ10iu2AUk+iAybikHToHYZBzj3mjmY3KWUA6BaTk9LtKYNRgilxOtcsRnWkJMQGVvKv\n\tli6BuCHS2ndvBMjzXn2ZZq44BssxpXKzEICNQTXVjCUSrxrKeAuI1vFl58u9jls0FN4k\n\tU5jg==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:content-transfer-encoding\n\t:in-reply-to;\n\tbh=bb6pmP/q+6k+MW8lW36Q9A6nRKuu3u1uPDd93j9QsCs=;\n\tb=OJgI00GirKkwR1Ifczs8UBKOOTgCQE4y7C8RzukmaGmR1GU9c6Br8aJxEGqu5EHIPY\n\txPwYY1dAhoxHe5Snp81iFCfO3kYWvFI3yykumKcq7RGbFj0D9HmFdqPIAght7rDJHYSJ\n\tP8GGK8yC1M6hkoQtyf5ws0hhq8XpRraKPzhefann+SqmOaq+RuCkl9uyWaTARXcADn/i\n\tHq3V1xd5Jz7LQtCCTudZpJgPcaOXlS1f4VdMYWW91zEwLmT97eF+bVeRCm/rK9Zb7HSz\n\tXadMwEPS6kkoemBeLQJH9kxao/vaRCknvr/rbmaRjhIlmxL76wGTasCMNhjYO6M+jJmo\n\tZ9mA==","X-Gm-Message-State":"AOAM531u8sOalkF9Dt7/ejKPnD4Y4gdx9IBt/NJzsCbwZ0jg/TQYC3gA\n\tBU1XaOiX7LJ41P8KXw/75oT7dA==","X-Google-Smtp-Source":"ABdhPJwRadNPlOfCw2RGvGkRD1UkkHJGobBqmqdCiASlanfO43ibruTWwqYl7gCx4N0lVW83jKbPqg==","X-Received":"by 2002:a19:6b13:: with SMTP id\n\td19mr15810708lfa.126.1589896958211; \n\tTue, 19 May 2020 07:02:38 -0700 (PDT)","Date":"Tue, 19 May 2020 16:02:36 +0200","From":"Niklas =?iso-8859-1?q?S=F6derlund?= <niklas.soderlund@ragnatech.se>","To":"Laurent Pinchart <laurent.pinchart@ideasonboard.com>","Cc":"libcamera-devel@lists.libcamera.org","Message-ID":"<20200519140236.GI470768@oden.dyn.berto.se>","References":"<20200519030624.15985-1-laurent.pinchart@ideasonboard.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=iso-8859-1","Content-Disposition":"inline","Content-Transfer-Encoding":"8bit","In-Reply-To":"<20200519030624.15985-1-laurent.pinchart@ideasonboard.com>","Subject":"Re: [libcamera-devel] [PATCH 1/2] libcamera: pipeline: simple:\n\tconverter: Add scaling support","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>","X-List-Received-Date":"Tue, 19 May 2020 14:02:39 -0000"}},{"id":4855,"web_url":"https://patchwork.libcamera.org/comment/4855/","msgid":"<20200519140310.GC3820@pendragon.ideasonboard.com>","date":"2020-05-19T14:03:10","subject":"Re: [libcamera-devel] [PATCH 1/2] libcamera: pipeline: simple:\n\tconverter: Add scaling support","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"Hi Niklas,\n\nOn Tue, May 19, 2020 at 04:02:36PM +0200, Niklas Söderlund wrote:\n> On 2020-05-19 06:06:23 +0300, Laurent Pinchart wrote:\n> > Extend the SimpleConverter to support scaling, with reporting of the\n> > minimum and maximum output sizes supported for a given input size.\n> > \n> > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>\n> > ---\n> >  src/libcamera/pipeline/simple/converter.cpp | 67 ++++++++++++++++++---\n> >  src/libcamera/pipeline/simple/converter.h   |  7 ++-\n> >  src/libcamera/pipeline/simple/simple.cpp    |  2 +-\n> >  3 files changed, 63 insertions(+), 13 deletions(-)\n> > \n> > diff --git a/src/libcamera/pipeline/simple/converter.cpp b/src/libcamera/pipeline/simple/converter.cpp\n> > index 02443e7f78ed..e5e2f0fddb62 100644\n> > --- a/src/libcamera/pipeline/simple/converter.cpp\n> > +++ b/src/libcamera/pipeline/simple/converter.cpp\n> > @@ -8,10 +8,12 @@\n> >  #include \"converter.h\"\n> >  \n> >  #include <algorithm>\n> > +#include <limits.h>\n> >  \n> >  #include <libcamera/buffer.h>\n> >  #include <libcamera/geometry.h>\n> >  #include <libcamera/signal.h>\n> > +#include <libcamera/stream.h>\n> >  \n> >  #include \"libcamera/internal/log.h\"\n> >  #include \"libcamera/internal/media_device.h\"\n> > @@ -92,15 +94,60 @@ std::vector<PixelFormat> SimpleConverter::formats(PixelFormat input)\n> >  \treturn pixelFormats;\n> >  }\n> >  \n> > -int SimpleConverter::configure(PixelFormat inputFormat,\n> > -\t\t\t       PixelFormat outputFormat, const Size &size)\n> > +SizeRange SimpleConverter::sizes(const Size &input)\n> > +{\n> > +\tif (!m2m_)\n> > +\t\treturn {};\n> > +\n> > +\t/*\n> > +\t * Set the size on the input side (V4L2 output) of the converter to\n> > +\t * enumerate the scaling capabilities on its output (V4L2 capture).\n> > +\t */\n> > +\tV4L2DeviceFormat format;\n> > +\tformat.fourcc = V4L2PixelFormat();\n> > +\tformat.size = input;\n> > +\n> > +\tint ret = m2m_->output()->setFormat(&format);\n> > +\tif (ret < 0) {\n> > +\t\tLOG(SimplePipeline, Error)\n> > +\t\t\t<< \"Failed to set format: \" << strerror(-ret);\n> > +\t\treturn {};\n> > +\t}\n> > +\n> > +\tSizeRange sizes;\n> > +\n> > +\tformat.size = { 1, 1 };\n> > +\tret = m2m_->capture()->setFormat(&format);\n> > +\tif (ret < 0) {\n> > +\t\tLOG(SimplePipeline, Error)\n> > +\t\t\t<< \"Failed to set format: \" << strerror(-ret);\n> > +\t\treturn {};\n> > +\t}\n> > +\n> > +\tsizes.min = format.size;\n> > +\n> > +\tformat.size = { UINT_MAX, UINT_MAX };\n> > +\tret = m2m_->capture()->setFormat(&format);\n> > +\tif (ret < 0) {\n> > +\t\tLOG(SimplePipeline, Error)\n> > +\t\t\t<< \"Failed to set format: \" << strerror(-ret);\n> > +\t\treturn {};\n> > +\t}\n> > +\n> > +\tsizes.max = format.size;\n> > +\n> > +\treturn sizes;\n> > +}\n> > +\n> > +int SimpleConverter::configure(PixelFormat inputFormat, const Size &inputSize,\n> > +\t\t\t       StreamConfiguration *cfg)\n> >  {\n> >  \tV4L2DeviceFormat format;\n> >  \tint ret;\n> >  \n> >  \tV4L2PixelFormat videoFormat = m2m_->output()->toV4L2PixelFormat(inputFormat);\n> >  \tformat.fourcc = videoFormat;\n> > -\tformat.size = size;\n> > +\tformat.size = inputSize;\n> >  \n> >  \tret = m2m_->output()->setFormat(&format);\n> >  \tif (ret < 0) {\n> > @@ -109,18 +156,16 @@ int SimpleConverter::configure(PixelFormat inputFormat,\n> >  \t\treturn ret;\n> >  \t}\n> >  \n> > -\tif (format.fourcc != videoFormat || format.size != size) {\n> > +\tif (format.fourcc != videoFormat || format.size != inputSize) {\n> >  \t\tLOG(SimplePipeline, Error)\n> >  \t\t\t<< \"Input format not supported\";\n> >  \t\treturn -EINVAL;\n> >  \t}\n> >  \n> > -\t/*\n> > -\t * Set the pixel format on the output, the size is identical to the\n> > -\t * input as we don't support scaling.\n> > -\t */\n> > -\tvideoFormat = m2m_->capture()->toV4L2PixelFormat(outputFormat);\n> > +\t/* Set the pixel format and size on the output. */\n> > +\tvideoFormat = m2m_->capture()->toV4L2PixelFormat(cfg->pixelFormat);\n> >  \tformat.fourcc = videoFormat;\n> > +\tformat.size = cfg->size;\n> >  \n> >  \tret = m2m_->capture()->setFormat(&format);\n> >  \tif (ret < 0) {\n> > @@ -129,12 +174,14 @@ int SimpleConverter::configure(PixelFormat inputFormat,\n> >  \t\treturn ret;\n> >  \t}\n> >  \n> > -\tif (format.fourcc != videoFormat || format.size != size) {\n> > +\tif (format.fourcc != videoFormat || format.size != cfg->size) {\n> >  \t\tLOG(SimplePipeline, Error)\n> >  \t\t\t<< \"Output format not supported\";\n> >  \t\treturn -EINVAL;\n> >  \t}\n> >  \n> > +\tcfg->stride = format.planes[0].bpl;\n> \n> This seems to be an unrelated change, but nice that it's fixed ;-)\n\nIndeed. I'll split it out to a separate patch.\n\n> Reviewed-by: Niklas Söderlund <niklas.soderlund@ragnatech.se>\n> \n> > +\n> >  \treturn 0;\n> >  }\n> >  \n> > diff --git a/src/libcamera/pipeline/simple/converter.h b/src/libcamera/pipeline/simple/converter.h\n> > index a33071fa8578..1f770eb844b5 100644\n> > --- a/src/libcamera/pipeline/simple/converter.h\n> > +++ b/src/libcamera/pipeline/simple/converter.h\n> > @@ -20,6 +20,8 @@ namespace libcamera {\n> >  class FrameBuffer;\n> >  class MediaDevice;\n> >  struct Size;\n> > +class SizeRange;\n> > +struct StreamConfiguration;\n> >  class V4L2M2MDevice;\n> >  \n> >  class SimpleConverter\n> > @@ -32,9 +34,10 @@ public:\n> >  \tvoid close();\n> >  \n> >  \tstd::vector<PixelFormat> formats(PixelFormat input);\n> > +\tSizeRange sizes(const Size &input);\n> >  \n> > -\tint configure(PixelFormat inputFormat, PixelFormat outputFormat,\n> > -\t\t      const Size &size);\n> > +\tint configure(PixelFormat inputFormat, const Size &inputSize,\n> > +\t\t      StreamConfiguration *cfg);\n> >  \tint exportBuffers(unsigned int count,\n> >  \t\t\t  std::vector<std::unique_ptr<FrameBuffer>> *buffers);\n> >  \n> > diff --git a/src/libcamera/pipeline/simple/simple.cpp b/src/libcamera/pipeline/simple/simple.cpp\n> > index 2565190082c8..b21ad82e1a05 100644\n> > --- a/src/libcamera/pipeline/simple/simple.cpp\n> > +++ b/src/libcamera/pipeline/simple/simple.cpp\n> > @@ -554,7 +554,7 @@ int SimplePipelineHandler::configure(Camera *camera, CameraConfiguration *c)\n> >  \n> >  \tif (useConverter_) {\n> >  \t\tint ret = converter_->configure(pipeConfig.pixelFormat,\n> > -\t\t\t\t\t\tcfg.pixelFormat, cfg.size);\n> > +\t\t\t\t\t\tcfg.size, &cfg);\n> >  \t\tif (ret < 0) {\n> >  \t\t\tLOG(SimplePipeline, Error)\n> >  \t\t\t\t<< \"Unable to configure converter\";","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 D5BDD603D9\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue, 19 May 2020 16:03:36 +0200 (CEST)","from pendragon.ideasonboard.com (81-175-216-236.bb.dnainternet.fi\n\t[81.175.216.236])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 26BF930C;\n\tTue, 19 May 2020 16:03:36 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key; \n\tunprotected) header.d=ideasonboard.com\n\theader.i=@ideasonboard.com\n\theader.b=\"DGNNCheA\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1589897016;\n\tbh=CdQDV9+QRFUbB/Ys0DdLRv+3HXZ56XRcfXGPvjYBTgc=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=DGNNCheA/tkig81zg1IJRgLxGyiH7dTGgVc6f81HzApwafCEfpdykSCXNl4ebE2BA\n\t0iovbbIFmXeBsumh3TCp08trk8W+V3sU41fuGyVIJpNU1zx+kGNdCrV6WIuVU68WY1\n\tddha23TIeXLkuK3cD4D0lajndRkC9bM3QrLHDGB8=","Date":"Tue, 19 May 2020 17:03:10 +0300","From":"Laurent Pinchart <laurent.pinchart@ideasonboard.com>","To":"Niklas =?utf-8?q?S=C3=B6derlund?= <niklas.soderlund@ragnatech.se>","Cc":"libcamera-devel@lists.libcamera.org","Message-ID":"<20200519140310.GC3820@pendragon.ideasonboard.com>","References":"<20200519030624.15985-1-laurent.pinchart@ideasonboard.com>\n\t<20200519140236.GI470768@oden.dyn.berto.se>","MIME-Version":"1.0","Content-Type":"text/plain; charset=utf-8","Content-Disposition":"inline","Content-Transfer-Encoding":"8bit","In-Reply-To":"<20200519140236.GI470768@oden.dyn.berto.se>","Subject":"Re: [libcamera-devel] [PATCH 1/2] libcamera: pipeline: simple:\n\tconverter: Add scaling support","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>","X-List-Received-Date":"Tue, 19 May 2020 14:03:37 -0000"}}]