Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <e398346a-286a-c467-89b6-46f590acdb84@quicinc.com>
Date:   Mon, 24 Apr 2023 16:59:35 -0700
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.6.2
Subject: Re: [PATCH v2 00/17] drm/msm/dpu: Implement tearcheck support on INTF
 block
Content-Language: en-US
To:     Marijn Suijten <marijn.suijten@somainline.org>,
        Rob Clark <robdclark@gmail.com>,
        Dmitry Baryshkov <dmitry.baryshkov@linaro.org>,
        "Sean Paul" <sean@poorly.run>, David Airlie <airlied@gmail.com>,
        Daniel Vetter <daniel@ffwll.ch>,
        Adam Skladowski <a39.skl@gmail.com>,
        Loic Poulain <loic.poulain@linaro.org>,
        Bjorn Andersson <andersson@kernel.org>,
        "Kuogee Hsieh" <quic_khsieh@quicinc.com>,
        Robert Foss <rfoss@kernel.org>, Vinod Koul <vkoul@kernel.org>,
        Rajesh Yadav <ryadav@codeaurora.org>,
        Jeykumar Sankaran <jsanka@codeaurora.org>,
        Neil Armstrong <neil.armstrong@linaro.org>,
        "Chandan Uddaraju" <chandanu@codeaurora.org>,
        Jessica Zhang <quic_jesszhan@quicinc.com>
CC:     <~postmarketos/upstreaming@lists.sr.ht>,
        AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@somainline.org>,
        Konrad Dybcio <konrad.dybcio@linaro.org>,
        Martin Botka <martin.botka@somainline.org>,
        "Jami Kettunen" <jami.kettunen@somainline.org>,
        <linux-arm-msm@vger.kernel.org>, <dri-devel@lists.freedesktop.org>,
        <freedreno@lists.freedesktop.org>, <linux-kernel@vger.kernel.org>,
        Jordan Crouse <jordan@cosmicpenguin.net>,
        Archit Taneja <architt@codeaurora.org>,
        Sravanthi Kollukuduru <skolluku@codeaurora.org>,
        Konrad Dybcio <konrad.dybcio@somainline.org>
References: <20230411-dpu-intf-te-v2-0-ef76c877eb97@somainline.org>
From:   Abhinav Kumar <quic_abhinavk@quicinc.com>
In-Reply-To: <20230411-dpu-intf-te-v2-0-ef76c877eb97@somainline.org>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 4/17/2023 1:21 PM, Marijn Suijten wrote:
> Since DPU 5.0.0 the TEARCHECK registers and interrupts moved out of the
> PINGPONG block and into the INTF.  Implement the necessary callbacks in
> the INTF block, and use these callbacks together with the INTF_TEAR
> interrupts.  Additionally, disable previous register writes and remove
> unused interrupts in the PINGPONG and MDP_TOP blocks for these newer
> platforms.
> 
> With these patches the devices on DPU >= 5.0.0 listed below now update
> their panels at 60fps without tearing (nor sluggishness), and without
> repeated timeouts in dmesg.
> 
> Tested on the following devices with command-mode panels and TE pins:
> 
> - Sony Xperia XZ3 (sdm845, DPU 4.0.0, cmdmode panel): no regressions on
>    PINGPONG TE;
> - Sony Xperia 5 (sm8150, DPU 5.0.0);
> - Sony Xperia 10 II (sm6125, DPU 5.0.4).
> 
> ---

I will pickup the fixes from this one and for the rest, since jessica 
has been rebasing the DSC 1.2 over DSI on top of this series since the 
RFC stage, will let her review and test this out and we can pick up the 
rest for 6.5

So please keep her CCed on the next revisions of this if there are more.

> Changes in v2:
> - Rebase on -next with all the new SC8280XP and SM8[345]50 support;
>    - Remove duplicate PP_BLK_TE macro now that .features is an argument;
>    - Fix PP_BLK_DIPHER -> DITHER typo that was added recently;
>    - Add INTF_TEAR interrupt blocks for DPU 7.0.0 (moved to different
>      register range);
>    - Describe INTF_TEAR support for the newly added SM8350, SM8450,
>      SM8550 and SC8280XP SoCs;
>    - Remove TE2 subblocks from 8[34]50 and sc8280xp (new patch);
> - Rebase on -next with DPU catalog rework;
>    - Remove dpu_hw_intf_v1_get_status which was inlined in the original
>      dpu_hw_intf_get_status function in e3969eadc8ee ("drm/msm/disp/dpu:
>      get timing engine status from intf status register");
>    - Many changes to move all catalog edits to separate files;
> - Add documentation for DPU_MDP_VSYNC_SEL;
> - Fix sdm8150_mdp typo, should be sm8150_mdp;
> - Move unrelated INTF_INTR offsets out of hwio header (new patch);
> - Remove _reg argument from INTF_BLK, since we now have a third
>    interrupt with a different base register.  To prevent confusion all
>    three interrupts should provide the final value from DPU_IRQ_IDX
>    directly.
> - Only request the new tear_rd_ptr in a new INTF_BLK_DSI_TE macro;
> - Drop stray INTF_MISR_SIGNATURE register definition;
> - Clean up registers in dpu_hw_intf.c (many new patches);
> - merged setup_tearcheck() and enable_tearcheck() callbacks;
> - replaced enable_tearcheck(false) with new disable_tearcheck()
>    callback;
> - Moved dpu_encoder_phys_cmd_enable_te intestines (just autorefresh
>    disablement) to INTF and PP block, replacing 3 callbacks in both
>    blocks with just a single disable_autorefresh() callback.
> 
> v1: https://lore.kernel.org/r/20221231215006.211860-1-marijn.suijten@somainline.org
> 
> ---
> Konrad Dybcio (1):
>        drm/msm/dpu: Move dpu_hw_{tear_check,pp_vsync_info} to dpu_hw_mdss.h
> 
> Marijn Suijten (16):
>        drm/msm/dpu: Remove unused INTF0 interrupt mask from SM6115/QCM2290
>        drm/msm/dpu: Remove TE2 block and feature from DPU >= 7.0.0 hardware
>        drm/msm/dpu: Move non-MDP_TOP INTF_INTR offsets out of hwio header
>        drm/msm/dpu: Fix PP_BLK_DIPHER -> DITHER typo
>        drm/msm/dpu: Remove duplicate register defines from INTF
>        drm/msm/dpu: Remove extraneous register define indentation
>        drm/msm/dpu: Sort INTF registers numerically
>        drm/msm/dpu: Drop unused poll_timeout_wr_ptr PINGPONG callback
>        drm/msm/dpu: Move autorefresh disable from CMD encoder to pingpong
>        drm/msm/dpu: Disable pingpong TE on DPU 5.0.0 and above
>        drm/msm/dpu: Disable MDP vsync source selection on DPU 5.0.0 and above
>        drm/msm/dpu: Factor out shared interrupt register in INTF_BLK macro
>        drm/msm/dpu: Document and enable TEAR interrupts on DSI interfaces
>        drm/msm/dpu: Merge setup_- and enable_tearcheck pingpong callbacks
>        drm/msm/dpu: Implement tearcheck support on INTF block
>        drm/msm/dpu: Remove intr_rdptr from DPU >= 5.0.0 pingpong config
> 
>   .../drm/msm/disp/dpu1/catalog/dpu_3_0_msm8998.h    |  26 +-
>   .../gpu/drm/msm/disp/dpu1/catalog/dpu_4_0_sdm845.h |  26 +-
>   .../gpu/drm/msm/disp/dpu1/catalog/dpu_5_0_sm8150.h |  40 +--
>   .../drm/msm/disp/dpu1/catalog/dpu_5_1_sc8180x.h    |  48 ++--
>   .../gpu/drm/msm/disp/dpu1/catalog/dpu_6_0_sm8250.h |  40 +--
>   .../gpu/drm/msm/disp/dpu1/catalog/dpu_6_2_sc7180.h |  16 +-
>   .../gpu/drm/msm/disp/dpu1/catalog/dpu_6_3_sm6115.h |  15 +-
>   .../drm/msm/disp/dpu1/catalog/dpu_6_5_qcm2290.h    |  15 +-
>   .../gpu/drm/msm/disp/dpu1/catalog/dpu_7_0_sm8350.h |  40 +--
>   .../gpu/drm/msm/disp/dpu1/catalog/dpu_7_2_sc7280.h |  22 +-
>   .../drm/msm/disp/dpu1/catalog/dpu_8_0_sc8280xp.h   |  64 +++--
>   .../gpu/drm/msm/disp/dpu1/catalog/dpu_8_1_sm8450.h |  46 ++--
>   .../gpu/drm/msm/disp/dpu1/catalog/dpu_9_0_sm8550.h |  36 ++-
>   drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c        |  11 +
>   drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h   |  10 +-
>   .../gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c   | 210 ++++++++--------
>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c     |  48 ++--
>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h     |   9 +-
>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_interrupts.c  |  32 ++-
>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_interrupts.h  |   4 +
>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c        | 268 ++++++++++++++++++---
>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.h        |  25 ++
>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_mdss.h        |  48 ++++
>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_pingpong.c    |  83 ++++---
>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_pingpong.h    |  64 +----
>   drivers/gpu/drm/msm/disp/dpu1/dpu_hw_top.c         |  52 ++--
>   drivers/gpu/drm/msm/disp/dpu1/dpu_hwio.h           |   3 -
>   drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h            |   4 +
>   drivers/gpu/drm/msm/disp/dpu1/dpu_trace.h          |  14 ++
>   29 files changed, 909 insertions(+), 410 deletions(-)
> ---
> base-commit: 4aa1da8d99724f6c0b762b58a71cee7c5e2e109b
> change-id: 20230411-dpu-intf-te-ea684f13e083
> 
> Best regards,