From fcc6eb1d59dc87dc603b8b3cdef1aae96064f9d9 Mon Sep 17 00:00:00 2001 From: Kurtis Dinelle Date: Wed, 11 Feb 2026 09:48:25 -0800 Subject: [PATCH] battery-service: Fix ACPI response --- battery-service/src/lib.rs | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/battery-service/src/lib.rs b/battery-service/src/lib.rs index d308fd10..7b5d21f6 100644 --- a/battery-service/src/lib.rs +++ b/battery-service/src/lib.rs @@ -66,18 +66,18 @@ impl Service { } Event::AcpiRequest(acpi_msg) => { trace!("Battery service: ACPI cmd recvd"); - match self.context.process_acpi_cmd(&acpi_msg).await { - Ok(response) => { - // TODO We should probably be responding to the requestor rather than just assuming the request came from the host - self.comms_send( - crate::EndpointID::External(embedded_services::comms::External::Host), - &response, - ) - .await - .expect("comms_send is infallible") - } - Err(e) => error!("Battery service command failed: {:?}", e), + let response = self.context.process_acpi_cmd(&acpi_msg).await; + if let Err(e) = response { + error!("Battery service command failed: {:?}", e) } + + // TODO We should probably be responding to the requestor rather than just assuming the request came from the host + self.comms_send( + crate::EndpointID::External(embedded_services::comms::External::Host), + &response, + ) + .await + .expect("comms_send is infallible") } } }