Re: [PATCH 07/15] drm/i915: Remove unused loop from intel_dp_mst_compute_config

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

 



On Mon, Aug 22, 2016 at 02:43:04PM +0200, Maarten Lankhorst wrote:
> Op 18-08-16 om 15:34 schreef Daniel Vetter:
> > On Tue, Aug 09, 2016 at 05:04:06PM +0200, Maarten Lankhorst wrote:
> >> conn_state is passed as argument now, if anything required conn_state
> >> they can get it without having to look it up.
> > Commit message seems off, this has been dead code before. It seems to be
> > duct-tape from before we had a proper intel_mst_best_encoder.
> What about this?
> 
> Now that conn_state is passed in as argument to compute_config, it's guaranteed
> that there is a connector for the argument. The code that looks for the connector
> is now dead, and completely unused. Delete it.

Sounds good. Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx> with
that.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
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