2022-02-28 20:03:27

by José Expósito

[permalink] [raw]
Subject: [PATCH] drm/bridge: parade-ps8622: switch to devm_drm_of_get_bridge

The function "drm_of_find_panel_or_bridge" has been deprecated in
favor of "devm_drm_of_get_bridge".

Switch to the new function and reduce boilerplate.

Signed-off-by: José Expósito <[email protected]>
---
drivers/gpu/drm/bridge/parade-ps8622.c | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/bridge/parade-ps8622.c b/drivers/gpu/drm/bridge/parade-ps8622.c
index 614b19f0f1b7..37b308850b4e 100644
--- a/drivers/gpu/drm/bridge/parade-ps8622.c
+++ b/drivers/gpu/drm/bridge/parade-ps8622.c
@@ -452,18 +452,13 @@ static int ps8622_probe(struct i2c_client *client,
struct device *dev = &client->dev;
struct ps8622_bridge *ps8622;
struct drm_bridge *panel_bridge;
- struct drm_panel *panel;
int ret;

ps8622 = devm_kzalloc(dev, sizeof(*ps8622), GFP_KERNEL);
if (!ps8622)
return -ENOMEM;

- ret = drm_of_find_panel_or_bridge(dev->of_node, 0, 0, &panel, NULL);
- if (ret)
- return ret;
-
- panel_bridge = devm_drm_panel_bridge_add(dev, panel);
+ panel_bridge = devm_drm_of_get_bridge(dev, dev->of_node, 0, 0);
if (IS_ERR(panel_bridge))
return PTR_ERR(panel_bridge);

--
2.25.1


2022-03-08 16:43:23

by Maxime Ripard

[permalink] [raw]
Subject: Re: (subset) [PATCH] drm/bridge: parade-ps8622: switch to devm_drm_of_get_bridge

On Mon, 28 Feb 2022 19:29:04 +0100, José Expósito wrote:
> The function "drm_of_find_panel_or_bridge" has been deprecated in
> favor of "devm_drm_of_get_bridge".
>
> Switch to the new function and reduce boilerplate.
>
>

Applied to drm/drm-misc (drm-misc-next).

Thanks!
Maxime

2022-03-08 23:05:05

by Jagan Teki

[permalink] [raw]
Subject: Re: (subset) [PATCH] drm/bridge: parade-ps8622: switch to devm_drm_of_get_bridge

Hi Maxime,

On Tue, Mar 8, 2022 at 4:51 PM Maxime Ripard <[email protected]> wrote:
>
> On Mon, 28 Feb 2022 19:29:04 +0100, José Expósito wrote:
> > The function "drm_of_find_panel_or_bridge" has been deprecated in
> > favor of "devm_drm_of_get_bridge".
> >
> > Switch to the new function and reduce boilerplate.
> >
> >
>
> Applied to drm/drm-misc (drm-misc-next).

Not sure whether it was intentionally or accidentally applied? the
same patch has sent before this date and has sent the v3 a few hours
ago.

https://patchwork.kernel.org/project/dri-devel/patch/[email protected]/

Thanks,
Jagan.