Modify atomic_get_input_bus_format function to return input formats
based on the output format instead of using hardcoded value.

Signed-off-by: Yuti Amonkar <yamon...@cadence.com>
---
 .../drm/bridge/cadence/cdns-mhdp8546-core.c   | 110 ++++++++++++++++--
 1 file changed, 100 insertions(+), 10 deletions(-)

diff --git a/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c 
b/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c
index bdb0d95aa412..623eadb8948f 100644
--- a/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c
+++ b/drivers/gpu/drm/bridge/cadence/cdns-mhdp8546-core.c
@@ -2078,27 +2078,117 @@ cdns_mhdp_bridge_atomic_reset(struct drm_bridge 
*bridge)
        return &cdns_mhdp_state->base;
 }
 
+static const u32 cdns_mhdp_bus_fmts[] = {
+       MEDIA_BUS_FMT_YUV16_1X48,
+       MEDIA_BUS_FMT_RGB161616_1X48,
+       MEDIA_BUS_FMT_UYVY12_1X24,
+       MEDIA_BUS_FMT_YUV12_1X36,
+       MEDIA_BUS_FMT_RGB121212_1X36,
+       MEDIA_BUS_FMT_UYVY10_1X20,
+       MEDIA_BUS_FMT_YUV10_1X30,
+       MEDIA_BUS_FMT_RGB101010_1X30,
+       MEDIA_BUS_FMT_UYVY8_1X16,
+       MEDIA_BUS_FMT_YUV8_1X24,
+       MEDIA_BUS_FMT_RGB888_1X24
+};
+
+static bool cdns_mhdp_format_supported(u32 output_fmt)
+{
+       unsigned int i;
+
+       for (i = 0; i < ARRAY_SIZE(cdns_mhdp_bus_fmts); i++) {
+               if (output_fmt == cdns_mhdp_bus_fmts[i])
+                       return true;
+       }
+
+       return false;
+}
+
+#define MAX_INPUT_FORMAT 4
+
 static u32 *cdns_mhdp_get_input_bus_fmts(struct drm_bridge *bridge,
-                                     struct drm_bridge_state *bridge_state,
-                                     struct drm_crtc_state *crtc_state,
-                                     struct drm_connector_state *conn_state,
-                                     u32 output_fmt,
-                                     unsigned int *num_input_fmts)
+                                        struct drm_bridge_state *bridge_state,
+                                        struct drm_crtc_state *crtc_state,
+                                        struct drm_connector_state *conn_state,
+                                        u32 output_fmt,
+                                        unsigned int *num_input_fmts)
 {
        u32 *input_fmts;
-       u32 default_bus_format = MEDIA_BUS_FMT_RGB121212_1X36;
+       unsigned int i = 0;
 
        *num_input_fmts = 0;
 
-       if (output_fmt != MEDIA_BUS_FMT_FIXED)
+       if (!cdns_mhdp_format_supported(output_fmt) &&
+           output_fmt != MEDIA_BUS_FMT_FIXED)
                return NULL;
 
-       input_fmts = kzalloc(sizeof(*input_fmts), GFP_KERNEL);
+       input_fmts = kcalloc(MAX_INPUT_FORMAT,
+                            sizeof(*input_fmts), GFP_KERNEL);
        if (!input_fmts)
                return NULL;
 
-       *num_input_fmts = 1;
-       input_fmts[0] = default_bus_format;
+       switch (output_fmt) {
+       /* RGB */
+       case MEDIA_BUS_FMT_RGB161616_1X48:
+               input_fmts[i++] = MEDIA_BUS_FMT_RGB161616_1X48;
+               input_fmts[i++] = MEDIA_BUS_FMT_RGB121212_1X36;
+               input_fmts[i++] = MEDIA_BUS_FMT_RGB101010_1X30;
+               input_fmts[i++] = MEDIA_BUS_FMT_RGB888_1X24;
+               break;
+       case MEDIA_BUS_FMT_RGB121212_1X36:
+               input_fmts[i++] = MEDIA_BUS_FMT_RGB121212_1X36;
+               input_fmts[i++] = MEDIA_BUS_FMT_RGB101010_1X30;
+               input_fmts[i++] = MEDIA_BUS_FMT_RGB888_1X24;
+               break;
+       case MEDIA_BUS_FMT_RGB101010_1X30:
+               input_fmts[i++] = MEDIA_BUS_FMT_RGB101010_1X30;
+               input_fmts[i++] = MEDIA_BUS_FMT_RGB888_1X24;
+               break;
+       case MEDIA_BUS_FMT_RGB888_1X24:
+               input_fmts[i++] = MEDIA_BUS_FMT_RGB888_1X24;
+               break;
+
+       /* YUV444 */
+       case MEDIA_BUS_FMT_YUV16_1X48:
+               input_fmts[i++] = MEDIA_BUS_FMT_YUV16_1X48;
+               input_fmts[i++] = MEDIA_BUS_FMT_YUV12_1X36;
+               input_fmts[i++] = MEDIA_BUS_FMT_YUV10_1X30;
+               input_fmts[i++] = MEDIA_BUS_FMT_YUV8_1X24;
+               break;
+       case MEDIA_BUS_FMT_YUV12_1X36:
+               input_fmts[i++] = MEDIA_BUS_FMT_YUV12_1X36;
+               input_fmts[i++] = MEDIA_BUS_FMT_YUV10_1X30;
+               input_fmts[i++] = MEDIA_BUS_FMT_YUV8_1X24;
+               break;
+       case MEDIA_BUS_FMT_YUV10_1X30:
+               input_fmts[i++] = MEDIA_BUS_FMT_YUV10_1X30;
+               input_fmts[i++] = MEDIA_BUS_FMT_YUV8_1X24;
+               break;
+       case MEDIA_BUS_FMT_YUV8_1X24:
+               input_fmts[i++] = MEDIA_BUS_FMT_YUV8_1X24;
+               break;
+
+       /* YUV422 */
+       case MEDIA_BUS_FMT_UYVY12_1X24:
+               input_fmts[i++] = MEDIA_BUS_FMT_UYVY12_1X24;
+               input_fmts[i++] = MEDIA_BUS_FMT_UYVY10_1X20;
+               input_fmts[i++] = MEDIA_BUS_FMT_UYVY8_1X16;
+               break;
+       case MEDIA_BUS_FMT_UYVY10_1X20:
+               input_fmts[i++] = MEDIA_BUS_FMT_UYVY10_1X20;
+               input_fmts[i++] = MEDIA_BUS_FMT_UYVY8_1X16;
+               break;
+       case MEDIA_BUS_FMT_UYVY8_1X16:
+               input_fmts[i++] = MEDIA_BUS_FMT_UYVY8_1X16;
+               break;
+
+       case MEDIA_BUS_FMT_FIXED:
+               input_fmts[i++] = MEDIA_BUS_FMT_RGB121212_1X36;
+               break;
+       }
+
+       *num_input_fmts = i;
+
        return input_fmts;
 }
 
-- 
2.17.1

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to