Hi,
On Tue, 2018-07-10 at 10:17 +0200, Hans Verkuil wrote:
> On 10/07/18 10:13, Paul Kocialkowski wrote:
> > Hi,
> >
> > On Tue, 2018-07-10 at 10:07 +0200, Geert Uytterhoeven wrote:
> > > On Tue, Jul 10, 2018 at 10:02 AM Paul Kocialkowski
> > > <paul.kocialkowski@xxxxxxxxxxx> wrote:
> > > > [PATCH v5 02/22] fixup! v4l2-ctrls: add v4l2_ctrl_request_hdl_find/put/ctrl_find functions
> > >
> > > git rebase -i ;-)
> >
> > Although I should have mentionned it (and did not), this is totally
> > intentional! The first patch (from Hans Verkuil) requires said fixup to
> > work properly. I didn't want to squash that change into the commit to
> > make the diff obvious.
>
> Just squash the two for the next version you post.
That works for me! I must admit I was rather unsure this was a sensible
way to do things.
> > Ultimately, this framework patch is not really part of the series but is
> > one of its underlying requirements, that should be merged separately (as
> > part of the requests API series).
>
> There is a good chance that this patch will go in via your series anyway
> since it is not needed by vivid or vim2m.
Alright, I will keep it around then.
Cheers,
Paul
> Regards,
>
> Hans
>
> >
> > I hope this clears up some of the confusion about this patch :)
> >
> > Cheers!
> >
> > > Gr{oetje,eeting}s,
> > >
> > > Geert
> > >
>
>
--
Paul Kocialkowski, Bootlin (formerly Free Electrons)
Embedded Linux and kernel engineering
https://bootlin.comAttachment:
signature.asc
Description: This is a digitally signed message part
