On Wed, Jul 13, 2011 at 5:40 PM, Benjamin Herrenschmidt <[email protected]> wrote: > On Wed, 2011-07-13 at 10:38 -0400, Alex Deucher wrote: > >> > case 6: >> > - args.v6.ulCrtcPclkFreq.ucCRTC = crtc_id; >> > - args.v6.ulCrtcPclkFreq.ulPixelClock = >> > cpu_to_le32(clock / 10); >> > + args.v6.ulDispEngClkFreq = cpu_to_le32(crtc_id << >> > 24 | clock / 10); >> >> For clarity (i can never remember op precedence), you might put: >> >> cpu_to_le32((crtc_id << 24) | (clock / 10)); > > I can't either but I have a nice chart printed on my wall :-) > > I can respin if you really want but the above is correct.
That's fine as is then. Alex > > Cheers, > Ben. > >> Other than that, >> >> Reviewed-by: Alex Deucher <[email protected]> >> >> >> > args.v6.ucRefDiv = ref_div; >> > args.v6.usFbDiv = cpu_to_le16(fb_div); >> > args.v6.ulFbDivDecFrac = cpu_to_le32(frac_fb_div * >> > 100000); >> > >> > >> > >> > > > > _______________________________________________ dri-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/dri-devel
