Hi, YT:

On Mon, 2016-09-12 at 18:15 +0800, YT Shen wrote:

Hi CK,

On Wed, 2016-09-07 at 12:58 +0800, CK Hu wrote:

quoted

Hi, YT:

On Fri, 2016-09-02 at 19:24 +0800, YT Shen wrote:

quoted

This patch update enable/disable flow of DSI module and MIPI TX module

Signed-off-by: shaoming chen <redacted>
Signed-off-by: YT Shen <redacted>
---

I think the description is too simple. Please briefly describe WHY of
this patch. The original enable/disable flow is workable, so why do you
need this patch? Without this patch, what problem would happen?

Got it, we will update more descriptions in the next version.
There is no transfer/interrupt function in the upstream DSI driver.
We also implement the following function [1][2] in this patch series.

Original flow works on there is a bridge chip: DSI -> bridge -> panel.
In this case: DSI -> panel, the DSI sub driver flow should be updated.
We need to initialize DSI first so that we can send commands to panel.

[1] https://patchwork.kernel.org/patch/9310819/
drm/mediatek: add dsi interrupt control
[2] https://patchwork.kernel.org/patch/9310823/
drm/mediatek: add dsi transfer function

I suggest you to separate "DSI directly connect to panel" related
patches to another series because MT8173 could also apply it and it is
not essential for MT2701 if MT2701 use bridge IC for dsi.

Regards,
CK

quoted

Regards,
CK