<div dir="ltr"><span style="font-size:13px;line-height:19.5px">> +            /* The kernel returns 12 for some cards for an unknown reason.</span><br style="font-size:13px;line-height:19.5px"><span style="font-size:13px;line-height:19.5px">> +             * I thought this was supposed to be a power of two.</span><br style="font-size:13px;line-height:19.5px"><span style="font-size:13px;line-height:19.5px">> +             */</span><br style="font-size:13px;line-height:19.5px"><div><span style="font-size:13px;line-height:19.5px">> +            if (ws->gen == DRV_SI && ws->info.num_tile_pipes == 12)</span><br style="font-size:13px;line-height:19.5px"><span style="font-size:13px;line-height:19.5px">> +                ws->info.num_tile_pipes = 8;</span><br style="font-size:13px;line-height:19.5px"><span style="font-size:13px;line-height:19.5px">> +</span><br style="font-size:13px;line-height:19.5px"></div><div><span style="font-size:13px;line-height:19.5px"><br></span></div><div><span style="line-height:19.5px">I may be late in the conversation, but shouldn't we have a look at why the value reported by the kernel is wrong for "some" cards? Which ones and why should be identified. It seems to be limited to Southern Islands as far as we know for now, which limits the scope for now.</span></div><div><span style="font-size:13px;line-height:19.5px"><br></span></div><div><span style="font-size:13px;line-height:19.5px">Also, about the patch itself, even if only some cards were reported to be problematic, why would we limit it to "</span><span style="font-size:13px;line-height:19.5px">ws->gen == DRV_SI"? Any cards reporting a wrong value should be treated the same way by mapping its value from 12 to 8, no?</span></div><div><span style="font-size:13px;line-height:19.5px"><br></span></div><div><span style="font-size:13px;line-height:19.5px">My late two cents here.</span></div><div><span style="font-size:13px;line-height:19.5px">Alexandre Demers</span></div></div>