Hi,
On 3/2/2017 4:17 PM, Laurent Pinchart wrote:
The LVDS encoder driver is a DRM bridge driver that supports the parallel to LVDS encoders that don't require any configuration. The driver thus doesn't interact with the device, but creates an LVDS connector for the panel and exposes its size and timing based on information retrieved from DT.
Signed-off-by: Laurent Pinchart laurent.pinchart+renesas@ideasonboard.com Acked-by: Daniel Vetter daniel@ffwll.ch
Changes since v3:
- Dropped encoder_type
- Added a Kconfig dependency on DRM_PANEL
drivers/gpu/drm/bridge/Kconfig | 9 ++ drivers/gpu/drm/bridge/Makefile | 1 + drivers/gpu/drm/bridge/lvds-encoder.c | 209 ++++++++++++++++++++++++++++++++++ 3 files changed, 219 insertions(+) create mode 100644 drivers/gpu/drm/bridge/lvds-encoder.c
<snip>
+static int lvds_encoder_remove(struct platform_device *pdev) +{
- struct lvds_encoder *encoder = platform_get_drvdata(pdev);
- drm_bridge_remove(&encoder->bridge);
- return 0;
+}
+static const struct of_device_id lvds_encoder_match[] = {
- { .compatible = "lvds-encoder" },
- {},
+}; +MODULE_DEVICE_TABLE(of, lvds_encoder_match);
+struct platform_driver lvds_encoder_driver = {
I changed this to static to fix a sparse warning. Pushed this and others in this set to drm-misc-next.
Thanks, Archit
- .probe = lvds_encoder_probe,
- .remove = lvds_encoder_remove,
- .driver = {
.name = "lvds-encoder",.of_match_table = lvds_encoder_match,- },
+}; +module_platform_driver(lvds_encoder_driver);
+MODULE_AUTHOR("Laurent Pinchart laurent.pinchart@ideasonboard.com"); +MODULE_DESCRIPTION("Transparent parallel to LVDS encoder"); +MODULE_LICENSE("GPL");