mlxsw: item: Make char *buf arg constant for getters

Enforce const for getter buf args.

Signed-off-by: Jiri Pirko <jiri@mellanox.com>
Acked-by: Ido Schimmel <idosch@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/ethernet/mellanox/mlxsw/item.h b/drivers/net/ethernet/mellanox/mlxsw/item.h
index 1c55a34e0aa21f9e376b488437ff9ba319a9e4e4..3c95e3ddd9c25c18ed0daff92518194652e2d2af 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlxsw/item.h
+++ b/drivers/net/ethernet/mellanox/mlxsw/item.h
@@ -72,7 +72,8 @@ __mlxsw_item_offset(const struct mlxsw_item *item, unsigned short index,
                typesize);
 }
 
-static inline u16 __mlxsw_item_get16(char *buf, const struct mlxsw_item *item,
+static inline u16 __mlxsw_item_get16(const char *buf,
+                                    const struct mlxsw_item *item,
                                     unsigned short index)
 {
        unsigned int offset = __mlxsw_item_offset(item, index, sizeof(u16));
@@ -105,7 +106,8 @@ static inline void __mlxsw_item_set16(char *buf, const struct mlxsw_item *item,
        b[offset] = cpu_to_be16(tmp);
 }
 
-static inline u32 __mlxsw_item_get32(char *buf, const struct mlxsw_item *item,
+static inline u32 __mlxsw_item_get32(const char *buf,
+                                    const struct mlxsw_item *item,
                                     unsigned short index)
 {
        unsigned int offset = __mlxsw_item_offset(item, index, sizeof(u32));
@@ -138,7 +140,8 @@ static inline void __mlxsw_item_set32(char *buf, const struct mlxsw_item *item,
        b[offset] = cpu_to_be32(tmp);
 }
 
-static inline u64 __mlxsw_item_get64(char *buf, const struct mlxsw_item *item,
+static inline u64 __mlxsw_item_get64(const char *buf,
+                                    const struct mlxsw_item *item,
                                     unsigned short index)
 {
        unsigned int offset = __mlxsw_item_offset(item, index, sizeof(u64));
@@ -170,7 +173,7 @@ static inline void __mlxsw_item_set64(char *buf, const struct mlxsw_item *item,
        b[offset] = cpu_to_be64(tmp);
 }
 
-static inline void __mlxsw_item_memcpy_from(char *buf, char *dst,
+static inline void __mlxsw_item_memcpy_from(const char *buf, char *dst,
                                            const struct mlxsw_item *item,
                                            unsigned short index)
 {
@@ -213,7 +216,7 @@ __mlxsw_item_bit_array_offset(const struct mlxsw_item *item,
        return item->offset + offset;
 }
 
-static inline u8 __mlxsw_item_bit_array_get(char *buf,
+static inline u8 __mlxsw_item_bit_array_get(const char *buf,
                                            const struct mlxsw_item *item,
                                            u16 index)
 {
@@ -257,7 +260,7 @@ static struct mlxsw_item __ITEM_NAME(_type, _cname, _iname) = {                     \
        .size = {.bits = _sizebits,},                                           \
        .name = #_type "_" #_cname "_" #_iname,                                 \
 };                                                                             \
-static inline u16 mlxsw_##_type##_##_cname##_##_iname##_get(char *buf)         \
+static inline u16 mlxsw_##_type##_##_cname##_##_iname##_get(const char *buf)   \
 {                                                                              \
        return __mlxsw_item_get16(buf, &__ITEM_NAME(_type, _cname, _iname), 0); \
 }                                                                              \
@@ -278,7 +281,7 @@ static struct mlxsw_item __ITEM_NAME(_type, _cname, _iname) = {                     \
        .name = #_type "_" #_cname "_" #_iname,                                 \
 };                                                                             \
 static inline u16                                                              \
-mlxsw_##_type##_##_cname##_##_iname##_get(char *buf, unsigned short index)     \
+mlxsw_##_type##_##_cname##_##_iname##_get(const char *buf, unsigned short index)\
 {                                                                              \
        return __mlxsw_item_get16(buf, &__ITEM_NAME(_type, _cname, _iname),     \
                                  index);                                       \
@@ -298,7 +301,7 @@ static struct mlxsw_item __ITEM_NAME(_type, _cname, _iname) = {                     \
        .size = {.bits = _sizebits,},                                           \
        .name = #_type "_" #_cname "_" #_iname,                                 \
 };                                                                             \
-static inline u32 mlxsw_##_type##_##_cname##_##_iname##_get(char *buf)         \
+static inline u32 mlxsw_##_type##_##_cname##_##_iname##_get(const char *buf)   \
 {                                                                              \
        return __mlxsw_item_get32(buf, &__ITEM_NAME(_type, _cname, _iname), 0); \
 }                                                                              \
@@ -319,7 +322,7 @@ static struct mlxsw_item __ITEM_NAME(_type, _cname, _iname) = {                     \
        .name = #_type "_" #_cname "_" #_iname,                                 \
 };                                                                             \
 static inline u32                                                              \
-mlxsw_##_type##_##_cname##_##_iname##_get(char *buf, unsigned short index)     \
+mlxsw_##_type##_##_cname##_##_iname##_get(const char *buf, unsigned short index)\
 {                                                                              \
        return __mlxsw_item_get32(buf, &__ITEM_NAME(_type, _cname, _iname),     \
                                  index);                                       \
@@ -339,7 +342,7 @@ static struct mlxsw_item __ITEM_NAME(_type, _cname, _iname) = {                     \
        .size = {.bits = _sizebits,},                                           \
        .name = #_type "_" #_cname "_" #_iname,                                 \
 };                                                                             \
-static inline u64 mlxsw_##_type##_##_cname##_##_iname##_get(char *buf)         \
+static inline u64 mlxsw_##_type##_##_cname##_##_iname##_get(const char *buf)   \
 {                                                                              \
        return __mlxsw_item_get64(buf, &__ITEM_NAME(_type, _cname, _iname), 0); \
 }                                                                              \
@@ -360,7 +363,7 @@ static struct mlxsw_item __ITEM_NAME(_type, _cname, _iname) = {                     \
        .name = #_type "_" #_cname "_" #_iname,                                 \
 };                                                                             \
 static inline u64                                                              \
-mlxsw_##_type##_##_cname##_##_iname##_get(char *buf, unsigned short index)     \
+mlxsw_##_type##_##_cname##_##_iname##_get(const char *buf, unsigned short index)\
 {                                                                              \
        return __mlxsw_item_get64(buf, &__ITEM_NAME(_type, _cname, _iname),     \
                                  index);                                       \
@@ -380,7 +383,7 @@ static struct mlxsw_item __ITEM_NAME(_type, _cname, _iname) = {                     \
        .name = #_type "_" #_cname "_" #_iname,                                 \
 };                                                                             \
 static inline void                                                             \
-mlxsw_##_type##_##_cname##_##_iname##_memcpy_from(char *buf, char *dst)                \
+mlxsw_##_type##_##_cname##_##_iname##_memcpy_from(const char *buf, char *dst)  \
 {                                                                              \
        __mlxsw_item_memcpy_from(buf, dst,                                      \
                                 &__ITEM_NAME(_type, _cname, _iname), 0);       \
@@ -402,7 +405,7 @@ static struct mlxsw_item __ITEM_NAME(_type, _cname, _iname) = {                     \
        .name = #_type "_" #_cname "_" #_iname,                                 \
 };                                                                             \
 static inline void                                                             \
-mlxsw_##_type##_##_cname##_##_iname##_memcpy_from(char *buf,                   \
+mlxsw_##_type##_##_cname##_##_iname##_memcpy_from(const char *buf,             \
                                                  unsigned short index,         \
                                                  char *dst)                    \
 {                                                                              \
@@ -427,7 +430,7 @@ static struct mlxsw_item __ITEM_NAME(_type, _cname, _iname) = {                     \
        .name = #_type "_" #_cname "_" #_iname,                                 \
 };                                                                             \
 static inline u8                                                               \
-mlxsw_##_type##_##_cname##_##_iname##_get(char *buf, u16 index)                        \
+mlxsw_##_type##_##_cname##_##_iname##_get(const char *buf, u16 index)          \
 {                                                                              \
        return __mlxsw_item_bit_array_get(buf,                                  \
                                          &__ITEM_NAME(_type, _cname, _iname),  \

diff --git a/drivers/net/ethernet/mellanox/mlxsw/pci.c b/drivers/net/ethernet/mellanox/mlxsw/pci.c
index e742bd4e8894a4493d60251a44dbd1209b310601..6f368068aa15c62689ad099078e8fd8c6ae38958 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlxsw/pci.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/pci.c
@@ -238,8 +238,9 @@ static bool mlxsw_pci_elem_hw_owned(struct mlxsw_pci_queue *q, bool owner_bit)
        return owner_bit != !!(q->consumer_counter & q->count);
 }
 
-static char *mlxsw_pci_queue_sw_elem_get(struct mlxsw_pci_queue *q,
-                                        u32 (*get_elem_owner_func)(char *))
+static char *
+mlxsw_pci_queue_sw_elem_get(struct mlxsw_pci_queue *q,
+                           u32 (*get_elem_owner_func)(const char *))
 {
        struct mlxsw_pci_queue_elem_info *elem_info;
        char *elem;

diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum.c b/drivers/net/ethernet/mellanox/mlxsw/spectrum.c
index 6d8cb22579ee7f0a87fb2de9f3f5c9c376215afc..991a320470a9e75f98f4e85fa9803f9cbad7deaa 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlxsw/spectrum.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum.c
@@ -1413,7 +1413,7 @@ err_port_pause_configure:
 
 struct mlxsw_sp_port_hw_stats {
        char str[ETH_GSTRING_LEN];
-       u64 (*getter)(char *payload);
+       u64 (*getter)(const char *payload);
 };
 
 static struct mlxsw_sp_port_hw_stats mlxsw_sp_port_hw_stats[] = {
@@ -1534,7 +1534,7 @@ static struct mlxsw_sp_port_hw_stats mlxsw_sp_port_hw_prio_stats[] = {
 
 #define MLXSW_SP_PORT_HW_PRIO_STATS_LEN ARRAY_SIZE(mlxsw_sp_port_hw_prio_stats)
 
-static u64 mlxsw_reg_ppcnt_tc_transmit_queue_bytes_get(char *ppcnt_pl)
+static u64 mlxsw_reg_ppcnt_tc_transmit_queue_bytes_get(const char *ppcnt_pl)
 {
        u64 transmit_queue = mlxsw_reg_ppcnt_tc_transmit_queue_get(ppcnt_pl);
 

diff --git a/drivers/net/ethernet/mellanox/mlxsw/switchx2.c b/drivers/net/ethernet/mellanox/mlxsw/switchx2.c
index 66af63d99b8b05deca7e8e23cafcf3a22d725ac4..963618da81a9d59adffd6fc7b3ce960578b948dc 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlxsw/switchx2.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/switchx2.c
@@ -410,7 +410,7 @@ static void mlxsw_sx_port_get_drvinfo(struct net_device *dev,
 
 struct mlxsw_sx_port_hw_stats {
        char str[ETH_GSTRING_LEN];
-       u64 (*getter)(char *payload);
+       u64 (*getter)(const char *payload);
 };
 
 static const struct mlxsw_sx_port_hw_stats mlxsw_sx_port_hw_stats[] = {