On 30/09/2017 03:43, Algea Cao wrote:
> To determine type of SOC, we add a parameter dev_type.
>
> Signed-off-by: Algea Cao <algea.cao@xxxxxxxxxxxxxx>
> ---
> drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c | 5 +++++
> include/drm/bridge/dw_hdmi.h | 10 ++++++++++
> 2 files changed, 15 insertions(+)
>
> diff --git a/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c b/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c
> index 719452d..09c77f9 100644
> --- a/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c
> +++ b/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c
> @@ -46,6 +46,7 @@ struct rockchip_hdmi {
> struct regmap *regmap;
> struct drm_encoder encoder;
> const struct rockchip_hdmi_chip_data *chip_data;
> + enum dw_hdmi_devtype dev_type;
> struct clk *vpll_clk;
> struct clk *grf_clk;
> };
> @@ -305,6 +306,7 @@ static const struct dw_hdmi_plat_data rk3288_hdmi_drv_data = {
> .cur_ctr = rockchip_cur_ctr,
> .phy_config = rockchip_phy_config,
> .phy_data = &rk3288_chip_data,
> + .dev_type = RK3288_HDMI,
> };
>
> static struct rockchip_hdmi_chip_data rk3399_chip_data = {
> @@ -315,6 +317,7 @@ static struct rockchip_hdmi_chip_data rk3399_chip_data = {
>
> static const struct dw_hdmi_plat_data rk3328_hdmi_drv_data = {
> .mode_valid = dw_hdmi_rockchip_mode_valid,
> + .dev_type = RK3328_HDMI,
> };
> static const struct dw_hdmi_plat_data rk3399_hdmi_drv_data = {
> .mode_valid = dw_hdmi_rockchip_mode_valid,
> @@ -322,6 +325,7 @@ static const struct dw_hdmi_plat_data rk3399_hdmi_drv_data = {
> .cur_ctr = rockchip_cur_ctr,
> .phy_config = rockchip_phy_config,
> .phy_data = &rk3399_chip_data,
> + .dev_type = RK3399_HDMI,
> };
>
> static const struct of_device_id dw_hdmi_rockchip_dt_ids[] = {
> @@ -360,6 +364,7 @@ static int dw_hdmi_rockchip_bind(struct device *dev, struct device *master,
> plat_data = match->data;
> hdmi->dev = &pdev->dev;
> hdmi->chip_data = plat_data->phy_data;
> + hdmi->dev_type = plat_data->dev_type;
> encoder = &hdmi->encoder;
>
> encoder->possible_crtcs = drm_of_find_possible_crtcs(drm, dev->of_node);
> diff --git a/include/drm/bridge/dw_hdmi.h b/include/drm/bridge/dw_hdmi.h
> index 182f832..fdb1f0a 100644
> --- a/include/drm/bridge/dw_hdmi.h
> +++ b/include/drm/bridge/dw_hdmi.h
> @@ -92,6 +92,15 @@ enum dw_hdmi_phy_type {
> DW_HDMI_PHY_VENDOR_PHY = 0xfe,
> };
>
> +enum dw_hdmi_devtype {
> + RK3228_HDMI,
> + RK3288_HDMI,
> + RK3328_HDMI,
> + RK3366_HDMI,
> + RK3368_HDMI,
> + RK3399_HDMI,
> +};
> +
> struct dw_hdmi_mpll_config {
> unsigned long mpixelclock;
> struct {
> @@ -124,6 +133,7 @@ struct dw_hdmi_phy_ops {
>
> struct dw_hdmi_plat_data {
> struct regmap *regm;
> + enum dw_hdmi_devtype dev_type;
Please don't do that, dw-hdmi driver should not have knowledge of the device type,
if the vendor PHY reporting is broken, add a way to force phy_type to DW_HDMI_PHY_VENDOR_PHY.
Neil
> enum drm_mode_status (*mode_valid)(struct drm_connector *connector,
> const struct drm_display_mode *mode);
> unsigned long input_bus_format;
>
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel
