Hi,
this patch doesn't build.
linux/drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c: In function
‘boe_tv101wum_ll2_probe’:
linux/drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c:241:23: error:
unterminated argument list invoking macro "devm_drm_panel_alloc"
241 | MODULE_LICENSE("GPL");
| ^
linux/drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c:169:15: error:
‘devm_drm_panel_alloc’ undeclared (first use in this function); did you
mean ‘devm_drm_panel_add_follower’?
169 | ctx = devm_drm_panel_alloc(dev, struct
boe_tv101wum_ll2, panel,
| ^~~~~~~~~~~~~~~~~~~~
| devm_drm_panel_add_follower
linux/drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c:169:15: note: each
undeclared identifier is reported only once for each function it appears in
linux/drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c:169:35: error:
expected ‘;’ at end of input
169 | ctx = devm_drm_panel_alloc(dev, struct
boe_tv101wum_ll2, panel,
| ^
| ;
......
linux/drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c:169:9: error:
expected declaration or statement at end of input
169 | ctx = devm_drm_panel_alloc(dev, struct
boe_tv101wum_ll2, panel,
| ^~~
linux/drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c:167:13: warning:
unused variable ‘ret’ [-Wunused-variable]
167 | int ret;
| ^~~
linux/drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c:166:34: warning:
variable ‘ctx’ set but not used [-Wunused-but-set-variable]
166 | struct boe_tv101wum_ll2 *ctx;
| ^~~
linux/drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c:165:24: warning:
unused variable ‘dev’ [-Wunused-variable]
165 | struct device *dev = &dsi->dev;
| ^~~
linux/drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c:169:9: error: no
return statement in function returning non-void [-Werror=return-type]
169 | ctx = devm_drm_panel_alloc(dev, struct
boe_tv101wum_ll2, panel,
| ^~~
linux/drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c: At top level:
linux/drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c:163:12: warning:
‘boe_tv101wum_ll2_probe’ defined but not used [-Wunused-function]
163 | static int boe_tv101wum_ll2_probe(struct mipi_dsi_device *dsi)
| ^~~~~~~~~~~~~~~~~~~~~~
linux/drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c:157:37: warning:
‘boe_tv101wum_ll2_panel_funcs’ defined but not used
[-Wunused-const-variable=]
157 | static const struct drm_panel_funcs
boe_tv101wum_ll2_panel_funcs = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
Please fix.
Best regard
Thomas
Am 01.04.25 um 18:03 schrieb Anusha Srivatsa:
Move to using the new API devm_drm_panel_alloc() to allocate the
panel.
Signed-off-by: Anusha Srivatsa <asriv...@redhat.com>
---
drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c
b/drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c
index
50e4a5341bc65727b5ed6ba43a11f5ab9ac9f5b9..04c7890cc51db43bdc6e38cdae8f7f21fd48009f
100644
--- a/drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c
+++ b/drivers/gpu/drm/panel/panel-boe-tv101wum-ll2.c
@@ -166,9 +166,11 @@ static int boe_tv101wum_ll2_probe(struct mipi_dsi_device
*dsi)
struct boe_tv101wum_ll2 *ctx;
int ret;
- ctx = devm_kzalloc(dev, sizeof(*ctx), GFP_KERNEL);
- if (!ctx)
- return -ENOMEM;
+ ctx = devm_drm_panel_alloc(dev, struct boe_tv101wum_ll2, panel,
+ &boe_tv101wum_ll2_panel_funcs,
+ DRM_MODE_CONNECTOR_DSI
+ if (IS_ERR(panel))
+ return PTR_ERR(panel);
ret = devm_regulator_bulk_get_const(&dsi->dev,
ARRAY_SIZE(boe_tv101wum_ll2_supplies),
@@ -190,8 +192,6 @@ static int boe_tv101wum_ll2_probe(struct mipi_dsi_device
*dsi)
dsi->mode_flags = MIPI_DSI_MODE_VIDEO | MIPI_DSI_MODE_VIDEO_BURST |
MIPI_DSI_MODE_VIDEO_HSE;
- drm_panel_init(&ctx->panel, dev, &boe_tv101wum_ll2_panel_funcs,
- DRM_MODE_CONNECTOR_DSI);
ctx->panel.prepare_prev_first = true;
ret = drm_panel_of_backlight(&ctx->panel);
--
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)
