linux-kernel - Re: [PATCH] drm/rcar-du: Convert drm_atomic_helper

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <1998688.b6VzpE9tkn@avalon>
Date:   Sun, 30 Sep 2018 09:57:45 +0300
From:   Laurent Pinchart <laurent.pinchart@...asonboard.com>
To:     Souptick Joarder <jrdr.linux@...il.com>
Cc:     linux-renesas-soc@...r.kernel.org, linux-kernel@...r.kernel.org,
        Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
        Gustavo Padovan <gustavo@...ovan.org>,
        Daniel Vetter <daniel@...ll.ch>, airlied@...ux.ie,
        sean@...rly.run, dri-devel@...ts.freedesktop.org
Subject: Re: [PATCH] drm/rcar-du: Convert drm_atomic_helper_suspend/resume()

Hi Souptick,

On Friday, 28 September 2018 23:02:32 EEST Souptick Joarder wrote:
> On 28-Sep-2018 9:00 PM, "Laurent Pinchart" wrote:
> > On Friday, 28 September 2018 18:05:18 EEST Laurent Pinchart wrote:
> >> On Thursday, 27 September 2018 09:34:18 EEST Souptick Joarder wrote:
> >>> On Tue, Sep 18, 2018 at 10:05 PM Souptick Joarder wrote:
> >>>> convert drm_atomic_helper_suspend/resume() to use
> >>>> drm_mode_config_helper_suspend/resume().
> >>>> 
> >>>> remove suspend_state field from the rcar_du_device
> >>>> structure as it is no more required.
> >>>> 
> >>>> With this conversion, also drm_fbdev_cma_set_suspend_unlocked()
> >>>> will left with no consumer. So this function can be removed.
> >>>> 
> >>>> Signed-off-by: Souptick Joarder <jrdr.linux@...il.com>
> >>> 
> >>> Laurent, any comment on this patch ??
> >> 
> >> Sorry for the delay, and thanks for pinging me.
> >> 
> >> Reviewed-by: Laurent Pinchart <laurent.pinchart@...asonboard.com>
> >> 
> >> Should I apply this to my tree or do you plan to merge it through
> >> drm-misc as it touches drm_fb_cma_helper.c ?
> > 
> > I just realized that the same patch got supplied by Noralf Trønnes nearly
> > a year ago, and was later superseded by https://patchwork.freedesktop.org/
> > patch/247861/. I think we should thus apply Noralf's patches instead (once
> > he sends v4 out).
> 
> Sure, you can go ahead with his patch.
> 
> Shall I remove drm_fbdev_cma_set_suspend_unlocked in a separate patch once
> all the consumer remove this function ?

Sure, if the function isn't removed as part of Noralf's patch series, please 
send a separate patch to remove it.

-- 
Regards,

Laurent Pinchart