Skip to content

Commit dbb2f36

Browse files
committed
eth: bnxt: make ulp_id unsigned to make GCC 12 happy
GCC array bounds checking complains that ulp_id is validated only against upper bound. Make it unsigned. Reviewed-by: Michael Chan <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Jakub Kicinski <[email protected]>
1 parent 5feba47 commit dbb2f36

File tree

2 files changed

+12
-12
lines changed

2 files changed

+12
-12
lines changed

drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
#include "bnxt_hwrm.h"
2626
#include "bnxt_ulp.h"
2727

28-
static int bnxt_register_dev(struct bnxt_en_dev *edev, int ulp_id,
28+
static int bnxt_register_dev(struct bnxt_en_dev *edev, unsigned int ulp_id,
2929
struct bnxt_ulp_ops *ulp_ops, void *handle)
3030
{
3131
struct net_device *dev = edev->net;
@@ -62,7 +62,7 @@ static int bnxt_register_dev(struct bnxt_en_dev *edev, int ulp_id,
6262
return 0;
6363
}
6464

65-
static int bnxt_unregister_dev(struct bnxt_en_dev *edev, int ulp_id)
65+
static int bnxt_unregister_dev(struct bnxt_en_dev *edev, unsigned int ulp_id)
6666
{
6767
struct net_device *dev = edev->net;
6868
struct bnxt *bp = netdev_priv(dev);
@@ -115,7 +115,7 @@ static void bnxt_fill_msix_vecs(struct bnxt *bp, struct bnxt_msix_entry *ent)
115115
}
116116
}
117117

118-
static int bnxt_req_msix_vecs(struct bnxt_en_dev *edev, int ulp_id,
118+
static int bnxt_req_msix_vecs(struct bnxt_en_dev *edev, unsigned int ulp_id,
119119
struct bnxt_msix_entry *ent, int num_msix)
120120
{
121121
struct net_device *dev = edev->net;
@@ -179,7 +179,7 @@ static int bnxt_req_msix_vecs(struct bnxt_en_dev *edev, int ulp_id,
179179
return avail_msix;
180180
}
181181

182-
static int bnxt_free_msix_vecs(struct bnxt_en_dev *edev, int ulp_id)
182+
static int bnxt_free_msix_vecs(struct bnxt_en_dev *edev, unsigned int ulp_id)
183183
{
184184
struct net_device *dev = edev->net;
185185
struct bnxt *bp = netdev_priv(dev);
@@ -233,7 +233,7 @@ int bnxt_get_ulp_stat_ctxs(struct bnxt *bp)
233233
return 0;
234234
}
235235

236-
static int bnxt_send_msg(struct bnxt_en_dev *edev, int ulp_id,
236+
static int bnxt_send_msg(struct bnxt_en_dev *edev, unsigned int ulp_id,
237237
struct bnxt_fw_msg *fw_msg)
238238
{
239239
struct net_device *dev = edev->net;
@@ -447,7 +447,7 @@ void bnxt_ulp_async_events(struct bnxt *bp, struct hwrm_async_event_cmpl *cmpl)
447447
rcu_read_unlock();
448448
}
449449

450-
static int bnxt_register_async_events(struct bnxt_en_dev *edev, int ulp_id,
450+
static int bnxt_register_async_events(struct bnxt_en_dev *edev, unsigned int ulp_id,
451451
unsigned long *events_bmap, u16 max_id)
452452
{
453453
struct net_device *dev = edev->net;

drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.h

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -77,15 +77,15 @@ struct bnxt_en_dev {
7777
};
7878

7979
struct bnxt_en_ops {
80-
int (*bnxt_register_device)(struct bnxt_en_dev *, int,
80+
int (*bnxt_register_device)(struct bnxt_en_dev *, unsigned int,
8181
struct bnxt_ulp_ops *, void *);
82-
int (*bnxt_unregister_device)(struct bnxt_en_dev *, int);
83-
int (*bnxt_request_msix)(struct bnxt_en_dev *, int,
82+
int (*bnxt_unregister_device)(struct bnxt_en_dev *, unsigned int);
83+
int (*bnxt_request_msix)(struct bnxt_en_dev *, unsigned int,
8484
struct bnxt_msix_entry *, int);
85-
int (*bnxt_free_msix)(struct bnxt_en_dev *, int);
86-
int (*bnxt_send_fw_msg)(struct bnxt_en_dev *, int,
85+
int (*bnxt_free_msix)(struct bnxt_en_dev *, unsigned int);
86+
int (*bnxt_send_fw_msg)(struct bnxt_en_dev *, unsigned int,
8787
struct bnxt_fw_msg *);
88-
int (*bnxt_register_fw_async_events)(struct bnxt_en_dev *, int,
88+
int (*bnxt_register_fw_async_events)(struct bnxt_en_dev *, unsigned int,
8989
unsigned long *, u16);
9090
};
9191

0 commit comments

Comments
 (0)