All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paulo Zanoni <paulo.r.zanoni@intel.com>
To: Manasi Navare <manasi.d.navare@intel.com>,
	intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH v3] drm/i915/icl: Add remaining registers and bitfields for MG PHY DDI
Date: Mon, 16 Jul 2018 14:01:01 -0700	[thread overview]
Message-ID: <1531774861.2503.3.camel@intel.com> (raw)
In-Reply-To: <1531510993-6606-1-git-send-email-manasi.d.navare@intel.com>

Em Sex, 2018-07-13 às 12:43 -0700, Manasi Navare escreveu:
> This patch adds the remaining register definitions and bit fields
> required for MG PHy DDI buffer initializations and voltage
> swing programming for MG PHy DDI ports.
> 
> While at it this patch also fixes the naming for previously defined
> MG PHY registers in original commit id (c92f47b5ec977a "drm/i915/icl:
> Add register defs for voltage swing sequences for MG PHY DDI").
> Since the MG PHY registers are first defined in ICL platform, there
> is no need for _ICL prefix.
> 
> v3:
> * Fix register names, add spaces for MASK defines, correct the order
> of #defines (Paulo)
> 
> v2:
> * Change the MG_TX_DRVCTL registers names to match the spec (Anusha)
> 
> Signed-off-by: Manasi Navare <manasi.d.navare@intel.com>
> Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
> Cc: James Ausmus <james.ausmus@intel.com>
> ---
>  drivers/gpu/drm/i915/i915_reg.h | 266 +++++++++++++++++++++++-------
> ----------
>  1 file changed, 155 insertions(+), 111 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_reg.h
> b/drivers/gpu/drm/i915/i915_reg.h
> index c30cfcd..1e4ed68 100644
> --- a/drivers/gpu/drm/i915/i915_reg.h
> +++ b/drivers/gpu/drm/i915/i915_reg.h
> @@ -1872,121 +1872,165 @@ enum i915_power_well_id {
>  #define   N_SCALAR(x)			((x) << 24)
>  #define   N_SCALAR_MASK			(0x7F << 24)
>  
> -#define _ICL_MG_PHY_PORT_LN(port, ln, ln0p1, ln0p2, ln1p1) \
> +#define MG_PHY_PORT_LN(port, ln, ln0p1, ln0p2, ln1p1) \
>  	_MMIO(_PORT((port) - PORT_C, ln0p1, ln0p2) + (ln) * ((ln1p1)
> - (ln0p1)))
>  
> -#define _ICL_MG_TX_LINK_PARAMS_TX1LN0_PORT1		0x16812C
> -#define _ICL_MG_TX_LINK_PARAMS_TX1LN1_PORT1		0x16852C
> -#define _ICL_MG_TX_LINK_PARAMS_TX1LN0_PORT2		0x16912C
> -#define _ICL_MG_TX_LINK_PARAMS_TX1LN1_PORT2		0x16952C
> -#define _ICL_MG_TX_LINK_PARAMS_TX1LN0_PORT3		0x16A12C
> -#define _ICL_MG_TX_LINK_PARAMS_TX1LN1_PORT3		0x16A52C
> -#define _ICL_MG_TX_LINK_PARAMS_TX1LN0_PORT4		0x16B12C
> -#define _ICL_MG_TX_LINK_PARAMS_TX1LN1_PORT4		0x16B52C
> -#define ICL_PORT_MG_TX1_LINK_PARAMS(port, ln) \
> -	_ICL_MG_PHY_PORT_LN(port, ln,
> _ICL_MG_TX_LINK_PARAMS_TX1LN0_PORT1, \
> -				      _ICL_MG_TX_LINK_PARAMS_TX1LN0_
> PORT2, \
> -				      _ICL_MG_TX_LINK_PARAMS_TX1LN1_
> PORT1)
> -
> -#define _ICL_MG_TX_LINK_PARAMS_TX2LN0_PORT1		0x1680AC
> -#define _ICL_MG_TX_LINK_PARAMS_TX2LN1_PORT1		0x1684AC
> -#define _ICL_MG_TX_LINK_PARAMS_TX2LN0_PORT2		0x1690AC
> -#define _ICL_MG_TX_LINK_PARAMS_TX2LN1_PORT2		0x1694AC
> -#define _ICL_MG_TX_LINK_PARAMS_TX2LN0_PORT3		0x16A0AC
> -#define _ICL_MG_TX_LINK_PARAMS_TX2LN1_PORT3		0x16A4AC
> -#define _ICL_MG_TX_LINK_PARAMS_TX2LN0_PORT4		0x16B0AC
> -#define _ICL_MG_TX_LINK_PARAMS_TX2LN1_PORT4		0x16B4AC
> -#define ICL_PORT_MG_TX2_LINK_PARAMS(port, ln) \
> -	_ICL_MG_PHY_PORT_LN(port, ln,
> _ICL_MG_TX_LINK_PARAMS_TX2LN0_PORT1, \
> -				      _ICL_MG_TX_LINK_PARAMS_TX2LN0_
> PORT2, \
> -				      _ICL_MG_TX_LINK_PARAMS_TX2LN1_
> PORT1)
> -#define CRI_USE_FS32			(1 << 5)
> -
> -#define _ICL_MG_TX_PISO_READLOAD_TX1LN0_PORT1		0x16814
> C
> -#define _ICL_MG_TX_PISO_READLOAD_TX1LN1_PORT1		0x16854
> C
> -#define _ICL_MG_TX_PISO_READLOAD_TX1LN0_PORT2		0x16914
> C
> -#define _ICL_MG_TX_PISO_READLOAD_TX1LN1_PORT2		0x16954
> C
> -#define _ICL_MG_TX_PISO_READLOAD_TX1LN0_PORT3		0x16A14
> C
> -#define _ICL_MG_TX_PISO_READLOAD_TX1LN1_PORT3		0x16A54
> C
> -#define _ICL_MG_TX_PISO_READLOAD_TX1LN0_PORT4		0x16B14
> C
> -#define _ICL_MG_TX_PISO_READLOAD_TX1LN1_PORT4		0x16B54
> C
> -#define ICL_PORT_MG_TX1_PISO_READLOAD(port, ln) \
> -	_ICL_MG_PHY_PORT_LN(port, ln,
> _ICL_MG_TX_PISO_READLOAD_TX1LN0_PORT1, \
> -				      _ICL_MG_TX_PISO_READLOAD_TX1LN
> 0_PORT2, \
> -				      _ICL_MG_TX_PISO_READLOAD_TX1LN
> 1_PORT1)
> -
> -#define _ICL_MG_TX_PISO_READLOAD_TX2LN0_PORT1		0x1680C
> C
> -#define _ICL_MG_TX_PISO_READLOAD_TX2LN1_PORT1		0x1684C
> C
> -#define _ICL_MG_TX_PISO_READLOAD_TX2LN0_PORT2		0x1690C
> C
> -#define _ICL_MG_TX_PISO_READLOAD_TX2LN1_PORT2		0x1694C
> C
> -#define _ICL_MG_TX_PISO_READLOAD_TX2LN0_PORT3		0x16A0C
> C
> -#define _ICL_MG_TX_PISO_READLOAD_TX2LN1_PORT3		0x16A4C
> C
> -#define _ICL_MG_TX_PISO_READLOAD_TX2LN0_PORT4		0x16B0C
> C
> -#define _ICL_MG_TX_PISO_READLOAD_TX2LN1_PORT4		0x16B4C
> C
> -#define ICL_PORT_MG_TX2_PISO_READLOAD(port, ln) \
> -	_ICL_MG_PHY_PORT_LN(port, ln,
> _ICL_MG_TX_PISO_READLOAD_TX2LN0_PORT1, \
> -				      _ICL_MG_TX_PISO_READLOAD_TX2LN
> 0_PORT2, \
> -				      _ICL_MG_TX_PISO_READLOAD_TX2LN
> 1_PORT1)
> +#define MG_TX_LINK_PARAMS_TX1LN0_PORT1		0x16812C
> +#define MG_TX_LINK_PARAMS_TX1LN1_PORT1		0x16852C
> +#define MG_TX_LINK_PARAMS_TX1LN0_PORT2		0x16912C
> +#define MG_TX_LINK_PARAMS_TX1LN1_PORT2		0x16952C
> +#define MG_TX_LINK_PARAMS_TX1LN0_PORT3		0x16A12C
> +#define MG_TX_LINK_PARAMS_TX1LN1_PORT3		0x16A52C
> +#define MG_TX_LINK_PARAMS_TX1LN0_PORT4		0x16B12C
> +#define MG_TX_LINK_PARAMS_TX1LN1_PORT4		0x16B52C
> +#define MG_TX1_LINK_PARAMS(port, ln) \
> +	MG_PHY_PORT_LN(port, ln, MG_TX_LINK_PARAMS_TX1LN0_PORT1, \
> +				 MG_TX_LINK_PARAMS_TX1LN0_PORT2, \
> +				 MG_TX_LINK_PARAMS_TX1LN1_PORT1)
> +
> +#define MG_TX_LINK_PARAMS_TX2LN0_PORT1		0x1680AC
> +#define MG_TX_LINK_PARAMS_TX2LN1_PORT1		0x1684AC
> +#define MG_TX_LINK_PARAMS_TX2LN0_PORT2		0x1690AC
> +#define MG_TX_LINK_PARAMS_TX2LN1_PORT2		0x1694AC
> +#define MG_TX_LINK_PARAMS_TX2LN0_PORT3		0x16A0AC
> +#define MG_TX_LINK_PARAMS_TX2LN1_PORT3		0x16A4AC
> +#define MG_TX_LINK_PARAMS_TX2LN0_PORT4		0x16B0AC
> +#define MG_TX_LINK_PARAMS_TX2LN1_PORT4		0x16B4AC
> +#define MG_TX2_LINK_PARAMS(port, ln) \
> +	MG_PHY_PORT_LN(port, ln, MG_TX_LINK_PARAMS_TX2LN0_PORT1, \
> +				 MG_TX_LINK_PARAMS_TX2LN0_PORT2, \
> +				 MG_TX_LINK_PARAMS_TX2LN1_PORT1)
> +#define   CRI_USE_FS32			(1 << 5)
> +
> +#define MG_TX_PISO_READLOAD_TX1LN0_PORT1		0x16814C
> +#define MG_TX_PISO_READLOAD_TX1LN1_PORT1		0x16854C
> +#define MG_TX_PISO_READLOAD_TX1LN0_PORT2		0x16914C
> +#define MG_TX_PISO_READLOAD_TX1LN1_PORT2		0x16954C
> +#define MG_TX_PISO_READLOAD_TX1LN0_PORT3		0x16A14C
> +#define MG_TX_PISO_READLOAD_TX1LN1_PORT3		0x16A54C
> +#define MG_TX_PISO_READLOAD_TX1LN0_PORT4		0x16B14C
> +#define MG_TX_PISO_READLOAD_TX1LN1_PORT4		0x16B54C
> +#define MG_TX1_PISO_READLOAD(port, ln) \
> +	MG_PHY_PORT_LN(port, ln, MG_TX_PISO_READLOAD_TX1LN0_PORT1, \
> +				 MG_TX_PISO_READLOAD_TX1LN0_PORT2, \
> +				 MG_TX_PISO_READLOAD_TX1LN1_PORT1)
> +
> +#define MG_TX_PISO_READLOAD_TX2LN0_PORT1		0x1680CC
> +#define MG_TX_PISO_READLOAD_TX2LN1_PORT1		0x1684CC
> +#define MG_TX_PISO_READLOAD_TX2LN0_PORT2		0x1690CC
> +#define MG_TX_PISO_READLOAD_TX2LN1_PORT2		0x1694CC
> +#define MG_TX_PISO_READLOAD_TX2LN0_PORT3		0x16A0CC
> +#define MG_TX_PISO_READLOAD_TX2LN1_PORT3		0x16A4CC
> +#define MG_TX_PISO_READLOAD_TX2LN0_PORT4		0x16B0CC
> +#define MG_TX_PISO_READLOAD_TX2LN1_PORT4		0x16B4CC
> +#define MG_TX2_PISO_READLOAD(port, ln) \
> +	MG_PHY_PORT_LN(port, ln, MG_TX_PISO_READLOAD_TX2LN0_PORT1, \
> +				 MG_TX_PISO_READLOAD_TX2LN0_PORT2, \
> +				 MG_TX_PISO_READLOAD_TX2LN1_PORT1)
>  #define CRI_CALCINIT					(1 << 1)

You missed the extra spaces on this one.

With that:
Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com>

>  
> -#define _ICL_MG_TX_SWINGCTRL_TX1LN0_PORT1		0x168148
> -#define _ICL_MG_TX_SWINGCTRL_TX1LN1_PORT1		0x168548
> -#define _ICL_MG_TX_SWINGCTRL_TX1LN0_PORT2		0x169148
> -#define _ICL_MG_TX_SWINGCTRL_TX1LN1_PORT2		0x169548
> -#define _ICL_MG_TX_SWINGCTRL_TX1LN0_PORT3		0x16A148
> -#define _ICL_MG_TX_SWINGCTRL_TX1LN1_PORT3		0x16A548
> -#define _ICL_MG_TX_SWINGCTRL_TX1LN0_PORT4		0x16B148
> -#define _ICL_MG_TX_SWINGCTRL_TX1LN1_PORT4		0x16B548
> -#define ICL_PORT_MG_TX1_SWINGCTRL(port, ln) \
> -	_ICL_MG_PHY_PORT_LN(port, ln,
> _ICL_MG_TX_SWINGCTRL_TX1LN0_PORT1, \
> -				      _ICL_MG_TX_SWINGCTRL_TX1LN0_PO
> RT2, \
> -				      _ICL_MG_TX_SWINGCTRL_TX1LN1_PO
> RT1)
> -
> -#define _ICL_MG_TX_SWINGCTRL_TX2LN0_PORT1		0x1680C8
> -#define _ICL_MG_TX_SWINGCTRL_TX2LN1_PORT1		0x1684C8
> -#define _ICL_MG_TX_SWINGCTRL_TX2LN0_PORT2		0x1690C8
> -#define _ICL_MG_TX_SWINGCTRL_TX2LN1_PORT2		0x1694C8
> -#define _ICL_MG_TX_SWINGCTRL_TX2LN0_PORT3		0x16A0C8
> -#define _ICL_MG_TX_SWINGCTRL_TX2LN1_PORT3		0x16A4C8
> -#define _ICL_MG_TX_SWINGCTRL_TX2LN0_PORT4		0x16B0C8
> -#define _ICL_MG_TX_SWINGCTRL_TX2LN1_PORT4		0x16B4C8
> -#define ICL_PORT_MG_TX2_SWINGCTRL(port, ln) \
> -	_ICL_MG_PHY_PORT_LN(port, ln,
> _ICL_MG_TX_SWINGCTRL_TX2LN0_PORT1, \
> -				      _ICL_MG_TX_SWINGCTRL_TX2LN0_PO
> RT2, \
> -				      _ICL_MG_TX_SWINGCTRL_TX2LN1_PO
> RT1)
> -#define CRI_TXDEEMPH_OVERRIDE_17_12(x)			((x)
> << 0)
> -#define CRI_TXDEEMPH_OVERRIDE_17_12_MASK		(0x3F << 0)
> -
> -#define _ICL_MG_TX_DRVCTRL_TX1LN0_PORT1			0x168
> 144
> -#define _ICL_MG_TX_DRVCTRL_TX1LN1_PORT1			0x168
> 544
> -#define _ICL_MG_TX_DRVCTRL_TX1LN0_PORT2			0x169
> 144
> -#define _ICL_MG_TX_DRVCTRL_TX1LN1_PORT2			0x169
> 544
> -#define _ICL_MG_TX_DRVCTRL_TX1LN0_PORT3			0x16A
> 144
> -#define _ICL_MG_TX_DRVCTRL_TX1LN1_PORT3			0x16A
> 544
> -#define _ICL_MG_TX_DRVCTRL_TX1LN0_PORT4			0x16B
> 144
> -#define _ICL_MG_TX_DRVCTRL_TX1LN1_PORT4			0x16B
> 544
> -#define ICL_PORT_MG_TX1_DRVCTRL(port, ln) \
> -	_ICL_MG_PHY_PORT_LN(port, ln,
> _ICL_MG_TX_DRVCTRL_TX1LN0_PORT1, \
> -				      _ICL_MG_TX_DRVCTRL_TX1LN0_PORT
> 2, \
> -				      _ICL_MG_TX_DRVCTRL_TX1LN1_PORT
> 1)
> -
> -#define _ICL_MG_TX_DRVCTRL_TX2LN0_PORT1			0x168
> 0C4
> -#define _ICL_MG_TX_DRVCTRL_TX2LN1_PORT1			0x168
> 4C4
> -#define _ICL_MG_TX_DRVCTRL_TX2LN0_PORT2			0x169
> 0C4
> -#define _ICL_MG_TX_DRVCTRL_TX2LN1_PORT2			0x169
> 4C4
> -#define _ICL_MG_TX_DRVCTRL_TX2LN0_PORT3			0x16A
> 0C4
> -#define _ICL_MG_TX_DRVCTRL_TX2LN1_PORT3			0x16A
> 4C4
> -#define _ICL_MG_TX_DRVCTRL_TX2LN0_PORT4			0x16B
> 0C4
> -#define _ICL_MG_TX_DRVCTRL_TX2LN1_PORT4			0x16B
> 4C4
> -#define ICL_PORT_MG_TX2_DRVCTRL(port, ln) \
> -	_ICL_MG_PHY_PORT_LN(port, ln,
> _ICL_MG_TX_DRVCTRL_TX2LN0_PORT1, \
> -				      _ICL_MG_TX_DRVCTRL_TX2LN0_PORT
> 2, \
> -				      _ICL_MG_TX_DRVCTRL_TX2LN1_PORT
> 1)
> -#define CRI_TXDEEMPH_OVERRIDE_11_6(x)			((x) <<
> 24)
> -#define CRI_TXDEEMPH_OVERRIDE_11_6_MASK			(0x3F
> << 24)
> -#define CRI_TXDEEMPH_OVERRIDE_EN			(1 << 22)
> -#define CRI_TXDEEMPH_OVERRIDE_5_0(x)			((x) <<
> 16)
> -#define CRI_TXDEEMPH_OVERRIDE_5_0_MASK			(0x3F
> << 16)
> +#define MG_TX_SWINGCTRL_TX1LN0_PORT1		0x168148
> +#define MG_TX_SWINGCTRL_TX1LN1_PORT1		0x168548
> +#define MG_TX_SWINGCTRL_TX1LN0_PORT2		0x169148
> +#define MG_TX_SWINGCTRL_TX1LN1_PORT2		0x169548
> +#define MG_TX_SWINGCTRL_TX1LN0_PORT3		0x16A148
> +#define MG_TX_SWINGCTRL_TX1LN1_PORT3		0x16A548
> +#define MG_TX_SWINGCTRL_TX1LN0_PORT4		0x16B148
> +#define MG_TX_SWINGCTRL_TX1LN1_PORT4		0x16B548
> +#define MG_TX1_SWINGCTRL(port, ln) \
> +	MG_PHY_PORT_LN(port, ln, MG_TX_SWINGCTRL_TX1LN0_PORT1, \
> +				 MG_TX_SWINGCTRL_TX1LN0_PORT2, \
> +				 MG_TX_SWINGCTRL_TX1LN1_PORT1)
> +
> +#define MG_TX_SWINGCTRL_TX2LN0_PORT1		0x1680C8
> +#define MG_TX_SWINGCTRL_TX2LN1_PORT1		0x1684C8
> +#define MG_TX_SWINGCTRL_TX2LN0_PORT2		0x1690C8
> +#define MG_TX_SWINGCTRL_TX2LN1_PORT2		0x1694C8
> +#define MG_TX_SWINGCTRL_TX2LN0_PORT3		0x16A0C8
> +#define MG_TX_SWINGCTRL_TX2LN1_PORT3		0x16A4C8
> +#define MG_TX_SWINGCTRL_TX2LN0_PORT4		0x16B0C8
> +#define MG_TX_SWINGCTRL_TX2LN1_PORT4		0x16B4C8
> +#define MG_TX2_SWINGCTRL(port, ln) \
> +	MG_PHY_PORT_LN(port, ln, MG_TX_SWINGCTRL_TX2LN0_PORT1, \
> +				 MG_TX_SWINGCTRL_TX2LN0_PORT2, \
> +				 MG_TX_SWINGCTRL_TX2LN1_PORT1)
> +#define   CRI_TXDEEMPH_OVERRIDE_17_12(x)		((x) << 0)
> +#define   CRI_TXDEEMPH_OVERRIDE_17_12_MASK		(0x3F <<
> 0)
> +
> +#define MG_TX_DRVCTRL_TX1LN0_TXPORT1			0x168144
> +#define MG_TX_DRVCTRL_TX1LN1_TXPORT1			0x168544
> +#define MG_TX_DRVCTRL_TX1LN0_TXPORT2			0x169144
> +#define MG_TX_DRVCTRL_TX1LN1_TXPORT2			0x169544
> +#define MG_TX_DRVCTRL_TX1LN0_TXPORT3			0x16A144
> +#define MG_TX_DRVCTRL_TX1LN1_TXPORT3			0x16A544
> +#define MG_TX_DRVCTRL_TX1LN0_TXPORT4			0x16B144
> +#define MG_TX_DRVCTRL_TX1LN1_TXPORT4			0x16B544
> +#define MG_TX1_DRVCTRL(port, ln) \
> +	MG_PHY_PORT_LN(port, ln, MG_TX_DRVCTRL_TX1LN0_TXPORT1, \
> +				 MG_TX_DRVCTRL_TX1LN0_TXPORT2, \
> +				 MG_TX_DRVCTRL_TX1LN1_TXPORT1)
> +
> +#define MG_TX_DRVCTRL_TX2LN0_PORT1			0x1680C4
> +#define MG_TX_DRVCTRL_TX2LN1_PORT1			0x1684C4
> +#define MG_TX_DRVCTRL_TX2LN0_PORT2			0x1690C4
> +#define MG_TX_DRVCTRL_TX2LN1_PORT2			0x1694C4
> +#define MG_TX_DRVCTRL_TX2LN0_PORT3			0x16A0C4
> +#define MG_TX_DRVCTRL_TX2LN1_PORT3			0x16A4C4
> +#define MG_TX_DRVCTRL_TX2LN0_PORT4			0x16B0C4
> +#define MG_TX_DRVCTRL_TX2LN1_PORT4			0x16B4C4
> +#define MG_TX2_DRVCTRL(port, ln) \
> +	MG_PHY_PORT_LN(port, ln, MG_TX_DRVCTRL_TX2LN0_PORT1, \
> +				 MG_TX_DRVCTRL_TX2LN0_PORT2, \
> +				 MG_TX_DRVCTRL_TX2LN1_PORT1)
> +#define   CRI_TXDEEMPH_OVERRIDE_11_6(x)			((x)
> << 24)
> +#define   CRI_TXDEEMPH_OVERRIDE_11_6_MASK		(0x3F <<
> 24)
> +#define   CRI_TXDEEMPH_OVERRIDE_EN			(1 << 22)
> +#define   CRI_TXDEEMPH_OVERRIDE_5_0(x)			((x)
> << 16)
> +#define   CRI_TXDEEMPH_OVERRIDE_5_0_MASK		(0x3F << 16)
> +#define   CRI_LOADGEN_SEL(x)				((x) <<
> 12)
> +#define   CRI_LOADGEN_SEL_MASK				(0x3
> << 12)
> +
> +#define MG_CLKHUB_LN0_PORT1			0x16839C
> +#define MG_CLKHUB_LN1_PORT1			0x16879C
> +#define MG_CLKHUB_LN0_PORT2			0x16939C
> +#define MG_CLKHUB_LN1_PORT2			0x16979C
> +#define MG_CLKHUB_LN0_PORT3			0x16A39C
> +#define MG_CLKHUB_LN1_PORT3			0x16A79C
> +#define MG_CLKHUB_LN0_PORT4			0x16B39C
> +#define MG_CLKHUB_LN1_PORT4			0x16B79C
> +#define MG_CLKHUB(port, ln) \
> +	MG_PHY_PORT_LN(port, ln, MG_CLKHUB_LN0_PORT1, \
> +				 MG_CLKHUB_LN0_PORT2, \
> +				 MG_CLKHUB_LN1_PORT1)
> +#define   CFG_LOW_RATE_LKREN_EN				(1 <<
> 11)
> +
> +#define MG_TX_DCC_TX1LN0_PORT1			0x168110
> +#define MG_TX_DCC_TX1LN1_PORT1			0x168510
> +#define MG_TX_DCC_TX1LN0_PORT2			0x169110
> +#define MG_TX_DCC_TX1LN1_PORT2			0x169510
> +#define MG_TX_DCC_TX1LN0_PORT3			0x16A110
> +#define MG_TX_DCC_TX1LN1_PORT3			0x16A510
> +#define MG_TX_DCC_TX1LN0_PORT4			0x16B110
> +#define MG_TX_DCC_TX1LN1_PORT4			0x16B510
> +#define MG_TX1_DCC(port, ln) \
> +	MG_PHY_PORT_LN(port, ln, MG_TX_DCC_TX1LN0_PORT1, \
> +				 MG_TX_DCC_TX1LN0_PORT2, \
> +				 MG_TX_DCC_TX1LN1_PORT1)
> +#define MG_TX_DCC_TX2LN0_PORT1			0x168090
> +#define MG_TX_DCC_TX2LN1_PORT1			0x168490
> +#define MG_TX_DCC_TX2LN0_PORT2			0x169090
> +#define MG_TX_DCC_TX2LN1_PORT2			0x169490
> +#define MG_TX_DCC_TX2LN0_PORT3			0x16A090
> +#define MG_TX_DCC_TX2LN1_PORT3			0x16A490
> +#define MG_TX_DCC_TX2LN0_PORT4			0x16B090
> +#define MG_TX_DCC_TX2LN1_PORT4			0x16B490
> +#define MG_TX2_DCC(port, ln) \
> +	MG_PHY_PORT_LN(port, ln, MG_TX_DCC_TX2LN0_PORT1, \
> +				 MG_TX_DCC_TX2LN0_PORT2, \
> +				 MG_TX_DCC_TX2LN1_PORT1)
> +#define   CFG_AMI_CK_DIV_OVERRIDE_VAL(x)	((x) << 25)
> +#define   CFG_AMI_CK_DIV_OVERRIDE_VAL_MASK	(0x3 << 25)
> +#define   CFG_AMI_CK_DIV_OVERRIDE_EN		(1 << 24)
>  
>  /* The spec defines this only for BXT PHY0, but lets assume that
> this
>   * would exist for PHY1 too if it had a second channel.
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

  reply	other threads:[~2018-07-16 21:01 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-28 22:35 [PATCH v2 1/2] drm/i915/icl: Add remaining registers and bitfields for MG PHY DDI Manasi Navare
2018-06-28 22:35 ` [PATCH v2 2/2] drm/i915/icl: Implement voltage swing programming sequence " Manasi Navare
2018-07-16 23:48   ` Paulo Zanoni
2018-06-28 23:26 ` ✗ Fi.CI.CHECKPATCH: warning for series starting with [v2,1/2] drm/i915/icl: Add remaining registers and bitfields " Patchwork
2018-06-28 23:42 ` ✓ Fi.CI.BAT: success " Patchwork
2018-06-29  5:46 ` ✓ Fi.CI.IGT: " Patchwork
2018-06-29 20:47 ` [PATCH v2 1/2] " Srivatsa, Anusha
2018-06-29 21:33 ` ✗ Fi.CI.BAT: failure for series starting with [v2,1/2] drm/i915/icl: Add remaining registers and bitfields for MG PHY DDI (rev2) Patchwork
2018-07-03 18:07 ` [PATCH v2 1/2] drm/i915/icl: Add remaining registers and bitfields for MG PHY DDI Srivatsa, Anusha
2018-07-13  0:00 ` Paulo Zanoni
2018-07-13 18:44   ` Manasi Navare
2018-07-13 19:43 ` [PATCH v3] " Manasi Navare
2018-07-16 21:01   ` Paulo Zanoni [this message]
2018-07-13 19:50 ` ✗ Fi.CI.CHECKPATCH: warning for series starting with [v3] drm/i915/icl: Add remaining registers and bitfields for MG PHY DDI (rev3) Patchwork
2018-07-13 20:06 ` ✓ Fi.CI.BAT: success " Patchwork

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=1531774861.2503.3.camel@intel.com \
    --to=paulo.r.zanoni@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=manasi.d.navare@intel.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.