Re: [PATCH 1/2] drm: vblank: use ktime_t instead of timeval

From: Sean Paul
Date: Thu Oct 12 2017 - 09:04:19 EST


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@xxxxxxxxxx> wrote:
> > Sean Paul <seanpaul@xxxxxxxxxxxx> 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@xxxxxxxxxxxx>

Sean


> Sean
>
>
> > --
> > -keith

--
Sean Paul, Software Engineer, Google / Chromium OS