On 11/26/2025 8:36 AM, Dmitry Baryshkov wrote:
On Tue, Nov 25, 2025 at 04:22:12PM +0800, Yongxing Mou wrote:
From: Abhinav Kumar <[email protected]>
Interface type of MST interfaces is currently INTF_NONE. Update this to
INTF_DP. And correct the intf_6 intr_underrun/intr_vsync index for
dpu_8_4_sa8775p.
Signed-off-by: Abhinav Kumar <[email protected]>
Signed-off-by: Yongxing Mou <[email protected]>
Reviewed-by: Dmitry Baryshkov <[email protected]>
---
This patch was split out from the MST V3 patch series:
https://lore.kernel.org/all/[email protected]/
---
Changes in v4:
- Splite chagne out from the MST V3 series.
Why?
I would like to send out some patches that are independent and have no
dependencies on the overall series, to make maintenance and review
easier. For example, in V3, patches 1, 29, 33, and 37. I mentioned this
approach in the first patch of V3 before. Please let me know if this is
appropriate. If not, I will stop immediately.
- Link to v3:
https://lore.kernel.org/all/[email protected]/
Changes in v3:
- Fix through the whole catalog
- Link to v2:
https://lore.kernel.org/all/[email protected]/
Changes in v2:
- Change the patch order in the series.
- Link to v1:
https://lore.kernel.org/all/[email protected]/
---
drivers/gpu/drm/msm/disp/dpu1/catalog/dpu_8_0_sc8280xp.h | 6 +++---
drivers/gpu/drm/msm/disp/dpu1/catalog/dpu_8_4_sa8775p.h | 12 ++++++------
drivers/gpu/drm/msm/disp/dpu1/catalog/dpu_9_2_x1e80100.h | 6 +++---
3 files changed, 12 insertions(+), 12 deletions(-)
@@ -363,15 +363,15 @@ static const struct dpu_intf_cfg sa8775p_intf[] = {
}, {
.name = "intf_6", .id = INTF_6,
.base = 0x3A000, .len = 0x280,
- .type = INTF_NONE,
+ .type = INTF_DP,
.controller_id = MSM_DP_CONTROLLER_0, /* pair with intf_0 for
DP MST */
.prog_fetch_lines_worst_case = 24,
- .intr_underrun = DPU_IRQ_IDX(MDP_SSPP_TOP0_INTR, 17),
- .intr_vsync = DPU_IRQ_IDX(MDP_SSPP_TOP0_INTR, 16),
+ .intr_underrun = DPU_IRQ_IDX(MDP_SSPP_TOP0_INTR, 16),
+ .intr_vsync = DPU_IRQ_IDX(MDP_SSPP_TOP0_INTR, 17),
This is a separate fix, it should be a separate patch.
Got that, thanks..
