Chris Wilson <[email protected]> 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
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to