
<html xmlns="http://www.w3.org/1999/xhtml">

<head>

<title></title>

</head>

<body>

<div name="messageBodySection">

<div dir="auto">Yes, I resent it as people told me it was missing tags :)<br />

<br />

So it’s missing more tags now? I admit, I didn’t see the reviewed by tags appear anywhere.<br />

<br />

So to clarify, to get this right, I need to resend again with a v3 and add reviewed by tags?</div>

</div>

<div name="messageSignatureSection"><br />

<div class="matchFont">Chris</div>

</div>

<div name="messageReplySection">On 23 Jun 2020, 19:38 +0200, Umang Jain <email@uajain.com>, wrote:<br />

<blockquote type="cite" style="border-left-color: grey; border-left-width: thin; border-left-style: solid; margin: 5px 5px;padding-left: 10px;">Hi Chris,<br />

<br />

Did you re-send this patch by mistake? It's identical with the patch<br />

you sent on 6/15/2020.<br />

<br />

When you re-send the patch, you need to also mention that the patch is<br />

the second (or third, fourth ...) iteration of the initial patch.<br />

You can do that by adding "-v2" to "git format-patch ..." commandline<br />

to denote that it's the second version of that patch.<br />

<br />

Also, since you got Reviewed-by tags by me and Laurent, you need to collect<br />

them too and add it to your commit message while re-sending them for the<br />

second round of review.<br />

<br />

On 6/23/20 4:27 PM, chris@gregariousmammal.com wrote:<br />

<blockquote type="cite">From: Chris Chinchilla <chris@gregariousmammal.com><br />

<br />

Fix incorrect method in cameradata docs<br />

<br />

Fixes: b581b9576abd ("libcamera: pipeline_handler: Make pipeline-specific data mandatory")<br />

Signed-off-by: Chris Chinchilla <chris@gregariousmammal.com><br />

---<br />

src/libcamera/pipeline_handler.cpp | 2 +-<br />

1 file changed, 1 insertion(+), 1 deletion(-)<br />

<br />

diff --git a/src/libcamera/pipeline_handler.cpp b/src/libcamera/pipeline_handler.cpp<br />

index 14dfba0..15cdc17 100644<br />

--- a/src/libcamera/pipeline_handler.cpp<br />

+++ b/src/libcamera/pipeline_handler.cpp<br />

@@ -46,7 +46,7 @@ LOG_DEFINE_CATEGORY(Pipeline)<br />

*<br />

* Pipeline handlers are expected to extend this base class with platform<br />

* specific implementation, associate instances of the derived classes<br />

- * using the setCameraData() method, and access them at a later time<br />

+ * using the registerCamera() method, and access them at a later time<br />

* with cameraData().<br />

*/<br />

<br /></blockquote>

</blockquote>

</div>

</body>

</html>