From nobody Mon Jun 8 14:39:56 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 77C543FCB2B; Thu, 28 May 2026 16:09:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779984582; cv=none; b=KaTB0pk0N5XmZpg04fp9k2mgxPSlJKUCA1bB8JaP3KlQ/AoCZ3ZWklvcZtiAkqX7Efb1IPwQVPYrPC+frF15eBeKJFe9bPtV1PPhlUxxo6J5AnQr7gzPvtpp5YMIz6xh0MVonEISUZG6YgsCb+H0ZUvdzNg/giESFG+g1ugdBAg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779984582; c=relaxed/simple; bh=VdBUolhNyo8yi7fcXoYRSQ+Yv/xh8n4VXOHyJf6wnGc=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=E7aFqMEKF93s9eMUFX/8ZVR89Q53cPJ8Vl3vAQvO9tAYHjed9AGbeHGbo3tE9dLe8PfKkQO+rVaAyeR3ifkEbYM4ofmdXcLlBgtlaD4GoUlduUjyb8S3CrUpE1mhCPB2E5bvGmdzqXDa9BWYewr5yNVT8CxVJ77NVMXDron/bBM= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=GS6kXqUW; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="GS6kXqUW" Received: by smtp.kernel.org (Postfix) with ESMTPS id 3E4FCC2BCC4; Thu, 28 May 2026 16:09:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1779984582; bh=VdBUolhNyo8yi7fcXoYRSQ+Yv/xh8n4VXOHyJf6wnGc=; h=From:Date:Subject:References:In-Reply-To:To:Cc:Reply-To:From; b=GS6kXqUWXsaWn9K1neFeiSiRcgOpLdbmo1GpgI4zqG6rLZ/d513W+q6ypDoS5m1GH 1jNcX9C9mNpfP9iM61sprUbsAbj5OY0W6jU27q1F68Qe4wqib5KEZrRZMQ+jcQZBm7 WrhSdt4dAdo25vkKtPUNa6+DGBPdUsB5cobmCTH8lB0nYtIInFzKaJZcu19cP1qn+2 7oDz5XCMOb2ZSHjxwXaxESHNbjGh6UWBCvb+hI4+sGU7OH3cC3VB8s1QmKVq+UEVEm jqCkgbglEFJ0WNzqTjN/vtt3j8LbIs2CoOoRpWNW1FuE5c3b/bQhsqaRoNNQeWAfvf AZhb+mesdz/8A== Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id 29E33CD4F54; Thu, 28 May 2026 16:09:42 +0000 (UTC) From: Cedric Jehasse via B4 Relay Date: Thu, 28 May 2026 18:09:32 +0200 Subject: [PATCH net-next v4 1/2] net: dsa: mv88e6xxx: use the hw tx queues Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260528-net-next-mv88e6xxx-cbs-v4-1-8bd13b906457@luminex.be> References: <20260528-net-next-mv88e6xxx-cbs-v4-0-8bd13b906457@luminex.be> In-Reply-To: <20260528-net-next-mv88e6xxx-cbs-v4-0-8bd13b906457@luminex.be> To: Andrew Lunn , Vladimir Oltean , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Simon Horman , Russell King Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Luke Howard , =?utf-8?q?Marek_Beh=C3=BAn?= , Cedric Jehasse , Cedric Jehasse X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=ed25519-sha256; t=1779984580; l=11505; i=cedric.jehasse@luminex.be; s=20260213; h=from:subject:message-id; bh=N73EMbDeMk4TXkTfSVwg8qX1Kj5tDWt/w+RuUrgiY1M=; b=uSNDGYA/EuSh3u41NuKHbZu7Huid465w7o3F14xkBASOKNLAnDgE0vAFnaPN45aQLNMLu25LS QT9+F/pmjfVD1jeHgJHRoYG+7yiSay+xySJSb8MmzK5g+/tgvDMag9p X-Developer-Key: i=cedric.jehasse@luminex.be; a=ed25519; pk=DL2RaHc2bc7tTDTMJdugm42BtJk4Ip+7V41Gkj+Avjk= X-Endpoint-Received: by B4 Relay for cedric.jehasse@luminex.be/20260213 with auth_id=638 X-Original-From: Cedric Jehasse Reply-To: cedric.jehasse@luminex.be From: Cedric Jehasse When transmitting fill in the PRI field in the dsa tag to select the egress queue is sent to. From the datasheets i've looked at these switches have 4 or 8 transmit queues per port. Note: skbs with skb->fw_offload_fwd_mark set use the DSA_CMD_FORWARD tag. These are processed as normal ingress frames, meaning the queue they end up in can still be altered by other switch config. eg. priority overrides, tcam policies. This isn't done for vlan tagged frames because this would overwrite the PCP value in the vlan tag (The PRI field in the dsa tag is used as the PCP value in the vlan tag). Signed-off-by: Cedric Jehasse --- drivers/net/dsa/mv88e6xxx/chip.c | 35 +++++++++++++++++++++++++++++++++++ drivers/net/dsa/mv88e6xxx/chip.h | 1 + net/dsa/tag_dsa.c | 11 ++++++++++- 3 files changed, 46 insertions(+), 1 deletion(-) diff --git a/drivers/net/dsa/mv88e6xxx/chip.c b/drivers/net/dsa/mv88e6xxx/c= hip.c index 8ca5fd40df92..ffd4fa41b7c5 100644 --- a/drivers/net/dsa/mv88e6xxx/chip.c +++ b/drivers/net/dsa/mv88e6xxx/chip.c @@ -3979,6 +3979,7 @@ static int mv88e6xxx_setup(struct dsa_switch *ds) =20 chip->ds =3D ds; ds->user_mii_bus =3D mv88e6xxx_default_mdio_bus(chip); + ds->num_tx_queues =3D chip->info->num_tx_queues; =20 /* Since virtual bridges are mapped in the PVT, the number we support * depends on the physical switch topology. We need to let DSA figure @@ -5689,6 +5690,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { */ .num_ports =3D 7, .num_internal_phys =3D 2, + .num_tx_queues =3D 4, .invalid_port_mask =3D BIT(2) | BIT(3) | BIT(4), .max_vid =3D 4095, .port_base_addr =3D 0x8, @@ -5711,6 +5713,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_databases =3D 64, .num_ports =3D 7, .num_internal_phys =3D 5, + .num_tx_queues =3D 4, .max_vid =3D 4095, .port_base_addr =3D 0x08, .phy_base_addr =3D 0x00, @@ -5733,6 +5736,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_macs =3D 8192, .num_ports =3D 10, .num_internal_phys =3D 5, + .num_tx_queues =3D 4, .max_vid =3D 4095, .max_sid =3D 63, .port_base_addr =3D 0x10, @@ -5757,6 +5761,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_macs =3D 8192, .num_ports =3D 11, .num_internal_phys =3D 0, + .num_tx_queues =3D 4, .max_vid =3D 4095, .port_base_addr =3D 0x10, .phy_base_addr =3D 0x0, @@ -5778,6 +5783,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_macs =3D 8192, .num_ports =3D 11, .num_internal_phys =3D 8, + .num_tx_queues =3D 4, .max_vid =3D 4095, .max_sid =3D 63, .port_base_addr =3D 0x10, @@ -5803,6 +5809,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_macs =3D 1024, .num_ports =3D 3, .num_internal_phys =3D 5, + .num_tx_queues =3D 4, .max_vid =3D 4095, .max_sid =3D 63, .port_base_addr =3D 0x10, @@ -5828,6 +5835,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_macs =3D 8192, .num_ports =3D 8, .num_internal_phys =3D 0, + .num_tx_queues =3D 4, .max_vid =3D 4095, .port_base_addr =3D 0x10, .phy_base_addr =3D 0x0, @@ -5850,6 +5858,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_ports =3D 6, .num_internal_phys =3D 5, .num_gpio =3D 11, + .num_tx_queues =3D 4, .max_vid =3D 4095, .max_sid =3D 63, .port_base_addr =3D 0x10, @@ -5875,6 +5884,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_macs =3D 1024, .num_ports =3D 6, .num_internal_phys =3D 5, + .num_tx_queues =3D 4, .max_vid =3D 4095, .max_sid =3D 63, .port_base_addr =3D 0x10, @@ -5901,6 +5911,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_macs =3D 8192, .num_ports =3D 6, .num_internal_phys =3D 0, + .num_tx_queues =3D 4, .max_vid =3D 4095, .max_sid =3D 63, .port_base_addr =3D 0x10, @@ -5926,6 +5937,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_macs =3D 8192, .num_ports =3D 7, .num_internal_phys =3D 5, + .num_tx_queues =3D 4, .max_vid =3D 4095, .max_sid =3D 63, .port_base_addr =3D 0x10, @@ -5952,6 +5964,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_ports =3D 7, .num_internal_phys =3D 5, .num_gpio =3D 15, + .num_tx_queues =3D 4, .max_vid =3D 4095, .max_sid =3D 63, .port_base_addr =3D 0x10, @@ -5977,6 +5990,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_macs =3D 8192, .num_ports =3D 7, .num_internal_phys =3D 5, + .num_tx_queues =3D 4, .max_vid =3D 4095, .max_sid =3D 63, .port_base_addr =3D 0x10, @@ -6003,6 +6017,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_ports =3D 7, .num_internal_phys =3D 5, .num_gpio =3D 15, + .num_tx_queues =3D 4, .max_vid =3D 4095, .max_sid =3D 63, .port_base_addr =3D 0x10, @@ -6028,6 +6043,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_macs =3D 8192, .num_ports =3D 10, .num_internal_phys =3D 0, + .num_tx_queues =3D 4, .max_vid =3D 4095, .port_base_addr =3D 0x10, .phy_base_addr =3D 0x0, @@ -6051,6 +6067,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_ports =3D 11, /* 10 + Z80 */ .num_internal_phys =3D 9, .num_gpio =3D 16, + .num_tx_queues =3D 8, .max_vid =3D 8191, .max_sid =3D 63, .port_base_addr =3D 0x0, @@ -6076,6 +6093,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_ports =3D 11, /* 10 + Z80 */ .num_internal_phys =3D 9, .num_gpio =3D 16, + .num_tx_queues =3D 8, .max_vid =3D 8191, .max_sid =3D 63, .port_base_addr =3D 0x0, @@ -6100,6 +6118,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_macs =3D 16384, .num_ports =3D 11, /* 10 + Z80 */ .num_internal_phys =3D 9, + .num_tx_queues =3D 8, .max_vid =3D 8191, .max_sid =3D 63, .port_base_addr =3D 0x0, @@ -6125,6 +6144,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_ports =3D 11, /* 10 + Z80 */ .num_internal_phys =3D 8, .internal_phys_offset =3D 1, + .num_tx_queues =3D 8, .max_vid =3D 8191, .max_sid =3D 63, .port_base_addr =3D 0x0, @@ -6151,6 +6171,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_internal_phys =3D 8, .num_tcam_entries =3D 256, .internal_phys_offset =3D 1, + .num_tx_queues =3D 8, .max_vid =3D 8191, .max_sid =3D 63, .port_base_addr =3D 0x0, @@ -6181,6 +6202,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_ports =3D 7, .num_internal_phys =3D 2, .invalid_port_mask =3D BIT(2) | BIT(3) | BIT(4), + .num_tx_queues =3D 4, .max_vid =3D 4095, .port_base_addr =3D 0x08, .phy_base_addr =3D 0x00, @@ -6205,6 +6227,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_ports =3D 7, .num_internal_phys =3D 5, .num_gpio =3D 15, + .num_tx_queues =3D 4, .max_vid =3D 4095, .max_sid =3D 63, .port_base_addr =3D 0x10, @@ -6230,6 +6253,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_databases =3D 64, .num_ports =3D 7, .num_internal_phys =3D 5, + .num_tx_queues =3D 4, .max_vid =3D 4095, .port_base_addr =3D 0x08, .phy_base_addr =3D 0x00, @@ -6254,6 +6278,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_internal_phys =3D 9, .num_gpio =3D 16, .num_tcam_entries =3D 256, + .num_tx_queues =3D 8, .max_vid =3D 8191, .max_sid =3D 63, .port_base_addr =3D 0x0, @@ -6282,6 +6307,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_internal_phys =3D 2, .internal_phys_offset =3D 3, .num_gpio =3D 15, + .num_tx_queues =3D 4, .max_vid =3D 4095, .max_sid =3D 63, .port_base_addr =3D 0x10, @@ -6310,6 +6336,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_internal_phys =3D 2, .internal_phys_offset =3D 3, .num_gpio =3D 15, + .num_tx_queues =3D 4, .max_vid =3D 4095, .max_sid =3D 63, .port_base_addr =3D 0x10, @@ -6337,6 +6364,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_internal_phys =3D 5, .num_ports =3D 6, .num_gpio =3D 11, + .num_tx_queues =3D 4, .max_vid =3D 4095, .max_sid =3D 63, .port_base_addr =3D 0x10, @@ -6363,6 +6391,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_macs =3D 8192, .num_ports =3D 7, .num_internal_phys =3D 5, + .num_tx_queues =3D 4, .max_vid =3D 4095, .max_sid =3D 63, .port_base_addr =3D 0x10, @@ -6388,6 +6417,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_macs =3D 8192, .num_ports =3D 7, .num_internal_phys =3D 5, + .num_tx_queues =3D 4, .max_vid =3D 4095, .max_sid =3D 63, .port_base_addr =3D 0x10, @@ -6414,6 +6444,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_ports =3D 7, .num_internal_phys =3D 5, .num_gpio =3D 15, + .num_tx_queues =3D 4, .max_vid =3D 4095, .max_sid =3D 63, .port_base_addr =3D 0x10, @@ -6442,6 +6473,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .invalid_port_mask =3D BIT(1) | BIT(2) | BIT(8), .num_internal_phys =3D 5, .internal_phys_offset =3D 3, + .num_tx_queues =3D 8, .max_vid =3D 8191, .max_sid =3D 63, .port_base_addr =3D 0x0, @@ -6468,6 +6500,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_internal_phys =3D 9, .num_gpio =3D 16, .num_tcam_entries =3D 256, + .num_tx_queues =3D 8, .max_vid =3D 8191, .max_sid =3D 63, .port_base_addr =3D 0x0, @@ -6495,6 +6528,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_ports =3D 11, /* 10 + Z80 */ .num_internal_phys =3D 9, .num_gpio =3D 16, + .num_tx_queues =3D 8, .max_vid =3D 8191, .max_sid =3D 63, .port_base_addr =3D 0x0, @@ -6521,6 +6555,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .num_ports =3D 11, /* 10 + Z80 */ .num_internal_phys =3D 8, .num_tcam_entries =3D 256, + .num_tx_queues =3D 8, .internal_phys_offset =3D 1, .max_vid =3D 8191, .max_sid =3D 63, diff --git a/drivers/net/dsa/mv88e6xxx/chip.h b/drivers/net/dsa/mv88e6xxx/c= hip.h index cde71828e9d9..19d8eda19b78 100644 --- a/drivers/net/dsa/mv88e6xxx/chip.h +++ b/drivers/net/dsa/mv88e6xxx/chip.h @@ -136,6 +136,7 @@ struct mv88e6xxx_info { unsigned int num_internal_phys; unsigned int num_gpio; unsigned int num_tcam_entries; + unsigned int num_tx_queues; unsigned int max_vid; unsigned int max_sid; unsigned int port_base_addr; diff --git a/net/dsa/tag_dsa.c b/net/dsa/tag_dsa.c index 2a2c4fb61a65..cc27b2994c6b 100644 --- a/net/dsa/tag_dsa.c +++ b/net/dsa/tag_dsa.c @@ -179,8 +179,17 @@ static struct sk_buff *dsa_xmit_ll(struct sk_buff *skb= , struct net_device *dev, dsa_header[2] &=3D ~0x10; } } else { + u16 queue =3D skb_get_queue_mapping(skb) & 0x7; u16 vid; =20 + /* The PRI field is 3 bits. According to the documentation the + * 2 highest bits specify the egress queue in From_CPU DSA + * tagged frames. On devices with 8 queues it's possible to + * send to the 8 queues, which means the 3 bits are used. + */ + if (dp->ds->num_tx_queues =3D=3D 4) + queue <<=3D 1; + vid =3D br_dev ? MV88E6XXX_VID_BRIDGED : MV88E6XXX_VID_STANDALONE; =20 skb_push(skb, DSA_HLEN + extra); @@ -191,7 +200,7 @@ static struct sk_buff *dsa_xmit_ll(struct sk_buff *skb,= struct net_device *dev, =20 dsa_header[0] =3D (cmd << 6) | tag_dev; dsa_header[1] =3D tag_port << 3; - dsa_header[2] =3D vid >> 8; + dsa_header[2] =3D (queue << 5) | vid >> 8; dsa_header[3] =3D vid & 0xff; } =20 --=20 2.43.0 From nobody Mon Jun 8 14:39:56 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 77BBA3FCB22; Thu, 28 May 2026 16:09:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779984582; cv=none; b=HTKueTOp+qUZjU69tvnEcxRSnjIll0NcoSP5tM5EQJBJefuf4TqZgv94pOC2zMvYrErT0FqAsVXj3YZZycnN00y59eD8vlnyukdUDLLVRAqtNj/8onTlDa2DDHIg9PqRTDJFGs4hy+Et2+dD8SFsyzd/Erh79oTBiDA9urrzs5k= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779984582; c=relaxed/simple; bh=Jn4zCyDtdzZFKJBpv0GY3uJJdOnFX6YBRr9wlEmxd88=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=VjXPN938WKwdK/M2vi/omJXcGtJw54U5JGP1VVIngIJLxbSZnDCc+BNpvK7jJTitPTeWQ4DO9rjZlimyCE5y15Es7DHuJZ5FMQUbnQ6D6UnPamgqgCQMM2FjBBtYrwf1og+Z0iKzZXEk6lLWKC02Jm/CszIR50khAzqLCTlt7EU= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=gE6ni/Eh; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="gE6ni/Eh" Received: by smtp.kernel.org (Postfix) with ESMTPS id 4C7A6C2BCC7; Thu, 28 May 2026 16:09:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1779984582; bh=Jn4zCyDtdzZFKJBpv0GY3uJJdOnFX6YBRr9wlEmxd88=; h=From:Date:Subject:References:In-Reply-To:To:Cc:Reply-To:From; b=gE6ni/Eh5n0NZQVqVDukCpZHaX483wzzx3OzOFPukqNz4uSTt48gdyOIBrlrQSkdO ZLWcoLXdobiY49u10L2uw3lfJhgaoIY4QR2xQtL95jhwv7cRj1HryveNiDPEfkn3zH wT423FJ58+W8zquTycdWnki8C7BZ1bqMH62+3kbPcJBrqr+iMr5tgjMuGKasXO632y GrWyS6TF6EKRfleASav7VPgwG+1FmuTQoFo3R/JvkhwadAlMJNAZP3jiHSErrpwG0T 61sjfLILMfgnTl0Mgr2mZkVnAENHXcGe5hdwjcIg52rjfFIFZZeDNVnUZUMNu3foy9 NOerWZLF7is7w== Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id 38570CD5BD5; Thu, 28 May 2026 16:09:42 +0000 (UTC) From: Cedric Jehasse via B4 Relay Date: Thu, 28 May 2026 18:09:33 +0200 Subject: [PATCH net-next v4 2/2] net: dsa: mv88e6xxx: add support for credit based shaper Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260528-net-next-mv88e6xxx-cbs-v4-2-8bd13b906457@luminex.be> References: <20260528-net-next-mv88e6xxx-cbs-v4-0-8bd13b906457@luminex.be> In-Reply-To: <20260528-net-next-mv88e6xxx-cbs-v4-0-8bd13b906457@luminex.be> To: Andrew Lunn , Vladimir Oltean , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Simon Horman , Russell King Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Luke Howard , =?utf-8?q?Marek_Beh=C3=BAn?= , Cedric Jehasse , Cedric Jehasse X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=ed25519-sha256; t=1779984580; l=19671; i=cedric.jehasse@luminex.be; s=20260213; h=from:subject:message-id; bh=obl+8460QY/BjRTf9Vh/8gqIfyNaX7yRNjyaJ7NHzUU=; b=KBEtqLLlc2XxSe2Y5OVy1UwQxNiMYHUuYEXilMaT4qUCyG09Hcy7KHuBGtbEsvNN2ttYldI5D PVEDAOp7CHZBgSMyRacPYO5tTEQ3/1CbwfFtngrtXxHGJ9CPe6b4RPv X-Developer-Key: i=cedric.jehasse@luminex.be; a=ed25519; pk=DL2RaHc2bc7tTDTMJdugm42BtJk4Ip+7V41Gkj+Avjk= X-Endpoint-Received: by B4 Relay for cedric.jehasse@luminex.be/20260213 with auth_id=638 X-Original-From: Cedric Jehasse Reply-To: cedric.jehasse@luminex.be From: Cedric Jehasse Some of the chips supported by this driver have credit based shaper support. Support is added for the 6341, 6352, 6390 and 6393 families. This is configured using the Qav registers in the AVB register block. There are small differences in the Qav registers between the chip families (eg. the unit used for the rate and number of bits in the registers). mv88e6xxx_qav_info is introduced to configure this per chip. Eg. setting up 20mbps credit based shaper on a 1GBit link: tc qdisc add dev p8 parent root handle 100: mqprio \ num_tc 8 \ map 0 0 6 7 0 5 0 0 0 0 0 0 0 0 0 0 \ queues 1@0 1@1 1@2 1@3 1@4 1@5 1@6 1@7 \ hw 0 tc qdisc replace dev p8 parent 100:8 cbs locredit -1470 hicredit 30 \ sendslope -980000 idleslope 20000 offload 1 Note: only idleslope and hicredit can be programmed in the switch registers, other parameters won't affect settings. Signed-off-by: Cedric Jehasse --- drivers/net/dsa/mv88e6xxx/chip.c | 130 ++++++++++++++++++++++++++++= ++++ drivers/net/dsa/mv88e6xxx/chip.h | 20 +++++ drivers/net/dsa/mv88e6xxx/global2.h | 3 + drivers/net/dsa/mv88e6xxx/global2_avb.c | 21 ++++++ drivers/net/dsa/mv88e6xxx/port.c | 63 ++++++++++++++++ drivers/net/dsa/mv88e6xxx/port.h | 20 +++++ 6 files changed, 257 insertions(+) diff --git a/drivers/net/dsa/mv88e6xxx/chip.c b/drivers/net/dsa/mv88e6xxx/c= hip.c index ffd4fa41b7c5..5dc5961f37d6 100644 --- a/drivers/net/dsa/mv88e6xxx/chip.c +++ b/drivers/net/dsa/mv88e6xxx/chip.c @@ -32,6 +32,7 @@ #include #include #include +#include =20 #include "chip.h" #include "devlink.h" @@ -5015,6 +5016,7 @@ static const struct mv88e6xxx_ops mv88e6240_ops =3D { .port_set_ether_type =3D mv88e6351_port_set_ether_type, .port_set_jumbo_size =3D mv88e6165_port_set_jumbo_size, .port_egress_rate_limiting =3D mv88e6097_port_egress_rate_limiting, + .port_set_scheduling_mode =3D mv88e6352_port_set_scheduling_mode, .port_pause_limit =3D mv88e6097_port_pause_limit, .port_disable_learn_limit =3D mv88e6xxx_port_disable_learn_limit, .port_disable_pri_override =3D mv88e6xxx_port_disable_pri_override, @@ -5288,6 +5290,7 @@ static const struct mv88e6xxx_ops mv88e6341_ops =3D { .port_set_ether_type =3D mv88e6351_port_set_ether_type, .port_set_jumbo_size =3D mv88e6165_port_set_jumbo_size, .port_egress_rate_limiting =3D mv88e6097_port_egress_rate_limiting, + .port_set_scheduling_mode =3D mv88e6352_port_set_scheduling_mode, .port_pause_limit =3D mv88e6097_port_pause_limit, .port_disable_learn_limit =3D mv88e6xxx_port_disable_learn_limit, .port_disable_pri_override =3D mv88e6xxx_port_disable_pri_override, @@ -5446,6 +5449,7 @@ static const struct mv88e6xxx_ops mv88e6352_ops =3D { .port_set_ether_type =3D mv88e6351_port_set_ether_type, .port_set_jumbo_size =3D mv88e6165_port_set_jumbo_size, .port_egress_rate_limiting =3D mv88e6097_port_egress_rate_limiting, + .port_set_scheduling_mode =3D mv88e6352_port_set_scheduling_mode, .port_pause_limit =3D mv88e6097_port_pause_limit, .port_disable_learn_limit =3D mv88e6xxx_port_disable_learn_limit, .port_disable_pri_override =3D mv88e6xxx_port_disable_pri_override, @@ -5515,6 +5519,7 @@ static const struct mv88e6xxx_ops mv88e6390_ops =3D { .port_get_cmode =3D mv88e6352_port_get_cmode, .port_set_cmode =3D mv88e6390_port_set_cmode, .port_setup_message_port =3D mv88e6xxx_setup_message_port, + .port_set_scheduling_mode =3D mv88e6390_port_set_scheduling_mode, .stats_snapshot =3D mv88e6390_g1_stats_snapshot, .stats_set_histogram =3D mv88e6390_g1_stats_set_histogram, .stats_get_sset_count =3D mv88e6320_stats_get_sset_count, @@ -5580,6 +5585,7 @@ static const struct mv88e6xxx_ops mv88e6390x_ops =3D { .port_get_cmode =3D mv88e6352_port_get_cmode, .port_set_cmode =3D mv88e6390x_port_set_cmode, .port_setup_message_port =3D mv88e6xxx_setup_message_port, + .port_set_scheduling_mode =3D mv88e6390_port_set_scheduling_mode, .stats_snapshot =3D mv88e6390_g1_stats_snapshot, .stats_set_histogram =3D mv88e6390_g1_stats_set_histogram, .stats_get_sset_count =3D mv88e6320_stats_get_sset_count, @@ -5637,6 +5643,7 @@ static const struct mv88e6xxx_ops mv88e6393x_ops =3D { .port_set_ether_type =3D mv88e6393x_port_set_ether_type, .port_set_jumbo_size =3D mv88e6165_port_set_jumbo_size, .port_egress_rate_limiting =3D mv88e6097_port_egress_rate_limiting, + .port_set_scheduling_mode =3D mv88e6390_port_set_scheduling_mode, .port_pause_limit =3D mv88e6390_port_pause_limit, .port_disable_learn_limit =3D mv88e6xxx_port_disable_learn_limit, .port_disable_pri_override =3D mv88e6xxx_port_disable_pri_override, @@ -5679,6 +5686,27 @@ static const struct mv88e6xxx_ops mv88e6393x_ops =3D= { .tcam_ops =3D &mv88e6393_tcam_ops, }; =20 +static const struct mv88e6xxx_qav_info mv88e6352_qav_info =3D { + .rate_unit =3D 32, + .rate_mask =3D GENMASK(14, 0), + .hi_limit_mask =3D GENMASK(14, 0), + .queue_mask =3D GENMASK(3, 0), +}; + +static const struct mv88e6xxx_qav_info mv88e6341_qav_info =3D { + .rate_unit =3D 64, + .rate_mask =3D GENMASK(15, 0), + .hi_limit_mask =3D GENMASK(14, 0), + .queue_mask =3D GENMASK(3, 0), +}; + +static const struct mv88e6xxx_qav_info mv88e6390_qav_info =3D { + .rate_unit =3D 64, + .rate_mask =3D GENMASK(15, 0), + .hi_limit_mask =3D GENMASK(13, 0), + .queue_mask =3D GENMASK(7, 0), +}; + static const struct mv88e6xxx_info mv88e6xxx_table[] =3D { [MV88E6020] =3D { .prod_num =3D MV88E6XXX_PORT_SWITCH_ID_PROD_6020, @@ -6243,6 +6271,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .multi_chip =3D true, .edsa_support =3D MV88E6XXX_EDSA_SUPPORTED, .ptp_support =3D true, + .qav =3D &mv88e6352_qav_info, .ops =3D &mv88e6240_ops, }, =20 @@ -6380,6 +6409,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .multi_chip =3D true, .edsa_support =3D MV88E6XXX_EDSA_SUPPORTED, .ptp_support =3D true, + .qav =3D &mv88e6341_qav_info, .ops =3D &mv88e6341_ops, }, =20 @@ -6460,6 +6490,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .multi_chip =3D true, .edsa_support =3D MV88E6XXX_EDSA_SUPPORTED, .ptp_support =3D true, + .qav =3D &mv88e6352_qav_info, .ops =3D &mv88e6352_ops, }, [MV88E6361] =3D { @@ -6517,6 +6548,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .multi_chip =3D true, .edsa_support =3D MV88E6XXX_EDSA_UNDOCUMENTED, .ptp_support =3D true, + .qav =3D &mv88e6390_qav_info, .ops =3D &mv88e6390_ops, }, [MV88E6390X] =3D { @@ -6544,6 +6576,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .multi_chip =3D true, .edsa_support =3D MV88E6XXX_EDSA_UNDOCUMENTED, .ptp_support =3D true, + .qav =3D &mv88e6390_qav_info, .ops =3D &mv88e6390x_ops, }, =20 @@ -6572,6 +6605,7 @@ static const struct mv88e6xxx_info mv88e6xxx_table[] = =3D { .pvt =3D true, .multi_chip =3D true, .ptp_support =3D true, + .qav =3D &mv88e6390_qav_info, .ops =3D &mv88e6393x_ops, }, }; @@ -7193,6 +7227,101 @@ static int mv88e6xxx_crosschip_lag_leave(struct dsa= _switch *ds, int sw_index, return err_sync ? : err_pvt; } =20 +static int mv88e6xxx_setup_tc_cbs(struct dsa_switch *ds, int port, + struct tc_cbs_qopt_offload *cbs) +{ + const struct mv88e6xxx_avb_ops *avb_ops; + struct mv88e6xxx_chip *chip =3D ds->priv; + const struct mv88e6xxx_qav_info *qav; + const struct mv88e6xxx_ops *ops; + int hilimit_reg; + int rate_reg; + u8 queue_bit; + u16 hi_limit; + u16 rate =3D 0; + int err; + + ops =3D chip->info->ops; + avb_ops =3D ops->avb_ops; + qav =3D chip->info->qav; + + if (!qav || !avb_ops || !avb_ops->port_qav_write || + !ops->port_set_scheduling_mode) + return -EOPNOTSUPP; + + if (!dsa_is_user_port(ds, port)) + return -EOPNOTSUPP; + + if (!(qav->queue_mask & BIT(cbs->queue))) + return -EOPNOTSUPP; + + queue_bit =3D BIT(cbs->queue); + rate_reg =3D MV88E6XXX_PORT_QAV_CFG_RATE(cbs->queue); + hilimit_reg =3D MV88E6XXX_PORT_QAV_CFG_HI_LIMIT(cbs->queue); + + if (cbs->enable) { + if (cbs->hicredit <=3D 0 || + cbs->hicredit > qav->hi_limit_mask) + return -ERANGE; + + rate =3D DIV_ROUND_UP(cbs->idleslope, qav->rate_unit); + if (rate > qav->rate_mask) + return -ERANGE; + /* avoid using zero rate */ + rate =3D max_t(u16, rate, 1); + } + + mv88e6xxx_reg_lock(chip); + + if (!cbs->enable) { + err =3D mv88e6xxx_port_qav_write(chip, port, rate_reg, 0); + if (err) + goto unlock; + + if (!(chip->ports[port].cbs_active_queues & ~queue_bit)) { + err =3D mv88e6xxx_port_set_scheduling_mode(chip, port, 0); + if (err) + goto unlock; + } + chip->ports[port].cbs_active_queues &=3D ~queue_bit; + goto unlock; + } + + hi_limit =3D cbs->hicredit & qav->hi_limit_mask; + err =3D mv88e6xxx_port_qav_write(chip, port, hilimit_reg, hi_limit); + if (err) + goto unlock; + + err =3D mv88e6xxx_port_qav_write(chip, port, rate_reg, rate); + if (err) + goto unlock; + + err =3D mv88e6xxx_port_set_scheduling_mode(chip, port, + chip->info->num_tx_queues - 1); + if (err) { + mv88e6xxx_port_qav_write(chip, port, rate_reg, 0); + chip->ports[port].cbs_active_queues &=3D ~queue_bit; + goto unlock; + } + chip->ports[port].cbs_active_queues |=3D queue_bit; + +unlock: + mv88e6xxx_reg_unlock(chip); + + return err; +} + +static int mv88e6xxx_port_setup_tc(struct dsa_switch *ds, int port, + enum tc_setup_type type, void *type_data) +{ + switch (type) { + case TC_SETUP_QDISC_CBS: + return mv88e6xxx_setup_tc_cbs(ds, port, type_data); + default: + return -EOPNOTSUPP; + } +} + static const struct phylink_mac_ops mv88e6xxx_phylink_mac_ops =3D { .mac_select_pcs =3D mv88e6xxx_mac_select_pcs, .mac_prepare =3D mv88e6xxx_mac_prepare, @@ -7252,6 +7381,7 @@ static const struct dsa_switch_ops mv88e6xxx_switch_o= ps =3D { .port_hwtstamp_get =3D mv88e6xxx_port_hwtstamp_get, .port_txtstamp =3D mv88e6xxx_port_txtstamp, .port_rxtstamp =3D mv88e6xxx_port_rxtstamp, + .port_setup_tc =3D mv88e6xxx_port_setup_tc, .cls_flower_add =3D mv88e6xxx_cls_flower_add, .cls_flower_del =3D mv88e6xxx_cls_flower_del, .get_ts_info =3D mv88e6xxx_get_ts_info, diff --git a/drivers/net/dsa/mv88e6xxx/chip.h b/drivers/net/dsa/mv88e6xxx/c= hip.h index 19d8eda19b78..8768d599f8fc 100644 --- a/drivers/net/dsa/mv88e6xxx/chip.h +++ b/drivers/net/dsa/mv88e6xxx/chip.h @@ -125,6 +125,7 @@ enum mv88e6xxx_edsa_support { }; =20 struct mv88e6xxx_ops; +struct mv88e6xxx_qav_info; =20 struct mv88e6xxx_info { enum mv88e6xxx_family family; @@ -177,6 +178,9 @@ struct mv88e6xxx_info { /* Supports PTP */ bool ptp_support; =20 + /* 802.1Qav credit based shaping */ + const struct mv88e6xxx_qav_info *qav; + /* Internal PHY start index. 0 means that internal PHYs range starts at * port 0, 1 means internal PHYs range starts at port 1, etc */ @@ -304,6 +308,9 @@ struct mv88e6xxx_port { =20 /* MacAuth Bypass control flag */ bool mab; + + /* Queues with CBS currently enabled. */ + u8 cbs_active_queues; }; =20 enum mv88e6xxx_region_id { @@ -607,6 +614,8 @@ struct mv88e6xxx_ops { size_t size); =20 int (*port_egress_rate_limiting)(struct mv88e6xxx_chip *chip, int port); + int (*port_set_scheduling_mode)(struct mv88e6xxx_chip *chip, int port, + u8 mode); int (*port_pause_limit)(struct mv88e6xxx_chip *chip, int port, u8 in, u8 out); int (*port_disable_learn_limit)(struct mv88e6xxx_chip *chip, int port); @@ -764,6 +773,10 @@ struct mv88e6xxx_avb_ops { int (*tai_read)(struct mv88e6xxx_chip *chip, int addr, u16 *data, int len); int (*tai_write)(struct mv88e6xxx_chip *chip, int addr, u16 data); + + /* Access port-scoped 802.1Qav registers */ + int (*port_qav_write)(struct mv88e6xxx_chip *chip, int port, int addr, + u16 data); }; =20 struct mv88e6xxx_ptp_ops { @@ -799,6 +812,13 @@ struct mv88e6xxx_tcam_ops { int (*flush_tcam)(struct mv88e6xxx_chip *chip); }; =20 +struct mv88e6xxx_qav_info { + u16 rate_unit; /* in kbps */ + u16 rate_mask; /* QPri Rate valid bits mask */ + u16 hi_limit_mask; /* Qpri Hi Limit bits mask*/ + u8 queue_mask; /* supported queues bitmask */ +}; + static inline bool mv88e6xxx_has_stu(struct mv88e6xxx_chip *chip) { return chip->info->max_sid > 0 && diff --git a/drivers/net/dsa/mv88e6xxx/global2.h b/drivers/net/dsa/mv88e6xx= x/global2.h index 82f9b410de0b..f1cdc33dd418 100644 --- a/drivers/net/dsa/mv88e6xxx/global2.h +++ b/drivers/net/dsa/mv88e6xxx/global2.h @@ -186,6 +186,9 @@ #define MV88E6352_G2_AVB_CMD_BLOCK_MASK 0x00e0 #define MV88E6352_G2_AVB_CMD_ADDR_MASK 0x001f =20 +#define MV88E6XXX_PORT_QAV_CFG_RATE(queue) (((queue) & 0x7) << 1) +#define MV88E6XXX_PORT_QAV_CFG_HI_LIMIT(queue) ((((queue) & 0x7) << 1) + 1) + /* Offset 0x17: AVB Data Register */ #define MV88E6352_G2_AVB_DATA 0x17 =20 diff --git a/drivers/net/dsa/mv88e6xxx/global2_avb.c b/drivers/net/dsa/mv88= e6xxx/global2_avb.c index 657783e043ff..6b54e275d21a 100644 --- a/drivers/net/dsa/mv88e6xxx/global2_avb.c +++ b/drivers/net/dsa/mv88e6xxx/global2_avb.c @@ -110,6 +110,15 @@ static int mv88e6352_g2_avb_port_ptp_write(struct mv88= e6xxx_chip *chip, return mv88e6xxx_g2_avb_write(chip, writeop, data); } =20 +static int mv88e6352_g2_avb_port_qav_write(struct mv88e6xxx_chip *chip, + int port, int addr, u16 data) +{ + u16 writeop =3D MV88E6352_G2_AVB_CMD_OP_WRITE | (port << 8) | + (MV88E6352_G2_AVB_CMD_BLOCK_QAV << 5) | addr; + + return mv88e6xxx_g2_avb_write(chip, writeop, data); +} + static int mv88e6352_g2_avb_ptp_read(struct mv88e6xxx_chip *chip, int addr, u16 *data, int len) { @@ -149,6 +158,7 @@ const struct mv88e6xxx_avb_ops mv88e6352_avb_ops =3D { .ptp_write =3D mv88e6352_g2_avb_ptp_write, .tai_read =3D mv88e6352_g2_avb_tai_read, .tai_write =3D mv88e6352_g2_avb_tai_write, + .port_qav_write =3D mv88e6352_g2_avb_port_qav_write, }; =20 static int mv88e6165_g2_avb_tai_read(struct mv88e6xxx_chip *chip, int addr, @@ -174,6 +184,7 @@ const struct mv88e6xxx_avb_ops mv88e6165_avb_ops =3D { .ptp_write =3D mv88e6352_g2_avb_ptp_write, .tai_read =3D mv88e6165_g2_avb_tai_read, .tai_write =3D mv88e6165_g2_avb_tai_write, + .port_qav_write =3D mv88e6352_g2_avb_port_qav_write, }; =20 static int mv88e6390_g2_avb_port_ptp_read(struct mv88e6xxx_chip *chip, @@ -197,6 +208,15 @@ static int mv88e6390_g2_avb_port_ptp_write(struct mv88= e6xxx_chip *chip, return mv88e6xxx_g2_avb_write(chip, writeop, data); } =20 +static int mv88e6390_g2_avb_port_qav_write(struct mv88e6xxx_chip *chip, + int port, int addr, u16 data) +{ + u16 writeop =3D MV88E6390_G2_AVB_CMD_OP_WRITE | (port << 8) | + (MV88E6352_G2_AVB_CMD_BLOCK_QAV << 5) | addr; + + return mv88e6xxx_g2_avb_write(chip, writeop, data); +} + static int mv88e6390_g2_avb_ptp_read(struct mv88e6xxx_chip *chip, int addr, u16 *data, int len) { @@ -236,4 +256,5 @@ const struct mv88e6xxx_avb_ops mv88e6390_avb_ops =3D { .ptp_write =3D mv88e6390_g2_avb_ptp_write, .tai_read =3D mv88e6390_g2_avb_tai_read, .tai_write =3D mv88e6390_g2_avb_tai_write, + .port_qav_write =3D mv88e6390_g2_avb_port_qav_write, }; diff --git a/drivers/net/dsa/mv88e6xxx/port.c b/drivers/net/dsa/mv88e6xxx/p= ort.c index c90117d2dd83..23d1435db0d8 100644 --- a/drivers/net/dsa/mv88e6xxx/port.c +++ b/drivers/net/dsa/mv88e6xxx/port.c @@ -1348,6 +1348,51 @@ int mv88e6097_port_egress_rate_limiting(struct mv88e= 6xxx_chip *chip, int port) 0x0001); } =20 +int mv88e6352_port_set_scheduling_mode(struct mv88e6xxx_chip *chip, int po= rt, + u8 mode) +{ + u16 reg; + int err; + + if (mode > 3) + return -EINVAL; + + err =3D mv88e6xxx_port_read(chip, port, MV88E6XXX_PORT_EGRESS_RATE_CTL2, + ®); + if (err) + return err; + + reg &=3D ~MV88E6XXX_PORT_EGRESS_RATE_CTL2_SCHEDULE_MASK; + reg |=3D mode << MV88E6XXX_PORT_EGRESS_RATE_CTL2_SCHEDULE_SHIFT; + + return mv88e6xxx_port_write(chip, port, + MV88E6XXX_PORT_EGRESS_RATE_CTL2, reg); +} + +int mv88e6390_port_set_scheduling_mode(struct mv88e6xxx_chip *chip, int po= rt, + u8 mode) +{ + u16 reg; + int err; + + if (mode > MV88E6390_PORT_QUEUE_CTL_SCHEDULE_MASK) + return -EINVAL; + + reg =3D MV88E6390_PORT_QUEUE_CTL_UPDATE | + (MV88E6390_PORT_QUEUE_CTL_SCHEDULE << + MV88E6390_PORT_QUEUE_CTL_PTR_SHIFT) | + (mode & MV88E6390_PORT_QUEUE_CTL_SCHEDULE_MASK); + + err =3D mv88e6xxx_port_write(chip, port, MV88E6390_PORT_QUEUE_CTL, + reg); + if (err) + return err; + + return mv88e6xxx_port_wait_bit(chip, port, MV88E6390_PORT_QUEUE_CTL, + __bf_shf(MV88E6390_PORT_QUEUE_CTL_UPDATE) + , 0); +} + /* Offset 0x0B: Port Association Vector */ =20 int mv88e6xxx_port_set_assoc_vector(struct mv88e6xxx_chip *chip, int port, @@ -1761,3 +1806,21 @@ int mv88e6393x_port_set_policy(struct mv88e6xxx_chip= *chip, int port, =20 return mv88e6393x_port_policy_write(chip, port, ptr, reg); } + +int mv88e6xxx_port_qav_write(struct mv88e6xxx_chip *chip, int port, int ad= dr, + u16 data) +{ + if (!chip->info->ops->avb_ops->port_qav_write) + return -EOPNOTSUPP; + + return chip->info->ops->avb_ops->port_qav_write(chip, port, addr, data); +} + +int mv88e6xxx_port_set_scheduling_mode(struct mv88e6xxx_chip *chip, int po= rt, + u8 mode) +{ + if (!chip->info->ops->port_set_scheduling_mode) + return -EOPNOTSUPP; + + return chip->info->ops->port_set_scheduling_mode(chip, port, mode); +} diff --git a/drivers/net/dsa/mv88e6xxx/port.h b/drivers/net/dsa/mv88e6xxx/p= ort.h index f6041f91215e..cf8655a13729 100644 --- a/drivers/net/dsa/mv88e6xxx/port.h +++ b/drivers/net/dsa/mv88e6xxx/port.h @@ -241,6 +241,18 @@ =20 /* Offset 0x0A: Egress Rate Control 2 */ #define MV88E6XXX_PORT_EGRESS_RATE_CTL2 0x0a +#define MV88E6XXX_PORT_EGRESS_RATE_CTL2_SCHEDULE_MASK 0x3000 +#define MV88E6XXX_PORT_EGRESS_RATE_CTL2_SCHEDULE_SHIFT 12 + +/* Offset 0x1C: Port Queue Control */ +#define MV88E6390_PORT_QUEUE_CTL 0x1c +#define MV88E6390_PORT_QUEUE_CTL_UPDATE 0x8000 +#define MV88E6390_PORT_QUEUE_CTL_PTR_MASK 0x7f00 +#define MV88E6390_PORT_QUEUE_CTL_PTR_SHIFT 8 +#define MV88E6390_PORT_QUEUE_CTL_DATA_MASK 0x00ff +#define MV88E6390_PORT_QUEUE_CTL_SCHEDULE 0x00 +#define MV88E6390_PORT_QUEUE_CTL_SCHEDULE_MASK 0x07 + =20 /* Offset 0x0B: Port Association Vector */ #define MV88E6XXX_PORT_ASSOC_VECTOR 0x0b @@ -569,6 +581,10 @@ int mv88e6165_port_set_jumbo_size(struct mv88e6xxx_chi= p *chip, int port, size_t size); int mv88e6095_port_egress_rate_limiting(struct mv88e6xxx_chip *chip, int p= ort); int mv88e6097_port_egress_rate_limiting(struct mv88e6xxx_chip *chip, int p= ort); +int mv88e6352_port_set_scheduling_mode(struct mv88e6xxx_chip *chip, int po= rt, + u8 mode); +int mv88e6390_port_set_scheduling_mode(struct mv88e6xxx_chip *chip, int po= rt, + u8 mode); int mv88e6xxx_port_set_assoc_vector(struct mv88e6xxx_chip *chip, int port, u16 pav); int mv88e6097_port_pause_limit(struct mv88e6xxx_chip *chip, int port, u8 i= n, @@ -613,5 +629,9 @@ int mv88e6xxx_port_hidden_read(struct mv88e6xxx_chip *c= hip, int block, int port, int reg, u16 *val); =20 int mv88e6xxx_port_enable_tcam(struct mv88e6xxx_chip *chip, int port); +int mv88e6xxx_port_qav_write(struct mv88e6xxx_chip *chip, int port, int ad= dr, + u16 data); +int mv88e6xxx_port_set_scheduling_mode(struct mv88e6xxx_chip *chip, int po= rt, + u8 mode); =20 #endif /* _MV88E6XXX_PORT_H */ --=20 2.43.0