]> git.proxmox.com Git - mirror_ubuntu-jammy-kernel.git/commitdiff
bcma: remove set but not used variable 'sizel'
authoryu kuai <yukuai3@huawei.com>
Wed, 27 Nov 2019 12:44:33 +0000 (20:44 +0800)
committerKalle Valo <kvalo@codeaurora.org>
Wed, 18 Dec 2019 18:55:25 +0000 (20:55 +0200)
Fixes gcc '-Wunused-but-set-variable' warning:

drivers/bcma/scan.c: In function ‘bcma_erom_get_addr_desc’:

drivers/bcma/scan.c:222:20: warning: variable ‘sizel’ set but
not used [-Wunused-but-set-variable]

It is never used, and so can be removed.

Fixes: 8369ae33b705 ("bcma: add Broadcom specific AMBA bus driver")
Signed-off-by: yu kuai <yukuai3@huawei.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/bcma/scan.c

index 4a2d1b235fb5af3e51ac8b044e6a18aed5753cec..1f2de714b40174bea06494f05919fa6e930b98f1 100644 (file)
@@ -219,7 +219,7 @@ static s32 bcma_erom_get_mst_port(struct bcma_bus *bus, u32 __iomem **eromptr)
 static u32 bcma_erom_get_addr_desc(struct bcma_bus *bus, u32 __iomem **eromptr,
                                  u32 type, u8 port)
 {
-       u32 addrl, addrh, sizel, sizeh = 0;
+       u32 addrl, addrh, sizeh = 0;
        u32 size;
 
        u32 ent = bcma_erom_get_ent(bus, eromptr);
@@ -239,12 +239,9 @@ static u32 bcma_erom_get_addr_desc(struct bcma_bus *bus, u32 __iomem **eromptr,
 
        if ((ent & SCAN_ADDR_SZ) == SCAN_ADDR_SZ_SZD) {
                size = bcma_erom_get_ent(bus, eromptr);
-               sizel = size & SCAN_SIZE_SZ;
                if (size & SCAN_SIZE_SG32)
                        sizeh = bcma_erom_get_ent(bus, eromptr);
-       } else
-               sizel = SCAN_ADDR_SZ_BASE <<
-                               ((ent & SCAN_ADDR_SZ) >> SCAN_ADDR_SZ_SHIFT);
+       }
 
        return addrl;
 }