[libcamera-devel] rkisp1: Fix spelling
Kieran Bingham
kieran.bingham at ideasonboard.com
Wed Jul 1 11:43:43 CEST 2020
Hi Andrej
On 30/06/2020 15:35, Andrej Shadura wrote:
> On Tue, 30 Jun 2020 at 16:31, Andrej Shadura <andrewsh at debian.org> wrote:
>> On Tue, 30 Jun 2020 at 16:22, Kieran Bingham
>> <kieran.bingham at ideasonboard.com> wrote:
>>> On 30/06/2020 14:58, Andrej Shadura wrote:
>>>> Signed-off-by: Andrej Shadura <andrewsh at debian.org>
>
>>> We require a Signed-off-by: from the Author, which is marked as
>>> umlaeute at debian.org in the header here.
>
>>> Have you authored this patch ? Or are you forwarding on from someone else?
>>
>> IOhannes and I are co-maintainers of libcamera in Debian. He wrote
Ahh excellent, welcome to the project, and thanks for your help.
>> this patch and I’m forwarding it for him. He didn’t leave a sign-off,
>> but since this patch is trivial and arguably non-copyrightable, I
Agreed in this case, it's trivial. It's just the missing SoB was
noticable ;)
>> understand this should not be a problem (I can "re-author" it myself
>> or so can you). In fact, I wrote the patch description in its
>> entirely, so in a way I wrote more of the patch than he did :)
>
> Or I could simply cc him and let him provide the missing sign-off :)
Either option is good for me.
Just a reply here will do in either case and we can update it while
applying :-)
>
>>> For the actual content of the patch:
>>>
>>> Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
>>>
>>>> ---
>>>> src/ipa/rkisp1/rkisp1.cpp | 2 +-
>>>> src/libcamera/pipeline/rkisp1/rkisp1.cpp | 2 +-
>>>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/src/ipa/rkisp1/rkisp1.cpp b/src/ipa/rkisp1/rkisp1.cpp
>>>> index fbdc908..bfd76cf 100644
>>>> --- a/src/ipa/rkisp1/rkisp1.cpp
>>>> +++ b/src/ipa/rkisp1/rkisp1.cpp
>>>> @@ -176,7 +176,7 @@ void IPARkISP1::processEvent(const IPAOperationData &event)
>>>> break;
>>>> }
>>>> default:
>>>> - LOG(IPARkISP1, Error) << "Unkown event " << event.operation;
>>>> + LOG(IPARkISP1, Error) << "Unknown event " << event.operation;
>>>> break;
>>>> }
>>>> }
>>>> diff --git a/src/libcamera/pipeline/rkisp1/rkisp1.cpp b/src/libcamera/pipeline/rkisp1/rkisp1.cpp
>>>> index 3c01821..3c3f3f3 100644
>>>> --- a/src/libcamera/pipeline/rkisp1/rkisp1.cpp
>>>> +++ b/src/libcamera/pipeline/rkisp1/rkisp1.cpp
>>>> @@ -429,7 +429,7 @@ void RkISP1CameraData::queueFrameAction(unsigned int frame,
>>>> metadataReady(frame, action.controls[0]);
>>>> break;
>>>> default:
>>>> - LOG(RkISP1, Error) << "Unkown action " << action.operation;
>>>> + LOG(RkISP1, Error) << "Unknown action " << action.operation;
>>>> break;
>>>> }
>>>> }
>
--
Regards
--
Kieran
More information about the libcamera-devel
mailing list