On Fri, Aug 08, 2014 at 05:47:25PM +0530, sonika.jindal@xxxxxxxxx wrote: > /* Start the training iterating through available voltages and emphasis, > * testing each value twice. */ > - for (i = 0; i < ARRAY_SIZE(hsw_ddi_buf_ctl_values) * 2; i++) { > + for (i = 0; i < ARRAY_SIZE(hsw_ddi_translations_fdi); i++) { This is correct but obsfucated. You're actually doing ARRAY_SIZE(hsw_ddi_translations_fdi) / 2 * 2 and we might as well give the ARRAY_SIZE(hsw_ddi_translations_fdi) / 2 a proper name like I suggested. With that fixed, you can add my: Reviewed-by: Damien Lespiau <damien.lespiau@xxxxxxxxx> -- Damien _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx