• Alastair Bridgewater's avatar
    drm/nouveau: Clean up nv50_head_atomic_check_mode() and fix blankus calculation · 35dd9874
    Alastair Bridgewater authored
    drm_mode_set_crtcinfo() does compensation for interlace and
    doublescan timing effects already, so do it first and use the
    compensated figures instead of the constant "vscan / ilace" terms
    that we had before.
    
    And then it turns out that the hardware model for how the timing
    parameters are configured is basically the standard model, but
    starting one clock before the sync pulse rather than at the start
    of the display area, which lets us drastically simplify the
    overall timing calculations (verifying the changes by algebraic
    operations is left as an exercise for the reader).
    
    Finally, there were a couple of issues with the computation of
    m->v.blankus that are addressed here.  Interlaced modes would
    generate a negative intermediate result.  Double scan modes would
    generate an overestimate rather than an underestimate.  And when
    enabling frame-packing modes, a rather extreme overestimate would
    be generated.  Fixed, by using the timings as adjusted for the
    CRTC to find the length of the vertical blanking period instead of
    mixing adjusted and pre-adjustment timing parameters.
    Signed-off-by: default avatarAlastair Bridgewater <alastair.bridgewater@gmail.com>
    Signed-off-by: default avatarBen Skeggs <bskeggs@redhat.com>
    35dd9874
nv50_display.c 116 KB