All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
To: cip-dev@lists.cip-project.org,
	Nobuhiro Iwamatsu <nobuhiro1.iwamatsu@toshiba.co.jp>,
	Pavel Machek <pavel@denx.de>
Cc: Biju Das <biju.das.jz@bp.renesas.com>
Subject: [PATCH 5.10.y-cip 50/61] ravb: Fillup ravb_rx_gbeth() stub
Date: Mon, 10 Jan 2022 12:23:20 +0000	[thread overview]
Message-ID: <20220110122331.24114-51-prabhakar.mahadev-lad.rj@bp.renesas.com> (raw)
In-Reply-To: <20220110122331.24114-1-prabhakar.mahadev-lad.rj@bp.renesas.com>

From: Biju Das <biju.das.jz@bp.renesas.com>

commit 1c59eb678cbd8d322d06d3a5514d36e8e1a4e84c upstream.

Fillup ravb_rx_gbeth() function to support RZ/G2L.

This patch also renames ravb_rcar_rx to ravb_rx_rcar to be
consistent with the naming convention used in sh_eth driver.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Reviewed-by: Sergey Shtylyov <s.shtylyov@omp.ru>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
 drivers/net/ethernet/renesas/ravb.h      |   1 +
 drivers/net/ethernet/renesas/ravb_main.c | 146 ++++++++++++++++++++++-
 2 files changed, 142 insertions(+), 5 deletions(-)

diff --git a/drivers/net/ethernet/renesas/ravb.h b/drivers/net/ethernet/renesas/ravb.h
index e9de3f8306ce..99164318db26 100644
--- a/drivers/net/ethernet/renesas/ravb.h
+++ b/drivers/net/ethernet/renesas/ravb.h
@@ -1043,6 +1043,7 @@ struct ravb_private {
 	struct ravb_ex_rx_desc *rx_ring[NUM_RX_QUEUE];
 	struct ravb_tx_desc *tx_ring[NUM_TX_QUEUE];
 	void *tx_align[NUM_TX_QUEUE];
+	struct sk_buff *rx_1st_skb;
 	struct sk_buff **rx_skb[NUM_RX_QUEUE];
 	struct sk_buff **tx_skb[NUM_TX_QUEUE];
 	u32 rx_over_errors;
diff --git a/drivers/net/ethernet/renesas/ravb_main.c b/drivers/net/ethernet/renesas/ravb_main.c
index 2412b570e1a1..2bfb903699cf 100644
--- a/drivers/net/ethernet/renesas/ravb_main.c
+++ b/drivers/net/ethernet/renesas/ravb_main.c
@@ -732,15 +732,151 @@ static void ravb_rx_csum(struct sk_buff *skb)
 	skb_trim(skb, skb->len - sizeof(__sum16));
 }
 
+static struct sk_buff *ravb_get_skb_gbeth(struct net_device *ndev, int entry,
+					  struct ravb_rx_desc *desc)
+{
+	struct ravb_private *priv = netdev_priv(ndev);
+	struct sk_buff *skb;
+
+	skb = priv->rx_skb[RAVB_BE][entry];
+	priv->rx_skb[RAVB_BE][entry] = NULL;
+	dma_unmap_single(ndev->dev.parent, le32_to_cpu(desc->dptr),
+			 ALIGN(GBETH_RX_BUFF_MAX, 16), DMA_FROM_DEVICE);
+
+	return skb;
+}
+
 /* Packet receive function for Gigabit Ethernet */
 static bool ravb_rx_gbeth(struct net_device *ndev, int *quota, int q)
 {
-	/* Place holder */
-	return true;
+	struct ravb_private *priv = netdev_priv(ndev);
+	const struct ravb_hw_info *info = priv->info;
+	struct net_device_stats *stats;
+	struct ravb_rx_desc *desc;
+	struct sk_buff *skb;
+	dma_addr_t dma_addr;
+	u8  desc_status;
+	int boguscnt;
+	u16 pkt_len;
+	u8  die_dt;
+	int entry;
+	int limit;
+
+	entry = priv->cur_rx[q] % priv->num_rx_ring[q];
+	boguscnt = priv->dirty_rx[q] + priv->num_rx_ring[q] - priv->cur_rx[q];
+	stats = &priv->stats[q];
+
+	boguscnt = min(boguscnt, *quota);
+	limit = boguscnt;
+	desc = &priv->gbeth_rx_ring[entry];
+	while (desc->die_dt != DT_FEMPTY) {
+		/* Descriptor type must be checked before all other reads */
+		dma_rmb();
+		desc_status = desc->msc;
+		pkt_len = le16_to_cpu(desc->ds_cc) & RX_DS;
+
+		if (--boguscnt < 0)
+			break;
+
+		/* We use 0-byte descriptors to mark the DMA mapping errors */
+		if (!pkt_len)
+			continue;
+
+		if (desc_status & MSC_MC)
+			stats->multicast++;
+
+		if (desc_status & (MSC_CRC | MSC_RFE | MSC_RTSF | MSC_RTLF | MSC_CEEF)) {
+			stats->rx_errors++;
+			if (desc_status & MSC_CRC)
+				stats->rx_crc_errors++;
+			if (desc_status & MSC_RFE)
+				stats->rx_frame_errors++;
+			if (desc_status & (MSC_RTLF | MSC_RTSF))
+				stats->rx_length_errors++;
+			if (desc_status & MSC_CEEF)
+				stats->rx_missed_errors++;
+		} else {
+			die_dt = desc->die_dt & 0xF0;
+			switch (die_dt) {
+			case DT_FSINGLE:
+				skb = ravb_get_skb_gbeth(ndev, entry, desc);
+				skb_put(skb, pkt_len);
+				skb->protocol = eth_type_trans(skb, ndev);
+				napi_gro_receive(&priv->napi[q], skb);
+				stats->rx_packets++;
+				stats->rx_bytes += pkt_len;
+				break;
+			case DT_FSTART:
+				priv->rx_1st_skb = ravb_get_skb_gbeth(ndev, entry, desc);
+				skb_put(priv->rx_1st_skb, pkt_len);
+				break;
+			case DT_FMID:
+				skb = ravb_get_skb_gbeth(ndev, entry, desc);
+				skb_copy_to_linear_data_offset(priv->rx_1st_skb,
+							       priv->rx_1st_skb->len,
+							       skb->data,
+							       pkt_len);
+				skb_put(priv->rx_1st_skb, pkt_len);
+				dev_kfree_skb(skb);
+				break;
+			case DT_FEND:
+				skb = ravb_get_skb_gbeth(ndev, entry, desc);
+				skb_copy_to_linear_data_offset(priv->rx_1st_skb,
+							       priv->rx_1st_skb->len,
+							       skb->data,
+							       pkt_len);
+				skb_put(priv->rx_1st_skb, pkt_len);
+				dev_kfree_skb(skb);
+				priv->rx_1st_skb->protocol =
+					eth_type_trans(priv->rx_1st_skb, ndev);
+				napi_gro_receive(&priv->napi[q],
+						 priv->rx_1st_skb);
+				stats->rx_packets++;
+				stats->rx_bytes += priv->rx_1st_skb->len;
+				break;
+			}
+		}
+
+		entry = (++priv->cur_rx[q]) % priv->num_rx_ring[q];
+		desc = &priv->gbeth_rx_ring[entry];
+	}
+
+	/* Refill the RX ring buffers. */
+	for (; priv->cur_rx[q] - priv->dirty_rx[q] > 0; priv->dirty_rx[q]++) {
+		entry = priv->dirty_rx[q] % priv->num_rx_ring[q];
+		desc = &priv->gbeth_rx_ring[entry];
+		desc->ds_cc = cpu_to_le16(GBETH_RX_DESC_DATA_SIZE);
+
+		if (!priv->rx_skb[q][entry]) {
+			skb = netdev_alloc_skb(ndev, info->max_rx_len);
+			if (!skb)
+				break;
+			ravb_set_buffer_align(skb);
+			dma_addr = dma_map_single(ndev->dev.parent,
+						  skb->data,
+						  GBETH_RX_BUFF_MAX,
+						  DMA_FROM_DEVICE);
+			skb_checksum_none_assert(skb);
+			/* We just set the data size to 0 for a failed mapping
+			 * which should prevent DMA  from happening...
+			 */
+			if (dma_mapping_error(ndev->dev.parent, dma_addr))
+				desc->ds_cc = cpu_to_le16(0);
+			desc->dptr = cpu_to_le32(dma_addr);
+			priv->rx_skb[q][entry] = skb;
+		}
+		/* Descriptor type must be set after all the above writes */
+		dma_wmb();
+		desc->die_dt = DT_FEMPTY;
+	}
+
+	*quota -= limit - (++boguscnt);
+
+	return boguscnt <= 0;
 }
 
 /* Packet receive function for Ethernet AVB */
-static bool ravb_rcar_rx(struct net_device *ndev, int *quota, int q)
+static bool ravb_rx_rcar(struct net_device *ndev, int *quota, int q)
 {
 	struct ravb_private *priv = netdev_priv(ndev);
 	const struct ravb_hw_info *info = priv->info;
@@ -2236,7 +2372,7 @@ static const struct ravb_hw_info ravb_gen3_hw_info = {
 	.rx_ring_free = ravb_rx_ring_free_rcar,
 	.rx_ring_format = ravb_rx_ring_format_rcar,
 	.alloc_rx_desc = ravb_alloc_rx_desc_rcar,
-	.receive = ravb_rcar_rx,
+	.receive = ravb_rx_rcar,
 	.set_rate = ravb_set_rate_rcar,
 	.set_feature = ravb_set_features_rcar,
 	.dmac_init = ravb_dmac_init_rcar,
@@ -2261,7 +2397,7 @@ static const struct ravb_hw_info ravb_gen2_hw_info = {
 	.rx_ring_free = ravb_rx_ring_free_rcar,
 	.rx_ring_format = ravb_rx_ring_format_rcar,
 	.alloc_rx_desc = ravb_alloc_rx_desc_rcar,
-	.receive = ravb_rcar_rx,
+	.receive = ravb_rx_rcar,
 	.set_rate = ravb_set_rate_rcar,
 	.set_feature = ravb_set_features_rcar,
 	.dmac_init = ravb_dmac_init_rcar,
-- 
2.17.1



  parent reply	other threads:[~2022-01-10 12:24 UTC|newest]

Thread overview: 64+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-10 12:22 [PATCH 5.10.y-cip 00/61] Add Ethernet support to RZ/G2L Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 01/61] dt-bindings: net: renesas,etheravb: Add additional clocks Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 02/61] dt-bindings: net: renesas,etheravb: Fix optional second clock name Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 03/61] dt-bindings: net: renesas,etheravb: Document Gigabit Ethernet IP Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 04/61] dt-bindings: net: renesas,etheravb: Drop "int_" prefix and "_n" suffix from interrupt names Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 05/61] net: ethernet: ravb: Enable optional refclk Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 06/61] net: ethernet: ravb: Fix release of refclk Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 07/61] net: ethernet: ravb: Use devm_platform_get_and_ioremap_resource() Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 08/61] ravb: Fix a typo in comment Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 09/61] ravb: Remove checks for unsupported internal delay modes Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 10/61] ravb: Use unsigned int for num_tx_desc variable in struct ravb_private Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 11/61] ravb: Add struct ravb_hw_info to driver data Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 12/61] ravb: Add aligned_tx to struct ravb_hw_info Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 13/61] ravb: Add max_rx_len " Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 14/61] ravb: Add stats_len " Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 15/61] ravb: Add gstrings_stats and gstrings_size " Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 16/61] ravb: Add net_features and net_hw_features " Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 17/61] ravb: Add internal delay hw feature " Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 18/61] ravb: Add tx_counters " Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 19/61] ravb: Remove the macros NUM_TX_DESC_GEN[23] Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 20/61] ravb: Add multi_irq to struct ravb_hw_info Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 21/61] ravb: Add no_ptp_cfg_active " Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 22/61] ravb: Add ptp_cfg_active " Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 23/61] ravb: Factorise ravb_ring_free function Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 24/61] ravb: Factorise ravb_ring_format function Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 25/61] ravb: Factorise ravb_ring_init function Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 26/61] ravb: Factorise ravb_rx function Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 27/61] ravb: Factorise ravb_adjust_link function Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 28/61] ravb: Factorise ravb_set_features Lad Prabhakar
2022-01-10 12:22 ` [PATCH 5.10.y-cip 29/61] ravb: Factorise ravb_dmac_init function Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 30/61] ravb: Factorise ravb_emac_init function Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 31/61] ravb: Add reset support Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 32/61] ravb: Rename "ravb_set_features_rx_csum" function to "ravb_set_features_rcar" Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 33/61] ravb: Rename "no_ptp_cfg_active" and "ptp_cfg_active" variables Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 34/61] ravb: Add nc_queue to struct ravb_hw_info Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 35/61] ravb: Add support for RZ/G2L SoC Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 36/61] ravb: Initialize GbEthernet DMAC Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 37/61] ravb: remove APSR_DM Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 38/61] ravb: Exclude gPTP feature support for RZ/G2L Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 39/61] ravb: Add tsrq to struct ravb_hw_info Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 40/61] ravb: Add magic_pkt " Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 41/61] ravb: Add half_duplex " Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 42/61] ravb: update "undocumented" annotations Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 43/61] ravb: Remove extra TAB Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 44/61] ravb: Initialize GbEthernet E-MAC Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 45/61] ravb: Add rx_max_buf_size to struct ravb_hw_info Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 46/61] ravb: Use ALIGN macro for max_rx_len Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 47/61] ravb: Fillup ravb_alloc_rx_desc_gbeth() stub Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 48/61] ravb: Fillup ravb_rx_ring_free_gbeth() stub Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 49/61] ravb: Fillup ravb_rx_ring_format_gbeth() stub Lad Prabhakar
2022-01-10 12:23 ` Lad Prabhakar [this message]
2022-01-10 12:23 ` [PATCH 5.10.y-cip 51/61] ravb: Add carrier_counters to struct ravb_hw_info Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 52/61] ravb: Add support to retrieve stats for GbEthernet Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 53/61] ravb: Rename "tsrq" variable Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 54/61] ravb: Optimize ravb_emac_init_gbeth function Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 55/61] ravb: Rename "nc_queue" feature bit Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 56/61] ravb: Update ravb_emac_init_gbeth() Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 57/61] ravb: Fix typo AVB->DMAC Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 58/61] clk: renesas: r9a07g044: Add ethernet clock sources Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 59/61] clk: renesas: r9a07g044: Add GbEthernet clock/reset Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 60/61] arm64: dts: renesas: r9a07g044: Add GbEthernet nodes Lad Prabhakar
2022-01-10 12:23 ` [PATCH 5.10.y-cip 61/61] arm64: dts: renesas: rzg2l-smarc-som: Enable Ethernet Lad Prabhakar
2022-01-11 11:54 ` [cip-dev] [PATCH 5.10.y-cip 00/61] Add Ethernet support to RZ/G2L Pavel Machek
     [not found] ` <16C93537A1693B9B.28404@lists.cip-project.org>
2022-01-12  9:45   ` Pavel Machek

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220110122331.24114-51-prabhakar.mahadev-lad.rj@bp.renesas.com \
    --to=prabhakar.mahadev-lad.rj@bp.renesas.com \
    --cc=biju.das.jz@bp.renesas.com \
    --cc=cip-dev@lists.cip-project.org \
    --cc=nobuhiro1.iwamatsu@toshiba.co.jp \
    --cc=pavel@denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.