From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: "Laurent Pinchart" <laurent.pinchart+renesas@ideasonboard.com>,
dri-devel@lists.freedesktop.org,
"Alison Wang" <alison.wang@freescale.com>,
"Thomas Hellstrom" <thellstrom@vmware.com>,
linux-renesas-soc@vger.kernel.org,
"Alex Deucher" <alexander.deucher@amd.com>,
"Daniel Vetter" <daniel.vetter@intel.com>,
"Christian König" <christian.koenig@amd.com>,
"Ben Skeggs" <bskeggs@redhat.com>
Subject: Re: [PATCH v3 01/13] drm: Don't include <drm/drm_encoder.h> in <drm/drm_crtc.h>
Date: Tue, 29 Nov 2016 11:37:28 +0200 [thread overview]
Message-ID: <2855608.Q3yz9DLsQo@avalon> (raw)
In-Reply-To: <20161129093036.lojkchkpiazcrahk@phenom.ffwll.local>
Hi Daniel,
On Tuesday 29 Nov 2016 10:30:36 Daniel Vetter wrote:
> On Tue, Nov 29, 2016 at 11:04:31AM +0200, Laurent Pinchart wrote:
> > <drm/drm_crtc.h> used to define most of the in-kernel KMS API. It has
> > now been split into separate files for each object type, but still
> > includes most other KMS headers to avoid breaking driver compilation.
> >
> > As a step towards fixing that problem, remove the inclusion of
> > <drm/drm_encoder.h> from <drm/drm_crtc.h> and include it instead where
> > appropriate. Also remove the forward declarations of the drm_encoder and
> > drm_encoder_helper_funcs structures from <drm/drm_crtc.h> as they're not
> > needed in the header.
> >
> > <drm/drm_encoder.h> now has to include <drm/drm_mode.h> and contain a
> > forward declaration of struct drm_encoder in order to allow including it
> > as the first header in a compilation unit.
> >
> > Signed-off-by: Laurent Pinchart
> > <laurent.pinchart+renesas@ideasonboard.com>
>
> Assuming it all still compiles the various defconfigs we have (I didn't
> check that, but lgtm otherwise):
I've done my best to try and compile all affected drivers. 0day hasn't
complained either so far, so we should be good.
> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
>
> > ---
> >
> > drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h | 1 +
> > drivers/gpu/drm/ast/ast_drv.h | 1 +
> > drivers/gpu/drm/bochs/bochs.h | 1 +
> > drivers/gpu/drm/cirrus/cirrus_drv.h | 1 +
> > drivers/gpu/drm/drm_connector.c | 1 +
> > drivers/gpu/drm/drm_crtc_helper.c | 1 +
> > drivers/gpu/drm/drm_edid.c | 1 +
> > drivers/gpu/drm/drm_mode_config.c | 1 +
> > drivers/gpu/drm/drm_of.c | 1 +
> > drivers/gpu/drm/drm_plane_helper.c | 1 +
> > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.h | 2 ++
> > drivers/gpu/drm/gma500/psb_intel_drv.h | 1 +
> > drivers/gpu/drm/i915/intel_drv.h | 1 +
> > drivers/gpu/drm/mgag200/mgag200_drv.h | 1 +
> > drivers/gpu/drm/nouveau/nouveau_connector.h | 1 +
> > drivers/gpu/drm/qxl/qxl_drv.h | 1 +
> > drivers/gpu/drm/radeon/radeon_mode.h | 1 +
> > drivers/gpu/drm/rcar-du/rcar_du_encoder.h | 1 +
> > drivers/gpu/drm/shmobile/shmob_drm_crtc.h | 1 +
> > drivers/gpu/drm/tegra/drm.h | 1 +
> > drivers/gpu/drm/vc4/vc4_drv.h | 2 ++
> > drivers/gpu/drm/virtio/virtgpu_drv.h | 1 +
> > drivers/gpu/drm/vmwgfx/vmwgfx_kms.h | 1 +
> > include/drm/drm_crtc.h | 3 ---
> > include/drm/drm_encoder.h | 3 +++
> > include/drm/drm_encoder_slave.h | 1 +
> > include/drm/drm_modeset_helper_vtables.h | 1 +
> > 27 files changed, 30 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h
> > b/drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h index
> > 1e23334b07fb..fac06064a8f5 100644
> > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h
> > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h
> > @@ -32,6 +32,7 @@
> >
> > #include <drm/drm_crtc.h>
> > #include <drm/drm_edid.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_dp_helper.h>
> > #include <drm/drm_fixed.h>
> > #include <drm/drm_crtc_helper.h>
> >
> > diff --git a/drivers/gpu/drm/ast/ast_drv.h b/drivers/gpu/drm/ast/ast_drv.h
> > index 908011d2c8f5..6f3b6f50cf52 100644
> > --- a/drivers/gpu/drm/ast/ast_drv.h
> > +++ b/drivers/gpu/drm/ast/ast_drv.h
> > @@ -28,6 +28,7 @@
> >
> > #ifndef __AST_DRV_H__
> > #define __AST_DRV_H__
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_fb_helper.h>
> >
> > #include <drm/ttm/ttm_bo_api.h>
> >
> > diff --git a/drivers/gpu/drm/bochs/bochs.h b/drivers/gpu/drm/bochs/bochs.h
> > index 32dfe418cc98..f626bab7f5e3 100644
> > --- a/drivers/gpu/drm/bochs/bochs.h
> > +++ b/drivers/gpu/drm/bochs/bochs.h
> > @@ -4,6 +4,7 @@
> >
> > #include <drm/drmP.h>
> > #include <drm/drm_crtc.h>
> > #include <drm/drm_crtc_helper.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_fb_helper.h>
> >
> > #include <drm/drm_gem.h>
> >
> > diff --git a/drivers/gpu/drm/cirrus/cirrus_drv.h
> > b/drivers/gpu/drm/cirrus/cirrus_drv.h index 2188d6b61b3e..b59aeef4635a
> > 100644
> > --- a/drivers/gpu/drm/cirrus/cirrus_drv.h
> > +++ b/drivers/gpu/drm/cirrus/cirrus_drv.h
> > @@ -13,6 +13,7 @@
> >
> > #include <video/vga.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_fb_helper.h>
> >
> > #include <drm/ttm/ttm_bo_api.h>
> >
> > diff --git a/drivers/gpu/drm/drm_connector.c
> > b/drivers/gpu/drm/drm_connector.c index b5c6a8ee831e..5f1e1f190d30 100644
> > --- a/drivers/gpu/drm/drm_connector.c
> > +++ b/drivers/gpu/drm/drm_connector.c
> > @@ -23,6 +23,7 @@
> >
> > #include <drm/drmP.h>
> > #include <drm/drm_connector.h>
> > #include <drm/drm_edid.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include "drm_crtc_internal.h"
> > #include "drm_internal.h"
> >
> > diff --git a/drivers/gpu/drm/drm_crtc_helper.c
> > b/drivers/gpu/drm/drm_crtc_helper.c index 5d2cb138eba6..b3fc23313cc3
> > 100644
> > --- a/drivers/gpu/drm/drm_crtc_helper.c
> > +++ b/drivers/gpu/drm/drm_crtc_helper.c
> > @@ -36,6 +36,7 @@
> >
> > #include <drm/drmP.h>
> > #include <drm/drm_atomic.h>
> > #include <drm/drm_crtc.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_fourcc.h>
> > #include <drm/drm_crtc_helper.h>
> > #include <drm/drm_fb_helper.h>
> >
> > diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> > index 7eec18925b70..a9e3cc3990c1 100644
> > --- a/drivers/gpu/drm/drm_edid.c
> > +++ b/drivers/gpu/drm/drm_edid.c
> > @@ -35,6 +35,7 @@
> >
> > #include <linux/vga_switcheroo.h>
> > #include <drm/drmP.h>
> > #include <drm/drm_edid.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_displayid.h>
> >
> > #define version_greater(edid, maj, min) \
> >
> > diff --git a/drivers/gpu/drm/drm_mode_config.c
> > b/drivers/gpu/drm/drm_mode_config.c index 2735a5847ffa..09b1d8f267a6
> > 100644
> > --- a/drivers/gpu/drm/drm_mode_config.c
> > +++ b/drivers/gpu/drm/drm_mode_config.c
> > @@ -20,6 +20,7 @@
> >
> > * OF THIS SOFTWARE.
> > */
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_mode_config.h>
> > #include <drm/drmP.h>
> >
> > diff --git a/drivers/gpu/drm/drm_of.c b/drivers/gpu/drm/drm_of.c
> > index 47848ed8ca48..b5f2f0fece99 100644
> > --- a/drivers/gpu/drm/drm_of.c
> > +++ b/drivers/gpu/drm/drm_of.c
> > @@ -4,6 +4,7 @@
> >
> > #include <linux/of_graph.h>
> > #include <drm/drmP.h>
> > #include <drm/drm_crtc.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_of.h>
> >
> > static void drm_release_of(struct device *dev, void *data)
> >
> > diff --git a/drivers/gpu/drm/drm_plane_helper.c
> > b/drivers/gpu/drm/drm_plane_helper.c index 7a7dddf604d7..191a5f095cf9
> > 100644
> > --- a/drivers/gpu/drm/drm_plane_helper.c
> > +++ b/drivers/gpu/drm/drm_plane_helper.c
> > @@ -29,6 +29,7 @@
> >
> > #include <drm/drm_rect.h>
> > #include <drm/drm_atomic.h>
> > #include <drm/drm_crtc_helper.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_atomic_helper.h>
> >
> > #define SUBPIXEL_MASK 0xffff
> >
> > diff --git a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.h
> > b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.h index
> > 3b371fe7491e..4c22edd4c746 100644
> > --- a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.h
> > +++ b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.h
> > @@ -12,6 +12,8 @@
> >
> > #ifndef __FSL_DCU_DRM_DRV_H__
> > #define __FSL_DCU_DRM_DRV_H__
> >
> > +#include <drm/drm_encoder.h>
> > +
> >
> > #include "fsl_dcu_drm_crtc.h"
> > #include "fsl_dcu_drm_output.h"
> > #include "fsl_dcu_drm_plane.h"
> >
> > diff --git a/drivers/gpu/drm/gma500/psb_intel_drv.h
> > b/drivers/gpu/drm/gma500/psb_intel_drv.h index 2a3b7c684db2..6a10215fc42d
> > 100644
> > --- a/drivers/gpu/drm/gma500/psb_intel_drv.h
> > +++ b/drivers/gpu/drm/gma500/psb_intel_drv.h
> > @@ -23,6 +23,7 @@
> >
> > #include <linux/i2c-algo-bit.h>
> > #include <drm/drm_crtc.h>
> > #include <drm/drm_crtc_helper.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <linux/gpio.h>
> > #include "gma_display.h"
> >
> > diff --git a/drivers/gpu/drm/i915/intel_drv.h
> > b/drivers/gpu/drm/i915/intel_drv.h index 398195bf6dd1..7fb8ac824b01
> > 100644
> > --- a/drivers/gpu/drm/i915/intel_drv.h
> > +++ b/drivers/gpu/drm/i915/intel_drv.h
> > @@ -32,6 +32,7 @@
> >
> > #include "i915_drv.h"
> > #include <drm/drm_crtc.h>
> > #include <drm/drm_crtc_helper.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_fb_helper.h>
> > #include <drm/drm_dp_dual_mode_helper.h>
> > #include <drm/drm_dp_mst_helper.h>
> >
> > diff --git a/drivers/gpu/drm/mgag200/mgag200_drv.h
> > b/drivers/gpu/drm/mgag200/mgag200_drv.h index 3e02ac20777c..87e0934773de
> > 100644
> > --- a/drivers/gpu/drm/mgag200/mgag200_drv.h
> > +++ b/drivers/gpu/drm/mgag200/mgag200_drv.h
> > @@ -15,6 +15,7 @@
> >
> > #include <video/vga.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_fb_helper.h>
> > #include <drm/ttm/ttm_bo_api.h>
> > #include <drm/ttm/ttm_bo_driver.h>
> >
> > diff --git a/drivers/gpu/drm/nouveau/nouveau_connector.h
> > b/drivers/gpu/drm/nouveau/nouveau_connector.h index
> > 096983c42a1f..a4d1a059bd3d 100644
> > --- a/drivers/gpu/drm/nouveau/nouveau_connector.h
> > +++ b/drivers/gpu/drm/nouveau/nouveau_connector.h
> > @@ -30,6 +30,7 @@
> >
> > #include <nvif/notify.h>
> >
> > #include <drm/drm_edid.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_dp_helper.h>
> > #include "nouveau_crtc.h"
> >
> > diff --git a/drivers/gpu/drm/qxl/qxl_drv.h b/drivers/gpu/drm/qxl/qxl_drv.h
> > index 84995ebc6ffc..56df6b4ddbab 100644
> > --- a/drivers/gpu/drm/qxl/qxl_drv.h
> > +++ b/drivers/gpu/drm/qxl/qxl_drv.h
> > @@ -43,6 +43,7 @@
> >
> > #include <ttm/ttm_placement.h>
> > #include <ttm/ttm_module.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_gem.h>
> >
> > /* just for ttm_validate_buffer */
> >
> > diff --git a/drivers/gpu/drm/radeon/radeon_mode.h
> > b/drivers/gpu/drm/radeon/radeon_mode.h index bb75201a24ba..7ceea8ffd0db
> > 100644
> > --- a/drivers/gpu/drm/radeon/radeon_mode.h
> > +++ b/drivers/gpu/drm/radeon/radeon_mode.h
> > @@ -32,6 +32,7 @@
> >
> > #include <drm/drm_crtc.h>
> > #include <drm/drm_edid.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_dp_helper.h>
> > #include <drm/drm_dp_mst_helper.h>
> > #include <drm/drm_fixed.h>
> >
> > diff --git a/drivers/gpu/drm/rcar-du/rcar_du_encoder.h
> > b/drivers/gpu/drm/rcar-du/rcar_du_encoder.h index
> > 269fbab15907..b79b2f075a74 100644
> > --- a/drivers/gpu/drm/rcar-du/rcar_du_encoder.h
> > +++ b/drivers/gpu/drm/rcar-du/rcar_du_encoder.h
> > @@ -15,6 +15,7 @@
> >
> > #define __RCAR_DU_ENCODER_H__
> >
> > #include <drm/drm_crtc.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > struct drm_panel;
> > struct rcar_du_device;
> >
> > diff --git a/drivers/gpu/drm/shmobile/shmob_drm_crtc.h
> > b/drivers/gpu/drm/shmobile/shmob_drm_crtc.h index
> > 38ed4ff8aaf2..818b31549ddc 100644
> > --- a/drivers/gpu/drm/shmobile/shmob_drm_crtc.h
> > +++ b/drivers/gpu/drm/shmobile/shmob_drm_crtc.h
> > @@ -16,6 +16,7 @@
> >
> > #include <drm/drmP.h>
> > #include <drm/drm_crtc.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > struct backlight_device;
> > struct shmob_drm_device;
> >
> > diff --git a/drivers/gpu/drm/tegra/drm.h b/drivers/gpu/drm/tegra/drm.h
> > index 0ddcce1b420d..5205790dd679 100644
> > --- a/drivers/gpu/drm/tegra/drm.h
> > +++ b/drivers/gpu/drm/tegra/drm.h
> > @@ -17,6 +17,7 @@
> >
> > #include <drm/drmP.h>
> > #include <drm/drm_crtc_helper.h>
> > #include <drm/drm_edid.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_fb_helper.h>
> > #include <drm/drm_fixed.h>
> >
> > diff --git a/drivers/gpu/drm/vc4/vc4_drv.h b/drivers/gpu/drm/vc4/vc4_drv.h
> > index fef172804345..f89a95cb98cf 100644
> > --- a/drivers/gpu/drm/vc4/vc4_drv.h
> > +++ b/drivers/gpu/drm/vc4/vc4_drv.h
> > @@ -9,6 +9,8 @@
> >
> > #include "drmP.h"
> > #include "drm_gem_cma_helper.h"
> >
> > +#include <drm/drm_encoder.h>
> > +
> >
> > struct vc4_dev {
> >
> > struct drm_device *dev;
> >
> > diff --git a/drivers/gpu/drm/virtio/virtgpu_drv.h
> > b/drivers/gpu/drm/virtio/virtgpu_drv.h index ec1ebdcfe80b..475ac93848c3
> > 100644
> > --- a/drivers/gpu/drm/virtio/virtgpu_drv.h
> > +++ b/drivers/gpu/drm/virtio/virtgpu_drv.h
> > @@ -35,6 +35,7 @@
> >
> > #include <drm/drm_gem.h>
> > #include <drm/drm_atomic.h>
> > #include <drm/drm_crtc_helper.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <ttm/ttm_bo_api.h>
> > #include <ttm/ttm_bo_driver.h>
> > #include <ttm/ttm_placement.h>
> >
> > diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.h
> > b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.h index ff4803c107bc..5ee25fb07ea2
> > 100644
> > --- a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.h
> > +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.h
> > @@ -30,6 +30,7 @@
> >
> > #include <drm/drmP.h>
> > #include <drm/drm_crtc_helper.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include "vmwgfx_drv.h"
> >
> > /**
> >
> > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
> > index 946672f97e1e..9ef0ac0ce69f 100644
> > --- a/include/drm/drm_crtc.h
> > +++ b/include/drm/drm_crtc.h
> > @@ -39,7 +39,6 @@
> >
> > #include <drm/drm_framebuffer.h>
> > #include <drm/drm_modes.h>
> > #include <drm/drm_connector.h>
> >
> > -#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_property.h>
> > #include <drm/drm_bridge.h>
> > #include <drm/drm_edid.h>
> >
> > @@ -68,14 +67,12 @@ static inline uint64_t I642U64(int64_t val)
> >
> > }
> >
> > struct drm_crtc;
> >
> > -struct drm_encoder;
> >
> > struct drm_pending_vblank_event;
> > struct drm_plane;
> > struct drm_bridge;
> > struct drm_atomic_state;
> >
> > struct drm_crtc_helper_funcs;
> >
> > -struct drm_encoder_helper_funcs;
> >
> > struct drm_plane_helper_funcs;
> >
> > /**
> >
> > diff --git a/include/drm/drm_encoder.h b/include/drm/drm_encoder.h
> > index c7438ff0d609..cec6ac45c6cc 100644
> > --- a/include/drm/drm_encoder.h
> > +++ b/include/drm/drm_encoder.h
> > @@ -25,8 +25,11 @@
> >
> > #include <linux/list.h>
> > #include <linux/ctype.h>
> >
> > +#include <drm/drm_mode.h>
> >
> > #include <drm/drm_mode_object.h>
> >
> > +struct drm_encoder;
> > +
> >
> > /**
> >
> > * struct drm_encoder_funcs - encoder controls
> > *
> >
> > diff --git a/include/drm/drm_encoder_slave.h
> > b/include/drm/drm_encoder_slave.h index 82cdf611393d..1107b4b1c599 100644
> > --- a/include/drm/drm_encoder_slave.h
> > +++ b/include/drm/drm_encoder_slave.h
> > @@ -29,6 +29,7 @@
> >
> > #include <drm/drmP.h>
> > #include <drm/drm_crtc.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > /**
> >
> > * struct drm_encoder_slave_funcs - Entry points exposed by a slave
> > encoder driver>
> > diff --git a/include/drm/drm_modeset_helper_vtables.h
> > b/include/drm/drm_modeset_helper_vtables.h index
> > 72478cf82147..21d9823d4872 100644
> > --- a/include/drm/drm_modeset_helper_vtables.h
> > +++ b/include/drm/drm_modeset_helper_vtables.h
> > @@ -30,6 +30,7 @@
> >
> > #define __DRM_MODESET_HELPER_VTABLES_H__
> >
> > #include <drm/drm_crtc.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > /**
> >
> > * DOC: overview
--
Regards,
Laurent Pinchart
WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: "Thomas Hellstrom" <thellstrom@vmware.com>,
"Laurent Pinchart" <laurent.pinchart+renesas@ideasonboard.com>,
"Alison Wang" <alison.wang@freescale.com>,
dri-devel@lists.freedesktop.org,
linux-renesas-soc@vger.kernel.org,
"Ben Skeggs" <bskeggs@redhat.com>,
"Alex Deucher" <alexander.deucher@amd.com>,
"Daniel Vetter" <daniel.vetter@intel.com>,
"Christian König" <christian.koenig@amd.com>
Subject: Re: [PATCH v3 01/13] drm: Don't include <drm/drm_encoder.h> in <drm/drm_crtc.h>
Date: Tue, 29 Nov 2016 11:37:28 +0200 [thread overview]
Message-ID: <2855608.Q3yz9DLsQo@avalon> (raw)
In-Reply-To: <20161129093036.lojkchkpiazcrahk@phenom.ffwll.local>
Hi Daniel,
On Tuesday 29 Nov 2016 10:30:36 Daniel Vetter wrote:
> On Tue, Nov 29, 2016 at 11:04:31AM +0200, Laurent Pinchart wrote:
> > <drm/drm_crtc.h> used to define most of the in-kernel KMS API. It has
> > now been split into separate files for each object type, but still
> > includes most other KMS headers to avoid breaking driver compilation.
> >
> > As a step towards fixing that problem, remove the inclusion of
> > <drm/drm_encoder.h> from <drm/drm_crtc.h> and include it instead where
> > appropriate. Also remove the forward declarations of the drm_encoder and
> > drm_encoder_helper_funcs structures from <drm/drm_crtc.h> as they're not
> > needed in the header.
> >
> > <drm/drm_encoder.h> now has to include <drm/drm_mode.h> and contain a
> > forward declaration of struct drm_encoder in order to allow including it
> > as the first header in a compilation unit.
> >
> > Signed-off-by: Laurent Pinchart
> > <laurent.pinchart+renesas@ideasonboard.com>
>
> Assuming it all still compiles the various defconfigs we have (I didn't
> check that, but lgtm otherwise):
I've done my best to try and compile all affected drivers. 0day hasn't
complained either so far, so we should be good.
> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
>
> > ---
> >
> > drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h | 1 +
> > drivers/gpu/drm/ast/ast_drv.h | 1 +
> > drivers/gpu/drm/bochs/bochs.h | 1 +
> > drivers/gpu/drm/cirrus/cirrus_drv.h | 1 +
> > drivers/gpu/drm/drm_connector.c | 1 +
> > drivers/gpu/drm/drm_crtc_helper.c | 1 +
> > drivers/gpu/drm/drm_edid.c | 1 +
> > drivers/gpu/drm/drm_mode_config.c | 1 +
> > drivers/gpu/drm/drm_of.c | 1 +
> > drivers/gpu/drm/drm_plane_helper.c | 1 +
> > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.h | 2 ++
> > drivers/gpu/drm/gma500/psb_intel_drv.h | 1 +
> > drivers/gpu/drm/i915/intel_drv.h | 1 +
> > drivers/gpu/drm/mgag200/mgag200_drv.h | 1 +
> > drivers/gpu/drm/nouveau/nouveau_connector.h | 1 +
> > drivers/gpu/drm/qxl/qxl_drv.h | 1 +
> > drivers/gpu/drm/radeon/radeon_mode.h | 1 +
> > drivers/gpu/drm/rcar-du/rcar_du_encoder.h | 1 +
> > drivers/gpu/drm/shmobile/shmob_drm_crtc.h | 1 +
> > drivers/gpu/drm/tegra/drm.h | 1 +
> > drivers/gpu/drm/vc4/vc4_drv.h | 2 ++
> > drivers/gpu/drm/virtio/virtgpu_drv.h | 1 +
> > drivers/gpu/drm/vmwgfx/vmwgfx_kms.h | 1 +
> > include/drm/drm_crtc.h | 3 ---
> > include/drm/drm_encoder.h | 3 +++
> > include/drm/drm_encoder_slave.h | 1 +
> > include/drm/drm_modeset_helper_vtables.h | 1 +
> > 27 files changed, 30 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h
> > b/drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h index
> > 1e23334b07fb..fac06064a8f5 100644
> > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h
> > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h
> > @@ -32,6 +32,7 @@
> >
> > #include <drm/drm_crtc.h>
> > #include <drm/drm_edid.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_dp_helper.h>
> > #include <drm/drm_fixed.h>
> > #include <drm/drm_crtc_helper.h>
> >
> > diff --git a/drivers/gpu/drm/ast/ast_drv.h b/drivers/gpu/drm/ast/ast_drv.h
> > index 908011d2c8f5..6f3b6f50cf52 100644
> > --- a/drivers/gpu/drm/ast/ast_drv.h
> > +++ b/drivers/gpu/drm/ast/ast_drv.h
> > @@ -28,6 +28,7 @@
> >
> > #ifndef __AST_DRV_H__
> > #define __AST_DRV_H__
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_fb_helper.h>
> >
> > #include <drm/ttm/ttm_bo_api.h>
> >
> > diff --git a/drivers/gpu/drm/bochs/bochs.h b/drivers/gpu/drm/bochs/bochs.h
> > index 32dfe418cc98..f626bab7f5e3 100644
> > --- a/drivers/gpu/drm/bochs/bochs.h
> > +++ b/drivers/gpu/drm/bochs/bochs.h
> > @@ -4,6 +4,7 @@
> >
> > #include <drm/drmP.h>
> > #include <drm/drm_crtc.h>
> > #include <drm/drm_crtc_helper.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_fb_helper.h>
> >
> > #include <drm/drm_gem.h>
> >
> > diff --git a/drivers/gpu/drm/cirrus/cirrus_drv.h
> > b/drivers/gpu/drm/cirrus/cirrus_drv.h index 2188d6b61b3e..b59aeef4635a
> > 100644
> > --- a/drivers/gpu/drm/cirrus/cirrus_drv.h
> > +++ b/drivers/gpu/drm/cirrus/cirrus_drv.h
> > @@ -13,6 +13,7 @@
> >
> > #include <video/vga.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_fb_helper.h>
> >
> > #include <drm/ttm/ttm_bo_api.h>
> >
> > diff --git a/drivers/gpu/drm/drm_connector.c
> > b/drivers/gpu/drm/drm_connector.c index b5c6a8ee831e..5f1e1f190d30 100644
> > --- a/drivers/gpu/drm/drm_connector.c
> > +++ b/drivers/gpu/drm/drm_connector.c
> > @@ -23,6 +23,7 @@
> >
> > #include <drm/drmP.h>
> > #include <drm/drm_connector.h>
> > #include <drm/drm_edid.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include "drm_crtc_internal.h"
> > #include "drm_internal.h"
> >
> > diff --git a/drivers/gpu/drm/drm_crtc_helper.c
> > b/drivers/gpu/drm/drm_crtc_helper.c index 5d2cb138eba6..b3fc23313cc3
> > 100644
> > --- a/drivers/gpu/drm/drm_crtc_helper.c
> > +++ b/drivers/gpu/drm/drm_crtc_helper.c
> > @@ -36,6 +36,7 @@
> >
> > #include <drm/drmP.h>
> > #include <drm/drm_atomic.h>
> > #include <drm/drm_crtc.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_fourcc.h>
> > #include <drm/drm_crtc_helper.h>
> > #include <drm/drm_fb_helper.h>
> >
> > diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c
> > index 7eec18925b70..a9e3cc3990c1 100644
> > --- a/drivers/gpu/drm/drm_edid.c
> > +++ b/drivers/gpu/drm/drm_edid.c
> > @@ -35,6 +35,7 @@
> >
> > #include <linux/vga_switcheroo.h>
> > #include <drm/drmP.h>
> > #include <drm/drm_edid.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_displayid.h>
> >
> > #define version_greater(edid, maj, min) \
> >
> > diff --git a/drivers/gpu/drm/drm_mode_config.c
> > b/drivers/gpu/drm/drm_mode_config.c index 2735a5847ffa..09b1d8f267a6
> > 100644
> > --- a/drivers/gpu/drm/drm_mode_config.c
> > +++ b/drivers/gpu/drm/drm_mode_config.c
> > @@ -20,6 +20,7 @@
> >
> > * OF THIS SOFTWARE.
> > */
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_mode_config.h>
> > #include <drm/drmP.h>
> >
> > diff --git a/drivers/gpu/drm/drm_of.c b/drivers/gpu/drm/drm_of.c
> > index 47848ed8ca48..b5f2f0fece99 100644
> > --- a/drivers/gpu/drm/drm_of.c
> > +++ b/drivers/gpu/drm/drm_of.c
> > @@ -4,6 +4,7 @@
> >
> > #include <linux/of_graph.h>
> > #include <drm/drmP.h>
> > #include <drm/drm_crtc.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_of.h>
> >
> > static void drm_release_of(struct device *dev, void *data)
> >
> > diff --git a/drivers/gpu/drm/drm_plane_helper.c
> > b/drivers/gpu/drm/drm_plane_helper.c index 7a7dddf604d7..191a5f095cf9
> > 100644
> > --- a/drivers/gpu/drm/drm_plane_helper.c
> > +++ b/drivers/gpu/drm/drm_plane_helper.c
> > @@ -29,6 +29,7 @@
> >
> > #include <drm/drm_rect.h>
> > #include <drm/drm_atomic.h>
> > #include <drm/drm_crtc_helper.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_atomic_helper.h>
> >
> > #define SUBPIXEL_MASK 0xffff
> >
> > diff --git a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.h
> > b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.h index
> > 3b371fe7491e..4c22edd4c746 100644
> > --- a/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.h
> > +++ b/drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.h
> > @@ -12,6 +12,8 @@
> >
> > #ifndef __FSL_DCU_DRM_DRV_H__
> > #define __FSL_DCU_DRM_DRV_H__
> >
> > +#include <drm/drm_encoder.h>
> > +
> >
> > #include "fsl_dcu_drm_crtc.h"
> > #include "fsl_dcu_drm_output.h"
> > #include "fsl_dcu_drm_plane.h"
> >
> > diff --git a/drivers/gpu/drm/gma500/psb_intel_drv.h
> > b/drivers/gpu/drm/gma500/psb_intel_drv.h index 2a3b7c684db2..6a10215fc42d
> > 100644
> > --- a/drivers/gpu/drm/gma500/psb_intel_drv.h
> > +++ b/drivers/gpu/drm/gma500/psb_intel_drv.h
> > @@ -23,6 +23,7 @@
> >
> > #include <linux/i2c-algo-bit.h>
> > #include <drm/drm_crtc.h>
> > #include <drm/drm_crtc_helper.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <linux/gpio.h>
> > #include "gma_display.h"
> >
> > diff --git a/drivers/gpu/drm/i915/intel_drv.h
> > b/drivers/gpu/drm/i915/intel_drv.h index 398195bf6dd1..7fb8ac824b01
> > 100644
> > --- a/drivers/gpu/drm/i915/intel_drv.h
> > +++ b/drivers/gpu/drm/i915/intel_drv.h
> > @@ -32,6 +32,7 @@
> >
> > #include "i915_drv.h"
> > #include <drm/drm_crtc.h>
> > #include <drm/drm_crtc_helper.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_fb_helper.h>
> > #include <drm/drm_dp_dual_mode_helper.h>
> > #include <drm/drm_dp_mst_helper.h>
> >
> > diff --git a/drivers/gpu/drm/mgag200/mgag200_drv.h
> > b/drivers/gpu/drm/mgag200/mgag200_drv.h index 3e02ac20777c..87e0934773de
> > 100644
> > --- a/drivers/gpu/drm/mgag200/mgag200_drv.h
> > +++ b/drivers/gpu/drm/mgag200/mgag200_drv.h
> > @@ -15,6 +15,7 @@
> >
> > #include <video/vga.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_fb_helper.h>
> > #include <drm/ttm/ttm_bo_api.h>
> > #include <drm/ttm/ttm_bo_driver.h>
> >
> > diff --git a/drivers/gpu/drm/nouveau/nouveau_connector.h
> > b/drivers/gpu/drm/nouveau/nouveau_connector.h index
> > 096983c42a1f..a4d1a059bd3d 100644
> > --- a/drivers/gpu/drm/nouveau/nouveau_connector.h
> > +++ b/drivers/gpu/drm/nouveau/nouveau_connector.h
> > @@ -30,6 +30,7 @@
> >
> > #include <nvif/notify.h>
> >
> > #include <drm/drm_edid.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_dp_helper.h>
> > #include "nouveau_crtc.h"
> >
> > diff --git a/drivers/gpu/drm/qxl/qxl_drv.h b/drivers/gpu/drm/qxl/qxl_drv.h
> > index 84995ebc6ffc..56df6b4ddbab 100644
> > --- a/drivers/gpu/drm/qxl/qxl_drv.h
> > +++ b/drivers/gpu/drm/qxl/qxl_drv.h
> > @@ -43,6 +43,7 @@
> >
> > #include <ttm/ttm_placement.h>
> > #include <ttm/ttm_module.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_gem.h>
> >
> > /* just for ttm_validate_buffer */
> >
> > diff --git a/drivers/gpu/drm/radeon/radeon_mode.h
> > b/drivers/gpu/drm/radeon/radeon_mode.h index bb75201a24ba..7ceea8ffd0db
> > 100644
> > --- a/drivers/gpu/drm/radeon/radeon_mode.h
> > +++ b/drivers/gpu/drm/radeon/radeon_mode.h
> > @@ -32,6 +32,7 @@
> >
> > #include <drm/drm_crtc.h>
> > #include <drm/drm_edid.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_dp_helper.h>
> > #include <drm/drm_dp_mst_helper.h>
> > #include <drm/drm_fixed.h>
> >
> > diff --git a/drivers/gpu/drm/rcar-du/rcar_du_encoder.h
> > b/drivers/gpu/drm/rcar-du/rcar_du_encoder.h index
> > 269fbab15907..b79b2f075a74 100644
> > --- a/drivers/gpu/drm/rcar-du/rcar_du_encoder.h
> > +++ b/drivers/gpu/drm/rcar-du/rcar_du_encoder.h
> > @@ -15,6 +15,7 @@
> >
> > #define __RCAR_DU_ENCODER_H__
> >
> > #include <drm/drm_crtc.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > struct drm_panel;
> > struct rcar_du_device;
> >
> > diff --git a/drivers/gpu/drm/shmobile/shmob_drm_crtc.h
> > b/drivers/gpu/drm/shmobile/shmob_drm_crtc.h index
> > 38ed4ff8aaf2..818b31549ddc 100644
> > --- a/drivers/gpu/drm/shmobile/shmob_drm_crtc.h
> > +++ b/drivers/gpu/drm/shmobile/shmob_drm_crtc.h
> > @@ -16,6 +16,7 @@
> >
> > #include <drm/drmP.h>
> > #include <drm/drm_crtc.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > struct backlight_device;
> > struct shmob_drm_device;
> >
> > diff --git a/drivers/gpu/drm/tegra/drm.h b/drivers/gpu/drm/tegra/drm.h
> > index 0ddcce1b420d..5205790dd679 100644
> > --- a/drivers/gpu/drm/tegra/drm.h
> > +++ b/drivers/gpu/drm/tegra/drm.h
> > @@ -17,6 +17,7 @@
> >
> > #include <drm/drmP.h>
> > #include <drm/drm_crtc_helper.h>
> > #include <drm/drm_edid.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_fb_helper.h>
> > #include <drm/drm_fixed.h>
> >
> > diff --git a/drivers/gpu/drm/vc4/vc4_drv.h b/drivers/gpu/drm/vc4/vc4_drv.h
> > index fef172804345..f89a95cb98cf 100644
> > --- a/drivers/gpu/drm/vc4/vc4_drv.h
> > +++ b/drivers/gpu/drm/vc4/vc4_drv.h
> > @@ -9,6 +9,8 @@
> >
> > #include "drmP.h"
> > #include "drm_gem_cma_helper.h"
> >
> > +#include <drm/drm_encoder.h>
> > +
> >
> > struct vc4_dev {
> >
> > struct drm_device *dev;
> >
> > diff --git a/drivers/gpu/drm/virtio/virtgpu_drv.h
> > b/drivers/gpu/drm/virtio/virtgpu_drv.h index ec1ebdcfe80b..475ac93848c3
> > 100644
> > --- a/drivers/gpu/drm/virtio/virtgpu_drv.h
> > +++ b/drivers/gpu/drm/virtio/virtgpu_drv.h
> > @@ -35,6 +35,7 @@
> >
> > #include <drm/drm_gem.h>
> > #include <drm/drm_atomic.h>
> > #include <drm/drm_crtc_helper.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include <ttm/ttm_bo_api.h>
> > #include <ttm/ttm_bo_driver.h>
> > #include <ttm/ttm_placement.h>
> >
> > diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.h
> > b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.h index ff4803c107bc..5ee25fb07ea2
> > 100644
> > --- a/drivers/gpu/drm/vmwgfx/vmwgfx_kms.h
> > +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_kms.h
> > @@ -30,6 +30,7 @@
> >
> > #include <drm/drmP.h>
> > #include <drm/drm_crtc_helper.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > #include "vmwgfx_drv.h"
> >
> > /**
> >
> > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
> > index 946672f97e1e..9ef0ac0ce69f 100644
> > --- a/include/drm/drm_crtc.h
> > +++ b/include/drm/drm_crtc.h
> > @@ -39,7 +39,6 @@
> >
> > #include <drm/drm_framebuffer.h>
> > #include <drm/drm_modes.h>
> > #include <drm/drm_connector.h>
> >
> > -#include <drm/drm_encoder.h>
> >
> > #include <drm/drm_property.h>
> > #include <drm/drm_bridge.h>
> > #include <drm/drm_edid.h>
> >
> > @@ -68,14 +67,12 @@ static inline uint64_t I642U64(int64_t val)
> >
> > }
> >
> > struct drm_crtc;
> >
> > -struct drm_encoder;
> >
> > struct drm_pending_vblank_event;
> > struct drm_plane;
> > struct drm_bridge;
> > struct drm_atomic_state;
> >
> > struct drm_crtc_helper_funcs;
> >
> > -struct drm_encoder_helper_funcs;
> >
> > struct drm_plane_helper_funcs;
> >
> > /**
> >
> > diff --git a/include/drm/drm_encoder.h b/include/drm/drm_encoder.h
> > index c7438ff0d609..cec6ac45c6cc 100644
> > --- a/include/drm/drm_encoder.h
> > +++ b/include/drm/drm_encoder.h
> > @@ -25,8 +25,11 @@
> >
> > #include <linux/list.h>
> > #include <linux/ctype.h>
> >
> > +#include <drm/drm_mode.h>
> >
> > #include <drm/drm_mode_object.h>
> >
> > +struct drm_encoder;
> > +
> >
> > /**
> >
> > * struct drm_encoder_funcs - encoder controls
> > *
> >
> > diff --git a/include/drm/drm_encoder_slave.h
> > b/include/drm/drm_encoder_slave.h index 82cdf611393d..1107b4b1c599 100644
> > --- a/include/drm/drm_encoder_slave.h
> > +++ b/include/drm/drm_encoder_slave.h
> > @@ -29,6 +29,7 @@
> >
> > #include <drm/drmP.h>
> > #include <drm/drm_crtc.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > /**
> >
> > * struct drm_encoder_slave_funcs - Entry points exposed by a slave
> > encoder driver>
> > diff --git a/include/drm/drm_modeset_helper_vtables.h
> > b/include/drm/drm_modeset_helper_vtables.h index
> > 72478cf82147..21d9823d4872 100644
> > --- a/include/drm/drm_modeset_helper_vtables.h
> > +++ b/include/drm/drm_modeset_helper_vtables.h
> > @@ -30,6 +30,7 @@
> >
> > #define __DRM_MODESET_HELPER_VTABLES_H__
> >
> > #include <drm/drm_crtc.h>
> >
> > +#include <drm/drm_encoder.h>
> >
> > /**
> >
> > * DOC: overview
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2016-11-29 9:37 UTC|newest]
Thread overview: 100+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-29 9:04 [PATCH v3 00/13] R-Car DU: Use drm bridge API Laurent Pinchart
2016-11-29 9:04 ` Laurent Pinchart
2016-11-29 9:04 ` [PATCH v3 01/13] drm: Don't include <drm/drm_encoder.h> in <drm/drm_crtc.h> Laurent Pinchart
2016-11-29 9:30 ` Daniel Vetter
2016-11-29 9:30 ` Daniel Vetter
2016-11-29 9:37 ` Laurent Pinchart [this message]
2016-11-29 9:37 ` Laurent Pinchart
2016-12-02 21:21 ` Sinclair Yeh
2016-12-02 21:21 ` Sinclair Yeh
2016-11-29 9:04 ` [PATCH v3 02/13] drm: Fix compilation warning caused by static inline forward declaration Laurent Pinchart
2016-11-29 9:04 ` Laurent Pinchart
2016-11-29 9:31 ` Daniel Vetter
2016-11-29 9:31 ` Daniel Vetter
2016-11-29 9:04 ` [PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code Laurent Pinchart
2016-11-29 9:04 ` Laurent Pinchart
2016-11-29 9:35 ` Daniel Vetter
2016-11-29 9:35 ` Daniel Vetter
2016-11-29 9:43 ` Laurent Pinchart
2016-11-29 10:05 ` Daniel Vetter
2016-11-29 10:05 ` Daniel Vetter
2016-11-29 18:02 ` Laurent Pinchart
2016-11-29 18:02 ` Laurent Pinchart
2016-11-29 18:51 ` Laurent Pinchart
2016-11-29 10:27 ` Archit Taneja
2016-11-29 10:27 ` Archit Taneja
2016-11-29 17:57 ` Laurent Pinchart
2016-11-29 17:57 ` Laurent Pinchart
2016-11-30 5:05 ` Archit Taneja
2016-11-30 5:05 ` Archit Taneja
2016-11-30 10:23 ` Laurent Pinchart
2016-11-30 10:23 ` Laurent Pinchart
2016-11-30 11:00 ` Archit Taneja
2016-11-30 11:00 ` Archit Taneja
2016-11-30 11:05 ` Laurent Pinchart
2016-11-30 11:05 ` Laurent Pinchart
2016-11-30 13:27 ` Archit Taneja
2016-11-30 13:27 ` Archit Taneja
2016-11-29 17:01 ` Stefan Agner
2016-11-29 17:01 ` Stefan Agner
2016-11-29 19:58 ` Boris Brezillon
2016-11-29 19:58 ` Boris Brezillon
2016-11-30 15:30 ` Vincent ABRIOU
2016-11-30 15:30 ` Vincent ABRIOU
2016-11-29 9:04 ` [PATCH v3 04/13] drm: bridge: Detach bridge from encoder at encoder cleanup time Laurent Pinchart
2016-11-29 9:04 ` Laurent Pinchart
2016-11-29 9:48 ` Daniel Vetter
2016-11-29 9:48 ` Daniel Vetter
2016-11-29 19:00 ` Laurent Pinchart
2016-11-29 10:34 ` Archit Taneja
2016-11-29 18:56 ` Laurent Pinchart
2016-11-29 18:56 ` Laurent Pinchart
2016-11-29 20:22 ` Daniel Vetter
2016-11-29 20:22 ` Daniel Vetter
2016-11-29 21:54 ` [PATCH] drm: bridge: Detach all bridges in a chain " Laurent Pinchart
[not found] ` <1480410283-28698-1-git-send-email-laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>
2016-11-29 9:04 ` [PATCH v3 05/13] drm: bridge: Add LVDS encoder DT bindings Laurent Pinchart
2016-11-29 9:04 ` Laurent Pinchart
2016-11-29 9:04 ` [PATCH v3 06/13] drm: bridge: Add LVDS encoder driver Laurent Pinchart
2016-11-29 9:54 ` Daniel Vetter
2016-11-29 20:57 ` Laurent Pinchart
2016-11-29 20:57 ` Laurent Pinchart
2017-01-04 1:33 ` Laurent Pinchart
2017-01-04 8:18 ` Daniel Vetter
2017-01-04 8:18 ` Daniel Vetter
2017-01-04 13:08 ` Laurent Pinchart
2017-01-04 13:08 ` Laurent Pinchart
2017-01-04 13:51 ` Daniel Vetter
2017-01-04 14:33 ` Laurent Pinchart
2017-01-04 14:33 ` Laurent Pinchart
2017-01-04 14:58 ` Daniel Vetter
2017-01-04 14:58 ` Daniel Vetter
2017-01-04 15:13 ` Laurent Pinchart
2017-01-04 15:13 ` Laurent Pinchart
2017-03-02 0:30 ` Laurent Pinchart
2017-03-02 0:30 ` Laurent Pinchart
2017-03-02 7:05 ` Daniel Vetter
2017-03-02 7:05 ` Daniel Vetter
2016-11-29 9:04 ` [PATCH v3 07/13] drm: bridge: vga-dac: Add adi,adv7123 compatible string Laurent Pinchart
2016-11-29 9:04 ` [PATCH v3 07/13] drm: bridge: vga-dac: Add adi, adv7123 " Laurent Pinchart
2016-11-29 9:50 ` [PATCH v3 07/13] drm: bridge: vga-dac: Add adi,adv7123 " Maxime Ripard
2016-11-29 9:50 ` Maxime Ripard
2016-11-29 9:04 ` [PATCH v3 08/13] drm: bridge: lvds-encoder: Add thine,thc63lvdm83d " Laurent Pinchart
2016-11-29 9:04 ` [PATCH v3 08/13] drm: bridge: lvds-encoder: Add thine, thc63lvdm83d " Laurent Pinchart
2016-11-29 9:04 ` [PATCH v3 09/13] drm: Add encoder_type field to the drm_bridge structure Laurent Pinchart
2016-11-29 9:56 ` Daniel Vetter
2016-11-29 9:58 ` Laurent Pinchart
2016-11-29 9:58 ` Laurent Pinchart
2016-11-29 10:27 ` Daniel Vetter
2016-11-29 10:27 ` Daniel Vetter
2016-11-29 17:49 ` Laurent Pinchart
2016-11-29 20:25 ` Daniel Vetter
2016-11-29 22:42 ` Laurent Pinchart
2016-11-29 9:04 ` [PATCH v3 10/13] drm: bridge: Set bridges' encoder type Laurent Pinchart
2016-11-29 9:04 ` [PATCH v3 11/13] drm: Set on-chip " Laurent Pinchart
2016-11-29 9:04 ` Laurent Pinchart
2016-11-30 15:28 ` Vincent ABRIOU
2016-11-30 15:28 ` Vincent ABRIOU
2016-11-29 9:04 ` [PATCH v3 12/13] drm: rcar-du: Replace manual bridge implementation with DRM bridge Laurent Pinchart
2016-12-27 12:40 ` Geert Uytterhoeven
2016-12-27 12:40 ` Geert Uytterhoeven
2016-11-29 9:04 ` [PATCH v3 13/13] drm: rcar-du: Initialize encoder's type based on the bridge's type Laurent Pinchart
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2855608.Q3yz9DLsQo@avalon \
--to=laurent.pinchart@ideasonboard.com \
--cc=alexander.deucher@amd.com \
--cc=alison.wang@freescale.com \
--cc=bskeggs@redhat.com \
--cc=christian.koenig@amd.com \
--cc=daniel.vetter@intel.com \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=laurent.pinchart+renesas@ideasonboard.com \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=thellstrom@vmware.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.