[PATCH v1 5/5] drm/i915/gvt: Refine port select logic for CFL platform

Zhenyu Wang zhenyuw at linux.intel.com
Mon Jan 7 03:19:33 UTC 2019


On 2019.01.04 03:05:30 +0800, fred gao wrote:
> Refine the code since the port select definition for CFL is different
> than SKL/BXT.
> 
> Signed-off-by: fred gao <fred.gao at intel.com>
> ---
>  drivers/gpu/drm/i915/gvt/edid.c | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/gvt/edid.c b/drivers/gpu/drm/i915/gvt/edid.c
> index 752aa0fd1cc9..689961d1ce3a 100644
> --- a/drivers/gpu/drm/i915/gvt/edid.c
> +++ b/drivers/gpu/drm/i915/gvt/edid.c
> @@ -77,6 +77,22 @@ static unsigned char edid_get_byte(struct intel_vgpu *vgpu)
>  	return chr;
>  }
>  
> +static inline int cnp_get_port_from_gmbus0(u32 gmbus0)
> +{
> +	int port_select = gmbus0 & _GMBUS_PIN_SEL_MASK;
> +	int port = -EINVAL;
> +
> +	if (port_select == GMBUS_PIN_1_BXT)
> +		port = PORT_B;
> +	else if (port_select == GMBUS_PIN_2_BXT)
> +		port = PORT_C;
> +	else if (port_select == GMBUS_PIN_3_BXT)
> +		port = PORT_D;
> +	else if (port_select == GMBUS_PIN_4_CNP)
> +		port = PORT_E;
> +	return port;
> +}
> +
>  static inline int bxt_get_port_from_gmbus0(u32 gmbus0)
>  {
>  	int port_select = gmbus0 & _GMBUS_PIN_SEL_MASK;
> @@ -133,6 +149,8 @@ static int gmbus0_mmio_write(struct intel_vgpu *vgpu,
>  
>  	if (IS_BROXTON(dev_priv))
>  		port = bxt_get_port_from_gmbus0(pin_select);
> +	else if (HAS_PCH_CNP(dev_priv))
> +		port = cnp_get_port_from_gmbus0(pin_select);
>  	else
>  		port = get_port_from_gmbus0(pin_select);
>  	if (WARN_ON(port < 0))

I think we need more explanation here, is PCH_CNP the only
possible PCH for CFL? Or this is just to say for CNP we should
do like that?

I don't know why we would still adjust something according
with hw config for our virtual display, maybe there's still
places to take wrong assumption...

-- 
Open Source Technology Center, Intel ltd.

$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/intel-gvt-dev/attachments/20190107/0d1a7e78/attachment.sig>


More information about the intel-gvt-dev mailing list