Skip to content

Commit

Permalink
drm/i915: Remove plane_config from struct intel_crtc, v2.
Browse files Browse the repository at this point in the history
Nothing depends on this outside initial hw readout, so keep this
struct on the stack instead.

Changes since v1:
- Remove unrelated changes.

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Reviewed-by: Daniel Stone <daniels@collabora.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
  • Loading branch information
Maarten Lankhorst authored and Daniel Vetter committed Jul 14, 2015
1 parent be5651f commit eeebeac
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 10 deletions.
19 changes: 10 additions & 9 deletions drivers/gpu/drm/i915/intel_display.c
Original file line number Diff line number Diff line change
Expand Up @@ -15202,6 +15202,8 @@ void intel_modeset_init(struct drm_device *dev)
drm_modeset_unlock_all(dev);

for_each_intel_crtc(dev, crtc) {
struct intel_initial_plane_config plane_config = {};

if (!crtc->active)
continue;

Expand All @@ -15212,15 +15214,14 @@ void intel_modeset_init(struct drm_device *dev)
* can even allow for smooth boot transitions if the BIOS
* fb is large enough for the active pipe configuration.
*/
if (dev_priv->display.get_initial_plane_config) {
dev_priv->display.get_initial_plane_config(crtc,
&crtc->plane_config);
/*
* If the fb is shared between multiple heads, we'll
* just get the first one.
*/
intel_find_initial_plane_obj(crtc, &crtc->plane_config);
}
dev_priv->display.get_initial_plane_config(crtc,
&plane_config);

/*
* If the fb is shared between multiple heads, we'll
* just get the first one.
*/
intel_find_initial_plane_obj(crtc, &plane_config);
}
}

Expand Down
1 change: 0 additions & 1 deletion drivers/gpu/drm/i915/intel_drv.h
Original file line number Diff line number Diff line change
Expand Up @@ -550,7 +550,6 @@ struct intel_crtc {
uint32_t cursor_size;
uint32_t cursor_base;

struct intel_initial_plane_config plane_config;
struct intel_crtc_state *config;
bool new_enabled;

Expand Down

0 comments on commit eeebeac

Please sign in to comment.