MIME-Version: 1.0
References: <20211028101912.4624-1-jason-jh.lin@mediatek.com>
In-Reply-To: <20211028101912.4624-1-jason-jh.lin@mediatek.com>
From:   Chun-Kuang Hu <chunkuang.hu@kernel.org>
Date:   Thu, 18 Nov 2021 07:54:10 +0800
Message-ID: <CAAOTY__ZbHGef_nVA9N0cXNN5xNwdVGE9DCSo04KZvDiowbgww@mail.gmail.com>
Subject: Re: [PATCH v6 0/5] CMDQ refinement of Mediatek DRM driver
To:     "jason-jh.lin" <jason-jh.lin@mediatek.com>
Cc:     Chun-Kuang Hu <chunkuang.hu@kernel.org>,
        Philipp Zabel <p.zabel@pengutronix.de>,
        Matthias Brugger <matthias.bgg@gmail.com>,
        Jassi Brar <jassisinghbrar@gmail.com>,
        Yongqiang Niu <yongqiang.niu@mediatek.com>,
        Fei Shao <fshao@chromium.org>, David Airlie <airlied@linux.ie>,
        Daniel Vetter <daniel@ffwll.ch>,
        DRI Development <dri-devel@lists.freedesktop.org>,
        "moderated list:ARM/Mediatek SoC support" 
        <linux-mediatek@lists.infradead.org>,
        Linux ARM <linux-arm-kernel@lists.infradead.org>,
        linux-kernel <linux-kernel@vger.kernel.org>,
        Hsin-Yi Wang <hsinyi@chromium.org>,
        Nancy Lin <nancy.lin@mediatek.com>, singo.chang@mediatek.com
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hi, Jason:

For this series except [v6,3/6] drm/mediatek: Detect CMDQ execution
timeout (I pick v5), applied to mediatek-drm-next [1], thanks.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/chunkuang.hu/linux.git/=
log/?h=3Dmediatek-drm-next

Regards,
Chun-Kuang.

jason-jh.lin <jason-jh.lin@mediatek.com> =E6=96=BC 2021=E5=B9=B410=E6=9C=88=
28=E6=97=A5 =E9=80=B1=E5=9B=9B =E4=B8=8B=E5=8D=886:19=E5=AF=AB=E9=81=93=EF=
=BC=9A
>
> Change in v6:
> 1. Drop the redundant checking of cmdq_vblank_cnt .
> 2. fix the indent.
>
> Change in v5:
> 1. Move mbox_free_channel to a independent patch.
>
> Change in v4:
> 1. Add cmdq_vblank_cnt initial value to 3.
> 2. Move mtk_drm_cmdq_pkt_create to the same define scope with
>    mtk_drm_cmdq_pkt_destroy.
>
> Change in v3:
> 1. Revert "drm/mediatek: clear pending flag when cmdq packet is done"
>    and add it after the CMDQ refinement patches.
> 2. Change the remove of struct cmdq_client to remove the pointer of
>    struct cmdq_client.
> 3. Fix pkt buf alloc once but free many times.
>
> Changes in v2:
> 1. Define mtk_drm_cmdq_pkt_create() and mtk_drm_cmdq_pkt_destroy()
>    when CONFIG_MTK_CMDQ is reachable.
>
> Chun-Kuang Hu (4):
>   drm/mediatek: Use mailbox rx_callback instead of cmdq_task_cb
>   drm/mediatek: Remove the pointer of struct cmdq_client
>   drm/mediatek: Detect CMDQ execution timeout
>   drm/mediatek: Add cmdq_handle in mtk_crtc
>
> Yongqiang Niu (1):
>   drm/mediatek: Clear pending flag when cmdq packet is done
>
> jason-jh.lin (1):
>   drm/mediatek: Add mbox_free_channel in mtk_drm_crtc_destroy
>
>  drivers/gpu/drm/mediatek/mtk_drm_crtc.c | 175 ++++++++++++++++++++----
>  1 file changed, 151 insertions(+), 24 deletions(-)
>
> --
> 2.18.0
>