Re: [PATCH 2/2] drm/panel: Add driver for BOE RM692E5 AMOLED panel

From: Jessica Zhang
Date: Thu Sep 28 2023 - 18:01:13 EST

Next message: Qais Yousef: "Re: [PATCH 2/4] sched: cpufreq: Fix apply_dvfs_headroom() escaping uclamp constraints"
Previous message: Bjorn Helgaas: "Re: [PATCH 0/5] Add the pci_get_base_class() helper and use it"
In reply to: Konrad Dybcio: "[PATCH 2/2] drm/panel: Add driver for BOE RM692E5 AMOLED panel"
Next in thread: Konrad Dybcio: "Re: [PATCH 2/2] drm/panel: Add driver for BOE RM692E5 AMOLED panel"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Konrad,

On 9/27/2023 6:19 AM, Konrad Dybcio wrote:

Add support for the 2700x1224 AMOLED BOE panel bundled with a RM692E5
driver IC, as found on the Fairphone 5 smartphone.

Co-developed-by: Luca Weiss <luca.weiss@xxxxxxxxxxxxx>
Signed-off-by: Luca Weiss <luca.weiss@xxxxxxxxxxxxx>
Signed-off-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxx>
---
drivers/gpu/drm/panel/Kconfig | 9 +
drivers/gpu/drm/panel/Makefile | 1 +
drivers/gpu/drm/panel/panel-raydium-rm692e5.c | 496 ++++++++++++++++++++++++++
3 files changed, 506 insertions(+)

diff --git a/drivers/gpu/drm/panel/Kconfig b/drivers/gpu/drm/panel/Kconfig
index 2d6d96ee3547..ecb22ea326cb 100644
--- a/drivers/gpu/drm/panel/Kconfig
+++ b/drivers/gpu/drm/panel/Kconfig
@@ -516,6 +516,15 @@ config DRM_PANEL_RAYDIUM_RM68200
Say Y here if you want to enable support for Raydium RM68200
720x1280 DSI video mode panel.
+config DRM_PANEL_RAYDIUM_RM692E5
+ tristate "Raydium RM692E5-based DSI panel"
+ depends on OF
+ depends on DRM_MIPI_DSI
+ depends on BACKLIGHT_CLASS_DEVICE
+ help
+ Say Y here if you want to enable support for Raydium RM692E5-based
+ display panels, such as the one found in the Fairphone 5 smartphone.
+
config DRM_PANEL_RONBO_RB070D30
tristate "Ronbo Electronics RB070D30 panel"
depends on OF
diff --git a/drivers/gpu/drm/panel/Makefile b/drivers/gpu/drm/panel/Makefile
index 157c77ff157f..e14ce55a0875 100644
--- a/drivers/gpu/drm/panel/Makefile
+++ b/drivers/gpu/drm/panel/Makefile
@@ -49,6 +49,7 @@ obj-$(CONFIG_DRM_PANEL_PANASONIC_VVX10F034N00) += panel-panasonic-vvx10f034n00.o
obj-$(CONFIG_DRM_PANEL_RASPBERRYPI_TOUCHSCREEN) += panel-raspberrypi-touchscreen.o
obj-$(CONFIG_DRM_PANEL_RAYDIUM_RM67191) += panel-raydium-rm67191.o
obj-$(CONFIG_DRM_PANEL_RAYDIUM_RM68200) += panel-raydium-rm68200.o
+obj-$(CONFIG_DRM_PANEL_RAYDIUM_RM692E5) += panel-raydium-rm692e5.o
obj-$(CONFIG_DRM_PANEL_RONBO_RB070D30) += panel-ronbo-rb070d30.o
obj-$(CONFIG_DRM_PANEL_SAMSUNG_ATNA33XC20) += panel-samsung-atna33xc20.o
obj-$(CONFIG_DRM_PANEL_SAMSUNG_DB7430) += panel-samsung-db7430.o
diff --git a/drivers/gpu/drm/panel/panel-raydium-rm692e5.c b/drivers/gpu/drm/panel/panel-raydium-rm692e5.c
new file mode 100644
index 000000000000..bd753d1b12dd
--- /dev/null
+++ b/drivers/gpu/drm/panel/panel-raydium-rm692e5.c
@@ -0,0 +1,496 @@
+// SPDX-License-Identifier: GPL-2.0-only
+/*
+ * Generated with linux-mdss-dsi-panel-driver-generator from vendor device tree.
+ * Copyright (c) 2023 Linaro Limited
+ */
+
+#include <linux/backlight.h>
+#include <linux/delay.h>
+#include <linux/gpio/consumer.h>
+#include <linux/module.h>
+#include <linux/of.h>
+#include <linux/regulator/consumer.h>
+
+#include <drm/display/drm_dsc.h>
+#include <drm/display/drm_dsc_helper.h>
+#include <drm/drm_mipi_dsi.h>
+#include <drm/drm_modes.h>
+#include <drm/drm_panel.h>
+
+struct rm692e5_panel {
+ struct drm_panel panel;
+ struct mipi_dsi_device *dsi;
+ struct drm_dsc_config dsc;
+ struct regulator_bulk_data supplies[3];
+ struct gpio_desc *reset_gpio;
+ bool prepared;
+};
+
+static inline struct rm692e5_panel *to_rm692e5_panel(struct drm_panel *panel)
+{
+ return container_of(panel, struct rm692e5_panel, panel);
+}
+
+static void rm692e5_reset(struct rm692e5_panel *ctx)
+{
+ gpiod_set_value_cansleep(ctx->reset_gpio, 0);
+ usleep_range(10000, 11000);
+ gpiod_set_value_cansleep(ctx->reset_gpio, 1);
+ usleep_range(5000, 6000);
+ gpiod_set_value_cansleep(ctx->reset_gpio, 0);
+ usleep_range(10000, 11000);
+}
+
+static int rm692e5_on(struct rm692e5_panel *ctx)
+{
+ struct mipi_dsi_device *dsi = ctx->dsi;
+ struct device *dev = &dsi->dev;
+ int ret;
+
+ dsi->mode_flags |= MIPI_DSI_MODE_LPM;
+
+ mipi_dsi_generic_write_seq(dsi, 0xfe, 0x41);
+ usleep_range(1000, 2000);

I'm not familiar with this panel, but is calling usleep() after almost every single DCS command necessary or specified by the spec?

+ mipi_dsi_generic_write_seq(dsi, 0xd6, 0x00);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xfe, 0x16);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x8a, 0x87);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xfe, 0x71);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x82, 0x01);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xc6, 0x00);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xc7, 0x2c);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xc8, 0x64);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xc9, 0x3c);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xca, 0x80);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xcb, 0x02);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xcc, 0x02);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xfe, 0x38);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x18, 0x13);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xfe, 0xf4);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x00, 0xff);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x01, 0xff);

Are these generic DCS commands? If so, can you use the MIPI_DCS_* command macros/helpers when applicable?

+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x02, 0xcf);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x03, 0xbc);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x04, 0xb9);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x05, 0x99);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x06, 0x02);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x07, 0x0a);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x08, 0xe0);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x09, 0x4c);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x0a, 0xeb);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x0b, 0xe8);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x0c, 0x32);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x0d, 0x07);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xfe, 0xf4);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x0d, 0xc0);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x0e, 0xff);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x0f, 0xff);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x10, 0x33);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x11, 0x6f);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x12, 0x6e);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x13, 0xa6);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x14, 0x80);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x15, 0x02);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x16, 0x38);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x17, 0xd3);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x18, 0x3a);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x19, 0xba);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x1a, 0xcc);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x1b, 0x01);
+ usleep_range(1000, 2000);
+
+ ret = mipi_dsi_dcs_nop(dsi);
+ if (ret < 0) {
+ dev_err(dev, "Failed to nop: %d\n", ret);
+ return ret;
+ }
+ msleep(32);
+
+ mipi_dsi_generic_write_seq(dsi, 0xfe, 0x38);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x18, 0x13);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xfe, 0xd1);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xd3, 0x00);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xd0, 0x00);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xd2, 0x00);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xd4, 0x00);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xb4, 0x01);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xfe, 0xf9);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x00, 0xaf);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x1d, 0x37);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x44, 0x0a, 0x7b);
+ mipi_dsi_generic_write_seq(dsi, 0xfe, 0x00);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xfa, 0x01);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0xc2, 0x08);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x35, 0x00);
+ usleep_range(1000, 2000);
+ mipi_dsi_generic_write_seq(dsi, 0x51, 0x05, 0x42);
+ usleep_range(1000, 2000);
+
+ ret = mipi_dsi_dcs_exit_sleep_mode(dsi);
+ if (ret < 0) {
+ dev_err(dev, "Failed to exit sleep mode: %d\n", ret);
+ return ret;
+ }
+ msleep(100);
+
+ ret = mipi_dsi_dcs_set_display_on(dsi);
+ if (ret < 0) {
+ dev_err(dev, "Failed to set display on: %d\n", ret);
+ return ret;
+ }
+ usleep_range(1000, 2000);
+
+ return 0;
+}
+
+static int rm692e5_off(struct rm692e5_panel *ctx)
+{
+ struct mipi_dsi_device *dsi = ctx->dsi;
+ struct device *dev = &dsi->dev;
+ int ret;
+
+ dsi->mode_flags &= ~MIPI_DSI_MODE_LPM;
+
+ mipi_dsi_generic_write_seq(dsi, 0xfe, 0x00);
+ usleep_range(1000, 2000);
+
+ ret = mipi_dsi_dcs_set_display_off(dsi);
+ if (ret < 0) {
+ dev_err(dev, "Failed to set display off: %d\n", ret);
+ return ret;
+ }
+ usleep_range(1000, 2000);
+
+ ret = mipi_dsi_dcs_enter_sleep_mode(dsi);
+ if (ret < 0) {
+ dev_err(dev, "Failed to enter sleep mode: %d\n", ret);
+ return ret;
+ }
+ msleep(100);
+
+ return 0;
+}
+
+static int rm692e5_prepare(struct drm_panel *panel)
+{
+ struct rm692e5_panel *ctx = to_rm692e5_panel(panel);
+ struct drm_dsc_picture_parameter_set pps;
+ struct device *dev = &ctx->dsi->dev;
+ int ret;
+
+ if (ctx->prepared)
+ return 0;
+
+ ret = regulator_bulk_enable(ARRAY_SIZE(ctx->supplies), ctx->supplies);
+ if (ret < 0) {
+ dev_err(dev, "Failed to enable regulators: %d\n", ret);
+ return ret;
+ }
+
+ rm692e5_reset(ctx);
+
+ ret = rm692e5_on(ctx);
+ if (ret < 0) {
+ dev_err(dev, "Failed to initialize panel: %d\n", ret);
+ gpiod_set_value_cansleep(ctx->reset_gpio, 1);
+ regulator_bulk_disable(ARRAY_SIZE(ctx->supplies), ctx->supplies);
+ return ret;
+ }
+
+ drm_dsc_pps_payload_pack(&pps, &ctx->dsc);
+
+ ret = mipi_dsi_picture_parameter_set(ctx->dsi, &pps);
+ if (ret < 0) {
+ dev_err(panel->dev, "failed to transmit PPS: %d\n", ret);
+ return ret;
+ }
+
+ ret = mipi_dsi_compression_mode(ctx->dsi, true);
+ if (ret < 0) {
+ dev_err(dev, "failed to enable compression mode: %d\n", ret);
+ return ret;
+ }
+
+ msleep(28);
+
+ mipi_dsi_generic_write_seq(ctx->dsi, 0xfe, 0x40);
+ usleep_range(1000, 2000);
+
+ /* 0x05 -> 90Hz, 0x00 -> 60Hz */
+ mipi_dsi_generic_write_seq(ctx->dsi, 0xbd, 0x05);
+ usleep_range(1000, 2000);

Just to check my understanding of the comment here.. so the above DCS command will set the panel to 90Hz, and if we change the parameter to 0x00, it will be set to 60Hz instead?

Thanks,

Jessica Zhang

+
+ mipi_dsi_generic_write_seq(ctx->dsi, 0xfe, 0x00);
+ usleep_range(1000, 2000);
+
+ ctx->prepared = true;
+
+ return 0;
+}
+
+static int rm692e5_unprepare(struct drm_panel *panel)
+{
+ struct rm692e5_panel *ctx = to_rm692e5_panel(panel);
+ struct device *dev = &ctx->dsi->dev;
+ int ret;
+
+ if (!ctx->prepared)
+ return 0;
+
+ ret = rm692e5_off(ctx);
+ if (ret < 0)
+ dev_err(dev, "Failed to un-initialize panel: %d\n", ret);
+
+ gpiod_set_value_cansleep(ctx->reset_gpio, 1);
+ regulator_bulk_disable(ARRAY_SIZE(ctx->supplies), ctx->supplies);
+
+ ctx->prepared = false;
+ return 0;
+}
+
+static const struct drm_display_mode rm692e5_mode = {
+ .clock = (1224 + 32 + 8 + 8) * (2700 + 8 + 2 + 8) * 90 / 1000,
+ .hdisplay = 1224,
+ .hsync_start = 1224 + 32,
+ .hsync_end = 1224 + 32 + 8,
+ .htotal = 1224 + 32 + 8 + 8,
+ .vdisplay = 2700,
+ .vsync_start = 2700 + 8,
+ .vsync_end = 2700 + 8 + 2,
+ .vtotal = 2700 + 8 + 2 + 8,
+ .width_mm = 68,
+ .height_mm = 150,
+};
+
+static int rm692e5_get_modes(struct drm_panel *panel,
+ struct drm_connector *connector)
+{
+ struct drm_display_mode *mode;
+
+ mode = drm_mode_duplicate(connector->dev, &rm692e5_mode);
+ if (!mode)
+ return -ENOMEM;
+
+ drm_mode_set_name(mode);
+
+ mode->type = DRM_MODE_TYPE_DRIVER | DRM_MODE_TYPE_PREFERRED;
+ connector->display_info.width_mm = mode->width_mm;
+ connector->display_info.height_mm = mode->height_mm;
+ drm_mode_probed_add(connector, mode);
+
+ return 1;
+}
+
+static const struct drm_panel_funcs rm692e5_panel_funcs = {
+ .prepare = rm692e5_prepare,
+ .unprepare = rm692e5_unprepare,
+ .get_modes = rm692e5_get_modes,
+};
+
+static int rm692e5_bl_update_status(struct backlight_device *bl)
+{
+ struct mipi_dsi_device *dsi = bl_get_data(bl);
+ u16 brightness = backlight_get_brightness(bl);
+ int ret;
+
+ dsi->mode_flags &= ~MIPI_DSI_MODE_LPM;
+
+ ret = mipi_dsi_dcs_set_display_brightness_large(dsi, brightness);
+ if (ret < 0)
+ return ret;
+
+ dsi->mode_flags |= MIPI_DSI_MODE_LPM;
+
+ return 0;
+}
+
+static int rm692e5_bl_get_brightness(struct backlight_device *bl)
+{
+ struct mipi_dsi_device *dsi = bl_get_data(bl);
+ u16 brightness;
+ int ret;
+
+ dsi->mode_flags &= ~MIPI_DSI_MODE_LPM;
+
+ ret = mipi_dsi_dcs_get_display_brightness_large(dsi, &brightness);
+ if (ret < 0)
+ return ret;
+
+ dsi->mode_flags |= MIPI_DSI_MODE_LPM;
+
+ return brightness;
+}
+
+static const struct backlight_ops rm692e5_bl_ops = {
+ .update_status = rm692e5_bl_update_status,
+ .get_brightness = rm692e5_bl_get_brightness,
+};
+
+static struct backlight_device *
+rm692e5_create_backlight(struct mipi_dsi_device *dsi)
+{
+ struct device *dev = &dsi->dev;
+ const struct backlight_properties props = {
+ .type = BACKLIGHT_RAW,
+ .brightness = 4095,
+ .max_brightness = 4095,
+ };
+
+ return devm_backlight_device_register(dev, dev_name(dev), dev, dsi,
+ &rm692e5_bl_ops, &props);
+}
+
+static int rm692e5_probe(struct mipi_dsi_device *dsi)
+{
+ struct device *dev = &dsi->dev;
+ struct rm692e5_panel *ctx;
+ int ret;
+
+ ctx = devm_kzalloc(dev, sizeof(*ctx), GFP_KERNEL);
+ if (!ctx)
+ return -ENOMEM;
+
+ ctx->supplies[0].supply = "vddio";
+ ctx->supplies[1].supply = "dvdd";
+ ctx->supplies[2].supply = "vci";
+ ret = devm_regulator_bulk_get(dev, ARRAY_SIZE(ctx->supplies),
+ ctx->supplies);
+ if (ret < 0)
+ return dev_err_probe(dev, ret, "Failed to get regulators\n");
+
+ ctx->reset_gpio = devm_gpiod_get(dev, "reset", GPIOD_OUT_HIGH);
+ if (IS_ERR(ctx->reset_gpio))
+ return dev_err_probe(dev, PTR_ERR(ctx->reset_gpio),
+ "Failed to get reset-gpios\n");
+
+ ctx->dsi = dsi;
+ mipi_dsi_set_drvdata(dsi, ctx);
+
+ dsi->lanes = 4;
+ dsi->format = MIPI_DSI_FMT_RGB888;
+ dsi->mode_flags = MIPI_DSI_MODE_NO_EOT_PACKET |
+ MIPI_DSI_CLOCK_NON_CONTINUOUS;
+
+ drm_panel_init(&ctx->panel, dev, &rm692e5_panel_funcs,
+ DRM_MODE_CONNECTOR_DSI);
+ ctx->panel.prepare_prev_first = true;
+
+ ctx->panel.backlight = rm692e5_create_backlight(dsi);
+ if (IS_ERR(ctx->panel.backlight))
+ return dev_err_probe(dev, PTR_ERR(ctx->panel.backlight),
+ "Failed to create backlight\n");
+
+ drm_panel_add(&ctx->panel);
+
+ /* This panel only supports DSC; unconditionally enable it */
+ dsi->dsc = &ctx->dsc;
+
+ /* TODO: Pass slice_per_pkt = 2 */
+ ctx->dsc.dsc_version_major = 1;
+ ctx->dsc.dsc_version_minor = 1;
+ ctx->dsc.slice_height = 60;
+ ctx->dsc.slice_width = 1224;
+
+ WARN_ON(1224 % ctx->dsc.slice_width);
+ ctx->dsc.slice_count = 1224 / ctx->dsc.slice_width;
+ ctx->dsc.bits_per_component = 8;
+ ctx->dsc.bits_per_pixel = 8 << 4; /* 4 fractional bits */
+ ctx->dsc.block_pred_enable = true;
+
+ ret = mipi_dsi_attach(dsi);
+ if (ret < 0) {
+ dev_err(dev, "Failed to attach to DSI host: %d\n", ret);
+ drm_panel_remove(&ctx->panel);
+ return ret;
+ }
+
+ return 0;
+}
+
+static void rm692e5_remove(struct mipi_dsi_device *dsi)
+{
+ struct rm692e5_panel *ctx = mipi_dsi_get_drvdata(dsi);
+ int ret;
+
+ ret = mipi_dsi_detach(dsi);
+ if (ret < 0)
+ dev_err(&dsi->dev, "Failed to detach from DSI host: %d\n", ret);
+
+ drm_panel_remove(&ctx->panel);
+}
+
+static const struct of_device_id rm692e5_of_match[] = {
+ { .compatible = "fairphone,fp5-rm692e5-boe" },
+ { }
+};
+MODULE_DEVICE_TABLE(of, rm692e5_of_match);
+
+static struct mipi_dsi_driver rm692e5_driver = {
+ .probe = rm692e5_probe,
+ .remove = rm692e5_remove,
+ .driver = {
+ .name = "panel-rm692e5-boe-amoled",
+ .of_match_table = rm692e5_of_match,
+ },
+};
+module_mipi_dsi_driver(rm692e5_driver);
+
+MODULE_DESCRIPTION("DRM driver for rm692e5-equipped DSI panels");
+MODULE_LICENSE("GPL");

--
2.42.0

Next message: Qais Yousef: "Re: [PATCH 2/4] sched: cpufreq: Fix apply_dvfs_headroom() escaping uclamp constraints"
Previous message: Bjorn Helgaas: "Re: [PATCH 0/5] Add the pci_get_base_class() helper and use it"
In reply to: Konrad Dybcio: "[PATCH 2/2] drm/panel: Add driver for BOE RM692E5 AMOLED panel"
Next in thread: Konrad Dybcio: "Re: [PATCH 2/2] drm/panel: Add driver for BOE RM692E5 AMOLED panel"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]