[libcamera-devel] [PATCHv2] Simple typo fixes

Kieran Bingham kieran.bingham at ideasonboard.com
Wed Dec 21 14:07:29 CET 2022


Quoting Kieran Bingham (2022-12-21 13:03:45)
> Hi Pavel,
> 
> How are you sending the patch to the list? It's malformed so it isn't
> being picked up as a patch by the patchwork tools.

Or rather - it isn't being correctly picked up as a patch - because it's
mixed between E-Mail content and patch content.

Note how when applied, it has this 'email' content in the patch that
shouldn't be there.

- https://git.uk.ideasonboard.com/kbingham/libcamera/commit/40bb9471e39362591f9e57318f5f7bb63d7fa37c

--
Kieran


> 
> https://patchwork.libcamera.org/patch/18041/
> 
> Are you using git-send-email ? https://git-send-email.io/ helps with the
> configuration of that if you are not.
> 
> --
> Kieran
> 
> Quoting Pavel Machek via libcamera-devel (2022-12-20 22:02:35)
> > Hi!
> > 
> > During reading the code, I found some typos that could be fixed.
> > 
> > Signed-off-by: Pavel Machek <pavel at ucw.cz>
> > Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> > 
> > --
> > v2: rebased to today's tree.
> > 
> > commit f214f8fe259b485af926bb6299d07fc17f1272fc
> > Author: Pavel Machek <pavel at ucw.cz>
> > Date:   Fri Dec 9 18:43:20 2022 +0000
> > 
> >     cleanup: Fix typos.
> > 
> > diff --git a/src/ipa/vimc/vimc.cpp b/src/ipa/vimc/vimc.cpp
> > index 9256bbba..d5fabaf1 100644
> > --- a/src/ipa/vimc/vimc.cpp
> > +++ b/src/ipa/vimc/vimc.cpp
> > @@ -2,7 +2,7 @@
> >  /*
> >   * Copyright (C) 2019, Google Inc.
> >   *
> > - * ipa_vimc.cpp - Vimc Image Processing Algorithm module
> > + * vimc.cpp - Vimc Image Processing Algorithm module
> >   */
> >  #include <libcamera/ipa/vimc_ipa_interface.h>
> >  
> > diff --git a/src/libcamera/pipeline/rkisp1/rkisp1.cpp b/src/libcamera/pipeline/rkisp1/rkisp1.cpp
> > index eb9ad65c..c27b3ef9 100644
> > --- a/src/libcamera/pipeline/rkisp1/rkisp1.cpp
> > +++ b/src/libcamera/pipeline/rkisp1/rkisp1.cpp
> > @@ -220,7 +220,7 @@ RkISP1FrameInfo *RkISP1Frames::create(const RkISP1CameraData *data, Request *req
> >                 }
> >  
> >                 if (pipe_->availableStatBuffers_.empty()) {
> > -                       LOG(RkISP1, Error) << "Statisitc buffer underrun";
> > +                       LOG(RkISP1, Error) << "Statistic buffer underrun";
> >                         return nullptr;
> >                 }
> >  
> > @@ -340,7 +340,7 @@ int RkISP1CameraData::loadIPA(unsigned int hwRevision)
> >  
> >         /*
> >          * The API tuning file is made from the sensor name unless the
> > -        * environment variable overrides it. If
> > +        * environment variable overrides it.
> >          */
> >         std::string ipaTuningFile;
> >         char const *configFromEnv = utils::secure_getenv("LIBCAMERA_RKISP1_TUNING_FILE");
> > 
> > -- 
> > People of Russia, stop Putin before his war on Ukraine escalates.


More information about the libcamera-devel mailing list