drm/i915: Add aliases for uapi and hw to crtc_state

Prepare to split up hw and uapi machinally, by adding a uapi and
hw alias. We will remove the base in a bit. This is a split from the
original uapi/hw patch, which did it all in one go.

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20191031112610.27608-2-maarten.lankhorst@linux.intel.com
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>

diff --git a/drivers/gpu/drm/i915/display/intel_atomic.c b/drivers/gpu/drm/i915/display/intel_atomic.c
index 9cd6d2348a1edcc1f414268f0f3cb550be5c6630..4826aa4ee8e77547503c20cdc840c8571b3760ea 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_atomic.c
+++ b/drivers/gpu/drm/i915/display/intel_atomic.c
@@ -186,9 +186,10 @@ intel_digital_connector_duplicate_state(struct drm_connector *connector)
 struct drm_crtc_state *
 intel_crtc_duplicate_state(struct drm_crtc *crtc)
 {
+       const struct intel_crtc_state *old_crtc_state = to_intel_crtc_state(crtc->state);
        struct intel_crtc_state *crtc_state;
 
-       crtc_state = kmemdup(crtc->state, sizeof(*crtc_state), GFP_KERNEL);
+       crtc_state = kmemdup(old_crtc_state, sizeof(*crtc_state), GFP_KERNEL);
        if (!crtc_state)
                return NULL;
 
@@ -219,7 +220,10 @@ void
 intel_crtc_destroy_state(struct drm_crtc *crtc,
                         struct drm_crtc_state *state)
 {
-       drm_atomic_helper_crtc_destroy_state(crtc, state);
+       struct intel_crtc_state *crtc_state = to_intel_crtc_state(state);
+
+       __drm_atomic_helper_crtc_destroy_state(&crtc_state->base);
+       kfree(crtc_state);
 }
 
 static void intel_atomic_setup_scaler(struct intel_crtc_scaler_state *scaler_state,

diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index 9b7c5596d74a7df25bb28c6b487dba642955bf06..88e81fad36f1147be0cecc7d17408d2f23085820 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -12592,6 +12592,8 @@ clear_intel_crtc_state(struct intel_crtc_state *crtc_state)
 
        /* Keep base drm_crtc_state intact, only clear our extended struct */
        BUILD_BUG_ON(offsetof(struct intel_crtc_state, base));
+       BUILD_BUG_ON(offsetof(struct intel_crtc_state, uapi));
+       BUILD_BUG_ON(offsetof(struct intel_crtc_state, hw));
        memcpy(&crtc_state->base + 1, &saved_state->base + 1,
               sizeof(*crtc_state) - sizeof(crtc_state->base));
 

diff --git a/drivers/gpu/drm/i915/display/intel_display.h b/drivers/gpu/drm/i915/display/intel_display.h
index 355c50088589bf35775d8e87550ebb812711224d..476bc71e6a83deb18707ca5f49db5cfdf7ccd129 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_display.h
+++ b/drivers/gpu/drm/i915/display/intel_display.h
@@ -447,10 +447,10 @@ enum phy_fia {
 #define intel_atomic_crtc_state_for_each_plane_state( \
                  plane, plane_state, \
                  crtc_state) \
-       for_each_intel_plane_mask(((crtc_state)->base.state->dev), (plane), \
-                               ((crtc_state)->base.plane_mask)) \
+       for_each_intel_plane_mask(((crtc_state)->uapi.state->dev), (plane), \
+                               ((crtc_state)->uapi.plane_mask)) \
                for_each_if ((plane_state = \
-                             to_intel_plane_state(__drm_atomic_get_current_plane_state((crtc_state)->base.state, &plane->base))))
+                             to_intel_plane_state(__drm_atomic_get_current_plane_state((crtc_state)->uapi.state, &plane->base))))
 
 void intel_link_compute_m_n(u16 bpp, int nlanes,
                            int pixel_clock, int link_clock,

diff --git a/drivers/gpu/drm/i915/display/intel_display_types.h b/drivers/gpu/drm/i915/display/intel_display_types.h
index 4341bd66a4187e1510613480d7cdb4f5170632c2..285b17c9d8d09bd07003c433847069262f576920 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_display_types.h
+++ b/drivers/gpu/drm/i915/display/intel_display_types.h
@@ -757,7 +757,32 @@ enum intel_output_format {
 };
 
 struct intel_crtc_state {
+       union {
        struct drm_crtc_state base;
+       /*
+        * uapi (drm) state. This is the software state shown to userspace.
+        * In particular, the following members are used for bookkeeping:
+        * - crtc
+        * - state
+        * - *_changed
+        * - event
+        * - commit
+        * - mode_blob
+        */
+       struct drm_crtc_state uapi;
+
+       /*
+        * actual hardware state, the state we program to the hardware.
+        * The following members are used to verify the hardware state:
+        * - enable
+        * - active
+        * - mode / adjusted_mode
+        * - color property blobs.
+        *
+        * During initial hw readout, they need to be copied to uapi.
+        */
+       struct drm_crtc_state hw;
+       };
 
        /**
         * quirks - bitfield with hw state readout quirks
@@ -1112,7 +1137,7 @@ struct cxsr_latency {
 
 #define to_intel_atomic_state(x) container_of(x, struct intel_atomic_state, base)
 #define to_intel_crtc(x) container_of(x, struct intel_crtc, base)
-#define to_intel_crtc_state(x) container_of(x, struct intel_crtc_state, base)
+#define to_intel_crtc_state(x) container_of(x, struct intel_crtc_state, uapi)
 #define to_intel_connector(x) container_of(x, struct intel_connector, base)
 #define to_intel_encoder(x) container_of(x, struct intel_encoder, base)
 #define to_intel_framebuffer(x) container_of(x, struct intel_framebuffer, base)