search

riscvarchive / riscv-platform-specs

RISC-V Profiles and Platform Specification
Creative Commons Attribution 4.0 International
112 stars 39 forks source link

Update UEFI and ACPI sections. #59

Closed vlsunil closed 3 years ago

vlsunil commented 3 years ago

1) Fix: interrrupt -> interrupt

2) Update the UEFI requirements to refer to the UEFI spec.

3) More clarity around ACPI requirement.

Signed-off-by: Sunil V L sunilvl@ventanamicro.com

kumarsankaran commented 3 years ago

I merged this patch. I will fix this to add “support ACPI”.

Regards

Kumar

From: Atish Patra @.> Sent: Thursday, September 30, 2021 11:06 PM To: riscv/riscv-platform-specs @.> Cc: Subscribed @.**> Subject:* Re: [riscv/riscv-platform-specs] Update UEFI and ACPI sections. (#59)

@.**** commented on this pull request.

Otherwise looks good.

In riscv-platform-spec.adoc https://github.com/riscv/riscv-platform-specs/pull/59#discussion_r719965142 :

|===

===== Hardware Discovery Mechanisms

====== ACPI

ACPI is the required mechanism for the hardware discovery and configuration.

Server platforms are required to adhere to the RISC-V ACPI Platform Requirements

-Specification<>.

+Specification<>. Platform firmware must implement ACPI and the

I & Kumar had a discussion about "implement". https://docs.google.com/document/d/1PfvLw3f-tPTRjl6Rw614O_RaA62oKeHlTfu7g6KZ4Vs/edit?disco=AAAAPk-SXW0

If you agree with the reasoning, lets change it to "Platform firmware must support ACPI"

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/riscv/riscv-platform-specs/pull/59#pullrequestreview-768589911, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABLRGVTY6QLB5T7PGBO56V3UEVFT3ANCNFSM5FD4VZPQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.