[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20200812083603.4320-1-algea.cao@rock-chips.com>
Date: Wed, 12 Aug 2020 16:36:03 +0800
From: Algea Cao <algea.cao@...k-chips.com>
To: a.hajda@...sung.com, kuankuan.y@...il.com, hjc@...k-chips.com,
tzimmermann@...e.de, dri-devel@...ts.freedesktop.org,
sam@...nborg.org, airlied@...ux.ie, heiko@...ech.de,
jernej.skrabec@...l.net, algea.cao@...k-chips.com,
Laurent.pinchart@...asonboard.com,
laurent.pinchart+renesas@...asonboard.com, jonas@...boo.se,
mripard@...nel.org, darekm@...gle.com,
linux-rockchip@...ts.infradead.org,
linux-arm-kernel@...ts.infradead.org, cychiang@...omium.org,
linux-kernel@...r.kernel.org, narmstrong@...libre.com,
jbrunet@...libre.com, maarten.lankhorst@...ux.intel.com,
daniel@...ll.ch
Subject: [PATCH 5/6] drm/rockchip: dw_hdmi: Add get_output_bus_format
Add get_output_bus_format in dw_hdmi_plat_data to get
hdmi output bus format. The output bus format is determined
by color format and depth, which can be set by rockchip
hdmi properties.
Signed-off-by: Algea Cao <algea.cao@...k-chips.com>
---
drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c | 47 +++++++++++++++++++++
include/drm/bridge/dw_hdmi.h | 1 +
2 files changed, 48 insertions(+)
diff --git a/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c b/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c
index 8f22d9a566db..a602d25639a7 100644
--- a/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c
+++ b/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c
@@ -100,6 +100,7 @@ struct rockchip_hdmi {
unsigned int colordepth;
enum drm_hdmi_output_type hdmi_output;
+ unsigned long output_bus_format;
};
#define to_rockchip_hdmi(x) container_of(x, struct rockchip_hdmi, x)
@@ -498,6 +499,50 @@ static const struct dw_hdmi_property_ops dw_hdmi_rockchip_property_ops = {
.get_property = dw_hdmi_rockchip_get_property,
};
+static void
+dw_hdmi_rockchip_output_bus_format_select(struct rockchip_hdmi *hdmi)
+{
+ unsigned long bus_format;
+
+ if (hdmi->hdmi_output == DRM_HDMI_OUTPUT_YCBCR420) {
+ if (hdmi->colordepth > 8)
+ bus_format = MEDIA_BUS_FMT_UYYVYY10_0_5X30;
+ else
+ bus_format = MEDIA_BUS_FMT_UYYVYY8_0_5X24;
+ } else if (hdmi->hdmi_output == DRM_HDMI_OUTPUT_YCBCR422) {
+ if (hdmi->colordepth == 12)
+ bus_format = MEDIA_BUS_FMT_UYVY12_1X24;
+ else if (hdmi->colordepth == 10)
+ bus_format = MEDIA_BUS_FMT_UYVY10_1X20;
+ else
+ bus_format = MEDIA_BUS_FMT_UYVY8_1X16;
+ } else {
+ if (hdmi->colordepth > 8) {
+ if (hdmi->hdmi_output != DRM_HDMI_OUTPUT_DEFAULT_RGB)
+ bus_format = MEDIA_BUS_FMT_YUV10_1X30;
+ else
+ bus_format = MEDIA_BUS_FMT_RGB101010_1X30;
+ } else {
+ if (hdmi->hdmi_output != DRM_HDMI_OUTPUT_DEFAULT_RGB)
+ bus_format = MEDIA_BUS_FMT_YUV8_1X24;
+ else
+ bus_format = MEDIA_BUS_FMT_RGB888_1X24;
+ }
+ }
+
+ hdmi->output_bus_format = bus_format;
+}
+
+static unsigned long
+dw_hdmi_rockchip_get_output_bus_format(void *data)
+{
+ struct rockchip_hdmi *hdmi = (struct rockchip_hdmi *)data;
+
+ dw_hdmi_rockchip_output_bus_format_select(hdmi);
+
+ return hdmi->output_bus_format;
+}
+
static void dw_hdmi_rk3228_setup_hpd(struct dw_hdmi *dw_hdmi, void *data)
{
struct rockchip_hdmi *hdmi = (struct rockchip_hdmi *)data;
@@ -685,6 +730,8 @@ static int dw_hdmi_rockchip_bind(struct device *dev, struct device *master,
plat_data->property_ops = &dw_hdmi_rockchip_property_ops;
+ plat_data->get_output_bus_format =
+ dw_hdmi_rockchip_get_output_bus_format;
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 dc561ebe7a9b..13495f810328 100644
--- a/include/drm/bridge/dw_hdmi.h
+++ b/include/drm/bridge/dw_hdmi.h
@@ -175,6 +175,7 @@ struct dw_hdmi_plat_data {
const struct dw_hdmi_phy_config *phy_config;
int (*configure_phy)(struct dw_hdmi *hdmi, void *data,
unsigned long mpixelclock);
+ unsigned long (*get_output_bus_format)(void *data);
};
struct dw_hdmi *dw_hdmi_probe(struct platform_device *pdev,
--
2.25.1
Powered by blists - more mailing lists