From: Laurent Pinchart <laurent.pinchart@ideasonboard.com> To: Sam Ravnborg <sam@ravnborg.org> Cc: Neil Armstrong <narmstrong@baylibre.com>, David Airlie <airlied@linux.ie>, dri-devel@lists.freedesktop.org, Thierry Reding <thierry.reding@gmail.com>, Marek Vasut <marex@denx.de>, Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>, Vincent Abriou <vincent.abriou@st.com>, Krzysztof Kozlowski <krzk@kernel.org>, Jonathan Hunter <jonathanh@nvidia.com>, Maxime Ripard <maxime.ripard@bootlin.com>, Kukjin Kim <kgene@kernel.org>, linux-arm-kernel@lists.infradead.org, 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>, Allison Randal <allison@lohutok.net> Subject: Re: [PATCH v1 13/16] drm/panel: drop return code from drm_panel_detach() Date: Mon, 5 Aug 2019 13:56:47 +0300 [thread overview] Message-ID: <20190805105647.GH29747@pendragon.ideasonboard.com> (raw) In-Reply-To: <20190804201637.1240-14-sam@ravnborg.org> Hi Sam, Thank you for the patch. On Sun, Aug 04, 2019 at 10:16:34PM +0200, Sam Ravnborg wrote: > There are no errors that can be reported by this function, > so drop the return code. > Fix the only bridge driver that checked the return result. > > Signed-off-by: Sam Ravnborg <sam@ravnborg.org> > Cc: Thierry Reding <thierry.reding@gmail.com> > Cc: Sam Ravnborg <sam@ravnborg.org> > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> > Cc: Maxime Ripard <maxime.ripard@bootlin.com> > Cc: Sean Paul <sean@poorly.run> > Cc: David Airlie <airlied@linux.ie> > Cc: Daniel Vetter <daniel@ffwll.ch> > Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > Cc: Andrzej Hajda <a.hajda@samsung.com> > Cc: Gwan-gyeong Mun <gwan-gyeong.mun@intel.com> > Cc: Thomas Gleixner <tglx@linutronix.de> > Cc: Linus Walleij <linus.walleij@linaro.org> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > --- > drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 3 +-- > drivers/gpu/drm/drm_panel.c | 6 +----- > include/drm/drm_panel.h | 2 +- > 3 files changed, 3 insertions(+), 8 deletions(-) > > diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c > index f2f7f69d6cc3..22885dceaa17 100644 > --- a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c > +++ b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c > @@ -1780,8 +1780,7 @@ void analogix_dp_unbind(struct analogix_dp_device *dp) > if (dp->plat_data->panel) { > if (drm_panel_unprepare(dp->plat_data->panel)) > DRM_ERROR("failed to turnoff the panel\n"); > - if (drm_panel_detach(dp->plat_data->panel)) > - DRM_ERROR("failed to detach the panel\n"); > + drm_panel_detach(dp->plat_data->panel); > } > > drm_dp_aux_unregister(&dp->aux); > diff --git a/drivers/gpu/drm/drm_panel.c b/drivers/gpu/drm/drm_panel.c > index 9946b8d9bacc..da19d5b4a2f4 100644 > --- a/drivers/gpu/drm/drm_panel.c > +++ b/drivers/gpu/drm/drm_panel.c > @@ -219,15 +219,11 @@ EXPORT_SYMBOL(drm_panel_attach); > * > * This function should not be called by the panel device itself. It > * is only for the drm device that called drm_panel_attach(). > - * > - * Return: 0 on success or a negative error code on failure. > */ > -int drm_panel_detach(struct drm_panel *panel) > +void drm_panel_detach(struct drm_panel *panel) > { > panel->connector = NULL; > panel->drm = NULL; > - > - return 0; > } > EXPORT_SYMBOL(drm_panel_detach); > > diff --git a/include/drm/drm_panel.h b/include/drm/drm_panel.h > index 5e62deea49ba..624bd15ecfab 100644 > --- a/include/drm/drm_panel.h > +++ b/include/drm/drm_panel.h > @@ -153,7 +153,7 @@ int drm_panel_add(struct drm_panel *panel); > void drm_panel_remove(struct drm_panel *panel); > > int drm_panel_attach(struct drm_panel *panel, struct drm_connector *connector); > -int drm_panel_detach(struct drm_panel *panel); > +void drm_panel_detach(struct drm_panel *panel); > > int drm_panel_prepare(struct drm_panel *panel); > int drm_panel_unprepare(struct drm_panel *panel); -- Regards, Laurent Pinchart _______________________________________________ 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: Laurent Pinchart <laurent.pinchart@ideasonboard.com> To: Sam Ravnborg <sam@ravnborg.org> Cc: Neil Armstrong <narmstrong@baylibre.com>, David Airlie <airlied@linux.ie>, Linus Walleij <linus.walleij@linaro.org>, dri-devel@lists.freedesktop.org, Andrzej Hajda <a.hajda@samsung.com>, Thierry Reding <thierry.reding@gmail.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>, Vincent Abriou <vincent.abriou@st.com>, Krzysztof Kozlowski <krzk@kernel.org>, Jonathan Hunter <jonathanh@nvidia.com>, Maxime Ripard <maxime.ripard@bootlin.com>, Kukjin Kim <kgene@kernel.org>, linux-arm-kernel@lists.infradead.org, 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, Stefan Agner <stefan@agner.ch>, linux-tegra@vger.kernel.org, Thomas Gleixner <tglx@linutronix.de>, Sean Paul <sean@poorly.run>, Allison Randal <allison@lohutok.net>, Jernej Skrabec <jernej.skrabec@siol.net>, Shawn Guo <shawnguo@kernel.org>, Seung-Woo Kim <sw0312.kim@samsung.com>, Kyungmin Park <kyungmin.park@samsung.com>, Daniel Vetter <daniel@ffwll.ch>, Enrico Weigelt <info@metux.net> Subject: Re: [PATCH v1 13/16] drm/panel: drop return code from drm_panel_detach() Date: Mon, 5 Aug 2019 13:56:47 +0300 [thread overview] Message-ID: <20190805105647.GH29747@pendragon.ideasonboard.com> (raw) In-Reply-To: <20190804201637.1240-14-sam@ravnborg.org> Hi Sam, Thank you for the patch. On Sun, Aug 04, 2019 at 10:16:34PM +0200, Sam Ravnborg wrote: > There are no errors that can be reported by this function, > so drop the return code. > Fix the only bridge driver that checked the return result. > > Signed-off-by: Sam Ravnborg <sam@ravnborg.org> > Cc: Thierry Reding <thierry.reding@gmail.com> > Cc: Sam Ravnborg <sam@ravnborg.org> > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> > Cc: Maxime Ripard <maxime.ripard@bootlin.com> > Cc: Sean Paul <sean@poorly.run> > Cc: David Airlie <airlied@linux.ie> > Cc: Daniel Vetter <daniel@ffwll.ch> > Cc: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > Cc: Andrzej Hajda <a.hajda@samsung.com> > Cc: Gwan-gyeong Mun <gwan-gyeong.mun@intel.com> > Cc: Thomas Gleixner <tglx@linutronix.de> > Cc: Linus Walleij <linus.walleij@linaro.org> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > --- > drivers/gpu/drm/bridge/analogix/analogix_dp_core.c | 3 +-- > drivers/gpu/drm/drm_panel.c | 6 +----- > include/drm/drm_panel.h | 2 +- > 3 files changed, 3 insertions(+), 8 deletions(-) > > diff --git a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c > index f2f7f69d6cc3..22885dceaa17 100644 > --- a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c > +++ b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c > @@ -1780,8 +1780,7 @@ void analogix_dp_unbind(struct analogix_dp_device *dp) > if (dp->plat_data->panel) { > if (drm_panel_unprepare(dp->plat_data->panel)) > DRM_ERROR("failed to turnoff the panel\n"); > - if (drm_panel_detach(dp->plat_data->panel)) > - DRM_ERROR("failed to detach the panel\n"); > + drm_panel_detach(dp->plat_data->panel); > } > > drm_dp_aux_unregister(&dp->aux); > diff --git a/drivers/gpu/drm/drm_panel.c b/drivers/gpu/drm/drm_panel.c > index 9946b8d9bacc..da19d5b4a2f4 100644 > --- a/drivers/gpu/drm/drm_panel.c > +++ b/drivers/gpu/drm/drm_panel.c > @@ -219,15 +219,11 @@ EXPORT_SYMBOL(drm_panel_attach); > * > * This function should not be called by the panel device itself. It > * is only for the drm device that called drm_panel_attach(). > - * > - * Return: 0 on success or a negative error code on failure. > */ > -int drm_panel_detach(struct drm_panel *panel) > +void drm_panel_detach(struct drm_panel *panel) > { > panel->connector = NULL; > panel->drm = NULL; > - > - return 0; > } > EXPORT_SYMBOL(drm_panel_detach); > > diff --git a/include/drm/drm_panel.h b/include/drm/drm_panel.h > index 5e62deea49ba..624bd15ecfab 100644 > --- a/include/drm/drm_panel.h > +++ b/include/drm/drm_panel.h > @@ -153,7 +153,7 @@ int drm_panel_add(struct drm_panel *panel); > void drm_panel_remove(struct drm_panel *panel); > > int drm_panel_attach(struct drm_panel *panel, struct drm_connector *connector); > -int drm_panel_detach(struct drm_panel *panel); > +void drm_panel_detach(struct drm_panel *panel); > > int drm_panel_prepare(struct drm_panel *panel); > int drm_panel_unprepare(struct drm_panel *panel); -- Regards, Laurent Pinchart _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2019-08-05 10:56 UTC|newest] Thread overview: 94+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-08-04 20:16 [PATCH v1 0/16] drm: panel related updates Sam Ravnborg 2019-08-04 20:16 ` 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 [this message] 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=20190805105647.GH29747@pendragon.ideasonboard.com \ --to=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=sam@ravnborg.org \ --cc=sean@poorly.run \ --cc=tglx@linutronix.de \ --cc=thierry.reding@gmail.com \ --cc=vincent.abriou@st.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.