diff options
| author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2021-01-12 09:43:53 +0100 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2021-02-25 13:03:11 +0100 |
| commit | 0a9422d279e9d07bfe070eedee482f1a43821802 (patch) | |
| tree | ee2e63b8d7dcbf5f0e295b416389e19d86a4a1fb /drivers/gpu/drm/arc | |
| parent | 3bc8a91a830b9c79377058bad64f485865ee5ba0 (diff) | |
| download | lwn-0a9422d279e9d07bfe070eedee482f1a43821802.tar.gz lwn-0a9422d279e9d07bfe070eedee482f1a43821802.zip | |
drm/arc: Drop crtc check in arc_pgu_update
It's redundant, drm core guarantees that state->fb is set iff
state->crtc is set.
v2: I had a misconception about simple helpers here and thought they
filter this out. They don't. Issue reported by Eugeniy.
Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
Cc: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Cc: Alexey Brodkin <abrodkin@synopsys.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210112084358.2771527-10-daniel.vetter@ffwll.ch
Diffstat (limited to 'drivers/gpu/drm/arc')
| -rw-r--r-- | drivers/gpu/drm/arc/arcpgu_crtc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/arc/arcpgu_crtc.c b/drivers/gpu/drm/arc/arcpgu_crtc.c index 19accc5a2f93..bfd892a9fa3d 100644 --- a/drivers/gpu/drm/arc/arcpgu_crtc.c +++ b/drivers/gpu/drm/arc/arcpgu_crtc.c @@ -135,7 +135,7 @@ static void arc_pgu_update(struct drm_simple_display_pipe *pipe, struct arcpgu_drm_private *arcpgu; struct drm_gem_cma_object *gem; - if (!pipe->plane.state->crtc || !pipe->plane.state->fb) + if (!pipe->plane.state->fb) return; arcpgu = pipe_to_arcpgu_priv(pipe); |