Re: [PATCH v3 07/12] drm/i915: Move plane_state->scaler_id initialization into intel_create_plane_state()

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

 



Quoting Ville Syrjala (2018-06-01 18:00:29)
> From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> 
> No point in having each caller of intel_create_plane_state() initialize
> the scaler_id to -1. Instead just do it in intel_create_plane_state().

No tricks spotted.
 
> Previously we left scaler_id at 0 for pre-SKL platforms, but I can't
> see how initializing it to -1 always would cause any harm.

Seems fine.
 
> Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




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

  Powered by Linux