Updated with MemoryRegion.

Author: jonathanpallant

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "neotron-common-bios"
-version = "0.1.0"
+version = "0.4.0-alpha"
 authors = ["Jonathan 'theJPster' Pallant <[email protected]>"]
 edition = "2018"
 description = "Common BIOS code and API for all Neotron systems."

README.md

@@ -36,3 +36,13 @@ This BIOS API crate is a work in progress.
 Unless you explicitly state otherwise, any contribution intentionally submitted
 for inclusion in the work by you, as defined in the Apache-2.0 license, shall be
 dual licensed as above, without any additional terms or conditions.
+
+## Changelog
+
+### Unreleased Changes
+
+* Changed `memory_get_region` to return a `MemoryRegion`
+
+### v0.3.0
+
+* First published version. `Cargo.toml` reports at `0.1.0`.

src/lib.rs

@@ -163,20 +163,12 @@ pub struct Api {
 	/// Other regions may be located at other addresses (e.g. external DRAM or
 	/// PSRAM).
 	///
-	/// The OS will always load non-relocatable applications into the bottom
-	/// of Region 0. It can allocate OS specific structures from any other
-	/// Region (if any), or from the top of Region 0 (although this reduces
-	/// the maximum application space available). The OS will prefer lower
-	/// numbered regions (other than Region 0), so faster memory should be
-	/// listed first.
-	///
-	/// If the region number given is invalid, the function returns `(null,
-	/// 0)`.
-	pub memory_get_region: extern "C" fn(
-		region: u8,
-		out_start: *mut *mut u8,
-		out_len: *mut usize,
-	) -> crate::Result<()>,
+	/// The OS will always load non-relocatable applications into the bottom of
+	/// Region 0. It can allocate OS specific structures from any other Region
+	/// (if any), or from the top of Region 0 (although this reduces the maximum
+	/// application space available). The OS will prefer lower numbered regions
+	/// (other than Region 0), so faster memory should be listed first.
+	pub memory_get_region: extern "C" fn(region_index: u8) -> crate::Result<types::MemoryRegion>,
 }
 
 // End of file

src/types.rs

@@ -126,6 +126,28 @@ pub struct Time {
 	pub nsecs: u32,
 }
 
+/// The kinds of memory we know about
+#[repr(C)]
+#[derive(Debug, Clone)]
+pub enum MemoryKind {
+	/// Read-write memory
+	Ram,
+	/// Read-only memory
+	Rom,
+}
+
+/// Represents a region in memory.
+#[repr(C)]
+#[derive(Debug, Clone)]
+pub struct MemoryRegion {
+	/// The address the region starts at
+	pub start: *mut u8,
+	/// The length of the region
+	pub length: usize,
+	/// The kind of memory found at this region
+	pub kind: MemoryKind,
+}
+
 // ============================================================================
 // Impls
 // ============================================================================
@@ -302,6 +324,35 @@ impl From<&Time> for chrono::DateTime<chrono::Utc> {
 	}
 }
 
+// MemoryKind
+
+impl core::fmt::Display for MemoryKind {
+	fn fmt(&self, f: &mut core::fmt::Formatter) -> core::fmt::Result {
+		write!(
+			f,
+			"{}",
+			match self {
+				MemoryKind::Rom => "ROM",
+				MemoryKind::Ram => "RAM",
+			}
+		)
+	}
+}
+
+// MemoryRegion
+
+impl core::fmt::Display for MemoryRegion {
+	fn fmt(&self, f: &mut core::fmt::Formatter) -> core::fmt::Result {
+		write!(
+			f,
+			"{} KiB {} @ {:p}",
+			self.length / 1024,
+			self.kind,
+			self.start
+		)
+	}
+}
+
 // ============================================================================
 // End of File
 // ============================================================================