Commit eee21566 authored by Imre Deak's avatar Imre Deak Committed by Daniel Vetter

drm/i915/bxt: add description about the BXT PHYs

Extend the VLV/CHV DPIO (PHY) documentation with the BXT specifics.

v2:
- add more detail about the mapping between ports and transcoders (ville)
Signed-off-by: default avatarImre Deak <imre.deak@intel.com>
Reviewed-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent 5c6706e5
...@@ -4067,7 +4067,7 @@ int num_ioctls;</synopsis> ...@@ -4067,7 +4067,7 @@ int num_ioctls;</synopsis>
<title>DPIO</title> <title>DPIO</title>
!Pdrivers/gpu/drm/i915/i915_reg.h DPIO !Pdrivers/gpu/drm/i915/i915_reg.h DPIO
<table id="dpiox2"> <table id="dpiox2">
<title>Dual channel PHY (VLV/CHV)</title> <title>Dual channel PHY (VLV/CHV/BXT)</title>
<tgroup cols="8"> <tgroup cols="8">
<colspec colname="c0" /> <colspec colname="c0" />
<colspec colname="c1" /> <colspec colname="c1" />
...@@ -4118,7 +4118,7 @@ int num_ioctls;</synopsis> ...@@ -4118,7 +4118,7 @@ int num_ioctls;</synopsis>
</tgroup> </tgroup>
</table> </table>
<table id="dpiox1"> <table id="dpiox1">
<title>Single channel PHY (CHV)</title> <title>Single channel PHY (CHV/BXT)</title>
<tgroup cols="4"> <tgroup cols="4">
<colspec colname="c0" /> <colspec colname="c0" />
<colspec colname="c1" /> <colspec colname="c1" />
......
...@@ -715,7 +715,7 @@ enum skl_disp_power_wells { ...@@ -715,7 +715,7 @@ enum skl_disp_power_wells {
/** /**
* DOC: DPIO * DOC: DPIO
* *
* VLV and CHV have slightly peculiar display PHYs for driving DP/HDMI * VLV, CHV and BXT have slightly peculiar display PHYs for driving DP/HDMI
* ports. DPIO is the name given to such a display PHY. These PHYs * ports. DPIO is the name given to such a display PHY. These PHYs
* don't follow the standard programming model using direct MMIO * don't follow the standard programming model using direct MMIO
* registers, and instead their registers must be accessed trough IOSF * registers, and instead their registers must be accessed trough IOSF
...@@ -746,7 +746,7 @@ enum skl_disp_power_wells { ...@@ -746,7 +746,7 @@ enum skl_disp_power_wells {
* controlled from the display controller side. No DPIO registers * controlled from the display controller side. No DPIO registers
* need to be accessed during AUX communication, * need to be accessed during AUX communication,
* *
* Generally the common lane corresponds to the pipe and * Generally on VLV/CHV the common lane corresponds to the pipe and
* the spline (PCS/TX) corresponds to the port. * the spline (PCS/TX) corresponds to the port.
* *
* For dual channel PHY (VLV/CHV): * For dual channel PHY (VLV/CHV):
...@@ -768,11 +768,17 @@ enum skl_disp_power_wells { ...@@ -768,11 +768,17 @@ enum skl_disp_power_wells {
* *
* port D == PCS/TX CH0 * port D == PCS/TX CH0
* *
* Note: digital port B is DDI0, digital port C is DDI1, * On BXT the entire PHY channel corresponds to the port. That means
* digital port D is DDI2 * the PLL is also now associated with the port rather than the pipe,
* and so the clock needs to be routed to the appropriate transcoder.
* Port A PLL is directly connected to transcoder EDP and port B/C
* PLLs can be routed to any transcoder A/B/C.
*
* Note: DDI0 is digital port B, DD1 is digital port C, and DDI2 is
* digital port D (CHV) or port A (BXT).
*/ */
/* /*
* Dual channel PHY (VLV/CHV) * Dual channel PHY (VLV/CHV/BXT)
* --------------------------------- * ---------------------------------
* | CH0 | CH1 | * | CH0 | CH1 |
* | CMN/PLL/REF | CMN/PLL/REF | * | CMN/PLL/REF | CMN/PLL/REF |
...@@ -784,7 +790,7 @@ enum skl_disp_power_wells { ...@@ -784,7 +790,7 @@ enum skl_disp_power_wells {
* | DDI0 | DDI1 | DP/HDMI ports * | DDI0 | DDI1 | DP/HDMI ports
* --------------------------------- * ---------------------------------
* *
* Single channel PHY (CHV) * Single channel PHY (CHV/BXT)
* ----------------- * -----------------
* | CH0 | * | CH0 |
* | CMN/PLL/REF | * | CMN/PLL/REF |
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment