| .. | .. |
|---|
| 1 | +// SPDX-License-Identifier: GPL-2.0-only |
|---|
| 1 | 2 | /* |
|---|
| 2 | 3 | * Copyright (C) 2015 Texas Instruments |
|---|
| 3 | 4 | * Author: Jyri Sarha <jsarha@ti.com> |
|---|
| 4 | | - * |
|---|
| 5 | | - * This program is free software; you can redistribute it and/or modify it |
|---|
| 6 | | - * under the terms of the GNU General Public License version 2 as published by |
|---|
| 7 | | - * the Free Software Foundation. |
|---|
| 8 | | - * |
|---|
| 9 | 5 | */ |
|---|
| 10 | 6 | |
|---|
| 11 | 7 | #include <linux/component.h> |
|---|
| 12 | 8 | #include <linux/of_graph.h> |
|---|
| 9 | + |
|---|
| 10 | +#include <drm/drm_atomic_helper.h> |
|---|
| 11 | +#include <drm/drm_bridge.h> |
|---|
| 13 | 12 | #include <drm/drm_of.h> |
|---|
| 13 | +#include <drm/drm_simple_kms_helper.h> |
|---|
| 14 | 14 | |
|---|
| 15 | 15 | #include "tilcdc_drv.h" |
|---|
| 16 | 16 | #include "tilcdc_external.h" |
|---|
| .. | .. |
|---|
| 40 | 40 | .raster_order = 0, |
|---|
| 41 | 41 | }; |
|---|
| 42 | 42 | |
|---|
| 43 | | -static int tilcdc_external_mode_valid(struct drm_connector *connector, |
|---|
| 44 | | - struct drm_display_mode *mode) |
|---|
| 45 | | -{ |
|---|
| 46 | | - struct tilcdc_drm_private *priv = connector->dev->dev_private; |
|---|
| 47 | | - int ret; |
|---|
| 48 | | - |
|---|
| 49 | | - ret = tilcdc_crtc_mode_valid(priv->crtc, mode); |
|---|
| 50 | | - if (ret != MODE_OK) |
|---|
| 51 | | - return ret; |
|---|
| 52 | | - |
|---|
| 53 | | - BUG_ON(priv->external_connector != connector); |
|---|
| 54 | | - BUG_ON(!priv->connector_funcs); |
|---|
| 55 | | - |
|---|
| 56 | | - /* If the connector has its own mode_valid call it. */ |
|---|
| 57 | | - if (!IS_ERR(priv->connector_funcs) && |
|---|
| 58 | | - priv->connector_funcs->mode_valid) |
|---|
| 59 | | - return priv->connector_funcs->mode_valid(connector, mode); |
|---|
| 60 | | - |
|---|
| 61 | | - return MODE_OK; |
|---|
| 62 | | -} |
|---|
| 63 | | - |
|---|
| 64 | | -static int tilcdc_add_external_connector(struct drm_device *dev, |
|---|
| 65 | | - struct drm_connector *connector) |
|---|
| 66 | | -{ |
|---|
| 67 | | - struct tilcdc_drm_private *priv = dev->dev_private; |
|---|
| 68 | | - struct drm_connector_helper_funcs *connector_funcs; |
|---|
| 69 | | - |
|---|
| 70 | | - /* There should never be more than one connector */ |
|---|
| 71 | | - if (WARN_ON(priv->external_connector)) |
|---|
| 72 | | - return -EINVAL; |
|---|
| 73 | | - |
|---|
| 74 | | - priv->external_connector = connector; |
|---|
| 75 | | - connector_funcs = devm_kzalloc(dev->dev, sizeof(*connector_funcs), |
|---|
| 76 | | - GFP_KERNEL); |
|---|
| 77 | | - if (!connector_funcs) |
|---|
| 78 | | - return -ENOMEM; |
|---|
| 79 | | - |
|---|
| 80 | | - /* connector->helper_private contains always struct |
|---|
| 81 | | - * connector_helper_funcs pointer. For tilcdc crtc to have a |
|---|
| 82 | | - * say if a specific mode is Ok, we need to install our own |
|---|
| 83 | | - * helper functions. In our helper functions we copy |
|---|
| 84 | | - * everything else but use our own mode_valid() (above). |
|---|
| 85 | | - */ |
|---|
| 86 | | - if (connector->helper_private) { |
|---|
| 87 | | - priv->connector_funcs = connector->helper_private; |
|---|
| 88 | | - *connector_funcs = *priv->connector_funcs; |
|---|
| 89 | | - } else { |
|---|
| 90 | | - priv->connector_funcs = ERR_PTR(-ENOENT); |
|---|
| 91 | | - } |
|---|
| 92 | | - connector_funcs->mode_valid = tilcdc_external_mode_valid; |
|---|
| 93 | | - drm_connector_helper_add(connector, connector_funcs); |
|---|
| 94 | | - |
|---|
| 95 | | - dev_dbg(dev->dev, "External connector '%s' connected\n", |
|---|
| 96 | | - connector->name); |
|---|
| 97 | | - |
|---|
| 98 | | - return 0; |
|---|
| 99 | | -} |
|---|
| 100 | | - |
|---|
| 101 | 43 | static |
|---|
| 102 | 44 | struct drm_connector *tilcdc_encoder_find_connector(struct drm_device *ddev, |
|---|
| 103 | 45 | struct drm_encoder *encoder) |
|---|
| .. | .. |
|---|
| 118 | 60 | int tilcdc_add_component_encoder(struct drm_device *ddev) |
|---|
| 119 | 61 | { |
|---|
| 120 | 62 | struct tilcdc_drm_private *priv = ddev->dev_private; |
|---|
| 121 | | - struct drm_connector *connector; |
|---|
| 122 | | - struct drm_encoder *encoder; |
|---|
| 63 | + struct drm_encoder *encoder = NULL, *iter; |
|---|
| 123 | 64 | |
|---|
| 124 | | - list_for_each_entry(encoder, &ddev->mode_config.encoder_list, head) |
|---|
| 125 | | - if (encoder->possible_crtcs & (1 << priv->crtc->index)) |
|---|
| 65 | + list_for_each_entry(iter, &ddev->mode_config.encoder_list, head) |
|---|
| 66 | + if (iter->possible_crtcs & (1 << priv->crtc->index)) { |
|---|
| 67 | + encoder = iter; |
|---|
| 126 | 68 | break; |
|---|
| 69 | + } |
|---|
| 127 | 70 | |
|---|
| 128 | 71 | if (!encoder) { |
|---|
| 129 | 72 | dev_err(ddev->dev, "%s: No suitable encoder found\n", __func__); |
|---|
| 130 | 73 | return -ENODEV; |
|---|
| 131 | 74 | } |
|---|
| 132 | 75 | |
|---|
| 133 | | - connector = tilcdc_encoder_find_connector(ddev, encoder); |
|---|
| 76 | + priv->external_connector = |
|---|
| 77 | + tilcdc_encoder_find_connector(ddev, encoder); |
|---|
| 134 | 78 | |
|---|
| 135 | | - if (!connector) |
|---|
| 79 | + if (!priv->external_connector) |
|---|
| 136 | 80 | return -ENODEV; |
|---|
| 137 | 81 | |
|---|
| 138 | 82 | /* Only tda998x is supported at the moment. */ |
|---|
| 139 | 83 | tilcdc_crtc_set_simulate_vesa_sync(priv->crtc, true); |
|---|
| 140 | 84 | tilcdc_crtc_set_panel_info(priv->crtc, &panel_info_tda998x); |
|---|
| 141 | 85 | |
|---|
| 142 | | - return tilcdc_add_external_connector(ddev, connector); |
|---|
| 86 | + return 0; |
|---|
| 143 | 87 | } |
|---|
| 144 | | - |
|---|
| 145 | | -void tilcdc_remove_external_device(struct drm_device *dev) |
|---|
| 146 | | -{ |
|---|
| 147 | | - struct tilcdc_drm_private *priv = dev->dev_private; |
|---|
| 148 | | - |
|---|
| 149 | | - /* Restore the original helper functions, if any. */ |
|---|
| 150 | | - if (IS_ERR(priv->connector_funcs)) |
|---|
| 151 | | - drm_connector_helper_add(priv->external_connector, NULL); |
|---|
| 152 | | - else if (priv->connector_funcs) |
|---|
| 153 | | - drm_connector_helper_add(priv->external_connector, |
|---|
| 154 | | - priv->connector_funcs); |
|---|
| 155 | | -} |
|---|
| 156 | | - |
|---|
| 157 | | -static const struct drm_encoder_funcs tilcdc_external_encoder_funcs = { |
|---|
| 158 | | - .destroy = drm_encoder_cleanup, |
|---|
| 159 | | -}; |
|---|
| 160 | 88 | |
|---|
| 161 | 89 | static |
|---|
| 162 | 90 | int tilcdc_attach_bridge(struct drm_device *ddev, struct drm_bridge *bridge) |
|---|
| 163 | 91 | { |
|---|
| 164 | 92 | struct tilcdc_drm_private *priv = ddev->dev_private; |
|---|
| 165 | | - struct drm_connector *connector; |
|---|
| 166 | 93 | int ret; |
|---|
| 167 | 94 | |
|---|
| 168 | 95 | priv->external_encoder->possible_crtcs = BIT(0); |
|---|
| 169 | 96 | |
|---|
| 170 | | - ret = drm_bridge_attach(priv->external_encoder, bridge, NULL); |
|---|
| 97 | + ret = drm_bridge_attach(priv->external_encoder, bridge, NULL, 0); |
|---|
| 171 | 98 | if (ret) { |
|---|
| 172 | 99 | dev_err(ddev->dev, "drm_bridge_attach() failed %d\n", ret); |
|---|
| 173 | 100 | return ret; |
|---|
| .. | .. |
|---|
| 175 | 102 | |
|---|
| 176 | 103 | tilcdc_crtc_set_panel_info(priv->crtc, &panel_info_default); |
|---|
| 177 | 104 | |
|---|
| 178 | | - connector = tilcdc_encoder_find_connector(ddev, priv->external_encoder); |
|---|
| 179 | | - if (!connector) |
|---|
| 105 | + priv->external_connector = |
|---|
| 106 | + tilcdc_encoder_find_connector(ddev, priv->external_encoder); |
|---|
| 107 | + if (!priv->external_connector) |
|---|
| 180 | 108 | return -ENODEV; |
|---|
| 181 | 109 | |
|---|
| 182 | | - ret = tilcdc_add_external_connector(ddev, connector); |
|---|
| 183 | | - |
|---|
| 184 | | - return ret; |
|---|
| 110 | + return 0; |
|---|
| 185 | 111 | } |
|---|
| 186 | 112 | |
|---|
| 187 | 113 | int tilcdc_attach_external_device(struct drm_device *ddev) |
|---|
| .. | .. |
|---|
| 204 | 130 | if (!priv->external_encoder) |
|---|
| 205 | 131 | return -ENOMEM; |
|---|
| 206 | 132 | |
|---|
| 207 | | - ret = drm_encoder_init(ddev, priv->external_encoder, |
|---|
| 208 | | - &tilcdc_external_encoder_funcs, |
|---|
| 209 | | - DRM_MODE_ENCODER_NONE, NULL); |
|---|
| 133 | + ret = drm_simple_encoder_init(ddev, priv->external_encoder, |
|---|
| 134 | + DRM_MODE_ENCODER_NONE); |
|---|
| 210 | 135 | if (ret) { |
|---|
| 211 | 136 | dev_err(ddev->dev, "drm_encoder_init() failed %d\n", ret); |
|---|
| 212 | 137 | return ret; |
|---|
| 213 | 138 | } |
|---|
| 214 | 139 | |
|---|
| 215 | 140 | if (panel) { |
|---|
| 216 | | - bridge = devm_drm_panel_bridge_add(ddev->dev, panel, |
|---|
| 217 | | - DRM_MODE_CONNECTOR_DPI); |
|---|
| 141 | + bridge = devm_drm_panel_bridge_add_typed(ddev->dev, panel, |
|---|
| 142 | + DRM_MODE_CONNECTOR_DPI); |
|---|
| 218 | 143 | if (IS_ERR(bridge)) { |
|---|
| 219 | 144 | ret = PTR_ERR(bridge); |
|---|
| 220 | 145 | goto err_encoder_cleanup; |
|---|