Commit 331c201a authored by Jani Nikula's avatar Jani Nikula
Browse files

drm/i915: extract intel_audio.h from intel_drv.h 



It used to be handy that we only had a couple of headers, but over time
intel_drv.h has become unwieldy. Extract declarations to a separate
header file corresponding to the implementation module, clarifying the
modularity of the driver.

Ensure the new header is self-contained, and do so with minimal further
includes, using forward declarations as needed. Include the new header
only where needed, and sort the modified include directives while at it
and as needed.

No functional changes.

Reviewed-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Acked-by: default avatarJoonas Lahtinen <joonas.lahtinen@linux.intel.com>
Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/e0284c4d62effa5bad72ce034206c26e3aa02884.1554461791.git.jani.nikula@intel.com
parent e1ef734e

drivers/gpu/drm/i915/Makefile.header-test

+1 −0
Original line number Diff line number Diff line
@@ -8,6 +8,7 @@ header_test := \ 
	i915_priolist_types.h \

	i915_scheduler_types.h \

	i915_timeline_types.h \

	intel_audio.h \

	intel_context_types.h \

	intel_engine_types.h \

	intel_frontbuffer.h \

drivers/gpu/drm/i915/i915_drv.c

+3 −2
Original line number Diff line number Diff line
@@ -48,11 +48,12 @@ 
#include <drm/i915_drm.h>



#include "i915_drv.h"

#include "i915_trace.h"

#include "i915_pmu.h"

#include "i915_reset.h"

#include "i915_query.h"

#include "i915_reset.h"

#include "i915_trace.h"

#include "i915_vgpu.h"

#include "intel_audio.h"

#include "intel_drv.h"

#include "intel_uc.h"

#include "intel_workarounds.h"

drivers/gpu/drm/i915/intel_audio.c

+7 −5
Original line number Diff line number Diff line
@@ -21,14 +21,16 @@ 
 * DEALINGS IN THE SOFTWARE.

 */



#include <linux/kernel.h>

#include <linux/component.h>

#include <linux/kernel.h>



#include <drm/drm_edid.h>

#include <drm/i915_component.h>

#include <drm/intel_lpe_audio.h>

#include "intel_drv.h"



#include <drm/drm_edid.h>

#include "i915_drv.h"

#include "intel_audio.h"

#include "intel_drv.h"



/**

 * DOC: High Definition Audio over HDMI and Display Port
@@ -1045,7 +1047,7 @@ static const struct component_ops i915_audio_component_bind_ops = { 
 * We ignore any error during registration and continue with reduced

 * functionality (i.e. without HDMI audio).

 */

void i915_audio_component_init(struct drm_i915_private *dev_priv)

static void i915_audio_component_init(struct drm_i915_private *dev_priv)

{

	int ret;
@@ -1068,7 +1070,7 @@ void i915_audio_component_init(struct drm_i915_private *dev_priv) 
 * Deregisters the audio component, breaking any existing binding to the

 * corresponding snd_hda_intel driver's master component.

 */

void i915_audio_component_cleanup(struct drm_i915_private *dev_priv)

static void i915_audio_component_cleanup(struct drm_i915_private *dev_priv)

{

	if (!dev_priv->audio_component_registered)

		return;

drivers/gpu/drm/i915/intel_audio.h

0 → 100644
+24 −0
Original line number Diff line number Diff line 
/* SPDX-License-Identifier: MIT */

/*

 * Copyright © 2019 Intel Corporation

 */



#ifndef __INTEL_AUDIO_H__

#define __INTEL_AUDIO_H__



struct drm_connector_state;

struct drm_i915_private;

struct intel_crtc_state;

struct intel_encoder;



void intel_init_audio_hooks(struct drm_i915_private *dev_priv);

void intel_audio_codec_enable(struct intel_encoder *encoder,

			      const struct intel_crtc_state *crtc_state,

			      const struct drm_connector_state *conn_state);

void intel_audio_codec_disable(struct intel_encoder *encoder,

			       const struct intel_crtc_state *old_crtc_state,

			       const struct drm_connector_state *old_conn_state);

void intel_audio_init(struct drm_i915_private *dev_priv);

void intel_audio_deinit(struct drm_i915_private *dev_priv);



#endif /* __INTEL_AUDIO_H__ */

drivers/gpu/drm/i915/intel_ddi.c

+2 −0
Original line number Diff line number Diff line
@@ -26,7 +26,9 @@ 
 */



#include <drm/drm_scdc_helper.h>



#include "i915_drv.h"

#include "intel_audio.h"

#include "intel_drv.h"

#include "intel_dsi.h"

CRA Git | Maintained and supported by SUSTech CRA and CCSE