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