Re: [PATCH 14/28] drm: Remove crtc/connector->save/restore hooks

Thierry Reding
2015-12-07T11:55:09Z

[-- Attachment #1.1: Type: text/plain, Size: 690 bytes --]

On Fri, Dec 04, 2015 at 09:45:55AM +0100, Daniel Vetter wrote:
> They're not how system suspend/resume should be done with atomic
> (there's new helpers for that developed by Thierry Redding), and for

s/Redding/Reding/ =)

> legacy drivers this really should be a helper hook and not a core one.
>
> But there's not even helper code to use them, and only 2 drivers
> (which now have their own private hooks) set them. Ditch them.
>
> Saves me typing some kerneldoc, too ;-)
>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> ---
> include/drm/drm_crtc.h | 11 -----------
> 1 file changed, 11 deletions(-)

Reviewed-by: Thierry Reding <treding@nvidia.com>

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]