diff options
| -rw-r--r-- | drivers/gpu/drm/vc4/vc4_dpi.c | 25 |
1 files changed, 22 insertions, 3 deletions
diff --git a/drivers/gpu/drm/vc4/vc4_dpi.c b/drivers/gpu/drm/vc4/vc4_dpi.c index 72c9dbd81d7f..f185812970da 100644 --- a/drivers/gpu/drm/vc4/vc4_dpi.c +++ b/drivers/gpu/drm/vc4/vc4_dpi.c | |||
| @@ -96,7 +96,6 @@ struct vc4_dpi { | |||
| 96 | struct platform_device *pdev; | 96 | struct platform_device *pdev; |
| 97 | 97 | ||
| 98 | struct drm_encoder *encoder; | 98 | struct drm_encoder *encoder; |
| 99 | struct drm_connector *connector; | ||
| 100 | 99 | ||
| 101 | void __iomem *regs; | 100 | void __iomem *regs; |
| 102 | 101 | ||
| @@ -164,14 +163,31 @@ static void vc4_dpi_encoder_disable(struct drm_encoder *encoder) | |||
| 164 | 163 | ||
| 165 | static void vc4_dpi_encoder_enable(struct drm_encoder *encoder) | 164 | static void vc4_dpi_encoder_enable(struct drm_encoder *encoder) |
| 166 | { | 165 | { |
| 166 | struct drm_device *dev = encoder->dev; | ||
| 167 | struct drm_display_mode *mode = &encoder->crtc->mode; | 167 | struct drm_display_mode *mode = &encoder->crtc->mode; |
| 168 | struct vc4_dpi_encoder *vc4_encoder = to_vc4_dpi_encoder(encoder); | 168 | struct vc4_dpi_encoder *vc4_encoder = to_vc4_dpi_encoder(encoder); |
| 169 | struct vc4_dpi *dpi = vc4_encoder->dpi; | 169 | struct vc4_dpi *dpi = vc4_encoder->dpi; |
| 170 | struct drm_connector_list_iter conn_iter; | ||
| 171 | struct drm_connector *connector = NULL, *connector_scan; | ||
| 170 | u32 dpi_c = DPI_ENABLE | DPI_OUTPUT_ENABLE_MODE; | 172 | u32 dpi_c = DPI_ENABLE | DPI_OUTPUT_ENABLE_MODE; |
| 171 | int ret; | 173 | int ret; |
| 172 | 174 | ||
| 173 | if (dpi->connector->display_info.num_bus_formats) { | 175 | /* Look up the connector attached to DPI so we can get the |
| 174 | u32 bus_format = dpi->connector->display_info.bus_formats[0]; | 176 | * bus_format. Ideally the bridge would tell us the |
| 177 | * bus_format we want, but it doesn't yet, so assume that it's | ||
| 178 | * uniform throughout the bridge chain. | ||
| 179 | */ | ||
| 180 | drm_connector_list_iter_begin(dev, &conn_iter); | ||
| 181 | drm_for_each_connector_iter(connector_scan, &conn_iter) { | ||
| 182 | if (connector_scan->encoder == encoder) { | ||
| 183 | connector = connector_scan; | ||
| 184 | break; | ||
| 185 | } | ||
| 186 | } | ||
| 187 | drm_connector_list_iter_end(&conn_iter); | ||
| 188 | |||
| 189 | if (connector && connector->display_info.num_bus_formats) { | ||
| 190 | u32 bus_format = connector->display_info.bus_formats[0]; | ||
| 175 | 191 | ||
| 176 | switch (bus_format) { | 192 | switch (bus_format) { |
| 177 | case MEDIA_BUS_FMT_RGB888_1X24: | 193 | case MEDIA_BUS_FMT_RGB888_1X24: |
| @@ -199,6 +215,9 @@ static void vc4_dpi_encoder_enable(struct drm_encoder *encoder) | |||
| 199 | DRM_ERROR("Unknown media bus format %d\n", bus_format); | 215 | DRM_ERROR("Unknown media bus format %d\n", bus_format); |
| 200 | break; | 216 | break; |
| 201 | } | 217 | } |
| 218 | } else { | ||
| 219 | /* Default to 24bit if no connector found. */ | ||
| 220 | dpi_c |= VC4_SET_FIELD(DPI_FORMAT_24BIT_888_RGB, DPI_FORMAT); | ||
| 202 | } | 221 | } |
| 203 | 222 | ||
| 204 | if (mode->flags & DRM_MODE_FLAG_NHSYNC) | 223 | if (mode->flags & DRM_MODE_FLAG_NHSYNC) |
