[PATCH v9 18/18] drm: bridge: samsung-dsim: Add i.MX8M Plus support

[Jagan Teki]

Add extras to support i.MX8M Plus. The main change is the removal of
HS/VS/DE signal inversion in the LCDIFv3-DSIM glue logic, otherwise
the implementation of this IP in i.MX8M Plus is very much compatible
with the i.MX8M Mini/Nano one.

Signed-off-by: Marek Vasut <ma...@denx.de>
Signed-off-by: Jagan Teki <ja...@amarulasolutions.com>
---
Changes for v9:
- added im8mp in DSIM_STATE_REINITIALIZED check

 drivers/gpu/drm/bridge/samsung-dsim.c | 26 +++++++++++++++++++++++++-
 include/drm/bridge/samsung-dsim.h     |  1 +
 2 files changed, 26 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/bridge/samsung-dsim.c 
b/drivers/gpu/drm/bridge/samsung-dsim.c
index 7ff10308a7ad..6e9ad955ebd3 100644
--- a/drivers/gpu/drm/bridge/samsung-dsim.c
+++ b/drivers/gpu/drm/bridge/samsung-dsim.c
@@ -479,6 +479,7 @@ samsung_dsim_types[SAMSUNG_DSIM_TYPE_COUNT] = {
        [SAMSUNG_DSIM_TYPE_EXYNOS5422] = &exynos5422_dsi_driver_data,
        [SAMSUNG_DSIM_TYPE_EXYNOS5433] = &exynos5433_dsi_driver_data,
        [SAMSUNG_DSIM_TYPE_IMX8MM] = &imx8mm_dsi_driver_data,
+       [SAMSUNG_DSIM_TYPE_IMX8MP] = &imx8mm_dsi_driver_data,
 };
 
 static inline struct samsung_dsim *host_to_dsi(struct mipi_dsi_host *h)
@@ -1305,7 +1306,8 @@ static int samsung_dsim_init(struct samsung_dsim *dsi, 
unsigned int flag)
         * This host reinitialization is handled via DSIM_STATE_REINITIALIZED
         * flag and triggers from host transfer. Do this exclusively for Exynos.
         */
-       if ((dsi->plat_data->hw_type == SAMSUNG_DSIM_TYPE_IMX8MM) &&
+       if ((dsi->plat_data->hw_type == SAMSUNG_DSIM_TYPE_IMX8MM ||
+           dsi->plat_data->hw_type == SAMSUNG_DSIM_TYPE_IMX8MP) &&
            dsi->state & DSIM_STATE_REINITIALIZED)
                return 0;
 
@@ -1468,10 +1470,17 @@ static int samsung_dsim_atomic_check(struct drm_bridge 
*bridge,
         * 13.6.2.7.2 RGB interface
         * both claim "Vsync, Hsync, and VDEN are active high signals.", the
         * LCDIF must generate inverted HS/VS/DE signals, i.e. active LOW.
+        *
+        * The i.MX8M Plus glue logic between LCDIFv3 and DSIM does not
+        * implement the same behavior, therefore LCDIFv3 must generate
+        * HS/VS/DE signals active HIGH.
         */
        if (dsi->plat_data->hw_type == SAMSUNG_DSIM_TYPE_IMX8MM) {
                adjusted_mode->flags |= (DRM_MODE_FLAG_NHSYNC | 
DRM_MODE_FLAG_NVSYNC);
                adjusted_mode->flags &= ~(DRM_MODE_FLAG_PHSYNC | 
DRM_MODE_FLAG_PVSYNC);
+       } else if (dsi->plat_data->hw_type == SAMSUNG_DSIM_TYPE_IMX8MP) {
+               adjusted_mode->flags &= ~(DRM_MODE_FLAG_NHSYNC | 
DRM_MODE_FLAG_NVSYNC);
+               adjusted_mode->flags |= (DRM_MODE_FLAG_PHSYNC | 
DRM_MODE_FLAG_PVSYNC);
        }
 
        return 0;
@@ -1689,6 +1698,10 @@ static const struct samsung_dsim_host_ops 
samsung_dsim_generic_host_ops = {
        .unregister_host = samsung_dsim_unregister_host,
 };
 
+static const struct drm_bridge_timings samsung_dsim_bridge_timings_de_high = {
+       .input_bus_flags = DRM_BUS_FLAG_DE_HIGH,
+};
+
 static const struct drm_bridge_timings samsung_dsim_bridge_timings_de_low = {
        .input_bus_flags = DRM_BUS_FLAG_DE_LOW,
 };
@@ -1778,6 +1791,8 @@ int samsung_dsim_probe(struct platform_device *pdev)
        /* DE_LOW: i.MX8M Mini/Nano LCDIF-DSIM glue logic inverts HS/VS/DE */
        if (dsi->plat_data->hw_type == SAMSUNG_DSIM_TYPE_IMX8MM)
                dsi->bridge.timings = &samsung_dsim_bridge_timings_de_low;
+       else
+               dsi->bridge.timings = &samsung_dsim_bridge_timings_de_high;
 
        if (dsi->plat_data->host_ops && dsi->plat_data->host_ops->register_host)
                ret = dsi->plat_data->host_ops->register_host(dsi);
@@ -1883,11 +1898,20 @@ static const struct samsung_dsim_plat_data 
samsung_dsim_imx8mm_pdata = {
        .host_ops = &samsung_dsim_generic_host_ops,
 };
 
+static const struct samsung_dsim_plat_data samsung_dsim_imx8mp_pdata = {
+       .hw_type = SAMSUNG_DSIM_TYPE_IMX8MP,
+       .host_ops = &samsung_dsim_generic_host_ops,
+};
+
 static const struct of_device_id samsung_dsim_of_match[] = {
        {
                .compatible = "fsl,imx8mm-mipi-dsim",
                .data = &samsung_dsim_imx8mm_pdata,
        },
+       {
+               .compatible = "fsl,imx8mp-mipi-dsim",
+               .data = &samsung_dsim_imx8mp_pdata,
+       },
        { /* sentinel. */ }
 };
 MODULE_DEVICE_TABLE(of, samsung_dsim_of_match);
diff --git a/include/drm/bridge/samsung-dsim.h 
b/include/drm/bridge/samsung-dsim.h
index df3d030daec6..3789f9dbb238 100644
--- a/include/drm/bridge/samsung-dsim.h
+++ b/include/drm/bridge/samsung-dsim.h
@@ -28,6 +28,7 @@ enum samsung_dsim_type {
        SAMSUNG_DSIM_TYPE_EXYNOS5422,
        SAMSUNG_DSIM_TYPE_EXYNOS5433,
        SAMSUNG_DSIM_TYPE_IMX8MM,
+       SAMSUNG_DSIM_TYPE_IMX8MP,
        SAMSUNG_DSIM_TYPE_COUNT,
 };
 
-- 
2.25.1