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

Đang mở
#57 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
48/100
Loại issue
Tài liệu
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Đình trệ
Công nghệ
c
Lĩnh vực
embedded-iot

Hướng nghiên cứu

Xác định header FSP-M UPD trong ApolloLakeFspBinPkg[MR6] chứa PrimaryVideoAdaptor và đọc mô tả hiện tại của nó. Cập nhật các giá trị được ghi trong tài liệu để khớp với hành vi đã báo cáo, với 0x0 cho IGD và 0x1 cho PCI, sau đó xác minh rằng định nghĩa UPD xung quanh vẫn nhất quán.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

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

Ngôn ngữ chính
C
Star
366
Fork
143
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của intel/FSP

Tất cả issue của intel/FSP

Issue tương tự

Thêm issue về C

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.