]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
firewire: core: minor code refactoring to delete useless local variable
authorTakashi Sakamoto <o-takashi@sakamocchi.jp>
Thu, 18 Sep 2025 23:54:48 +0000 (08:54 +0900)
committerTakashi Sakamoto <o-takashi@sakamocchi.jp>
Thu, 18 Sep 2025 23:58:34 +0000 (08:58 +0900)
The do_reset local variable has less merit. Let's remove it.

Link: https://lore.kernel.org/r/20250918235448.129705-7-o-takashi@sakamocchi.jp
Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
drivers/firewire/core-card.c

index 58d1f58a4a0f1a61efa9d881ef7dbd9a05e7aef4..4a5459696093a1e75fbbe230e943ddad1d42bedc 100644 (file)
@@ -387,7 +387,6 @@ static void bm_work(struct work_struct *work)
        struct fw_node *root_node __free(node_unref) = NULL;
        int root_id, new_root_id, irm_id, local_id;
        int expected_gap_count, generation;
-       bool do_reset = false;
        bool stand_for_root = false;
 
        if (card->local_node == NULL)
@@ -499,16 +498,10 @@ static void bm_work(struct work_struct *work)
        else
                expected_gap_count = 63;
 
-       /*
-        * Finally, figure out if we should do a reset or not.  If we have
-        * done less than 5 resets with the same physical topology and we
-        * have either a new root or a new gap count setting, let's do it.
-        */
-
-       if (card->bm_retries++ < 5 && (card->gap_count != expected_gap_count || new_root_id != root_id))
-               do_reset = true;
-
-       if (do_reset) {
+       // Finally, figure out if we should do a reset or not. If we have done less than 5 resets
+       // with the same physical topology and we have either a new root or a new gap count
+       // setting, let's do it.
+       if (card->bm_retries++ < 5 && (card->gap_count != expected_gap_count || new_root_id != root_id)) {
                int card_gap_count = card->gap_count;
 
                fw_notice(card, "phy config: new root=%x, gap_count=%d\n",