ApolloLakeFspBinPkg[MR6] FSP-M PrimaryVideoAdaptor UPD possible values are wrong

Abierto
#57 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
48/100
Tipo de issue
Documentación
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
c
Área
embedded-iot

Línea de trabajo

Localice el encabezado FSP-M UPD de ApolloLakeFspBinPkg[MR6] que contiene PrimaryVideoAdaptor y lea su descripción actual. Actualice los valores documentados para que coincidan con el comportamiento informado, con 0x0 para IGD y 0x1 para PCI, y luego verifique que la definición UPD circundante siga siendo coherente.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Hi,

Looks like the documented possible values for the FSP-M PrimaryVideoAdaptor UPD are not correct. The UPD headers state the following:
/** Offset 0x004C - Primary Display Select which of IGD/PCI Graphics device should be Primary Display. 0x0:AUTO(Default), 0x2:IGD, 0x3:PCI 0x0:AUTO, 0x2:IGD, 0x3:PCI **/ UINT8 PrimaryVideoAdaptor;

However, testing showed that any non-zero value will result in FSP setting the IVD (IGD VGA Disable) bit in the GGC register (PCI B:D.F 0:0.0 offset 0x50), regardless of whether another video card is present in the system. When PrimaryVideoAdaptor is zero, the IVD bit is cleared as well. Of course, this only happens if the IGD is enabled.

The simplest solution (and what I would prefer) is to simply update the UPD's description to match FSP's behavior, that is: 0x0:IGD, 0x1:PCI.

Thanks in advance,
Angel

Lenguaje dominante
C
Estrellas
366
Forks
143
Métricas de merge de PR
Sin PR fusionados en 30 d

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de intel/FSP

Todos los issues de intel/FSP

Issues similares

Más issues de C

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.