[PATCH 7/9] drm: Extract drm_property.[hc]

Emil Velikov emil.l.velikov at gmail.com
Thu Aug 18 11:11:35 UTC 2016


Hi Daniel,

On 17 August 2016 at 21:56, Daniel Vetter <daniel.vetter at ffwll.ch> wrote:
> --- /dev/null
> +++ b/include/drm/drm_property.h

> +#ifndef __DRM_PROPERTY_H__
> +#define __DRM_PROPERTY_H__
> +
> +#include <linux/list.h>
> +#include <linux/ctype.h>
> +#include <drm/drm_mode_object.h>
> +
Add the following fwd declaration since we use a pointer to the said
struct ? From a brief look the other newly introduced headers
could/should use it as well.

struct drm_device;

The declarations in include/drm should be the ones meant for drivers
and there's no core drm internal ones ? Where/how does one manage API
that should be kept private between the different core DRM components,
even if there's symbol dependency between the different modules ?

Thanks !
Emil


More information about the dri-devel mailing list