vlv eDP BIOS Compatiblity to EMGD generated BIOS
diff mbox

Message ID 1434622631-19512-1-git-send-email-lampersperger.andreas@heidenhain.de
State New
Headers show

Commit Message

Lampersperger Andreas June 18, 2015, 10:17 a.m. UTC
When the i915.ko identify an eDP output on a valleyview
board, it should be more slackly. The reason for that is,
that BIOS DATA TABLES generated with intel BMP (Binary
Modification Program) do not set bits for NOT_HDMI or
DIGITAL_OUTPUT on the device type. Due to Adolfo
Sanchez from Intel EMGD, this is not possible.
To solve this problem and enable i915.ko on embedded
vlv boards with eDP, we ignore this two bits.
---
 drivers/gpu/drm/i915/intel_bios.h | 10 ++++++++++
 drivers/gpu/drm/i915/intel_dp.c   | 10 ++++++++--
 2 files changed, 18 insertions(+), 2 deletions(-)

Comments

Daniel Vetter June 22, 2015, 1:43 p.m. UTC | #1
On Thu, Jun 18, 2015 at 12:17:11PM +0200, Andreas Lampersperger wrote:
> When the i915.ko identify an eDP output on a valleyview
> board, it should be more slackly. The reason for that is,
> that BIOS DATA TABLES generated with intel BMP (Binary
> Modification Program) do not set bits for NOT_HDMI or
> DIGITAL_OUTPUT on the device type. Due to Adolfo
> Sanchez from Intel EMGD, this is not possible.
> To solve this problem and enable i915.ko on embedded
> vlv boards with eDP, we ignore this two bits.

s-o-b missing, see Documentation/SubmittingPatches.
-Daniel

> ---
>  drivers/gpu/drm/i915/intel_bios.h | 10 ++++++++++
>  drivers/gpu/drm/i915/intel_dp.c   | 10 ++++++++--
>  2 files changed, 18 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_bios.h b/drivers/gpu/drm/i915/intel_bios.h
> index af0b476..c42161f 100644
> --- a/drivers/gpu/drm/i915/intel_bios.h
> +++ b/drivers/gpu/drm/i915/intel_bios.h
> @@ -742,6 +742,16 @@ int intel_parse_bios(struct drm_device *dev);
>  	 DEVICE_TYPE_DIGITAL_OUTPUT | \
>  	 DEVICE_TYPE_ANALOG_OUTPUT)
>  
> +/*
> + * We dont look on DEVICE_TYPE_NOT_HDMI_OUTPUT an DEVICE_TYPE_DIGITAL_OUTPUT
> + * on valleyview, because intels BMP-generated BIOS don't sets these 
> + * BITS for eDP ports
> + */ 
> +#define DEVICE_TYPE_eDP_BITS_VLV \
> +	(DEVICE_TYPE_eDP_BITS &\
> +	 (~ DEVICE_TYPE_NOT_HDMI_OUTPUT ) &\
> +	 (~ DEVICE_TYPE_DIGITAL_OUTPUT ) )
> +	   
>  /* define the DVO port for HDMI output type */
>  #define		DVO_B		1
>  #define		DVO_C		2
> diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
> index f52eef1..51c753f 100644
> --- a/drivers/gpu/drm/i915/intel_dp.c
> +++ b/drivers/gpu/drm/i915/intel_dp.c
> @@ -5036,6 +5036,7 @@ bool intel_dp_is_edp(struct drm_device *dev, enum port port)
>  	struct drm_i915_private *dev_priv = dev->dev_private;
>  	union child_device_config *p_child;
>  	int i;
> +	u16 eDP_bits;
>  	static const short port_mapping[] = {
>  		[PORT_B] = PORT_IDPB,
>  		[PORT_C] = PORT_IDPC,
> @@ -5047,13 +5048,18 @@ bool intel_dp_is_edp(struct drm_device *dev, enum port port)
>  
>  	if (!dev_priv->vbt.child_dev_num)
>  		return false;
> +	
> +	if (IS_VALLEYVIEW(dev))
> +	  eDP_bits = DEVICE_TYPE_eDP_BITS_VLV;
> +	else
> +	  eDP_bits = DEVICE_TYPE_eDP_BITS;
>  
>  	for (i = 0; i < dev_priv->vbt.child_dev_num; i++) {
>  		p_child = dev_priv->vbt.child_dev + i;
>  
>  		if (p_child->common.dvo_port == port_mapping[port] &&
> -		    (p_child->common.device_type & DEVICE_TYPE_eDP_BITS) ==
> -		    (DEVICE_TYPE_eDP & DEVICE_TYPE_eDP_BITS))
> +		    (p_child->common.device_type & eDP_bits) ==
> +		    (DEVICE_TYPE_eDP & eDP_bits))
>  			return true;
>  	}
>  	return false;
> -- 
> 2.1.4
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Patch
diff mbox

diff --git a/drivers/gpu/drm/i915/intel_bios.h b/drivers/gpu/drm/i915/intel_bios.h
index af0b476..c42161f 100644
--- a/drivers/gpu/drm/i915/intel_bios.h
+++ b/drivers/gpu/drm/i915/intel_bios.h
@@ -742,6 +742,16 @@  int intel_parse_bios(struct drm_device *dev);
 	 DEVICE_TYPE_DIGITAL_OUTPUT | \
 	 DEVICE_TYPE_ANALOG_OUTPUT)
 
+/*
+ * We dont look on DEVICE_TYPE_NOT_HDMI_OUTPUT an DEVICE_TYPE_DIGITAL_OUTPUT
+ * on valleyview, because intels BMP-generated BIOS don't sets these 
+ * BITS for eDP ports
+ */ 
+#define DEVICE_TYPE_eDP_BITS_VLV \
+	(DEVICE_TYPE_eDP_BITS &\
+	 (~ DEVICE_TYPE_NOT_HDMI_OUTPUT ) &\
+	 (~ DEVICE_TYPE_DIGITAL_OUTPUT ) )
+	   
 /* define the DVO port for HDMI output type */
 #define		DVO_B		1
 #define		DVO_C		2
diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c
index f52eef1..51c753f 100644
--- a/drivers/gpu/drm/i915/intel_dp.c
+++ b/drivers/gpu/drm/i915/intel_dp.c
@@ -5036,6 +5036,7 @@  bool intel_dp_is_edp(struct drm_device *dev, enum port port)
 	struct drm_i915_private *dev_priv = dev->dev_private;
 	union child_device_config *p_child;
 	int i;
+	u16 eDP_bits;
 	static const short port_mapping[] = {
 		[PORT_B] = PORT_IDPB,
 		[PORT_C] = PORT_IDPC,
@@ -5047,13 +5048,18 @@  bool intel_dp_is_edp(struct drm_device *dev, enum port port)
 
 	if (!dev_priv->vbt.child_dev_num)
 		return false;
+	
+	if (IS_VALLEYVIEW(dev))
+	  eDP_bits = DEVICE_TYPE_eDP_BITS_VLV;
+	else
+	  eDP_bits = DEVICE_TYPE_eDP_BITS;
 
 	for (i = 0; i < dev_priv->vbt.child_dev_num; i++) {
 		p_child = dev_priv->vbt.child_dev + i;
 
 		if (p_child->common.dvo_port == port_mapping[port] &&
-		    (p_child->common.device_type & DEVICE_TYPE_eDP_BITS) ==
-		    (DEVICE_TYPE_eDP & DEVICE_TYPE_eDP_BITS))
+		    (p_child->common.device_type & eDP_bits) ==
+		    (DEVICE_TYPE_eDP & eDP_bits))
 			return true;
 	}
 	return false;