dri-devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v1 1/1] drm/komeda: fix build with drm_modeset_helper.h update
@ 2019-02-08 22:13 Sam Ravnborg
  2019-02-11  2:06 ` Stephen Rothwell
  2019-02-11  9:38 ` Maxime Ripard
  0 siblings, 2 replies; 3+ messages in thread
From: Sam Ravnborg @ 2019-02-08 22:13 UTC (permalink / raw)
  To: dri-devel
  Cc: Stephen Rothwell, David Airlie, Liviu Dudau, James Wang,
	Sam Ravnborg

With drmP.h removed from drm_modeset_helper.h the build of
komeda filed as reported by linux-next

Add missing include files to fix build.
For the files touched group include files and sort them.

The fix was tested on a tree with drm-misc-next merged.
And the patch was also tested to work without drm-misc-next merged.

Build tested on arm + x86.

Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au> [linux-next]
Cc: Stephen Rothwell <sfr@canb.auug.org.au>
Cc: James Wang <james.qian.wang@arm.com>
Cc: Liviu Dudau <liviu.dudau@arm.com>
Cc: Brian Starkey <brian.starkey@arm.com>
Cc: David Airlie <airlied@linux.ie>
Cc: Daniel Vetter <daniel@ffwll.ch>
---
 drivers/gpu/drm/arm/display/komeda/komeda_crtc.c        |  8 ++++++--
 drivers/gpu/drm/arm/display/komeda/komeda_dev.c         |  6 +++++-
 drivers/gpu/drm/arm/display/komeda/komeda_framebuffer.c |  6 ++++--
 drivers/gpu/drm/arm/display/komeda/komeda_kms.c         | 12 ++++++++----
 drivers/gpu/drm/arm/display/komeda/komeda_kms.h         |  1 +
 drivers/gpu/drm/arm/display/komeda/komeda_pipeline.c    |  2 ++
 6 files changed, 26 insertions(+), 9 deletions(-)

diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c b/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c
index 5bb5a55f6b31..3ca5718aa0c2 100644
--- a/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c
+++ b/drivers/gpu/drm/arm/display/komeda/komeda_crtc.c
@@ -5,12 +5,16 @@
  *
  */
 #include <linux/clk.h>
+#include <linux/pm_runtime.h>
 #include <linux/spinlock.h>
+
 #include <drm/drm_atomic.h>
 #include <drm/drm_atomic_helper.h>
-#include <drm/drm_plane_helper.h>
 #include <drm/drm_crtc_helper.h>
-#include <linux/pm_runtime.h>
+#include <drm/drm_plane_helper.h>
+#include <drm/drm_print.h>
+#include <drm/drm_vblank.h>
+
 #include "komeda_dev.h"
 #include "komeda_kms.h"
 
diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
index 0fe6954fbbf4..70e9bb7fa30c 100644
--- a/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
+++ b/drivers/gpu/drm/arm/display/komeda/komeda_dev.c
@@ -4,9 +4,13 @@
  * Author: James.Qian.Wang <james.qian.wang@arm.com>
  *
  */
-#include <linux/platform_device.h>
+#include <linux/io.h>
 #include <linux/of_device.h>
 #include <linux/of_graph.h>
+#include <linux/platform_device.h>
+
+#include <drm/drm_print.h>
+
 #include "komeda_dev.h"
 
 static int komeda_parse_pipe_dt(struct komeda_dev *mdev, struct device_node *np)
diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_framebuffer.c b/drivers/gpu/drm/arm/display/komeda/komeda_framebuffer.c
index 23ee74d42239..9cc9935024f7 100644
--- a/drivers/gpu/drm/arm/display/komeda/komeda_framebuffer.c
+++ b/drivers/gpu/drm/arm/display/komeda/komeda_framebuffer.c
@@ -4,10 +4,12 @@
  * Author: James.Qian.Wang <james.qian.wang@arm.com>
  *
  */
-#include <drm/drm_gem.h>
-#include <drm/drm_gem_framebuffer_helper.h>
+#include <drm/drm_device.h>
 #include <drm/drm_fb_cma_helper.h>
+#include <drm/drm_gem.h>
 #include <drm/drm_gem_cma_helper.h>
+#include <drm/drm_gem_framebuffer_helper.h>
+
 #include "komeda_framebuffer.h"
 #include "komeda_dev.h"
 
diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_kms.c b/drivers/gpu/drm/arm/display/komeda/komeda_kms.c
index 3fc096d3883e..47a58ab20434 100644
--- a/drivers/gpu/drm/arm/display/komeda/komeda_kms.c
+++ b/drivers/gpu/drm/arm/display/komeda/komeda_kms.c
@@ -5,15 +5,19 @@
  *
  */
 #include <linux/component.h>
+#include <linux/interrupt.h>
+
 #include <drm/drm_atomic.h>
 #include <drm/drm_atomic_helper.h>
-#include <drm/drm_gem_framebuffer_helper.h>
-#include <drm/drm_gem_cma_helper.h>
+#include <drm/drm_drv.h>
 #include <drm/drm_fb_helper.h>
-#include <linux/interrupt.h>
+#include <drm/drm_gem_cma_helper.h>
+#include <drm/drm_gem_framebuffer_helper.h>
+#include <drm/drm_vblank.h>
+
 #include "komeda_dev.h"
-#include "komeda_kms.h"
 #include "komeda_framebuffer.h"
+#include "komeda_kms.h"
 
 DEFINE_DRM_GEM_CMA_FOPS(komeda_cma_fops);
 
diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_kms.h b/drivers/gpu/drm/arm/display/komeda/komeda_kms.h
index f13666004a42..874e9c9f0749 100644
--- a/drivers/gpu/drm/arm/display/komeda/komeda_kms.h
+++ b/drivers/gpu/drm/arm/display/komeda/komeda_kms.h
@@ -10,6 +10,7 @@
 #include <drm/drm_atomic.h>
 #include <drm/drm_atomic_helper.h>
 #include <drm/drm_crtc_helper.h>
+#include <drm/drm_device.h>
 #include <drm/drm_writeback.h>
 
 /** struct komeda_plane - komeda instance of drm_plane */
diff --git a/drivers/gpu/drm/arm/display/komeda/komeda_pipeline.c b/drivers/gpu/drm/arm/display/komeda/komeda_pipeline.c
index edb1cd7795f9..f1908e9ef128 100644
--- a/drivers/gpu/drm/arm/display/komeda/komeda_pipeline.c
+++ b/drivers/gpu/drm/arm/display/komeda/komeda_pipeline.c
@@ -4,6 +4,8 @@
  * Author: James.Qian.Wang <james.qian.wang@arm.com>
  *
  */
+#include <drm/drm_print.h>
+
 #include "komeda_dev.h"
 #include "komeda_pipeline.h"
 
-- 
2.12.0

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v1 1/1] drm/komeda: fix build with drm_modeset_helper.h update
  2019-02-08 22:13 [PATCH v1 1/1] drm/komeda: fix build with drm_modeset_helper.h update Sam Ravnborg
@ 2019-02-11  2:06 ` Stephen Rothwell
  2019-02-11  9:38 ` Maxime Ripard
  1 sibling, 0 replies; 3+ messages in thread
From: Stephen Rothwell @ 2019-02-11  2:06 UTC (permalink / raw)
  To: Sam Ravnborg; +Cc: David Airlie, Liviu Dudau, dri-devel, James Wang


[-- Attachment #1.1: Type: text/plain, Size: 1514 bytes --]

Hi Sam,

On Fri,  8 Feb 2019 23:13:24 +0100 Sam Ravnborg <sam@ravnborg.org> wrote:
>
> With drmP.h removed from drm_modeset_helper.h the build of
> komeda filed as reported by linux-next
> 
> Add missing include files to fix build.
> For the files touched group include files and sort them.
> 
> The fix was tested on a tree with drm-misc-next merged.
> And the patch was also tested to work without drm-misc-next merged.
> 
> Build tested on arm + x86.
> 
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au> [linux-next]
> Cc: Stephen Rothwell <sfr@canb.auug.org.au>
> Cc: James Wang <james.qian.wang@arm.com>
> Cc: Liviu Dudau <liviu.dudau@arm.com>
> Cc: Brian Starkey <brian.starkey@arm.com>
> Cc: David Airlie <airlied@linux.ie>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> ---
>  drivers/gpu/drm/arm/display/komeda/komeda_crtc.c        |  8 ++++++--
>  drivers/gpu/drm/arm/display/komeda/komeda_dev.c         |  6 +++++-
>  drivers/gpu/drm/arm/display/komeda/komeda_framebuffer.c |  6 ++++--
>  drivers/gpu/drm/arm/display/komeda/komeda_kms.c         | 12 ++++++++----
>  drivers/gpu/drm/arm/display/komeda/komeda_kms.h         |  1 +
>  drivers/gpu/drm/arm/display/komeda/komeda_pipeline.c    |  2 ++
>  6 files changed, 26 insertions(+), 9 deletions(-)

Applied to linux-next as a merge fix patch for the merge of the drm-misc tree today.

Tested-by: Stephen Rothwell <sfr@canb.auug.org.au>

-- 
Cheers,
Stephen Rothwell

[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v1 1/1] drm/komeda: fix build with drm_modeset_helper.h update
  2019-02-08 22:13 [PATCH v1 1/1] drm/komeda: fix build with drm_modeset_helper.h update Sam Ravnborg
  2019-02-11  2:06 ` Stephen Rothwell
@ 2019-02-11  9:38 ` Maxime Ripard
  1 sibling, 0 replies; 3+ messages in thread
From: Maxime Ripard @ 2019-02-11  9:38 UTC (permalink / raw)
  To: Sam Ravnborg
  Cc: David Airlie, Stephen Rothwell, James Wang, Liviu Dudau,
	dri-devel


[-- Attachment #1.1: Type: text/plain, Size: 993 bytes --]

On Fri, Feb 08, 2019 at 11:13:24PM +0100, Sam Ravnborg wrote:
> With drmP.h removed from drm_modeset_helper.h the build of
> komeda filed as reported by linux-next
> 
> Add missing include files to fix build.
> For the files touched group include files and sort them.
> 
> The fix was tested on a tree with drm-misc-next merged.
> And the patch was also tested to work without drm-misc-next merged.
> 
> Build tested on arm + x86.
> 
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au> [linux-next]
> Cc: Stephen Rothwell <sfr@canb.auug.org.au>
> Cc: James Wang <james.qian.wang@arm.com>
> Cc: Liviu Dudau <liviu.dudau@arm.com>
> Cc: Brian Starkey <brian.starkey@arm.com>
> Cc: David Airlie <airlied@linux.ie>
> Cc: Daniel Vetter <daniel@ffwll.ch>

I've applied it to drm-misc-next (after a drm-next backmerge), thanks!
Maxime

-- 
Maxime Ripard, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2019-02-11  9:38 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-02-08 22:13 [PATCH v1 1/1] drm/komeda: fix build with drm_modeset_helper.h update Sam Ravnborg
2019-02-11  2:06 ` Stephen Rothwell
2019-02-11  9:38 ` Maxime Ripard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox