[PATCH v10 17/17] drm/exynos: split exynos_crtc->dpms in enable() and disable()

Gustavo Padovan gustavo at padovan.org
Tue Jun 2 07:06:23 PDT 2015


Hi Inki,

2015-06-02 Inki Dae <inki.dae at samsung.com>:

> Hi,
> 
> On 2015년 06월 02일 00:04, Gustavo Padovan wrote:
> > From: Gustavo Padovan <gustavo.padovan at collabora.co.uk>
> >
> > To follow more closely the new atomic API we split the dpms()
> > helper into the enable() and disable() helper to get exactly the
> > same semantics.
> 
> Below is the result from checkpatch.pl. Please fix all errors and check
> your patch with checkpatch.pl before posting it.
> 
> Thanks,
> Inki Dae
> 
> total: 62 errors, 0 warnings, 410 lines checked

I think you did something wrong when checking, for me it looks like
this:

0016-drm-exynos-split-exynos_crtc-dpms-in-enable-and-disa.patch has no
obvious style problems and is ready for submission.
WARNING: Do not use whitespace before Signed-off-by:
#12: 
    Signed-off-by: Gustavo Padovan <gustavo.padovan at collabora.co.uk>

total: 0 errors, 1 warnings, 594 lines checked

	Gustavo


More information about the dri-devel mailing list