Re: [PATCH 9/9] drm/i915/dsb: Nuke the 'dev' variables

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 29-Jan-20 11:50 PM, Ville Syrjala wrote:
From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>

No point in having these drm_device variables around. Get rid of them.

Reviewed-by: Swati Sharma <swati2.sharma@xxxxxxxxx>


Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
---
  drivers/gpu/drm/i915/display/intel_dsb.c | 6 ++----
  1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_dsb.c b/drivers/gpu/drm/i915/display/intel_dsb.c
index 56790fae6386..c97b64401c23 100644
--- a/drivers/gpu/drm/i915/display/intel_dsb.c
+++ b/drivers/gpu/drm/i915/display/intel_dsb.c
@@ -57,8 +57,7 @@ static inline bool is_dsb_busy(struct intel_dsb *dsb)
  struct intel_dsb *
  intel_dsb_get(struct intel_crtc *crtc)
  {
-	struct drm_device *dev = crtc->base.dev;
-	struct drm_i915_private *i915 = to_i915(dev);
+	struct drm_i915_private *i915 = to_i915(crtc->base.dev);
  	struct intel_dsb *dsb = &crtc->dsb;
  	struct drm_i915_gem_object *obj;
  	struct i915_vma *vma;
@@ -276,8 +275,7 @@ static u32 intel_dsb_align_tail(struct intel_dsb *dsb)
  void intel_dsb_commit(struct intel_dsb *dsb)
  {
  	struct intel_crtc *crtc = container_of(dsb, typeof(*crtc), dsb);
-	struct drm_device *dev = crtc->base.dev;
-	struct drm_i915_private *dev_priv = to_i915(dev);
+	struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
  	enum pipe pipe = crtc->pipe;
  	u32 tail;

--
~Swati Sharma
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux