Re: [PATCH] drm/msm: add missing include to msm_drv.c

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 30/04/2022 18:00, Abhinav Kumar wrote:
Change seems fine, with a couple of minor things below.

Reviewed-by: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx>

On 4/30/2022 3:48 AM, Dmitry Baryshkov wrote:
Add explicit include of drm_bridge.h to the msm_drm.c to fix the

this should be msm_drv.c

Ack.


following warning:

drivers/gpu/drm/msm/msm_drv.c:236:17: error: implicit declaration of function 'drm_bridge_remove'; did you mean 'drm_bridge_detach'? [-Werror=implicit-function-declaration]

Fixes: d28ea556267c ("drm/msm: properly add and remove internal bridges")

How did the this commit compile without this?

I think this manifests only in some situations depending on the drivers and features selected. The config generated by the robot is the PA/RISC randconfig. See [1]. For example the whole tree compile w/o any issues here. Otherwise I would have noticed it.

[1] https://download.01.org/0day-ci/archive/20220430/202204301804.hJxOQfse-lkp@xxxxxxxxx/config



Reported-by: kernel test robot <lkp@xxxxxxxxx>
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
---
  drivers/gpu/drm/msm/msm_drv.c | 1 +
  1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c
index 4f3dce334553..4a3dda23e3e0 100644
--- a/drivers/gpu/drm/msm/msm_drv.c
+++ b/drivers/gpu/drm/msm/msm_drv.c
@@ -11,6 +11,7 @@
  #include <linux/uaccess.h>
  #include <uapi/linux/sched/types.h>
+#include <drm/drm_bridge.h>
  #include <drm/drm_drv.h>
  #include <drm/drm_file.h>
  #include <drm/drm_ioctl.h>


--
With best wishes
Dmitry



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux