From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
Replace pointless memcpy()s of structures. A normal assignment
will do just fine, thank you.
Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
---
drivers/gpu/drm/i915/display/intel_atomic_plane.c | 3 +--
drivers/gpu/drm/i915/display/intel_display.c | 4 ++--
2 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
index 605f3b7ef4e0..ab7d5e6105ab 100644
--- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c
+++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
@@ -355,8 +355,7 @@ void intel_plane_copy_hw_state(struct intel_plane_state *plane_state,
{
intel_plane_clear_hw_state(plane_state);
- memcpy(&plane_state->hw, &from_plane_state->hw,
- sizeof(plane_state->hw));
+ plane_state->hw = from_plane_state->hw;
if (plane_state->hw.fb)
drm_framebuffer_get(plane_state->hw.fb);
diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index 3abae05ab04c..27ef77fb4324 100644
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -4480,7 +4480,7 @@ copy_bigjoiner_crtc_state_modeset(struct intel_atomic_state *state,
saved_state->crc_enabled = slave_crtc_state->crc_enabled;
intel_crtc_free_hw_state(slave_crtc_state);
- memcpy(slave_crtc_state, saved_state, sizeof(*slave_crtc_state));
+ *slave_crtc_state = *saved_state;
kfree(saved_state);
/* Re-init hw state */
@@ -4540,7 +4540,7 @@ intel_crtc_prepare_cleared_state(struct intel_atomic_state *state,
IS_VALLEYVIEW(dev_priv) || IS_CHERRYVIEW(dev_priv))
saved_state->wm = crtc_state->wm;
- memcpy(crtc_state, saved_state, sizeof(*crtc_state));
+ *crtc_state = *saved_state;
kfree(saved_state);
intel_crtc_copy_uapi_to_hw_state_modeset(state, crtc);
--
2.41.0
