From: Paul Kocialkowski <paul.kocialkowski@bootlin.com> To: Maxime Ripard <maxime.ripard@bootlin.com>, Maarten Lankhorst <maarten.lankhorst@linux.intel.com>, Sean Paul <seanpaul@chromium.org> Cc: Konstantin Sudakov <k.sudakov@integrasources.com>, bbrezillon@kernel.org, Sam Ravnborg <sam@ravnborg.org>, dri-devel@lists.freedesktop.org, Chen-Yu Tsai <wens@csie.org>, Thomas Petazzoni <thomas.petazzoni@bootlin.com>, Jagan Teki <jagan@openedev.com>, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v3 8/8] drm/panel: Add Ronbo RB070D30 panel Date: Wed, 13 Feb 2019 15:26:45 +0100 Message-ID: <25f4beadd3088b3fbb2751f120e4216969301136.camel@bootlin.com> (raw) In-Reply-To: <2663d767bffbc83f84f9086e614b0ec1dd343ced.1549896081.git-series.maxime.ripard@bootlin.com> Hi, On Mon, 2019-02-11 at 15:41 +0100, Maxime Ripard wrote: > From: Konstantin Sudakov <k.sudakov@integrasources.com> > > The Ronbo RB070D30 panel is a MIPI-DSI panel based on a Fitipower EK79007 > controller and a 1024x600 panel. > > Reviewed-by: Sam Ravnborg <sam@ravnborg.org> > Signed-off-by: Konstantin Sudakov <k.sudakov@integrasources.com> > Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> > --- > drivers/gpu/drm/panel/Kconfig | 9 +- > drivers/gpu/drm/panel/Makefile | 1 +- > drivers/gpu/drm/panel/panel-ronbo-rb070d30.c | 262 ++++++++++++++++++++- Looks like no bindings documentation is introduced for this panel. Shouldn't that be the case? The vendor also seems to be missing from vendor-prefixes.txt. For the driver itself: Reviewed-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> Cheers, Paul > 3 files changed, 272 insertions(+) > create mode 100644 drivers/gpu/drm/panel/panel-ronbo-rb070d30.c > > diff --git a/drivers/gpu/drm/panel/Kconfig b/drivers/gpu/drm/panel/Kconfig > index 3f3537719beb..870dcdbf2601 100644 > --- a/drivers/gpu/drm/panel/Kconfig > +++ b/drivers/gpu/drm/panel/Kconfig > @@ -138,6 +138,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_RONBO_RB070D30 > + tristate "Ronbo Electronics RB070D30 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 Ronbo Electronics > + RB070D30 1024x600 DSI panel. > + > config DRM_PANEL_SAMSUNG_S6D16D0 > tristate "Samsung S6D16D0 DSI video mode panel" > depends on OF > diff --git a/drivers/gpu/drm/panel/Makefile b/drivers/gpu/drm/panel/Makefile > index 4396658a7996..4a103d346809 100644 > --- a/drivers/gpu/drm/panel/Makefile > +++ b/drivers/gpu/drm/panel/Makefile > @@ -12,6 +12,7 @@ obj-$(CONFIG_DRM_PANEL_ORISETECH_OTM8009A) += panel-orisetech-otm8009a.o > 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_RM68200) += panel-raydium-rm68200.o > +obj-$(CONFIG_DRM_PANEL_RONBO_RB070D30) += panel-ronbo-rb070d30.o > obj-$(CONFIG_DRM_PANEL_SAMSUNG_LD9040) += panel-samsung-ld9040.o > obj-$(CONFIG_DRM_PANEL_SAMSUNG_S6D16D0) += panel-samsung-s6d16d0.o > obj-$(CONFIG_DRM_PANEL_SAMSUNG_S6E3HA2) += panel-samsung-s6e3ha2.o > diff --git a/drivers/gpu/drm/panel/panel-ronbo-rb070d30.c b/drivers/gpu/drm/panel/panel-ronbo-rb070d30.c > new file mode 100644 > index 000000000000..cb4f99252aa0 > --- /dev/null > +++ b/drivers/gpu/drm/panel/panel-ronbo-rb070d30.c > @@ -0,0 +1,262 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * Copyright (C) 2018-2019, Bridge Systems BV > + * Copyright (C) 2018-2019, Bootlin > + * Copyright (C) 2017, Free Electrons > + * > + * This file based on panel-ilitek-ili9881c.c > + */ > + > +#include <linux/backlight.h> > +#include <linux/delay.h> > +#include <linux/device.h> > +#include <linux/err.h> > +#include <linux/errno.h> > +#include <linux/fb.h> > +#include <linux/kernel.h> > +#include <linux/media-bus-format.h> > +#include <linux/module.h> > + > +#include <linux/gpio/consumer.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_print.h> > + > +#include <video/mipi_display.h> > +#include <video/of_display_timing.h> > +#include <video/videomode.h> > + > +struct rb070d30_panel { > + struct drm_panel panel; > + struct mipi_dsi_device *dsi; > + struct backlight_device *backlight; > + struct regulator *supply; > + > + struct { > + struct gpio_desc *power; > + struct gpio_desc *reset; > + struct gpio_desc *updn; > + struct gpio_desc *shlr; > + } gpios; > +}; > + > +static inline struct rb070d30_panel *panel_to_rb070d30_panel(struct drm_panel *panel) > +{ > + return container_of(panel, struct rb070d30_panel, panel); > +} > + > +static int rb070d30_panel_prepare(struct drm_panel *panel) > +{ > + struct rb070d30_panel *ctx = panel_to_rb070d30_panel(panel); > + int ret; > + > + ret = regulator_enable(ctx->supply); > + if (ret < 0) { > + DRM_DEV_ERROR(&ctx->dsi->dev, "Failed to enable supply: %d\n", ret); > + return ret; > + } > + > + /* Reset */ > + msleep(20); > + gpiod_set_value(ctx->gpios.power, 1); > + msleep(20); > + gpiod_set_value(ctx->gpios.reset, 1); > + msleep(20); > + return 0; > +} > + > +static int rb070d30_panel_unprepare(struct drm_panel *panel) > +{ > + struct rb070d30_panel *ctx = panel_to_rb070d30_panel(panel); > + > + gpiod_set_value(ctx->gpios.reset, 0); > + gpiod_set_value(ctx->gpios.power, 0); > + regulator_disable(ctx->supply); > + > + return 0; > +} > + > +static int rb070d30_panel_enable(struct drm_panel *panel) > +{ > + struct rb070d30_panel *ctx = panel_to_rb070d30_panel(panel); > + int ret; > + > + ret = mipi_dsi_dcs_exit_sleep_mode(ctx->dsi); > + if (ret) > + return ret; > + > + ret = backlight_enable(ctx->backlight); > + if (ret) > + goto out; > + > + return 0; > + > +out: > + mipi_dsi_dcs_enter_sleep_mode(ctx->dsi); > + return ret; > +} > + > +static int rb070d30_panel_disable(struct drm_panel *panel) > +{ > + struct rb070d30_panel *ctx = panel_to_rb070d30_panel(panel); > + > + backlight_disable(ctx->backlight); > + return mipi_dsi_dcs_enter_sleep_mode(ctx->dsi); > +} > + > +/* Default timings */ > +static const struct drm_display_mode default_mode = { > + .clock = 51206, > + .hdisplay = 1024, > + .hsync_start = 1024 + 160, > + .hsync_end = 1024 + 160 + 80, > + .htotal = 1024 + 160 + 80 + 80, > + .vdisplay = 600, > + .vsync_start = 600 + 12, > + .vsync_end = 600 + 12 + 10, > + .vtotal = 600 + 12 + 10 + 13, > + .vrefresh = 60, > + > + .width_mm = 154, > + .height_mm = 85, > +}; > + > +static int rb070d30_panel_get_modes(struct drm_panel *panel) > +{ > + struct drm_connector *connector = panel->connector; > + struct rb070d30_panel *ctx = panel_to_rb070d30_panel(panel); > + struct drm_display_mode *mode; > + static const u32 bus_format = MEDIA_BUS_FMT_RGB888_1X24; > + > + mode = drm_mode_duplicate(panel->drm, &default_mode); > + if (!mode) { > + DRM_DEV_ERROR(&ctx->dsi->dev, > + "Failed to add mode " DRM_MODE_FMT "\n", > + DRM_MODE_ARG(&default_mode)); > + return -EINVAL; > + } > + > + drm_mode_set_name(mode); > + > + mode->type = DRM_MODE_TYPE_DRIVER | DRM_MODE_TYPE_PREFERRED; > + drm_mode_probed_add(connector, mode); > + > + panel->connector->display_info.bpc = 8; > + panel->connector->display_info.width_mm = mode->width_mm; > + panel->connector->display_info.height_mm = mode->height_mm; > + drm_display_info_set_bus_formats(&connector->display_info, > + &bus_format, 1); > + > + return 1; > +} > + > +static const struct drm_panel_funcs rb070d30_panel_funcs = { > + .get_modes = rb070d30_panel_get_modes, > + .prepare = rb070d30_panel_prepare, > + .enable = rb070d30_panel_enable, > + .disable = rb070d30_panel_disable, > + .unprepare = rb070d30_panel_unprepare, > +}; > + > +static int rb070d30_panel_dsi_probe(struct mipi_dsi_device *dsi) > +{ > + struct rb070d30_panel *ctx; > + int ret; > + > + ctx = devm_kzalloc(&dsi->dev, sizeof(*ctx), GFP_KERNEL); > + if (!ctx) > + return -ENOMEM; > + > + ctx->supply = devm_regulator_get(&dsi->dev, "vcc-lcd"); > + if (IS_ERR(ctx->supply)) > + return PTR_ERR(ctx->supply); > + > + mipi_dsi_set_drvdata(dsi, ctx); > + ctx->dsi = dsi; > + > + drm_panel_init(&ctx->panel); > + ctx->panel.dev = &dsi->dev; > + ctx->panel.funcs = &rb070d30_panel_funcs; > + > + ctx->gpios.reset = devm_gpiod_get(&dsi->dev, "reset", GPIOD_OUT_LOW); > + if (IS_ERR(ctx->gpios.reset)) { > + DRM_DEV_ERROR(&dsi->dev, "Couldn't get our reset GPIO\n"); > + return PTR_ERR(ctx->gpios.reset); > + } > + > + ctx->gpios.power = devm_gpiod_get(&dsi->dev, "power", GPIOD_OUT_LOW); > + if (IS_ERR(ctx->gpios.power)) { > + DRM_DEV_ERROR(&dsi->dev, "Couldn't get our power GPIO\n"); > + return PTR_ERR(ctx->gpios.power); > + } > + > + /* > + * We don't change the state of that GPIO later on but we need > + * to force it into a low state. > + */ > + ctx->gpios.updn = devm_gpiod_get(&dsi->dev, "updn", GPIOD_OUT_LOW); > + if (IS_ERR(ctx->gpios.updn)) { > + DRM_DEV_ERROR(&dsi->dev, "Couldn't get our updn GPIO\n"); > + return PTR_ERR(ctx->gpios.updn); > + } > + > + /* > + * We don't change the state of that GPIO later on but we need > + * to force it into a low state. > + */ > + ctx->gpios.shlr = devm_gpiod_get(&dsi->dev, "shlr", GPIOD_OUT_LOW); > + if (IS_ERR(ctx->gpios.shlr)) { > + DRM_DEV_ERROR(&dsi->dev, "Couldn't get our shlr GPIO\n"); > + return PTR_ERR(ctx->gpios.shlr); > + } > + > + ctx->backlight = devm_of_find_backlight(&dsi->dev); > + if (IS_ERR(ctx->backlight)) { > + DRM_DEV_ERROR(&dsi->dev, "Couldn't get our backlight\n"); > + return PTR_ERR(ctx->backlight); > + } > + > + ret = drm_panel_add(&ctx->panel); > + if (ret < 0) > + return ret; > + > + dsi->mode_flags = MIPI_DSI_MODE_VIDEO | MIPI_DSI_MODE_VIDEO_BURST | MIPI_DSI_MODE_LPM; > + dsi->format = MIPI_DSI_FMT_RGB888; > + dsi->lanes = 4; > + > + return mipi_dsi_attach(dsi); > +} > + > +static int rb070d30_panel_dsi_remove(struct mipi_dsi_device *dsi) > +{ > + struct rb070d30_panel *ctx = mipi_dsi_get_drvdata(dsi); > + > + mipi_dsi_detach(dsi); > + drm_panel_remove(&ctx->panel); > + > + return 0; > +} > + > +static const struct of_device_id rb070d30_panel_of_match[] = { > + { .compatible = "ronbo,rb070d30" }, > + { /* sentinel */ }, > +}; > +MODULE_DEVICE_TABLE(of, rb070d30_panel_of_match); > + > +static struct mipi_dsi_driver rb070d30_panel_driver = { > + .probe = rb070d30_panel_dsi_probe, > + .remove = rb070d30_panel_dsi_remove, > + .driver = { > + .name = "panel-ronbo-rb070d30", > + .of_match_table = rb070d30_panel_of_match, > + }, > +}; > +module_mipi_dsi_driver(rb070d30_panel_driver); > + > +MODULE_AUTHOR("Boris Brezillon <boris.brezillon@bootlin.com>"); > +MODULE_AUTHOR("Konstantin Sudakov <k.sudakov@integrasources.com>"); > +MODULE_DESCRIPTION("Ronbo RB070D30 Panel Driver"); > +MODULE_LICENSE("GPL"); -- Paul Kocialkowski, Bootlin Embedded Linux and kernel engineering https://bootlin.com _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply index Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-02-11 14:41 [PATCH v3 0/8] drm/sun4i: dsi: Add burst mode support Maxime Ripard 2019-02-11 14:41 ` [PATCH v3 1/8] drm/sun4i: dsi: Restrict DSI tcon clock divider Maxime Ripard 2019-02-11 14:41 ` [PATCH v3 2/8] drm/sun4i: dsi: Change the start delay calculation Maxime Ripard 2019-02-12 15:28 ` Paul Kocialkowski 2019-02-12 15:45 ` Maxime Ripard 2019-02-11 14:41 ` [PATCH v3 3/8] drm/sun4i: dsi: Enforce boundaries on the start delay Maxime Ripard 2019-02-12 15:30 ` Paul Kocialkowski 2019-02-14 9:21 ` Maxime Ripard 2019-02-11 14:41 ` [PATCH v3 4/8] drm/sun4i: dsi: Fix front vs back porch calculation Maxime Ripard 2019-02-12 15:41 ` Paul Kocialkowski 2019-02-11 14:41 ` [PATCH v3 5/8] drm/sun4i: dsi: Fix DRQ calculation Maxime Ripard 2019-02-13 14:33 ` Paul Kocialkowski 2019-02-14 9:23 ` Maxime Ripard 2019-02-11 14:41 ` [PATCH v3 6/8] drm/sun4i: dsi: Rework a bit the hblk calculation Maxime Ripard 2019-02-13 14:41 ` Paul Kocialkowski 2019-02-11 14:41 ` [PATCH v3 7/8] drm/sun4i: dsi: Add burst support Maxime Ripard 2019-02-13 14:36 ` Paul Kocialkowski 2019-02-14 9:15 ` Maxime Ripard 2019-02-11 14:41 ` [PATCH v3 8/8] drm/panel: Add Ronbo RB070D30 panel Maxime Ripard 2019-02-11 15:13 ` Sam Ravnborg 2019-02-13 14:26 ` Paul Kocialkowski [this message] 2019-02-14 11:07 ` Re[2]: " Konstantin Sudakov 2019-02-12 10:32 ` [PATCH v3 0/8] drm/sun4i: dsi: Add burst mode support Jagan Teki 2019-02-15 14:12 ` Maxime Ripard [not found] ` <CAMty3ZAA90-fHPADJSE3ht9CiYWA3yBoyEy7wVv9=6C5JiaTVw@mail.gmail.com> 2019-02-15 17:07 ` Jagan Teki 2019-02-18 8:26 ` Paul Kocialkowski 2019-02-18 10:31 ` Jagan Teki 2019-02-20 16:35 ` Maxime Ripard 2019-02-26 6:48 ` Jagan Teki
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=25f4beadd3088b3fbb2751f120e4216969301136.camel@bootlin.com \ --to=paul.kocialkowski@bootlin.com \ --cc=bbrezillon@kernel.org \ --cc=dri-devel@lists.freedesktop.org \ --cc=jagan@openedev.com \ --cc=k.sudakov@integrasources.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=maarten.lankhorst@linux.intel.com \ --cc=maxime.ripard@bootlin.com \ --cc=sam@ravnborg.org \ --cc=seanpaul@chromium.org \ --cc=thomas.petazzoni@bootlin.com \ --cc=wens@csie.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Linux-ARM-Kernel Archive on lore.kernel.org Archives are clonable: git clone --mirror https://lore.kernel.org/linux-arm-kernel/0 linux-arm-kernel/git/0.git git clone --mirror https://lore.kernel.org/linux-arm-kernel/1 linux-arm-kernel/git/1.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 linux-arm-kernel linux-arm-kernel/ https://lore.kernel.org/linux-arm-kernel \ linux-arm-kernel@lists.infradead.org public-inbox-index linux-arm-kernel Example config snippet for mirrors Newsgroup available over NNTP: nntp://nntp.lore.kernel.org/org.infradead.lists.linux-arm-kernel AGPL code for this site: git clone https://public-inbox.org/public-inbox.git