Re: [PATCH v2 7/8] drm/msm/dpu: pull connector from dpu_encoder_phys to dpu_encoder_virt

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx>, Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>, Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>, Rob Clark <robdclark@xxxxxxxxx>, Sean Paul <sean@xxxxxxxxxx>
Subject: Re: [PATCH v2 7/8] drm/msm/dpu: pull connector from dpu_encoder_phys to dpu_encoder_virt
From: Stephen Boyd <swboyd@xxxxxxxxxxxx>
Date: Tue, 15 Feb 2022 18:00:15 -0800
Cc: David Airlie <airlied@xxxxxxxx>, Daniel Vetter <daniel@xxxxxxxx>, linux-arm-msm@xxxxxxxxxxxxxxx, dri-devel@xxxxxxxxxxxxxxxxxxxxx, freedreno@xxxxxxxxxxxxxxxxxxxxx
In-reply-to: <20220215141643.3444941-8-dmitry.baryshkov@linaro.org>
References: <20220215141643.3444941-1-dmitry.baryshkov@linaro.org> <20220215141643.3444941-8-dmitry.baryshkov@linaro.org>
User-agent: alot/0.10

Quoting Dmitry Baryshkov (2022-02-15 06:16:42)
> All physical encoders used by virtual encoder share the same connector,
> so pull the connector field from dpu_encoder_phys into dpu_encoder_virt
> structure.

What is the benefit? Less code? Clearer association?

Follow-Ups:
- Re: [PATCH v2 7/8] drm/msm/dpu: pull connector from dpu_encoder_phys to dpu_encoder_virt
  - From: Dmitry Baryshkov

References:
- [PATCH v2 0/8] drm/msm/dpu: cleanup dpu encoder code
  - From: Dmitry Baryshkov
- [PATCH v2 7/8] drm/msm/dpu: pull connector from dpu_encoder_phys to dpu_encoder_virt
  - From: Dmitry Baryshkov

Prev by Date: Re: [PATCH v2 6/8] drm/msm/dpu: switch dpu_encoder to use atomic_mode_set
Next by Date: Re: [PATCH] drm/msm/dpu: drop unused access macros
Previous by thread: [PATCH v2 7/8] drm/msm/dpu: pull connector from dpu_encoder_phys to dpu_encoder_virt
Next by thread: Re: [PATCH v2 7/8] drm/msm/dpu: pull connector from dpu_encoder_phys to dpu_encoder_virt
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [Linux for Sparc] [IETF Annouce] [Security] [Bugtraq] [Linux MIPS] [ECOS] [Asterisk Internet PBX] [Linux API]