* [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