Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     netdev@vger.kernel.org
Cc:     Andrew Lunn <andrew@lunn.ch>,
        Florian Fainelli <f.fainelli@gmail.com>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        UNGLinuxDriver@microchip.com,
        Xiaoliang Yang <xiaoliang.yang_1@nxp.com>,
        linux-kernel@vger.kernel.org
Subject: [PATCH net-next 1/7] net: mscc: ocelot: export a single ocelot_mm_irq()
Date:   Sat, 15 Apr 2023 20:05:45 +0300
Message-Id: <20230415170551.3939607-2-vladimir.oltean@nxp.com>
In-Reply-To: <20230415170551.3939607-1-vladimir.oltean@nxp.com>
References: <20230415170551.3939607-1-vladimir.oltean@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?xtb2jnmEp4j8Qtm5+w4KRepmBjJSqOjQJvUlZQ1n3WINX5VYPZE7XXGM+BOY?=
 =?us-ascii?Q?qDoXu49lkn/zk4pmvMb+zKzMnTDnFDKMsbuCFmN5yCYHHvEp8ugSXRY6a/gH?=
 =?us-ascii?Q?pJSkAFbpipgsyw3ZpMGtSNAysq60YEtNLPxMTxf4WW1hPalVHiK71QfhZyyt?=
 =?us-ascii?Q?M3etRQzfZmVT6ubsYQLOjM6EccRyfTvWdr3HpK0fp8MYn5pFVqUBXarAhzpa?=
 =?us-ascii?Q?J1ZDmwknB5jBjwZHvZqu07jAMHeH5uyTJ8WIkaGoBJEOSrfT5ywQEG3k74s+?=
 =?us-ascii?Q?OhREL8Z6NhhqoAc2ae+F/aiwtI0F1P1tL3kSL2r9rttJA7Up3NSZ/9plCwIU?=
 =?us-ascii?Q?dGVUEjsG0dMxBcS5b/GWr9cemBcwO02MXq81LJXF23QG3BPIgiKMU6Eo4Rj2?=
 =?us-ascii?Q?X+Rxq3eyY8sty18wGWdB3+/S9Fzr9YBlqqL8R2oAueZmhPcGym5bEa2o//f0?=
 =?us-ascii?Q?HoXfF5TZ9kZ0J2X8k+hLYWPSWEKU+9B/pEBkCygfJsbrLXm/liHXGw21MwUO?=
 =?us-ascii?Q?wnVJwSO7zSp/u6ZFzSCA4EvgBr4pc/eMRRpJCsgyIMMv0u5a28ZiaVyA6xVy?=
 =?us-ascii?Q?tV53O7oaii7Cinpg2GAZkd/3rhlE3jnLMFcj6QoJy5N6exaj8vd+80+YWkh+?=
 =?us-ascii?Q?8s8h4nJLf5p1iI0KdKYnqT7S8WufibGaiaCLzdMSg55bVjMUhxx6uf4v0i8D?=
 =?us-ascii?Q?j3oD638s7SE+HWaDX/MpEqalaSzwkXO5hQJi52x/W+LdUJDrRAs1lH5O+YOh?=
 =?us-ascii?Q?eBd9a1BgUAhyAWYSwJlJOdq4ktEun/EQsknzQ2oSuorRlmkPUv76vMxpXHgG?=
 =?us-ascii?Q?SZDjT4kyazvTXdyqh6+an62ticAp+aznAOsl2qp4phpfJpp7gpgzv0u3ju03?=
 =?us-ascii?Q?nYxfpuYz+6BNfOiuOTOMexsRfyr/4eRPlSN6oGWWt2MqbQ8vLzAISLGlTyD1?=
 =?us-ascii?Q?/INXbUtahUGE8WlXCEVh7IYKiJM4k9DxA5RTgKIk6kjd1F5OiIyPnToaEUYu?=
 =?us-ascii?Q?XAk8iddfzQQfWPjzreG8110aaSlw9e6/8YHHwVurasmAZ1cURkq4qKUEjWGN?=
 =?us-ascii?Q?n7Ueb9H6lsZmmsbmLVDSmJo5NmXv5AtevJFe62gSA7eBWh5VLUD5lgwKfNpU?=
 =?us-ascii?Q?vsH2btHp0p9OIuhfkZv2aINWQbKOxYcHvhx9gjrFk9s19CHz9KcXoIvdCqpk?=
 =?us-ascii?Q?iGS1qCd8om3QvxjRXpbb1xE6EpNLUqiz5eIuk7woXk4zx+nI5LSyNGM2UHXf?=
 =?us-ascii?Q?7BPrz0rtGMEQhTY5rlDattmtfFx4PW8VzZQUwNRsHYXOCUvT6l5Ahw5oQqpE?=
 =?us-ascii?Q?hjz77ND4nQdxh+ND0bhxpxciHTTtaYcytGy3KytTEXcgd5tjl9xbtzAyV7TL?=
 =?us-ascii?Q?uNReoOsE0S8VPt0Eh/nnpttYmIF2szUDNimIb+LERjYitP0xHzk+3FpNCV/p?=
 =?us-ascii?Q?979ZmfIBSK4jVP41hiGhzet1cDXaS8M3F4PJBvKWEwxgQCv5TlSiTtvbwvOo?=
 =?us-ascii?Q?Aacb3pG92fTlwvlMIvAnygh9PBV5eUIge1qsyF0gQ8A2hvcJCgGcxiNQ1nt2?=
 =?us-ascii?Q?N+ThaZbskTVe8+o5xmd00oDa59xUFZzGMKKxpjgx2JQcE7dSsaNS2VQxDtPm?=
 =?us-ascii?Q?ug=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: caa91a49-87d7-4c78-52e8-08db3dd3b242
X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB6452.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Apr 2023 17:06:04.3436
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: kbuJD2bI3iiFTw6wSzFAcsEz9HqNBiRl82h9ZmXyye6dmIbZhMarwRxFOkqKOQI0ASKTt6LBfY/xcUm848wqiA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB8158
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,URIBL_BLOCKED autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

When the switch emits an IRQ, we don't know what caused it, and we
iterate through all ports to check the MAC Merge status.

Move that iteration inside the ocelot lib; we will change the locking in
a future change and it would be good to encapsulate that lock completely
within the ocelot lib.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
---
Diff: patch is new.

 drivers/net/dsa/ocelot/felix_vsc9959.c |  5 +----
 drivers/net/ethernet/mscc/ocelot_mm.c  | 12 ++++++++++--
 include/soc/mscc/ocelot.h              |  2 +-
 3 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/drivers/net/dsa/ocelot/felix_vsc9959.c b/drivers/net/dsa/ocelot/felix_vsc9959.c
index dddb28984bdf..478893c06f56 100644
--- a/drivers/net/dsa/ocelot/felix_vsc9959.c
+++ b/drivers/net/dsa/ocelot/felix_vsc9959.c
@@ -2610,12 +2610,9 @@ static const struct felix_info felix_info_vsc9959 = {
 static irqreturn_t felix_irq_handler(int irq, void *data)
 {
 	struct ocelot *ocelot = (struct ocelot *)data;
-	int port;
 
 	ocelot_get_txtstamp(ocelot);
-
-	for (port = 0; port < ocelot->num_phys_ports; port++)
-		ocelot_port_mm_irq(ocelot, port);
+	ocelot_mm_irq(ocelot);
 
 	return IRQ_HANDLED;
 }
diff --git a/drivers/net/ethernet/mscc/ocelot_mm.c b/drivers/net/ethernet/mscc/ocelot_mm.c
index 0a8f21ae23f0..ddaf1fb05e48 100644
--- a/drivers/net/ethernet/mscc/ocelot_mm.c
+++ b/drivers/net/ethernet/mscc/ocelot_mm.c
@@ -49,7 +49,7 @@ static enum ethtool_mm_verify_status ocelot_mm_verify_status(u32 val)
 	}
 }
 
-void ocelot_port_mm_irq(struct ocelot *ocelot, int port)
+static void ocelot_mm_update_port_status(struct ocelot *ocelot, int port)
 {
 	struct ocelot_port *ocelot_port = ocelot->ports[port];
 	struct ocelot_mm_state *mm = &ocelot->mm[port];
@@ -91,7 +91,15 @@ void ocelot_port_mm_irq(struct ocelot *ocelot, int port)
 
 	mutex_unlock(&mm->lock);
 }
-EXPORT_SYMBOL_GPL(ocelot_port_mm_irq);
+
+void ocelot_mm_irq(struct ocelot *ocelot)
+{
+	int port;
+
+	for (port = 0; port < ocelot->num_phys_ports; port++)
+		ocelot_mm_update_port_status(ocelot, port);
+}
+EXPORT_SYMBOL_GPL(ocelot_mm_irq);
 
 int ocelot_port_set_mm(struct ocelot *ocelot, int port,
 		       struct ethtool_mm_cfg *cfg,
diff --git a/include/soc/mscc/ocelot.h b/include/soc/mscc/ocelot.h
index 277e6d1f2096..eb8e3935375d 100644
--- a/include/soc/mscc/ocelot.h
+++ b/include/soc/mscc/ocelot.h
@@ -1148,7 +1148,7 @@ int ocelot_vcap_policer_add(struct ocelot *ocelot, u32 pol_ix,
 			    struct ocelot_policer *pol);
 int ocelot_vcap_policer_del(struct ocelot *ocelot, u32 pol_ix);
 
-void ocelot_port_mm_irq(struct ocelot *ocelot, int port);
+void ocelot_mm_irq(struct ocelot *ocelot);
 int ocelot_port_set_mm(struct ocelot *ocelot, int port,
 		       struct ethtool_mm_cfg *cfg,
 		       struct netlink_ext_ack *extack);
-- 
2.34.1