Re: [PATCH 0/6] Use crtc helper drm_crtc_from_index() for more drivers

From: Sean Paul <seanpaul@chromium.org>
To: Shawn Guo <shawnguo@kernel.org>
Cc: Xinliang Liu <z.liuxinliang@hisilicon.com>,
	dri-devel <dri-devel@lists.freedesktop.org>,
	Ben Skeggs <bskeggs@redhat.com>
Subject: Re: [PATCH 0/6] Use crtc helper drm_crtc_from_index() for more drivers
Date: Wed, 18 Jan 2017 10:08:09 -0500	[thread overview]
Message-ID: <CAOw6vbLQ=xf-F6LreY3a3_x-dUWzuEmtMWZ5eCbO5G91VTtG-A@mail.gmail.com> (raw)
In-Reply-To: <CAOw6vbKo3UeJmM71y9prm=UG3pKu8FAoR3KLxtcCarAP0BGaMQ@mail.gmail.com>

On Mon, Jan 9, 2017 at 11:11 AM, Sean Paul <seanpaul@chromium.org> wrote:
> On Mon, Jan 9, 2017 at 6:25 AM, Shawn Guo <shawnguo@kernel.org> wrote:
>> From: Shawn Guo <shawn.guo@linaro.org>
>>
>> This is a follow-up series for "[PATCH 0/3] Add CRTC helper
>> drm_crtc_from_index()" per Daniel's comment [1].
>>
>> Basically, it changes some drivers to use helper drm_crtc_from_index()
>> for the vblank code, so that either they do not need to store CRTC
>> pointers in private struct, or the exactly same code for finding CRTC
>> can be saved.
>>
>> Compile tested against the following branch.
>>
>>  git://anongit.freedesktop.org/drm/drm-misc drm-misc-next
>>
>
> Thanks for the patches, Shawn. The entire series is
>
> Reviewed-by: Sean Paul <seanpaul@chromium.org>
>
> I'll apply it to -misc in a week if there are no objections between now and then
>

Didn't see any objections, so the series has been merged.

Sean

> Sean
>
>
>> Shawn
>>
>> [1] http://www.spinics.net/lists/dri-devel/msg128065.html
>>
>> Shawn Guo (6):
>>   drm: exynos: use crtc helper drm_crtc_from_index()
>>   drm: kirin: use crtc helper drm_crtc_from_index()
>>   drm: mediatek: use crtc helper drm_crtc_from_index()
>>   drm: nouveau: use crtc helper drm_crtc_from_index()
>>   drm: tegra: use crtc helper drm_crtc_from_index()
>>   drm: vc4: use crtc helper drm_crtc_from_index()
>>
>>  drivers/gpu/drm/exynos/exynos_drm_crtc.c        |  6 -----
>>  drivers/gpu/drm/exynos/exynos_drm_drv.h         | 10 ++------
>>  drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 10 +++-----
>>  drivers/gpu/drm/hisilicon/kirin/kirin_drm_drv.h |  1 -
>>  drivers/gpu/drm/mediatek/mtk_drm_crtc.c         |  9 +++----
>>  drivers/gpu/drm/mediatek/mtk_drm_drv.h          |  1 -
>>  drivers/gpu/drm/nouveau/nouveau_display.c       | 33 ++++++++++++++-----------
>>  drivers/gpu/drm/tegra/drm.c                     | 19 +++-----------
>>  drivers/gpu/drm/vc4/vc4_crtc.c                  | 17 ++++++-------
>>  drivers/gpu/drm/vc4/vc4_drv.h                   |  1 -
>>  10 files changed, 38 insertions(+), 69 deletions(-)
>>
>> --
>> 1.9.1
>>
>
>
>
> --
> Sean Paul, Software Engineer, Google / Chromium OS

-- 
Sean Paul, Software Engineer, Google / Chromium OS
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel