<div class="gmail_quote">On Sat, Feb 11, 2012 at 16:13, Kai Krakow <span dir="ltr"><<a href="mailto:hurikhan77@gmail.com">hurikhan77@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>> schrieb:<br>
<div class="im"><br>
>> +     if (rc6_mode > 0) {<br>
> I'm a little uneasy mixing signs and bitmasks, and this test looks a<br>
> little pointless as intel_enable_rc6 now returns the bitmask. Kill it<br>
> and we kill one level of indentation!<br>
<br>
</div>I think not, because Eugeni documented "-1" as the chipset default...<br></blockquote><div><br>Not quite, Chris is right, we can get rid of this if block. Chipset default values are being handled by the intel_enable_rc6(), that function returns a valid bitmask now. So it will always return some meaningful values to us and explicitly say if RC6 should be disabled or enabled (and if enabled, which states are active).<br clear="all">

</div></div><br>-- <br>Eugeni Dodonov<a href="http://eugeni.dodonov.net/" target="_blank"><br></a><br>