<div dir="ltr"><div dir="ltr">Hi all,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 8 Mar 2023 at 08:24, Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com">laurent.pinchart@ideasonboard.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, Mar 07, 2023 at 10:24:28PM +0000, Kieran Bingham wrote:<br>
> Hi Naush,<br>
> <br>
> Quoting Laurent Pinchart (2023-03-07 17:10:22)<br>
> > Hi Naush,<br>
> > <br>
> > Thanks for the pull request.<br>
> > <br>
> > In the future, could you try to follow the kernel convention for the<br>
> > subject line ? It should start with a [GIT PULL] tag, and then have a<br>
> > short one-line description of the pull request. This one could be, for<br>
> > instance<br>
> > <br>
> > [GIT PULL] Raspberry Pi timeout handling improvements & other misc changes<br>
> <br>
> Thank you for trying this as new way to handle your integrations!<br>
> <br>
> Running through the integration scripts on my side failed:<br>
> <br>
> Missing committer Signed-off-by in commit f194529f9ba35421b66fee5e5bfcb2b8a148bbbf<br>
> Missing committer Signed-off-by in commit 95e7b5415aadccf1a8d706c3460e114553a49e06<br>
> Found 2 errors in refs/heads/master, not pushing<br>
> <br>
> Which was due to our pre-merge validation scripts noting that the<br>
> patches you cherry-picked from David were not Signed-off by the<br>
> 'committer' (now you).<br>
> <br>
> To be able to merge these, I've run git rebase --signoff, which has<br>
> recommitted these with my Sign-off. I figured I could have asked you to<br>
> resubmit a new pull request with your SoB - but it didn't seem worth the<br>
> extra cycle at this time.<br>
<br>
Just to make it clear, Naush, could you make sure, in future pull<br>
request, that all commits that list you as the committer have your SoB ?<br>
This includes all commits that you cherry-pick, apply from an mbox file,<br>
or rebase. You can see the committer with `git log --pretty=fuller`.<br>
<br>
You can also use the pre-push hook from utils/hooks/ (just copy it to<br>
.git/hooks/) to perform validity checks when pushing a branch. By<br>
default the script only checks branches named `master` or<br>
`integration/*`, but it's easy to modify it to accommodate your<br>
workflow.<br></blockquote><div><br></div><div>Sure, I'll make sure to do that going forward.  This git request-pull stuff is<br>all new to me.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> With that, they passed my integration tests, and are now merged.<br></blockquote><div><br></div><div>Excellent, thanks!!</div><div><br></div><div>Naush</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> <br>
> > On Tue, Mar 07, 2023 at 03:58:20PM +0000, Naushir Patuck via libcamera-devel wrote:<br>
> > > The following changes since commit cde9293cf9899b0fc4ce9cf89dd29f26be77f35c:<br>
> > > <br>
> > >   ipa: rkisp1: lsc: Fix integer division error (2023-03-06 18:12:10 +0100)<br>
> > > <br>
> > > are available in the Git repository at:<br>
> > > <br>
> > >   <a href="https://github.com/naushir/libcamera.git" rel="noreferrer" target="_blank">https://github.com/naushir/libcamera.git</a><br>
> > > <br>
> > > for you to fetch changes up to f194529f9ba35421b66fee5e5bfcb2b8a148bbbf:<br>
> > > <br>
> > >   ipa: raspberrypi: imx296: Minor tuning updates (2023-03-07 15:43:08 +0000)<br>
> > > <br>
> > > ----------------------------------------------------------------<br>
> > > David Plowman (2):<br>
> > >       ipa: raspberrypi: agc: Fix overflow in Y value calculation<br>
> > >       ipa: raspberrypi: imx296: Minor tuning updates<br>
> > > <br>
> > > Naushir Patuck (4):<br>
> > >       pipeline: ipa: raspberrypi: Change Unicam timeout handling<br>
> > >       ipa: raspberrypi: Better heuristics for calculating Unicam timeout<br>
> > >       pipeline: raspberrypi: Add a Unicam timeout override config options<br>
> > >       pipeline: raspberrypi: Iterate over all Unicam instances in match()<br>
> > > <br>
> > >  include/libcamera/ipa/raspberrypi.mojom              |   2 +-<br>
> > >  src/ipa/raspberrypi/controller/rpi/agc.cpp           |   2 +-<br>
> > >  src/ipa/raspberrypi/data/imx296.json                 | 180 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------------<br>
> > >  src/ipa/raspberrypi/data/imx296_mono.json            |  12 ++++++++--<br>
> > >  src/ipa/raspberrypi/raspberrypi.cpp                  |  45 +++++++++++++++++++++++++++++++++++---<br>
> > >  src/libcamera/pipeline/raspberrypi/data/example.yaml |  11 +++++++++-<br>
> > >  src/libcamera/pipeline/raspberrypi/raspberrypi.cpp   | 105 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-----------------------------<br>
> > >  7 files changed, 282 insertions(+), 75 deletions(-)<br>
<br>
-- <br>
Regards,<br>
<br>
Laurent Pinchart<br>
</blockquote></div></div>