Re: [PATCH v12 4/4] drm/msm/dp: enable widebus feature for display port

[Kuogee Hsieh]

Hi Stephen,


Would you please to help land this serial of 4 patch into msm-next?

Thanks,

On 2/25/2022 2:17 PM, Stephen Boyd wrote:
> Quoting Kuogee Hsieh (2022-02-25 13:23:12)
> > Widebus feature will transmit two pixel data per pixel clock to interface.
> > This feature now is required to be enabled to easy migrant to higher
> > resolution applications in future. However since some legacy chipsets
> > does not support this feature, this feature is enabled by setting
> > wide_bus_en flag to true within msm_dp_desc struct.
> >
> > changes in v2:
> > -- remove compression related code from timing
> > -- remove op_info from  struct msm_drm_private
> > -- remove unnecessary wide_bus_en variables
> > -- pass wide_bus_en into timing configuration by struct msm_dp
> >
> > Changes in v3:
> > -- split patch into 3 patches
> > -- enable widebus feature base on chip hardware revision
> >
> > Changes in v5:
> > -- DP_INTF_CONFIG_DATABUS_WIDEN
> >
> > Changes in v6:
> > -- static inline bool msm_dp_wide_bus_enable() in msm_drv.h
> >
> > Changes in v7:
> > -- add Tested-by
> >
> > Changes in v9:
> > -- add wide_bus_en to msm_dp_desc
> >
> > Changes in v10:
> > -- add wide_bus_en boolean to dp_catalog struc to avoid passing it as parameter
> >
> > Changes in v11:
> > -- add const to dp_catalog_hw_revision()
> > -- add const to msm_dp_wide_bus_available()
> >
> > Changes in v12:
> > -- dp_catalog_hw_revision(const struct dp_catalog *dp_catalog)
> > -- msm_dp_wide_bus_available(const struct msm_dp *dp_display)
> >
> > Signed-off-by: Kuogee Hsieh <quic_khsieh@xxxxxxxxxxx>
> > Reported-by: kernel test robot <lkp@xxxxxxxxx>
> > Tested-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>
> > ---
> Reviewed-by: Stephen Boyd <swboyd@xxxxxxxxxxxx>