Update redox_uefi_std to 0.1.13

This version would allow compiling using a stable toolchain, if the use of `unaligned_references` could be removed from this project. Signed-off-by: Tim Crawford <[email protected]>
system76 · Nov 11, 2024 · 6b14cb5 · 6b14cb5
1 parent 769cb38
commit 6b14cb5
Show file tree

Hide file tree

Showing 14 changed files with 75 additions and 100 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -16,7 +16,7 @@ orbfont = { version = "0.1.12", default-features = false, features = ["no-std"]
 plain = "0.2.3"
 redox_dmi = "0.1.6"
 redox_hwio = { version = "0.1.6", default-features = false }
-redox_uefi_std = "0.1.11"
+redox_uefi_std = "0.1.13"
 spin = "0.9.4"
 
 [dependencies.system76_ectool]

diff --git a/src/display.rs b/src/display.rs
@@ -1,11 +1,11 @@
 // SPDX-License-Identifier: GPL-3.0-only
 
 use core::cell::Cell;
-use core::ops::Try;
 use orbclient::{Color, Mode, Renderer};
+use std::prelude::*;
 use std::proto::Protocol;
 use std::uefi::graphics::{GraphicsOutput, GraphicsBltOp, GraphicsBltPixel};
-use std::uefi::guid::{Guid, GRAPHICS_OUTPUT_PROTOCOL_GUID};
+use std::uefi::guid::GRAPHICS_OUTPUT_PROTOCOL_GUID;
 
 pub struct Output(pub &'static mut GraphicsOutput);
 
@@ -53,7 +53,7 @@ impl Display {
             h as usize,
             0
         );
-        status.branch().is_continue()
+        status.is_success()
     }
 }
 

diff --git a/src/dump_hii.rs b/src/dump_hii.rs
@@ -4,11 +4,11 @@ use hwio::{Io, Pio};
 use std::{char, mem, ptr, str};
 use std::ops::Try;
 use std::proto::Protocol;
+use std::prelude::*;
 use std::uefi::hii::database::HiiHandle;
 use std::uefi::hii::ifr::{IfrOpCode, IfrOpHeader, IfrForm, IfrAction};
 use std::uefi::hii::package::{HiiPackageHeader, HiiPackageKind, HiiPackageListHeader, HiiStringPackageHeader};
 use std::uefi::hii::sibt::{SibtHeader, SibtKind, SibtEnd, SibtSkip2, SibtStringUcs2, SibtStringsUcs2};
-use std::uefi::status::{Error, Result};
 
 use crate::hii;
 

diff --git a/src/fde.rs b/src/fde.rs
@@ -1,30 +1,26 @@
 // SPDX-License-Identifier: GPL-3.0-only
 
-use core::prelude::v1::derive;
-use core::ops::FromResidual;
 use orbclient::{Color, Renderer};
 use orbfont::Text;
-use std::{char, cmp, ffi, mem, ptr, slice};
-use std::ops::Try;
+use core::{char, cmp, mem, ptr, slice};
+use std::prelude::*;
 use std::proto::Protocol;
-use std::uefi::Event;
-use std::uefi::guid::Guid;
+use std::ffi;
 use std::uefi::hii::{AnimationId, ImageId, StringId};
 use std::uefi::hii::database::HiiHandle;
 use std::uefi::hii::ifr::{
     HiiValue,
     IfrOpCode, IfrOpHeader, IfrStatementHeader, IfrTypeValueEnum,
     IfrAction, IfrCheckbox, IfrNumeric, IfrOneOf, IfrOneOfOption, IfrOrderedList, IfrRef, IfrSubtitle
 };
-use std::uefi::status::{Error, Result, Status};
 use std::uefi::text::TextInputKey;
 
 use crate::display::{Display, Output};
 use crate::key::{raw_key, Key};
 use crate::ui::Ui;
 
 // TODO: Move to uefi library {
-pub const HII_STRING_PROTOCOL_GUID: Guid = Guid(0xfd96974, 0x23aa, 0x4cdc, [0xb9, 0xcb, 0x98, 0xd1, 0x77, 0x50, 0x32, 0x2a]);
+pub const HII_STRING_PROTOCOL_GUID: Guid = guid!("0fd96974-23aa-4cdc-b9cb-98d17750322a");
 
 #[repr(C)]
 pub struct HiiStringProtocol {
@@ -47,15 +43,15 @@ impl HiiStringProtocol {
     pub fn string(&self, PackageList: HiiHandle, StringId: StringId) -> Result<String> {
         let mut data = vec![0u16; 4096];
         let mut len = data.len();
-        (self.GetString)(
+        Result::from((self.GetString)(
             self,
             b"en-US\0".as_ptr(),
             PackageList,
             StringId,
             data.as_mut_ptr(),
             &mut len,
             0
-        )?;
+        ))?;
         data.truncate(len);
 
         let mut string = String::new();
@@ -354,7 +350,7 @@ fn wait_for_events(form: &Form) -> Result<EventType>  {
         events.push(form.FormRefreshEvent);
     }
 
-    (uefi.BootServices.WaitForEvent)(events.len(), events.as_mut_ptr(), &mut index)?;
+    Result::from((uefi.BootServices.WaitForEvent)(events.len(), events.as_mut_ptr(), &mut index))?;
 
     if index == 0 {
         Ok(EventType::Keyboard)
@@ -776,7 +772,7 @@ fn form_display_inner(form: &Form, user_input: &mut UserInput) -> Result<()> {
                 let raw_key = match raw_key(false) {
                     Ok(ok) => ok,
                     Err(err) => match err {
-                        Error::NotReady => break 'input,
+                        Status::NOT_READY => break 'input,
                         _ => return Err(err),
                     }
                 };
@@ -1030,10 +1026,7 @@ fn form_display_inner(form: &Form, user_input: &mut UserInput) -> Result<()> {
 }
 
 extern "efiapi" fn form_display(form: &Form, user_input: &mut UserInput) -> Status {
-    match form_display_inner(form, user_input) {
-        Ok(()) => Status::from_output(0),
-        Err(err) => Status::from_residual(err),
-    }
+    form_display_inner(form, user_input).into()
 }
 
 extern "efiapi" fn exit_display() {
@@ -1047,13 +1040,13 @@ extern "efiapi" fn confirm_data_change() -> usize {
 
 impl Fde {
     pub fn install() -> Result<()> {
-        let guid = Guid(0x9bbe29e9, 0xfda1, 0x41ec, [0xad, 0x52, 0x45, 0x22, 0x13, 0x74, 0x2d, 0x2e]);
+        let guid = guid!("9bbe29e9-fda1-41ec-ad52-452213742d2e");
 
         let uefi = unsafe { std::system_table_mut() };
 
         let current = unsafe {
             let mut interface = 0;
-            (uefi.BootServices.LocateProtocol)(&guid, 0, &mut interface)?;
+            Result::from((uefi.BootServices.LocateProtocol)(&guid, 0, &mut interface))?;
             &mut *(interface as *mut Fde)
         };
 

diff --git a/src/hii.rs b/src/hii.rs
@@ -1,8 +1,9 @@
 // SPDX-License-Identifier: GPL-3.0-only
 
 use std::proto::Protocol;
+use std::prelude::*;
 use std::uefi::hii::database::HiiDatabase;
-use std::uefi::guid::{Guid, HII_DATABASE_GUID};
+use std::uefi::guid::HII_DATABASE_GUID;
 
 pub struct Database(pub &'static mut HiiDatabase);
 

diff --git a/src/image/bmp.rs b/src/image/bmp.rs
@@ -2,7 +2,9 @@
 
 use super::Image;
 
-pub fn parse(file_data: &[u8]) -> Result<Image, String> {
+use std::prelude::*;
+
+pub fn parse(file_data: &[u8]) -> core::result::Result<Image, String> {
     use orbclient::Color;
 
     let get = |i: usize| -> u8 {

diff --git a/src/image/mod.rs b/src/image/mod.rs
@@ -2,7 +2,9 @@
 
 use core::cell::Cell;
 use core::cmp;
-use core::prelude::v1::derive;
+use core::default::Default;
+
+use std::prelude::*;
 
 use orbclient::{Color, Mode, Renderer};
 
@@ -51,7 +53,7 @@ impl Image {
     }
 
     /// Create a new image from a boxed slice of colors
-    pub fn from_data(width: u32, height: u32, data: Box<[Color]>) -> Result<Self, String> {
+    pub fn from_data(width: u32, height: u32, data: Box<[Color]>) -> core::result::Result<Self, String> {
         if (width * height) as usize != data.len() {
             return Err("not enough or too much data given compared to width and height".to_string())
         }

diff --git a/src/key.rs b/src/key.rs
@@ -1,8 +1,7 @@
 // SPDX-License-Identifier: GPL-3.0-only
 
 use core::char;
-use core::prelude::v1::derive;
-use std::uefi::status::Result;
+use std::prelude::*;
 use std::uefi::text::TextInputKey;
 
 #[derive(Debug)]
@@ -79,15 +78,15 @@ pub fn raw_key(wait: bool) -> Result<TextInputKey> {
 
     if wait {
         let mut index = 0;
-        (uefi.BootServices.WaitForEvent)(1, &uefi.ConsoleIn.WaitForKey, &mut index)?;
+        Result::from((uefi.BootServices.WaitForEvent)(1, &uefi.ConsoleIn.WaitForKey, &mut index))?;
     }
 
     let mut key = TextInputKey {
         ScanCode: 0,
         UnicodeChar: 0
     };
 
-    (uefi.ConsoleIn.ReadKeyStroke)(uefi.ConsoleIn, &mut key)?;
+    Result::from((uefi.ConsoleIn.ReadKeyStroke)(uefi.ConsoleIn, &mut key))?;
 
     Ok(key)
 }

diff --git a/src/main.rs b/src/main.rs
@@ -2,25 +2,16 @@
 
 #![no_std]
 #![no_main]
-#![feature(core_intrinsics)]
-#![feature(prelude_import)]
-#![feature(try_trait_v2)]
-#![feature(control_flow_enum)]
 #![allow(non_snake_case)]
 
-#[macro_use]
-extern crate bitflags;
 #[macro_use]
 extern crate memoffset;
 #[macro_use]
 extern crate uefi_std as std;
 
-#[allow(unused_imports)]
-#[prelude_import]
 use std::prelude::*;
 
 use core::ptr;
-use std::uefi::status::Status;
 
 #[macro_use]
 mod debug;

diff --git a/src/rng.rs b/src/rng.rs
@@ -1,24 +1,19 @@
-use std::{
-    proto::Protocol,
-    ptr,
-};
-use std::uefi::{
-    guid::Guid,
-    status::{Result, Status},
-};
+use core::ptr;
+use std::proto::Protocol;
+use std::prelude::*;
 
 pub const RNG_PROTOCOL_GUID: Guid = Guid(0x3152bca5, 0xeade, 0x433d, [0x86, 0x2e, 0xc0, 0x1c, 0xdc, 0x29, 0x1f, 0x44]);
 
 pub struct Rng(pub &'static mut RngProtocol);
 
 impl Rng {
     pub fn read(&self, buf: &mut [u8]) -> Result<()> {
-        (self.0.GetRNG)(
+        Result::from((self.0.GetRNG)(
             self.0,
             ptr::null(),
             buf.len(),
             buf.as_mut_ptr(),
-        )?;
+        ))?;
         Ok(())
     }
 }