firewire: ohci: use guard macro to maintain bus time

The 1394 OHCI driver maintains bus time to respond to querying request.
The concurrent access to the bus time is protected by spinlock.

This commit uses guard macro to maintain the spinlock.

Link: https://lore.kernel.org/r/20240805085408.251763-16-o-takashi@sakamocchi.jp
Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>

diff --git a/drivers/firewire/ohci.c b/drivers/firewire/ohci.c
index 1461e008d265701e9ce74d5b3ddd32a4dbd67e64..5cb7c7603c2cada2020986fc7d0572034585a2fb 100644 (file)
--- a/drivers/firewire/ohci.c
+++ b/drivers/firewire/ohci.c
@@ -2300,9 +2300,8 @@ static irqreturn_t irq_handler(int irq, void *data)
                handle_dead_contexts(ohci);
 
        if (event & OHCI1394_cycle64Seconds) {
-               spin_lock(&ohci->lock);
+               guard(spinlock)(&ohci->lock);
                update_bus_time(ohci);
-               spin_unlock(&ohci->lock);
        } else
                flush_writes(ohci);
 
@@ -2762,7 +2761,6 @@ static int ohci_enable_phys_dma(struct fw_card *card,
 static u32 ohci_read_csr(struct fw_card *card, int csr_offset)
 {
        struct fw_ohci *ohci = fw_ohci(card);
-       unsigned long flags;
        u32 value;
 
        switch (csr_offset) {
@@ -2786,16 +2784,14 @@ static u32 ohci_read_csr(struct fw_card *card, int csr_offset)
                return get_cycle_time(ohci);
 
        case CSR_BUS_TIME:
-               /*
-                * We might be called just after the cycle timer has wrapped
-                * around but just before the cycle64Seconds handler, so we
-                * better check here, too, if the bus time needs to be updated.
-                */
-               spin_lock_irqsave(&ohci->lock, flags);
-               value = update_bus_time(ohci);
-               spin_unlock_irqrestore(&ohci->lock, flags);
-               return value;
+       {
+               // We might be called just after the cycle timer has wrapped around but just before
+               // the cycle64Seconds handler, so we better check here, too, if the bus time needs
+               // to be updated.
 
+               guard(spinlock_irqsave)(&ohci->lock);
+               return update_bus_time(ohci);
+       }
        case CSR_BUSY_TIMEOUT:
                value = reg_read(ohci, OHCI1394_ATRetries);
                return (value >> 4) & 0x0ffff00f;
@@ -2813,7 +2809,6 @@ static u32 ohci_read_csr(struct fw_card *card, int csr_offset)
 static void ohci_write_csr(struct fw_card *card, int csr_offset, u32 value)
 {
        struct fw_ohci *ohci = fw_ohci(card);
-       unsigned long flags;
 
        switch (csr_offset) {
        case CSR_STATE_CLEAR:
@@ -2849,12 +2844,11 @@ static void ohci_write_csr(struct fw_card *card, int csr_offset, u32 value)
                break;
 
        case CSR_BUS_TIME:
-               spin_lock_irqsave(&ohci->lock, flags);
-               ohci->bus_time = (update_bus_time(ohci) & 0x40) |
-                                (value & ~0x7f);
-               spin_unlock_irqrestore(&ohci->lock, flags);
+       {
+               guard(spinlock_irqsave)(&ohci->lock);
+               ohci->bus_time = (update_bus_time(ohci) & 0x40) | (value & ~0x7f);
                break;
-
+       }
        case CSR_BUSY_TIMEOUT:
                value = (value & 0xf) | ((value & 0xf) << 4) |
                        ((value & 0xf) << 8) | ((value & 0x0ffff000) << 4);