From: Sam Ravnborg <sam@ravnborg.org> To: dri-devel@lists.freedesktop.org, Thierry Reding <thierry.reding@gmail.com> Cc: Neil Armstrong <narmstrong@baylibre.com>, David Airlie <airlied@linux.ie>, Laurent Pinchart <Laurent.pinchart@ideasonboard.com>, Marek Vasut <marex@denx.de>, Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>, Sam Ravnborg <sam@ravnborg.org>, Krzysztof Kozlowski <krzk@kernel.org>, Jonathan Hunter <jonathanh@nvidia.com>, Maxime Ripard <maxime.ripard@bootlin.com>, Kukjin Kim <kgene@kernel.org>, Allison Randal <allison@lohutok.net>, NXP Linux Team <linux-imx@nxp.com>, Pengutronix Kernel Team <kernel@pengutronix.de>, Jonas Karlman <jonas@kwiboo.se>, Alison Wang <alison.wang@nxp.com>, Gwan-gyeong Mun <gwan-gyeong.mun@intel.com>, Alexios Zavras <alexios.zavras@intel.com>, linux-samsung-soc@vger.kernel.org, linux-tegra@vger.kernel.org, Thomas Gleixner <tglx@linutronix.de>, Sean Paul <sean@poorly.run>, linux-arm-kernel@lists.infradead.org, Jernej Skrabec <jernej.skrab> Subject: [PATCH v1 0/16] drm: panel related updates Date: Sun, 4 Aug 2019 22:16:21 +0200 [thread overview] Message-ID: <20190804201637.1240-1-sam@ravnborg.org> (raw) The first 9 patches replaces direct use of the drm_panel function pointers with their drm_panel_* counterparts. The function pointers are only supposed to be used by the drm_panel infrastructure and direct use are discouraged. ili9322 is updated to handle bus_flags in get_modes like everyone else. This is in preparation for a later patch series where controller becomes an arugument to get_modes() and not like today where drm_panel is attached to a controller. The remaining patches move functionality to the drm_panel core that today are repeated in many drivers. As preparation for this the inline functions are moved to drm_panel.c and kernel-doc is made inline. panel-simple is updated to benefit from the additional infrastructure and is an example for the simplifications that can be done. The patchset has been tested on my embedded target, and build tested. Feedback welcome! The "fix opencoded" patches are all independent and can be applied out of order. They were kept here to keep panel related patches in one series. Sam Cc: Alexios Zavras <alexios.zavras@intel.com> Cc: Alison Wang <alison.wang@nxp.com> Cc: Allison Randal <allison@lohutok.net> Cc: Andrzej Hajda <a.hajda@samsung.com> Cc: Benjamin Gaignard <benjamin.gaignard@linaro.org> Cc: Daniel Vetter <daniel@ffwll.ch> Cc: David Airlie <airlied@linux.ie> Cc: Enrico Weigelt <info@metux.net> Cc: Fabio Estevam <festevam@gmail.com> Cc: Gwan-gyeong Mun <gwan-gyeong.mun@intel.com> Cc: Inki Dae <inki.dae@samsung.com> Cc: Jernej Skrabec <jernej.skrabec@siol.net> Cc: Jonas Karlman <jonas@kwiboo.se> Cc: Jonathan Hunter <jonathanh@nvidia.com> Cc: Joonyoung Shim <jy0922.shim@samsung.com> Cc: Krzysztof Kozlowski <krzk@kernel.org> Cc: Kukjin Kim <kgene@kernel.org> Cc: Kyungmin Park <kyungmin.park@samsung.com> Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com> Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> Cc: Linus Walleij <linus.walleij@linaro.org> Cc: linux-arm-kernel@lists.infradead.org Cc: linux-samsung-soc@vger.kernel.org Cc: linux-tegra@vger.kernel.org Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Cc: Marek Vasut <marex@denx.de> Cc: Maxime Ripard <maxime.ripard@bootlin.com> Cc: Neil Armstrong <narmstrong@baylibre.com> Cc: NXP Linux Team <linux-imx@nxp.com> Cc: Pengutronix Kernel Team <kernel@pengutronix.de> Cc: Philipp Zabel <p.zabel@pengutronix.de> Cc: Sam Ravnborg <sam@ravnborg.org> Cc: Sascha Hauer <s.hauer@pengutronix.de> Cc: Sean Paul <sean@poorly.run> Cc: Seung-Woo Kim <sw0312.kim@samsung.com> Cc: Shawn Guo <shawnguo@kernel.org> Cc: Stefan Agner <stefan@agner.ch> Cc: Thierry Reding <thierry.reding@gmail.com> Cc: Thomas Gleixner <tglx@linutronix.de> Cc: Vincent Abriou <vincent.abriou@st.com> Sam Ravnborg (16): drm/bridge: tc358767: fix opencoded use of drm_panel_* drm/exynos: fix opencoded use of drm_panel_* drm/exynos: fix opencoded use of drm_panel_* drm/imx: fix opencoded use of drm_panel_* drm/fsl-dcu: fix opencoded use of drm_panel_* drm/msm: fix opencoded use of drm_panel_* drm/mxsfb: fix opencoded use of drm_panel_* drm/sti: fix opencoded use of drm_panel_* drm/tegra: fix opencoded use of drm_panel_* drm/panel: ili9322: move bus_flags to get_modes() drm/panel: move drm_panel functions to .c file drm/panel: use inline comments in drm_panel.h drm/panel: drop return code from drm_panel_detach() drm/panel: call prepare/enable only once drm/panel: add backlight support drm/panel: simple: use drm_panel infrastructure drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 3 +- drivers/gpu/drm/bridge/tc358767.c | 10 +- drivers/gpu/drm/drm_panel.c | 185 ++++++++++++++++- drivers/gpu/drm/exynos/exynos_drm_dpi.c | 4 +- drivers/gpu/drm/exynos/exynos_drm_dsi.c | 2 +- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_rgb.c | 10 +- drivers/gpu/drm/imx/imx-ldb.c | 11 +- drivers/gpu/drm/imx/parallel-display.c | 11 +- .../gpu/drm/msm/disp/mdp4/mdp4_lvds_connector.c | 2 +- drivers/gpu/drm/mxsfb/mxsfb_out.c | 2 +- drivers/gpu/drm/panel/panel-ilitek-ili9322.c | 34 ++- drivers/gpu/drm/panel/panel-simple.c | 73 +------ drivers/gpu/drm/sti/sti_dvo.c | 8 +- drivers/gpu/drm/tegra/output.c | 2 +- include/drm/drm_panel.h | 227 +++++++++++---------- 15 files changed, 349 insertions(+), 235 deletions(-) _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
WARNING: multiple messages have this Message-ID (diff)
From: Sam Ravnborg <sam@ravnborg.org> To: dri-devel@lists.freedesktop.org, Thierry Reding <thierry.reding@gmail.com> Cc: Neil Armstrong <narmstrong@baylibre.com>, David Airlie <airlied@linux.ie>, Linus Walleij <linus.walleij@linaro.org>, Stefan Agner <stefan@agner.ch>, Andrzej Hajda <a.hajda@samsung.com>, Laurent Pinchart <Laurent.pinchart@ideasonboard.com>, Benjamin Gaignard <benjamin.gaignard@linaro.org>, Fabio Estevam <festevam@gmail.com>, Marek Vasut <marex@denx.de>, Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>, Joonyoung Shim <jy0922.shim@samsung.com>, Sam Ravnborg <sam@ravnborg.org>, Krzysztof Kozlowski <krzk@kernel.org>, Jonathan Hunter <jonathanh@nvidia.com>, Maxime Ripard <maxime.ripard@bootlin.com>, Kukjin Kim <kgene@kernel.org>, Allison Randal <allison@lohutok.net>, Philipp Zabel <p.zabel@pengutronix.de>, NXP Linux Team <linux-imx@nxp.com>, Pengutronix Kernel Team <kernel@pengutronix.de>, Jonas Karlman <jonas@kwiboo.se>, Sascha Hauer <s.hauer@pengutronix.de>, Alison Wang <alison.wang@nxp.com>, Maarten Lankhorst <maarten.lankhorst@linux.intel.com>, Gwan-gyeong Mun <gwan-gyeong.mun@intel.com>, Inki Dae <inki.dae@samsung.com>, Alexios Zavras <alexios.zavras@intel.com>, linux-samsung-soc@vger.kernel.org, linux-tegra@vger.kernel.org, Thomas Gleixner <tglx@linutronix.de>, Sean Paul <sean@poorly.run>, linux-arm-kernel@lists.infradead.org, Jernej Skrabec <jernej.skrabec@siol.net>, Shawn Guo <shawnguo@kernel.org>, Seung-Woo Kim <sw0312.kim@samsung.com>, Vincent Abriou <vincent.abriou@st.com>, Kyungmin Park <kyungmin.park@samsung.com>, Daniel Vetter <daniel@ffwll.ch>, Enrico Weigelt <info@metux.net> Subject: [PATCH v1 0/16] drm: panel related updates Date: Sun, 4 Aug 2019 22:16:21 +0200 [thread overview] Message-ID: <20190804201637.1240-1-sam@ravnborg.org> (raw) The first 9 patches replaces direct use of the drm_panel function pointers with their drm_panel_* counterparts. The function pointers are only supposed to be used by the drm_panel infrastructure and direct use are discouraged. ili9322 is updated to handle bus_flags in get_modes like everyone else. This is in preparation for a later patch series where controller becomes an arugument to get_modes() and not like today where drm_panel is attached to a controller. The remaining patches move functionality to the drm_panel core that today are repeated in many drivers. As preparation for this the inline functions are moved to drm_panel.c and kernel-doc is made inline. panel-simple is updated to benefit from the additional infrastructure and is an example for the simplifications that can be done. The patchset has been tested on my embedded target, and build tested. Feedback welcome! The "fix opencoded" patches are all independent and can be applied out of order. They were kept here to keep panel related patches in one series. Sam Cc: Alexios Zavras <alexios.zavras@intel.com> Cc: Alison Wang <alison.wang@nxp.com> Cc: Allison Randal <allison@lohutok.net> Cc: Andrzej Hajda <a.hajda@samsung.com> Cc: Benjamin Gaignard <benjamin.gaignard@linaro.org> Cc: Daniel Vetter <daniel@ffwll.ch> Cc: David Airlie <airlied@linux.ie> Cc: Enrico Weigelt <info@metux.net> Cc: Fabio Estevam <festevam@gmail.com> Cc: Gwan-gyeong Mun <gwan-gyeong.mun@intel.com> Cc: Inki Dae <inki.dae@samsung.com> Cc: Jernej Skrabec <jernej.skrabec@siol.net> Cc: Jonas Karlman <jonas@kwiboo.se> Cc: Jonathan Hunter <jonathanh@nvidia.com> Cc: Joonyoung Shim <jy0922.shim@samsung.com> Cc: Krzysztof Kozlowski <krzk@kernel.org> Cc: Kukjin Kim <kgene@kernel.org> Cc: Kyungmin Park <kyungmin.park@samsung.com> Cc: Laurent Pinchart <Laurent.pinchart@ideasonboard.com> Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> Cc: Linus Walleij <linus.walleij@linaro.org> Cc: linux-arm-kernel@lists.infradead.org Cc: linux-samsung-soc@vger.kernel.org Cc: linux-tegra@vger.kernel.org Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Cc: Marek Vasut <marex@denx.de> Cc: Maxime Ripard <maxime.ripard@bootlin.com> Cc: Neil Armstrong <narmstrong@baylibre.com> Cc: NXP Linux Team <linux-imx@nxp.com> Cc: Pengutronix Kernel Team <kernel@pengutronix.de> Cc: Philipp Zabel <p.zabel@pengutronix.de> Cc: Sam Ravnborg <sam@ravnborg.org> Cc: Sascha Hauer <s.hauer@pengutronix.de> Cc: Sean Paul <sean@poorly.run> Cc: Seung-Woo Kim <sw0312.kim@samsung.com> Cc: Shawn Guo <shawnguo@kernel.org> Cc: Stefan Agner <stefan@agner.ch> Cc: Thierry Reding <thierry.reding@gmail.com> Cc: Thomas Gleixner <tglx@linutronix.de> Cc: Vincent Abriou <vincent.abriou@st.com> Sam Ravnborg (16): drm/bridge: tc358767: fix opencoded use of drm_panel_* drm/exynos: fix opencoded use of drm_panel_* drm/exynos: fix opencoded use of drm_panel_* drm/imx: fix opencoded use of drm_panel_* drm/fsl-dcu: fix opencoded use of drm_panel_* drm/msm: fix opencoded use of drm_panel_* drm/mxsfb: fix opencoded use of drm_panel_* drm/sti: fix opencoded use of drm_panel_* drm/tegra: fix opencoded use of drm_panel_* drm/panel: ili9322: move bus_flags to get_modes() drm/panel: move drm_panel functions to .c file drm/panel: use inline comments in drm_panel.h drm/panel: drop return code from drm_panel_detach() drm/panel: call prepare/enable only once drm/panel: add backlight support drm/panel: simple: use drm_panel infrastructure drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 3 +- drivers/gpu/drm/bridge/tc358767.c | 10 +- drivers/gpu/drm/drm_panel.c | 185 ++++++++++++++++- drivers/gpu/drm/exynos/exynos_drm_dpi.c | 4 +- drivers/gpu/drm/exynos/exynos_drm_dsi.c | 2 +- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_rgb.c | 10 +- drivers/gpu/drm/imx/imx-ldb.c | 11 +- drivers/gpu/drm/imx/parallel-display.c | 11 +- .../gpu/drm/msm/disp/mdp4/mdp4_lvds_connector.c | 2 +- drivers/gpu/drm/mxsfb/mxsfb_out.c | 2 +- drivers/gpu/drm/panel/panel-ilitek-ili9322.c | 34 ++- drivers/gpu/drm/panel/panel-simple.c | 73 +------ drivers/gpu/drm/sti/sti_dvo.c | 8 +- drivers/gpu/drm/tegra/output.c | 2 +- include/drm/drm_panel.h | 227 +++++++++++---------- 15 files changed, 349 insertions(+), 235 deletions(-) _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next reply other threads:[~2019-08-04 20:16 UTC|newest] Thread overview: 94+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-08-04 20:16 Sam Ravnborg [this message] 2019-08-04 20:16 ` [PATCH v1 0/16] drm: panel related updates Sam Ravnborg 2019-08-04 20:16 ` [PATCH v1 01/16] drm/bridge: tc358767: fix opencoded use of drm_panel_* Sam Ravnborg 2019-08-04 20:16 ` Sam Ravnborg 2019-08-05 9:35 ` Philipp Zabel 2019-08-05 9:35 ` Philipp Zabel 2019-08-05 11:53 ` Sam Ravnborg 2019-08-05 11:53 ` Sam Ravnborg 2019-08-04 20:16 ` [PATCH v1 02/16] drm/exynos: " Sam Ravnborg 2019-08-04 20:16 ` Sam Ravnborg 2019-12-01 11:32 ` Sam Ravnborg 2019-12-01 11:32 ` Sam Ravnborg 2019-12-01 11:32 ` Sam Ravnborg 2019-12-01 11:32 ` Sam Ravnborg 2019-08-04 20:16 ` [PATCH v1 03/16] " Sam Ravnborg 2019-08-04 20:16 ` Sam Ravnborg 2019-12-01 11:32 ` Sam Ravnborg 2019-12-01 11:32 ` Sam Ravnborg 2019-12-01 11:32 ` Sam Ravnborg 2019-12-01 11:32 ` Sam Ravnborg 2019-08-04 20:16 ` [PATCH v1 04/16] drm/imx: " Sam Ravnborg 2019-08-04 20:16 ` Sam Ravnborg 2019-08-05 9:34 ` Philipp Zabel 2019-08-05 9:34 ` Philipp Zabel 2019-08-04 20:16 ` [PATCH v1 05/16] drm/fsl-dcu: " Sam Ravnborg 2019-08-04 20:16 ` Sam Ravnborg 2019-08-05 9:16 ` Stefan Agner 2019-08-05 9:16 ` Stefan Agner 2019-08-05 11:54 ` Sam Ravnborg 2019-08-05 11:54 ` Sam Ravnborg 2019-08-04 20:16 ` [PATCH v1 06/16] drm/msm: " Sam Ravnborg 2019-08-04 20:16 ` Sam Ravnborg 2019-12-01 11:33 ` Sam Ravnborg 2019-12-01 11:33 ` Sam Ravnborg 2019-12-01 11:33 ` Sam Ravnborg 2019-12-01 11:33 ` Sam Ravnborg 2019-08-04 20:16 ` [PATCH v1 07/16] drm/mxsfb: " Sam Ravnborg 2019-08-04 20:16 ` Sam Ravnborg 2019-08-05 9:20 ` Stefan Agner 2019-08-05 9:20 ` Stefan Agner 2019-08-04 20:16 ` [PATCH v1 08/16] drm/sti: " Sam Ravnborg 2019-08-04 20:16 ` Sam Ravnborg 2019-08-07 11:55 ` Benjamin Gaignard 2019-08-07 11:55 ` Benjamin Gaignard 2019-08-04 20:16 ` [PATCH v1 09/16] drm/tegra: " Sam Ravnborg 2019-08-04 20:16 ` Sam Ravnborg 2019-12-01 11:33 ` Sam Ravnborg 2019-12-01 11:33 ` Sam Ravnborg 2019-12-01 11:33 ` Sam Ravnborg 2019-12-01 11:33 ` Sam Ravnborg 2019-08-04 20:16 ` [PATCH v1 10/16] drm/panel: ili9322: move bus_flags to get_modes() Sam Ravnborg 2019-08-04 20:16 ` Sam Ravnborg 2019-08-06 12:56 ` Linus Walleij 2019-08-06 12:56 ` Linus Walleij 2019-08-04 20:16 ` [PATCH v1 11/16] drm/panel: move drm_panel functions to .c file Sam Ravnborg 2019-08-04 20:16 ` Sam Ravnborg 2019-08-05 10:45 ` Laurent Pinchart 2019-08-05 10:45 ` Laurent Pinchart 2019-08-04 20:16 ` [PATCH v1 12/16] drm/panel: use inline comments in drm_panel.h Sam Ravnborg 2019-08-04 20:16 ` Sam Ravnborg 2019-08-05 10:54 ` Laurent Pinchart 2019-08-05 10:54 ` Laurent Pinchart 2019-08-04 20:16 ` [PATCH v1 13/16] drm/panel: drop return code from drm_panel_detach() Sam Ravnborg 2019-08-04 20:16 ` Sam Ravnborg 2019-08-05 10:56 ` Laurent Pinchart 2019-08-05 10:56 ` Laurent Pinchart 2019-08-04 20:16 ` [PATCH v1 14/16] drm/panel: call prepare/enable only once Sam Ravnborg 2019-08-04 20:16 ` Sam Ravnborg 2019-08-05 10:59 ` Laurent Pinchart 2019-08-05 10:59 ` Laurent Pinchart 2019-08-05 13:15 ` Emil Velikov 2019-08-05 13:15 ` Emil Velikov 2019-08-05 16:51 ` Sam Ravnborg 2019-08-05 16:51 ` Sam Ravnborg 2019-12-02 15:22 ` Laurent Pinchart 2019-12-02 15:22 ` Laurent Pinchart 2019-12-02 15:22 ` Laurent Pinchart 2019-12-02 15:22 ` Laurent Pinchart 2019-08-05 17:01 ` Sean Paul 2019-08-05 17:01 ` Sean Paul 2019-12-02 15:15 ` Laurent Pinchart 2019-12-02 15:15 ` Laurent Pinchart 2019-12-02 15:15 ` Laurent Pinchart 2019-12-02 15:15 ` Laurent Pinchart 2019-08-04 20:16 ` [PATCH v1 15/16] drm/panel: add backlight support Sam Ravnborg 2019-08-04 20:16 ` Sam Ravnborg 2019-08-05 11:04 ` Laurent Pinchart 2019-08-05 11:04 ` Laurent Pinchart 2019-08-04 20:16 ` [PATCH v1 16/16] drm/panel: simple: use drm_panel infrastructure Sam Ravnborg 2019-08-04 20:16 ` Sam Ravnborg 2019-08-05 11:12 ` Laurent Pinchart 2019-08-05 11:12 ` Laurent Pinchart 2019-08-05 13:18 ` [PATCH v1 0/16] drm: panel related updates Emil Velikov 2019-08-05 13:18 ` Emil Velikov
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=20190804201637.1240-1-sam@ravnborg.org \ --to=sam@ravnborg.org \ --cc=Laurent.pinchart@ideasonboard.com \ --cc=airlied@linux.ie \ --cc=alexios.zavras@intel.com \ --cc=alison.wang@nxp.com \ --cc=allison@lohutok.net \ --cc=dri-devel@lists.freedesktop.org \ --cc=gwan-gyeong.mun@intel.com \ --cc=jonas@kwiboo.se \ --cc=jonathanh@nvidia.com \ --cc=kernel@pengutronix.de \ --cc=kgene@kernel.org \ --cc=krzk@kernel.org \ --cc=laurent.pinchart+renesas@ideasonboard.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-imx@nxp.com \ --cc=linux-samsung-soc@vger.kernel.org \ --cc=linux-tegra@vger.kernel.org \ --cc=marex@denx.de \ --cc=maxime.ripard@bootlin.com \ --cc=narmstrong@baylibre.com \ --cc=sean@poorly.run \ --cc=tglx@linutronix.de \ --cc=thierry.reding@gmail.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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.