* [PATCH 3/3] drm/panel: add Fitipower EK79007AD3 panel driver
2024-10-23 12:44 [PATCH 1/3] dt-bindings: vendor-prefixes: Add lxd Rouven Czerwinski
@ 2024-10-23 12:44 ` Rouven Czerwinski
2024-10-25 0:35 ` Dmitry Baryshkov
2024-10-25 14:34 ` Dan Carpenter
0 siblings, 2 replies; 4+ messages in thread
From: Rouven Czerwinski @ 2024-10-23 12:44 UTC (permalink / raw)
To: Jessica Zhang, Sam Ravnborg, linux-kernel, dri-devel
Cc: kernel, Rouven Czerwinski, Neil Armstrong, Maarten Lankhorst,
Maxime Ripard, Thomas Zimmermann, David Airlie, Daniel Vetter
The LXD M9189A panel is based on the EK79007AD3 DSI display controller.
It currently supports only 4 lane operation.
Signed-off-by: Rouven Czerwinski <r.czerwinski@pengutronix.de>
---
MAINTAINERS | 6 +
drivers/gpu/drm/panel/Kconfig | 9 +
drivers/gpu/drm/panel/Makefile | 1 +
drivers/gpu/drm/panel/panel-lxd-m9189a.c | 261 +++++++++++++++++++++++
4 files changed, 277 insertions(+)
create mode 100644 drivers/gpu/drm/panel/panel-lxd-m9189a.c
diff --git a/MAINTAINERS b/MAINTAINERS
index e9659a5a7fb33..e4d749b403c28 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -7179,6 +7179,12 @@ F: Documentation/devicetree/bindings/display/lvds.yaml
F: Documentation/devicetree/bindings/display/panel/panel-lvds.yaml
F: drivers/gpu/drm/panel/panel-lvds.c
+DRM DRIVER FOR LXD M9189A PANELS
+M: Rouven Czerwinski <r.czerwinski@pengutronix.de>
+S: Maintained
+F: Documentation/devicetree/bindings/display/panel/lxd,m9189a.yaml
+F: drivers/gpu/drm/panel/panel-lxd-m9189a.c
+
DRM DRIVER FOR MANTIX MLAF057WE51 PANELS
M: Guido Günther <agx@sigxcpu.org>
R: Purism Kernel Team <kernel@puri.sm>
diff --git a/drivers/gpu/drm/panel/Kconfig b/drivers/gpu/drm/panel/Kconfig
index d3a9a9fafe4ec..7667ac1ef80e3 100644
--- a/drivers/gpu/drm/panel/Kconfig
+++ b/drivers/gpu/drm/panel/Kconfig
@@ -135,6 +135,15 @@ config DRM_PANEL_FEIYANG_FY07024DI26A30D
Say Y if you want to enable support for panels based on the
Feiyang FY07024DI26A30-D MIPI-DSI interface.
+config DRM_PANEL_LXD_M9189A
+ tristate "LXD M9189A MIPI-DSI LCD panel"
+ depends on OF
+ depends on DRM_MIPI_DSI
+ depends on BACKLIGHT_CLASS_DEVICE
+ help
+ Say Y if you want to enable support for the LXD M9189A 4-Lane
+ 1024x600 MIPI DSI panel.
+
config DRM_PANEL_DSI_CM
tristate "Generic DSI command mode panels"
depends on OF
diff --git a/drivers/gpu/drm/panel/Makefile b/drivers/gpu/drm/panel/Makefile
index 987a087024103..6d77c304e7529 100644
--- a/drivers/gpu/drm/panel/Makefile
+++ b/drivers/gpu/drm/panel/Makefile
@@ -16,6 +16,7 @@ obj-$(CONFIG_DRM_PANEL_EBBG_FT8719) += panel-ebbg-ft8719.o
obj-$(CONFIG_DRM_PANEL_ELIDA_KD35T133) += panel-elida-kd35t133.o
obj-$(CONFIG_DRM_PANEL_FEIXIN_K101_IM2BA02) += panel-feixin-k101-im2ba02.o
obj-$(CONFIG_DRM_PANEL_FEIYANG_FY07024DI26A30D) += panel-feiyang-fy07024di26a30d.o
+obj-$(CONFIG_DRM_PANEL_LXD_M9189A) += panel-lxd-m9189a.o
obj-$(CONFIG_DRM_PANEL_HIMAX_HX83102) += panel-himax-hx83102.o
obj-$(CONFIG_DRM_PANEL_HIMAX_HX83112A) += panel-himax-hx83112a.o
obj-$(CONFIG_DRM_PANEL_HIMAX_HX8394) += panel-himax-hx8394.o
diff --git a/drivers/gpu/drm/panel/panel-lxd-m9189a.c b/drivers/gpu/drm/panel/panel-lxd-m9189a.c
new file mode 100644
index 0000000000000..71c5a18541bae
--- /dev/null
+++ b/drivers/gpu/drm/panel/panel-lxd-m9189a.c
@@ -0,0 +1,261 @@
+// SPDX-License-Identifier: GPL-2.0-only
+/*
+ * Generated with linux-mdss-dsi-panel-driver-generator from vendor device tree.
+ * Copyright (c) 2024 Luca Weiss <luca.weiss@fairphone.com>
+ */
+
+#include <linux/delay.h>
+#include <linux/gpio/consumer.h>
+#include <linux/module.h>
+#include <linux/of.h>
+#include <linux/regulator/consumer.h>
+
+#include <drm/drm_mipi_dsi.h>
+#include <drm/drm_modes.h>
+#include <drm/drm_panel.h>
+#include <drm/drm_probe_helper.h>
+
+/* Manufacturer specific DSI commands */
+#define EK79007AD3_GAMMA1 0x80
+#define EK79007AD3_GAMMA2 0x81
+#define EK79007AD3_GAMMA3 0x82
+#define EK79007AD3_GAMMA4 0x83
+#define EK79007AD3_GAMMA5 0x84
+#define EK79007AD3_GAMMA6 0x85
+#define EK79007AD3_GAMMA7 0x86
+#define EK79007AD3_PANEL_CTRL3 0xB2
+
+struct ek79007ad3_panel {
+ struct drm_panel panel;
+ struct mipi_dsi_device *dsi;
+ struct regulator *supply;
+ struct gpio_desc *reset_gpio;
+ struct gpio_desc *standby_gpio;
+};
+
+static inline struct ek79007ad3_panel *to_ek79007ad3_panel(struct drm_panel *panel)
+{
+ return container_of(panel, struct ek79007ad3_panel, panel);
+}
+
+static void ek79007ad3_reset(struct ek79007ad3_panel *ctx)
+{
+ gpiod_set_value_cansleep(ctx->reset_gpio, 0);
+ msleep(20);
+ gpiod_set_value_cansleep(ctx->reset_gpio, 1);
+ msleep(30);
+ gpiod_set_value_cansleep(ctx->reset_gpio, 0);
+ msleep(55);
+}
+
+static int ek79007ad3_on(struct ek79007ad3_panel *ctx)
+{
+ struct mipi_dsi_device *dsi = ctx->dsi;
+ struct device *dev = &dsi->dev;
+ int ret;
+
+ dsi->mode_flags |= MIPI_DSI_MODE_LPM;
+
+ /* Gamma 2.2 */
+ mipi_dsi_dcs_write_seq(dsi, EK79007AD3_GAMMA1, 0x48);
+ mipi_dsi_dcs_write_seq(dsi, EK79007AD3_GAMMA2, 0xB8);
+ mipi_dsi_dcs_write_seq(dsi, EK79007AD3_GAMMA3, 0x88);
+ mipi_dsi_dcs_write_seq(dsi, EK79007AD3_GAMMA4, 0x88);
+ mipi_dsi_dcs_write_seq(dsi, EK79007AD3_GAMMA5, 0x58);
+ mipi_dsi_dcs_write_seq(dsi, EK79007AD3_GAMMA6, 0xD2);
+ mipi_dsi_dcs_write_seq(dsi, EK79007AD3_GAMMA7, 0x88);
+ msleep(50);
+
+ /* 4 Lanes */
+ ret = mipi_dsi_generic_write(dsi, (u8[]){ EK79007AD3_PANEL_CTRL3, 0x70 }, 2);
+ if (ret)
+ goto fail;
+
+ 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(120);
+
+ ret = mipi_dsi_dcs_set_display_on(dsi);
+ msleep(120);
+
+fail:
+ return ret;
+}
+
+static int ek79007ad3_disable(struct drm_panel *panel)
+{
+ struct ek79007ad3_panel *ctx = to_ek79007ad3_panel(panel);
+ struct mipi_dsi_device *dsi = ctx->dsi;
+ struct device *dev = &dsi->dev;
+ int ret;
+
+ dsi->mode_flags &= ~MIPI_DSI_MODE_LPM;
+
+ 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(120);
+
+ gpiod_set_value_cansleep(ctx->standby_gpio, 1);
+
+ return 0;
+}
+
+static int ek79007ad3_prepare(struct drm_panel *panel)
+{
+ struct ek79007ad3_panel *ctx = to_ek79007ad3_panel(panel);
+ struct mipi_dsi_device *dsi = ctx->dsi;
+ struct device *dev = &ctx->dsi->dev;
+ int ret;
+
+ ret = regulator_enable(ctx->supply);
+ if (ret < 0) {
+ dev_err(dev, "Failed to enable regulators: %d\n", ret);
+ return ret;
+ }
+
+ gpiod_set_value_cansleep(ctx->standby_gpio, 0);
+ msleep(20);
+
+ mipi_dsi_dcs_nop(dsi);
+ usleep_range(1000, 2000);
+
+ ek79007ad3_reset(ctx);
+
+ ret = ek79007ad3_on(ctx);
+ if (ret < 0) {
+ dev_err(dev, "Failed to initialize panel: %d\n", ret);
+ gpiod_set_value_cansleep(ctx->reset_gpio, 1);
+ regulator_disable(ctx->supply);
+ return ret;
+ }
+
+ return 0;
+}
+
+static int ek79007ad3_unprepare(struct drm_panel *panel)
+{
+ struct ek79007ad3_panel *ctx = to_ek79007ad3_panel(panel);
+
+ gpiod_set_value_cansleep(ctx->standby_gpio, 1);
+ msleep(50);
+
+ gpiod_set_value_cansleep(ctx->reset_gpio, 1);
+ regulator_disable(ctx->supply);
+
+ return 0;
+}
+
+static const struct drm_display_mode ek79007ad3_mode = {
+ .clock = 51200,
+ .hdisplay = 1024,
+ .hsync_start = 1024 + 160,
+ .hsync_end = 1024 + 160 + 160,
+ .htotal = 1024 + 160 + 160 + 10,
+ .vdisplay = 600,
+ .vsync_start = 600 + 12,
+ .vsync_end = 600 + 12 + 23,
+ .vtotal = 600 + 23 + 12 + 1,
+ .width_mm = 154,
+ .height_mm = 86,
+};
+
+static int ek79007ad3_get_modes(struct drm_panel *panel,
+ struct drm_connector *connector)
+{
+ return drm_connector_helper_get_modes_fixed(connector, &ek79007ad3_mode);
+}
+
+static const struct drm_panel_funcs ek79007ad3_panel_funcs = {
+ .prepare = ek79007ad3_prepare,
+ .unprepare = ek79007ad3_unprepare,
+ .disable = ek79007ad3_disable,
+ .get_modes = ek79007ad3_get_modes,
+};
+
+static int ek79007ad3_probe(struct mipi_dsi_device *dsi)
+{
+ struct device *dev = &dsi->dev;
+ struct ek79007ad3_panel *ctx;
+ int ret;
+
+ ctx = devm_kzalloc(dev, sizeof(*ctx), GFP_KERNEL);
+ if (!ctx)
+ return -ENOMEM;
+
+ ctx->supply = devm_regulator_get(dev, "vdd");
+ if (IS_ERR(ctx->supply))
+ return dev_err_probe(dev, ret, "Failed to get regulator\n");
+
+ ctx->reset_gpio = devm_gpiod_get(dev, "reset", GPIOD_OUT_LOW);
+ if (IS_ERR(ctx->reset_gpio))
+ return dev_err_probe(dev, PTR_ERR(ctx->reset_gpio),
+ "Failed to get reset-gpios\n");
+
+ ctx->standby_gpio = devm_gpiod_get(dev, "standby", GPIOD_OUT_LOW);
+ if (IS_ERR(ctx->standby_gpio))
+ return dev_err_probe(dev, PTR_ERR(ctx->standby_gpio),
+ "Failed to get standby-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_VIDEO | MIPI_DSI_MODE_VIDEO_BURST;
+
+ drm_panel_init(&ctx->panel, dev, &ek79007ad3_panel_funcs,
+ DRM_MODE_CONNECTOR_DSI);
+ ctx->panel.prepare_prev_first = true;
+
+ ret = drm_panel_of_backlight(&ctx->panel);
+ if (ret)
+ return dev_err_probe(dev, ret, "Failed to get backlight\n");
+
+ drm_panel_add(&ctx->panel);
+
+ ret = mipi_dsi_attach(dsi);
+ if (ret < 0) {
+ dev_err_probe(dev, ret, "Failed to attach to DSI host\n");
+ drm_panel_remove(&ctx->panel);
+ return ret;
+ }
+
+ return 0;
+}
+
+static void ek79007ad3_remove(struct mipi_dsi_device *dsi)
+{
+ struct ek79007ad3_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 ek79007ad3_of_match[] = {
+ { .compatible = "lxd,m9189a" },
+ { /* sentinel */ }
+};
+MODULE_DEVICE_TABLE(of, ek79007ad3_of_match);
+
+static struct mipi_dsi_driver ek79007ad3_driver = {
+ .probe = ek79007ad3_probe,
+ .remove = ek79007ad3_remove,
+ .driver = {
+ .name = "panel-fitipower-ek79007ad3",
+ .of_match_table = ek79007ad3_of_match,
+ },
+};
+module_mipi_dsi_driver(ek79007ad3_driver);
+
+MODULE_DESCRIPTION("DRM driver for ek79007ad3-equipped DSI panels");
+MODULE_LICENSE("GPL");
--
2.39.5
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 3/3] drm/panel: add Fitipower EK79007AD3 panel driver
2024-10-23 12:44 ` [PATCH 3/3] drm/panel: add Fitipower EK79007AD3 panel driver Rouven Czerwinski
@ 2024-10-25 0:35 ` Dmitry Baryshkov
2024-10-25 14:34 ` Dan Carpenter
1 sibling, 0 replies; 4+ messages in thread
From: Dmitry Baryshkov @ 2024-10-25 0:35 UTC (permalink / raw)
To: Rouven Czerwinski
Cc: Jessica Zhang, Sam Ravnborg, linux-kernel, dri-devel, kernel,
Neil Armstrong, Maarten Lankhorst, Maxime Ripard,
Thomas Zimmermann, David Airlie, Daniel Vetter
On Wed, Oct 23, 2024 at 02:44:10PM +0200, Rouven Czerwinski wrote:
> The LXD M9189A panel is based on the EK79007AD3 DSI display controller.
> It currently supports only 4 lane operation.
>
> Signed-off-by: Rouven Czerwinski <r.czerwinski@pengutronix.de>
> ---
> MAINTAINERS | 6 +
> drivers/gpu/drm/panel/Kconfig | 9 +
> drivers/gpu/drm/panel/Makefile | 1 +
> drivers/gpu/drm/panel/panel-lxd-m9189a.c | 261 +++++++++++++++++++++++
> 4 files changed, 277 insertions(+)
> create mode 100644 drivers/gpu/drm/panel/panel-lxd-m9189a.c
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index e9659a5a7fb33..e4d749b403c28 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -7179,6 +7179,12 @@ F: Documentation/devicetree/bindings/display/lvds.yaml
> F: Documentation/devicetree/bindings/display/panel/panel-lvds.yaml
> F: drivers/gpu/drm/panel/panel-lvds.c
>
> +DRM DRIVER FOR LXD M9189A PANELS
> +M: Rouven Czerwinski <r.czerwinski@pengutronix.de>
> +S: Maintained
> +F: Documentation/devicetree/bindings/display/panel/lxd,m9189a.yaml
> +F: drivers/gpu/drm/panel/panel-lxd-m9189a.c
> +
> DRM DRIVER FOR MANTIX MLAF057WE51 PANELS
> M: Guido Günther <agx@sigxcpu.org>
> R: Purism Kernel Team <kernel@puri.sm>
> diff --git a/drivers/gpu/drm/panel/Kconfig b/drivers/gpu/drm/panel/Kconfig
> index d3a9a9fafe4ec..7667ac1ef80e3 100644
> --- a/drivers/gpu/drm/panel/Kconfig
> +++ b/drivers/gpu/drm/panel/Kconfig
> @@ -135,6 +135,15 @@ config DRM_PANEL_FEIYANG_FY07024DI26A30D
> Say Y if you want to enable support for panels based on the
> Feiyang FY07024DI26A30-D MIPI-DSI interface.
>
> +config DRM_PANEL_LXD_M9189A
> + tristate "LXD M9189A MIPI-DSI LCD panel"
> + depends on OF
> + depends on DRM_MIPI_DSI
> + depends on BACKLIGHT_CLASS_DEVICE
> + help
> + Say Y if you want to enable support for the LXD M9189A 4-Lane
> + 1024x600 MIPI DSI panel.
> +
> config DRM_PANEL_DSI_CM
> tristate "Generic DSI command mode panels"
> depends on OF
> diff --git a/drivers/gpu/drm/panel/Makefile b/drivers/gpu/drm/panel/Makefile
> index 987a087024103..6d77c304e7529 100644
> --- a/drivers/gpu/drm/panel/Makefile
> +++ b/drivers/gpu/drm/panel/Makefile
> @@ -16,6 +16,7 @@ obj-$(CONFIG_DRM_PANEL_EBBG_FT8719) += panel-ebbg-ft8719.o
> obj-$(CONFIG_DRM_PANEL_ELIDA_KD35T133) += panel-elida-kd35t133.o
> obj-$(CONFIG_DRM_PANEL_FEIXIN_K101_IM2BA02) += panel-feixin-k101-im2ba02.o
> obj-$(CONFIG_DRM_PANEL_FEIYANG_FY07024DI26A30D) += panel-feiyang-fy07024di26a30d.o
> +obj-$(CONFIG_DRM_PANEL_LXD_M9189A) += panel-lxd-m9189a.o
> obj-$(CONFIG_DRM_PANEL_HIMAX_HX83102) += panel-himax-hx83102.o
> obj-$(CONFIG_DRM_PANEL_HIMAX_HX83112A) += panel-himax-hx83112a.o
> obj-$(CONFIG_DRM_PANEL_HIMAX_HX8394) += panel-himax-hx8394.o
> diff --git a/drivers/gpu/drm/panel/panel-lxd-m9189a.c b/drivers/gpu/drm/panel/panel-lxd-m9189a.c
> new file mode 100644
> index 0000000000000..71c5a18541bae
> --- /dev/null
> +++ b/drivers/gpu/drm/panel/panel-lxd-m9189a.c
> @@ -0,0 +1,261 @@
> +// SPDX-License-Identifier: GPL-2.0-only
> +/*
> + * Generated with linux-mdss-dsi-panel-driver-generator from vendor device tree.
> + * Copyright (c) 2024 Luca Weiss <luca.weiss@fairphone.com>
> + */
> +
> +#include <linux/delay.h>
> +#include <linux/gpio/consumer.h>
> +#include <linux/module.h>
> +#include <linux/of.h>
> +#include <linux/regulator/consumer.h>
> +
> +#include <drm/drm_mipi_dsi.h>
> +#include <drm/drm_modes.h>
> +#include <drm/drm_panel.h>
> +#include <drm/drm_probe_helper.h>
> +
> +/* Manufacturer specific DSI commands */
> +#define EK79007AD3_GAMMA1 0x80
> +#define EK79007AD3_GAMMA2 0x81
> +#define EK79007AD3_GAMMA3 0x82
> +#define EK79007AD3_GAMMA4 0x83
> +#define EK79007AD3_GAMMA5 0x84
> +#define EK79007AD3_GAMMA6 0x85
> +#define EK79007AD3_GAMMA7 0x86
> +#define EK79007AD3_PANEL_CTRL3 0xB2
> +
> +struct ek79007ad3_panel {
> + struct drm_panel panel;
> + struct mipi_dsi_device *dsi;
> + struct regulator *supply;
> + struct gpio_desc *reset_gpio;
> + struct gpio_desc *standby_gpio;
> +};
> +
> +static inline struct ek79007ad3_panel *to_ek79007ad3_panel(struct drm_panel *panel)
> +{
> + return container_of(panel, struct ek79007ad3_panel, panel);
> +}
> +
> +static void ek79007ad3_reset(struct ek79007ad3_panel *ctx)
> +{
> + gpiod_set_value_cansleep(ctx->reset_gpio, 0);
> + msleep(20);
> + gpiod_set_value_cansleep(ctx->reset_gpio, 1);
> + msleep(30);
> + gpiod_set_value_cansleep(ctx->reset_gpio, 0);
> + msleep(55);
> +}
> +
> +static int ek79007ad3_on(struct ek79007ad3_panel *ctx)
> +{
> + struct mipi_dsi_device *dsi = ctx->dsi;
> + struct device *dev = &dsi->dev;
> + int ret;
> +
> + dsi->mode_flags |= MIPI_DSI_MODE_LPM;
> +
> + /* Gamma 2.2 */
> + mipi_dsi_dcs_write_seq(dsi, EK79007AD3_GAMMA1, 0x48);
> + mipi_dsi_dcs_write_seq(dsi, EK79007AD3_GAMMA2, 0xB8);
> + mipi_dsi_dcs_write_seq(dsi, EK79007AD3_GAMMA3, 0x88);
> + mipi_dsi_dcs_write_seq(dsi, EK79007AD3_GAMMA4, 0x88);
> + mipi_dsi_dcs_write_seq(dsi, EK79007AD3_GAMMA5, 0x58);
> + mipi_dsi_dcs_write_seq(dsi, EK79007AD3_GAMMA6, 0xD2);
> + mipi_dsi_dcs_write_seq(dsi, EK79007AD3_GAMMA7, 0x88);
mipi_dsi_dcs_write_seq_multi
> + msleep(50);
mipi_dsi_msleep(), here and further below
> +
> + /* 4 Lanes */
> + ret = mipi_dsi_generic_write(dsi, (u8[]){ EK79007AD3_PANEL_CTRL3, 0x70 }, 2);
mipi_dsi_generic_write_multi
> + if (ret)
> + goto fail;
> +
> + ret = mipi_dsi_dcs_exit_sleep_mode(dsi);
_multi
> + if (ret < 0) {
> + dev_err(dev, "Failed to exit sleep mode: %d\n", ret);
> + return ret;
> + }
> + msleep(120);
> +
> + ret = mipi_dsi_dcs_set_display_on(dsi);
_multi
> + msleep(120);
> +
> +fail:
> + return ret;
> +}
> +
> +static int ek79007ad3_disable(struct drm_panel *panel)
> +{
> + struct ek79007ad3_panel *ctx = to_ek79007ad3_panel(panel);
> + struct mipi_dsi_device *dsi = ctx->dsi;
> + struct device *dev = &dsi->dev;
> + int ret;
> +
> + dsi->mode_flags &= ~MIPI_DSI_MODE_LPM;
> +
> + ret = mipi_dsi_dcs_enter_sleep_mode(dsi);
_multi
> + if (ret < 0) {
> + dev_err(dev, "Failed to enter sleep mode: %d\n", ret);
> + return ret;
> + }
> + msleep(120);
> +
> + gpiod_set_value_cansleep(ctx->standby_gpio, 1);
> +
> + return 0;
> +}
> +
> +static int ek79007ad3_prepare(struct drm_panel *panel)
> +{
> + struct ek79007ad3_panel *ctx = to_ek79007ad3_panel(panel);
> + struct mipi_dsi_device *dsi = ctx->dsi;
> + struct device *dev = &ctx->dsi->dev;
> + int ret;
> +
> + ret = regulator_enable(ctx->supply);
> + if (ret < 0) {
> + dev_err(dev, "Failed to enable regulators: %d\n", ret);
> + return ret;
> + }
> +
> + gpiod_set_value_cansleep(ctx->standby_gpio, 0);
> + msleep(20);
> +
> + mipi_dsi_dcs_nop(dsi);
PLease move these calls to _on() and use mipi_dsi_dcs_nop_multi() and
mipi_dsi_usleep_range().
> + usleep_range(1000, 2000);
> +
> + ek79007ad3_reset(ctx);
> +
> + ret = ek79007ad3_on(ctx);
> + if (ret < 0) {
> + dev_err(dev, "Failed to initialize panel: %d\n", ret);
> + gpiod_set_value_cansleep(ctx->reset_gpio, 1);
> + regulator_disable(ctx->supply);
> + return ret;
> + }
> +
> + return 0;
> +}
> +
> +static int ek79007ad3_unprepare(struct drm_panel *panel)
> +{
> + struct ek79007ad3_panel *ctx = to_ek79007ad3_panel(panel);
> +
> + gpiod_set_value_cansleep(ctx->standby_gpio, 1);
> + msleep(50);
> +
> + gpiod_set_value_cansleep(ctx->reset_gpio, 1);
> + regulator_disable(ctx->supply);
> +
> + return 0;
> +}
> +
> +static const struct drm_display_mode ek79007ad3_mode = {
> + .clock = 51200,
.clock = (1024 + 160 + 160 + 10) * (600 + 23 + 12 + 1) * N / 1000;
> + .hdisplay = 1024,
> + .hsync_start = 1024 + 160,
> + .hsync_end = 1024 + 160 + 160,
> + .htotal = 1024 + 160 + 160 + 10,
> + .vdisplay = 600,
> + .vsync_start = 600 + 12,
> + .vsync_end = 600 + 12 + 23,
> + .vtotal = 600 + 23 + 12 + 1,
> + .width_mm = 154,
> + .height_mm = 86,
> +};
> +
> +static int ek79007ad3_get_modes(struct drm_panel *panel,
> + struct drm_connector *connector)
> +{
> + return drm_connector_helper_get_modes_fixed(connector, &ek79007ad3_mode);
> +}
> +
> +static const struct drm_panel_funcs ek79007ad3_panel_funcs = {
> + .prepare = ek79007ad3_prepare,
> + .unprepare = ek79007ad3_unprepare,
> + .disable = ek79007ad3_disable,
> + .get_modes = ek79007ad3_get_modes,
> +};
> +
> +static int ek79007ad3_probe(struct mipi_dsi_device *dsi)
> +{
> + struct device *dev = &dsi->dev;
> + struct ek79007ad3_panel *ctx;
> + int ret;
> +
> + ctx = devm_kzalloc(dev, sizeof(*ctx), GFP_KERNEL);
> + if (!ctx)
> + return -ENOMEM;
> +
> + ctx->supply = devm_regulator_get(dev, "vdd");
> + if (IS_ERR(ctx->supply))
> + return dev_err_probe(dev, ret, "Failed to get regulator\n");
> +
> + ctx->reset_gpio = devm_gpiod_get(dev, "reset", GPIOD_OUT_LOW);
> + if (IS_ERR(ctx->reset_gpio))
> + return dev_err_probe(dev, PTR_ERR(ctx->reset_gpio),
> + "Failed to get reset-gpios\n");
> +
> + ctx->standby_gpio = devm_gpiod_get(dev, "standby", GPIOD_OUT_LOW);
> + if (IS_ERR(ctx->standby_gpio))
> + return dev_err_probe(dev, PTR_ERR(ctx->standby_gpio),
> + "Failed to get standby-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_VIDEO | MIPI_DSI_MODE_VIDEO_BURST;
> +
> + drm_panel_init(&ctx->panel, dev, &ek79007ad3_panel_funcs,
> + DRM_MODE_CONNECTOR_DSI);
> + ctx->panel.prepare_prev_first = true;
> +
> + ret = drm_panel_of_backlight(&ctx->panel);
> + if (ret)
> + return dev_err_probe(dev, ret, "Failed to get backlight\n");
> +
> + drm_panel_add(&ctx->panel);
> +
> + ret = mipi_dsi_attach(dsi);
> + if (ret < 0) {
> + dev_err_probe(dev, ret, "Failed to attach to DSI host\n");
> + drm_panel_remove(&ctx->panel);
> + return ret;
> + }
> +
> + return 0;
> +}
> +
> +static void ek79007ad3_remove(struct mipi_dsi_device *dsi)
> +{
> + struct ek79007ad3_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 ek79007ad3_of_match[] = {
> + { .compatible = "lxd,m9189a" },
> + { /* sentinel */ }
> +};
> +MODULE_DEVICE_TABLE(of, ek79007ad3_of_match);
> +
> +static struct mipi_dsi_driver ek79007ad3_driver = {
> + .probe = ek79007ad3_probe,
> + .remove = ek79007ad3_remove,
> + .driver = {
> + .name = "panel-fitipower-ek79007ad3",
> + .of_match_table = ek79007ad3_of_match,
> + },
> +};
> +module_mipi_dsi_driver(ek79007ad3_driver);
> +
> +MODULE_DESCRIPTION("DRM driver for ek79007ad3-equipped DSI panels");
> +MODULE_LICENSE("GPL");
> --
> 2.39.5
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 3/3] drm/panel: add Fitipower EK79007AD3 panel driver
@ 2024-10-25 14:25 kernel test robot
0 siblings, 0 replies; 4+ messages in thread
From: kernel test robot @ 2024-10-25 14:25 UTC (permalink / raw)
To: oe-kbuild; +Cc: lkp, Dan Carpenter
BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
In-Reply-To: <20241023124411.1153552-3-r.czerwinski@pengutronix.de>
References: <20241023124411.1153552-3-r.czerwinski@pengutronix.de>
TO: Rouven Czerwinski <r.czerwinski@pengutronix.de>
TO: Jessica Zhang <quic_jesszhan@quicinc.com>
TO: Sam Ravnborg <sam@ravnborg.org>
TO: linux-kernel@vger.kernel.org
TO: dri-devel@lists.freedesktop.org
CC: kernel@pengutronix.de
CC: Rouven Czerwinski <r.czerwinski@pengutronix.de>
CC: Neil Armstrong <neil.armstrong@linaro.org>
CC: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
CC: Maxime Ripard <mripard@kernel.org>
CC: Thomas Zimmermann <tzimmermann@suse.de>
CC: David Airlie <airlied@gmail.com>
CC: Daniel Vetter <daniel@ffwll.ch>
Hi Rouven,
kernel test robot noticed the following build warnings:
[auto build test WARNING on c2ee9f594da826bea183ed14f2cc029c719bf4da]
url: https://github.com/intel-lab-lkp/linux/commits/Rouven-Czerwinski/dt-bindings-display-panel-add-YAML-schema-for-LXD-M9189A/20241023-204826
base: c2ee9f594da826bea183ed14f2cc029c719bf4da
patch link: https://lore.kernel.org/r/20241023124411.1153552-3-r.czerwinski%40pengutronix.de
patch subject: [PATCH 3/3] drm/panel: add Fitipower EK79007AD3 panel driver
:::::: branch date: 2 days ago
:::::: commit date: 2 days ago
config: nios2-randconfig-r072-20241025 (https://download.01.org/0day-ci/archive/20241025/202410252201.QglvauIC-lkp@intel.com/config)
compiler: nios2-linux-gcc (GCC) 14.1.0
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Reported-by: Dan Carpenter <error27@gmail.com>
| Closes: https://lore.kernel.org/r/202410252201.QglvauIC-lkp@intel.com/
smatch warnings:
drivers/gpu/drm/panel/panel-lxd-m9189a.c:193 ek79007ad3_probe() error: uninitialized symbol 'ret'.
vim +/ret +193 drivers/gpu/drm/panel/panel-lxd-m9189a.c
a69b961abc36b3 Rouven Czerwinski 2024-10-23 180
a69b961abc36b3 Rouven Czerwinski 2024-10-23 181 static int ek79007ad3_probe(struct mipi_dsi_device *dsi)
a69b961abc36b3 Rouven Czerwinski 2024-10-23 182 {
a69b961abc36b3 Rouven Czerwinski 2024-10-23 183 struct device *dev = &dsi->dev;
a69b961abc36b3 Rouven Czerwinski 2024-10-23 184 struct ek79007ad3_panel *ctx;
a69b961abc36b3 Rouven Czerwinski 2024-10-23 185 int ret;
a69b961abc36b3 Rouven Czerwinski 2024-10-23 186
a69b961abc36b3 Rouven Czerwinski 2024-10-23 187 ctx = devm_kzalloc(dev, sizeof(*ctx), GFP_KERNEL);
a69b961abc36b3 Rouven Czerwinski 2024-10-23 188 if (!ctx)
a69b961abc36b3 Rouven Czerwinski 2024-10-23 189 return -ENOMEM;
a69b961abc36b3 Rouven Czerwinski 2024-10-23 190
a69b961abc36b3 Rouven Czerwinski 2024-10-23 191 ctx->supply = devm_regulator_get(dev, "vdd");
a69b961abc36b3 Rouven Czerwinski 2024-10-23 192 if (IS_ERR(ctx->supply))
a69b961abc36b3 Rouven Czerwinski 2024-10-23 @193 return dev_err_probe(dev, ret, "Failed to get regulator\n");
a69b961abc36b3 Rouven Czerwinski 2024-10-23 194
a69b961abc36b3 Rouven Czerwinski 2024-10-23 195 ctx->reset_gpio = devm_gpiod_get(dev, "reset", GPIOD_OUT_LOW);
a69b961abc36b3 Rouven Czerwinski 2024-10-23 196 if (IS_ERR(ctx->reset_gpio))
a69b961abc36b3 Rouven Czerwinski 2024-10-23 197 return dev_err_probe(dev, PTR_ERR(ctx->reset_gpio),
a69b961abc36b3 Rouven Czerwinski 2024-10-23 198 "Failed to get reset-gpios\n");
a69b961abc36b3 Rouven Czerwinski 2024-10-23 199
a69b961abc36b3 Rouven Czerwinski 2024-10-23 200 ctx->standby_gpio = devm_gpiod_get(dev, "standby", GPIOD_OUT_LOW);
a69b961abc36b3 Rouven Czerwinski 2024-10-23 201 if (IS_ERR(ctx->standby_gpio))
a69b961abc36b3 Rouven Czerwinski 2024-10-23 202 return dev_err_probe(dev, PTR_ERR(ctx->standby_gpio),
a69b961abc36b3 Rouven Czerwinski 2024-10-23 203 "Failed to get standby-gpios\n");
a69b961abc36b3 Rouven Czerwinski 2024-10-23 204
a69b961abc36b3 Rouven Czerwinski 2024-10-23 205 ctx->dsi = dsi;
a69b961abc36b3 Rouven Czerwinski 2024-10-23 206 mipi_dsi_set_drvdata(dsi, ctx);
a69b961abc36b3 Rouven Czerwinski 2024-10-23 207
a69b961abc36b3 Rouven Czerwinski 2024-10-23 208 dsi->lanes = 4;
a69b961abc36b3 Rouven Czerwinski 2024-10-23 209 dsi->format = MIPI_DSI_FMT_RGB888;
a69b961abc36b3 Rouven Czerwinski 2024-10-23 210 dsi->mode_flags = MIPI_DSI_MODE_VIDEO | MIPI_DSI_MODE_VIDEO_BURST;
a69b961abc36b3 Rouven Czerwinski 2024-10-23 211
a69b961abc36b3 Rouven Czerwinski 2024-10-23 212 drm_panel_init(&ctx->panel, dev, &ek79007ad3_panel_funcs,
a69b961abc36b3 Rouven Czerwinski 2024-10-23 213 DRM_MODE_CONNECTOR_DSI);
a69b961abc36b3 Rouven Czerwinski 2024-10-23 214 ctx->panel.prepare_prev_first = true;
a69b961abc36b3 Rouven Czerwinski 2024-10-23 215
a69b961abc36b3 Rouven Czerwinski 2024-10-23 216 ret = drm_panel_of_backlight(&ctx->panel);
a69b961abc36b3 Rouven Czerwinski 2024-10-23 217 if (ret)
a69b961abc36b3 Rouven Czerwinski 2024-10-23 218 return dev_err_probe(dev, ret, "Failed to get backlight\n");
a69b961abc36b3 Rouven Czerwinski 2024-10-23 219
a69b961abc36b3 Rouven Czerwinski 2024-10-23 220 drm_panel_add(&ctx->panel);
a69b961abc36b3 Rouven Czerwinski 2024-10-23 221
a69b961abc36b3 Rouven Czerwinski 2024-10-23 222 ret = mipi_dsi_attach(dsi);
a69b961abc36b3 Rouven Czerwinski 2024-10-23 223 if (ret < 0) {
a69b961abc36b3 Rouven Czerwinski 2024-10-23 224 dev_err_probe(dev, ret, "Failed to attach to DSI host\n");
a69b961abc36b3 Rouven Czerwinski 2024-10-23 225 drm_panel_remove(&ctx->panel);
a69b961abc36b3 Rouven Czerwinski 2024-10-23 226 return ret;
a69b961abc36b3 Rouven Czerwinski 2024-10-23 227 }
a69b961abc36b3 Rouven Czerwinski 2024-10-23 228
a69b961abc36b3 Rouven Czerwinski 2024-10-23 229 return 0;
a69b961abc36b3 Rouven Czerwinski 2024-10-23 230 }
a69b961abc36b3 Rouven Czerwinski 2024-10-23 231
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 3/3] drm/panel: add Fitipower EK79007AD3 panel driver
2024-10-23 12:44 ` [PATCH 3/3] drm/panel: add Fitipower EK79007AD3 panel driver Rouven Czerwinski
2024-10-25 0:35 ` Dmitry Baryshkov
@ 2024-10-25 14:34 ` Dan Carpenter
1 sibling, 0 replies; 4+ messages in thread
From: Dan Carpenter @ 2024-10-25 14:34 UTC (permalink / raw)
To: oe-kbuild, Rouven Czerwinski, Jessica Zhang, Sam Ravnborg,
linux-kernel, dri-devel
Cc: lkp, oe-kbuild-all, kernel, Rouven Czerwinski, Neil Armstrong,
Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann, David Airlie,
Daniel Vetter
Hi Rouven,
kernel test robot noticed the following build warnings:
url: https://github.com/intel-lab-lkp/linux/commits/Rouven-Czerwinski/dt-bindings-display-panel-add-YAML-schema-for-LXD-M9189A/20241023-204826
base: c2ee9f594da826bea183ed14f2cc029c719bf4da
patch link: https://lore.kernel.org/r/20241023124411.1153552-3-r.czerwinski%40pengutronix.de
patch subject: [PATCH 3/3] drm/panel: add Fitipower EK79007AD3 panel driver
config: nios2-randconfig-r072-20241025 (https://download.01.org/0day-ci/archive/20241025/202410252201.QglvauIC-lkp@intel.com/config)
compiler: nios2-linux-gcc (GCC) 14.1.0
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Reported-by: Dan Carpenter <dan.carpenter@linaro.org>
| Closes: https://lore.kernel.org/r/202410252201.QglvauIC-lkp@intel.com/
smatch warnings:
drivers/gpu/drm/panel/panel-lxd-m9189a.c:193 ek79007ad3_probe() error: uninitialized symbol 'ret'.
vim +/ret +193 drivers/gpu/drm/panel/panel-lxd-m9189a.c
a69b961abc36b3 Rouven Czerwinski 2024-10-23 181 static int ek79007ad3_probe(struct mipi_dsi_device *dsi)
a69b961abc36b3 Rouven Czerwinski 2024-10-23 182 {
a69b961abc36b3 Rouven Czerwinski 2024-10-23 183 struct device *dev = &dsi->dev;
a69b961abc36b3 Rouven Czerwinski 2024-10-23 184 struct ek79007ad3_panel *ctx;
a69b961abc36b3 Rouven Czerwinski 2024-10-23 185 int ret;
a69b961abc36b3 Rouven Czerwinski 2024-10-23 186
a69b961abc36b3 Rouven Czerwinski 2024-10-23 187 ctx = devm_kzalloc(dev, sizeof(*ctx), GFP_KERNEL);
a69b961abc36b3 Rouven Czerwinski 2024-10-23 188 if (!ctx)
a69b961abc36b3 Rouven Czerwinski 2024-10-23 189 return -ENOMEM;
a69b961abc36b3 Rouven Czerwinski 2024-10-23 190
a69b961abc36b3 Rouven Czerwinski 2024-10-23 191 ctx->supply = devm_regulator_get(dev, "vdd");
a69b961abc36b3 Rouven Czerwinski 2024-10-23 192 if (IS_ERR(ctx->supply))
a69b961abc36b3 Rouven Czerwinski 2024-10-23 @193 return dev_err_probe(dev, ret, "Failed to get regulator\n");
s/ret/PTR_ERR(ctx->supply)/
a69b961abc36b3 Rouven Czerwinski 2024-10-23 194
a69b961abc36b3 Rouven Czerwinski 2024-10-23 195 ctx->reset_gpio = devm_gpiod_get(dev, "reset", GPIOD_OUT_LOW);
a69b961abc36b3 Rouven Czerwinski 2024-10-23 196 if (IS_ERR(ctx->reset_gpio))
a69b961abc36b3 Rouven Czerwinski 2024-10-23 197 return dev_err_probe(dev, PTR_ERR(ctx->reset_gpio),
a69b961abc36b3 Rouven Czerwinski 2024-10-23 198 "Failed to get reset-gpios\n");
a69b961abc36b3 Rouven Czerwinski 2024-10-23 199
a69b961abc36b3 Rouven Czerwinski 2024-10-23 200 ctx->standby_gpio = devm_gpiod_get(dev, "standby", GPIOD_OUT_LOW);
a69b961abc36b3 Rouven Czerwinski 2024-10-23 201 if (IS_ERR(ctx->standby_gpio))
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-10-25 14:34 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-25 14:25 [PATCH 3/3] drm/panel: add Fitipower EK79007AD3 panel driver kernel test robot
-- strict thread matches above, loose matches on Subject: below --
2024-10-23 12:44 [PATCH 1/3] dt-bindings: vendor-prefixes: Add lxd Rouven Czerwinski
2024-10-23 12:44 ` [PATCH 3/3] drm/panel: add Fitipower EK79007AD3 panel driver Rouven Czerwinski
2024-10-25 0:35 ` Dmitry Baryshkov
2024-10-25 14:34 ` Dan Carpenter
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.