Hi See below: 2012/9/18 Adam Jackson <ajax at redhat.com>: > Sources: DP, eDP, and DP interop specs, and a VESA slideshow about DP > 1.2 for the MST bits. All I needed to review every bit was DP spec version 1.2. > > Tested-by: Takashi Iwai <tiwai at suse.de> > Signed-off-by: Adam Jackson <ajax at redhat.com> > --- > include/drm/drm_dp_helper.h | 60 ++++++++++++++++++++++++++++++++++++++++--- > 1 files changed, 56 insertions(+), 4 deletions(-) > > diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h > index 1744b18c..f9888c3 100644 > --- a/include/drm/drm_dp_helper.h > +++ b/include/drm/drm_dp_helper.h > @@ -69,16 +69,30 @@ > /* 10b = TMDS or HDMI */ > /* 11b = Other */ > # define DP_FORMAT_CONVERSION (1 << 3) > +# define DP_DETAILED_CAP_INFO_AVAILABLE (1 << 4) > > #define DP_MAIN_LINK_CHANNEL_CODING 0x006 > > #define DP_DOWN_STREAM_PORT_COUNT 0x007 > -#define DP_PORT_COUNT_MASK 0x0f > -#define DP_OUI_SUPPORT (1 << 7) > +# define DP_PORT_COUNT_MASK 0x0f > +# define DP_MSA_TIMING_PAR_IGNORED (1 << 6) > +# define DP_OUI_SUPPORT (1 << 7) > + > +#define DP_I2C_SPEED_CAP 0x00c > +# define DP_I2C_SPEED_1K 0x01 > +# define DP_I2C_SPEED_5K 0x02 > +# define DP_I2C_SPEED_10K 0x04 > +# define DP_I2C_SPEED_100K 0x08 > +# define DP_I2C_SPEED_400K 0x10 > +# define DP_I2C_SPEED_1M 0x20 > > #define DP_EDP_CONFIGURATION_CAP 0x00d > #define DP_TRAINING_AUX_RD_INTERVAL 0x00e > > +/* Multiple stream transport */ > +#define DP_MSTM_CAP 0x021 > +# define DP_MST_CAP (1 << 0) > + > #define DP_PSR_SUPPORT 0x070 > # define DP_PSR_IS_SUPPORTED 1 > #define DP_PSR_CAPS 0x071 > @@ -93,6 +107,31 @@ > # define DP_PSR_SETUP_TIME_MASK (7 << 1) > # define DP_PSR_SETUP_TIME_SHIFT 1 > > +/* > + * 0x80-0x8f describe downstream port capabilities, but there are two layouts > + * based on whether DP_DETAILED_CAP_INFO_AVAILABLE was set. If it was not, > + * each port's descriptor is one byte wide. If it was set, each port's is > + * four bytes wide, starting with the one byte from the base info. As of > + * DP interop v1.1a only VGA defines additional detail. > + */ > + > +/* offset 0 */ > +#define DP_DOWNSTREAM_PORT_0 0x80 > +# define DP_DS_PORT_TYPE_MASK (7 << 0) > +# define DP_DS_PORT_TYPE_DP 0 > +# define DP_DS_PORT_TYPE_VGA 1 > +# define DP_DS_PORT_TYPE_DVI 2 > +# define DP_DS_PORT_TYPE_HDMI 3 > +# define DP_DS_PORT_TYPE_NON_EDID 4 > +# define DP_DS_PORT_HPD (1 << 3) > +/* offset 1 for VGA is maximum megapixels per second / 8 */ > +/* offset 2 */ > +# define DP_DS_VGA_MAX_BPC_MASK (3 << 0) > +# define DP_DS_VGA_8BPC 0 > +# define DP_DS_VGA_10BPC 1 > +# define DP_DS_VGA_12BPC 2 > +# define DP_DS_VGA_16BPC 3 > + > /* link configuration */ > #define DP_LINK_BW_SET 0x100 > # define DP_LINK_BW_1_62 0x06 > @@ -148,24 +187,37 @@ > > #define DP_DOWNSPREAD_CTRL 0x107 > # define DP_SPREAD_AMP_0_5 (1 << 4) > +# define DP_MSA_TIMING_PAR_IGNORE_EN (1 << 7) > > #define DP_MAIN_LINK_CHANNEL_CODING_SET 0x108 > # define DP_SET_ANSI_8B10B (1 << 0) > > +#define DP_I2C_SPEED_CONTROL_STATUS 0x109 > +/* bitmask as for DP_I2C_SPEED_CAP */ > + > +#define DP_EDP_CONFIGURATION_SET 0x10a > + > +#define DP_MSTM_CTRL 0x111 > +# define DP_MST_EN (1 << 0) > +# define DP_UP_REQ_EN (1 << 1) > +# define DP_UPSTREAM_IS_SRC (1 << 2) > + > #define DP_PSR_EN_CFG 0x170 > # define DP_PSR_ENABLE (1 << 0) > # define DP_PSR_MAIN_LINK_ACTIVE (1 << 1) > # define DP_PSR_CRC_VERIFICATION (1 << 2) > # define DP_PSR_FRAME_CAPTURE (1 << 3) > > +#define DP_SINK_COUNT 0x200 > +# define DP_SINK_COUNT_MASK (31 << 0) My DP spec version 1.2 says "bits 7 and 5:0", but the DP 1.1 spec says it's just 5:0 and "Bits 7 = RESERVED". So should we treat bit 7 as the most-significant-bit? Notice that this will affect patch 4 of this series. Idea for a follow-up patch: maybe we should try to add some comments explaining which bits appeared only in some specific DPCD x.y revision? With the "bit 7" problem fixed somehow: Reviewed-by: Paulo Zanoni <paulo.r.zanoni at intel.com> > +# define DP_SINK_CP_READY (1 << 6) > + > #define DP_DEVICE_SERVICE_IRQ_VECTOR 0x201 > # define DP_REMOTE_CONTROL_COMMAND_PENDING (1 << 0) > # define DP_AUTOMATED_TEST_REQUEST (1 << 1) > # define DP_CP_IRQ (1 << 2) > # define DP_SINK_SPECIFIC_IRQ (1 << 6) > > -#define DP_EDP_CONFIGURATION_SET 0x10a > - > #define DP_LANE0_1_STATUS 0x202 > #define DP_LANE2_3_STATUS 0x203 > # define DP_LANE_CR_DONE (1 << 0) > -- > 1.7.7.6 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx at lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Paulo Zanoni
