uefi: protocols: cleanup register_protocol_notify[_by_guid]()

Author: phip1611

uefi-test-runner/src/boot/misc.rs

@@ -145,12 +145,11 @@ fn test_register_protocol_notify() {
         info!("in callback for test_register_protocol_notify")
     }
 
-    let protocol = &TestProtocol::GUID;
     let event = unsafe {
         boot::create_event(EventType::NOTIFY_SIGNAL, Tpl::NOTIFY, Some(callback), None).unwrap()
     };
 
-    boot::register_protocol_notify(protocol, &event)
+    boot::register_protocol_notify::<TestProtocol>(&event)
         .expect("Failed to register protocol notify fn");
 }
 

uefi/CHANGELOG.md

@@ -8,6 +8,7 @@
 - Added `proto::nvme::pass_thru::NvmePassThru::broadcast()`.
 - Added `proto::media::block::BlockIO2`.
 - Added `boot::test_protocol_by_guid()`
+- Added `boot::register_protocol_notify_by_guid()`
 
 ## Changed
 - Changed ordering of `proto::pci::PciIoAddress` to (bus -> dev -> fun -> reg -> ext_reg).
@@ -16,6 +17,8 @@
   returns `Option<Event>` instead of `&Event`.
 - `Http::get_mode_data` doesn't consume a parameter anymore and instead return
   an owned value of type `HttpConfigData`
+- **Breaking**: `boot::register_protocol_notify()` now follows our generic-based
+  API and now longer consumes a `&Guid` parameter.
 
 # uefi - v0.36.1 (2025-11-05)
 

uefi/src/boot.rs

@@ -819,7 +819,15 @@ pub unsafe fn uninstall_protocol_interface(
 /// # Errors
 ///
 /// * [`Status::OUT_OF_RESOURCES`]: the event could not be allocated.
-pub fn register_protocol_notify(
+pub fn register_protocol_notify<P: ProtocolPointer + ?Sized>(
+    event: &Event,
+) -> Result<SearchType<'static>> {
+    register_protocol_notify_by_guid(&P::GUID, event)
+}
+
+/// Variant of [`register_protocol_notify`] that consumes the [`Guid`] as
+/// parameter.
+pub fn register_protocol_notify_by_guid(
     protocol: &'static Guid,
     event: &Event,
 ) -> Result<SearchType<'static>> {