]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
net: ieee802154: Use the IEEE802154_MAX_PAGE define when relevant
authorMiquel Raynal <miquel.raynal@bootlin.com>
Tue, 25 Jan 2022 12:25:39 +0000 (13:25 +0100)
committerStefan Schmidt <stefan@datenfreihafen.org>
Fri, 28 Jan 2022 10:23:30 +0000 (11:23 +0100)
This define already exist but is hardcoded in nl-phy.c. Use the
definition when relevant.

While at it, also convert the type from uint32_t to u32.

Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Acked-by: Alexander Aring <aahringo@redhat.com>
Link: https://lore.kernel.org/r/20220125122540.855604-3-miquel.raynal@bootlin.com
Signed-off-by: Stefan Schmidt <stefan@datenfreihafen.org>
net/ieee802154/nl-phy.c

index dd5a45f8a78a07557e229ede483a7d1048e79b22..359249ab77bf3d0f14c666dde77376de580d3e05 100644 (file)
@@ -30,7 +30,7 @@ static int ieee802154_nl_fill_phy(struct sk_buff *msg, u32 portid,
 {
        void *hdr;
        int i, pages = 0;
-       uint32_t *buf = kcalloc(32, sizeof(uint32_t), GFP_KERNEL);
+       u32 *buf = kcalloc(IEEE802154_MAX_PAGE + 1, sizeof(u32), GFP_KERNEL);
 
        pr_debug("%s\n", __func__);
 
@@ -47,7 +47,7 @@ static int ieee802154_nl_fill_phy(struct sk_buff *msg, u32 portid,
            nla_put_u8(msg, IEEE802154_ATTR_PAGE, phy->current_page) ||
            nla_put_u8(msg, IEEE802154_ATTR_CHANNEL, phy->current_channel))
                goto nla_put_failure;
-       for (i = 0; i < 32; i++) {
+       for (i = 0; i <= IEEE802154_MAX_PAGE; i++) {
                if (phy->supported.channels[i])
                        buf[pages++] = phy->supported.channels[i] | (i << 27);
        }