Actually, having said that I think it would make more sense to change
the line to:

      return brw->gen >= 9 ? j * 4 : j;

The 4 that it was returning for Gen<9 is supposed to represent the block
height and I don't think there's any reason to hard-code it seeing as we
are now querying the block height anyway. The later patch about
combining the two functions would need to be changed too.

Regards,
- Neil

Neil Roberts <n...@linux.intel.com> writes:

> Looks good to me.
>
> Reviewed-by: Neil Roberts <n...@linux.intel.com>
>
> - Neil
>
> Anuj Phogat <anuj.pho...@gmail.com> writes:
>
>> Adding Neil to Cc who committed 4ab8d59.
>>
>> Reviewed-by: Anuj Phogat <anuj.pho...@gmail.com>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to