Messages in this thread | | | Date | Thu, 12 Oct 2017 09:04:13 -0400 | From | Sean Paul <> | Subject | Re: [PATCH 1/2] drm: vblank: use ktime_t instead of timeval |
| |
On Wed, Oct 11, 2017 at 04:28:39PM -0400, Sean Paul wrote: > On Wed, Oct 11, 2017 at 4:18 PM, Keith Packard <keithp@keithp.com> wrote: > > Sean Paul <seanpaul@chromium.org> writes: > > > >> It looks like perhaps Keith missed one of the comment tweaks that you have > >> below. > >> > >> Keith, perhaps you can rebase your widening patch on this one? > > > > I'm easy; either order works for me just fine. Having the time > > change separated from the sequence change might be nice? > > > > Yeah, that's what I was thinking. I'd normally default to first come > first serve, but since your patch is a superset, rebasing has the nice > feature of teasing out the widening change. > > Would it be easier for you to respin this into your series, or for me > to just apply it to drm-misc-next? >
It seems like we have consensus, and Daniel has offered to pick both of Arnd's patches into -misc-next.
So thanks to Keith for being flexible, and
Reviewed-by: Sean Paul <seanpaul@chromium.org>
Sean
> Sean > > > > -- > > -keith
-- Sean Paul, Software Engineer, Google / Chromium OS
| |