* [PATCH] video: fsl-ldb: add ported driver from linux kernel
@ 2025-11-11 9:12 Michael Grzeschik
0 siblings, 0 replies; only message in thread
From: Michael Grzeschik @ 2025-11-11 9:12 UTC (permalink / raw)
To: Sascha Hauer, BAREBOX
Import the driver from Linux and change the interface from drm to vpl.
This has been tested on the IMX93.
Signed-off-by: Michael Grzeschik <m.grzeschik@pengutronix.de>
---
drivers/video/Kconfig | 8 ++
drivers/video/Makefile | 1 +
drivers/video/fsl-ldb.c | 311 ++++++++++++++++++++++++++++++++++++++++++++++++
3 files changed, 320 insertions(+)
diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig
index b2eccd5db7fe05999f3fe64db214fb569b3fb1df..e1b5d862234d6489a02cdb51fe7f68b52a146d6d 100644
--- a/drivers/video/Kconfig
+++ b/drivers/video/Kconfig
@@ -242,4 +242,12 @@ config DRIVER_VIDEO_DW_MIPI_DSI
controller. To make use of this driver you need platform specific
glue code to be selected as well.
+config DRIVER_VIDEO_FSL_LDB
+ tristate "DRM support for FSL_LDB DPI-to-LVDS controller"
+ depends on OFTREE
+ select VIDEO_VPL
+ help
+ Say Y here if you want to enable support for the i.MX6SX/8MP/93
+ DPI-to-LVDS on-SoC encoder.
+
endif
diff --git a/drivers/video/Makefile b/drivers/video/Makefile
index 470a5abaa450b6477b7cf5a78b2a747f9a3ec743..e27aad87f98842411484a690a0e5f75fe2867278 100644
--- a/drivers/video/Makefile
+++ b/drivers/video/Makefile
@@ -38,3 +38,4 @@ obj-$(CONFIG_DRIVER_VIDEO_DW_HDMI) += dw-hdmi.o
obj-$(CONFIG_DRIVER_VIDEO_DW_MIPI_DSI) += dw_mipi_dsi.o
obj-$(CONFIG_DRIVER_VIDEO_BOCHS) += bochs/
obj-$(CONFIG_DRIVER_VIDEO_ROCKCHIP) += rockchip/
+obj-$(CONFIG_DRIVER_VIDEO_FSL_LDB) += fsl-ldb.o
diff --git a/drivers/video/fsl-ldb.c b/drivers/video/fsl-ldb.c
new file mode 100644
index 0000000000000000000000000000000000000000..3fca7867a2a3d6f4649d4bfb3f453feaba9fad61
--- /dev/null
+++ b/drivers/video/fsl-ldb.c
@@ -0,0 +1,311 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+/*
+ * Copyright (C) 2022 Marek Vasut <marex@denx.de>
+ *
+ * Limitations:
+ * - ldb is currently hardcoded to MEDIA_BUS_FMT_RGB888_1X7X4_SPWG
+ */
+
+#include <linux/clk.h>
+#include <video/media-bus-format.h>
+#include <mfd/syscon.h>
+#include <linux/module.h>
+#include <of.h>
+#include <of_graph.h>
+#include <regmap.h>
+
+#include <video/vpl.h>
+
+#define LDB_CTRL_CH0_ENABLE BIT(0)
+#define LDB_CTRL_CH0_DI_SELECT BIT(1)
+#define LDB_CTRL_CH1_ENABLE BIT(2)
+#define LDB_CTRL_CH1_DI_SELECT BIT(3)
+#define LDB_CTRL_SPLIT_MODE BIT(4)
+#define LDB_CTRL_CH0_DATA_WIDTH BIT(5)
+#define LDB_CTRL_CH0_BIT_MAPPING BIT(6)
+#define LDB_CTRL_CH1_DATA_WIDTH BIT(7)
+#define LDB_CTRL_CH1_BIT_MAPPING BIT(8)
+#define LDB_CTRL_DI0_VSYNC_POLARITY BIT(9)
+#define LDB_CTRL_DI1_VSYNC_POLARITY BIT(10)
+#define LDB_CTRL_REG_CH0_FIFO_RESET BIT(11)
+#define LDB_CTRL_REG_CH1_FIFO_RESET BIT(12)
+#define LDB_CTRL_ASYNC_FIFO_ENABLE BIT(24)
+#define LDB_CTRL_ASYNC_FIFO_THRESHOLD_MASK GENMASK(27, 25)
+
+#define LVDS_CTRL_CH0_EN BIT(0)
+#define LVDS_CTRL_CH1_EN BIT(1)
+/*
+ * LVDS_CTRL_LVDS_EN bit is poorly named in i.MX93 reference manual.
+ * Clear it to enable LVDS and set it to disable LVDS.
+ */
+#define LVDS_CTRL_LVDS_EN BIT(1)
+#define LVDS_CTRL_VBG_EN BIT(2)
+#define LVDS_CTRL_HS_EN BIT(3)
+#define LVDS_CTRL_PRE_EMPH_EN BIT(4)
+#define LVDS_CTRL_PRE_EMPH_ADJ(n) (((n) & 0x7) << 5)
+#define LVDS_CTRL_PRE_EMPH_ADJ_MASK GENMASK(7, 5)
+#define LVDS_CTRL_CM_ADJ(n) (((n) & 0x7) << 8)
+#define LVDS_CTRL_CM_ADJ_MASK GENMASK(10, 8)
+#define LVDS_CTRL_CC_ADJ(n) (((n) & 0x7) << 11)
+#define LVDS_CTRL_CC_ADJ_MASK GENMASK(13, 11)
+#define LVDS_CTRL_SLEW_ADJ(n) (((n) & 0x7) << 14)
+#define LVDS_CTRL_SLEW_ADJ_MASK GENMASK(16, 14)
+#define LVDS_CTRL_VBG_ADJ(n) (((n) & 0x7) << 17)
+#define LVDS_CTRL_VBG_ADJ_MASK GENMASK(19, 17)
+
+enum fsl_ldb_devtype {
+ IMX6SX_LDB,
+ IMX8MP_LDB,
+ IMX93_LDB,
+};
+
+struct fsl_ldb_devdata {
+ u32 ldb_ctrl;
+ u32 lvds_ctrl;
+ bool lvds_en_bit;
+ bool single_ctrl_reg;
+};
+
+static const struct fsl_ldb_devdata fsl_ldb_devdata[] = {
+ [IMX6SX_LDB] = {
+ .ldb_ctrl = 0x18,
+ .single_ctrl_reg = true,
+ },
+ [IMX8MP_LDB] = {
+ .ldb_ctrl = 0x5c,
+ .lvds_ctrl = 0x128,
+ },
+ [IMX93_LDB] = {
+ .ldb_ctrl = 0x20,
+ .lvds_ctrl = 0x24,
+ .lvds_en_bit = true,
+ },
+};
+
+struct fsl_ldb {
+ struct device *dev;
+ struct clk *clk;
+ struct vpl vpl;
+ struct regmap *regmap;
+ u32 bus_format;
+ const struct fsl_ldb_devdata *devdata;
+ struct display_timings *modes;
+ int output_port;
+ bool ch0_enabled;
+ bool ch1_enabled;
+};
+
+static bool fsl_ldb_is_dual(const struct fsl_ldb *fsl_ldb)
+{
+ return (fsl_ldb->ch0_enabled && fsl_ldb->ch1_enabled);
+}
+
+static unsigned long fsl_ldb_link_frequency(struct fsl_ldb *fsl_ldb, int clock)
+{
+ if (fsl_ldb_is_dual(fsl_ldb))
+ return clock * 3500;
+ else
+ return clock * 7000;
+}
+
+static void fsl_ldb_atomic_enable(struct fsl_ldb *fsl_ldb,
+ struct fb_videomode *mode)
+{
+ unsigned long configured_link_freq;
+ unsigned long requested_link_freq;
+ bool lvds_format_24bpp;
+ bool lvds_format_jeida;
+ u32 reg;
+
+ switch (fsl_ldb->bus_format) {
+ case MEDIA_BUS_FMT_RGB666_1X7X3_SPWG:
+ lvds_format_24bpp = false;
+ lvds_format_jeida = true;
+ break;
+ case MEDIA_BUS_FMT_RGB888_1X7X4_JEIDA:
+ lvds_format_24bpp = true;
+ lvds_format_jeida = true;
+ break;
+ case MEDIA_BUS_FMT_RGB888_1X7X4_SPWG:
+ lvds_format_24bpp = true;
+ lvds_format_jeida = false;
+ break;
+ default:
+ /*
+ * Some bridges still don't set the correct LVDS bus pixel
+ * format, use SPWG24 default format until those are fixed.
+ */
+ lvds_format_24bpp = true;
+ lvds_format_jeida = false;
+ dev_warn(fsl_ldb->dev,
+ "Unsupported LVDS bus format 0x%04x, please check output bridge driver. Falling back to SPWG24.\n",
+ fsl_ldb->bus_format);
+ break;
+ }
+
+ if (mode->pixclock.ps > (fsl_ldb_is_dual(fsl_ldb) ? 160000 : 80000))
+ return;
+
+ requested_link_freq = fsl_ldb_link_frequency(fsl_ldb, PICOS2KHZ(mode->pixclock));
+ clk_set_rate(fsl_ldb->clk, requested_link_freq);
+
+ configured_link_freq = clk_get_rate(fsl_ldb->clk);
+ if (configured_link_freq != requested_link_freq)
+ dev_warn(fsl_ldb->dev, "Configured LDB clock (%lu Hz) does not match requested LVDS clock: %lu Hz\n",
+ configured_link_freq,
+ requested_link_freq);
+
+ clk_prepare_enable(fsl_ldb->clk);
+
+ /* Program LDB_CTRL */
+ reg = (fsl_ldb->ch0_enabled ? LDB_CTRL_CH0_ENABLE : 0) |
+ (fsl_ldb->ch1_enabled ? LDB_CTRL_CH1_ENABLE : 0) |
+ (fsl_ldb_is_dual(fsl_ldb) ? LDB_CTRL_SPLIT_MODE : 0);
+
+ if (lvds_format_24bpp)
+ reg |= (fsl_ldb->ch0_enabled ? LDB_CTRL_CH0_DATA_WIDTH : 0) |
+ (fsl_ldb->ch1_enabled ? LDB_CTRL_CH1_DATA_WIDTH : 0);
+
+ if (lvds_format_jeida)
+ reg |= (fsl_ldb->ch0_enabled ? LDB_CTRL_CH0_BIT_MAPPING : 0) |
+ (fsl_ldb->ch1_enabled ? LDB_CTRL_CH1_BIT_MAPPING : 0);
+
+ if (mode->sync & FB_SYNC_VERT_HIGH_ACT)
+ reg |= (fsl_ldb->ch0_enabled ? LDB_CTRL_DI0_VSYNC_POLARITY : 0) |
+ (fsl_ldb->ch1_enabled ? LDB_CTRL_DI1_VSYNC_POLARITY : 0);
+
+ regmap_write(fsl_ldb->regmap, fsl_ldb->devdata->ldb_ctrl, reg);
+
+ if (fsl_ldb->devdata->single_ctrl_reg)
+ return;
+
+ /* Program LVDS_CTRL */
+ reg = LVDS_CTRL_CC_ADJ(2) | LVDS_CTRL_PRE_EMPH_EN |
+ LVDS_CTRL_PRE_EMPH_ADJ(3) | LVDS_CTRL_VBG_EN;
+ regmap_write(fsl_ldb->regmap, fsl_ldb->devdata->lvds_ctrl, reg);
+
+ /* Wait for VBG to stabilize. */
+ udelay(20);
+
+ reg |= (fsl_ldb->ch0_enabled ? LVDS_CTRL_CH0_EN : 0) |
+ (fsl_ldb->ch1_enabled ? LVDS_CTRL_CH1_EN : 0);
+
+ regmap_write(fsl_ldb->regmap, fsl_ldb->devdata->lvds_ctrl, reg);
+}
+
+static void fsl_ldb_atomic_disable(struct fsl_ldb *fsl_ldb)
+{
+ /* Stop channel(s). */
+ if (fsl_ldb->devdata->lvds_en_bit)
+ /* Set LVDS_CTRL_LVDS_EN bit to disable. */
+ regmap_write(fsl_ldb->regmap, fsl_ldb->devdata->lvds_ctrl,
+ LVDS_CTRL_LVDS_EN);
+ else
+ if (!fsl_ldb->devdata->single_ctrl_reg)
+ regmap_write(fsl_ldb->regmap, fsl_ldb->devdata->lvds_ctrl, 0);
+ regmap_write(fsl_ldb->regmap, fsl_ldb->devdata->ldb_ctrl, 0);
+
+ clk_disable_unprepare(fsl_ldb->clk);
+}
+
+static int fsl_ldb_ioctl(struct vpl *vpl, unsigned int port,
+ unsigned int cmd, void *data)
+{
+ struct fsl_ldb *fsl_ldb = container_of(vpl, struct fsl_ldb, vpl);
+
+ switch (cmd) {
+ case VPL_ENABLE:
+ break;
+ case VPL_DISABLE:
+ break;
+ case VPL_UNPREPARE:
+ fsl_ldb_atomic_disable(fsl_ldb);
+ case VPL_PREPARE:
+ fsl_ldb_atomic_enable(fsl_ldb, data);
+ case VPL_GET_VIDEOMODES:
+ if (fsl_ldb->modes) {
+ struct display_timings *timings = data;
+
+ timings->num_modes = fsl_ldb->modes->num_modes;
+ timings->modes = fsl_ldb->modes->modes;
+ dev_info(fsl_ldb->dev, "Using ldb provided timings\n");
+ return 0;
+ }
+ }
+
+ if (fsl_ldb->output_port > 0)
+ return vpl_ioctl(vpl, fsl_ldb->output_port, cmd, data);
+
+ return 0;
+}
+
+static int fsl_ldb_probe(struct device *dev)
+{
+ struct device_node *remote1, *remote2;
+ struct device_node *panel_node;
+ struct fsl_ldb *fsl_ldb;
+
+ fsl_ldb = xzalloc(sizeof(*fsl_ldb));
+ if (!fsl_ldb)
+ return -ENOMEM;
+
+ fsl_ldb->devdata = device_get_match_data(dev);
+ if (!fsl_ldb->devdata)
+ return -EINVAL;
+
+ fsl_ldb->dev = dev;
+
+ fsl_ldb->clk = clk_get(dev, "ldb");
+ if (IS_ERR(fsl_ldb->clk))
+ return PTR_ERR(fsl_ldb->clk);
+
+ fsl_ldb->regmap = syscon_node_to_regmap(dev->of_node->parent);
+ if (IS_ERR(fsl_ldb->regmap))
+ return PTR_ERR(fsl_ldb->regmap);
+
+ /* Locate the remote ports and the panel node */
+ remote1 = of_graph_get_remote_node(dev->of_node, 1, 0);
+ remote2 = of_graph_get_remote_node(dev->of_node, 2, 0);
+ panel_node = of_node_get(remote1 ? remote1 : remote2);
+ fsl_ldb->ch0_enabled = (remote1 != NULL);
+ fsl_ldb->ch1_enabled = (remote2 != NULL);
+
+ if (!fsl_ldb->ch0_enabled && !fsl_ldb->ch1_enabled)
+ return dev_err_probe(dev, -ENXIO, "No panel node found");
+
+ fsl_ldb->modes = of_get_display_timings(dev->of_node);
+
+ /* Only MEDIA_BUS_FMT_RGB888_1X7X4_SPWG is supported for now */
+ fsl_ldb->bus_format = MEDIA_BUS_FMT_RGB888_1X7X4_SPWG;
+ fsl_ldb->output_port = remote1 ? 1 : remote2 ? 2 : 0;
+
+ dev_dbg(dev, "Using %s\n",
+ fsl_ldb_is_dual(fsl_ldb) ? "dual-link mode" :
+ fsl_ldb->ch0_enabled ? "channel 0" : "channel 1");
+
+ /* Only DRM_LVDS_DUAL_LINK_ODD_EVEN_PIXELS is supported */
+
+ dev_info(dev, "LVDS channel pixel swap not supported.\n");
+
+ fsl_ldb->vpl.node = dev->of_node;
+ fsl_ldb->vpl.ioctl = &fsl_ldb_ioctl;
+ return vpl_register(&fsl_ldb->vpl);
+}
+
+static const struct of_device_id fsl_ldb_match[] = {
+ { .compatible = "fsl,imx6sx-ldb",
+ .data = &fsl_ldb_devdata[IMX6SX_LDB], },
+ { .compatible = "fsl,imx8mp-ldb",
+ .data = &fsl_ldb_devdata[IMX8MP_LDB], },
+ { .compatible = "fsl,imx93-ldb",
+ .data = &fsl_ldb_devdata[IMX93_LDB], },
+ { /* sentinel */ },
+};
+MODULE_DEVICE_TABLE(of, fsl_ldb_match);
+
+static struct driver fsl_ldb_driver = {
+ .probe = fsl_ldb_probe,
+ .name = "fsl-ldb",
+ .of_compatible = fsl_ldb_match,
+};
+device_platform_driver(fsl_ldb_driver);
---
base-commit: 0633a4db9d2351234b565b55978ce068b387eb2d
change-id: 20251111-imx-ldb-225cf3c9a4b7
Best regards,
--
Michael Grzeschik <m.grzeschik@pengutronix.de>
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2025-11-11 9:13 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-11-11 9:12 [PATCH] video: fsl-ldb: add ported driver from linux kernel Michael Grzeschik
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox