* [PATCH v2 0/6] drm/i915: start hiding away vbt structure from the driver
@ 2016-01-11 19:54 Jani Nikula
2016-01-11 19:54 ` [PATCH v2 1/6] drm/i915: move VBT based TV presence check to intel_bios.c Jani Nikula
` (7 more replies)
0 siblings, 8 replies; 17+ messages in thread
From: Jani Nikula @ 2016-01-11 19:54 UTC (permalink / raw)
To: intel-gfx; +Cc: jani.nikula
Hi all, first real patches since the RFC at [1].
The VBT is a monster and it keeps growing. Originally we've extracted
bits and pieces out of there, and added them cleanly to our own
structures in dev_priv->vbt, with our own macros. Later on we've been
slipping and we have copied stuff from VBT verbatim, using the same
structs and defines as in VBT.
Start abstracting the VBT more, hiding away the VBT specific stuff that
should not be fiddled with outside of intel_bios.c. Make it more
explicit when we're doing so. At this stage, mostly just fork the
private parts into intel_bios_private.h. As prep work, move some stuff
to intel_bios.c to limit the amount of macros needing to be exposed
across the whole driver.
With the MIPI DSI VBT stuff out of the way, this now applies on top of
nightly.
BR,
Jani.
[1] http://mid.gmane.org/cover.1450192823.git.jani.nikula@intel.com
Jani Nikula (6):
drm/i915: move VBT based TV presence check to intel_bios.c
drm/i915: move VBT based LVDS presence check to intel_bios.c
drm/i915: move VBT based eDP port check to intel_bios.c
drm/i915: move VBT based DSI presence check to intel_bios.c
drm/i915/panel: setup pwm backlight based on connector type
drm/i915: hide away VBT private data in a separate header
Documentation/DocBook/gpu.tmpl | 2 +-
drivers/gpu/drm/i915/i915_drv.h | 5 +-
drivers/gpu/drm/i915/intel_bios.c | 156 +++++-
drivers/gpu/drm/i915/intel_bios.h | 860 ++----------------------------
drivers/gpu/drm/i915/intel_bios_private.h | 823 ++++++++++++++++++++++++++++
drivers/gpu/drm/i915/intel_dp.c | 21 +-
drivers/gpu/drm/i915/intel_dsi.c | 23 +-
drivers/gpu/drm/i915/intel_lvds.c | 53 +-
drivers/gpu/drm/i915/intel_panel.c | 2 +-
drivers/gpu/drm/i915/intel_tv.c | 43 +-
10 files changed, 1034 insertions(+), 954 deletions(-)
create mode 100644 drivers/gpu/drm/i915/intel_bios_private.h
--
2.1.4
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 17+ messages in thread* [PATCH v2 1/6] drm/i915: move VBT based TV presence check to intel_bios.c 2016-01-11 19:54 [PATCH v2 0/6] drm/i915: start hiding away vbt structure from the driver Jani Nikula @ 2016-01-11 19:54 ` Jani Nikula 2016-01-12 9:45 ` Daniel Vetter 2016-01-14 12:51 ` Mika Kahola 2016-01-11 19:54 ` [PATCH v2 2/6] drm/i915: move VBT based LVDS " Jani Nikula ` (6 subsequent siblings) 7 siblings, 2 replies; 17+ messages in thread From: Jani Nikula @ 2016-01-11 19:54 UTC (permalink / raw) To: intel-gfx; +Cc: jani.nikula Hide knowledge about VBT child devices in intel_bios.c. Signed-off-by: Jani Nikula <jani.nikula@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/gpu/drm/i915/intel_bios.c | 38 ++++++++++++++++++++++++++++++++++ drivers/gpu/drm/i915/intel_tv.c | 43 +-------------------------------------- 3 files changed, 40 insertions(+), 42 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h index 104bd1809936..3822c465d3dc 100644 --- a/drivers/gpu/drm/i915/i915_drv.h +++ b/drivers/gpu/drm/i915/i915_drv.h @@ -3347,6 +3347,7 @@ extern void intel_i2c_reset(struct drm_device *dev); /* intel_bios.c */ int intel_bios_init(struct drm_i915_private *dev_priv); bool intel_bios_is_valid_vbt(const void *buf, size_t size); +bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv); /* intel_opregion.c */ #ifdef CONFIG_ACPI diff --git a/drivers/gpu/drm/i915/intel_bios.c b/drivers/gpu/drm/i915/intel_bios.c index 15ba52bd2538..bb9e8b086b63 100644 --- a/drivers/gpu/drm/i915/intel_bios.c +++ b/drivers/gpu/drm/i915/intel_bios.c @@ -1415,3 +1415,41 @@ intel_bios_init(struct drm_i915_private *dev_priv) return 0; } + +/** + * intel_bios_is_tv_present - is integrated TV present in VBT + * @dev_priv: i915 device instance + * + * Return true if TV is present. If no child devices were parsed from VBT, + * assume TV is present. + */ +bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv) +{ + union child_device_config *p_child; + int i; + + if (!dev_priv->vbt.child_dev_num) + return true; + + for (i = 0; i < dev_priv->vbt.child_dev_num; i++) { + p_child = dev_priv->vbt.child_dev + i; + /* + * If the device type is not TV, continue. + */ + switch (p_child->old.device_type) { + case DEVICE_TYPE_INT_TV: + case DEVICE_TYPE_TV: + case DEVICE_TYPE_TV_SVIDEO_COMPOSITE: + break; + default: + continue; + } + /* Only when the addin_offset is non-zero, it is regarded + * as present. + */ + if (p_child->old.addin_offset) + return true; + } + + return false; +} diff --git a/drivers/gpu/drm/i915/intel_tv.c b/drivers/gpu/drm/i915/intel_tv.c index 948cbff6c62e..29e68859b9b7 100644 --- a/drivers/gpu/drm/i915/intel_tv.c +++ b/drivers/gpu/drm/i915/intel_tv.c @@ -1529,47 +1529,6 @@ static const struct drm_encoder_funcs intel_tv_enc_funcs = { .destroy = intel_encoder_destroy, }; -/* - * Enumerate the child dev array parsed from VBT to check whether - * the integrated TV is present. - * If it is present, return 1. - * If it is not present, return false. - * If no child dev is parsed from VBT, it assumes that the TV is present. - */ -static int tv_is_present_in_vbt(struct drm_device *dev) -{ - struct drm_i915_private *dev_priv = dev->dev_private; - union child_device_config *p_child; - int i, ret; - - if (!dev_priv->vbt.child_dev_num) - return 1; - - ret = 0; - for (i = 0; i < dev_priv->vbt.child_dev_num; i++) { - p_child = dev_priv->vbt.child_dev + i; - /* - * If the device type is not TV, continue. - */ - switch (p_child->old.device_type) { - case DEVICE_TYPE_INT_TV: - case DEVICE_TYPE_TV: - case DEVICE_TYPE_TV_SVIDEO_COMPOSITE: - break; - default: - continue; - } - /* Only when the addin_offset is non-zero, it is regarded - * as present. - */ - if (p_child->old.addin_offset) { - ret = 1; - break; - } - } - return ret; -} - void intel_tv_init(struct drm_device *dev) { @@ -1585,7 +1544,7 @@ intel_tv_init(struct drm_device *dev) if ((I915_READ(TV_CTL) & TV_FUSE_STATE_MASK) == TV_FUSE_STATE_DISABLED) return; - if (!tv_is_present_in_vbt(dev)) { + if (!intel_bios_is_tv_present(dev_priv)) { DRM_DEBUG_KMS("Integrated TV is not present.\n"); return; } -- 2.1.4 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH v2 1/6] drm/i915: move VBT based TV presence check to intel_bios.c 2016-01-11 19:54 ` [PATCH v2 1/6] drm/i915: move VBT based TV presence check to intel_bios.c Jani Nikula @ 2016-01-12 9:45 ` Daniel Vetter 2016-01-14 12:51 ` Mika Kahola 1 sibling, 0 replies; 17+ messages in thread From: Daniel Vetter @ 2016-01-12 9:45 UTC (permalink / raw) To: Jani Nikula; +Cc: intel-gfx On Mon, Jan 11, 2016 at 09:54:37PM +0200, Jani Nikula wrote: > Hide knowledge about VBT child devices in intel_bios.c. > > Signed-off-by: Jani Nikula <jani.nikula@intel.com> Assuming you copypasted correctly ;-) Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> > --- > drivers/gpu/drm/i915/i915_drv.h | 1 + > drivers/gpu/drm/i915/intel_bios.c | 38 ++++++++++++++++++++++++++++++++++ > drivers/gpu/drm/i915/intel_tv.c | 43 +-------------------------------------- > 3 files changed, 40 insertions(+), 42 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h > index 104bd1809936..3822c465d3dc 100644 > --- a/drivers/gpu/drm/i915/i915_drv.h > +++ b/drivers/gpu/drm/i915/i915_drv.h > @@ -3347,6 +3347,7 @@ extern void intel_i2c_reset(struct drm_device *dev); > /* intel_bios.c */ > int intel_bios_init(struct drm_i915_private *dev_priv); > bool intel_bios_is_valid_vbt(const void *buf, size_t size); > +bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv); > > /* intel_opregion.c */ > #ifdef CONFIG_ACPI > diff --git a/drivers/gpu/drm/i915/intel_bios.c b/drivers/gpu/drm/i915/intel_bios.c > index 15ba52bd2538..bb9e8b086b63 100644 > --- a/drivers/gpu/drm/i915/intel_bios.c > +++ b/drivers/gpu/drm/i915/intel_bios.c > @@ -1415,3 +1415,41 @@ intel_bios_init(struct drm_i915_private *dev_priv) > > return 0; > } > + > +/** > + * intel_bios_is_tv_present - is integrated TV present in VBT > + * @dev_priv: i915 device instance > + * > + * Return true if TV is present. If no child devices were parsed from VBT, > + * assume TV is present. > + */ > +bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv) > +{ > + union child_device_config *p_child; > + int i; > + > + if (!dev_priv->vbt.child_dev_num) > + return true; > + > + for (i = 0; i < dev_priv->vbt.child_dev_num; i++) { > + p_child = dev_priv->vbt.child_dev + i; > + /* > + * If the device type is not TV, continue. > + */ > + switch (p_child->old.device_type) { > + case DEVICE_TYPE_INT_TV: > + case DEVICE_TYPE_TV: > + case DEVICE_TYPE_TV_SVIDEO_COMPOSITE: > + break; > + default: > + continue; > + } > + /* Only when the addin_offset is non-zero, it is regarded > + * as present. > + */ > + if (p_child->old.addin_offset) > + return true; > + } > + > + return false; > +} > diff --git a/drivers/gpu/drm/i915/intel_tv.c b/drivers/gpu/drm/i915/intel_tv.c > index 948cbff6c62e..29e68859b9b7 100644 > --- a/drivers/gpu/drm/i915/intel_tv.c > +++ b/drivers/gpu/drm/i915/intel_tv.c > @@ -1529,47 +1529,6 @@ static const struct drm_encoder_funcs intel_tv_enc_funcs = { > .destroy = intel_encoder_destroy, > }; > > -/* > - * Enumerate the child dev array parsed from VBT to check whether > - * the integrated TV is present. > - * If it is present, return 1. > - * If it is not present, return false. > - * If no child dev is parsed from VBT, it assumes that the TV is present. > - */ > -static int tv_is_present_in_vbt(struct drm_device *dev) > -{ > - struct drm_i915_private *dev_priv = dev->dev_private; > - union child_device_config *p_child; > - int i, ret; > - > - if (!dev_priv->vbt.child_dev_num) > - return 1; > - > - ret = 0; > - for (i = 0; i < dev_priv->vbt.child_dev_num; i++) { > - p_child = dev_priv->vbt.child_dev + i; > - /* > - * If the device type is not TV, continue. > - */ > - switch (p_child->old.device_type) { > - case DEVICE_TYPE_INT_TV: > - case DEVICE_TYPE_TV: > - case DEVICE_TYPE_TV_SVIDEO_COMPOSITE: > - break; > - default: > - continue; > - } > - /* Only when the addin_offset is non-zero, it is regarded > - * as present. > - */ > - if (p_child->old.addin_offset) { > - ret = 1; > - break; > - } > - } > - return ret; > -} > - > void > intel_tv_init(struct drm_device *dev) > { > @@ -1585,7 +1544,7 @@ intel_tv_init(struct drm_device *dev) > if ((I915_READ(TV_CTL) & TV_FUSE_STATE_MASK) == TV_FUSE_STATE_DISABLED) > return; > > - if (!tv_is_present_in_vbt(dev)) { > + if (!intel_bios_is_tv_present(dev_priv)) { > DRM_DEBUG_KMS("Integrated TV is not present.\n"); > return; > } > -- > 2.1.4 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v2 1/6] drm/i915: move VBT based TV presence check to intel_bios.c 2016-01-11 19:54 ` [PATCH v2 1/6] drm/i915: move VBT based TV presence check to intel_bios.c Jani Nikula 2016-01-12 9:45 ` Daniel Vetter @ 2016-01-14 12:51 ` Mika Kahola 1 sibling, 0 replies; 17+ messages in thread From: Mika Kahola @ 2016-01-14 12:51 UTC (permalink / raw) To: Jani Nikula; +Cc: intel-gfx On Mon, 2016-01-11 at 21:54 +0200, Jani Nikula wrote: > Hide knowledge about VBT child devices in intel_bios.c. > Tested-by: Mika Kahola <mika.kahola@intel.com> > Signed-off-by: Jani Nikula <jani.nikula@intel.com> > --- > drivers/gpu/drm/i915/i915_drv.h | 1 + > drivers/gpu/drm/i915/intel_bios.c | 38 ++++++++++++++++++++++++++++++++++ > drivers/gpu/drm/i915/intel_tv.c | 43 +-------------------------------------- > 3 files changed, 40 insertions(+), 42 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h > index 104bd1809936..3822c465d3dc 100644 > --- a/drivers/gpu/drm/i915/i915_drv.h > +++ b/drivers/gpu/drm/i915/i915_drv.h > @@ -3347,6 +3347,7 @@ extern void intel_i2c_reset(struct drm_device *dev); > /* intel_bios.c */ > int intel_bios_init(struct drm_i915_private *dev_priv); > bool intel_bios_is_valid_vbt(const void *buf, size_t size); > +bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv); > > /* intel_opregion.c */ > #ifdef CONFIG_ACPI > diff --git a/drivers/gpu/drm/i915/intel_bios.c b/drivers/gpu/drm/i915/intel_bios.c > index 15ba52bd2538..bb9e8b086b63 100644 > --- a/drivers/gpu/drm/i915/intel_bios.c > +++ b/drivers/gpu/drm/i915/intel_bios.c > @@ -1415,3 +1415,41 @@ intel_bios_init(struct drm_i915_private *dev_priv) > > return 0; > } > + > +/** > + * intel_bios_is_tv_present - is integrated TV present in VBT > + * @dev_priv: i915 device instance > + * > + * Return true if TV is present. If no child devices were parsed from VBT, > + * assume TV is present. > + */ > +bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv) > +{ > + union child_device_config *p_child; > + int i; > + > + if (!dev_priv->vbt.child_dev_num) > + return true; > + > + for (i = 0; i < dev_priv->vbt.child_dev_num; i++) { > + p_child = dev_priv->vbt.child_dev + i; > + /* > + * If the device type is not TV, continue. > + */ > + switch (p_child->old.device_type) { > + case DEVICE_TYPE_INT_TV: > + case DEVICE_TYPE_TV: > + case DEVICE_TYPE_TV_SVIDEO_COMPOSITE: > + break; > + default: > + continue; > + } > + /* Only when the addin_offset is non-zero, it is regarded > + * as present. > + */ > + if (p_child->old.addin_offset) > + return true; > + } > + > + return false; > +} > diff --git a/drivers/gpu/drm/i915/intel_tv.c b/drivers/gpu/drm/i915/intel_tv.c > index 948cbff6c62e..29e68859b9b7 100644 > --- a/drivers/gpu/drm/i915/intel_tv.c > +++ b/drivers/gpu/drm/i915/intel_tv.c > @@ -1529,47 +1529,6 @@ static const struct drm_encoder_funcs intel_tv_enc_funcs = { > .destroy = intel_encoder_destroy, > }; > > -/* > - * Enumerate the child dev array parsed from VBT to check whether > - * the integrated TV is present. > - * If it is present, return 1. > - * If it is not present, return false. > - * If no child dev is parsed from VBT, it assumes that the TV is present. > - */ > -static int tv_is_present_in_vbt(struct drm_device *dev) > -{ > - struct drm_i915_private *dev_priv = dev->dev_private; > - union child_device_config *p_child; > - int i, ret; > - > - if (!dev_priv->vbt.child_dev_num) > - return 1; > - > - ret = 0; > - for (i = 0; i < dev_priv->vbt.child_dev_num; i++) { > - p_child = dev_priv->vbt.child_dev + i; > - /* > - * If the device type is not TV, continue. > - */ > - switch (p_child->old.device_type) { > - case DEVICE_TYPE_INT_TV: > - case DEVICE_TYPE_TV: > - case DEVICE_TYPE_TV_SVIDEO_COMPOSITE: > - break; > - default: > - continue; > - } > - /* Only when the addin_offset is non-zero, it is regarded > - * as present. > - */ > - if (p_child->old.addin_offset) { > - ret = 1; > - break; > - } > - } > - return ret; > -} > - > void > intel_tv_init(struct drm_device *dev) > { > @@ -1585,7 +1544,7 @@ intel_tv_init(struct drm_device *dev) > if ((I915_READ(TV_CTL) & TV_FUSE_STATE_MASK) == TV_FUSE_STATE_DISABLED) > return; > > - if (!tv_is_present_in_vbt(dev)) { > + if (!intel_bios_is_tv_present(dev_priv)) { > DRM_DEBUG_KMS("Integrated TV is not present.\n"); > return; > } _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH v2 2/6] drm/i915: move VBT based LVDS presence check to intel_bios.c 2016-01-11 19:54 [PATCH v2 0/6] drm/i915: start hiding away vbt structure from the driver Jani Nikula 2016-01-11 19:54 ` [PATCH v2 1/6] drm/i915: move VBT based TV presence check to intel_bios.c Jani Nikula @ 2016-01-11 19:54 ` Jani Nikula 2016-01-14 12:52 ` Mika Kahola 2016-01-11 19:54 ` [PATCH v2 3/6] drm/i915: move VBT based eDP port " Jani Nikula ` (5 subsequent siblings) 7 siblings, 1 reply; 17+ messages in thread From: Jani Nikula @ 2016-01-11 19:54 UTC (permalink / raw) To: intel-gfx; +Cc: jani.nikula Hide knowledge about VBT child devices in intel_bios.c. Signed-off-by: Jani Nikula <jani.nikula@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/gpu/drm/i915/intel_bios.c | 50 ++++++++++++++++++++++++++++++++++++ drivers/gpu/drm/i915/intel_lvds.c | 53 +-------------------------------------- 3 files changed, 52 insertions(+), 52 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h index 3822c465d3dc..5f0d44b380aa 100644 --- a/drivers/gpu/drm/i915/i915_drv.h +++ b/drivers/gpu/drm/i915/i915_drv.h @@ -3348,6 +3348,7 @@ extern void intel_i2c_reset(struct drm_device *dev); int intel_bios_init(struct drm_i915_private *dev_priv); bool intel_bios_is_valid_vbt(const void *buf, size_t size); bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv); +bool intel_bios_is_lvds_present(struct drm_i915_private *dev_priv, u8 *i2c_pin); /* intel_opregion.c */ #ifdef CONFIG_ACPI diff --git a/drivers/gpu/drm/i915/intel_bios.c b/drivers/gpu/drm/i915/intel_bios.c index bb9e8b086b63..099f3f0002a2 100644 --- a/drivers/gpu/drm/i915/intel_bios.c +++ b/drivers/gpu/drm/i915/intel_bios.c @@ -1453,3 +1453,53 @@ bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv) return false; } + +/** + * intel_bios_is_lvds_present - is LVDS present in VBT + * @dev_priv: i915 device instance + * @i2c_pin: i2c pin for LVDS if present + * + * Return true if LVDS is present. If no child devices were parsed from VBT, + * assume LVDS is present. + */ +bool intel_bios_is_lvds_present(struct drm_i915_private *dev_priv, u8 *i2c_pin) +{ + int i; + + if (!dev_priv->vbt.child_dev_num) + return true; + + for (i = 0; i < dev_priv->vbt.child_dev_num; i++) { + union child_device_config *uchild = dev_priv->vbt.child_dev + i; + struct old_child_dev_config *child = &uchild->old; + + /* If the device type is not LFP, continue. + * We have to check both the new identifiers as well as the + * old for compatibility with some BIOSes. + */ + if (child->device_type != DEVICE_TYPE_INT_LFP && + child->device_type != DEVICE_TYPE_LFP) + continue; + + if (intel_gmbus_is_valid_pin(dev_priv, child->i2c_pin)) + *i2c_pin = child->i2c_pin; + + /* However, we cannot trust the BIOS writers to populate + * the VBT correctly. Since LVDS requires additional + * information from AIM blocks, a non-zero addin offset is + * a good indicator that the LVDS is actually present. + */ + if (child->addin_offset) + return true; + + /* But even then some BIOS writers perform some black magic + * and instantiate the device without reference to any + * additional data. Trust that if the VBT was written into + * the OpRegion then they have validated the LVDS's existence. + */ + if (dev_priv->opregion.vbt) + return true; + } + + return false; +} diff --git a/drivers/gpu/drm/i915/intel_lvds.c b/drivers/gpu/drm/i915/intel_lvds.c index 0da0240caf81..80f8684e5137 100644 --- a/drivers/gpu/drm/i915/intel_lvds.c +++ b/drivers/gpu/drm/i915/intel_lvds.c @@ -772,57 +772,6 @@ static const struct dmi_system_id intel_no_lvds[] = { { } /* terminating entry */ }; -/* - * Enumerate the child dev array parsed from VBT to check whether - * the LVDS is present. - * If it is present, return 1. - * If it is not present, return false. - * If no child dev is parsed from VBT, it assumes that the LVDS is present. - */ -static bool lvds_is_present_in_vbt(struct drm_device *dev, - u8 *i2c_pin) -{ - struct drm_i915_private *dev_priv = dev->dev_private; - int i; - - if (!dev_priv->vbt.child_dev_num) - return true; - - for (i = 0; i < dev_priv->vbt.child_dev_num; i++) { - union child_device_config *uchild = dev_priv->vbt.child_dev + i; - struct old_child_dev_config *child = &uchild->old; - - /* If the device type is not LFP, continue. - * We have to check both the new identifiers as well as the - * old for compatibility with some BIOSes. - */ - if (child->device_type != DEVICE_TYPE_INT_LFP && - child->device_type != DEVICE_TYPE_LFP) - continue; - - if (intel_gmbus_is_valid_pin(dev_priv, child->i2c_pin)) - *i2c_pin = child->i2c_pin; - - /* However, we cannot trust the BIOS writers to populate - * the VBT correctly. Since LVDS requires additional - * information from AIM blocks, a non-zero addin offset is - * a good indicator that the LVDS is actually present. - */ - if (child->addin_offset) - return true; - - /* But even then some BIOS writers perform some black magic - * and instantiate the device without reference to any - * additional data. Trust that if the VBT was written into - * the OpRegion then they have validated the LVDS's existence. - */ - if (dev_priv->opregion.vbt) - return true; - } - - return false; -} - static int intel_dual_link_lvds_callback(const struct dmi_system_id *id) { DRM_INFO("Forcing lvds to dual link mode on %s\n", id->ident); @@ -979,7 +928,7 @@ void intel_lvds_init(struct drm_device *dev) } pin = GMBUS_PIN_PANEL; - if (!lvds_is_present_in_vbt(dev, &pin)) { + if (!intel_bios_is_lvds_present(dev_priv, &pin)) { if ((lvds & LVDS_PORT_EN) == 0) { DRM_DEBUG_KMS("LVDS is not present in VBT\n"); return; -- 2.1.4 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH v2 2/6] drm/i915: move VBT based LVDS presence check to intel_bios.c 2016-01-11 19:54 ` [PATCH v2 2/6] drm/i915: move VBT based LVDS " Jani Nikula @ 2016-01-14 12:52 ` Mika Kahola 0 siblings, 0 replies; 17+ messages in thread From: Mika Kahola @ 2016-01-14 12:52 UTC (permalink / raw) To: Jani Nikula; +Cc: intel-gfx On Mon, 2016-01-11 at 21:54 +0200, Jani Nikula wrote: > Hide knowledge about VBT child devices in intel_bios.c. > Tested-by: Mika Kahola <mika.kahola@intel.com> > Signed-off-by: Jani Nikula <jani.nikula@intel.com> > --- > drivers/gpu/drm/i915/i915_drv.h | 1 + > drivers/gpu/drm/i915/intel_bios.c | 50 ++++++++++++++++++++++++++++++++++++ > drivers/gpu/drm/i915/intel_lvds.c | 53 +-------------------------------------- > 3 files changed, 52 insertions(+), 52 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h > index 3822c465d3dc..5f0d44b380aa 100644 > --- a/drivers/gpu/drm/i915/i915_drv.h > +++ b/drivers/gpu/drm/i915/i915_drv.h > @@ -3348,6 +3348,7 @@ extern void intel_i2c_reset(struct drm_device *dev); > int intel_bios_init(struct drm_i915_private *dev_priv); > bool intel_bios_is_valid_vbt(const void *buf, size_t size); > bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv); > +bool intel_bios_is_lvds_present(struct drm_i915_private *dev_priv, u8 *i2c_pin); > > /* intel_opregion.c */ > #ifdef CONFIG_ACPI > diff --git a/drivers/gpu/drm/i915/intel_bios.c b/drivers/gpu/drm/i915/intel_bios.c > index bb9e8b086b63..099f3f0002a2 100644 > --- a/drivers/gpu/drm/i915/intel_bios.c > +++ b/drivers/gpu/drm/i915/intel_bios.c > @@ -1453,3 +1453,53 @@ bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv) > > return false; > } > + > +/** > + * intel_bios_is_lvds_present - is LVDS present in VBT > + * @dev_priv: i915 device instance > + * @i2c_pin: i2c pin for LVDS if present > + * > + * Return true if LVDS is present. If no child devices were parsed from VBT, > + * assume LVDS is present. > + */ > +bool intel_bios_is_lvds_present(struct drm_i915_private *dev_priv, u8 *i2c_pin) > +{ > + int i; > + > + if (!dev_priv->vbt.child_dev_num) > + return true; > + > + for (i = 0; i < dev_priv->vbt.child_dev_num; i++) { > + union child_device_config *uchild = dev_priv->vbt.child_dev + i; > + struct old_child_dev_config *child = &uchild->old; > + > + /* If the device type is not LFP, continue. > + * We have to check both the new identifiers as well as the > + * old for compatibility with some BIOSes. > + */ > + if (child->device_type != DEVICE_TYPE_INT_LFP && > + child->device_type != DEVICE_TYPE_LFP) > + continue; > + > + if (intel_gmbus_is_valid_pin(dev_priv, child->i2c_pin)) > + *i2c_pin = child->i2c_pin; > + > + /* However, we cannot trust the BIOS writers to populate > + * the VBT correctly. Since LVDS requires additional > + * information from AIM blocks, a non-zero addin offset is > + * a good indicator that the LVDS is actually present. > + */ > + if (child->addin_offset) > + return true; > + > + /* But even then some BIOS writers perform some black magic > + * and instantiate the device without reference to any > + * additional data. Trust that if the VBT was written into > + * the OpRegion then they have validated the LVDS's existence. > + */ > + if (dev_priv->opregion.vbt) > + return true; > + } > + > + return false; > +} > diff --git a/drivers/gpu/drm/i915/intel_lvds.c b/drivers/gpu/drm/i915/intel_lvds.c > index 0da0240caf81..80f8684e5137 100644 > --- a/drivers/gpu/drm/i915/intel_lvds.c > +++ b/drivers/gpu/drm/i915/intel_lvds.c > @@ -772,57 +772,6 @@ static const struct dmi_system_id intel_no_lvds[] = { > { } /* terminating entry */ > }; > > -/* > - * Enumerate the child dev array parsed from VBT to check whether > - * the LVDS is present. > - * If it is present, return 1. > - * If it is not present, return false. > - * If no child dev is parsed from VBT, it assumes that the LVDS is present. > - */ > -static bool lvds_is_present_in_vbt(struct drm_device *dev, > - u8 *i2c_pin) > -{ > - struct drm_i915_private *dev_priv = dev->dev_private; > - int i; > - > - if (!dev_priv->vbt.child_dev_num) > - return true; > - > - for (i = 0; i < dev_priv->vbt.child_dev_num; i++) { > - union child_device_config *uchild = dev_priv->vbt.child_dev + i; > - struct old_child_dev_config *child = &uchild->old; > - > - /* If the device type is not LFP, continue. > - * We have to check both the new identifiers as well as the > - * old for compatibility with some BIOSes. > - */ > - if (child->device_type != DEVICE_TYPE_INT_LFP && > - child->device_type != DEVICE_TYPE_LFP) > - continue; > - > - if (intel_gmbus_is_valid_pin(dev_priv, child->i2c_pin)) > - *i2c_pin = child->i2c_pin; > - > - /* However, we cannot trust the BIOS writers to populate > - * the VBT correctly. Since LVDS requires additional > - * information from AIM blocks, a non-zero addin offset is > - * a good indicator that the LVDS is actually present. > - */ > - if (child->addin_offset) > - return true; > - > - /* But even then some BIOS writers perform some black magic > - * and instantiate the device without reference to any > - * additional data. Trust that if the VBT was written into > - * the OpRegion then they have validated the LVDS's existence. > - */ > - if (dev_priv->opregion.vbt) > - return true; > - } > - > - return false; > -} > - > static int intel_dual_link_lvds_callback(const struct dmi_system_id *id) > { > DRM_INFO("Forcing lvds to dual link mode on %s\n", id->ident); > @@ -979,7 +928,7 @@ void intel_lvds_init(struct drm_device *dev) > } > > pin = GMBUS_PIN_PANEL; > - if (!lvds_is_present_in_vbt(dev, &pin)) { > + if (!intel_bios_is_lvds_present(dev_priv, &pin)) { > if ((lvds & LVDS_PORT_EN) == 0) { > DRM_DEBUG_KMS("LVDS is not present in VBT\n"); > return; _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH v2 3/6] drm/i915: move VBT based eDP port check to intel_bios.c 2016-01-11 19:54 [PATCH v2 0/6] drm/i915: start hiding away vbt structure from the driver Jani Nikula 2016-01-11 19:54 ` [PATCH v2 1/6] drm/i915: move VBT based TV presence check to intel_bios.c Jani Nikula 2016-01-11 19:54 ` [PATCH v2 2/6] drm/i915: move VBT based LVDS " Jani Nikula @ 2016-01-11 19:54 ` Jani Nikula 2016-01-14 12:52 ` Mika Kahola 2016-01-11 19:54 ` [PATCH v2 4/6] drm/i915: move VBT based DSI presence " Jani Nikula ` (4 subsequent siblings) 7 siblings, 1 reply; 17+ messages in thread From: Jani Nikula @ 2016-01-11 19:54 UTC (permalink / raw) To: intel-gfx; +Cc: jani.nikula Hide knowledge about VBT child devices in intel_bios.c. Signed-off-by: Jani Nikula <jani.nikula@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/gpu/drm/i915/intel_bios.c | 33 +++++++++++++++++++++++++++++++++ drivers/gpu/drm/i915/intel_dp.c | 21 +-------------------- 3 files changed, 35 insertions(+), 20 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h index 5f0d44b380aa..6ae828f984de 100644 --- a/drivers/gpu/drm/i915/i915_drv.h +++ b/drivers/gpu/drm/i915/i915_drv.h @@ -3349,6 +3349,7 @@ int intel_bios_init(struct drm_i915_private *dev_priv); bool intel_bios_is_valid_vbt(const void *buf, size_t size); bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv); bool intel_bios_is_lvds_present(struct drm_i915_private *dev_priv, u8 *i2c_pin); +bool intel_bios_is_port_edp(struct drm_i915_private *dev_priv, enum port port); /* intel_opregion.c */ #ifdef CONFIG_ACPI diff --git a/drivers/gpu/drm/i915/intel_bios.c b/drivers/gpu/drm/i915/intel_bios.c index 099f3f0002a2..a7491a450cc8 100644 --- a/drivers/gpu/drm/i915/intel_bios.c +++ b/drivers/gpu/drm/i915/intel_bios.c @@ -1503,3 +1503,36 @@ bool intel_bios_is_lvds_present(struct drm_i915_private *dev_priv, u8 *i2c_pin) return false; } + +/** + * intel_bios_is_port_edp - is the device in given port eDP + * @dev_priv: i915 device instance + * @port: port to check + * + * Return true if the device in %port is eDP. + */ +bool intel_bios_is_port_edp(struct drm_i915_private *dev_priv, enum port port) +{ + union child_device_config *p_child; + static const short port_mapping[] = { + [PORT_B] = DVO_PORT_DPB, + [PORT_C] = DVO_PORT_DPC, + [PORT_D] = DVO_PORT_DPD, + [PORT_E] = DVO_PORT_DPE, + }; + int i; + + if (!dev_priv->vbt.child_dev_num) + return false; + + 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)) + return true; + } + + return false; +} diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index 796e3d313cb9..02cb4e9bf5ea 100644 --- a/drivers/gpu/drm/i915/intel_dp.c +++ b/drivers/gpu/drm/i915/intel_dp.c @@ -5068,14 +5068,6 @@ put_power: 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; - static const short port_mapping[] = { - [PORT_B] = DVO_PORT_DPB, - [PORT_C] = DVO_PORT_DPC, - [PORT_D] = DVO_PORT_DPD, - [PORT_E] = DVO_PORT_DPE, - }; /* * eDP not supported on g4x. so bail out early just @@ -5087,18 +5079,7 @@ bool intel_dp_is_edp(struct drm_device *dev, enum port port) if (port == PORT_A) return true; - if (!dev_priv->vbt.child_dev_num) - return false; - - 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)) - return true; - } - return false; + return intel_bios_is_port_edp(dev_priv, port); } void -- 2.1.4 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH v2 3/6] drm/i915: move VBT based eDP port check to intel_bios.c 2016-01-11 19:54 ` [PATCH v2 3/6] drm/i915: move VBT based eDP port " Jani Nikula @ 2016-01-14 12:52 ` Mika Kahola 0 siblings, 0 replies; 17+ messages in thread From: Mika Kahola @ 2016-01-14 12:52 UTC (permalink / raw) To: Jani Nikula; +Cc: intel-gfx On Mon, 2016-01-11 at 21:54 +0200, Jani Nikula wrote: > Hide knowledge about VBT child devices in intel_bios.c. > Tested-by: Mika Kahola <mika.kahola@intel.com> > Signed-off-by: Jani Nikula <jani.nikula@intel.com> > --- > drivers/gpu/drm/i915/i915_drv.h | 1 + > drivers/gpu/drm/i915/intel_bios.c | 33 +++++++++++++++++++++++++++++++++ > drivers/gpu/drm/i915/intel_dp.c | 21 +-------------------- > 3 files changed, 35 insertions(+), 20 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h > index 5f0d44b380aa..6ae828f984de 100644 > --- a/drivers/gpu/drm/i915/i915_drv.h > +++ b/drivers/gpu/drm/i915/i915_drv.h > @@ -3349,6 +3349,7 @@ int intel_bios_init(struct drm_i915_private *dev_priv); > bool intel_bios_is_valid_vbt(const void *buf, size_t size); > bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv); > bool intel_bios_is_lvds_present(struct drm_i915_private *dev_priv, u8 *i2c_pin); > +bool intel_bios_is_port_edp(struct drm_i915_private *dev_priv, enum port port); > > /* intel_opregion.c */ > #ifdef CONFIG_ACPI > diff --git a/drivers/gpu/drm/i915/intel_bios.c b/drivers/gpu/drm/i915/intel_bios.c > index 099f3f0002a2..a7491a450cc8 100644 > --- a/drivers/gpu/drm/i915/intel_bios.c > +++ b/drivers/gpu/drm/i915/intel_bios.c > @@ -1503,3 +1503,36 @@ bool intel_bios_is_lvds_present(struct drm_i915_private *dev_priv, u8 *i2c_pin) > > return false; > } > + > +/** > + * intel_bios_is_port_edp - is the device in given port eDP > + * @dev_priv: i915 device instance > + * @port: port to check > + * > + * Return true if the device in %port is eDP. > + */ > +bool intel_bios_is_port_edp(struct drm_i915_private *dev_priv, enum port port) > +{ > + union child_device_config *p_child; > + static const short port_mapping[] = { > + [PORT_B] = DVO_PORT_DPB, > + [PORT_C] = DVO_PORT_DPC, > + [PORT_D] = DVO_PORT_DPD, > + [PORT_E] = DVO_PORT_DPE, > + }; > + int i; > + > + if (!dev_priv->vbt.child_dev_num) > + return false; > + > + 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)) > + return true; > + } > + > + return false; > +} > diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c > index 796e3d313cb9..02cb4e9bf5ea 100644 > --- a/drivers/gpu/drm/i915/intel_dp.c > +++ b/drivers/gpu/drm/i915/intel_dp.c > @@ -5068,14 +5068,6 @@ put_power: > 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; > - static const short port_mapping[] = { > - [PORT_B] = DVO_PORT_DPB, > - [PORT_C] = DVO_PORT_DPC, > - [PORT_D] = DVO_PORT_DPD, > - [PORT_E] = DVO_PORT_DPE, > - }; > > /* > * eDP not supported on g4x. so bail out early just > @@ -5087,18 +5079,7 @@ bool intel_dp_is_edp(struct drm_device *dev, enum port port) > if (port == PORT_A) > return true; > > - if (!dev_priv->vbt.child_dev_num) > - return false; > - > - 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)) > - return true; > - } > - return false; > + return intel_bios_is_port_edp(dev_priv, port); > } > > void _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH v2 4/6] drm/i915: move VBT based DSI presence check to intel_bios.c 2016-01-11 19:54 [PATCH v2 0/6] drm/i915: start hiding away vbt structure from the driver Jani Nikula ` (2 preceding siblings ...) 2016-01-11 19:54 ` [PATCH v2 3/6] drm/i915: move VBT based eDP port " Jani Nikula @ 2016-01-11 19:54 ` Jani Nikula 2016-01-14 12:52 ` Mika Kahola 2016-01-11 19:54 ` [PATCH v2 5/6] drm/i915/panel: setup pwm backlight based on connector type Jani Nikula ` (3 subsequent siblings) 7 siblings, 1 reply; 17+ messages in thread From: Jani Nikula @ 2016-01-11 19:54 UTC (permalink / raw) To: intel-gfx; +Cc: jani.nikula Hide knowledge about VBT child devices in intel_bios.c. Signed-off-by: Jani Nikula <jani.nikula@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 2 +- drivers/gpu/drm/i915/intel_bios.c | 33 ++++++++++++++++++++++++++++++++- drivers/gpu/drm/i915/intel_dsi.c | 23 ++++++++++++++--------- 3 files changed, 47 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h index 6ae828f984de..f8516f564d0a 100644 --- a/drivers/gpu/drm/i915/i915_drv.h +++ b/drivers/gpu/drm/i915/i915_drv.h @@ -1474,7 +1474,6 @@ struct intel_vbt_data { /* MIPI DSI */ struct { - u16 port; u16 panel_id; struct mipi_config *config; struct mipi_pps_data *pps; @@ -3350,6 +3349,7 @@ bool intel_bios_is_valid_vbt(const void *buf, size_t size); bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv); bool intel_bios_is_lvds_present(struct drm_i915_private *dev_priv, u8 *i2c_pin); bool intel_bios_is_port_edp(struct drm_i915_private *dev_priv, enum port port); +bool intel_bios_is_dsi_present(struct drm_i915_private *dev_priv, enum port *port); /* intel_opregion.c */ #ifdef CONFIG_ACPI diff --git a/drivers/gpu/drm/i915/intel_bios.c b/drivers/gpu/drm/i915/intel_bios.c index a7491a450cc8..94c2baa50db7 100644 --- a/drivers/gpu/drm/i915/intel_bios.c +++ b/drivers/gpu/drm/i915/intel_bios.c @@ -1221,7 +1221,6 @@ parse_device_mapping(struct drm_i915_private *dev_priv, &&p_child->common.device_type & DEVICE_TYPE_MIPI_OUTPUT) { DRM_DEBUG_KMS("Found MIPI as LFP\n"); dev_priv->vbt.has_mipi = 1; - dev_priv->vbt.dsi.port = p_child->common.dvo_port; } child_dev_ptr = dev_priv->vbt.child_dev + count; @@ -1536,3 +1535,35 @@ bool intel_bios_is_port_edp(struct drm_i915_private *dev_priv, enum port port) return false; } + +/** + * intel_bios_is_dsi_present - is DSI present in VBT + * @dev_priv: i915 device instance + * @port: port for DSI if present + * + * Return true if DSI is present, and return the port in %port. + */ +bool intel_bios_is_dsi_present(struct drm_i915_private *dev_priv, + enum port *port) +{ + union child_device_config *p_child; + int i; + + for (i = 0; i < dev_priv->vbt.child_dev_num; i++) { + p_child = dev_priv->vbt.child_dev + i; + + if (!(p_child->common.device_type & DEVICE_TYPE_MIPI_OUTPUT)) + continue; + + switch (p_child->common.dvo_port) { + case DVO_PORT_MIPIA: + case DVO_PORT_MIPIB: + case DVO_PORT_MIPIC: + case DVO_PORT_MIPID: + *port = p_child->common.dvo_port - DVO_PORT_MIPIA; + return true; + } + } + + return false; +} diff --git a/drivers/gpu/drm/i915/intel_dsi.c b/drivers/gpu/drm/i915/intel_dsi.c index 91cef3525c93..33f219cfba43 100644 --- a/drivers/gpu/drm/i915/intel_dsi.c +++ b/drivers/gpu/drm/i915/intel_dsi.c @@ -1112,9 +1112,15 @@ void intel_dsi_init(struct drm_device *dev) DRM_DEBUG_KMS("\n"); /* There is no detection method for MIPI so rely on VBT */ - if (!dev_priv->vbt.has_mipi) + if (!intel_bios_is_dsi_present(dev_priv, &port)) return; + if (port != PORT_A && port != PORT_C) { + DRM_DEBUG_KMS("VBT has unsupported DSI port %c\n", + port_name(port)); + return; + } + if (IS_VALLEYVIEW(dev) || IS_CHERRYVIEW(dev)) { dev_priv->mipi_mmio_base = VLV_MIPI_BASE; } else { @@ -1153,16 +1159,15 @@ void intel_dsi_init(struct drm_device *dev) intel_connector->unregister = intel_connector_unregister; /* Pipe A maps to MIPI DSI port A, pipe B maps to MIPI DSI port C */ - if (dev_priv->vbt.dsi.port == DVO_PORT_MIPIA) { - intel_encoder->crtc_mask = (1 << PIPE_A); - intel_dsi->ports = (1 << PORT_A); - } else if (dev_priv->vbt.dsi.port == DVO_PORT_MIPIC) { - intel_encoder->crtc_mask = (1 << PIPE_B); - intel_dsi->ports = (1 << PORT_C); - } + if (port == PORT_A) + intel_encoder->crtc_mask = 1 << PIPE_A; + else + intel_encoder->crtc_mask = 1 << PIPE_B; if (dev_priv->vbt.dsi.config->dual_link) - intel_dsi->ports = ((1 << PORT_A) | (1 << PORT_C)); + intel_dsi->ports = (1 << PORT_A) | (1 << PORT_C); + else + intel_dsi->ports = 1 << port; /* Create a DSI host (and a device) for each port. */ for_each_dsi_port(port, intel_dsi->ports) { -- 2.1.4 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH v2 4/6] drm/i915: move VBT based DSI presence check to intel_bios.c 2016-01-11 19:54 ` [PATCH v2 4/6] drm/i915: move VBT based DSI presence " Jani Nikula @ 2016-01-14 12:52 ` Mika Kahola 0 siblings, 0 replies; 17+ messages in thread From: Mika Kahola @ 2016-01-14 12:52 UTC (permalink / raw) To: Jani Nikula; +Cc: intel-gfx On Mon, 2016-01-11 at 21:54 +0200, Jani Nikula wrote: > Hide knowledge about VBT child devices in intel_bios.c. > Tested-by: Mika Kahola <mika.kahola@intel.com> > Signed-off-by: Jani Nikula <jani.nikula@intel.com> > --- > drivers/gpu/drm/i915/i915_drv.h | 2 +- > drivers/gpu/drm/i915/intel_bios.c | 33 ++++++++++++++++++++++++++++++++- > drivers/gpu/drm/i915/intel_dsi.c | 23 ++++++++++++++--------- > 3 files changed, 47 insertions(+), 11 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h > index 6ae828f984de..f8516f564d0a 100644 > --- a/drivers/gpu/drm/i915/i915_drv.h > +++ b/drivers/gpu/drm/i915/i915_drv.h > @@ -1474,7 +1474,6 @@ struct intel_vbt_data { > > /* MIPI DSI */ > struct { > - u16 port; > u16 panel_id; > struct mipi_config *config; > struct mipi_pps_data *pps; > @@ -3350,6 +3349,7 @@ bool intel_bios_is_valid_vbt(const void *buf, size_t size); > bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv); > bool intel_bios_is_lvds_present(struct drm_i915_private *dev_priv, u8 *i2c_pin); > bool intel_bios_is_port_edp(struct drm_i915_private *dev_priv, enum port port); > +bool intel_bios_is_dsi_present(struct drm_i915_private *dev_priv, enum port *port); > > /* intel_opregion.c */ > #ifdef CONFIG_ACPI > diff --git a/drivers/gpu/drm/i915/intel_bios.c b/drivers/gpu/drm/i915/intel_bios.c > index a7491a450cc8..94c2baa50db7 100644 > --- a/drivers/gpu/drm/i915/intel_bios.c > +++ b/drivers/gpu/drm/i915/intel_bios.c > @@ -1221,7 +1221,6 @@ parse_device_mapping(struct drm_i915_private *dev_priv, > &&p_child->common.device_type & DEVICE_TYPE_MIPI_OUTPUT) { > DRM_DEBUG_KMS("Found MIPI as LFP\n"); > dev_priv->vbt.has_mipi = 1; > - dev_priv->vbt.dsi.port = p_child->common.dvo_port; > } > > child_dev_ptr = dev_priv->vbt.child_dev + count; > @@ -1536,3 +1535,35 @@ bool intel_bios_is_port_edp(struct drm_i915_private *dev_priv, enum port port) > > return false; > } > + > +/** > + * intel_bios_is_dsi_present - is DSI present in VBT > + * @dev_priv: i915 device instance > + * @port: port for DSI if present > + * > + * Return true if DSI is present, and return the port in %port. > + */ > +bool intel_bios_is_dsi_present(struct drm_i915_private *dev_priv, > + enum port *port) > +{ > + union child_device_config *p_child; > + int i; > + > + for (i = 0; i < dev_priv->vbt.child_dev_num; i++) { > + p_child = dev_priv->vbt.child_dev + i; > + > + if (!(p_child->common.device_type & DEVICE_TYPE_MIPI_OUTPUT)) > + continue; > + > + switch (p_child->common.dvo_port) { > + case DVO_PORT_MIPIA: > + case DVO_PORT_MIPIB: > + case DVO_PORT_MIPIC: > + case DVO_PORT_MIPID: > + *port = p_child->common.dvo_port - DVO_PORT_MIPIA; > + return true; > + } > + } > + > + return false; > +} > diff --git a/drivers/gpu/drm/i915/intel_dsi.c b/drivers/gpu/drm/i915/intel_dsi.c > index 91cef3525c93..33f219cfba43 100644 > --- a/drivers/gpu/drm/i915/intel_dsi.c > +++ b/drivers/gpu/drm/i915/intel_dsi.c > @@ -1112,9 +1112,15 @@ void intel_dsi_init(struct drm_device *dev) > DRM_DEBUG_KMS("\n"); > > /* There is no detection method for MIPI so rely on VBT */ > - if (!dev_priv->vbt.has_mipi) > + if (!intel_bios_is_dsi_present(dev_priv, &port)) > return; > > + if (port != PORT_A && port != PORT_C) { > + DRM_DEBUG_KMS("VBT has unsupported DSI port %c\n", > + port_name(port)); > + return; > + } > + > if (IS_VALLEYVIEW(dev) || IS_CHERRYVIEW(dev)) { > dev_priv->mipi_mmio_base = VLV_MIPI_BASE; > } else { > @@ -1153,16 +1159,15 @@ void intel_dsi_init(struct drm_device *dev) > intel_connector->unregister = intel_connector_unregister; > > /* Pipe A maps to MIPI DSI port A, pipe B maps to MIPI DSI port C */ > - if (dev_priv->vbt.dsi.port == DVO_PORT_MIPIA) { > - intel_encoder->crtc_mask = (1 << PIPE_A); > - intel_dsi->ports = (1 << PORT_A); > - } else if (dev_priv->vbt.dsi.port == DVO_PORT_MIPIC) { > - intel_encoder->crtc_mask = (1 << PIPE_B); > - intel_dsi->ports = (1 << PORT_C); > - } > + if (port == PORT_A) > + intel_encoder->crtc_mask = 1 << PIPE_A; > + else > + intel_encoder->crtc_mask = 1 << PIPE_B; > > if (dev_priv->vbt.dsi.config->dual_link) > - intel_dsi->ports = ((1 << PORT_A) | (1 << PORT_C)); > + intel_dsi->ports = (1 << PORT_A) | (1 << PORT_C); > + else > + intel_dsi->ports = 1 << port; > > /* Create a DSI host (and a device) for each port. */ > for_each_dsi_port(port, intel_dsi->ports) { _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH v2 5/6] drm/i915/panel: setup pwm backlight based on connector type 2016-01-11 19:54 [PATCH v2 0/6] drm/i915: start hiding away vbt structure from the driver Jani Nikula ` (3 preceding siblings ...) 2016-01-11 19:54 ` [PATCH v2 4/6] drm/i915: move VBT based DSI presence " Jani Nikula @ 2016-01-11 19:54 ` Jani Nikula 2016-01-11 19:54 ` [PATCH v2 6/6] drm/i915: hide away VBT private data in a separate header Jani Nikula ` (2 subsequent siblings) 7 siblings, 0 replies; 17+ messages in thread From: Jani Nikula @ 2016-01-11 19:54 UTC (permalink / raw) To: intel-gfx; +Cc: jani.nikula Use the connector type instead of VBT directly to decide which backlight mechanism to use on VLV/CHV. (Indirectly, this is the same thing, but hides the VBT use.) Signed-off-by: Jani Nikula <jani.nikula@intel.com> --- drivers/gpu/drm/i915/intel_panel.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_panel.c b/drivers/gpu/drm/i915/intel_panel.c index 21ee6477bf98..f7fbb7c223b1 100644 --- a/drivers/gpu/drm/i915/intel_panel.c +++ b/drivers/gpu/drm/i915/intel_panel.c @@ -1745,7 +1745,7 @@ intel_panel_init_backlight_funcs(struct intel_panel *panel) panel->backlight.get = pch_get_backlight; panel->backlight.hz_to_pwm = pch_hz_to_pwm; } else if (IS_VALLEYVIEW(dev_priv) || IS_CHERRYVIEW(dev_priv)) { - if (dev_priv->vbt.has_mipi) { + if (connector->base.connector_type == DRM_MODE_CONNECTOR_DSI) { panel->backlight.setup = pwm_setup_backlight; panel->backlight.enable = pwm_enable_backlight; panel->backlight.disable = pwm_disable_backlight; -- 2.1.4 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH v2 6/6] drm/i915: hide away VBT private data in a separate header 2016-01-11 19:54 [PATCH v2 0/6] drm/i915: start hiding away vbt structure from the driver Jani Nikula ` (4 preceding siblings ...) 2016-01-11 19:54 ` [PATCH v2 5/6] drm/i915/panel: setup pwm backlight based on connector type Jani Nikula @ 2016-01-11 19:54 ` Jani Nikula 2016-01-14 12:53 ` Mika Kahola 2016-01-11 20:07 ` [PATCH v2 0/6] drm/i915: start hiding away vbt structure from the driver Lukas Wunner 2016-01-12 8:20 ` ✓ success: Fi.CI.BAT Patchwork 7 siblings, 1 reply; 17+ messages in thread From: Jani Nikula @ 2016-01-11 19:54 UTC (permalink / raw) To: intel-gfx; +Cc: jani.nikula We've been accumulating code across the driver that depends on the VBT specific structures and defines. The VBT is an uncontrollable beast. Encourage encapsulation of the VBT data by hiding the structures and defines in a private header only to be included from intel_bios.c. Signed-off-by: Jani Nikula <jani.nikula@intel.com> --- Documentation/DocBook/gpu.tmpl | 2 +- drivers/gpu/drm/i915/intel_bios.c | 2 +- drivers/gpu/drm/i915/intel_bios.h | 860 ++---------------------------- drivers/gpu/drm/i915/intel_bios_private.h | 823 ++++++++++++++++++++++++++++ 4 files changed, 859 insertions(+), 828 deletions(-) create mode 100644 drivers/gpu/drm/i915/intel_bios_private.h diff --git a/Documentation/DocBook/gpu.tmpl b/Documentation/DocBook/gpu.tmpl index 351e801cead9..4ba7c6f8e8b1 100644 --- a/Documentation/DocBook/gpu.tmpl +++ b/Documentation/DocBook/gpu.tmpl @@ -3323,7 +3323,7 @@ int num_ioctls;</synopsis> <title>Video BIOS Table (VBT)</title> !Pdrivers/gpu/drm/i915/intel_bios.c Video BIOS Table (VBT) !Idrivers/gpu/drm/i915/intel_bios.c -!Idrivers/gpu/drm/i915/intel_bios.h +!Idrivers/gpu/drm/i915/intel_bios_private.h </sect2> </sect1> diff --git a/drivers/gpu/drm/i915/intel_bios.c b/drivers/gpu/drm/i915/intel_bios.c index 94c2baa50db7..83b78e0df306 100644 --- a/drivers/gpu/drm/i915/intel_bios.c +++ b/drivers/gpu/drm/i915/intel_bios.c @@ -29,7 +29,7 @@ #include <drm/drmP.h> #include <drm/i915_drm.h> #include "i915_drv.h" -#include "intel_bios.h" +#include "intel_bios_private.h" /** * DOC: Video BIOS Table (VBT) diff --git a/drivers/gpu/drm/i915/intel_bios.h b/drivers/gpu/drm/i915/intel_bios.h index 350d4e0f75a4..536d90f0c3eb 100644 --- a/drivers/gpu/drm/i915/intel_bios.h +++ b/drivers/gpu/drm/i915/intel_bios.h @@ -1,5 +1,5 @@ /* - * Copyright © 2006 Intel Corporation + * Copyright © 2016 Intel Corporation * * Permission is hereby granted, free of charge, to any person obtaining a * copy of this software and associated documentation files (the "Software"), @@ -19,543 +19,15 @@ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE * SOFTWARE. - * - * Authors: - * Eric Anholt <eric@anholt.net> - * - */ - -#ifndef _INTEL_BIOS_H_ -#define _INTEL_BIOS_H_ - -/** - * struct vbt_header - VBT Header structure - * @signature: VBT signature, always starts with "$VBT" - * @version: Version of this structure - * @header_size: Size of this structure - * @vbt_size: Size of VBT (VBT Header, BDB Header and data blocks) - * @vbt_checksum: Checksum - * @reserved0: Reserved - * @bdb_offset: Offset of &struct bdb_header from beginning of VBT - * @aim_offset: Offsets of add-in data blocks from beginning of VBT - */ -struct vbt_header { - u8 signature[20]; - u16 version; - u16 header_size; - u16 vbt_size; - u8 vbt_checksum; - u8 reserved0; - u32 bdb_offset; - u32 aim_offset[4]; -} __packed; - -/** - * struct bdb_header - BDB Header structure - * @signature: BDB signature "BIOS_DATA_BLOCK" - * @version: Version of the data block definitions - * @header_size: Size of this structure - * @bdb_size: Size of BDB (BDB Header and data blocks) - */ -struct bdb_header { - u8 signature[16]; - u16 version; - u16 header_size; - u16 bdb_size; -} __packed; - -/* strictly speaking, this is a "skip" block, but it has interesting info */ -struct vbios_data { - u8 type; /* 0 == desktop, 1 == mobile */ - u8 relstage; - u8 chipset; - u8 lvds_present:1; - u8 tv_present:1; - u8 rsvd2:6; /* finish byte */ - u8 rsvd3[4]; - u8 signon[155]; - u8 copyright[61]; - u16 code_segment; - u8 dos_boot_mode; - u8 bandwidth_percent; - u8 rsvd4; /* popup memory size */ - u8 resize_pci_bios; - u8 rsvd5; /* is crt already on ddc2 */ -} __packed; - -/* - * There are several types of BIOS data blocks (BDBs), each block has - * an ID and size in the first 3 bytes (ID in first, size in next 2). - * Known types are listed below. */ -#define BDB_GENERAL_FEATURES 1 -#define BDB_GENERAL_DEFINITIONS 2 -#define BDB_OLD_TOGGLE_LIST 3 -#define BDB_MODE_SUPPORT_LIST 4 -#define BDB_GENERIC_MODE_TABLE 5 -#define BDB_EXT_MMIO_REGS 6 -#define BDB_SWF_IO 7 -#define BDB_SWF_MMIO 8 -#define BDB_PSR 9 -#define BDB_MODE_REMOVAL_TABLE 10 -#define BDB_CHILD_DEVICE_TABLE 11 -#define BDB_DRIVER_FEATURES 12 -#define BDB_DRIVER_PERSISTENCE 13 -#define BDB_EXT_TABLE_PTRS 14 -#define BDB_DOT_CLOCK_OVERRIDE 15 -#define BDB_DISPLAY_SELECT 16 -/* 17 rsvd */ -#define BDB_DRIVER_ROTATION 18 -#define BDB_DISPLAY_REMOVE 19 -#define BDB_OEM_CUSTOM 20 -#define BDB_EFP_LIST 21 /* workarounds for VGA hsync/vsync */ -#define BDB_SDVO_LVDS_OPTIONS 22 -#define BDB_SDVO_PANEL_DTDS 23 -#define BDB_SDVO_LVDS_PNP_IDS 24 -#define BDB_SDVO_LVDS_POWER_SEQ 25 -#define BDB_TV_OPTIONS 26 -#define BDB_EDP 27 -#define BDB_LVDS_OPTIONS 40 -#define BDB_LVDS_LFP_DATA_PTRS 41 -#define BDB_LVDS_LFP_DATA 42 -#define BDB_LVDS_BACKLIGHT 43 -#define BDB_LVDS_POWER 44 -#define BDB_MIPI_CONFIG 52 -#define BDB_MIPI_SEQUENCE 53 -#define BDB_SKIP 254 /* VBIOS private block, ignore */ - -struct bdb_general_features { - /* bits 1 */ - u8 panel_fitting:2; - u8 flexaim:1; - u8 msg_enable:1; - u8 clear_screen:3; - u8 color_flip:1; - - /* bits 2 */ - u8 download_ext_vbt:1; - u8 enable_ssc:1; - u8 ssc_freq:1; - u8 enable_lfp_on_override:1; - u8 disable_ssc_ddt:1; - u8 rsvd7:1; - u8 display_clock_mode:1; - u8 rsvd8:1; /* finish byte */ - - /* bits 3 */ - u8 disable_smooth_vision:1; - u8 single_dvi:1; - u8 rsvd9:1; - u8 fdi_rx_polarity_inverted:1; - u8 rsvd10:4; /* finish byte */ - - /* bits 4 */ - u8 legacy_monitor_detect; - - /* bits 5 */ - u8 int_crt_support:1; - u8 int_tv_support:1; - u8 int_efp_support:1; - u8 dp_ssc_enb:1; /* PCH attached eDP supports SSC */ - u8 dp_ssc_freq:1; /* SSC freq for PCH attached eDP */ - u8 rsvd11:3; /* finish byte */ -} __packed; - -/* pre-915 */ -#define GPIO_PIN_DVI_LVDS 0x03 /* "DVI/LVDS DDC GPIO pins" */ -#define GPIO_PIN_ADD_I2C 0x05 /* "ADDCARD I2C GPIO pins" */ -#define GPIO_PIN_ADD_DDC 0x04 /* "ADDCARD DDC GPIO pins" */ -#define GPIO_PIN_ADD_DDC_I2C 0x06 /* "ADDCARD DDC/I2C GPIO pins" */ - -/* Pre 915 */ -#define DEVICE_TYPE_NONE 0x00 -#define DEVICE_TYPE_CRT 0x01 -#define DEVICE_TYPE_TV 0x09 -#define DEVICE_TYPE_EFP 0x12 -#define DEVICE_TYPE_LFP 0x22 -/* On 915+ */ -#define DEVICE_TYPE_CRT_DPMS 0x6001 -#define DEVICE_TYPE_CRT_DPMS_HOTPLUG 0x4001 -#define DEVICE_TYPE_TV_COMPOSITE 0x0209 -#define DEVICE_TYPE_TV_MACROVISION 0x0289 -#define DEVICE_TYPE_TV_RF_COMPOSITE 0x020c -#define DEVICE_TYPE_TV_SVIDEO_COMPOSITE 0x0609 -#define DEVICE_TYPE_TV_SCART 0x0209 -#define DEVICE_TYPE_TV_CODEC_HOTPLUG_PWR 0x6009 -#define DEVICE_TYPE_EFP_HOTPLUG_PWR 0x6012 -#define DEVICE_TYPE_EFP_DVI_HOTPLUG_PWR 0x6052 -#define DEVICE_TYPE_EFP_DVI_I 0x6053 -#define DEVICE_TYPE_EFP_DVI_D_DUAL 0x6152 -#define DEVICE_TYPE_EFP_DVI_D_HDCP 0x60d2 -#define DEVICE_TYPE_OPENLDI_HOTPLUG_PWR 0x6062 -#define DEVICE_TYPE_OPENLDI_DUALPIX 0x6162 -#define DEVICE_TYPE_LFP_PANELLINK 0x5012 -#define DEVICE_TYPE_LFP_CMOS_PWR 0x5042 -#define DEVICE_TYPE_LFP_LVDS_PWR 0x5062 -#define DEVICE_TYPE_LFP_LVDS_DUAL 0x5162 -#define DEVICE_TYPE_LFP_LVDS_DUAL_HDCP 0x51e2 - -#define DEVICE_CFG_NONE 0x00 -#define DEVICE_CFG_12BIT_DVOB 0x01 -#define DEVICE_CFG_12BIT_DVOC 0x02 -#define DEVICE_CFG_24BIT_DVOBC 0x09 -#define DEVICE_CFG_24BIT_DVOCB 0x0a -#define DEVICE_CFG_DUAL_DVOB 0x11 -#define DEVICE_CFG_DUAL_DVOC 0x12 -#define DEVICE_CFG_DUAL_DVOBC 0x13 -#define DEVICE_CFG_DUAL_LINK_DVOBC 0x19 -#define DEVICE_CFG_DUAL_LINK_DVOCB 0x1a - -#define DEVICE_WIRE_NONE 0x00 -#define DEVICE_WIRE_DVOB 0x01 -#define DEVICE_WIRE_DVOC 0x02 -#define DEVICE_WIRE_DVOBC 0x03 -#define DEVICE_WIRE_DVOBB 0x05 -#define DEVICE_WIRE_DVOCC 0x06 -#define DEVICE_WIRE_DVOB_MASTER 0x0d -#define DEVICE_WIRE_DVOC_MASTER 0x0e - -#define DEVICE_PORT_DVOA 0x00 /* none on 845+ */ -#define DEVICE_PORT_DVOB 0x01 -#define DEVICE_PORT_DVOC 0x02 /* - * We used to keep this struct but without any version control. We should avoid - * using it in the future, but it should be safe to keep using it in the old - * code. Do not change; we rely on its size. + * Please use intel_bios_private.h for VBT private data, and add the parsed data + * to struct intel_vbt_data. */ -struct old_child_dev_config { - u16 handle; - u16 device_type; - u8 device_id[10]; /* ascii string */ - u16 addin_offset; - u8 dvo_port; /* See Device_PORT_* above */ - u8 i2c_pin; - u8 slave_addr; - u8 ddc_pin; - u16 edid_ptr; - u8 dvo_cfg; /* See DEVICE_CFG_* above */ - u8 dvo2_port; - u8 i2c2_pin; - u8 slave2_addr; - u8 ddc2_pin; - u8 capabilities; - u8 dvo_wiring;/* See DEVICE_WIRE_* above */ - u8 dvo2_wiring; - u16 extended_type; - u8 dvo_function; -} __packed; - -/* This one contains field offsets that are known to be common for all BDB - * versions. Notice that the meaning of the contents contents may still change, - * but at least the offsets are consistent. */ - -/* Definitions for flags_1 */ -#define IBOOST_ENABLE (1<<3) - -struct common_child_dev_config { - u16 handle; - u16 device_type; - u8 not_common1[12]; - u8 dvo_port; - u8 not_common2[2]; - u8 ddc_pin; - u16 edid_ptr; - u8 obsolete; - u8 flags_1; - u8 not_common3[13]; - u8 iboost_level; -} __packed; - - -/* This field changes depending on the BDB version, so the most reliable way to - * read it is by checking the BDB version and reading the raw pointer. */ -union child_device_config { - /* This one is safe to be used anywhere, but the code should still check - * the BDB version. */ - u8 raw[33]; - /* This one should only be kept for legacy code. */ - struct old_child_dev_config old; - /* This one should also be safe to use anywhere, even without version - * checks. */ - struct common_child_dev_config common; -} __packed; - -struct bdb_general_definitions { - /* DDC GPIO */ - u8 crt_ddc_gmbus_pin; - - /* DPMS bits */ - u8 dpms_acpi:1; - u8 skip_boot_crt_detect:1; - u8 dpms_aim:1; - u8 rsvd1:5; /* finish byte */ - - /* boot device bits */ - u8 boot_display[2]; - u8 child_dev_size; - - /* - * Device info: - * If TV is present, it'll be at devices[0]. - * LVDS will be next, either devices[0] or [1], if present. - * On some platforms the number of device is 6. But could be as few as - * 4 if both TV and LVDS are missing. - * And the device num is related with the size of general definition - * block. It is obtained by using the following formula: - * number = (block_size - sizeof(bdb_general_definitions))/ - * defs->child_dev_size; - */ - uint8_t devices[0]; -} __packed; - -/* Mask for DRRS / Panel Channel / SSC / BLT control bits extraction */ -#define MODE_MASK 0x3 - -struct bdb_lvds_options { - u8 panel_type; - u8 rsvd1; - /* LVDS capabilities, stored in a dword */ - u8 pfit_mode:2; - u8 pfit_text_mode_enhanced:1; - u8 pfit_gfx_mode_enhanced:1; - u8 pfit_ratio_auto:1; - u8 pixel_dither:1; - u8 lvds_edid:1; - u8 rsvd2:1; - u8 rsvd4; - /* LVDS Panel channel bits stored here */ - u32 lvds_panel_channel_bits; - /* LVDS SSC (Spread Spectrum Clock) bits stored here. */ - u16 ssc_bits; - u16 ssc_freq; - u16 ssc_ddt; - /* Panel color depth defined here */ - u16 panel_color_depth; - /* LVDS panel type bits stored here */ - u32 dps_panel_type_bits; - /* LVDS backlight control type bits stored here */ - u32 blt_control_type_bits; -} __packed; - -/* LFP pointer table contains entries to the struct below */ -struct bdb_lvds_lfp_data_ptr { - u16 fp_timing_offset; /* offsets are from start of bdb */ - u8 fp_table_size; - u16 dvo_timing_offset; - u8 dvo_table_size; - u16 panel_pnp_id_offset; - u8 pnp_table_size; -} __packed; - -struct bdb_lvds_lfp_data_ptrs { - u8 lvds_entries; /* followed by one or more lvds_data_ptr structs */ - struct bdb_lvds_lfp_data_ptr ptr[16]; -} __packed; - -/* LFP data has 3 blocks per entry */ -struct lvds_fp_timing { - u16 x_res; - u16 y_res; - u32 lvds_reg; - u32 lvds_reg_val; - u32 pp_on_reg; - u32 pp_on_reg_val; - u32 pp_off_reg; - u32 pp_off_reg_val; - u32 pp_cycle_reg; - u32 pp_cycle_reg_val; - u32 pfit_reg; - u32 pfit_reg_val; - u16 terminator; -} __packed; - -struct lvds_dvo_timing { - u16 clock; /**< In 10khz */ - u8 hactive_lo; - u8 hblank_lo; - u8 hblank_hi:4; - u8 hactive_hi:4; - u8 vactive_lo; - u8 vblank_lo; - u8 vblank_hi:4; - u8 vactive_hi:4; - u8 hsync_off_lo; - u8 hsync_pulse_width; - u8 vsync_pulse_width:4; - u8 vsync_off:4; - u8 rsvd0:6; - u8 hsync_off_hi:2; - u8 h_image; - u8 v_image; - u8 max_hv; - u8 h_border; - u8 v_border; - u8 rsvd1:3; - u8 digital:2; - u8 vsync_positive:1; - u8 hsync_positive:1; - u8 rsvd2:1; -} __packed; - -struct lvds_pnp_id { - u16 mfg_name; - u16 product_code; - u32 serial; - u8 mfg_week; - u8 mfg_year; -} __packed; - -struct bdb_lvds_lfp_data_entry { - struct lvds_fp_timing fp_timing; - struct lvds_dvo_timing dvo_timing; - struct lvds_pnp_id pnp_id; -} __packed; - -struct bdb_lvds_lfp_data { - struct bdb_lvds_lfp_data_entry data[16]; -} __packed; - -#define BDB_BACKLIGHT_TYPE_NONE 0 -#define BDB_BACKLIGHT_TYPE_PWM 2 - -struct bdb_lfp_backlight_data_entry { - u8 type:2; - u8 active_low_pwm:1; - u8 obsolete1:5; - u16 pwm_freq_hz; - u8 min_brightness; - u8 obsolete2; - u8 obsolete3; -} __packed; - -struct bdb_lfp_backlight_data { - u8 entry_size; - struct bdb_lfp_backlight_data_entry data[16]; - u8 level[16]; -} __packed; - -struct aimdb_header { - char signature[16]; - char oem_device[20]; - u16 aimdb_version; - u16 aimdb_header_size; - u16 aimdb_size; -} __packed; - -struct aimdb_block { - u8 aimdb_id; - u16 aimdb_size; -} __packed; -struct vch_panel_data { - u16 fp_timing_offset; - u8 fp_timing_size; - u16 dvo_timing_offset; - u8 dvo_timing_size; - u16 text_fitting_offset; - u8 text_fitting_size; - u16 graphics_fitting_offset; - u8 graphics_fitting_size; -} __packed; - -struct vch_bdb_22 { - struct aimdb_block aimdb_block; - struct vch_panel_data panels[16]; -} __packed; - -struct bdb_sdvo_lvds_options { - u8 panel_backlight; - u8 h40_set_panel_type; - u8 panel_type; - u8 ssc_clk_freq; - u16 als_low_trip; - u16 als_high_trip; - u8 sclalarcoeff_tab_row_num; - u8 sclalarcoeff_tab_row_size; - u8 coefficient[8]; - u8 panel_misc_bits_1; - u8 panel_misc_bits_2; - u8 panel_misc_bits_3; - u8 panel_misc_bits_4; -} __packed; - - -#define BDB_DRIVER_FEATURE_NO_LVDS 0 -#define BDB_DRIVER_FEATURE_INT_LVDS 1 -#define BDB_DRIVER_FEATURE_SDVO_LVDS 2 -#define BDB_DRIVER_FEATURE_EDP 3 - -struct bdb_driver_features { - u8 boot_dev_algorithm:1; - u8 block_display_switch:1; - u8 allow_display_switch:1; - u8 hotplug_dvo:1; - u8 dual_view_zoom:1; - u8 int15h_hook:1; - u8 sprite_in_clone:1; - u8 primary_lfp_id:1; - - u16 boot_mode_x; - u16 boot_mode_y; - u8 boot_mode_bpp; - u8 boot_mode_refresh; - - u16 enable_lfp_primary:1; - u16 selective_mode_pruning:1; - u16 dual_frequency:1; - u16 render_clock_freq:1; /* 0: high freq; 1: low freq */ - u16 nt_clone_support:1; - u16 power_scheme_ui:1; /* 0: CUI; 1: 3rd party */ - u16 sprite_display_assign:1; /* 0: secondary; 1: primary */ - u16 cui_aspect_scaling:1; - u16 preserve_aspect_ratio:1; - u16 sdvo_device_power_down:1; - u16 crt_hotplug:1; - u16 lvds_config:2; - u16 tv_hotplug:1; - u16 hdmi_config:2; - - u8 static_display:1; - u8 reserved2:7; - u16 legacy_crt_max_x; - u16 legacy_crt_max_y; - u8 legacy_crt_max_refresh; - - u8 hdmi_termination; - u8 custom_vbt_version; - /* Driver features data block */ - u16 rmpm_enabled:1; - u16 s2ddt_enabled:1; - u16 dpst_enabled:1; - u16 bltclt_enabled:1; - u16 adb_enabled:1; - u16 drrs_enabled:1; - u16 grs_enabled:1; - u16 gpmt_enabled:1; - u16 tbt_enabled:1; - u16 psr_enabled:1; - u16 ips_enabled:1; - u16 reserved3:4; - u16 pc_feature_valid:1; -} __packed; - -#define EDP_18BPP 0 -#define EDP_24BPP 1 -#define EDP_30BPP 2 -#define EDP_RATE_1_62 0 -#define EDP_RATE_2_7 1 -#define EDP_LANE_1 0 -#define EDP_LANE_2 1 -#define EDP_LANE_4 3 -#define EDP_PREEMPHASIS_NONE 0 -#define EDP_PREEMPHASIS_3_5dB 1 -#define EDP_PREEMPHASIS_6dB 2 -#define EDP_PREEMPHASIS_9_5dB 3 -#define EDP_VSWING_0_4V 0 -#define EDP_VSWING_0_6V 1 -#define EDP_VSWING_0_8V 2 -#define EDP_VSWING_1_2V 3 +#ifndef _INTEL_BIOS_H_ +#define _INTEL_BIOS_H_ struct edp_power_seq { u16 t1_t3; @@ -565,245 +37,37 @@ struct edp_power_seq { u16 t11_t12; } __packed; -struct edp_link_params { - u8 rate:4; - u8 lanes:4; - u8 preemphasis:4; - u8 vswing:4; -} __packed; - -struct bdb_edp { - struct edp_power_seq power_seqs[16]; - u32 color_depth; - struct edp_link_params link_params[16]; - u32 sdrrs_msa_timing_delay; - - /* ith bit indicates enabled/disabled for (i+1)th panel */ - u16 edp_s3d_feature; - u16 edp_t3_optimization; - u64 edp_vswing_preemph; /* v173 */ -} __packed; - -struct psr_table { - /* Feature bits */ - u8 full_link:1; - u8 require_aux_to_wakeup:1; - u8 feature_bits_rsvd:6; - - /* Wait times */ - u8 idle_frames:4; - u8 lines_to_wait:3; - u8 wait_times_rsvd:1; - - /* TP wake up time in multiple of 100 */ - u16 tp1_wakeup_time; - u16 tp2_tp3_wakeup_time; -} __packed; - -struct bdb_psr { - struct psr_table psr_table[16]; -} __packed; - -/* - * Driver<->VBIOS interaction occurs through scratch bits in - * GR18 & SWF*. - */ - -/* GR18 bits are set on display switch and hotkey events */ -#define GR18_DRIVER_SWITCH_EN (1<<7) /* 0: VBIOS control, 1: driver control */ -#define GR18_HOTKEY_MASK 0x78 /* See also SWF4 15:0 */ -#define GR18_HK_NONE (0x0<<3) -#define GR18_HK_LFP_STRETCH (0x1<<3) -#define GR18_HK_TOGGLE_DISP (0x2<<3) -#define GR18_HK_DISP_SWITCH (0x4<<3) /* see SWF14 15:0 for what to enable */ -#define GR18_HK_POPUP_DISABLED (0x6<<3) -#define GR18_HK_POPUP_ENABLED (0x7<<3) -#define GR18_HK_PFIT (0x8<<3) -#define GR18_HK_APM_CHANGE (0xa<<3) -#define GR18_HK_MULTIPLE (0xc<<3) -#define GR18_USER_INT_EN (1<<2) -#define GR18_A0000_FLUSH_EN (1<<1) -#define GR18_SMM_EN (1<<0) - -/* Set by driver, cleared by VBIOS */ -#define SWF00_YRES_SHIFT 16 -#define SWF00_XRES_SHIFT 0 -#define SWF00_RES_MASK 0xffff - -/* Set by VBIOS at boot time and driver at runtime */ -#define SWF01_TV2_FORMAT_SHIFT 8 -#define SWF01_TV1_FORMAT_SHIFT 0 -#define SWF01_TV_FORMAT_MASK 0xffff - -#define SWF10_VBIOS_BLC_I2C_EN (1<<29) -#define SWF10_GTT_OVERRIDE_EN (1<<28) -#define SWF10_LFP_DPMS_OVR (1<<27) /* override DPMS on display switch */ -#define SWF10_ACTIVE_TOGGLE_LIST_MASK (7<<24) -#define SWF10_OLD_TOGGLE 0x0 -#define SWF10_TOGGLE_LIST_1 0x1 -#define SWF10_TOGGLE_LIST_2 0x2 -#define SWF10_TOGGLE_LIST_3 0x3 -#define SWF10_TOGGLE_LIST_4 0x4 -#define SWF10_PANNING_EN (1<<23) -#define SWF10_DRIVER_LOADED (1<<22) -#define SWF10_EXTENDED_DESKTOP (1<<21) -#define SWF10_EXCLUSIVE_MODE (1<<20) -#define SWF10_OVERLAY_EN (1<<19) -#define SWF10_PLANEB_HOLDOFF (1<<18) -#define SWF10_PLANEA_HOLDOFF (1<<17) -#define SWF10_VGA_HOLDOFF (1<<16) -#define SWF10_ACTIVE_DISP_MASK 0xffff -#define SWF10_PIPEB_LFP2 (1<<15) -#define SWF10_PIPEB_EFP2 (1<<14) -#define SWF10_PIPEB_TV2 (1<<13) -#define SWF10_PIPEB_CRT2 (1<<12) -#define SWF10_PIPEB_LFP (1<<11) -#define SWF10_PIPEB_EFP (1<<10) -#define SWF10_PIPEB_TV (1<<9) -#define SWF10_PIPEB_CRT (1<<8) -#define SWF10_PIPEA_LFP2 (1<<7) -#define SWF10_PIPEA_EFP2 (1<<6) -#define SWF10_PIPEA_TV2 (1<<5) -#define SWF10_PIPEA_CRT2 (1<<4) -#define SWF10_PIPEA_LFP (1<<3) -#define SWF10_PIPEA_EFP (1<<2) -#define SWF10_PIPEA_TV (1<<1) -#define SWF10_PIPEA_CRT (1<<0) - -#define SWF11_MEMORY_SIZE_SHIFT 16 -#define SWF11_SV_TEST_EN (1<<15) -#define SWF11_IS_AGP (1<<14) -#define SWF11_DISPLAY_HOLDOFF (1<<13) -#define SWF11_DPMS_REDUCED (1<<12) -#define SWF11_IS_VBE_MODE (1<<11) -#define SWF11_PIPEB_ACCESS (1<<10) /* 0 here means pipe a */ -#define SWF11_DPMS_MASK 0x07 -#define SWF11_DPMS_OFF (1<<2) -#define SWF11_DPMS_SUSPEND (1<<1) -#define SWF11_DPMS_STANDBY (1<<0) -#define SWF11_DPMS_ON 0 - -#define SWF14_GFX_PFIT_EN (1<<31) -#define SWF14_TEXT_PFIT_EN (1<<30) -#define SWF14_LID_STATUS_CLOSED (1<<29) /* 0 here means open */ -#define SWF14_POPUP_EN (1<<28) -#define SWF14_DISPLAY_HOLDOFF (1<<27) -#define SWF14_DISP_DETECT_EN (1<<26) -#define SWF14_DOCKING_STATUS_DOCKED (1<<25) /* 0 here means undocked */ -#define SWF14_DRIVER_STATUS (1<<24) -#define SWF14_OS_TYPE_WIN9X (1<<23) -#define SWF14_OS_TYPE_WINNT (1<<22) -/* 21:19 rsvd */ -#define SWF14_PM_TYPE_MASK 0x00070000 -#define SWF14_PM_ACPI_VIDEO (0x4 << 16) -#define SWF14_PM_ACPI (0x3 << 16) -#define SWF14_PM_APM_12 (0x2 << 16) -#define SWF14_PM_APM_11 (0x1 << 16) -#define SWF14_HK_REQUEST_MASK 0x0000ffff /* see GR18 6:3 for event type */ - /* if GR18 indicates a display switch */ -#define SWF14_DS_PIPEB_LFP2_EN (1<<15) -#define SWF14_DS_PIPEB_EFP2_EN (1<<14) -#define SWF14_DS_PIPEB_TV2_EN (1<<13) -#define SWF14_DS_PIPEB_CRT2_EN (1<<12) -#define SWF14_DS_PIPEB_LFP_EN (1<<11) -#define SWF14_DS_PIPEB_EFP_EN (1<<10) -#define SWF14_DS_PIPEB_TV_EN (1<<9) -#define SWF14_DS_PIPEB_CRT_EN (1<<8) -#define SWF14_DS_PIPEA_LFP2_EN (1<<7) -#define SWF14_DS_PIPEA_EFP2_EN (1<<6) -#define SWF14_DS_PIPEA_TV2_EN (1<<5) -#define SWF14_DS_PIPEA_CRT2_EN (1<<4) -#define SWF14_DS_PIPEA_LFP_EN (1<<3) -#define SWF14_DS_PIPEA_EFP_EN (1<<2) -#define SWF14_DS_PIPEA_TV_EN (1<<1) -#define SWF14_DS_PIPEA_CRT_EN (1<<0) - /* if GR18 indicates a panel fitting request */ -#define SWF14_PFIT_EN (1<<0) /* 0 means disable */ - /* if GR18 indicates an APM change request */ -#define SWF14_APM_HIBERNATE 0x4 -#define SWF14_APM_SUSPEND 0x3 -#define SWF14_APM_STANDBY 0x1 -#define SWF14_APM_RESTORE 0x0 - -/* Add the device class for LFP, TV, HDMI */ -#define DEVICE_TYPE_INT_LFP 0x1022 -#define DEVICE_TYPE_INT_TV 0x1009 -#define DEVICE_TYPE_HDMI 0x60D2 -#define DEVICE_TYPE_DP 0x68C6 -#define DEVICE_TYPE_eDP 0x78C6 - -#define DEVICE_TYPE_CLASS_EXTENSION (1 << 15) -#define DEVICE_TYPE_POWER_MANAGEMENT (1 << 14) -#define DEVICE_TYPE_HOTPLUG_SIGNALING (1 << 13) -#define DEVICE_TYPE_INTERNAL_CONNECTOR (1 << 12) -#define DEVICE_TYPE_NOT_HDMI_OUTPUT (1 << 11) -#define DEVICE_TYPE_MIPI_OUTPUT (1 << 10) -#define DEVICE_TYPE_COMPOSITE_OUTPUT (1 << 9) -#define DEVICE_TYPE_DUAL_CHANNEL (1 << 8) -#define DEVICE_TYPE_HIGH_SPEED_LINK (1 << 6) -#define DEVICE_TYPE_LVDS_SINGALING (1 << 5) -#define DEVICE_TYPE_TMDS_DVI_SIGNALING (1 << 4) -#define DEVICE_TYPE_VIDEO_SIGNALING (1 << 3) -#define DEVICE_TYPE_DISPLAYPORT_OUTPUT (1 << 2) -#define DEVICE_TYPE_DIGITAL_OUTPUT (1 << 1) -#define DEVICE_TYPE_ANALOG_OUTPUT (1 << 0) - -/* - * Bits we care about when checking for DEVICE_TYPE_eDP - * Depending on the system, the other bits may or may not - * be set for eDP outputs. - */ -#define DEVICE_TYPE_eDP_BITS \ - (DEVICE_TYPE_INTERNAL_CONNECTOR | \ - DEVICE_TYPE_MIPI_OUTPUT | \ - DEVICE_TYPE_COMPOSITE_OUTPUT | \ - DEVICE_TYPE_DUAL_CHANNEL | \ - DEVICE_TYPE_LVDS_SINGALING | \ - DEVICE_TYPE_TMDS_DVI_SIGNALING | \ - DEVICE_TYPE_VIDEO_SIGNALING | \ - DEVICE_TYPE_DISPLAYPORT_OUTPUT | \ - DEVICE_TYPE_ANALOG_OUTPUT) - -/* define the DVO port for HDMI output type */ -#define DVO_B 1 -#define DVO_C 2 -#define DVO_D 3 - -/* Possible values for the "DVO Port" field for versions >= 155: */ -#define DVO_PORT_HDMIA 0 -#define DVO_PORT_HDMIB 1 -#define DVO_PORT_HDMIC 2 -#define DVO_PORT_HDMID 3 -#define DVO_PORT_LVDS 4 -#define DVO_PORT_TV 5 -#define DVO_PORT_CRT 6 -#define DVO_PORT_DPB 7 -#define DVO_PORT_DPC 8 -#define DVO_PORT_DPD 9 -#define DVO_PORT_DPA 10 -#define DVO_PORT_DPE 11 -#define DVO_PORT_HDMIE 12 -#define DVO_PORT_MIPIA 21 -#define DVO_PORT_MIPIB 22 -#define DVO_PORT_MIPIC 23 -#define DVO_PORT_MIPID 24 +/* MIPI Sequence Block definitions */ +enum mipi_seq { + MIPI_SEQ_END = 0, + MIPI_SEQ_ASSERT_RESET, + MIPI_SEQ_INIT_OTP, + MIPI_SEQ_DISPLAY_ON, + MIPI_SEQ_DISPLAY_OFF, + MIPI_SEQ_DEASSERT_RESET, + MIPI_SEQ_BACKLIGHT_ON, /* sequence block v2+ */ + MIPI_SEQ_BACKLIGHT_OFF, /* sequence block v2+ */ + MIPI_SEQ_TEAR_ON, /* sequence block v2+ */ + MIPI_SEQ_TEAR_OFF, /* sequence block v3+ */ + MIPI_SEQ_POWER_ON, /* sequence block v3+ */ + MIPI_SEQ_POWER_OFF, /* sequence block v3+ */ + MIPI_SEQ_MAX +}; -/* Block 52 contains MIPI Panel info - * 6 such enteries will there. Index into correct - * entery is based on the panel_index in #40 LFP - */ -#define MAX_MIPI_CONFIGURATIONS 6 +enum mipi_seq_element { + MIPI_SEQ_ELEM_END = 0, + MIPI_SEQ_ELEM_SEND_PKT, + MIPI_SEQ_ELEM_DELAY, + MIPI_SEQ_ELEM_GPIO, + MIPI_SEQ_ELEM_I2C, /* sequence block v2+ */ + MIPI_SEQ_ELEM_SPI, /* sequence block v3+ */ + MIPI_SEQ_ELEM_PMIC, /* sequence block v3+ */ + MIPI_SEQ_ELEM_MAX +}; #define MIPI_DSI_UNDEFINED_PANEL_ID 0 #define MIPI_DSI_GENERIC_PANEL_ID 1 -/* - * PMIC vs SoC Backlight support specified in pwm_blc - * field in mipi_config block below. -*/ -#define PPS_BLC_PMIC 0 -#define PPS_BLC_SOC 1 - struct mipi_config { u16 panel_id; @@ -821,6 +85,8 @@ struct mipi_config { u32 video_transfer_mode:2; u32 cabc_supported:1; +#define PPS_BLC_PMIC 0 +#define PPS_BLC_SOC 1 u32 pwm_blc:1; /* Bit 13:10 */ @@ -924,12 +190,7 @@ struct mipi_config { } __packed; -/* Block 52 contains MIPI configuration block - * 6 * bdb_mipi_config, followed by 6 pps data - * block below - * - * all delays has a unit of 100us - */ +/* all delays have a unit of 100us */ struct mipi_pps_data { u16 panel_on_delay; u16 bl_enable_delay; @@ -938,57 +199,4 @@ struct mipi_pps_data { u16 panel_power_cycle_delay; } __packed; -struct bdb_mipi_config { - struct mipi_config config[MAX_MIPI_CONFIGURATIONS]; - struct mipi_pps_data pps[MAX_MIPI_CONFIGURATIONS]; -} __packed; - -/* Block 53 contains MIPI sequences as needed by the panel - * for enabling it. This block can be variable in size and - * can be maximum of 6 blocks - */ -struct bdb_mipi_sequence { - u8 version; - u8 data[0]; -} __packed; - -/* MIPI Sequnece Block definitions */ -enum mipi_seq { - MIPI_SEQ_END = 0, - MIPI_SEQ_ASSERT_RESET, - MIPI_SEQ_INIT_OTP, - MIPI_SEQ_DISPLAY_ON, - MIPI_SEQ_DISPLAY_OFF, - MIPI_SEQ_DEASSERT_RESET, - MIPI_SEQ_BACKLIGHT_ON, /* sequence block v2+ */ - MIPI_SEQ_BACKLIGHT_OFF, /* sequence block v2+ */ - MIPI_SEQ_TEAR_ON, /* sequence block v2+ */ - MIPI_SEQ_TEAR_OFF, /* sequence block v3+ */ - MIPI_SEQ_POWER_ON, /* sequence block v3+ */ - MIPI_SEQ_POWER_OFF, /* sequence block v3+ */ - MIPI_SEQ_MAX -}; - -enum mipi_seq_element { - MIPI_SEQ_ELEM_END = 0, - MIPI_SEQ_ELEM_SEND_PKT, - MIPI_SEQ_ELEM_DELAY, - MIPI_SEQ_ELEM_GPIO, - MIPI_SEQ_ELEM_I2C, /* sequence block v2+ */ - MIPI_SEQ_ELEM_SPI, /* sequence block v3+ */ - MIPI_SEQ_ELEM_PMIC, /* sequence block v3+ */ - MIPI_SEQ_ELEM_MAX -}; - -enum mipi_gpio_pin_index { - MIPI_GPIO_UNDEFINED = 0, - MIPI_GPIO_PANEL_ENABLE, - MIPI_GPIO_BL_ENABLE, - MIPI_GPIO_PWM_ENABLE, - MIPI_GPIO_RESET_N, - MIPI_GPIO_PWR_DOWN_R, - MIPI_GPIO_STDBY_RST_N, - MIPI_GPIO_MAX -}; - #endif /* _INTEL_BIOS_H_ */ diff --git a/drivers/gpu/drm/i915/intel_bios_private.h b/drivers/gpu/drm/i915/intel_bios_private.h new file mode 100644 index 000000000000..56d3aecae064 --- /dev/null +++ b/drivers/gpu/drm/i915/intel_bios_private.h @@ -0,0 +1,823 @@ +/* + * Copyright © 2006-2016 Intel Corporation + * + * Permission is hereby granted, free of charge, to any person obtaining a + * copy of this software and associated documentation files (the "Software"), + * to deal in the Software without restriction, including without limitation + * the rights to use, copy, modify, merge, publish, distribute, sublicense, + * and/or sell copies of the Software, and to permit persons to whom the + * Software is furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice (including the next + * paragraph) shall be included in all copies or substantial portions of the + * Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE + * SOFTWARE. + * + * Authors: + * Eric Anholt <eric@anholt.net> + * + */ + +/* + * This information is private to VBT parsing in intel_bios.c. + * + * Please do NOT include anywhere else. + */ + +#ifndef _INTEL_BIOS_PRIVATE_H_ +#define _INTEL_BIOS_PRIVATE_H_ + +#include "intel_bios.h" + +/** + * struct vbt_header - VBT Header structure + * @signature: VBT signature, always starts with "$VBT" + * @version: Version of this structure + * @header_size: Size of this structure + * @vbt_size: Size of VBT (VBT Header, BDB Header and data blocks) + * @vbt_checksum: Checksum + * @reserved0: Reserved + * @bdb_offset: Offset of &struct bdb_header from beginning of VBT + * @aim_offset: Offsets of add-in data blocks from beginning of VBT + */ +struct vbt_header { + u8 signature[20]; + u16 version; + u16 header_size; + u16 vbt_size; + u8 vbt_checksum; + u8 reserved0; + u32 bdb_offset; + u32 aim_offset[4]; +} __packed; + +/** + * struct bdb_header - BDB Header structure + * @signature: BDB signature "BIOS_DATA_BLOCK" + * @version: Version of the data block definitions + * @header_size: Size of this structure + * @bdb_size: Size of BDB (BDB Header and data blocks) + */ +struct bdb_header { + u8 signature[16]; + u16 version; + u16 header_size; + u16 bdb_size; +} __packed; + +/* strictly speaking, this is a "skip" block, but it has interesting info */ +struct vbios_data { + u8 type; /* 0 == desktop, 1 == mobile */ + u8 relstage; + u8 chipset; + u8 lvds_present:1; + u8 tv_present:1; + u8 rsvd2:6; /* finish byte */ + u8 rsvd3[4]; + u8 signon[155]; + u8 copyright[61]; + u16 code_segment; + u8 dos_boot_mode; + u8 bandwidth_percent; + u8 rsvd4; /* popup memory size */ + u8 resize_pci_bios; + u8 rsvd5; /* is crt already on ddc2 */ +} __packed; + +/* + * There are several types of BIOS data blocks (BDBs), each block has + * an ID and size in the first 3 bytes (ID in first, size in next 2). + * Known types are listed below. + */ +#define BDB_GENERAL_FEATURES 1 +#define BDB_GENERAL_DEFINITIONS 2 +#define BDB_OLD_TOGGLE_LIST 3 +#define BDB_MODE_SUPPORT_LIST 4 +#define BDB_GENERIC_MODE_TABLE 5 +#define BDB_EXT_MMIO_REGS 6 +#define BDB_SWF_IO 7 +#define BDB_SWF_MMIO 8 +#define BDB_PSR 9 +#define BDB_MODE_REMOVAL_TABLE 10 +#define BDB_CHILD_DEVICE_TABLE 11 +#define BDB_DRIVER_FEATURES 12 +#define BDB_DRIVER_PERSISTENCE 13 +#define BDB_EXT_TABLE_PTRS 14 +#define BDB_DOT_CLOCK_OVERRIDE 15 +#define BDB_DISPLAY_SELECT 16 +/* 17 rsvd */ +#define BDB_DRIVER_ROTATION 18 +#define BDB_DISPLAY_REMOVE 19 +#define BDB_OEM_CUSTOM 20 +#define BDB_EFP_LIST 21 /* workarounds for VGA hsync/vsync */ +#define BDB_SDVO_LVDS_OPTIONS 22 +#define BDB_SDVO_PANEL_DTDS 23 +#define BDB_SDVO_LVDS_PNP_IDS 24 +#define BDB_SDVO_LVDS_POWER_SEQ 25 +#define BDB_TV_OPTIONS 26 +#define BDB_EDP 27 +#define BDB_LVDS_OPTIONS 40 +#define BDB_LVDS_LFP_DATA_PTRS 41 +#define BDB_LVDS_LFP_DATA 42 +#define BDB_LVDS_BACKLIGHT 43 +#define BDB_LVDS_POWER 44 +#define BDB_MIPI_CONFIG 52 +#define BDB_MIPI_SEQUENCE 53 +#define BDB_SKIP 254 /* VBIOS private block, ignore */ + +struct bdb_general_features { + /* bits 1 */ + u8 panel_fitting:2; + u8 flexaim:1; + u8 msg_enable:1; + u8 clear_screen:3; + u8 color_flip:1; + + /* bits 2 */ + u8 download_ext_vbt:1; + u8 enable_ssc:1; + u8 ssc_freq:1; + u8 enable_lfp_on_override:1; + u8 disable_ssc_ddt:1; + u8 rsvd7:1; + u8 display_clock_mode:1; + u8 rsvd8:1; /* finish byte */ + + /* bits 3 */ + u8 disable_smooth_vision:1; + u8 single_dvi:1; + u8 rsvd9:1; + u8 fdi_rx_polarity_inverted:1; + u8 rsvd10:4; /* finish byte */ + + /* bits 4 */ + u8 legacy_monitor_detect; + + /* bits 5 */ + u8 int_crt_support:1; + u8 int_tv_support:1; + u8 int_efp_support:1; + u8 dp_ssc_enb:1; /* PCH attached eDP supports SSC */ + u8 dp_ssc_freq:1; /* SSC freq for PCH attached eDP */ + u8 rsvd11:3; /* finish byte */ +} __packed; + +/* pre-915 */ +#define GPIO_PIN_DVI_LVDS 0x03 /* "DVI/LVDS DDC GPIO pins" */ +#define GPIO_PIN_ADD_I2C 0x05 /* "ADDCARD I2C GPIO pins" */ +#define GPIO_PIN_ADD_DDC 0x04 /* "ADDCARD DDC GPIO pins" */ +#define GPIO_PIN_ADD_DDC_I2C 0x06 /* "ADDCARD DDC/I2C GPIO pins" */ + +/* Pre 915 */ +#define DEVICE_TYPE_NONE 0x00 +#define DEVICE_TYPE_CRT 0x01 +#define DEVICE_TYPE_TV 0x09 +#define DEVICE_TYPE_EFP 0x12 +#define DEVICE_TYPE_LFP 0x22 +/* On 915+ */ +#define DEVICE_TYPE_CRT_DPMS 0x6001 +#define DEVICE_TYPE_CRT_DPMS_HOTPLUG 0x4001 +#define DEVICE_TYPE_TV_COMPOSITE 0x0209 +#define DEVICE_TYPE_TV_MACROVISION 0x0289 +#define DEVICE_TYPE_TV_RF_COMPOSITE 0x020c +#define DEVICE_TYPE_TV_SVIDEO_COMPOSITE 0x0609 +#define DEVICE_TYPE_TV_SCART 0x0209 +#define DEVICE_TYPE_TV_CODEC_HOTPLUG_PWR 0x6009 +#define DEVICE_TYPE_EFP_HOTPLUG_PWR 0x6012 +#define DEVICE_TYPE_EFP_DVI_HOTPLUG_PWR 0x6052 +#define DEVICE_TYPE_EFP_DVI_I 0x6053 +#define DEVICE_TYPE_EFP_DVI_D_DUAL 0x6152 +#define DEVICE_TYPE_EFP_DVI_D_HDCP 0x60d2 +#define DEVICE_TYPE_OPENLDI_HOTPLUG_PWR 0x6062 +#define DEVICE_TYPE_OPENLDI_DUALPIX 0x6162 +#define DEVICE_TYPE_LFP_PANELLINK 0x5012 +#define DEVICE_TYPE_LFP_CMOS_PWR 0x5042 +#define DEVICE_TYPE_LFP_LVDS_PWR 0x5062 +#define DEVICE_TYPE_LFP_LVDS_DUAL 0x5162 +#define DEVICE_TYPE_LFP_LVDS_DUAL_HDCP 0x51e2 + +#define DEVICE_CFG_NONE 0x00 +#define DEVICE_CFG_12BIT_DVOB 0x01 +#define DEVICE_CFG_12BIT_DVOC 0x02 +#define DEVICE_CFG_24BIT_DVOBC 0x09 +#define DEVICE_CFG_24BIT_DVOCB 0x0a +#define DEVICE_CFG_DUAL_DVOB 0x11 +#define DEVICE_CFG_DUAL_DVOC 0x12 +#define DEVICE_CFG_DUAL_DVOBC 0x13 +#define DEVICE_CFG_DUAL_LINK_DVOBC 0x19 +#define DEVICE_CFG_DUAL_LINK_DVOCB 0x1a + +#define DEVICE_WIRE_NONE 0x00 +#define DEVICE_WIRE_DVOB 0x01 +#define DEVICE_WIRE_DVOC 0x02 +#define DEVICE_WIRE_DVOBC 0x03 +#define DEVICE_WIRE_DVOBB 0x05 +#define DEVICE_WIRE_DVOCC 0x06 +#define DEVICE_WIRE_DVOB_MASTER 0x0d +#define DEVICE_WIRE_DVOC_MASTER 0x0e + +#define DEVICE_PORT_DVOA 0x00 /* none on 845+ */ +#define DEVICE_PORT_DVOB 0x01 +#define DEVICE_PORT_DVOC 0x02 + +/* + * We used to keep this struct but without any version control. We should avoid + * using it in the future, but it should be safe to keep using it in the old + * code. Do not change; we rely on its size. + */ +struct old_child_dev_config { + u16 handle; + u16 device_type; + u8 device_id[10]; /* ascii string */ + u16 addin_offset; + u8 dvo_port; /* See Device_PORT_* above */ + u8 i2c_pin; + u8 slave_addr; + u8 ddc_pin; + u16 edid_ptr; + u8 dvo_cfg; /* See DEVICE_CFG_* above */ + u8 dvo2_port; + u8 i2c2_pin; + u8 slave2_addr; + u8 ddc2_pin; + u8 capabilities; + u8 dvo_wiring;/* See DEVICE_WIRE_* above */ + u8 dvo2_wiring; + u16 extended_type; + u8 dvo_function; +} __packed; + +/* This one contains field offsets that are known to be common for all BDB + * versions. Notice that the meaning of the contents contents may still change, + * but at least the offsets are consistent. */ + +/* Definitions for flags_1 */ +#define IBOOST_ENABLE (1<<3) + +struct common_child_dev_config { + u16 handle; + u16 device_type; + u8 not_common1[12]; + u8 dvo_port; + u8 not_common2[2]; + u8 ddc_pin; + u16 edid_ptr; + u8 obsolete; + u8 flags_1; + u8 not_common3[13]; + u8 iboost_level; +} __packed; + + +/* This field changes depending on the BDB version, so the most reliable way to + * read it is by checking the BDB version and reading the raw pointer. */ +union child_device_config { + /* This one is safe to be used anywhere, but the code should still check + * the BDB version. */ + u8 raw[33]; + /* This one should only be kept for legacy code. */ + struct old_child_dev_config old; + /* This one should also be safe to use anywhere, even without version + * checks. */ + struct common_child_dev_config common; +} __packed; + +struct bdb_general_definitions { + /* DDC GPIO */ + u8 crt_ddc_gmbus_pin; + + /* DPMS bits */ + u8 dpms_acpi:1; + u8 skip_boot_crt_detect:1; + u8 dpms_aim:1; + u8 rsvd1:5; /* finish byte */ + + /* boot device bits */ + u8 boot_display[2]; + u8 child_dev_size; + + /* + * Device info: + * If TV is present, it'll be at devices[0]. + * LVDS will be next, either devices[0] or [1], if present. + * On some platforms the number of device is 6. But could be as few as + * 4 if both TV and LVDS are missing. + * And the device num is related with the size of general definition + * block. It is obtained by using the following formula: + * number = (block_size - sizeof(bdb_general_definitions))/ + * defs->child_dev_size; + */ + uint8_t devices[0]; +} __packed; + +/* Mask for DRRS / Panel Channel / SSC / BLT control bits extraction */ +#define MODE_MASK 0x3 + +struct bdb_lvds_options { + u8 panel_type; + u8 rsvd1; + /* LVDS capabilities, stored in a dword */ + u8 pfit_mode:2; + u8 pfit_text_mode_enhanced:1; + u8 pfit_gfx_mode_enhanced:1; + u8 pfit_ratio_auto:1; + u8 pixel_dither:1; + u8 lvds_edid:1; + u8 rsvd2:1; + u8 rsvd4; + /* LVDS Panel channel bits stored here */ + u32 lvds_panel_channel_bits; + /* LVDS SSC (Spread Spectrum Clock) bits stored here. */ + u16 ssc_bits; + u16 ssc_freq; + u16 ssc_ddt; + /* Panel color depth defined here */ + u16 panel_color_depth; + /* LVDS panel type bits stored here */ + u32 dps_panel_type_bits; + /* LVDS backlight control type bits stored here */ + u32 blt_control_type_bits; +} __packed; + +/* LFP pointer table contains entries to the struct below */ +struct bdb_lvds_lfp_data_ptr { + u16 fp_timing_offset; /* offsets are from start of bdb */ + u8 fp_table_size; + u16 dvo_timing_offset; + u8 dvo_table_size; + u16 panel_pnp_id_offset; + u8 pnp_table_size; +} __packed; + +struct bdb_lvds_lfp_data_ptrs { + u8 lvds_entries; /* followed by one or more lvds_data_ptr structs */ + struct bdb_lvds_lfp_data_ptr ptr[16]; +} __packed; + +/* LFP data has 3 blocks per entry */ +struct lvds_fp_timing { + u16 x_res; + u16 y_res; + u32 lvds_reg; + u32 lvds_reg_val; + u32 pp_on_reg; + u32 pp_on_reg_val; + u32 pp_off_reg; + u32 pp_off_reg_val; + u32 pp_cycle_reg; + u32 pp_cycle_reg_val; + u32 pfit_reg; + u32 pfit_reg_val; + u16 terminator; +} __packed; + +struct lvds_dvo_timing { + u16 clock; /**< In 10khz */ + u8 hactive_lo; + u8 hblank_lo; + u8 hblank_hi:4; + u8 hactive_hi:4; + u8 vactive_lo; + u8 vblank_lo; + u8 vblank_hi:4; + u8 vactive_hi:4; + u8 hsync_off_lo; + u8 hsync_pulse_width; + u8 vsync_pulse_width:4; + u8 vsync_off:4; + u8 rsvd0:6; + u8 hsync_off_hi:2; + u8 h_image; + u8 v_image; + u8 max_hv; + u8 h_border; + u8 v_border; + u8 rsvd1:3; + u8 digital:2; + u8 vsync_positive:1; + u8 hsync_positive:1; + u8 rsvd2:1; +} __packed; + +struct lvds_pnp_id { + u16 mfg_name; + u16 product_code; + u32 serial; + u8 mfg_week; + u8 mfg_year; +} __packed; + +struct bdb_lvds_lfp_data_entry { + struct lvds_fp_timing fp_timing; + struct lvds_dvo_timing dvo_timing; + struct lvds_pnp_id pnp_id; +} __packed; + +struct bdb_lvds_lfp_data { + struct bdb_lvds_lfp_data_entry data[16]; +} __packed; + +#define BDB_BACKLIGHT_TYPE_NONE 0 +#define BDB_BACKLIGHT_TYPE_PWM 2 + +struct bdb_lfp_backlight_data_entry { + u8 type:2; + u8 active_low_pwm:1; + u8 obsolete1:5; + u16 pwm_freq_hz; + u8 min_brightness; + u8 obsolete2; + u8 obsolete3; +} __packed; + +struct bdb_lfp_backlight_data { + u8 entry_size; + struct bdb_lfp_backlight_data_entry data[16]; + u8 level[16]; +} __packed; + +struct aimdb_header { + char signature[16]; + char oem_device[20]; + u16 aimdb_version; + u16 aimdb_header_size; + u16 aimdb_size; +} __packed; + +struct aimdb_block { + u8 aimdb_id; + u16 aimdb_size; +} __packed; + +struct vch_panel_data { + u16 fp_timing_offset; + u8 fp_timing_size; + u16 dvo_timing_offset; + u8 dvo_timing_size; + u16 text_fitting_offset; + u8 text_fitting_size; + u16 graphics_fitting_offset; + u8 graphics_fitting_size; +} __packed; + +struct vch_bdb_22 { + struct aimdb_block aimdb_block; + struct vch_panel_data panels[16]; +} __packed; + +struct bdb_sdvo_lvds_options { + u8 panel_backlight; + u8 h40_set_panel_type; + u8 panel_type; + u8 ssc_clk_freq; + u16 als_low_trip; + u16 als_high_trip; + u8 sclalarcoeff_tab_row_num; + u8 sclalarcoeff_tab_row_size; + u8 coefficient[8]; + u8 panel_misc_bits_1; + u8 panel_misc_bits_2; + u8 panel_misc_bits_3; + u8 panel_misc_bits_4; +} __packed; + + +#define BDB_DRIVER_FEATURE_NO_LVDS 0 +#define BDB_DRIVER_FEATURE_INT_LVDS 1 +#define BDB_DRIVER_FEATURE_SDVO_LVDS 2 +#define BDB_DRIVER_FEATURE_EDP 3 + +struct bdb_driver_features { + u8 boot_dev_algorithm:1; + u8 block_display_switch:1; + u8 allow_display_switch:1; + u8 hotplug_dvo:1; + u8 dual_view_zoom:1; + u8 int15h_hook:1; + u8 sprite_in_clone:1; + u8 primary_lfp_id:1; + + u16 boot_mode_x; + u16 boot_mode_y; + u8 boot_mode_bpp; + u8 boot_mode_refresh; + + u16 enable_lfp_primary:1; + u16 selective_mode_pruning:1; + u16 dual_frequency:1; + u16 render_clock_freq:1; /* 0: high freq; 1: low freq */ + u16 nt_clone_support:1; + u16 power_scheme_ui:1; /* 0: CUI; 1: 3rd party */ + u16 sprite_display_assign:1; /* 0: secondary; 1: primary */ + u16 cui_aspect_scaling:1; + u16 preserve_aspect_ratio:1; + u16 sdvo_device_power_down:1; + u16 crt_hotplug:1; + u16 lvds_config:2; + u16 tv_hotplug:1; + u16 hdmi_config:2; + + u8 static_display:1; + u8 reserved2:7; + u16 legacy_crt_max_x; + u16 legacy_crt_max_y; + u8 legacy_crt_max_refresh; + + u8 hdmi_termination; + u8 custom_vbt_version; + /* Driver features data block */ + u16 rmpm_enabled:1; + u16 s2ddt_enabled:1; + u16 dpst_enabled:1; + u16 bltclt_enabled:1; + u16 adb_enabled:1; + u16 drrs_enabled:1; + u16 grs_enabled:1; + u16 gpmt_enabled:1; + u16 tbt_enabled:1; + u16 psr_enabled:1; + u16 ips_enabled:1; + u16 reserved3:4; + u16 pc_feature_valid:1; +} __packed; + +#define EDP_18BPP 0 +#define EDP_24BPP 1 +#define EDP_30BPP 2 +#define EDP_RATE_1_62 0 +#define EDP_RATE_2_7 1 +#define EDP_LANE_1 0 +#define EDP_LANE_2 1 +#define EDP_LANE_4 3 +#define EDP_PREEMPHASIS_NONE 0 +#define EDP_PREEMPHASIS_3_5dB 1 +#define EDP_PREEMPHASIS_6dB 2 +#define EDP_PREEMPHASIS_9_5dB 3 +#define EDP_VSWING_0_4V 0 +#define EDP_VSWING_0_6V 1 +#define EDP_VSWING_0_8V 2 +#define EDP_VSWING_1_2V 3 + + +struct edp_link_params { + u8 rate:4; + u8 lanes:4; + u8 preemphasis:4; + u8 vswing:4; +} __packed; + +struct bdb_edp { + struct edp_power_seq power_seqs[16]; + u32 color_depth; + struct edp_link_params link_params[16]; + u32 sdrrs_msa_timing_delay; + + /* ith bit indicates enabled/disabled for (i+1)th panel */ + u16 edp_s3d_feature; + u16 edp_t3_optimization; + u64 edp_vswing_preemph; /* v173 */ +} __packed; + +struct psr_table { + /* Feature bits */ + u8 full_link:1; + u8 require_aux_to_wakeup:1; + u8 feature_bits_rsvd:6; + + /* Wait times */ + u8 idle_frames:4; + u8 lines_to_wait:3; + u8 wait_times_rsvd:1; + + /* TP wake up time in multiple of 100 */ + u16 tp1_wakeup_time; + u16 tp2_tp3_wakeup_time; +} __packed; + +struct bdb_psr { + struct psr_table psr_table[16]; +} __packed; + +/* + * Driver<->VBIOS interaction occurs through scratch bits in + * GR18 & SWF*. + */ + +/* GR18 bits are set on display switch and hotkey events */ +#define GR18_DRIVER_SWITCH_EN (1<<7) /* 0: VBIOS control, 1: driver control */ +#define GR18_HOTKEY_MASK 0x78 /* See also SWF4 15:0 */ +#define GR18_HK_NONE (0x0<<3) +#define GR18_HK_LFP_STRETCH (0x1<<3) +#define GR18_HK_TOGGLE_DISP (0x2<<3) +#define GR18_HK_DISP_SWITCH (0x4<<3) /* see SWF14 15:0 for what to enable */ +#define GR18_HK_POPUP_DISABLED (0x6<<3) +#define GR18_HK_POPUP_ENABLED (0x7<<3) +#define GR18_HK_PFIT (0x8<<3) +#define GR18_HK_APM_CHANGE (0xa<<3) +#define GR18_HK_MULTIPLE (0xc<<3) +#define GR18_USER_INT_EN (1<<2) +#define GR18_A0000_FLUSH_EN (1<<1) +#define GR18_SMM_EN (1<<0) + +/* Set by driver, cleared by VBIOS */ +#define SWF00_YRES_SHIFT 16 +#define SWF00_XRES_SHIFT 0 +#define SWF00_RES_MASK 0xffff + +/* Set by VBIOS at boot time and driver at runtime */ +#define SWF01_TV2_FORMAT_SHIFT 8 +#define SWF01_TV1_FORMAT_SHIFT 0 +#define SWF01_TV_FORMAT_MASK 0xffff + +#define SWF10_VBIOS_BLC_I2C_EN (1<<29) +#define SWF10_GTT_OVERRIDE_EN (1<<28) +#define SWF10_LFP_DPMS_OVR (1<<27) /* override DPMS on display switch */ +#define SWF10_ACTIVE_TOGGLE_LIST_MASK (7<<24) +#define SWF10_OLD_TOGGLE 0x0 +#define SWF10_TOGGLE_LIST_1 0x1 +#define SWF10_TOGGLE_LIST_2 0x2 +#define SWF10_TOGGLE_LIST_3 0x3 +#define SWF10_TOGGLE_LIST_4 0x4 +#define SWF10_PANNING_EN (1<<23) +#define SWF10_DRIVER_LOADED (1<<22) +#define SWF10_EXTENDED_DESKTOP (1<<21) +#define SWF10_EXCLUSIVE_MODE (1<<20) +#define SWF10_OVERLAY_EN (1<<19) +#define SWF10_PLANEB_HOLDOFF (1<<18) +#define SWF10_PLANEA_HOLDOFF (1<<17) +#define SWF10_VGA_HOLDOFF (1<<16) +#define SWF10_ACTIVE_DISP_MASK 0xffff +#define SWF10_PIPEB_LFP2 (1<<15) +#define SWF10_PIPEB_EFP2 (1<<14) +#define SWF10_PIPEB_TV2 (1<<13) +#define SWF10_PIPEB_CRT2 (1<<12) +#define SWF10_PIPEB_LFP (1<<11) +#define SWF10_PIPEB_EFP (1<<10) +#define SWF10_PIPEB_TV (1<<9) +#define SWF10_PIPEB_CRT (1<<8) +#define SWF10_PIPEA_LFP2 (1<<7) +#define SWF10_PIPEA_EFP2 (1<<6) +#define SWF10_PIPEA_TV2 (1<<5) +#define SWF10_PIPEA_CRT2 (1<<4) +#define SWF10_PIPEA_LFP (1<<3) +#define SWF10_PIPEA_EFP (1<<2) +#define SWF10_PIPEA_TV (1<<1) +#define SWF10_PIPEA_CRT (1<<0) + +#define SWF11_MEMORY_SIZE_SHIFT 16 +#define SWF11_SV_TEST_EN (1<<15) +#define SWF11_IS_AGP (1<<14) +#define SWF11_DISPLAY_HOLDOFF (1<<13) +#define SWF11_DPMS_REDUCED (1<<12) +#define SWF11_IS_VBE_MODE (1<<11) +#define SWF11_PIPEB_ACCESS (1<<10) /* 0 here means pipe a */ +#define SWF11_DPMS_MASK 0x07 +#define SWF11_DPMS_OFF (1<<2) +#define SWF11_DPMS_SUSPEND (1<<1) +#define SWF11_DPMS_STANDBY (1<<0) +#define SWF11_DPMS_ON 0 + +#define SWF14_GFX_PFIT_EN (1<<31) +#define SWF14_TEXT_PFIT_EN (1<<30) +#define SWF14_LID_STATUS_CLOSED (1<<29) /* 0 here means open */ +#define SWF14_POPUP_EN (1<<28) +#define SWF14_DISPLAY_HOLDOFF (1<<27) +#define SWF14_DISP_DETECT_EN (1<<26) +#define SWF14_DOCKING_STATUS_DOCKED (1<<25) /* 0 here means undocked */ +#define SWF14_DRIVER_STATUS (1<<24) +#define SWF14_OS_TYPE_WIN9X (1<<23) +#define SWF14_OS_TYPE_WINNT (1<<22) +/* 21:19 rsvd */ +#define SWF14_PM_TYPE_MASK 0x00070000 +#define SWF14_PM_ACPI_VIDEO (0x4 << 16) +#define SWF14_PM_ACPI (0x3 << 16) +#define SWF14_PM_APM_12 (0x2 << 16) +#define SWF14_PM_APM_11 (0x1 << 16) +#define SWF14_HK_REQUEST_MASK 0x0000ffff /* see GR18 6:3 for event type */ + /* if GR18 indicates a display switch */ +#define SWF14_DS_PIPEB_LFP2_EN (1<<15) +#define SWF14_DS_PIPEB_EFP2_EN (1<<14) +#define SWF14_DS_PIPEB_TV2_EN (1<<13) +#define SWF14_DS_PIPEB_CRT2_EN (1<<12) +#define SWF14_DS_PIPEB_LFP_EN (1<<11) +#define SWF14_DS_PIPEB_EFP_EN (1<<10) +#define SWF14_DS_PIPEB_TV_EN (1<<9) +#define SWF14_DS_PIPEB_CRT_EN (1<<8) +#define SWF14_DS_PIPEA_LFP2_EN (1<<7) +#define SWF14_DS_PIPEA_EFP2_EN (1<<6) +#define SWF14_DS_PIPEA_TV2_EN (1<<5) +#define SWF14_DS_PIPEA_CRT2_EN (1<<4) +#define SWF14_DS_PIPEA_LFP_EN (1<<3) +#define SWF14_DS_PIPEA_EFP_EN (1<<2) +#define SWF14_DS_PIPEA_TV_EN (1<<1) +#define SWF14_DS_PIPEA_CRT_EN (1<<0) + /* if GR18 indicates a panel fitting request */ +#define SWF14_PFIT_EN (1<<0) /* 0 means disable */ + /* if GR18 indicates an APM change request */ +#define SWF14_APM_HIBERNATE 0x4 +#define SWF14_APM_SUSPEND 0x3 +#define SWF14_APM_STANDBY 0x1 +#define SWF14_APM_RESTORE 0x0 + +/* Add the device class for LFP, TV, HDMI */ +#define DEVICE_TYPE_INT_LFP 0x1022 +#define DEVICE_TYPE_INT_TV 0x1009 +#define DEVICE_TYPE_HDMI 0x60D2 +#define DEVICE_TYPE_DP 0x68C6 +#define DEVICE_TYPE_eDP 0x78C6 + +#define DEVICE_TYPE_CLASS_EXTENSION (1 << 15) +#define DEVICE_TYPE_POWER_MANAGEMENT (1 << 14) +#define DEVICE_TYPE_HOTPLUG_SIGNALING (1 << 13) +#define DEVICE_TYPE_INTERNAL_CONNECTOR (1 << 12) +#define DEVICE_TYPE_NOT_HDMI_OUTPUT (1 << 11) +#define DEVICE_TYPE_MIPI_OUTPUT (1 << 10) +#define DEVICE_TYPE_COMPOSITE_OUTPUT (1 << 9) +#define DEVICE_TYPE_DUAL_CHANNEL (1 << 8) +#define DEVICE_TYPE_HIGH_SPEED_LINK (1 << 6) +#define DEVICE_TYPE_LVDS_SINGALING (1 << 5) +#define DEVICE_TYPE_TMDS_DVI_SIGNALING (1 << 4) +#define DEVICE_TYPE_VIDEO_SIGNALING (1 << 3) +#define DEVICE_TYPE_DISPLAYPORT_OUTPUT (1 << 2) +#define DEVICE_TYPE_DIGITAL_OUTPUT (1 << 1) +#define DEVICE_TYPE_ANALOG_OUTPUT (1 << 0) + +/* + * Bits we care about when checking for DEVICE_TYPE_eDP + * Depending on the system, the other bits may or may not + * be set for eDP outputs. + */ +#define DEVICE_TYPE_eDP_BITS \ + (DEVICE_TYPE_INTERNAL_CONNECTOR | \ + DEVICE_TYPE_MIPI_OUTPUT | \ + DEVICE_TYPE_COMPOSITE_OUTPUT | \ + DEVICE_TYPE_DUAL_CHANNEL | \ + DEVICE_TYPE_LVDS_SINGALING | \ + DEVICE_TYPE_TMDS_DVI_SIGNALING | \ + DEVICE_TYPE_VIDEO_SIGNALING | \ + DEVICE_TYPE_DISPLAYPORT_OUTPUT | \ + DEVICE_TYPE_ANALOG_OUTPUT) + +/* define the DVO port for HDMI output type */ +#define DVO_B 1 +#define DVO_C 2 +#define DVO_D 3 + +/* Possible values for the "DVO Port" field for versions >= 155: */ +#define DVO_PORT_HDMIA 0 +#define DVO_PORT_HDMIB 1 +#define DVO_PORT_HDMIC 2 +#define DVO_PORT_HDMID 3 +#define DVO_PORT_LVDS 4 +#define DVO_PORT_TV 5 +#define DVO_PORT_CRT 6 +#define DVO_PORT_DPB 7 +#define DVO_PORT_DPC 8 +#define DVO_PORT_DPD 9 +#define DVO_PORT_DPA 10 +#define DVO_PORT_DPE 11 +#define DVO_PORT_HDMIE 12 +#define DVO_PORT_MIPIA 21 +#define DVO_PORT_MIPIB 22 +#define DVO_PORT_MIPIC 23 +#define DVO_PORT_MIPID 24 + +/* Block 52 contains MIPI configuration block + * 6 * bdb_mipi_config, followed by 6 pps data block + * block below + */ +#define MAX_MIPI_CONFIGURATIONS 6 + +struct bdb_mipi_config { + struct mipi_config config[MAX_MIPI_CONFIGURATIONS]; + struct mipi_pps_data pps[MAX_MIPI_CONFIGURATIONS]; +} __packed; + +/* Block 53 contains MIPI sequences as needed by the panel + * for enabling it. This block can be variable in size and + * can be maximum of 6 blocks + */ +struct bdb_mipi_sequence { + u8 version; + u8 data[0]; +} __packed; + +enum mipi_gpio_pin_index { + MIPI_GPIO_UNDEFINED = 0, + MIPI_GPIO_PANEL_ENABLE, + MIPI_GPIO_BL_ENABLE, + MIPI_GPIO_PWM_ENABLE, + MIPI_GPIO_RESET_N, + MIPI_GPIO_PWR_DOWN_R, + MIPI_GPIO_STDBY_RST_N, + MIPI_GPIO_MAX +}; + +#endif /* _INTEL_BIOS_PRIVATE_H_ */ -- 2.1.4 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH v2 6/6] drm/i915: hide away VBT private data in a separate header 2016-01-11 19:54 ` [PATCH v2 6/6] drm/i915: hide away VBT private data in a separate header Jani Nikula @ 2016-01-14 12:53 ` Mika Kahola 0 siblings, 0 replies; 17+ messages in thread From: Mika Kahola @ 2016-01-14 12:53 UTC (permalink / raw) To: Jani Nikula; +Cc: intel-gfx On Mon, 2016-01-11 at 21:54 +0200, Jani Nikula wrote: > We've been accumulating code across the driver that depends on the VBT > specific structures and defines. The VBT is an uncontrollable > beast. Encourage encapsulation of the VBT data by hiding the structures > and defines in a private header only to be included from intel_bios.c. > Tested-by: Mika Kahola <mika.kahola@intel.com> > Signed-off-by: Jani Nikula <jani.nikula@intel.com> > --- > Documentation/DocBook/gpu.tmpl | 2 +- > drivers/gpu/drm/i915/intel_bios.c | 2 +- > drivers/gpu/drm/i915/intel_bios.h | 860 ++---------------------------- > drivers/gpu/drm/i915/intel_bios_private.h | 823 ++++++++++++++++++++++++++++ > 4 files changed, 859 insertions(+), 828 deletions(-) > create mode 100644 drivers/gpu/drm/i915/intel_bios_private.h > > diff --git a/Documentation/DocBook/gpu.tmpl b/Documentation/DocBook/gpu.tmpl > index 351e801cead9..4ba7c6f8e8b1 100644 > --- a/Documentation/DocBook/gpu.tmpl > +++ b/Documentation/DocBook/gpu.tmpl > @@ -3323,7 +3323,7 @@ int num_ioctls;</synopsis> > <title>Video BIOS Table (VBT)</title> > !Pdrivers/gpu/drm/i915/intel_bios.c Video BIOS Table (VBT) > !Idrivers/gpu/drm/i915/intel_bios.c > -!Idrivers/gpu/drm/i915/intel_bios.h > +!Idrivers/gpu/drm/i915/intel_bios_private.h > </sect2> > </sect1> > > diff --git a/drivers/gpu/drm/i915/intel_bios.c b/drivers/gpu/drm/i915/intel_bios.c > index 94c2baa50db7..83b78e0df306 100644 > --- a/drivers/gpu/drm/i915/intel_bios.c > +++ b/drivers/gpu/drm/i915/intel_bios.c > @@ -29,7 +29,7 @@ > #include <drm/drmP.h> > #include <drm/i915_drm.h> > #include "i915_drv.h" > -#include "intel_bios.h" > +#include "intel_bios_private.h" > > /** > * DOC: Video BIOS Table (VBT) > diff --git a/drivers/gpu/drm/i915/intel_bios.h b/drivers/gpu/drm/i915/intel_bios.h > index 350d4e0f75a4..536d90f0c3eb 100644 > --- a/drivers/gpu/drm/i915/intel_bios.h > +++ b/drivers/gpu/drm/i915/intel_bios.h > @@ -1,5 +1,5 @@ > /* > - * Copyright © 2006 Intel Corporation > + * Copyright © 2016 Intel Corporation > * > * Permission is hereby granted, free of charge, to any person obtaining a > * copy of this software and associated documentation files (the "Software"), > @@ -19,543 +19,15 @@ > * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, > * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE > * SOFTWARE. > - * > - * Authors: > - * Eric Anholt <eric@anholt.net> > - * > - */ > - > -#ifndef _INTEL_BIOS_H_ > -#define _INTEL_BIOS_H_ > - > -/** > - * struct vbt_header - VBT Header structure > - * @signature: VBT signature, always starts with "$VBT" > - * @version: Version of this structure > - * @header_size: Size of this structure > - * @vbt_size: Size of VBT (VBT Header, BDB Header and data blocks) > - * @vbt_checksum: Checksum > - * @reserved0: Reserved > - * @bdb_offset: Offset of &struct bdb_header from beginning of VBT > - * @aim_offset: Offsets of add-in data blocks from beginning of VBT > - */ > -struct vbt_header { > - u8 signature[20]; > - u16 version; > - u16 header_size; > - u16 vbt_size; > - u8 vbt_checksum; > - u8 reserved0; > - u32 bdb_offset; > - u32 aim_offset[4]; > -} __packed; > - > -/** > - * struct bdb_header - BDB Header structure > - * @signature: BDB signature "BIOS_DATA_BLOCK" > - * @version: Version of the data block definitions > - * @header_size: Size of this structure > - * @bdb_size: Size of BDB (BDB Header and data blocks) > - */ > -struct bdb_header { > - u8 signature[16]; > - u16 version; > - u16 header_size; > - u16 bdb_size; > -} __packed; > - > -/* strictly speaking, this is a "skip" block, but it has interesting info */ > -struct vbios_data { > - u8 type; /* 0 == desktop, 1 == mobile */ > - u8 relstage; > - u8 chipset; > - u8 lvds_present:1; > - u8 tv_present:1; > - u8 rsvd2:6; /* finish byte */ > - u8 rsvd3[4]; > - u8 signon[155]; > - u8 copyright[61]; > - u16 code_segment; > - u8 dos_boot_mode; > - u8 bandwidth_percent; > - u8 rsvd4; /* popup memory size */ > - u8 resize_pci_bios; > - u8 rsvd5; /* is crt already on ddc2 */ > -} __packed; > - > -/* > - * There are several types of BIOS data blocks (BDBs), each block has > - * an ID and size in the first 3 bytes (ID in first, size in next 2). > - * Known types are listed below. > */ > -#define BDB_GENERAL_FEATURES 1 > -#define BDB_GENERAL_DEFINITIONS 2 > -#define BDB_OLD_TOGGLE_LIST 3 > -#define BDB_MODE_SUPPORT_LIST 4 > -#define BDB_GENERIC_MODE_TABLE 5 > -#define BDB_EXT_MMIO_REGS 6 > -#define BDB_SWF_IO 7 > -#define BDB_SWF_MMIO 8 > -#define BDB_PSR 9 > -#define BDB_MODE_REMOVAL_TABLE 10 > -#define BDB_CHILD_DEVICE_TABLE 11 > -#define BDB_DRIVER_FEATURES 12 > -#define BDB_DRIVER_PERSISTENCE 13 > -#define BDB_EXT_TABLE_PTRS 14 > -#define BDB_DOT_CLOCK_OVERRIDE 15 > -#define BDB_DISPLAY_SELECT 16 > -/* 17 rsvd */ > -#define BDB_DRIVER_ROTATION 18 > -#define BDB_DISPLAY_REMOVE 19 > -#define BDB_OEM_CUSTOM 20 > -#define BDB_EFP_LIST 21 /* workarounds for VGA hsync/vsync */ > -#define BDB_SDVO_LVDS_OPTIONS 22 > -#define BDB_SDVO_PANEL_DTDS 23 > -#define BDB_SDVO_LVDS_PNP_IDS 24 > -#define BDB_SDVO_LVDS_POWER_SEQ 25 > -#define BDB_TV_OPTIONS 26 > -#define BDB_EDP 27 > -#define BDB_LVDS_OPTIONS 40 > -#define BDB_LVDS_LFP_DATA_PTRS 41 > -#define BDB_LVDS_LFP_DATA 42 > -#define BDB_LVDS_BACKLIGHT 43 > -#define BDB_LVDS_POWER 44 > -#define BDB_MIPI_CONFIG 52 > -#define BDB_MIPI_SEQUENCE 53 > -#define BDB_SKIP 254 /* VBIOS private block, ignore */ > - > -struct bdb_general_features { > - /* bits 1 */ > - u8 panel_fitting:2; > - u8 flexaim:1; > - u8 msg_enable:1; > - u8 clear_screen:3; > - u8 color_flip:1; > - > - /* bits 2 */ > - u8 download_ext_vbt:1; > - u8 enable_ssc:1; > - u8 ssc_freq:1; > - u8 enable_lfp_on_override:1; > - u8 disable_ssc_ddt:1; > - u8 rsvd7:1; > - u8 display_clock_mode:1; > - u8 rsvd8:1; /* finish byte */ > - > - /* bits 3 */ > - u8 disable_smooth_vision:1; > - u8 single_dvi:1; > - u8 rsvd9:1; > - u8 fdi_rx_polarity_inverted:1; > - u8 rsvd10:4; /* finish byte */ > - > - /* bits 4 */ > - u8 legacy_monitor_detect; > - > - /* bits 5 */ > - u8 int_crt_support:1; > - u8 int_tv_support:1; > - u8 int_efp_support:1; > - u8 dp_ssc_enb:1; /* PCH attached eDP supports SSC */ > - u8 dp_ssc_freq:1; /* SSC freq for PCH attached eDP */ > - u8 rsvd11:3; /* finish byte */ > -} __packed; > - > -/* pre-915 */ > -#define GPIO_PIN_DVI_LVDS 0x03 /* "DVI/LVDS DDC GPIO pins" */ > -#define GPIO_PIN_ADD_I2C 0x05 /* "ADDCARD I2C GPIO pins" */ > -#define GPIO_PIN_ADD_DDC 0x04 /* "ADDCARD DDC GPIO pins" */ > -#define GPIO_PIN_ADD_DDC_I2C 0x06 /* "ADDCARD DDC/I2C GPIO pins" */ > - > -/* Pre 915 */ > -#define DEVICE_TYPE_NONE 0x00 > -#define DEVICE_TYPE_CRT 0x01 > -#define DEVICE_TYPE_TV 0x09 > -#define DEVICE_TYPE_EFP 0x12 > -#define DEVICE_TYPE_LFP 0x22 > -/* On 915+ */ > -#define DEVICE_TYPE_CRT_DPMS 0x6001 > -#define DEVICE_TYPE_CRT_DPMS_HOTPLUG 0x4001 > -#define DEVICE_TYPE_TV_COMPOSITE 0x0209 > -#define DEVICE_TYPE_TV_MACROVISION 0x0289 > -#define DEVICE_TYPE_TV_RF_COMPOSITE 0x020c > -#define DEVICE_TYPE_TV_SVIDEO_COMPOSITE 0x0609 > -#define DEVICE_TYPE_TV_SCART 0x0209 > -#define DEVICE_TYPE_TV_CODEC_HOTPLUG_PWR 0x6009 > -#define DEVICE_TYPE_EFP_HOTPLUG_PWR 0x6012 > -#define DEVICE_TYPE_EFP_DVI_HOTPLUG_PWR 0x6052 > -#define DEVICE_TYPE_EFP_DVI_I 0x6053 > -#define DEVICE_TYPE_EFP_DVI_D_DUAL 0x6152 > -#define DEVICE_TYPE_EFP_DVI_D_HDCP 0x60d2 > -#define DEVICE_TYPE_OPENLDI_HOTPLUG_PWR 0x6062 > -#define DEVICE_TYPE_OPENLDI_DUALPIX 0x6162 > -#define DEVICE_TYPE_LFP_PANELLINK 0x5012 > -#define DEVICE_TYPE_LFP_CMOS_PWR 0x5042 > -#define DEVICE_TYPE_LFP_LVDS_PWR 0x5062 > -#define DEVICE_TYPE_LFP_LVDS_DUAL 0x5162 > -#define DEVICE_TYPE_LFP_LVDS_DUAL_HDCP 0x51e2 > - > -#define DEVICE_CFG_NONE 0x00 > -#define DEVICE_CFG_12BIT_DVOB 0x01 > -#define DEVICE_CFG_12BIT_DVOC 0x02 > -#define DEVICE_CFG_24BIT_DVOBC 0x09 > -#define DEVICE_CFG_24BIT_DVOCB 0x0a > -#define DEVICE_CFG_DUAL_DVOB 0x11 > -#define DEVICE_CFG_DUAL_DVOC 0x12 > -#define DEVICE_CFG_DUAL_DVOBC 0x13 > -#define DEVICE_CFG_DUAL_LINK_DVOBC 0x19 > -#define DEVICE_CFG_DUAL_LINK_DVOCB 0x1a > - > -#define DEVICE_WIRE_NONE 0x00 > -#define DEVICE_WIRE_DVOB 0x01 > -#define DEVICE_WIRE_DVOC 0x02 > -#define DEVICE_WIRE_DVOBC 0x03 > -#define DEVICE_WIRE_DVOBB 0x05 > -#define DEVICE_WIRE_DVOCC 0x06 > -#define DEVICE_WIRE_DVOB_MASTER 0x0d > -#define DEVICE_WIRE_DVOC_MASTER 0x0e > - > -#define DEVICE_PORT_DVOA 0x00 /* none on 845+ */ > -#define DEVICE_PORT_DVOB 0x01 > -#define DEVICE_PORT_DVOC 0x02 > > /* > - * We used to keep this struct but without any version control. We should avoid > - * using it in the future, but it should be safe to keep using it in the old > - * code. Do not change; we rely on its size. > + * Please use intel_bios_private.h for VBT private data, and add the parsed data > + * to struct intel_vbt_data. > */ > -struct old_child_dev_config { > - u16 handle; > - u16 device_type; > - u8 device_id[10]; /* ascii string */ > - u16 addin_offset; > - u8 dvo_port; /* See Device_PORT_* above */ > - u8 i2c_pin; > - u8 slave_addr; > - u8 ddc_pin; > - u16 edid_ptr; > - u8 dvo_cfg; /* See DEVICE_CFG_* above */ > - u8 dvo2_port; > - u8 i2c2_pin; > - u8 slave2_addr; > - u8 ddc2_pin; > - u8 capabilities; > - u8 dvo_wiring;/* See DEVICE_WIRE_* above */ > - u8 dvo2_wiring; > - u16 extended_type; > - u8 dvo_function; > -} __packed; > - > -/* This one contains field offsets that are known to be common for all BDB > - * versions. Notice that the meaning of the contents contents may still change, > - * but at least the offsets are consistent. */ > - > -/* Definitions for flags_1 */ > -#define IBOOST_ENABLE (1<<3) > - > -struct common_child_dev_config { > - u16 handle; > - u16 device_type; > - u8 not_common1[12]; > - u8 dvo_port; > - u8 not_common2[2]; > - u8 ddc_pin; > - u16 edid_ptr; > - u8 obsolete; > - u8 flags_1; > - u8 not_common3[13]; > - u8 iboost_level; > -} __packed; > - > - > -/* This field changes depending on the BDB version, so the most reliable way to > - * read it is by checking the BDB version and reading the raw pointer. */ > -union child_device_config { > - /* This one is safe to be used anywhere, but the code should still check > - * the BDB version. */ > - u8 raw[33]; > - /* This one should only be kept for legacy code. */ > - struct old_child_dev_config old; > - /* This one should also be safe to use anywhere, even without version > - * checks. */ > - struct common_child_dev_config common; > -} __packed; > - > -struct bdb_general_definitions { > - /* DDC GPIO */ > - u8 crt_ddc_gmbus_pin; > - > - /* DPMS bits */ > - u8 dpms_acpi:1; > - u8 skip_boot_crt_detect:1; > - u8 dpms_aim:1; > - u8 rsvd1:5; /* finish byte */ > - > - /* boot device bits */ > - u8 boot_display[2]; > - u8 child_dev_size; > - > - /* > - * Device info: > - * If TV is present, it'll be at devices[0]. > - * LVDS will be next, either devices[0] or [1], if present. > - * On some platforms the number of device is 6. But could be as few as > - * 4 if both TV and LVDS are missing. > - * And the device num is related with the size of general definition > - * block. It is obtained by using the following formula: > - * number = (block_size - sizeof(bdb_general_definitions))/ > - * defs->child_dev_size; > - */ > - uint8_t devices[0]; > -} __packed; > - > -/* Mask for DRRS / Panel Channel / SSC / BLT control bits extraction */ > -#define MODE_MASK 0x3 > - > -struct bdb_lvds_options { > - u8 panel_type; > - u8 rsvd1; > - /* LVDS capabilities, stored in a dword */ > - u8 pfit_mode:2; > - u8 pfit_text_mode_enhanced:1; > - u8 pfit_gfx_mode_enhanced:1; > - u8 pfit_ratio_auto:1; > - u8 pixel_dither:1; > - u8 lvds_edid:1; > - u8 rsvd2:1; > - u8 rsvd4; > - /* LVDS Panel channel bits stored here */ > - u32 lvds_panel_channel_bits; > - /* LVDS SSC (Spread Spectrum Clock) bits stored here. */ > - u16 ssc_bits; > - u16 ssc_freq; > - u16 ssc_ddt; > - /* Panel color depth defined here */ > - u16 panel_color_depth; > - /* LVDS panel type bits stored here */ > - u32 dps_panel_type_bits; > - /* LVDS backlight control type bits stored here */ > - u32 blt_control_type_bits; > -} __packed; > - > -/* LFP pointer table contains entries to the struct below */ > -struct bdb_lvds_lfp_data_ptr { > - u16 fp_timing_offset; /* offsets are from start of bdb */ > - u8 fp_table_size; > - u16 dvo_timing_offset; > - u8 dvo_table_size; > - u16 panel_pnp_id_offset; > - u8 pnp_table_size; > -} __packed; > - > -struct bdb_lvds_lfp_data_ptrs { > - u8 lvds_entries; /* followed by one or more lvds_data_ptr structs */ > - struct bdb_lvds_lfp_data_ptr ptr[16]; > -} __packed; > - > -/* LFP data has 3 blocks per entry */ > -struct lvds_fp_timing { > - u16 x_res; > - u16 y_res; > - u32 lvds_reg; > - u32 lvds_reg_val; > - u32 pp_on_reg; > - u32 pp_on_reg_val; > - u32 pp_off_reg; > - u32 pp_off_reg_val; > - u32 pp_cycle_reg; > - u32 pp_cycle_reg_val; > - u32 pfit_reg; > - u32 pfit_reg_val; > - u16 terminator; > -} __packed; > - > -struct lvds_dvo_timing { > - u16 clock; /**< In 10khz */ > - u8 hactive_lo; > - u8 hblank_lo; > - u8 hblank_hi:4; > - u8 hactive_hi:4; > - u8 vactive_lo; > - u8 vblank_lo; > - u8 vblank_hi:4; > - u8 vactive_hi:4; > - u8 hsync_off_lo; > - u8 hsync_pulse_width; > - u8 vsync_pulse_width:4; > - u8 vsync_off:4; > - u8 rsvd0:6; > - u8 hsync_off_hi:2; > - u8 h_image; > - u8 v_image; > - u8 max_hv; > - u8 h_border; > - u8 v_border; > - u8 rsvd1:3; > - u8 digital:2; > - u8 vsync_positive:1; > - u8 hsync_positive:1; > - u8 rsvd2:1; > -} __packed; > - > -struct lvds_pnp_id { > - u16 mfg_name; > - u16 product_code; > - u32 serial; > - u8 mfg_week; > - u8 mfg_year; > -} __packed; > - > -struct bdb_lvds_lfp_data_entry { > - struct lvds_fp_timing fp_timing; > - struct lvds_dvo_timing dvo_timing; > - struct lvds_pnp_id pnp_id; > -} __packed; > - > -struct bdb_lvds_lfp_data { > - struct bdb_lvds_lfp_data_entry data[16]; > -} __packed; > - > -#define BDB_BACKLIGHT_TYPE_NONE 0 > -#define BDB_BACKLIGHT_TYPE_PWM 2 > - > -struct bdb_lfp_backlight_data_entry { > - u8 type:2; > - u8 active_low_pwm:1; > - u8 obsolete1:5; > - u16 pwm_freq_hz; > - u8 min_brightness; > - u8 obsolete2; > - u8 obsolete3; > -} __packed; > - > -struct bdb_lfp_backlight_data { > - u8 entry_size; > - struct bdb_lfp_backlight_data_entry data[16]; > - u8 level[16]; > -} __packed; > - > -struct aimdb_header { > - char signature[16]; > - char oem_device[20]; > - u16 aimdb_version; > - u16 aimdb_header_size; > - u16 aimdb_size; > -} __packed; > - > -struct aimdb_block { > - u8 aimdb_id; > - u16 aimdb_size; > -} __packed; > > -struct vch_panel_data { > - u16 fp_timing_offset; > - u8 fp_timing_size; > - u16 dvo_timing_offset; > - u8 dvo_timing_size; > - u16 text_fitting_offset; > - u8 text_fitting_size; > - u16 graphics_fitting_offset; > - u8 graphics_fitting_size; > -} __packed; > - > -struct vch_bdb_22 { > - struct aimdb_block aimdb_block; > - struct vch_panel_data panels[16]; > -} __packed; > - > -struct bdb_sdvo_lvds_options { > - u8 panel_backlight; > - u8 h40_set_panel_type; > - u8 panel_type; > - u8 ssc_clk_freq; > - u16 als_low_trip; > - u16 als_high_trip; > - u8 sclalarcoeff_tab_row_num; > - u8 sclalarcoeff_tab_row_size; > - u8 coefficient[8]; > - u8 panel_misc_bits_1; > - u8 panel_misc_bits_2; > - u8 panel_misc_bits_3; > - u8 panel_misc_bits_4; > -} __packed; > - > - > -#define BDB_DRIVER_FEATURE_NO_LVDS 0 > -#define BDB_DRIVER_FEATURE_INT_LVDS 1 > -#define BDB_DRIVER_FEATURE_SDVO_LVDS 2 > -#define BDB_DRIVER_FEATURE_EDP 3 > - > -struct bdb_driver_features { > - u8 boot_dev_algorithm:1; > - u8 block_display_switch:1; > - u8 allow_display_switch:1; > - u8 hotplug_dvo:1; > - u8 dual_view_zoom:1; > - u8 int15h_hook:1; > - u8 sprite_in_clone:1; > - u8 primary_lfp_id:1; > - > - u16 boot_mode_x; > - u16 boot_mode_y; > - u8 boot_mode_bpp; > - u8 boot_mode_refresh; > - > - u16 enable_lfp_primary:1; > - u16 selective_mode_pruning:1; > - u16 dual_frequency:1; > - u16 render_clock_freq:1; /* 0: high freq; 1: low freq */ > - u16 nt_clone_support:1; > - u16 power_scheme_ui:1; /* 0: CUI; 1: 3rd party */ > - u16 sprite_display_assign:1; /* 0: secondary; 1: primary */ > - u16 cui_aspect_scaling:1; > - u16 preserve_aspect_ratio:1; > - u16 sdvo_device_power_down:1; > - u16 crt_hotplug:1; > - u16 lvds_config:2; > - u16 tv_hotplug:1; > - u16 hdmi_config:2; > - > - u8 static_display:1; > - u8 reserved2:7; > - u16 legacy_crt_max_x; > - u16 legacy_crt_max_y; > - u8 legacy_crt_max_refresh; > - > - u8 hdmi_termination; > - u8 custom_vbt_version; > - /* Driver features data block */ > - u16 rmpm_enabled:1; > - u16 s2ddt_enabled:1; > - u16 dpst_enabled:1; > - u16 bltclt_enabled:1; > - u16 adb_enabled:1; > - u16 drrs_enabled:1; > - u16 grs_enabled:1; > - u16 gpmt_enabled:1; > - u16 tbt_enabled:1; > - u16 psr_enabled:1; > - u16 ips_enabled:1; > - u16 reserved3:4; > - u16 pc_feature_valid:1; > -} __packed; > - > -#define EDP_18BPP 0 > -#define EDP_24BPP 1 > -#define EDP_30BPP 2 > -#define EDP_RATE_1_62 0 > -#define EDP_RATE_2_7 1 > -#define EDP_LANE_1 0 > -#define EDP_LANE_2 1 > -#define EDP_LANE_4 3 > -#define EDP_PREEMPHASIS_NONE 0 > -#define EDP_PREEMPHASIS_3_5dB 1 > -#define EDP_PREEMPHASIS_6dB 2 > -#define EDP_PREEMPHASIS_9_5dB 3 > -#define EDP_VSWING_0_4V 0 > -#define EDP_VSWING_0_6V 1 > -#define EDP_VSWING_0_8V 2 > -#define EDP_VSWING_1_2V 3 > +#ifndef _INTEL_BIOS_H_ > +#define _INTEL_BIOS_H_ > > struct edp_power_seq { > u16 t1_t3; > @@ -565,245 +37,37 @@ struct edp_power_seq { > u16 t11_t12; > } __packed; > > -struct edp_link_params { > - u8 rate:4; > - u8 lanes:4; > - u8 preemphasis:4; > - u8 vswing:4; > -} __packed; > - > -struct bdb_edp { > - struct edp_power_seq power_seqs[16]; > - u32 color_depth; > - struct edp_link_params link_params[16]; > - u32 sdrrs_msa_timing_delay; > - > - /* ith bit indicates enabled/disabled for (i+1)th panel */ > - u16 edp_s3d_feature; > - u16 edp_t3_optimization; > - u64 edp_vswing_preemph; /* v173 */ > -} __packed; > - > -struct psr_table { > - /* Feature bits */ > - u8 full_link:1; > - u8 require_aux_to_wakeup:1; > - u8 feature_bits_rsvd:6; > - > - /* Wait times */ > - u8 idle_frames:4; > - u8 lines_to_wait:3; > - u8 wait_times_rsvd:1; > - > - /* TP wake up time in multiple of 100 */ > - u16 tp1_wakeup_time; > - u16 tp2_tp3_wakeup_time; > -} __packed; > - > -struct bdb_psr { > - struct psr_table psr_table[16]; > -} __packed; > - > -/* > - * Driver<->VBIOS interaction occurs through scratch bits in > - * GR18 & SWF*. > - */ > - > -/* GR18 bits are set on display switch and hotkey events */ > -#define GR18_DRIVER_SWITCH_EN (1<<7) /* 0: VBIOS control, 1: driver control */ > -#define GR18_HOTKEY_MASK 0x78 /* See also SWF4 15:0 */ > -#define GR18_HK_NONE (0x0<<3) > -#define GR18_HK_LFP_STRETCH (0x1<<3) > -#define GR18_HK_TOGGLE_DISP (0x2<<3) > -#define GR18_HK_DISP_SWITCH (0x4<<3) /* see SWF14 15:0 for what to enable */ > -#define GR18_HK_POPUP_DISABLED (0x6<<3) > -#define GR18_HK_POPUP_ENABLED (0x7<<3) > -#define GR18_HK_PFIT (0x8<<3) > -#define GR18_HK_APM_CHANGE (0xa<<3) > -#define GR18_HK_MULTIPLE (0xc<<3) > -#define GR18_USER_INT_EN (1<<2) > -#define GR18_A0000_FLUSH_EN (1<<1) > -#define GR18_SMM_EN (1<<0) > - > -/* Set by driver, cleared by VBIOS */ > -#define SWF00_YRES_SHIFT 16 > -#define SWF00_XRES_SHIFT 0 > -#define SWF00_RES_MASK 0xffff > - > -/* Set by VBIOS at boot time and driver at runtime */ > -#define SWF01_TV2_FORMAT_SHIFT 8 > -#define SWF01_TV1_FORMAT_SHIFT 0 > -#define SWF01_TV_FORMAT_MASK 0xffff > - > -#define SWF10_VBIOS_BLC_I2C_EN (1<<29) > -#define SWF10_GTT_OVERRIDE_EN (1<<28) > -#define SWF10_LFP_DPMS_OVR (1<<27) /* override DPMS on display switch */ > -#define SWF10_ACTIVE_TOGGLE_LIST_MASK (7<<24) > -#define SWF10_OLD_TOGGLE 0x0 > -#define SWF10_TOGGLE_LIST_1 0x1 > -#define SWF10_TOGGLE_LIST_2 0x2 > -#define SWF10_TOGGLE_LIST_3 0x3 > -#define SWF10_TOGGLE_LIST_4 0x4 > -#define SWF10_PANNING_EN (1<<23) > -#define SWF10_DRIVER_LOADED (1<<22) > -#define SWF10_EXTENDED_DESKTOP (1<<21) > -#define SWF10_EXCLUSIVE_MODE (1<<20) > -#define SWF10_OVERLAY_EN (1<<19) > -#define SWF10_PLANEB_HOLDOFF (1<<18) > -#define SWF10_PLANEA_HOLDOFF (1<<17) > -#define SWF10_VGA_HOLDOFF (1<<16) > -#define SWF10_ACTIVE_DISP_MASK 0xffff > -#define SWF10_PIPEB_LFP2 (1<<15) > -#define SWF10_PIPEB_EFP2 (1<<14) > -#define SWF10_PIPEB_TV2 (1<<13) > -#define SWF10_PIPEB_CRT2 (1<<12) > -#define SWF10_PIPEB_LFP (1<<11) > -#define SWF10_PIPEB_EFP (1<<10) > -#define SWF10_PIPEB_TV (1<<9) > -#define SWF10_PIPEB_CRT (1<<8) > -#define SWF10_PIPEA_LFP2 (1<<7) > -#define SWF10_PIPEA_EFP2 (1<<6) > -#define SWF10_PIPEA_TV2 (1<<5) > -#define SWF10_PIPEA_CRT2 (1<<4) > -#define SWF10_PIPEA_LFP (1<<3) > -#define SWF10_PIPEA_EFP (1<<2) > -#define SWF10_PIPEA_TV (1<<1) > -#define SWF10_PIPEA_CRT (1<<0) > - > -#define SWF11_MEMORY_SIZE_SHIFT 16 > -#define SWF11_SV_TEST_EN (1<<15) > -#define SWF11_IS_AGP (1<<14) > -#define SWF11_DISPLAY_HOLDOFF (1<<13) > -#define SWF11_DPMS_REDUCED (1<<12) > -#define SWF11_IS_VBE_MODE (1<<11) > -#define SWF11_PIPEB_ACCESS (1<<10) /* 0 here means pipe a */ > -#define SWF11_DPMS_MASK 0x07 > -#define SWF11_DPMS_OFF (1<<2) > -#define SWF11_DPMS_SUSPEND (1<<1) > -#define SWF11_DPMS_STANDBY (1<<0) > -#define SWF11_DPMS_ON 0 > - > -#define SWF14_GFX_PFIT_EN (1<<31) > -#define SWF14_TEXT_PFIT_EN (1<<30) > -#define SWF14_LID_STATUS_CLOSED (1<<29) /* 0 here means open */ > -#define SWF14_POPUP_EN (1<<28) > -#define SWF14_DISPLAY_HOLDOFF (1<<27) > -#define SWF14_DISP_DETECT_EN (1<<26) > -#define SWF14_DOCKING_STATUS_DOCKED (1<<25) /* 0 here means undocked */ > -#define SWF14_DRIVER_STATUS (1<<24) > -#define SWF14_OS_TYPE_WIN9X (1<<23) > -#define SWF14_OS_TYPE_WINNT (1<<22) > -/* 21:19 rsvd */ > -#define SWF14_PM_TYPE_MASK 0x00070000 > -#define SWF14_PM_ACPI_VIDEO (0x4 << 16) > -#define SWF14_PM_ACPI (0x3 << 16) > -#define SWF14_PM_APM_12 (0x2 << 16) > -#define SWF14_PM_APM_11 (0x1 << 16) > -#define SWF14_HK_REQUEST_MASK 0x0000ffff /* see GR18 6:3 for event type */ > - /* if GR18 indicates a display switch */ > -#define SWF14_DS_PIPEB_LFP2_EN (1<<15) > -#define SWF14_DS_PIPEB_EFP2_EN (1<<14) > -#define SWF14_DS_PIPEB_TV2_EN (1<<13) > -#define SWF14_DS_PIPEB_CRT2_EN (1<<12) > -#define SWF14_DS_PIPEB_LFP_EN (1<<11) > -#define SWF14_DS_PIPEB_EFP_EN (1<<10) > -#define SWF14_DS_PIPEB_TV_EN (1<<9) > -#define SWF14_DS_PIPEB_CRT_EN (1<<8) > -#define SWF14_DS_PIPEA_LFP2_EN (1<<7) > -#define SWF14_DS_PIPEA_EFP2_EN (1<<6) > -#define SWF14_DS_PIPEA_TV2_EN (1<<5) > -#define SWF14_DS_PIPEA_CRT2_EN (1<<4) > -#define SWF14_DS_PIPEA_LFP_EN (1<<3) > -#define SWF14_DS_PIPEA_EFP_EN (1<<2) > -#define SWF14_DS_PIPEA_TV_EN (1<<1) > -#define SWF14_DS_PIPEA_CRT_EN (1<<0) > - /* if GR18 indicates a panel fitting request */ > -#define SWF14_PFIT_EN (1<<0) /* 0 means disable */ > - /* if GR18 indicates an APM change request */ > -#define SWF14_APM_HIBERNATE 0x4 > -#define SWF14_APM_SUSPEND 0x3 > -#define SWF14_APM_STANDBY 0x1 > -#define SWF14_APM_RESTORE 0x0 > - > -/* Add the device class for LFP, TV, HDMI */ > -#define DEVICE_TYPE_INT_LFP 0x1022 > -#define DEVICE_TYPE_INT_TV 0x1009 > -#define DEVICE_TYPE_HDMI 0x60D2 > -#define DEVICE_TYPE_DP 0x68C6 > -#define DEVICE_TYPE_eDP 0x78C6 > - > -#define DEVICE_TYPE_CLASS_EXTENSION (1 << 15) > -#define DEVICE_TYPE_POWER_MANAGEMENT (1 << 14) > -#define DEVICE_TYPE_HOTPLUG_SIGNALING (1 << 13) > -#define DEVICE_TYPE_INTERNAL_CONNECTOR (1 << 12) > -#define DEVICE_TYPE_NOT_HDMI_OUTPUT (1 << 11) > -#define DEVICE_TYPE_MIPI_OUTPUT (1 << 10) > -#define DEVICE_TYPE_COMPOSITE_OUTPUT (1 << 9) > -#define DEVICE_TYPE_DUAL_CHANNEL (1 << 8) > -#define DEVICE_TYPE_HIGH_SPEED_LINK (1 << 6) > -#define DEVICE_TYPE_LVDS_SINGALING (1 << 5) > -#define DEVICE_TYPE_TMDS_DVI_SIGNALING (1 << 4) > -#define DEVICE_TYPE_VIDEO_SIGNALING (1 << 3) > -#define DEVICE_TYPE_DISPLAYPORT_OUTPUT (1 << 2) > -#define DEVICE_TYPE_DIGITAL_OUTPUT (1 << 1) > -#define DEVICE_TYPE_ANALOG_OUTPUT (1 << 0) > - > -/* > - * Bits we care about when checking for DEVICE_TYPE_eDP > - * Depending on the system, the other bits may or may not > - * be set for eDP outputs. > - */ > -#define DEVICE_TYPE_eDP_BITS \ > - (DEVICE_TYPE_INTERNAL_CONNECTOR | \ > - DEVICE_TYPE_MIPI_OUTPUT | \ > - DEVICE_TYPE_COMPOSITE_OUTPUT | \ > - DEVICE_TYPE_DUAL_CHANNEL | \ > - DEVICE_TYPE_LVDS_SINGALING | \ > - DEVICE_TYPE_TMDS_DVI_SIGNALING | \ > - DEVICE_TYPE_VIDEO_SIGNALING | \ > - DEVICE_TYPE_DISPLAYPORT_OUTPUT | \ > - DEVICE_TYPE_ANALOG_OUTPUT) > - > -/* define the DVO port for HDMI output type */ > -#define DVO_B 1 > -#define DVO_C 2 > -#define DVO_D 3 > - > -/* Possible values for the "DVO Port" field for versions >= 155: */ > -#define DVO_PORT_HDMIA 0 > -#define DVO_PORT_HDMIB 1 > -#define DVO_PORT_HDMIC 2 > -#define DVO_PORT_HDMID 3 > -#define DVO_PORT_LVDS 4 > -#define DVO_PORT_TV 5 > -#define DVO_PORT_CRT 6 > -#define DVO_PORT_DPB 7 > -#define DVO_PORT_DPC 8 > -#define DVO_PORT_DPD 9 > -#define DVO_PORT_DPA 10 > -#define DVO_PORT_DPE 11 > -#define DVO_PORT_HDMIE 12 > -#define DVO_PORT_MIPIA 21 > -#define DVO_PORT_MIPIB 22 > -#define DVO_PORT_MIPIC 23 > -#define DVO_PORT_MIPID 24 > +/* MIPI Sequence Block definitions */ > +enum mipi_seq { > + MIPI_SEQ_END = 0, > + MIPI_SEQ_ASSERT_RESET, > + MIPI_SEQ_INIT_OTP, > + MIPI_SEQ_DISPLAY_ON, > + MIPI_SEQ_DISPLAY_OFF, > + MIPI_SEQ_DEASSERT_RESET, > + MIPI_SEQ_BACKLIGHT_ON, /* sequence block v2+ */ > + MIPI_SEQ_BACKLIGHT_OFF, /* sequence block v2+ */ > + MIPI_SEQ_TEAR_ON, /* sequence block v2+ */ > + MIPI_SEQ_TEAR_OFF, /* sequence block v3+ */ > + MIPI_SEQ_POWER_ON, /* sequence block v3+ */ > + MIPI_SEQ_POWER_OFF, /* sequence block v3+ */ > + MIPI_SEQ_MAX > +}; > > -/* Block 52 contains MIPI Panel info > - * 6 such enteries will there. Index into correct > - * entery is based on the panel_index in #40 LFP > - */ > -#define MAX_MIPI_CONFIGURATIONS 6 > +enum mipi_seq_element { > + MIPI_SEQ_ELEM_END = 0, > + MIPI_SEQ_ELEM_SEND_PKT, > + MIPI_SEQ_ELEM_DELAY, > + MIPI_SEQ_ELEM_GPIO, > + MIPI_SEQ_ELEM_I2C, /* sequence block v2+ */ > + MIPI_SEQ_ELEM_SPI, /* sequence block v3+ */ > + MIPI_SEQ_ELEM_PMIC, /* sequence block v3+ */ > + MIPI_SEQ_ELEM_MAX > +}; > > #define MIPI_DSI_UNDEFINED_PANEL_ID 0 > #define MIPI_DSI_GENERIC_PANEL_ID 1 > > -/* > - * PMIC vs SoC Backlight support specified in pwm_blc > - * field in mipi_config block below. > -*/ > -#define PPS_BLC_PMIC 0 > -#define PPS_BLC_SOC 1 > - > struct mipi_config { > u16 panel_id; > > @@ -821,6 +85,8 @@ struct mipi_config { > u32 video_transfer_mode:2; > > u32 cabc_supported:1; > +#define PPS_BLC_PMIC 0 > +#define PPS_BLC_SOC 1 > u32 pwm_blc:1; > > /* Bit 13:10 */ > @@ -924,12 +190,7 @@ struct mipi_config { > > } __packed; > > -/* Block 52 contains MIPI configuration block > - * 6 * bdb_mipi_config, followed by 6 pps data > - * block below > - * > - * all delays has a unit of 100us > - */ > +/* all delays have a unit of 100us */ > struct mipi_pps_data { > u16 panel_on_delay; > u16 bl_enable_delay; > @@ -938,57 +199,4 @@ struct mipi_pps_data { > u16 panel_power_cycle_delay; > } __packed; > > -struct bdb_mipi_config { > - struct mipi_config config[MAX_MIPI_CONFIGURATIONS]; > - struct mipi_pps_data pps[MAX_MIPI_CONFIGURATIONS]; > -} __packed; > - > -/* Block 53 contains MIPI sequences as needed by the panel > - * for enabling it. This block can be variable in size and > - * can be maximum of 6 blocks > - */ > -struct bdb_mipi_sequence { > - u8 version; > - u8 data[0]; > -} __packed; > - > -/* MIPI Sequnece Block definitions */ > -enum mipi_seq { > - MIPI_SEQ_END = 0, > - MIPI_SEQ_ASSERT_RESET, > - MIPI_SEQ_INIT_OTP, > - MIPI_SEQ_DISPLAY_ON, > - MIPI_SEQ_DISPLAY_OFF, > - MIPI_SEQ_DEASSERT_RESET, > - MIPI_SEQ_BACKLIGHT_ON, /* sequence block v2+ */ > - MIPI_SEQ_BACKLIGHT_OFF, /* sequence block v2+ */ > - MIPI_SEQ_TEAR_ON, /* sequence block v2+ */ > - MIPI_SEQ_TEAR_OFF, /* sequence block v3+ */ > - MIPI_SEQ_POWER_ON, /* sequence block v3+ */ > - MIPI_SEQ_POWER_OFF, /* sequence block v3+ */ > - MIPI_SEQ_MAX > -}; > - > -enum mipi_seq_element { > - MIPI_SEQ_ELEM_END = 0, > - MIPI_SEQ_ELEM_SEND_PKT, > - MIPI_SEQ_ELEM_DELAY, > - MIPI_SEQ_ELEM_GPIO, > - MIPI_SEQ_ELEM_I2C, /* sequence block v2+ */ > - MIPI_SEQ_ELEM_SPI, /* sequence block v3+ */ > - MIPI_SEQ_ELEM_PMIC, /* sequence block v3+ */ > - MIPI_SEQ_ELEM_MAX > -}; > - > -enum mipi_gpio_pin_index { > - MIPI_GPIO_UNDEFINED = 0, > - MIPI_GPIO_PANEL_ENABLE, > - MIPI_GPIO_BL_ENABLE, > - MIPI_GPIO_PWM_ENABLE, > - MIPI_GPIO_RESET_N, > - MIPI_GPIO_PWR_DOWN_R, > - MIPI_GPIO_STDBY_RST_N, > - MIPI_GPIO_MAX > -}; > - > #endif /* _INTEL_BIOS_H_ */ > diff --git a/drivers/gpu/drm/i915/intel_bios_private.h b/drivers/gpu/drm/i915/intel_bios_private.h > new file mode 100644 > index 000000000000..56d3aecae064 > --- /dev/null > +++ b/drivers/gpu/drm/i915/intel_bios_private.h > @@ -0,0 +1,823 @@ > +/* > + * Copyright © 2006-2016 Intel Corporation > + * > + * Permission is hereby granted, free of charge, to any person obtaining a > + * copy of this software and associated documentation files (the "Software"), > + * to deal in the Software without restriction, including without limitation > + * the rights to use, copy, modify, merge, publish, distribute, sublicense, > + * and/or sell copies of the Software, and to permit persons to whom the > + * Software is furnished to do so, subject to the following conditions: > + * > + * The above copyright notice and this permission notice (including the next > + * paragraph) shall be included in all copies or substantial portions of the > + * Software. > + * > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL > + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, > + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE > + * SOFTWARE. > + * > + * Authors: > + * Eric Anholt <eric@anholt.net> > + * > + */ > + > +/* > + * This information is private to VBT parsing in intel_bios.c. > + * > + * Please do NOT include anywhere else. > + */ > + > +#ifndef _INTEL_BIOS_PRIVATE_H_ > +#define _INTEL_BIOS_PRIVATE_H_ > + > +#include "intel_bios.h" > + > +/** > + * struct vbt_header - VBT Header structure > + * @signature: VBT signature, always starts with "$VBT" > + * @version: Version of this structure > + * @header_size: Size of this structure > + * @vbt_size: Size of VBT (VBT Header, BDB Header and data blocks) > + * @vbt_checksum: Checksum > + * @reserved0: Reserved > + * @bdb_offset: Offset of &struct bdb_header from beginning of VBT > + * @aim_offset: Offsets of add-in data blocks from beginning of VBT > + */ > +struct vbt_header { > + u8 signature[20]; > + u16 version; > + u16 header_size; > + u16 vbt_size; > + u8 vbt_checksum; > + u8 reserved0; > + u32 bdb_offset; > + u32 aim_offset[4]; > +} __packed; > + > +/** > + * struct bdb_header - BDB Header structure > + * @signature: BDB signature "BIOS_DATA_BLOCK" > + * @version: Version of the data block definitions > + * @header_size: Size of this structure > + * @bdb_size: Size of BDB (BDB Header and data blocks) > + */ > +struct bdb_header { > + u8 signature[16]; > + u16 version; > + u16 header_size; > + u16 bdb_size; > +} __packed; > + > +/* strictly speaking, this is a "skip" block, but it has interesting info */ > +struct vbios_data { > + u8 type; /* 0 == desktop, 1 == mobile */ > + u8 relstage; > + u8 chipset; > + u8 lvds_present:1; > + u8 tv_present:1; > + u8 rsvd2:6; /* finish byte */ > + u8 rsvd3[4]; > + u8 signon[155]; > + u8 copyright[61]; > + u16 code_segment; > + u8 dos_boot_mode; > + u8 bandwidth_percent; > + u8 rsvd4; /* popup memory size */ > + u8 resize_pci_bios; > + u8 rsvd5; /* is crt already on ddc2 */ > +} __packed; > + > +/* > + * There are several types of BIOS data blocks (BDBs), each block has > + * an ID and size in the first 3 bytes (ID in first, size in next 2). > + * Known types are listed below. > + */ > +#define BDB_GENERAL_FEATURES 1 > +#define BDB_GENERAL_DEFINITIONS 2 > +#define BDB_OLD_TOGGLE_LIST 3 > +#define BDB_MODE_SUPPORT_LIST 4 > +#define BDB_GENERIC_MODE_TABLE 5 > +#define BDB_EXT_MMIO_REGS 6 > +#define BDB_SWF_IO 7 > +#define BDB_SWF_MMIO 8 > +#define BDB_PSR 9 > +#define BDB_MODE_REMOVAL_TABLE 10 > +#define BDB_CHILD_DEVICE_TABLE 11 > +#define BDB_DRIVER_FEATURES 12 > +#define BDB_DRIVER_PERSISTENCE 13 > +#define BDB_EXT_TABLE_PTRS 14 > +#define BDB_DOT_CLOCK_OVERRIDE 15 > +#define BDB_DISPLAY_SELECT 16 > +/* 17 rsvd */ > +#define BDB_DRIVER_ROTATION 18 > +#define BDB_DISPLAY_REMOVE 19 > +#define BDB_OEM_CUSTOM 20 > +#define BDB_EFP_LIST 21 /* workarounds for VGA hsync/vsync */ > +#define BDB_SDVO_LVDS_OPTIONS 22 > +#define BDB_SDVO_PANEL_DTDS 23 > +#define BDB_SDVO_LVDS_PNP_IDS 24 > +#define BDB_SDVO_LVDS_POWER_SEQ 25 > +#define BDB_TV_OPTIONS 26 > +#define BDB_EDP 27 > +#define BDB_LVDS_OPTIONS 40 > +#define BDB_LVDS_LFP_DATA_PTRS 41 > +#define BDB_LVDS_LFP_DATA 42 > +#define BDB_LVDS_BACKLIGHT 43 > +#define BDB_LVDS_POWER 44 > +#define BDB_MIPI_CONFIG 52 > +#define BDB_MIPI_SEQUENCE 53 > +#define BDB_SKIP 254 /* VBIOS private block, ignore */ > + > +struct bdb_general_features { > + /* bits 1 */ > + u8 panel_fitting:2; > + u8 flexaim:1; > + u8 msg_enable:1; > + u8 clear_screen:3; > + u8 color_flip:1; > + > + /* bits 2 */ > + u8 download_ext_vbt:1; > + u8 enable_ssc:1; > + u8 ssc_freq:1; > + u8 enable_lfp_on_override:1; > + u8 disable_ssc_ddt:1; > + u8 rsvd7:1; > + u8 display_clock_mode:1; > + u8 rsvd8:1; /* finish byte */ > + > + /* bits 3 */ > + u8 disable_smooth_vision:1; > + u8 single_dvi:1; > + u8 rsvd9:1; > + u8 fdi_rx_polarity_inverted:1; > + u8 rsvd10:4; /* finish byte */ > + > + /* bits 4 */ > + u8 legacy_monitor_detect; > + > + /* bits 5 */ > + u8 int_crt_support:1; > + u8 int_tv_support:1; > + u8 int_efp_support:1; > + u8 dp_ssc_enb:1; /* PCH attached eDP supports SSC */ > + u8 dp_ssc_freq:1; /* SSC freq for PCH attached eDP */ > + u8 rsvd11:3; /* finish byte */ > +} __packed; > + > +/* pre-915 */ > +#define GPIO_PIN_DVI_LVDS 0x03 /* "DVI/LVDS DDC GPIO pins" */ > +#define GPIO_PIN_ADD_I2C 0x05 /* "ADDCARD I2C GPIO pins" */ > +#define GPIO_PIN_ADD_DDC 0x04 /* "ADDCARD DDC GPIO pins" */ > +#define GPIO_PIN_ADD_DDC_I2C 0x06 /* "ADDCARD DDC/I2C GPIO pins" */ > + > +/* Pre 915 */ > +#define DEVICE_TYPE_NONE 0x00 > +#define DEVICE_TYPE_CRT 0x01 > +#define DEVICE_TYPE_TV 0x09 > +#define DEVICE_TYPE_EFP 0x12 > +#define DEVICE_TYPE_LFP 0x22 > +/* On 915+ */ > +#define DEVICE_TYPE_CRT_DPMS 0x6001 > +#define DEVICE_TYPE_CRT_DPMS_HOTPLUG 0x4001 > +#define DEVICE_TYPE_TV_COMPOSITE 0x0209 > +#define DEVICE_TYPE_TV_MACROVISION 0x0289 > +#define DEVICE_TYPE_TV_RF_COMPOSITE 0x020c > +#define DEVICE_TYPE_TV_SVIDEO_COMPOSITE 0x0609 > +#define DEVICE_TYPE_TV_SCART 0x0209 > +#define DEVICE_TYPE_TV_CODEC_HOTPLUG_PWR 0x6009 > +#define DEVICE_TYPE_EFP_HOTPLUG_PWR 0x6012 > +#define DEVICE_TYPE_EFP_DVI_HOTPLUG_PWR 0x6052 > +#define DEVICE_TYPE_EFP_DVI_I 0x6053 > +#define DEVICE_TYPE_EFP_DVI_D_DUAL 0x6152 > +#define DEVICE_TYPE_EFP_DVI_D_HDCP 0x60d2 > +#define DEVICE_TYPE_OPENLDI_HOTPLUG_PWR 0x6062 > +#define DEVICE_TYPE_OPENLDI_DUALPIX 0x6162 > +#define DEVICE_TYPE_LFP_PANELLINK 0x5012 > +#define DEVICE_TYPE_LFP_CMOS_PWR 0x5042 > +#define DEVICE_TYPE_LFP_LVDS_PWR 0x5062 > +#define DEVICE_TYPE_LFP_LVDS_DUAL 0x5162 > +#define DEVICE_TYPE_LFP_LVDS_DUAL_HDCP 0x51e2 > + > +#define DEVICE_CFG_NONE 0x00 > +#define DEVICE_CFG_12BIT_DVOB 0x01 > +#define DEVICE_CFG_12BIT_DVOC 0x02 > +#define DEVICE_CFG_24BIT_DVOBC 0x09 > +#define DEVICE_CFG_24BIT_DVOCB 0x0a > +#define DEVICE_CFG_DUAL_DVOB 0x11 > +#define DEVICE_CFG_DUAL_DVOC 0x12 > +#define DEVICE_CFG_DUAL_DVOBC 0x13 > +#define DEVICE_CFG_DUAL_LINK_DVOBC 0x19 > +#define DEVICE_CFG_DUAL_LINK_DVOCB 0x1a > + > +#define DEVICE_WIRE_NONE 0x00 > +#define DEVICE_WIRE_DVOB 0x01 > +#define DEVICE_WIRE_DVOC 0x02 > +#define DEVICE_WIRE_DVOBC 0x03 > +#define DEVICE_WIRE_DVOBB 0x05 > +#define DEVICE_WIRE_DVOCC 0x06 > +#define DEVICE_WIRE_DVOB_MASTER 0x0d > +#define DEVICE_WIRE_DVOC_MASTER 0x0e > + > +#define DEVICE_PORT_DVOA 0x00 /* none on 845+ */ > +#define DEVICE_PORT_DVOB 0x01 > +#define DEVICE_PORT_DVOC 0x02 > + > +/* > + * We used to keep this struct but without any version control. We should avoid > + * using it in the future, but it should be safe to keep using it in the old > + * code. Do not change; we rely on its size. > + */ > +struct old_child_dev_config { > + u16 handle; > + u16 device_type; > + u8 device_id[10]; /* ascii string */ > + u16 addin_offset; > + u8 dvo_port; /* See Device_PORT_* above */ > + u8 i2c_pin; > + u8 slave_addr; > + u8 ddc_pin; > + u16 edid_ptr; > + u8 dvo_cfg; /* See DEVICE_CFG_* above */ > + u8 dvo2_port; > + u8 i2c2_pin; > + u8 slave2_addr; > + u8 ddc2_pin; > + u8 capabilities; > + u8 dvo_wiring;/* See DEVICE_WIRE_* above */ > + u8 dvo2_wiring; > + u16 extended_type; > + u8 dvo_function; > +} __packed; > + > +/* This one contains field offsets that are known to be common for all BDB > + * versions. Notice that the meaning of the contents contents may still change, > + * but at least the offsets are consistent. */ > + > +/* Definitions for flags_1 */ > +#define IBOOST_ENABLE (1<<3) > + > +struct common_child_dev_config { > + u16 handle; > + u16 device_type; > + u8 not_common1[12]; > + u8 dvo_port; > + u8 not_common2[2]; > + u8 ddc_pin; > + u16 edid_ptr; > + u8 obsolete; > + u8 flags_1; > + u8 not_common3[13]; > + u8 iboost_level; > +} __packed; > + > + > +/* This field changes depending on the BDB version, so the most reliable way to > + * read it is by checking the BDB version and reading the raw pointer. */ > +union child_device_config { > + /* This one is safe to be used anywhere, but the code should still check > + * the BDB version. */ > + u8 raw[33]; > + /* This one should only be kept for legacy code. */ > + struct old_child_dev_config old; > + /* This one should also be safe to use anywhere, even without version > + * checks. */ > + struct common_child_dev_config common; > +} __packed; > + > +struct bdb_general_definitions { > + /* DDC GPIO */ > + u8 crt_ddc_gmbus_pin; > + > + /* DPMS bits */ > + u8 dpms_acpi:1; > + u8 skip_boot_crt_detect:1; > + u8 dpms_aim:1; > + u8 rsvd1:5; /* finish byte */ > + > + /* boot device bits */ > + u8 boot_display[2]; > + u8 child_dev_size; > + > + /* > + * Device info: > + * If TV is present, it'll be at devices[0]. > + * LVDS will be next, either devices[0] or [1], if present. > + * On some platforms the number of device is 6. But could be as few as > + * 4 if both TV and LVDS are missing. > + * And the device num is related with the size of general definition > + * block. It is obtained by using the following formula: > + * number = (block_size - sizeof(bdb_general_definitions))/ > + * defs->child_dev_size; > + */ > + uint8_t devices[0]; > +} __packed; > + > +/* Mask for DRRS / Panel Channel / SSC / BLT control bits extraction */ > +#define MODE_MASK 0x3 > + > +struct bdb_lvds_options { > + u8 panel_type; > + u8 rsvd1; > + /* LVDS capabilities, stored in a dword */ > + u8 pfit_mode:2; > + u8 pfit_text_mode_enhanced:1; > + u8 pfit_gfx_mode_enhanced:1; > + u8 pfit_ratio_auto:1; > + u8 pixel_dither:1; > + u8 lvds_edid:1; > + u8 rsvd2:1; > + u8 rsvd4; > + /* LVDS Panel channel bits stored here */ > + u32 lvds_panel_channel_bits; > + /* LVDS SSC (Spread Spectrum Clock) bits stored here. */ > + u16 ssc_bits; > + u16 ssc_freq; > + u16 ssc_ddt; > + /* Panel color depth defined here */ > + u16 panel_color_depth; > + /* LVDS panel type bits stored here */ > + u32 dps_panel_type_bits; > + /* LVDS backlight control type bits stored here */ > + u32 blt_control_type_bits; > +} __packed; > + > +/* LFP pointer table contains entries to the struct below */ > +struct bdb_lvds_lfp_data_ptr { > + u16 fp_timing_offset; /* offsets are from start of bdb */ > + u8 fp_table_size; > + u16 dvo_timing_offset; > + u8 dvo_table_size; > + u16 panel_pnp_id_offset; > + u8 pnp_table_size; > +} __packed; > + > +struct bdb_lvds_lfp_data_ptrs { > + u8 lvds_entries; /* followed by one or more lvds_data_ptr structs */ > + struct bdb_lvds_lfp_data_ptr ptr[16]; > +} __packed; > + > +/* LFP data has 3 blocks per entry */ > +struct lvds_fp_timing { > + u16 x_res; > + u16 y_res; > + u32 lvds_reg; > + u32 lvds_reg_val; > + u32 pp_on_reg; > + u32 pp_on_reg_val; > + u32 pp_off_reg; > + u32 pp_off_reg_val; > + u32 pp_cycle_reg; > + u32 pp_cycle_reg_val; > + u32 pfit_reg; > + u32 pfit_reg_val; > + u16 terminator; > +} __packed; > + > +struct lvds_dvo_timing { > + u16 clock; /**< In 10khz */ > + u8 hactive_lo; > + u8 hblank_lo; > + u8 hblank_hi:4; > + u8 hactive_hi:4; > + u8 vactive_lo; > + u8 vblank_lo; > + u8 vblank_hi:4; > + u8 vactive_hi:4; > + u8 hsync_off_lo; > + u8 hsync_pulse_width; > + u8 vsync_pulse_width:4; > + u8 vsync_off:4; > + u8 rsvd0:6; > + u8 hsync_off_hi:2; > + u8 h_image; > + u8 v_image; > + u8 max_hv; > + u8 h_border; > + u8 v_border; > + u8 rsvd1:3; > + u8 digital:2; > + u8 vsync_positive:1; > + u8 hsync_positive:1; > + u8 rsvd2:1; > +} __packed; > + > +struct lvds_pnp_id { > + u16 mfg_name; > + u16 product_code; > + u32 serial; > + u8 mfg_week; > + u8 mfg_year; > +} __packed; > + > +struct bdb_lvds_lfp_data_entry { > + struct lvds_fp_timing fp_timing; > + struct lvds_dvo_timing dvo_timing; > + struct lvds_pnp_id pnp_id; > +} __packed; > + > +struct bdb_lvds_lfp_data { > + struct bdb_lvds_lfp_data_entry data[16]; > +} __packed; > + > +#define BDB_BACKLIGHT_TYPE_NONE 0 > +#define BDB_BACKLIGHT_TYPE_PWM 2 > + > +struct bdb_lfp_backlight_data_entry { > + u8 type:2; > + u8 active_low_pwm:1; > + u8 obsolete1:5; > + u16 pwm_freq_hz; > + u8 min_brightness; > + u8 obsolete2; > + u8 obsolete3; > +} __packed; > + > +struct bdb_lfp_backlight_data { > + u8 entry_size; > + struct bdb_lfp_backlight_data_entry data[16]; > + u8 level[16]; > +} __packed; > + > +struct aimdb_header { > + char signature[16]; > + char oem_device[20]; > + u16 aimdb_version; > + u16 aimdb_header_size; > + u16 aimdb_size; > +} __packed; > + > +struct aimdb_block { > + u8 aimdb_id; > + u16 aimdb_size; > +} __packed; > + > +struct vch_panel_data { > + u16 fp_timing_offset; > + u8 fp_timing_size; > + u16 dvo_timing_offset; > + u8 dvo_timing_size; > + u16 text_fitting_offset; > + u8 text_fitting_size; > + u16 graphics_fitting_offset; > + u8 graphics_fitting_size; > +} __packed; > + > +struct vch_bdb_22 { > + struct aimdb_block aimdb_block; > + struct vch_panel_data panels[16]; > +} __packed; > + > +struct bdb_sdvo_lvds_options { > + u8 panel_backlight; > + u8 h40_set_panel_type; > + u8 panel_type; > + u8 ssc_clk_freq; > + u16 als_low_trip; > + u16 als_high_trip; > + u8 sclalarcoeff_tab_row_num; > + u8 sclalarcoeff_tab_row_size; > + u8 coefficient[8]; > + u8 panel_misc_bits_1; > + u8 panel_misc_bits_2; > + u8 panel_misc_bits_3; > + u8 panel_misc_bits_4; > +} __packed; > + > + > +#define BDB_DRIVER_FEATURE_NO_LVDS 0 > +#define BDB_DRIVER_FEATURE_INT_LVDS 1 > +#define BDB_DRIVER_FEATURE_SDVO_LVDS 2 > +#define BDB_DRIVER_FEATURE_EDP 3 > + > +struct bdb_driver_features { > + u8 boot_dev_algorithm:1; > + u8 block_display_switch:1; > + u8 allow_display_switch:1; > + u8 hotplug_dvo:1; > + u8 dual_view_zoom:1; > + u8 int15h_hook:1; > + u8 sprite_in_clone:1; > + u8 primary_lfp_id:1; > + > + u16 boot_mode_x; > + u16 boot_mode_y; > + u8 boot_mode_bpp; > + u8 boot_mode_refresh; > + > + u16 enable_lfp_primary:1; > + u16 selective_mode_pruning:1; > + u16 dual_frequency:1; > + u16 render_clock_freq:1; /* 0: high freq; 1: low freq */ > + u16 nt_clone_support:1; > + u16 power_scheme_ui:1; /* 0: CUI; 1: 3rd party */ > + u16 sprite_display_assign:1; /* 0: secondary; 1: primary */ > + u16 cui_aspect_scaling:1; > + u16 preserve_aspect_ratio:1; > + u16 sdvo_device_power_down:1; > + u16 crt_hotplug:1; > + u16 lvds_config:2; > + u16 tv_hotplug:1; > + u16 hdmi_config:2; > + > + u8 static_display:1; > + u8 reserved2:7; > + u16 legacy_crt_max_x; > + u16 legacy_crt_max_y; > + u8 legacy_crt_max_refresh; > + > + u8 hdmi_termination; > + u8 custom_vbt_version; > + /* Driver features data block */ > + u16 rmpm_enabled:1; > + u16 s2ddt_enabled:1; > + u16 dpst_enabled:1; > + u16 bltclt_enabled:1; > + u16 adb_enabled:1; > + u16 drrs_enabled:1; > + u16 grs_enabled:1; > + u16 gpmt_enabled:1; > + u16 tbt_enabled:1; > + u16 psr_enabled:1; > + u16 ips_enabled:1; > + u16 reserved3:4; > + u16 pc_feature_valid:1; > +} __packed; > + > +#define EDP_18BPP 0 > +#define EDP_24BPP 1 > +#define EDP_30BPP 2 > +#define EDP_RATE_1_62 0 > +#define EDP_RATE_2_7 1 > +#define EDP_LANE_1 0 > +#define EDP_LANE_2 1 > +#define EDP_LANE_4 3 > +#define EDP_PREEMPHASIS_NONE 0 > +#define EDP_PREEMPHASIS_3_5dB 1 > +#define EDP_PREEMPHASIS_6dB 2 > +#define EDP_PREEMPHASIS_9_5dB 3 > +#define EDP_VSWING_0_4V 0 > +#define EDP_VSWING_0_6V 1 > +#define EDP_VSWING_0_8V 2 > +#define EDP_VSWING_1_2V 3 > + > + > +struct edp_link_params { > + u8 rate:4; > + u8 lanes:4; > + u8 preemphasis:4; > + u8 vswing:4; > +} __packed; > + > +struct bdb_edp { > + struct edp_power_seq power_seqs[16]; > + u32 color_depth; > + struct edp_link_params link_params[16]; > + u32 sdrrs_msa_timing_delay; > + > + /* ith bit indicates enabled/disabled for (i+1)th panel */ > + u16 edp_s3d_feature; > + u16 edp_t3_optimization; > + u64 edp_vswing_preemph; /* v173 */ > +} __packed; > + > +struct psr_table { > + /* Feature bits */ > + u8 full_link:1; > + u8 require_aux_to_wakeup:1; > + u8 feature_bits_rsvd:6; > + > + /* Wait times */ > + u8 idle_frames:4; > + u8 lines_to_wait:3; > + u8 wait_times_rsvd:1; > + > + /* TP wake up time in multiple of 100 */ > + u16 tp1_wakeup_time; > + u16 tp2_tp3_wakeup_time; > +} __packed; > + > +struct bdb_psr { > + struct psr_table psr_table[16]; > +} __packed; > + > +/* > + * Driver<->VBIOS interaction occurs through scratch bits in > + * GR18 & SWF*. > + */ > + > +/* GR18 bits are set on display switch and hotkey events */ > +#define GR18_DRIVER_SWITCH_EN (1<<7) /* 0: VBIOS control, 1: driver control */ > +#define GR18_HOTKEY_MASK 0x78 /* See also SWF4 15:0 */ > +#define GR18_HK_NONE (0x0<<3) > +#define GR18_HK_LFP_STRETCH (0x1<<3) > +#define GR18_HK_TOGGLE_DISP (0x2<<3) > +#define GR18_HK_DISP_SWITCH (0x4<<3) /* see SWF14 15:0 for what to enable */ > +#define GR18_HK_POPUP_DISABLED (0x6<<3) > +#define GR18_HK_POPUP_ENABLED (0x7<<3) > +#define GR18_HK_PFIT (0x8<<3) > +#define GR18_HK_APM_CHANGE (0xa<<3) > +#define GR18_HK_MULTIPLE (0xc<<3) > +#define GR18_USER_INT_EN (1<<2) > +#define GR18_A0000_FLUSH_EN (1<<1) > +#define GR18_SMM_EN (1<<0) > + > +/* Set by driver, cleared by VBIOS */ > +#define SWF00_YRES_SHIFT 16 > +#define SWF00_XRES_SHIFT 0 > +#define SWF00_RES_MASK 0xffff > + > +/* Set by VBIOS at boot time and driver at runtime */ > +#define SWF01_TV2_FORMAT_SHIFT 8 > +#define SWF01_TV1_FORMAT_SHIFT 0 > +#define SWF01_TV_FORMAT_MASK 0xffff > + > +#define SWF10_VBIOS_BLC_I2C_EN (1<<29) > +#define SWF10_GTT_OVERRIDE_EN (1<<28) > +#define SWF10_LFP_DPMS_OVR (1<<27) /* override DPMS on display switch */ > +#define SWF10_ACTIVE_TOGGLE_LIST_MASK (7<<24) > +#define SWF10_OLD_TOGGLE 0x0 > +#define SWF10_TOGGLE_LIST_1 0x1 > +#define SWF10_TOGGLE_LIST_2 0x2 > +#define SWF10_TOGGLE_LIST_3 0x3 > +#define SWF10_TOGGLE_LIST_4 0x4 > +#define SWF10_PANNING_EN (1<<23) > +#define SWF10_DRIVER_LOADED (1<<22) > +#define SWF10_EXTENDED_DESKTOP (1<<21) > +#define SWF10_EXCLUSIVE_MODE (1<<20) > +#define SWF10_OVERLAY_EN (1<<19) > +#define SWF10_PLANEB_HOLDOFF (1<<18) > +#define SWF10_PLANEA_HOLDOFF (1<<17) > +#define SWF10_VGA_HOLDOFF (1<<16) > +#define SWF10_ACTIVE_DISP_MASK 0xffff > +#define SWF10_PIPEB_LFP2 (1<<15) > +#define SWF10_PIPEB_EFP2 (1<<14) > +#define SWF10_PIPEB_TV2 (1<<13) > +#define SWF10_PIPEB_CRT2 (1<<12) > +#define SWF10_PIPEB_LFP (1<<11) > +#define SWF10_PIPEB_EFP (1<<10) > +#define SWF10_PIPEB_TV (1<<9) > +#define SWF10_PIPEB_CRT (1<<8) > +#define SWF10_PIPEA_LFP2 (1<<7) > +#define SWF10_PIPEA_EFP2 (1<<6) > +#define SWF10_PIPEA_TV2 (1<<5) > +#define SWF10_PIPEA_CRT2 (1<<4) > +#define SWF10_PIPEA_LFP (1<<3) > +#define SWF10_PIPEA_EFP (1<<2) > +#define SWF10_PIPEA_TV (1<<1) > +#define SWF10_PIPEA_CRT (1<<0) > + > +#define SWF11_MEMORY_SIZE_SHIFT 16 > +#define SWF11_SV_TEST_EN (1<<15) > +#define SWF11_IS_AGP (1<<14) > +#define SWF11_DISPLAY_HOLDOFF (1<<13) > +#define SWF11_DPMS_REDUCED (1<<12) > +#define SWF11_IS_VBE_MODE (1<<11) > +#define SWF11_PIPEB_ACCESS (1<<10) /* 0 here means pipe a */ > +#define SWF11_DPMS_MASK 0x07 > +#define SWF11_DPMS_OFF (1<<2) > +#define SWF11_DPMS_SUSPEND (1<<1) > +#define SWF11_DPMS_STANDBY (1<<0) > +#define SWF11_DPMS_ON 0 > + > +#define SWF14_GFX_PFIT_EN (1<<31) > +#define SWF14_TEXT_PFIT_EN (1<<30) > +#define SWF14_LID_STATUS_CLOSED (1<<29) /* 0 here means open */ > +#define SWF14_POPUP_EN (1<<28) > +#define SWF14_DISPLAY_HOLDOFF (1<<27) > +#define SWF14_DISP_DETECT_EN (1<<26) > +#define SWF14_DOCKING_STATUS_DOCKED (1<<25) /* 0 here means undocked */ > +#define SWF14_DRIVER_STATUS (1<<24) > +#define SWF14_OS_TYPE_WIN9X (1<<23) > +#define SWF14_OS_TYPE_WINNT (1<<22) > +/* 21:19 rsvd */ > +#define SWF14_PM_TYPE_MASK 0x00070000 > +#define SWF14_PM_ACPI_VIDEO (0x4 << 16) > +#define SWF14_PM_ACPI (0x3 << 16) > +#define SWF14_PM_APM_12 (0x2 << 16) > +#define SWF14_PM_APM_11 (0x1 << 16) > +#define SWF14_HK_REQUEST_MASK 0x0000ffff /* see GR18 6:3 for event type */ > + /* if GR18 indicates a display switch */ > +#define SWF14_DS_PIPEB_LFP2_EN (1<<15) > +#define SWF14_DS_PIPEB_EFP2_EN (1<<14) > +#define SWF14_DS_PIPEB_TV2_EN (1<<13) > +#define SWF14_DS_PIPEB_CRT2_EN (1<<12) > +#define SWF14_DS_PIPEB_LFP_EN (1<<11) > +#define SWF14_DS_PIPEB_EFP_EN (1<<10) > +#define SWF14_DS_PIPEB_TV_EN (1<<9) > +#define SWF14_DS_PIPEB_CRT_EN (1<<8) > +#define SWF14_DS_PIPEA_LFP2_EN (1<<7) > +#define SWF14_DS_PIPEA_EFP2_EN (1<<6) > +#define SWF14_DS_PIPEA_TV2_EN (1<<5) > +#define SWF14_DS_PIPEA_CRT2_EN (1<<4) > +#define SWF14_DS_PIPEA_LFP_EN (1<<3) > +#define SWF14_DS_PIPEA_EFP_EN (1<<2) > +#define SWF14_DS_PIPEA_TV_EN (1<<1) > +#define SWF14_DS_PIPEA_CRT_EN (1<<0) > + /* if GR18 indicates a panel fitting request */ > +#define SWF14_PFIT_EN (1<<0) /* 0 means disable */ > + /* if GR18 indicates an APM change request */ > +#define SWF14_APM_HIBERNATE 0x4 > +#define SWF14_APM_SUSPEND 0x3 > +#define SWF14_APM_STANDBY 0x1 > +#define SWF14_APM_RESTORE 0x0 > + > +/* Add the device class for LFP, TV, HDMI */ > +#define DEVICE_TYPE_INT_LFP 0x1022 > +#define DEVICE_TYPE_INT_TV 0x1009 > +#define DEVICE_TYPE_HDMI 0x60D2 > +#define DEVICE_TYPE_DP 0x68C6 > +#define DEVICE_TYPE_eDP 0x78C6 > + > +#define DEVICE_TYPE_CLASS_EXTENSION (1 << 15) > +#define DEVICE_TYPE_POWER_MANAGEMENT (1 << 14) > +#define DEVICE_TYPE_HOTPLUG_SIGNALING (1 << 13) > +#define DEVICE_TYPE_INTERNAL_CONNECTOR (1 << 12) > +#define DEVICE_TYPE_NOT_HDMI_OUTPUT (1 << 11) > +#define DEVICE_TYPE_MIPI_OUTPUT (1 << 10) > +#define DEVICE_TYPE_COMPOSITE_OUTPUT (1 << 9) > +#define DEVICE_TYPE_DUAL_CHANNEL (1 << 8) > +#define DEVICE_TYPE_HIGH_SPEED_LINK (1 << 6) > +#define DEVICE_TYPE_LVDS_SINGALING (1 << 5) > +#define DEVICE_TYPE_TMDS_DVI_SIGNALING (1 << 4) > +#define DEVICE_TYPE_VIDEO_SIGNALING (1 << 3) > +#define DEVICE_TYPE_DISPLAYPORT_OUTPUT (1 << 2) > +#define DEVICE_TYPE_DIGITAL_OUTPUT (1 << 1) > +#define DEVICE_TYPE_ANALOG_OUTPUT (1 << 0) > + > +/* > + * Bits we care about when checking for DEVICE_TYPE_eDP > + * Depending on the system, the other bits may or may not > + * be set for eDP outputs. > + */ > +#define DEVICE_TYPE_eDP_BITS \ > + (DEVICE_TYPE_INTERNAL_CONNECTOR | \ > + DEVICE_TYPE_MIPI_OUTPUT | \ > + DEVICE_TYPE_COMPOSITE_OUTPUT | \ > + DEVICE_TYPE_DUAL_CHANNEL | \ > + DEVICE_TYPE_LVDS_SINGALING | \ > + DEVICE_TYPE_TMDS_DVI_SIGNALING | \ > + DEVICE_TYPE_VIDEO_SIGNALING | \ > + DEVICE_TYPE_DISPLAYPORT_OUTPUT | \ > + DEVICE_TYPE_ANALOG_OUTPUT) > + > +/* define the DVO port for HDMI output type */ > +#define DVO_B 1 > +#define DVO_C 2 > +#define DVO_D 3 > + > +/* Possible values for the "DVO Port" field for versions >= 155: */ > +#define DVO_PORT_HDMIA 0 > +#define DVO_PORT_HDMIB 1 > +#define DVO_PORT_HDMIC 2 > +#define DVO_PORT_HDMID 3 > +#define DVO_PORT_LVDS 4 > +#define DVO_PORT_TV 5 > +#define DVO_PORT_CRT 6 > +#define DVO_PORT_DPB 7 > +#define DVO_PORT_DPC 8 > +#define DVO_PORT_DPD 9 > +#define DVO_PORT_DPA 10 > +#define DVO_PORT_DPE 11 > +#define DVO_PORT_HDMIE 12 > +#define DVO_PORT_MIPIA 21 > +#define DVO_PORT_MIPIB 22 > +#define DVO_PORT_MIPIC 23 > +#define DVO_PORT_MIPID 24 > + > +/* Block 52 contains MIPI configuration block > + * 6 * bdb_mipi_config, followed by 6 pps data block > + * block below > + */ > +#define MAX_MIPI_CONFIGURATIONS 6 > + > +struct bdb_mipi_config { > + struct mipi_config config[MAX_MIPI_CONFIGURATIONS]; > + struct mipi_pps_data pps[MAX_MIPI_CONFIGURATIONS]; > +} __packed; > + > +/* Block 53 contains MIPI sequences as needed by the panel > + * for enabling it. This block can be variable in size and > + * can be maximum of 6 blocks > + */ > +struct bdb_mipi_sequence { > + u8 version; > + u8 data[0]; > +} __packed; > + > +enum mipi_gpio_pin_index { > + MIPI_GPIO_UNDEFINED = 0, > + MIPI_GPIO_PANEL_ENABLE, > + MIPI_GPIO_BL_ENABLE, > + MIPI_GPIO_PWM_ENABLE, > + MIPI_GPIO_RESET_N, > + MIPI_GPIO_PWR_DOWN_R, > + MIPI_GPIO_STDBY_RST_N, > + MIPI_GPIO_MAX > +}; > + > +#endif /* _INTEL_BIOS_PRIVATE_H_ */ _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v2 0/6] drm/i915: start hiding away vbt structure from the driver 2016-01-11 19:54 [PATCH v2 0/6] drm/i915: start hiding away vbt structure from the driver Jani Nikula ` (5 preceding siblings ...) 2016-01-11 19:54 ` [PATCH v2 6/6] drm/i915: hide away VBT private data in a separate header Jani Nikula @ 2016-01-11 20:07 ` Lukas Wunner 2016-01-12 5:51 ` Jani Nikula 2016-01-12 8:20 ` ✓ success: Fi.CI.BAT Patchwork 7 siblings, 1 reply; 17+ messages in thread From: Lukas Wunner @ 2016-01-11 20:07 UTC (permalink / raw) To: Jani Nikula; +Cc: intel-gfx Hi, On Mon, Jan 11, 2016 at 09:54:36PM +0200, Jani Nikula wrote: > Hi all, first real patches since the RFC at [1]. > > The VBT is a monster and it keeps growing. Originally we've extracted > bits and pieces out of there, and added them cleanly to our own > structures in dev_priv->vbt, with our own macros. Later on we've been > slipping and we have copied stuff from VBT verbatim, using the same > structs and defines as in VBT. There's also code spread out across the driver to sanitize VBT data, e.g.: http://cgit.freedesktop.org/drm-intel/tree/drivers/gpu/drm/i915/intel_display.c?h=drm-intel-nightly#n15424 (cf. 69f92f67b68a, "drm/i915: Preserve SSC earlier") Would probably make sense to consolidate that in intel_bios.c as well. Best regards, Lukas > > Start abstracting the VBT more, hiding away the VBT specific stuff that > should not be fiddled with outside of intel_bios.c. Make it more > explicit when we're doing so. At this stage, mostly just fork the > private parts into intel_bios_private.h. As prep work, move some stuff > to intel_bios.c to limit the amount of macros needing to be exposed > across the whole driver. > > With the MIPI DSI VBT stuff out of the way, this now applies on top of > nightly. > > > BR, > Jani. > > [1] http://mid.gmane.org/cover.1450192823.git.jani.nikula@intel.com > > > Jani Nikula (6): > drm/i915: move VBT based TV presence check to intel_bios.c > drm/i915: move VBT based LVDS presence check to intel_bios.c > drm/i915: move VBT based eDP port check to intel_bios.c > drm/i915: move VBT based DSI presence check to intel_bios.c > drm/i915/panel: setup pwm backlight based on connector type > drm/i915: hide away VBT private data in a separate header > > Documentation/DocBook/gpu.tmpl | 2 +- > drivers/gpu/drm/i915/i915_drv.h | 5 +- > drivers/gpu/drm/i915/intel_bios.c | 156 +++++- > drivers/gpu/drm/i915/intel_bios.h | 860 ++---------------------------- > drivers/gpu/drm/i915/intel_bios_private.h | 823 ++++++++++++++++++++++++++++ > drivers/gpu/drm/i915/intel_dp.c | 21 +- > drivers/gpu/drm/i915/intel_dsi.c | 23 +- > drivers/gpu/drm/i915/intel_lvds.c | 53 +- > drivers/gpu/drm/i915/intel_panel.c | 2 +- > drivers/gpu/drm/i915/intel_tv.c | 43 +- > 10 files changed, 1034 insertions(+), 954 deletions(-) > create mode 100644 drivers/gpu/drm/i915/intel_bios_private.h > > -- > 2.1.4 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH v2 0/6] drm/i915: start hiding away vbt structure from the driver 2016-01-11 20:07 ` [PATCH v2 0/6] drm/i915: start hiding away vbt structure from the driver Lukas Wunner @ 2016-01-12 5:51 ` Jani Nikula 0 siblings, 0 replies; 17+ messages in thread From: Jani Nikula @ 2016-01-12 5:51 UTC (permalink / raw) To: Lukas Wunner; +Cc: intel-gfx On Mon, 11 Jan 2016, Lukas Wunner <lukas@wunner.de> wrote: > Hi, > > On Mon, Jan 11, 2016 at 09:54:36PM +0200, Jani Nikula wrote: >> Hi all, first real patches since the RFC at [1]. >> >> The VBT is a monster and it keeps growing. Originally we've extracted >> bits and pieces out of there, and added them cleanly to our own >> structures in dev_priv->vbt, with our own macros. Later on we've been >> slipping and we have copied stuff from VBT verbatim, using the same >> structs and defines as in VBT. > > There's also code spread out across the driver to sanitize VBT data, e.g.: > http://cgit.freedesktop.org/drm-intel/tree/drivers/gpu/drm/i915/intel_display.c?h=drm-intel-nightly#n15424 > > (cf. 69f92f67b68a, "drm/i915: Preserve SSC earlier") > > Would probably make sense to consolidate that in intel_bios.c as well. It's worth noting that there's no mmio register access in intel_bios.c. It's just about parsing the VBT. I'd prefer to keep it that way. BR, Jani. > > Best regards, > > Lukas > >> >> Start abstracting the VBT more, hiding away the VBT specific stuff that >> should not be fiddled with outside of intel_bios.c. Make it more >> explicit when we're doing so. At this stage, mostly just fork the >> private parts into intel_bios_private.h. As prep work, move some stuff >> to intel_bios.c to limit the amount of macros needing to be exposed >> across the whole driver. >> >> With the MIPI DSI VBT stuff out of the way, this now applies on top of >> nightly. >> >> >> BR, >> Jani. >> >> [1] http://mid.gmane.org/cover.1450192823.git.jani.nikula@intel.com >> >> >> Jani Nikula (6): >> drm/i915: move VBT based TV presence check to intel_bios.c >> drm/i915: move VBT based LVDS presence check to intel_bios.c >> drm/i915: move VBT based eDP port check to intel_bios.c >> drm/i915: move VBT based DSI presence check to intel_bios.c >> drm/i915/panel: setup pwm backlight based on connector type >> drm/i915: hide away VBT private data in a separate header >> >> Documentation/DocBook/gpu.tmpl | 2 +- >> drivers/gpu/drm/i915/i915_drv.h | 5 +- >> drivers/gpu/drm/i915/intel_bios.c | 156 +++++- >> drivers/gpu/drm/i915/intel_bios.h | 860 ++---------------------------- >> drivers/gpu/drm/i915/intel_bios_private.h | 823 ++++++++++++++++++++++++++++ >> drivers/gpu/drm/i915/intel_dp.c | 21 +- >> drivers/gpu/drm/i915/intel_dsi.c | 23 +- >> drivers/gpu/drm/i915/intel_lvds.c | 53 +- >> drivers/gpu/drm/i915/intel_panel.c | 2 +- >> drivers/gpu/drm/i915/intel_tv.c | 43 +- >> 10 files changed, 1034 insertions(+), 954 deletions(-) >> create mode 100644 drivers/gpu/drm/i915/intel_bios_private.h >> >> -- >> 2.1.4 >> >> _______________________________________________ >> Intel-gfx mailing list >> Intel-gfx@lists.freedesktop.org >> http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 17+ messages in thread
* ✓ success: Fi.CI.BAT 2016-01-11 19:54 [PATCH v2 0/6] drm/i915: start hiding away vbt structure from the driver Jani Nikula ` (6 preceding siblings ...) 2016-01-11 20:07 ` [PATCH v2 0/6] drm/i915: start hiding away vbt structure from the driver Lukas Wunner @ 2016-01-12 8:20 ` Patchwork 7 siblings, 0 replies; 17+ messages in thread From: Patchwork @ 2016-01-12 8:20 UTC (permalink / raw) To: Jani Nikula; +Cc: intel-gfx == Summary == Built on a90796840c30dac6d9907439bf98d1d08046c49d drm-intel-nightly: 2016y-01m-11d-17h-22m-54s UTC integration manifest Test gem_storedw_loop: Subgroup basic-render: pass -> DMESG-WARN (skl-i5k-2) UNSTABLE Test kms_pipe_crc_basic: Subgroup suspend-read-crc-pipe-b: dmesg-warn -> PASS (ilk-hp8440p) bdw-nuci7 total:138 pass:129 dwarn:0 dfail:0 fail:0 skip:9 bdw-ultra total:138 pass:132 dwarn:0 dfail:0 fail:0 skip:6 bsw-nuc-2 total:141 pass:115 dwarn:2 dfail:0 fail:0 skip:24 byt-nuc total:141 pass:123 dwarn:3 dfail:0 fail:0 skip:15 hsw-brixbox total:141 pass:134 dwarn:0 dfail:0 fail:0 skip:7 hsw-gt2 total:141 pass:137 dwarn:0 dfail:0 fail:0 skip:4 ilk-hp8440p total:141 pass:102 dwarn:2 dfail:0 fail:0 skip:37 ivb-t430s total:135 pass:122 dwarn:3 dfail:4 fail:0 skip:6 skl-i5k-2 total:141 pass:131 dwarn:2 dfail:0 fail:0 skip:8 skl-i7k-2 total:141 pass:132 dwarn:1 dfail:0 fail:0 skip:8 snb-dellxps total:141 pass:122 dwarn:5 dfail:0 fail:0 skip:14 snb-x220t total:141 pass:122 dwarn:5 dfail:0 fail:1 skip:13 Results at /archive/results/CI_IGT_test/Patchwork_1138/ _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH v2 1/6] drm/i915: move VBT based TV presence check to intel_bios.c @ 2016-02-09 15:29 Jani Nikula 2016-02-09 15:29 ` [PATCH v2 2/6] drm/i915: move VBT based LVDS " Jani Nikula 0 siblings, 1 reply; 17+ messages in thread From: Jani Nikula @ 2016-02-09 15:29 UTC (permalink / raw) To: intel-gfx; +Cc: jani.nikula Hide knowledge about VBT child devices in intel_bios.c. v2: also move int_tv_support check to intel_bios.c (Sivakumar) Signed-off-by: Jani Nikula <jani.nikula@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/gpu/drm/i915/intel_bios.c | 41 ++++++++++++++++++++++++++++++++++ drivers/gpu/drm/i915/intel_tv.c | 46 +-------------------------------------- 3 files changed, 43 insertions(+), 45 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h index 8216665405eb..cf3ef03856db 100644 --- a/drivers/gpu/drm/i915/i915_drv.h +++ b/drivers/gpu/drm/i915/i915_drv.h @@ -3393,6 +3393,7 @@ extern void intel_i2c_reset(struct drm_device *dev); /* intel_bios.c */ int intel_bios_init(struct drm_i915_private *dev_priv); bool intel_bios_is_valid_vbt(const void *buf, size_t size); +bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv); /* intel_opregion.c */ #ifdef CONFIG_ACPI diff --git a/drivers/gpu/drm/i915/intel_bios.c b/drivers/gpu/drm/i915/intel_bios.c index bf62a19c8f69..d5e6da01adaa 100644 --- a/drivers/gpu/drm/i915/intel_bios.c +++ b/drivers/gpu/drm/i915/intel_bios.c @@ -1431,3 +1431,44 @@ intel_bios_init(struct drm_i915_private *dev_priv) return 0; } + +/** + * intel_bios_is_tv_present - is integrated TV present in VBT + * @dev_priv: i915 device instance + * + * Return true if TV is present. If no child devices were parsed from VBT, + * assume TV is present. + */ +bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv) +{ + union child_device_config *p_child; + int i; + + if (!dev_priv->vbt.int_tv_support) + return false; + + if (!dev_priv->vbt.child_dev_num) + return true; + + for (i = 0; i < dev_priv->vbt.child_dev_num; i++) { + p_child = dev_priv->vbt.child_dev + i; + /* + * If the device type is not TV, continue. + */ + switch (p_child->old.device_type) { + case DEVICE_TYPE_INT_TV: + case DEVICE_TYPE_TV: + case DEVICE_TYPE_TV_SVIDEO_COMPOSITE: + break; + default: + continue; + } + /* Only when the addin_offset is non-zero, it is regarded + * as present. + */ + if (p_child->old.addin_offset) + return true; + } + + return false; +} diff --git a/drivers/gpu/drm/i915/intel_tv.c b/drivers/gpu/drm/i915/intel_tv.c index 5034b0055169..c27a6aec25e0 100644 --- a/drivers/gpu/drm/i915/intel_tv.c +++ b/drivers/gpu/drm/i915/intel_tv.c @@ -1530,47 +1530,6 @@ static const struct drm_encoder_funcs intel_tv_enc_funcs = { .destroy = intel_encoder_destroy, }; -/* - * Enumerate the child dev array parsed from VBT to check whether - * the integrated TV is present. - * If it is present, return 1. - * If it is not present, return false. - * If no child dev is parsed from VBT, it assumes that the TV is present. - */ -static int tv_is_present_in_vbt(struct drm_device *dev) -{ - struct drm_i915_private *dev_priv = dev->dev_private; - union child_device_config *p_child; - int i, ret; - - if (!dev_priv->vbt.child_dev_num) - return 1; - - ret = 0; - for (i = 0; i < dev_priv->vbt.child_dev_num; i++) { - p_child = dev_priv->vbt.child_dev + i; - /* - * If the device type is not TV, continue. - */ - switch (p_child->old.device_type) { - case DEVICE_TYPE_INT_TV: - case DEVICE_TYPE_TV: - case DEVICE_TYPE_TV_SVIDEO_COMPOSITE: - break; - default: - continue; - } - /* Only when the addin_offset is non-zero, it is regarded - * as present. - */ - if (p_child->old.addin_offset) { - ret = 1; - break; - } - } - return ret; -} - void intel_tv_init(struct drm_device *dev) { @@ -1586,13 +1545,10 @@ intel_tv_init(struct drm_device *dev) if ((I915_READ(TV_CTL) & TV_FUSE_STATE_MASK) == TV_FUSE_STATE_DISABLED) return; - if (!tv_is_present_in_vbt(dev)) { + if (!intel_bios_is_tv_present(dev_priv)) { DRM_DEBUG_KMS("Integrated TV is not present.\n"); return; } - /* Even if we have an encoder we may not have a connector */ - if (!dev_priv->vbt.int_tv_support) - return; /* * Sanity check the TV output by checking to see if the -- 2.1.4 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 17+ messages in thread
* [PATCH v2 2/6] drm/i915: move VBT based LVDS presence check to intel_bios.c 2016-02-09 15:29 [PATCH v2 1/6] drm/i915: move VBT based TV presence check to intel_bios.c Jani Nikula @ 2016-02-09 15:29 ` Jani Nikula 0 siblings, 0 replies; 17+ messages in thread From: Jani Nikula @ 2016-02-09 15:29 UTC (permalink / raw) To: intel-gfx; +Cc: jani.nikula Hide knowledge about VBT child devices in intel_bios.c. Signed-off-by: Jani Nikula <jani.nikula@intel.com> --- drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/gpu/drm/i915/intel_bios.c | 50 ++++++++++++++++++++++++++++++++++++ drivers/gpu/drm/i915/intel_lvds.c | 53 +-------------------------------------- 3 files changed, 52 insertions(+), 52 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h index cf3ef03856db..24bf770907ae 100644 --- a/drivers/gpu/drm/i915/i915_drv.h +++ b/drivers/gpu/drm/i915/i915_drv.h @@ -3394,6 +3394,7 @@ extern void intel_i2c_reset(struct drm_device *dev); int intel_bios_init(struct drm_i915_private *dev_priv); bool intel_bios_is_valid_vbt(const void *buf, size_t size); bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv); +bool intel_bios_is_lvds_present(struct drm_i915_private *dev_priv, u8 *i2c_pin); /* intel_opregion.c */ #ifdef CONFIG_ACPI diff --git a/drivers/gpu/drm/i915/intel_bios.c b/drivers/gpu/drm/i915/intel_bios.c index d5e6da01adaa..4f7eba36a849 100644 --- a/drivers/gpu/drm/i915/intel_bios.c +++ b/drivers/gpu/drm/i915/intel_bios.c @@ -1472,3 +1472,53 @@ bool intel_bios_is_tv_present(struct drm_i915_private *dev_priv) return false; } + +/** + * intel_bios_is_lvds_present - is LVDS present in VBT + * @dev_priv: i915 device instance + * @i2c_pin: i2c pin for LVDS if present + * + * Return true if LVDS is present. If no child devices were parsed from VBT, + * assume LVDS is present. + */ +bool intel_bios_is_lvds_present(struct drm_i915_private *dev_priv, u8 *i2c_pin) +{ + int i; + + if (!dev_priv->vbt.child_dev_num) + return true; + + for (i = 0; i < dev_priv->vbt.child_dev_num; i++) { + union child_device_config *uchild = dev_priv->vbt.child_dev + i; + struct old_child_dev_config *child = &uchild->old; + + /* If the device type is not LFP, continue. + * We have to check both the new identifiers as well as the + * old for compatibility with some BIOSes. + */ + if (child->device_type != DEVICE_TYPE_INT_LFP && + child->device_type != DEVICE_TYPE_LFP) + continue; + + if (intel_gmbus_is_valid_pin(dev_priv, child->i2c_pin)) + *i2c_pin = child->i2c_pin; + + /* However, we cannot trust the BIOS writers to populate + * the VBT correctly. Since LVDS requires additional + * information from AIM blocks, a non-zero addin offset is + * a good indicator that the LVDS is actually present. + */ + if (child->addin_offset) + return true; + + /* But even then some BIOS writers perform some black magic + * and instantiate the device without reference to any + * additional data. Trust that if the VBT was written into + * the OpRegion then they have validated the LVDS's existence. + */ + if (dev_priv->opregion.vbt) + return true; + } + + return false; +} diff --git a/drivers/gpu/drm/i915/intel_lvds.c b/drivers/gpu/drm/i915/intel_lvds.c index 811ddf7799f0..325acfbc02f3 100644 --- a/drivers/gpu/drm/i915/intel_lvds.c +++ b/drivers/gpu/drm/i915/intel_lvds.c @@ -773,57 +773,6 @@ static const struct dmi_system_id intel_no_lvds[] = { { } /* terminating entry */ }; -/* - * Enumerate the child dev array parsed from VBT to check whether - * the LVDS is present. - * If it is present, return 1. - * If it is not present, return false. - * If no child dev is parsed from VBT, it assumes that the LVDS is present. - */ -static bool lvds_is_present_in_vbt(struct drm_device *dev, - u8 *i2c_pin) -{ - struct drm_i915_private *dev_priv = dev->dev_private; - int i; - - if (!dev_priv->vbt.child_dev_num) - return true; - - for (i = 0; i < dev_priv->vbt.child_dev_num; i++) { - union child_device_config *uchild = dev_priv->vbt.child_dev + i; - struct old_child_dev_config *child = &uchild->old; - - /* If the device type is not LFP, continue. - * We have to check both the new identifiers as well as the - * old for compatibility with some BIOSes. - */ - if (child->device_type != DEVICE_TYPE_INT_LFP && - child->device_type != DEVICE_TYPE_LFP) - continue; - - if (intel_gmbus_is_valid_pin(dev_priv, child->i2c_pin)) - *i2c_pin = child->i2c_pin; - - /* However, we cannot trust the BIOS writers to populate - * the VBT correctly. Since LVDS requires additional - * information from AIM blocks, a non-zero addin offset is - * a good indicator that the LVDS is actually present. - */ - if (child->addin_offset) - return true; - - /* But even then some BIOS writers perform some black magic - * and instantiate the device without reference to any - * additional data. Trust that if the VBT was written into - * the OpRegion then they have validated the LVDS's existence. - */ - if (dev_priv->opregion.vbt) - return true; - } - - return false; -} - static int intel_dual_link_lvds_callback(const struct dmi_system_id *id) { DRM_INFO("Forcing lvds to dual link mode on %s\n", id->ident); @@ -980,7 +929,7 @@ void intel_lvds_init(struct drm_device *dev) } pin = GMBUS_PIN_PANEL; - if (!lvds_is_present_in_vbt(dev, &pin)) { + if (!intel_bios_is_lvds_present(dev_priv, &pin)) { if ((lvds & LVDS_PORT_EN) == 0) { DRM_DEBUG_KMS("LVDS is not present in VBT\n"); return; -- 2.1.4 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply related [flat|nested] 17+ messages in thread
end of thread, other threads:[~2016-02-09 15:29 UTC | newest] Thread overview: 17+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-01-11 19:54 [PATCH v2 0/6] drm/i915: start hiding away vbt structure from the driver Jani Nikula 2016-01-11 19:54 ` [PATCH v2 1/6] drm/i915: move VBT based TV presence check to intel_bios.c Jani Nikula 2016-01-12 9:45 ` Daniel Vetter 2016-01-14 12:51 ` Mika Kahola 2016-01-11 19:54 ` [PATCH v2 2/6] drm/i915: move VBT based LVDS " Jani Nikula 2016-01-14 12:52 ` Mika Kahola 2016-01-11 19:54 ` [PATCH v2 3/6] drm/i915: move VBT based eDP port " Jani Nikula 2016-01-14 12:52 ` Mika Kahola 2016-01-11 19:54 ` [PATCH v2 4/6] drm/i915: move VBT based DSI presence " Jani Nikula 2016-01-14 12:52 ` Mika Kahola 2016-01-11 19:54 ` [PATCH v2 5/6] drm/i915/panel: setup pwm backlight based on connector type Jani Nikula 2016-01-11 19:54 ` [PATCH v2 6/6] drm/i915: hide away VBT private data in a separate header Jani Nikula 2016-01-14 12:53 ` Mika Kahola 2016-01-11 20:07 ` [PATCH v2 0/6] drm/i915: start hiding away vbt structure from the driver Lukas Wunner 2016-01-12 5:51 ` Jani Nikula 2016-01-12 8:20 ` ✓ success: Fi.CI.BAT Patchwork -- strict thread matches above, loose matches on Subject: below -- 2016-02-09 15:29 [PATCH v2 1/6] drm/i915: move VBT based TV presence check to intel_bios.c Jani Nikula 2016-02-09 15:29 ` [PATCH v2 2/6] drm/i915: move VBT based LVDS " Jani Nikula
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).