On Sun, Apr 14, 2013 at 1:58 PM, Rafał Miłecki <zaj...@gmail.com> wrote: > 2013/4/14 Alex Deucher <alexdeuc...@gmail.com>: >>> + /* >>> + * According to the commens above we should use !DCE41 || >>> DCE5, >>> + * condition, however there isn't any DCE5 that is DCE41, so >>> + * DCE5 check is not needed. >>> + */ >> >> It would probably be more obvious to just loop over rdev->num_crtc >> since the number of hdmi blocks matches the number of crtcs. > > Not really. > > 1) CHIP_CAICOS > It has 4 CRTCs but is DCE5, so has 6 AFMTs > > 2) CHIP_CEDAR > It has 4 CRTCs but is DCE4, so also has 6 AFMTs >
Oh, right. I forgot about the 4 crtc parts. Alex _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel