kernel: bcma: fallback-sprom: optimize struct data
authorÁlvaro Fernández Rojas <noltari@gmail.com>
Mon, 24 Apr 2023 18:06:49 +0000 (20:06 +0200)
committerÁlvaro Fernández Rojas <noltari@gmail.com>
Mon, 24 Apr 2023 18:10:28 +0000 (20:10 +0200)
- Remove unneeded mac bytes from struct (it's already present in the SPROM).
- Convert devid_override to boolean.

Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
target/linux/generic/files/drivers/bcma/fallback-sprom.c

index 6da41cd3b473ad1f3a16c802e2ced65a2bd038ad..12a0b6405362957a5087d36e8a934025f91202a9 100644 (file)
@@ -48,8 +48,7 @@ struct bcma_fbs {
        struct ssb_sprom sprom;
        u32 pci_bus;
        u32 pci_dev;
-       u8 mac[ETH_ALEN];
-       int devid_override;
+       bool devid_override;
 };
 
 static DEFINE_SPINLOCK(bcma_fbs_lock);
@@ -624,8 +623,8 @@ static void bcma_fbs_fixup(struct bcma_fbs *priv, u16 *sprom)
        }
 }
 
-static int sprom_override_devid(struct bcma_fbs *priv, struct ssb_sprom *out,
-                               const u16 *in)
+static bool sprom_override_devid(struct bcma_fbs *priv, struct ssb_sprom *out,
+                                const u16 *in)
 {
        SPEX(dev_id, 0x0060, 0xFFFF, 0);
        return !!out->dev_id;
@@ -668,7 +667,7 @@ static void bcma_fbs_set(struct bcma_fbs *priv, struct device_node *node)
                sprom->itssi_bg = 0x00;
                sprom->boardflags_lo = 0x2848;
                sprom->boardflags_hi = 0x0000;
-               priv->devid_override = 0;
+               priv->devid_override = false;
 
                dev_warn(priv->dev, "using basic SPROM\n");
        } else {
@@ -694,6 +693,7 @@ static int bcma_fbs_probe(struct platform_device *pdev)
        struct device_node *node = dev->of_node;
        struct bcma_fbs *priv;
        unsigned long flags;
+       u8 mac[ETH_ALEN];
 
        priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
        if (!priv)
@@ -706,18 +706,18 @@ static int bcma_fbs_probe(struct platform_device *pdev)
        of_property_read_u32(node, "pci-bus", &priv->pci_bus);
        of_property_read_u32(node, "pci-dev", &priv->pci_dev);
 
-       of_get_mac_address(node, priv->mac);
-       if (is_valid_ether_addr(priv->mac)) {
-               dev_info(dev, "mtd mac %pM\n", priv->mac);
+       of_get_mac_address(node, mac);
+       if (is_valid_ether_addr(mac)) {
+               dev_info(dev, "mtd mac %pM\n", mac);
        } else {
-               random_ether_addr(priv->mac);
-               dev_info(dev, "random mac %pM\n", priv->mac);
+               random_ether_addr(mac);
+               dev_info(dev, "random mac %pM\n", mac);
        }
 
-       memcpy(priv->sprom.il0mac, priv->mac, ETH_ALEN);
-       memcpy(priv->sprom.et0mac, priv->mac, ETH_ALEN);
-       memcpy(priv->sprom.et1mac, priv->mac, ETH_ALEN);
-       memcpy(priv->sprom.et2mac, priv->mac, ETH_ALEN);
+       memcpy(priv->sprom.il0mac, mac, ETH_ALEN);
+       memcpy(priv->sprom.et0mac, mac, ETH_ALEN);
+       memcpy(priv->sprom.et1mac, mac, ETH_ALEN);
+       memcpy(priv->sprom.et2mac, mac, ETH_ALEN);
 
        spin_lock_irqsave(&bcma_fbs_lock, flags);
        list_add(&priv->list, &bcma_fbs_list);