On Wed, Nov 27, 2024 at 04:15:51PM +0800, Yongxing Mou wrote:
> The Qualcomm QCS8300 platform comes with a DisplayPort controller
> with same base offset with SA8775P, add support for this in the
> DisplayPort driver.
Can we reuse SA8775P config then? And SA8775p compatible as a fallback,
not requiring any driver modifications.
>
> Signed-off-by: Yongxing Mou <quic_yongmou@xxxxxxxxxxx>
> ---
> drivers/gpu/drm/msm/dp/dp_display.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/drivers/gpu/drm/msm/dp/dp_display.c b/drivers/gpu/drm/msm/dp/dp_display.c
> index aba925aab7ad7c6652e81004043864c1cb3ac370..f870faa89f26a8cb5bd7f4caf11f42e919c9efac 100644
> --- a/drivers/gpu/drm/msm/dp/dp_display.c
> +++ b/drivers/gpu/drm/msm/dp/dp_display.c
> @@ -118,6 +118,11 @@ struct msm_dp_desc {
> bool wide_bus_supported;
> };
>
> +static const struct msm_dp_desc msm_dp_desc_qcs8300[] = {
> + { .io_start = 0x0af54000, .id = MSM_DP_CONTROLLER_0, .wide_bus_supported = true },
> + {}
> +};
> +
> static const struct msm_dp_desc msm_dp_desc_sa8775p[] = {
> { .io_start = 0x0af54000, .id = MSM_DP_CONTROLLER_0, .wide_bus_supported = true },
> { .io_start = 0x0af5c000, .id = MSM_DP_CONTROLLER_1, .wide_bus_supported = true },
> @@ -170,6 +175,7 @@ static const struct msm_dp_desc msm_dp_desc_x1e80100[] = {
> };
>
> static const struct of_device_id msm_dp_dt_match[] = {
> + { .compatible = "qcom,qcs8300-dp", .data = &msm_dp_desc_qcs8300 },
> { .compatible = "qcom,sa8775p-dp", .data = &msm_dp_desc_sa8775p },
> { .compatible = "qcom,sc7180-dp", .data = &msm_dp_desc_sc7180 },
> { .compatible = "qcom,sc7280-dp", .data = &msm_dp_desc_sc7280 },
>
> --
> 2.34.1
>
--
With best wishes
Dmitry
