Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1709647 > unrolled thread

[PATCH] mlxsw: make mlxsw_config_profile const

Started byBhumika Goyal <bhumirks@gmail.com>
First post2017-08-11 15:50 +0200
Last post2017-08-12 00:00 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] mlxsw: make mlxsw_config_profile const Bhumika Goyal <bhumirks@gmail.com> - 2017-08-11 15:50 +0200
    Re: [PATCH] mlxsw: make mlxsw_config_profile const Jiri Pirko <jiri@resnulli.us> - 2017-08-11 17:40 +0200
    Re: [PATCH] mlxsw: make mlxsw_config_profile const David Miller <davem@davemloft.net> - 2017-08-12 00:00 +0200

#1709647 — [PATCH] mlxsw: make mlxsw_config_profile const

FromBhumika Goyal <bhumirks@gmail.com>
Date2017-08-11 15:50 +0200
Subject[PATCH] mlxsw: make mlxsw_config_profile const
Message-ID<udiu6-4b4-29@gated-at.bofh.it>
Make these structures const as they only stored in the profile field of
a mlxsw_driver structure, which is of type const.
Done using Coccinelle.

Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
---
 drivers/net/ethernet/mellanox/mlxsw/spectrum.c | 2 +-
 drivers/net/ethernet/mellanox/mlxsw/switchib.c | 2 +-
 drivers/net/ethernet/mellanox/mlxsw/switchx2.c | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum.c b/drivers/net/ethernet/mellanox/mlxsw/spectrum.c
index eb7c454..82f8ccd 100644
--- a/drivers/net/ethernet/mellanox/mlxsw/spectrum.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum.c
@@ -3743,7 +3743,7 @@ static void mlxsw_sp_fini(struct mlxsw_core *mlxsw_core)
 	mlxsw_sp_fids_fini(mlxsw_sp);
 }
 
-static struct mlxsw_config_profile mlxsw_sp_config_profile = {
+static const struct mlxsw_config_profile mlxsw_sp_config_profile = {
 	.used_max_vepa_channels		= 1,
 	.max_vepa_channels		= 0,
 	.used_max_mid			= 1,
diff --git a/drivers/net/ethernet/mellanox/mlxsw/switchib.c b/drivers/net/ethernet/mellanox/mlxsw/switchib.c
index 74341fe..ab7a298 100644
--- a/drivers/net/ethernet/mellanox/mlxsw/switchib.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/switchib.c
@@ -497,7 +497,7 @@ static void mlxsw_sib_fini(struct mlxsw_core *mlxsw_core)
 	mlxsw_sib_ports_remove(mlxsw_sib);
 }
 
-static struct mlxsw_config_profile mlxsw_sib_config_profile = {
+static const struct mlxsw_config_profile mlxsw_sib_config_profile = {
 	.used_max_system_port		= 1,
 	.max_system_port		= 48000,
 	.used_max_ib_mc			= 1,
diff --git a/drivers/net/ethernet/mellanox/mlxsw/switchx2.c b/drivers/net/ethernet/mellanox/mlxsw/switchx2.c
index 3b0f724..f3c29bb 100644
--- a/drivers/net/ethernet/mellanox/mlxsw/switchx2.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/switchx2.c
@@ -1674,7 +1674,7 @@ static void mlxsw_sx_fini(struct mlxsw_core *mlxsw_core)
 	mlxsw_sx_ports_remove(mlxsw_sx);
 }
 
-static struct mlxsw_config_profile mlxsw_sx_config_profile = {
+static const struct mlxsw_config_profile mlxsw_sx_config_profile = {
 	.used_max_vepa_channels		= 1,
 	.max_vepa_channels		= 0,
 	.used_max_mid			= 1,
-- 
1.9.1

[toc] | [next] | [standalone]


#1709752

FromJiri Pirko <jiri@resnulli.us>
Date2017-08-11 17:40 +0200
Message-ID<udkcz-5fX-35@gated-at.bofh.it>
In reply to#1709647
Fri, Aug 11, 2017 at 03:40:42PM CEST, bhumirks@gmail.com wrote:
>Make these structures const as they only stored in the profile field of
>a mlxsw_driver structure, which is of type const.
>Done using Coccinelle.
>
>Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>

Acked-by: Jiri Pirko <jiri@mellanox.com>

[toc] | [prev] | [next] | [standalone]


#1709997

FromDavid Miller <davem@davemloft.net>
Date2017-08-12 00:00 +0200
Message-ID<udq8i-la-25@gated-at.bofh.it>
In reply to#1709647
From: Bhumika Goyal <bhumirks@gmail.com>
Date: Fri, 11 Aug 2017 19:10:42 +0530

> Make these structures const as they only stored in the profile field of
> a mlxsw_driver structure, which is of type const.
> Done using Coccinelle.
> 
> Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>

Applied to net-next.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web