From: Sebastian Reichel <sre@kernel.org>
To: Tomi Valkeinen <tomi.valkeinen@ti.com>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Nikhil Devshatwar <nikhil.nd@ti.com>,
linux-omap@vger.kernel.org, dri-devel@lists.freedesktop.org,
Sekhar Nori <nsekhar@ti.com>, Tony Lindgren <tony@atomide.com>,
hns@goldelico.com
Subject: Re: [PATCH v4 50/80] drm/omap: squash omapdrm sub-modules into one
Date: Mon, 14 Dec 2020 13:53:15 +0100 [thread overview]
Message-ID: <20201214125315.234l3hln3d4txfsl@earth.universe> (raw)
In-Reply-To: <20201124124538.660710-51-tomi.valkeinen@ti.com>
[-- Attachment #1: Type: text/plain, Size: 22246 bytes --]
Hi,
On Tue, Nov 24, 2020 at 02:45:08PM +0200, Tomi Valkeinen wrote:
> At the moment we have three different modules: omapdss-base, omapdss,
> omapdrm. This setup is finally obsolete, as the last omapdrm specific
> panel has been converted to DRM panel.
>
> We can thus remove omapdss-base and omapdss, and just compile everything
> into omapdrm.ko.
>
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
Reviewed-by: Sebastian Reichel <sebastian.reichel@collabora.com>
-- Sebastian
> drivers/gpu/drm/omapdrm/Kconfig | 119 ++++++++++++++++++++++-
> drivers/gpu/drm/omapdrm/Makefile | 17 +++-
> drivers/gpu/drm/omapdrm/dss/Kconfig | 132 --------------------------
> drivers/gpu/drm/omapdrm/dss/Makefile | 18 ----
> drivers/gpu/drm/omapdrm/dss/base.c | 24 +----
> drivers/gpu/drm/omapdrm/dss/display.c | 2 -
> drivers/gpu/drm/omapdrm/dss/dss.c | 11 +--
> drivers/gpu/drm/omapdrm/dss/omapdss.h | 3 +
> drivers/gpu/drm/omapdrm/dss/output.c | 11 ---
> drivers/gpu/drm/omapdrm/omap_drv.c | 20 +++-
> 10 files changed, 155 insertions(+), 202 deletions(-)
> delete mode 100644 drivers/gpu/drm/omapdrm/dss/Kconfig
> delete mode 100644 drivers/gpu/drm/omapdrm/dss/Makefile
>
> diff --git a/drivers/gpu/drm/omapdrm/Kconfig b/drivers/gpu/drm/omapdrm/Kconfig
> index cea3f44ea6d4..e7281da5bc6a 100644
> --- a/drivers/gpu/drm/omapdrm/Kconfig
> +++ b/drivers/gpu/drm/omapdrm/Kconfig
> @@ -5,12 +5,129 @@ config DRM_OMAP
> depends on ARCH_OMAP2PLUS || ARCH_MULTIPLATFORM
> select OMAP2_DSS
> select DRM_KMS_HELPER
> + select VIDEOMODE_HELPERS
> + select HDMI
> default n
> help
> DRM display driver for OMAP2/3/4 based boards.
>
> if DRM_OMAP
>
> -source "drivers/gpu/drm/omapdrm/dss/Kconfig"
> +config OMAP2_DSS_DEBUG
> + bool "Debug support"
> + default n
> + help
> + This enables printing of debug messages. Alternatively, debug messages
> + can also be enabled by setting CONFIG_DYNAMIC_DEBUG and then setting
> + appropriate flags in <debugfs>/dynamic_debug/control.
> +
> +config OMAP2_DSS_DEBUGFS
> + bool "Debugfs filesystem support"
> + depends on DEBUG_FS
> + default n
> + help
> + This enables debugfs for OMAPDSS at <debugfs>/omapdss. This enables
> + querying about clock configuration and register configuration of dss,
> + dispc, dsi, hdmi and rfbi.
> +
> +config OMAP2_DSS_COLLECT_IRQ_STATS
> + bool "Collect DSS IRQ statistics"
> + depends on OMAP2_DSS_DEBUGFS
> + default n
> + help
> + Collect DSS IRQ statistics, printable via debugfs.
> +
> + The statistics can be found from
> + <debugfs>/omapdss/dispc_irq for DISPC interrupts, and
> + <debugfs>/omapdss/dsi_irq for DSI interrupts.
> +
> +config OMAP2_DSS_DPI
> + bool "DPI support"
> + default y
> + help
> + DPI Interface. This is the Parallel Display Interface.
> +
> +config OMAP2_DSS_VENC
> + bool "VENC support"
> + default y
> + help
> + OMAP Video Encoder support for S-Video and composite TV-out.
> +
> +config OMAP2_DSS_HDMI_COMMON
> + bool
> +
> +config OMAP4_DSS_HDMI
> + bool "HDMI support for OMAP4"
> + default y
> + select OMAP2_DSS_HDMI_COMMON
> + help
> + HDMI support for OMAP4 based SoCs.
> +
> +config OMAP4_DSS_HDMI_CEC
> + bool "Enable HDMI CEC support for OMAP4"
> + depends on OMAP4_DSS_HDMI
> + select CEC_CORE
> + default y
> + help
> + When selected the HDMI transmitter will support the CEC feature.
> +
> +config OMAP5_DSS_HDMI
> + bool "HDMI support for OMAP5"
> + default n
> + select OMAP2_DSS_HDMI_COMMON
> + help
> + HDMI Interface for OMAP5 and similar cores. This adds the High
> + Definition Multimedia Interface. See http://www.hdmi.org/ for HDMI
> + specification.
> +
> +config OMAP2_DSS_SDI
> + bool "SDI support"
> + default n
> + help
> + SDI (Serial Display Interface) support.
> +
> + SDI is a high speed one-way display serial bus between the host
> + processor and a display.
> +
> +config OMAP2_DSS_DSI
> + bool "DSI support"
> + default n
> + select DRM_MIPI_DSI
> + help
> + MIPI DSI (Display Serial Interface) support.
> +
> + DSI is a high speed half-duplex serial interface between the host
> + processor and a peripheral, such as a display or a framebuffer chip.
> +
> + See http://www.mipi.org/ for DSI specifications.
> +
> +config OMAP2_DSS_MIN_FCK_PER_PCK
> + int "Minimum FCK/PCK ratio (for scaling)"
> + range 0 32
> + default 0
> + help
> + This can be used to adjust the minimum FCK/PCK ratio.
> +
> + With this you can make sure that DISPC FCK is at least
> + n x PCK. Video plane scaling requires higher FCK than
> + normally.
> +
> + If this is set to 0, there's no extra constraint on the
> + DISPC FCK. However, the FCK will at minimum be
> + 2xPCK (if active matrix) or 3xPCK (if passive matrix).
> +
> + Max FCK is 173MHz, so this doesn't work if your PCK
> + is very high.
> +
> +config OMAP2_DSS_SLEEP_AFTER_VENC_RESET
> + bool "Sleep 20ms after VENC reset"
> + default y
> + help
> + There is a 20ms sleep after VENC reset which seemed to fix the
> + reset. The reason for the bug is unclear, and it's also unclear
> + on what platforms this happens.
> +
> + This option enables the sleep, and is enabled by default. You can
> + disable the sleep if it doesn't cause problems on your platform.
>
> endif
> diff --git a/drivers/gpu/drm/omapdrm/Makefile b/drivers/gpu/drm/omapdrm/Makefile
> index 2a404e045b78..33fe7e937680 100644
> --- a/drivers/gpu/drm/omapdrm/Makefile
> +++ b/drivers/gpu/drm/omapdrm/Makefile
> @@ -4,8 +4,6 @@
> # Direct Rendering Infrastructure (DRI)
> #
>
> -obj-y += dss/
> -
> omapdrm-y := omap_drv.o \
> omap_irq.o \
> omap_debugfs.o \
> @@ -20,4 +18,17 @@ omapdrm-y := omap_drv.o \
>
> omapdrm-$(CONFIG_DRM_FBDEV_EMULATION) += omap_fbdev.o
>
> -obj-$(CONFIG_DRM_OMAP) += omapdrm.o
> +omapdrm-y += dss/base.o dss/display.o dss/output.o dss/dss.o dss/dispc.o \
> + dss/dispc_coefs.o dss/pll.o dss/video-pll.o
> +omapdrm-$(CONFIG_OMAP2_DSS_DPI) += dss/dpi.o
> +omapdrm-$(CONFIG_OMAP2_DSS_VENC) += dss/venc.o
> +omapdrm-$(CONFIG_OMAP2_DSS_SDI) += dss/sdi.o
> +omapdrm-$(CONFIG_OMAP2_DSS_DSI) += dss/dsi.o
> +omapdrm-$(CONFIG_OMAP2_DSS_HDMI_COMMON) += dss/hdmi_common.o dss/hdmi_wp.o \
> + dss/hdmi_pll.o dss/hdmi_phy.o
> +omapdrm-$(CONFIG_OMAP4_DSS_HDMI) += dss/hdmi4.o dss/hdmi4_core.o
> +omapdrm-$(CONFIG_OMAP4_DSS_HDMI_CEC) += dss/hdmi4_cec.o
> +omapdrm-$(CONFIG_OMAP5_DSS_HDMI) += dss/hdmi5.o dss/hdmi5_core.o
> +ccflags-$(CONFIG_OMAP2_DSS_DEBUG) += -DDEBUG
> +
> +obj-$(CONFIG_DRM_OMAP) += omapdrm.o
> diff --git a/drivers/gpu/drm/omapdrm/dss/Kconfig b/drivers/gpu/drm/omapdrm/dss/Kconfig
> deleted file mode 100644
> index 6ec80771af3d..000000000000
> --- a/drivers/gpu/drm/omapdrm/dss/Kconfig
> +++ /dev/null
> @@ -1,132 +0,0 @@
> -# SPDX-License-Identifier: GPL-2.0-only
> -config OMAP_DSS_BASE
> - tristate
> -
> -menuconfig OMAP2_DSS
> - tristate "OMAP2+ Display Subsystem support"
> - select OMAP_DSS_BASE
> - select VIDEOMODE_HELPERS
> - select HDMI
> - help
> - OMAP2+ Display Subsystem support.
> -
> -if OMAP2_DSS
> -
> -config OMAP2_DSS_DEBUG
> - bool "Debug support"
> - default n
> - help
> - This enables printing of debug messages. Alternatively, debug messages
> - can also be enabled by setting CONFIG_DYNAMIC_DEBUG and then setting
> - appropriate flags in <debugfs>/dynamic_debug/control.
> -
> -config OMAP2_DSS_DEBUGFS
> - bool "Debugfs filesystem support"
> - depends on DEBUG_FS
> - default n
> - help
> - This enables debugfs for OMAPDSS at <debugfs>/omapdss. This enables
> - querying about clock configuration and register configuration of dss,
> - dispc, dsi, hdmi and rfbi.
> -
> -config OMAP2_DSS_COLLECT_IRQ_STATS
> - bool "Collect DSS IRQ statistics"
> - depends on OMAP2_DSS_DEBUGFS
> - default n
> - help
> - Collect DSS IRQ statistics, printable via debugfs.
> -
> - The statistics can be found from
> - <debugfs>/omapdss/dispc_irq for DISPC interrupts, and
> - <debugfs>/omapdss/dsi_irq for DSI interrupts.
> -
> -config OMAP2_DSS_DPI
> - bool "DPI support"
> - default y
> - help
> - DPI Interface. This is the Parallel Display Interface.
> -
> -config OMAP2_DSS_VENC
> - bool "VENC support"
> - default y
> - help
> - OMAP Video Encoder support for S-Video and composite TV-out.
> -
> -config OMAP2_DSS_HDMI_COMMON
> - bool
> -
> -config OMAP4_DSS_HDMI
> - bool "HDMI support for OMAP4"
> - default y
> - select OMAP2_DSS_HDMI_COMMON
> - help
> - HDMI support for OMAP4 based SoCs.
> -
> -config OMAP4_DSS_HDMI_CEC
> - bool "Enable HDMI CEC support for OMAP4"
> - depends on OMAP4_DSS_HDMI
> - select CEC_CORE
> - default y
> - help
> - When selected the HDMI transmitter will support the CEC feature.
> -
> -config OMAP5_DSS_HDMI
> - bool "HDMI support for OMAP5"
> - default n
> - select OMAP2_DSS_HDMI_COMMON
> - help
> - HDMI Interface for OMAP5 and similar cores. This adds the High
> - Definition Multimedia Interface. See http://www.hdmi.org/ for HDMI
> - specification.
> -
> -config OMAP2_DSS_SDI
> - bool "SDI support"
> - default n
> - help
> - SDI (Serial Display Interface) support.
> -
> - SDI is a high speed one-way display serial bus between the host
> - processor and a display.
> -
> -config OMAP2_DSS_DSI
> - bool "DSI support"
> - default n
> - select DRM_MIPI_DSI
> - help
> - MIPI DSI (Display Serial Interface) support.
> -
> - DSI is a high speed half-duplex serial interface between the host
> - processor and a peripheral, such as a display or a framebuffer chip.
> -
> - See http://www.mipi.org/ for DSI specifications.
> -
> -config OMAP2_DSS_MIN_FCK_PER_PCK
> - int "Minimum FCK/PCK ratio (for scaling)"
> - range 0 32
> - default 0
> - help
> - This can be used to adjust the minimum FCK/PCK ratio.
> -
> - With this you can make sure that DISPC FCK is at least
> - n x PCK. Video plane scaling requires higher FCK than
> - normally.
> -
> - If this is set to 0, there's no extra constraint on the
> - DISPC FCK. However, the FCK will at minimum be
> - 2xPCK (if active matrix) or 3xPCK (if passive matrix).
> -
> - Max FCK is 173MHz, so this doesn't work if your PCK
> - is very high.
> -
> -config OMAP2_DSS_SLEEP_AFTER_VENC_RESET
> - bool "Sleep 20ms after VENC reset"
> - default y
> - help
> - There is a 20ms sleep after VENC reset which seemed to fix the
> - reset. The reason for the bug is unclear, and it's also unclear
> - on what platforms this happens.
> -
> - This option enables the sleep, and is enabled by default. You can
> - disable the sleep if it doesn't cause problems on your platform.
> -
> -endif
> diff --git a/drivers/gpu/drm/omapdrm/dss/Makefile b/drivers/gpu/drm/omapdrm/dss/Makefile
> deleted file mode 100644
> index 811966cd7468..000000000000
> --- a/drivers/gpu/drm/omapdrm/dss/Makefile
> +++ /dev/null
> @@ -1,18 +0,0 @@
> -# SPDX-License-Identifier: GPL-2.0
> -obj-$(CONFIG_OMAP_DSS_BASE) += omapdss-base.o
> -omapdss-base-y := base.o display.o output.o
> -
> -obj-$(CONFIG_OMAP2_DSS) += omapdss.o
> -# Core DSS files
> -omapdss-y := dss.o dispc.o dispc_coefs.o \
> - pll.o video-pll.o
> -omapdss-$(CONFIG_OMAP2_DSS_DPI) += dpi.o
> -omapdss-$(CONFIG_OMAP2_DSS_VENC) += venc.o
> -omapdss-$(CONFIG_OMAP2_DSS_SDI) += sdi.o
> -omapdss-$(CONFIG_OMAP2_DSS_DSI) += dsi.o
> -omapdss-$(CONFIG_OMAP2_DSS_HDMI_COMMON) += hdmi_common.o hdmi_wp.o hdmi_pll.o \
> - hdmi_phy.o
> -omapdss-$(CONFIG_OMAP4_DSS_HDMI) += hdmi4.o hdmi4_core.o
> -omapdss-$(CONFIG_OMAP4_DSS_HDMI_CEC) += hdmi4_cec.o
> -omapdss-$(CONFIG_OMAP5_DSS_HDMI) += hdmi5.o hdmi5_core.o
> -ccflags-$(CONFIG_OMAP2_DSS_DEBUG) += -DDEBUG
> diff --git a/drivers/gpu/drm/omapdrm/dss/base.c b/drivers/gpu/drm/omapdrm/dss/base.c
> index d0d5d9c8ca76..6c57d9292415 100644
> --- a/drivers/gpu/drm/omapdrm/dss/base.c
> +++ b/drivers/gpu/drm/omapdrm/dss/base.c
> @@ -20,14 +20,11 @@ struct dispc_device *dispc_get_dispc(struct dss_device *dss)
> {
> return dss->dispc;
> }
> -EXPORT_SYMBOL(dispc_get_dispc);
>
> const struct dispc_ops *dispc_get_ops(struct dss_device *dss)
> {
> return dss->dispc_ops;
> }
> -EXPORT_SYMBOL(dispc_get_ops);
> -
>
> /* -----------------------------------------------------------------------------
> * OMAP DSS Devices Handling
> @@ -42,7 +39,6 @@ void omapdss_device_register(struct omap_dss_device *dssdev)
> list_add_tail(&dssdev->list, &omapdss_devices_list);
> mutex_unlock(&omapdss_devices_lock);
> }
> -EXPORT_SYMBOL_GPL(omapdss_device_register);
>
> void omapdss_device_unregister(struct omap_dss_device *dssdev)
> {
> @@ -50,7 +46,6 @@ void omapdss_device_unregister(struct omap_dss_device *dssdev)
> list_del(&dssdev->list);
> mutex_unlock(&omapdss_devices_lock);
> }
> -EXPORT_SYMBOL_GPL(omapdss_device_unregister);
>
> static bool omapdss_device_is_registered(struct device_node *node)
> {
> @@ -72,24 +67,16 @@ static bool omapdss_device_is_registered(struct device_node *node)
>
> struct omap_dss_device *omapdss_device_get(struct omap_dss_device *dssdev)
> {
> - if (!try_module_get(dssdev->owner))
> + if (get_device(dssdev->dev) == NULL)
> return NULL;
>
> - if (get_device(dssdev->dev) == NULL) {
> - module_put(dssdev->owner);
> - return NULL;
> - }
> -
> return dssdev;
> }
> -EXPORT_SYMBOL(omapdss_device_get);
>
> void omapdss_device_put(struct omap_dss_device *dssdev)
> {
> put_device(dssdev->dev);
> - module_put(dssdev->owner);
> }
> -EXPORT_SYMBOL(omapdss_device_put);
>
> struct omap_dss_device *omapdss_find_device_by_node(struct device_node *node)
> {
> @@ -150,7 +137,6 @@ struct omap_dss_device *omapdss_device_next_output(struct omap_dss_device *from)
> mutex_unlock(&omapdss_devices_lock);
> return dssdev;
> }
> -EXPORT_SYMBOL(omapdss_device_next_output);
>
> static bool omapdss_device_is_connected(struct omap_dss_device *dssdev)
> {
> @@ -181,7 +167,6 @@ int omapdss_device_connect(struct dss_device *dss,
>
> return 0;
> }
> -EXPORT_SYMBOL_GPL(omapdss_device_connect);
>
> void omapdss_device_disconnect(struct omap_dss_device *src,
> struct omap_dss_device *dst)
> @@ -204,7 +189,6 @@ void omapdss_device_disconnect(struct omap_dss_device *src,
>
> dst->dss = NULL;
> }
> -EXPORT_SYMBOL_GPL(omapdss_device_disconnect);
>
> /* -----------------------------------------------------------------------------
> * Components Handling
> @@ -290,7 +274,6 @@ void omapdss_gather_components(struct device *dev)
> for_each_available_child_of_node(dev->of_node, child)
> omapdss_walk_device(dev, child, true);
> }
> -EXPORT_SYMBOL(omapdss_gather_components);
>
> static bool omapdss_component_is_loaded(struct omapdss_comp_node *comp)
> {
> @@ -315,8 +298,3 @@ bool omapdss_stack_is_ready(void)
>
> return true;
> }
> -EXPORT_SYMBOL(omapdss_stack_is_ready);
> -
> -MODULE_AUTHOR("Tomi Valkeinen <tomi.valkeinen@ti.com>");
> -MODULE_DESCRIPTION("OMAP Display Subsystem Base");
> -MODULE_LICENSE("GPL v2");
> diff --git a/drivers/gpu/drm/omapdrm/dss/display.c b/drivers/gpu/drm/omapdrm/dss/display.c
> index 3b82158b1bfd..7b7ee2019eae 100644
> --- a/drivers/gpu/drm/omapdrm/dss/display.c
> +++ b/drivers/gpu/drm/omapdrm/dss/display.c
> @@ -38,7 +38,6 @@ void omapdss_display_init(struct omap_dss_device *dssdev)
> dssdev->name = devm_kasprintf(dssdev->dev, GFP_KERNEL,
> "display%u", id);
> }
> -EXPORT_SYMBOL_GPL(omapdss_display_init);
>
> int omapdss_display_get_modes(struct drm_connector *connector,
> const struct videomode *vm)
> @@ -57,4 +56,3 @@ int omapdss_display_get_modes(struct drm_connector *connector,
>
> return 1;
> }
> -EXPORT_SYMBOL_GPL(omapdss_display_get_modes);
> diff --git a/drivers/gpu/drm/omapdrm/dss/dss.c b/drivers/gpu/drm/omapdrm/dss/dss.c
> index 6a160138bf88..2d566ff81c55 100644
> --- a/drivers/gpu/drm/omapdrm/dss/dss.c
> +++ b/drivers/gpu/drm/omapdrm/dss/dss.c
> @@ -1638,21 +1638,14 @@ static struct platform_driver * const omap_dss_drivers[] = {
> #endif
> };
>
> -static int __init omap_dss_init(void)
> +int __init omap_dss_init(void)
> {
> return platform_register_drivers(omap_dss_drivers,
> ARRAY_SIZE(omap_dss_drivers));
> }
>
> -static void __exit omap_dss_exit(void)
> +void omap_dss_exit(void)
> {
> platform_unregister_drivers(omap_dss_drivers,
> ARRAY_SIZE(omap_dss_drivers));
> }
> -
> -module_init(omap_dss_init);
> -module_exit(omap_dss_exit);
> -
> -MODULE_AUTHOR("Tomi Valkeinen <tomi.valkeinen@ti.com>");
> -MODULE_DESCRIPTION("OMAP2/3/4/5 Display Subsystem");
> -MODULE_LICENSE("GPL v2");
> diff --git a/drivers/gpu/drm/omapdrm/dss/omapdss.h b/drivers/gpu/drm/omapdrm/dss/omapdss.h
> index 4a0826c8fed5..4e7ae164de4f 100644
> --- a/drivers/gpu/drm/omapdrm/dss/omapdss.h
> +++ b/drivers/gpu/drm/omapdrm/dss/omapdss.h
> @@ -458,4 +458,7 @@ const struct dispc_ops *dispc_get_ops(struct dss_device *dss);
> bool omapdss_stack_is_ready(void);
> void omapdss_gather_components(struct device *dev);
>
> +int omap_dss_init(void);
> +void omap_dss_exit(void);
> +
> #endif /* __OMAP_DRM_DSS_H */
> diff --git a/drivers/gpu/drm/omapdrm/dss/output.c b/drivers/gpu/drm/omapdrm/dss/output.c
> index 40cb353572f6..7a14d2b5b2f7 100644
> --- a/drivers/gpu/drm/omapdrm/dss/output.c
> +++ b/drivers/gpu/drm/omapdrm/dss/output.c
> @@ -73,7 +73,6 @@ int omapdss_device_init_output(struct omap_dss_device *out,
> omapdss_device_cleanup_output(out);
> return ret;
> }
> -EXPORT_SYMBOL(omapdss_device_init_output);
>
> void omapdss_device_cleanup_output(struct omap_dss_device *out)
> {
> @@ -81,7 +80,6 @@ void omapdss_device_cleanup_output(struct omap_dss_device *out)
> drm_panel_bridge_remove(out->next_bridge ?
> out->next_bridge : out->bridge);
> }
> -EXPORT_SYMBOL(omapdss_device_cleanup_output);
>
> int dss_install_mgr_ops(struct dss_device *dss,
> const struct dss_mgr_ops *mgr_ops,
> @@ -95,14 +93,12 @@ int dss_install_mgr_ops(struct dss_device *dss,
>
> return 0;
> }
> -EXPORT_SYMBOL(dss_install_mgr_ops);
>
> void dss_uninstall_mgr_ops(struct dss_device *dss)
> {
> dss->mgr_ops = NULL;
> dss->mgr_ops_priv = NULL;
> }
> -EXPORT_SYMBOL(dss_uninstall_mgr_ops);
>
> void dss_mgr_set_timings(struct omap_dss_device *dssdev,
> const struct videomode *vm)
> @@ -110,7 +106,6 @@ void dss_mgr_set_timings(struct omap_dss_device *dssdev,
> dssdev->dss->mgr_ops->set_timings(dssdev->dss->mgr_ops_priv,
> dssdev->dispc_channel, vm);
> }
> -EXPORT_SYMBOL(dss_mgr_set_timings);
>
> void dss_mgr_set_lcd_config(struct omap_dss_device *dssdev,
> const struct dss_lcd_mgr_config *config)
> @@ -118,28 +113,24 @@ void dss_mgr_set_lcd_config(struct omap_dss_device *dssdev,
> dssdev->dss->mgr_ops->set_lcd_config(dssdev->dss->mgr_ops_priv,
> dssdev->dispc_channel, config);
> }
> -EXPORT_SYMBOL(dss_mgr_set_lcd_config);
>
> int dss_mgr_enable(struct omap_dss_device *dssdev)
> {
> return dssdev->dss->mgr_ops->enable(dssdev->dss->mgr_ops_priv,
> dssdev->dispc_channel);
> }
> -EXPORT_SYMBOL(dss_mgr_enable);
>
> void dss_mgr_disable(struct omap_dss_device *dssdev)
> {
> dssdev->dss->mgr_ops->disable(dssdev->dss->mgr_ops_priv,
> dssdev->dispc_channel);
> }
> -EXPORT_SYMBOL(dss_mgr_disable);
>
> void dss_mgr_start_update(struct omap_dss_device *dssdev)
> {
> dssdev->dss->mgr_ops->start_update(dssdev->dss->mgr_ops_priv,
> dssdev->dispc_channel);
> }
> -EXPORT_SYMBOL(dss_mgr_start_update);
>
> int dss_mgr_register_framedone_handler(struct omap_dss_device *dssdev,
> void (*handler)(void *), void *data)
> @@ -150,7 +141,6 @@ int dss_mgr_register_framedone_handler(struct omap_dss_device *dssdev,
> dssdev->dispc_channel,
> handler, data);
> }
> -EXPORT_SYMBOL(dss_mgr_register_framedone_handler);
>
> void dss_mgr_unregister_framedone_handler(struct omap_dss_device *dssdev,
> void (*handler)(void *), void *data)
> @@ -161,4 +151,3 @@ void dss_mgr_unregister_framedone_handler(struct omap_dss_device *dssdev,
> dssdev->dispc_channel,
> handler, data);
> }
> -EXPORT_SYMBOL(dss_mgr_unregister_framedone_handler);
> diff --git a/drivers/gpu/drm/omapdrm/omap_drv.c b/drivers/gpu/drm/omapdrm/omap_drv.c
> index dbb8e95234b3..b6ba4a43f217 100644
> --- a/drivers/gpu/drm/omapdrm/omap_drv.c
> +++ b/drivers/gpu/drm/omapdrm/omap_drv.c
> @@ -741,9 +741,21 @@ static struct platform_driver * const drivers[] = {
>
> static int __init omap_drm_init(void)
> {
> + int r;
> +
> DBG("init");
>
> - return platform_register_drivers(drivers, ARRAY_SIZE(drivers));
> + r = omap_dss_init();
> + if (r)
> + return r;
> +
> + r = platform_register_drivers(drivers, ARRAY_SIZE(drivers));
> + if (r) {
> + omap_dss_exit();
> + return r;
> + }
> +
> + return 0;
> }
>
> static void __exit omap_drm_fini(void)
> @@ -751,13 +763,15 @@ static void __exit omap_drm_fini(void)
> DBG("fini");
>
> platform_unregister_drivers(drivers, ARRAY_SIZE(drivers));
> +
> + omap_dss_exit();
> }
>
> -/* need late_initcall() so we load after dss_driver's are loaded */
> -late_initcall(omap_drm_init);
> +module_init(omap_drm_init);
> module_exit(omap_drm_fini);
>
> MODULE_AUTHOR("Rob Clark <rob@ti.com>");
> +MODULE_AUTHOR("Tomi Valkeinen <tomi.valkeinen@ti.com>");
> MODULE_DESCRIPTION("OMAP DRM Display Driver");
> MODULE_ALIAS("platform:" DRIVER_NAME);
> MODULE_LICENSE("GPL v2");
> --
> Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
> Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2020-12-14 12:54 UTC|newest]
Thread overview: 152+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-24 12:44 [PATCH v4 00/80] Convert DSI code to use drm_mipi_dsi and drm_panel Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 01/80] Revert "drm/omap: dss: Remove unused omap_dss_device operations" Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 02/80] drm/omap: drop unused dsi.configure_pins Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 03/80] drm/omap: dsi: use MIPI_DSI_FMT_* instead of OMAP_DSS_DSI_FMT_* Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 04/80] drm/omap: constify write buffers Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 05/80] drm/omap: dsi: add generic transfer function Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 06/80] drm/omap: panel-dsi-cm: convert to transfer API Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 07/80] drm/omap: dsi: unexport specific data transfer functions Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 08/80] drm/omap: dsi: drop virtual channel logic Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 09/80] drm/omap: dsi: simplify write function Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 10/80] drm/omap: dsi: simplify read functions Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 11/80] drm/omap: dsi: switch dsi_vc_send_long/short to mipi_dsi_msg Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 12/80] drm/omap: dsi: introduce mipi_dsi_host Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 13/80] drm/omap: panel-dsi-cm: use DSI helpers Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 14/80] drm/omap: dsi: request VC via mipi_dsi_attach Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 15/80] drm/omap: panel-dsi-cm: drop hardcoded VC Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 16/80] drm/omap: panel-dsi-cm: use common MIPI DCS 1.3 defines Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 17/80] drm/omap: dsi: drop unused memory_read() Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 18/80] drm/omap: dsi: drop unused get_te() Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 19/80] drm/omap: dsi: drop unused enable_te() Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 20/80] drm/omap: dsi: drop useless sync() Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 21/80] drm/omap: dsi: use pixel-format and mode from attach Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 22/80] drm/omap: panel-dsi-cm: use bulk regulator API Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 23/80] drm/omap: dsi: lp/hs switching support for transfer() Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 24/80] drm/omap: dsi: move TE GPIO handling into core Tomi Valkeinen
2021-02-08 17:55 ` Tony Lindgren
2021-02-11 7:34 ` Tomi Valkeinen
2021-02-11 17:35 ` Tony Lindgren
2021-02-17 7:42 ` Tomi Valkeinen
2021-02-18 5:57 ` Tony Lindgren
2021-02-22 8:47 ` Tomi Valkeinen
2021-02-25 12:46 ` Tony Lindgren
2021-02-26 9:04 ` Tomi Valkeinen
2021-02-27 7:18 ` Tony Lindgren
2020-11-24 12:44 ` [PATCH v4 25/80] drm/omap: dsi: drop custom enable_te() API Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 26/80] drm/omap: dsi: do bus locking in host driver Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 27/80] drm/omap: dsi: untangle ulps ops from enable/disable Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 28/80] drm/omap: dsi: do ULPS in host driver Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 29/80] drm/omap: dsi: move panel refresh function to host Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 30/80] drm/omap: dsi: Reverse direction of the DSS device enable/disable operations Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 31/80] drm/omap: dsi: drop custom panel capability support Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 32/80] drm/omap: dsi: convert to drm_panel Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 33/80] drm/omap: drop omapdss-boot-init Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 34/80] drm/omap: dsi: implement check timings Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 35/80] drm/omap: panel-dsi-cm: use DEVICE_ATTR_RO Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 36/80] drm/omap: panel-dsi-cm: support unbinding Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 37/80] drm/omap: panel-dsi-cm: fix remove() Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 38/80] drm/omap: remove global dss_device variable Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 39/80] drm/panel: Move OMAP's DSI command mode panel driver Tomi Valkeinen
2021-01-08 21:23 ` Dixit, Ashutosh
2021-01-08 21:54 ` Sebastian Reichel
2020-11-24 12:44 ` [PATCH v4 40/80] drm/omap: dsi: Register a drm_bridge Tomi Valkeinen
2020-11-24 12:44 ` [PATCH v4 41/80] drm/omap: remove legacy DSS device operations Tomi Valkeinen
2020-11-24 12:45 ` [PATCH v4 42/80] drm/omap: remove unused omap_connector Tomi Valkeinen
2020-11-24 12:45 ` [PATCH v4 43/80] drm/omap: simplify omap_display_id Tomi Valkeinen
2020-11-24 12:45 ` [PATCH v4 44/80] drm/omap: drop unused DSS next pointer Tomi Valkeinen
2020-11-24 12:45 ` [PATCH v4 45/80] drm/omap: drop DSS ops_flags Tomi Valkeinen
2020-11-24 12:45 ` [PATCH v4 46/80] drm/omap: drop dssdev display field Tomi Valkeinen
2020-11-24 12:45 ` [PATCH v4 47/80] drm/omap: simplify DSI manual update code Tomi Valkeinen
2020-11-24 12:45 ` [PATCH v4 48/80] drm/omap: dsi: simplify pin config Tomi Valkeinen
2020-11-24 12:45 ` [PATCH v4 49/80] ARM: omap2plus_defconfig: Update for moved DSI command mode panel Tomi Valkeinen
2020-12-01 8:46 ` Tony Lindgren
2020-11-24 12:45 ` [PATCH v4 50/80] drm/omap: squash omapdrm sub-modules into one Tomi Valkeinen
2020-12-14 12:53 ` Sebastian Reichel [this message]
2020-11-24 12:45 ` [PATCH v4 51/80] drm/omap: remove unused display.c Tomi Valkeinen
2020-12-14 12:53 ` Sebastian Reichel
2020-11-24 12:45 ` [PATCH v4 52/80] drm/omap: drop unused owner field Tomi Valkeinen
2020-12-14 12:53 ` Sebastian Reichel
2020-11-24 12:45 ` [PATCH v4 53/80] drm/omap: remove dispc_ops Tomi Valkeinen
2020-12-14 12:54 ` Sebastian Reichel
2020-11-24 12:45 ` [PATCH v4 54/80] drm/omap: remove dss_mgr_ops Tomi Valkeinen
2020-12-14 12:54 ` Sebastian Reichel
2020-11-24 12:45 ` [PATCH v4 55/80] drm/panel: panel-dsi-cm: use MIPI_DCS_GET_ERROR_COUNT_ON_DSI Tomi Valkeinen
2020-11-24 16:18 ` Sam Ravnborg
2020-11-24 16:26 ` Tomi Valkeinen
2020-11-24 16:38 ` Sam Ravnborg
2020-11-25 8:52 ` Tomi Valkeinen
2020-11-30 9:50 ` Laurent Pinchart
2020-12-14 12:54 ` Sebastian Reichel
2020-11-24 12:45 ` [PATCH v4 56/80] drm/panel: panel-dsi-cm: cleanup tear enable Tomi Valkeinen
2020-11-24 17:25 ` Sam Ravnborg
2020-11-30 9:51 ` Laurent Pinchart
2020-12-14 12:54 ` Sebastian Reichel
2020-11-24 12:45 ` [PATCH v4 57/80] ARM: dts: omap5: add address-cells & size-cells to dsi Tomi Valkeinen
2020-11-30 9:53 ` Laurent Pinchart
2020-12-01 10:39 ` Tomi Valkeinen
2020-12-01 10:47 ` Tony Lindgren
2020-12-14 12:55 ` Sebastian Reichel
2020-11-24 12:45 ` [PATCH v4 58/80] drm/omap: pll: fix iteration loop check Tomi Valkeinen
2020-11-30 9:54 ` Laurent Pinchart
2020-12-14 12:55 ` Sebastian Reichel
2020-11-24 12:45 ` [PATCH v4 59/80] drm/omap: dsi: set trans_mode according to client mode_flags Tomi Valkeinen
2020-11-30 9:55 ` Laurent Pinchart
2020-12-14 12:56 ` Sebastian Reichel
2020-11-24 12:45 ` [PATCH v4 60/80] drm/panel: panel-dsi-cm: set column & page at setup Tomi Valkeinen
2020-11-24 17:30 ` Sam Ravnborg
2020-11-24 18:19 ` Tomi Valkeinen
2020-11-30 9:56 ` Laurent Pinchart
2020-12-14 12:59 ` Sebastian Reichel
2020-11-24 12:45 ` [PATCH v4 61/80] drm/omap: dsi: send nop instead of page & column Tomi Valkeinen
2020-11-30 9:58 ` Laurent Pinchart
2020-12-01 10:59 ` Tomi Valkeinen
2020-11-24 12:45 ` [PATCH v4 62/80] drm/omap: dsi: simplify VC handling Tomi Valkeinen
2020-11-30 10:00 ` Laurent Pinchart
2020-12-01 11:11 ` Tomi Valkeinen
2020-11-24 12:45 ` [PATCH v4 63/80] drm/omap: dsi: drop useless channel checks Tomi Valkeinen
2020-12-01 0:14 ` Laurent Pinchart
2020-11-24 12:45 ` [PATCH v4 64/80] drm/omap: dsi: cleanup channel usages Tomi Valkeinen
2020-12-01 0:17 ` Laurent Pinchart
2020-12-01 11:23 ` Tomi Valkeinen
2020-11-24 12:45 ` [PATCH v4 65/80] drm/omap: dsi: skip dsi_vc_enable_hs when already in correct mode Tomi Valkeinen
2020-12-01 0:18 ` Laurent Pinchart
2020-11-24 12:45 ` [PATCH v4 66/80] drm/omap: dsi: set LP/HS before update Tomi Valkeinen
2020-12-01 0:20 ` Laurent Pinchart
2020-11-24 12:45 ` [PATCH v4 67/80] drm/omap: dsi: use separate VCs for cmd and video Tomi Valkeinen
2020-12-01 0:21 ` Laurent Pinchart
2020-11-24 12:45 ` [PATCH v4 68/80] drm/panel: panel-dsi-cm: remove extra 'if' Tomi Valkeinen
2020-11-24 17:34 ` Sam Ravnborg
2020-12-01 0:22 ` Laurent Pinchart
2020-12-01 11:26 ` Tomi Valkeinen
2020-12-02 20:08 ` kernel test robot
2020-11-24 12:45 ` [PATCH v4 69/80] drm/panel: panel-dsi-cm: add panel database to driver Tomi Valkeinen
2020-11-24 17:35 ` Sam Ravnborg
2020-12-01 0:23 ` Laurent Pinchart
2020-12-01 11:29 ` Tomi Valkeinen
2020-11-24 12:45 ` [PATCH v4 70/80] drm/panel: panel-dsi-cm: drop unneeded includes Tomi Valkeinen
2020-11-24 17:36 ` Sam Ravnborg
2020-12-01 0:27 ` Laurent Pinchart
2020-12-01 11:33 ` Tomi Valkeinen
2020-11-24 12:45 ` [PATCH v4 71/80] drm/omap: dsi: move structs & defines to dsi.h Tomi Valkeinen
2020-12-01 0:31 ` Laurent Pinchart
2020-12-01 11:37 ` Tomi Valkeinen
2020-11-24 12:45 ` [PATCH v4 72/80] drm/omap: dsi: move enable/disable to bridge enable/disable Tomi Valkeinen
2020-12-01 0:32 ` Laurent Pinchart
2020-11-24 12:45 ` [PATCH v4 73/80] drm/omap: dsi: display_enable cleanup Tomi Valkeinen
2020-12-01 0:32 ` Laurent Pinchart
2020-11-24 12:45 ` [PATCH v4 74/80] drm/omap: dsi: display_disable cleanup Tomi Valkeinen
2020-12-01 0:33 ` Laurent Pinchart
2020-11-24 12:45 ` [PATCH v4 75/80] drm/omap: dsi: rename dsi_display_* functions Tomi Valkeinen
2020-12-01 0:34 ` Laurent Pinchart
2020-11-24 12:45 ` [PATCH v4 76/80] drm/omap: dsi: cleanup initial vc setup Tomi Valkeinen
2020-12-01 0:36 ` Laurent Pinchart
2020-12-01 11:51 ` Tomi Valkeinen
2020-11-24 12:45 ` [PATCH v4 77/80] drm/omap: dsi: split video mode enable/disable into separate func Tomi Valkeinen
2020-12-01 0:37 ` Laurent Pinchart
2020-11-24 12:45 ` [PATCH v4 78/80] drm/omap: dsi: fix and cleanup ddr_clk_always_on Tomi Valkeinen
2020-12-01 0:38 ` Laurent Pinchart
2020-11-24 12:45 ` [PATCH v4 79/80] drm/omap: dsi: remove ulps support Tomi Valkeinen
2020-12-01 0:39 ` Laurent Pinchart
2020-11-24 12:45 ` [PATCH v4 80/80] drm/omap: dsi: fix DCS_CMD_ENABLE Tomi Valkeinen
2020-12-01 0:40 ` Laurent Pinchart
2020-11-30 9:39 ` [PATCH v4 81/80] drm/omap: dsi: allow DSI commands to be sent early Tomi Valkeinen
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=20201214125315.234l3hln3d4txfsl@earth.universe \
--to=sre@kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=hns@goldelico.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-omap@vger.kernel.org \
--cc=nikhil.nd@ti.com \
--cc=nsekhar@ti.com \
--cc=tomi.valkeinen@ti.com \
--cc=tony@atomide.com \
/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
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).