On Fri, 12 Feb 2016, Shubhangi Shrivastava <shubhangi.shrivast...@intel.com>
wrote:
> This patch adds new fields that are not yet added in drm code
> in child devices struct
>
> Signed-off-by: Sivakumar Thulasimani <sivakumar.thulasim...@intel.com>
> Signed-off-by: Durgadoss R <durgados...@intel.com>
> Signed-off-by: Shubhangi Shrivastava <shubhangi.shrivast...@intel.com>
> ---
> drivers/gpu/drm/i915/intel_bios.c | 15 ++++++++++++++-
> drivers/gpu/drm/i915/intel_bios.h | 20 +++++++++++++-------
> 2 files changed, 27 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_bios.c
> b/drivers/gpu/drm/i915/intel_bios.c
> index bf62a19..a26d4b4 100644
> --- a/drivers/gpu/drm/i915/intel_bios.c
> +++ b/drivers/gpu/drm/i915/intel_bios.c
> @@ -1124,7 +1124,7 @@ static void parse_ddi_port(struct drm_i915_private
> *dev_priv, enum port port,
> }
>
> /* Parse the I_boost config for SKL and above */
> - if (bdb->version >= 196 && (child->common.flags_1 & IBOOST_ENABLE)) {
> + if (bdb->version >= 196 && child->common.iboost) {
> info->dp_boost_level =
> translate_iboost(child->common.iboost_level & 0xF);
> DRM_DEBUG_KMS("VBT (e)DP boost level for port %c: %d\n",
> port_name(port), info->dp_boost_level);
> @@ -1250,6 +1250,19 @@ parse_device_mapping(struct drm_i915_private *dev_priv,
> */
> memcpy(child_dev_ptr, p_child,
> min_t(size_t, p_defs->child_dev_size, sizeof(*p_child)));
> +
> + /*
> + * copied full block, now init values when they are not
> + * available in current version
> + */
> + if (bdb->version < 196) {
> + /* Set default values for bits added from v196 */
> + child_dev_ptr->common.iboost = 0;
> + child_dev_ptr->common.hpd_invert = 0;
> + }
> +
> + if (bdb->version < 192)
> + child_dev_ptr->common.lspcon = 0;
> }
> return;
> }
> diff --git a/drivers/gpu/drm/i915/intel_bios.h
> b/drivers/gpu/drm/i915/intel_bios.h
> index 350d4e0..833b82b 100644
> --- a/drivers/gpu/drm/i915/intel_bios.h
> +++ b/drivers/gpu/drm/i915/intel_bios.h
> @@ -250,9 +250,6 @@ struct old_child_dev_config {
> * 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;
> @@ -261,10 +258,19 @@ struct common_child_dev_config {
> u8 not_common2[2];
> u8 ddc_pin;
> u16 edid_ptr;
> - u8 obsolete;
> - u8 flags_1;
> - u8 not_common3[13];
> - u8 iboost_level;
> + u8 dvo_cfg; /* See DEVICE_CFG_* above */
> + u8 efp_routed:1;
> + u8 lane_reversal:1;
> + u8 lspcon:1;
> + u8 iboost:1;
> + u8 hpd_invert:1;
> + u8 flag_reserved:3;
> + u8 hdmi_support:1;
> + u8 dp_support:1;
> + u8 tmds_support:1;
> + u8 support_reserved:5;
> + u8 not_common3[13];
> + u8 iboost_level;
Please use just one space between type and member name, not two.
BR,
Jani.
> } __packed;
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
