[RESEND][PATCH v4] drm: kirin: Add mode_valid logic to avoid mode clocks we can't generate

[John Stultz]

On Fri, Aug 18, 2017 at 8:28 AM, Sean Paul <seanpaul at 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.vetter at intel.com>
>> Cc: Jani Nikula <jani.nikula at linux.intel.com>
>> Cc: Sean Paul <seanpaul at chromium.org>
>> Cc: David Airlie <airlied at linux.ie>
>> Cc: Rob Clark <robdclark at gmail.com>
>> Cc: Xinliang Liu <xinliang.liu at linaro.org>
>> Cc: Xinliang Liu <z.liuxinliang at hisilicon.com>
>> Cc: Rongrong Zou <zourongrong at gmail.com>
>> Cc: Xinwei Kong <kong.kongxinwei at hisilicon.com>
>> Cc: Chen Feng <puck.chen at hisilicon.com>
>> Cc: Jose Abreu <Jose.Abreu at synopsys.com>
>> Cc: Archit Taneja <architt at codeaurora.org>
>> Cc: dri-devel at lists.freedesktop.org
>> Signed-off-by: John Stultz <john.stultz at 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 <seanpaul at chromium.org>

Thanks so much for the review! I really appreciate it!
-john