Commit 6e007c3b authored by Jani Nikula's avatar Jani Nikula
Browse files

drm/i915/color: hide struct intel_color_funcs 



The struct is only needed in intel_color.c, move it there.

Signed-off-by: default avatarJani Nikula <jani.nikula@intel.com>
Reviewed-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/68cf44fab55c047253c3ed87f8afcf0a86fa157a.1643896905.git.jani.nikula@intel.com
parent 2ecf64a0

drivers/gpu/drm/i915/display/intel_color.c

+19 −0
Original line number Diff line number Diff line
@@ -28,6 +28,25 @@ 
#include "intel_dpll.h"

#include "vlv_dsi_pll.h"



struct intel_color_funcs {

	int (*color_check)(struct intel_crtc_state *crtc_state);

	/*

	 * Program double buffered color management registers during

	 * vblank evasion. The registers should then latch during the

	 * next vblank start, alongside any other double buffered registers

	 * involved with the same commit.

	 */

	void (*color_commit)(const struct intel_crtc_state *crtc_state);

	/*

	 * Load LUTs (and other single buffered color management

	 * registers). Will (hopefully) be called during the vblank

	 * following the latching of any double buffered registers

	 * involved with the same commit.

	 */

	void (*load_luts)(const struct intel_crtc_state *crtc_state);

	void (*read_luts)(struct intel_crtc_state *crtc_state);

};



#define CTM_COEFF_SIGN	(1ULL << 63)



#define CTM_COEFF_1_0	(1ULL << 32)

drivers/gpu/drm/i915/i915_drv.h

+1 −19
Original line number Diff line number Diff line
@@ -115,6 +115,7 @@ struct intel_cdclk_config; 
struct intel_cdclk_funcs;

struct intel_cdclk_state;

struct intel_cdclk_vals;

struct intel_color_funcs;

struct intel_connector;

struct intel_crtc;

struct intel_dp;
@@ -320,25 +321,6 @@ struct drm_i915_wm_disp_funcs { 
	int (*compute_global_watermarks)(struct intel_atomic_state *state);

};



struct intel_color_funcs {

	int (*color_check)(struct intel_crtc_state *crtc_state);

	/*

	 * Program double buffered color management registers during

	 * vblank evasion. The registers should then latch during the

	 * next vblank start, alongside any other double buffered registers

	 * involved with the same commit.

	 */

	void (*color_commit)(const struct intel_crtc_state *crtc_state);

	/*

	 * Load LUTs (and other single buffered color management

	 * registers). Will (hopefully) be called during the vblank

	 * following the latching of any double buffered registers

	 * involved with the same commit.

	 */

	void (*load_luts)(const struct intel_crtc_state *crtc_state);

	void (*read_luts)(struct intel_crtc_state *crtc_state);

};



struct intel_hotplug_funcs {

	void (*hpd_irq_setup)(struct drm_i915_private *dev_priv);

};