[libcamera-devel] [PATCH 4/7] qcam: viewfinder_gl: Rename YUV.vert to identity.vert
Niklas Söderlund
niklas.soderlund at ragnatech.se
Fri Nov 6 01:47:44 CET 2020
Hi Laurent,
Thanks for your work.
On 2020-11-03 23:02:11 +0000, Kieran Bingham wrote:
>
>
> On 03/11/2020 18:32, Andrey Konovalov wrote:
> > Hi Laurent,
> >
> > Thank you for the patch!
> >
> > On 03.11.2020 18:50, Laurent Pinchart wrote:
> >> In preparation for RGB formats support, rename the identity vertex
> >> shader from YUV.vert to identify.vert.
> >
> > - s/identify/identity in the above line
> >
> > Yes, identity is a better name for this shader.
> >
> > Reviewed-by: Andrey Konovalov <andrey.konovalov at linaro.org>
>
> and then
> Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
and then and then
Reviewed-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>
>
> >
> > Thanks,
> > Andrey
> >
> >> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> >> ---
> >> src/qcam/assets/shader/{YUV.vert => identity.vert} | 2 +-
> >> src/qcam/assets/shader/shaders.qrc | 2 +-
> >> src/qcam/viewfinder_gl.cpp | 2 +-
> >> 3 files changed, 3 insertions(+), 3 deletions(-)
> >> rename src/qcam/assets/shader/{YUV.vert => identity.vert} (77%)
> >>
> >> diff --git a/src/qcam/assets/shader/YUV.vert
> >> b/src/qcam/assets/shader/identity.vert
> >> similarity index 77%
> >> rename from src/qcam/assets/shader/YUV.vert
> >> rename to src/qcam/assets/shader/identity.vert
> >> index f38e8045db89..6d6f7551017e 100644
> >> --- a/src/qcam/assets/shader/YUV.vert
> >> +++ b/src/qcam/assets/shader/identity.vert
> >> @@ -2,7 +2,7 @@
> >> /*
> >> * Copyright (C) 2020, Linaro
> >> *
> >> - * YUV.vert - Vertex shader for YUV to RGB conversion
> >> + * identity.vert - Identity vertex shader for pixel format conversion
> >> */
> >> attribute vec4 vertexIn;
> >> diff --git a/src/qcam/assets/shader/shaders.qrc
> >> b/src/qcam/assets/shader/shaders.qrc
> >> index 857ed9fd5cde..863109146281 100644
> >> --- a/src/qcam/assets/shader/shaders.qrc
> >> +++ b/src/qcam/assets/shader/shaders.qrc
> >> @@ -1,9 +1,9 @@
> >> <!-- SPDX-License-Identifier: LGPL-2.1-or-later -->
> >> <!DOCTYPE RCC><RCC version="1.0">
> >> <qresource>
> >> - <file>YUV.vert</file>
> >> <file>YUV_2_planes.frag</file>
> >> <file>YUV_3_planes.frag</file>
> >> <file>YUV_packed.frag</file>
> >> + <file>identity.vert</file>
> >> </qresource>
> >> </RCC>
> >> diff --git a/src/qcam/viewfinder_gl.cpp b/src/qcam/viewfinder_gl.cpp
> >> index dc35d4fd8505..110f0fa673df 100644
> >> --- a/src/qcam/viewfinder_gl.cpp
> >> +++ b/src/qcam/viewfinder_gl.cpp
> >> @@ -191,7 +191,7 @@ bool ViewFinderGL::createVertexShader()
> >> vertexShader_ =
> >> std::make_unique<QOpenGLShader>(QOpenGLShader::Vertex, this);
> >> /* Compile the vertex shader */
> >> - if (!vertexShader_->compileSourceFile(":YUV.vert")) {
> >> + if (!vertexShader_->compileSourceFile(":identity.vert")) {
> >> qWarning() << "[ViewFinderGL]:" << vertexShader_->log();
> >> return false;
> >> }
> >>
> > _______________________________________________
> > libcamera-devel mailing list
> > libcamera-devel at lists.libcamera.org
> > https://lists.libcamera.org/listinfo/libcamera-devel
>
> --
> Regards
> --
> Kieran
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel
--
Regards,
Niklas Söderlund
More information about the libcamera-devel
mailing list