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
