Chris Wilson <ch...@chris-wilson.co.uk> schrieb:

>> +     if (rc6_mode > 0) {
> I'm a little uneasy mixing signs and bitmasks, and this test looks a
> little pointless as intel_enable_rc6 now returns the bitmask. Kill it
> and we kill one level of indentation!

I think not, because Eugeni documented "-1" as the chipset default...

Just my 2 cents...

Regards,
Kai

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to