subghz: replace Debug impl with Display for some structs

This improves the formatting for nested structures.

Author: newAM

CHANGELOG.md

@@ -10,6 +10,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Added alarm functionality to the RTC.
 - Added `Rtc.is_wakeup_timer_en`.
 
+### Changed
+- Replaced `Debug` implementation with `Display` implementation for:
+  - `subghz::FskPacketStatus`
+  - `subghz::LoRaPacketStatus`
+  - `subghz::Stats`
+  - `subghz::Status`
+
 ## [0.3.0] - 2021-12-20
 ### Added
 - Added `info::Core::CT` to get the CPU core at compile time.

hal/src/subghz/packet_status.rs

@@ -7,7 +7,7 @@ use super::Status;
 /// Returned by [`fsk_packet_status`].
 ///
 /// [`fsk_packet_status`]: super::SubGhz::fsk_packet_status
-#[derive(Debug, Clone, Copy, PartialEq, Eq)]
+#[derive(Clone, Copy, PartialEq, Eq)]
 pub struct FskPacketStatus {
     buf: [u8; 4],
 }
@@ -149,7 +149,7 @@ impl defmt::Format for FskPacketStatus {
     }
 }
 
-impl core::fmt::Display for FskPacketStatus {
+impl core::fmt::Debug for FskPacketStatus {
     fn fmt(&self, f: &mut core::fmt::Formatter<'_>) -> core::fmt::Result {
         f.debug_struct("FskPacketStatus")
             .field("status", &self.status())
@@ -172,7 +172,7 @@ impl core::fmt::Display for FskPacketStatus {
 /// Returned by [`lora_packet_status`].
 ///
 /// [`lora_packet_status`]: super::SubGhz::lora_packet_status
-#[derive(Debug, Clone, Copy, PartialEq, Eq)]
+#[derive(Clone, Copy, PartialEq, Eq)]
 pub struct LoRaPacketStatus {
     buf: [u8; 4],
 }
@@ -272,7 +272,7 @@ impl defmt::Format for LoRaPacketStatus {
     }
 }
 
-impl core::fmt::Display for LoRaPacketStatus {
+impl core::fmt::Debug for LoRaPacketStatus {
     fn fmt(&self, f: &mut core::fmt::Formatter<'_>) -> core::fmt::Result {
         f.debug_struct("LoRaPacketStatus")
             .field("status", &self.status())

hal/src/subghz/stats.rs

@@ -9,7 +9,7 @@ typestate!(FskStats, "FSK stats");
 ///
 /// [`fsk_stats`]: super::SubGhz::fsk_stats
 /// [`lora_stats`]: super::SubGhz::lora_stats
-#[derive(Debug, Eq, PartialEq, Clone, Copy)]
+#[derive(Eq, PartialEq, Clone, Copy)]
 #[cfg_attr(feature = "defmt", derive(defmt::Format))]
 pub struct Stats<ModType> {
     status: Status,
@@ -139,7 +139,7 @@ impl Stats<LoRaStats> {
     }
 }
 
-impl core::fmt::Display for Stats<FskStats> {
+impl core::fmt::Debug for Stats<FskStats> {
     fn fmt(&self, f: &mut core::fmt::Formatter<'_>) -> core::fmt::Result {
         f.debug_struct("Stats")
             .field("status", &self.status())

hal/src/subghz/status.rs

@@ -116,7 +116,7 @@ impl CmdStatus {
 /// This is returned by [`status`].
 ///
 /// [`status`]: super::SubGhz::status
-#[derive(Debug, PartialEq, Eq, Clone, Copy)]
+#[derive(PartialEq, Eq, Clone, Copy)]
 pub struct Status(u8);
 
 impl From<u8> for Status {
@@ -180,7 +180,7 @@ impl Status {
     }
 }
 
-impl core::fmt::Display for Status {
+impl core::fmt::Debug for Status {
     fn fmt(&self, f: &mut core::fmt::Formatter<'_>) -> core::fmt::Result {
         f.debug_struct("Status")
             .field("mode", &self.mode())