On Fri, Aug 18, 2017 at 8:28 AM, Sean Paul <seanp...@chromium.org> wrote: > On Thu, Aug 17, 2017 at 12:00:04PM -0700, John Stultz wrote: >> Currently the hikey dsi logic cannot generate accurate byte >> clocks values for all pixel clock values. Thus if a mode clock >> is selected that cannot match the calculated byte clock, the >> device will boot with a blank screen. >> >> This patch uses the new mode_valid callback (many thanks to >> Jose Abreu for upstreaming it!) to ensure we don't select >> modes we cannot generate. >> >> Also, since the ade crtc code will adjust the mode in mode_set, >> this patch also adds a mode_fixup callback which we use to make >> sure we are validating the mode clock that will eventually be >> used. >> >> Cc: Daniel Vetter <daniel.vet...@intel.com> >> Cc: Jani Nikula <jani.nik...@linux.intel.com> >> Cc: Sean Paul <seanp...@chromium.org> >> Cc: David Airlie <airl...@linux.ie> >> Cc: Rob Clark <robdcl...@gmail.com> >> Cc: Xinliang Liu <xinliang....@linaro.org> >> Cc: Xinliang Liu <z.liuxinli...@hisilicon.com> >> Cc: Rongrong Zou <zourongr...@gmail.com> >> Cc: Xinwei Kong <kong.kongxin...@hisilicon.com> >> Cc: Chen Feng <puck.c...@hisilicon.com> >> Cc: Jose Abreu <jose.ab...@synopsys.com> >> Cc: Archit Taneja <arch...@codeaurora.org> >> Cc: dri-devel@lists.freedesktop.org >> Signed-off-by: John Stultz <john.stu...@linaro.org> > > Hi John, > Thanks for continuing to send new versions for this patch. It looks good to me > (there's a small spelling mistake in a comment below that perhaps can be fixed > when applied, no biggy). > > Reviewed-by: Sean Paul <seanp...@chromium.org>
Thanks so much for the review! I really appreciate it! -john _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
