[Intel-gfx] [PATCH] drm/dp: DPCD register defines for link

Pandiyan, Dhinakaran dhinakaran.pandiyan at intel.com
Thu Sep 14 06:18:30 UTC 2017


On Thu, 2017-09-14 at 09:00 +0300, Jani Nikula wrote:
> On Wed, 13 Sep 2017, Dhinakaran Pandiyan <dhinakaran.pandiyan at intel.com> wrote:
> > Link status is available in the ESI field on devices with DPCD r1.2 or
> > higher. DP spec also says "An MST upstream device shall use this field
> > instead of the Link/Sink Device Status field registers, starting from DPCD
> > Address 00200h."
> >
> > Cc: Jani Nikula <jani.nikula at intel.com>
> > Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan at intel.com>
> > ---
> >  include/drm/drm_dp_helper.h | 5 +++++
> >  1 file changed, 5 insertions(+)
> >
> > diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h
> > index 2c412a15cfa1..0bf15525cdd0 100644
> > --- a/include/drm/drm_dp_helper.h
> > +++ b/include/drm/drm_dp_helper.h
> > @@ -738,6 +738,11 @@
> >  #define DP_RECEIVER_ALPM_STATUS		    0x200b  /* eDP 1.4 */
> >  # define DP_ALPM_LOCK_TIMEOUT_ERROR	    (1 << 0)
> >  
> > +#define LANE0_1_STATUS_ESI                  0x200c /* Same as status in 0x202 */
> > +#define LANE2_3_STATUS_ESI                  0x200d /* Same as status in 0x203 */
> > +#define LANE_ALIGN_STATUS_UPDATED_ESI       0x200e /* Same as status in 0x204 */
> > +#define SINK_STATUS_ESI                     0x200f /* Same as status in 0x205 */
> 
> With the macro names prefixed with DP_,
> 

I must be blind to have not seen that. Thanks!

> Reviewed-by: Jani Nikula <jani.nikula at intel.com>
> 
> 
> > +
> >  #define DP_DPRX_FEATURE_ENUMERATION_LIST    0x2210  /* DP 1.3 */
> >  # define DP_GTC_CAP					(1 << 0)  /* DP 1.3 */
> >  # define DP_SST_SPLIT_SDP_CAP				(1 << 1)  /* DP 1.4 */
> 


More information about the dri-devel mailing list