clippy: fix clippy warning

-------- Signed-off-by: guoweikang <[email protected]>
Arceos-monolithic · Mar 27, 2024 · 978035e · 978035e
1 parent 5fb1dc1
commit 978035e
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 9 deletions.
diff --git a/crates/arm_gic/src/gic_v3.rs b/crates/arm_gic/src/gic_v3.rs
@@ -117,7 +117,7 @@ impl GicDistributor {
     fn validate_dist_version(&self) {
         let pidr2 = self.regs().PIDR2.get() & Self::GIC_PIDR2_ARCH_MASK; 
         match pidr2 {
-            Self::GIC_PIDR2_ARCH_GICV3 | Self::GIC_PIDR2_ARCH_GICV4 => return,
+            Self::GIC_PIDR2_ARCH_GICV3 | Self::GIC_PIDR2_ARCH_GICV4 => (),
             _ => panic!("unvalid gic pidr2")
         }
     }
@@ -434,7 +434,7 @@ impl GenericArmGic for GicV3 {
 
     /// Enables the interrupt with the given ID.
     fn enable_interrupt(&mut self, intid: IntId) {
-        let index = (intid.0 / 32) as usize;
+        let index = intid.0 / 32;
         let bit = 1 << (intid.0 % 32);
 
         if intid.is_private() {
@@ -445,7 +445,7 @@ impl GenericArmGic for GicV3 {
     }
 
     fn disable_interrupt(&mut self, intid: IntId) {
-        let index = (intid.0 / 32) as usize;
+        let index = intid.0 / 32;
         let bit = 1 << (intid.0 % 32);
 
         if intid.is_private() {

diff --git a/crates/arm_gic/src/sysregs.rs b/crates/arm_gic/src/sysregs.rs
@@ -3,7 +3,7 @@ macro_rules! read_sysreg {
         {
             let mut value: u64;
             ::core::arch::asm!(
-                concat!("mrs {value:x}, ", ::core::stringify!($name)),
+                concat!("mrs {value}, ", ::core::stringify!($name)),
                 value = out(reg) value,
                 options(nomem, nostack),
             );
@@ -19,7 +19,7 @@ macro_rules! write_sysreg {
         {
             let v: u64 = $value;
             ::core::arch::asm!(
-                concat!("msr ", ::core::stringify!($name), ", {value:x}"),
+                concat!("msr ", ::core::stringify!($name), ", {value}"),
                 value = in(reg) v,
                 options(nomem, nostack),
             )

diff --git a/modules/axhal/src/platform/aarch64_common/gic.rs b/modules/axhal/src/platform/aarch64_common/gic.rs
@@ -60,12 +60,9 @@ pub fn register_handler(irq_num: usize, handler: IrqHandler) -> bool {
 pub fn dispatch_irq(_unused: usize) {
     // actually no need to lock
     let intid = unsafe {GIC.get_mut().get_and_acknowledge_interrupt()};
-    match intid {
-        Some(id) => {
+    if let Some(id) = intid {
             crate::irq::dispatch_irq_common(id.into());
             unsafe {GIC.get_mut().end_interrupt(id);}
-        }
-        None => (),
     }
 }