From patchwork Fri Apr 16 17:11:22 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Claudiu Manoil X-Patchwork-Id: 12208317 X-Patchwork-Delegate: kuba@kernel.org Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-18.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id AF1A0C433ED for ; Fri, 16 Apr 2021 17:14:50 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 9225C611BF for ; Fri, 16 Apr 2021 17:14:50 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240420AbhDPRPO (ORCPT ); Fri, 16 Apr 2021 13:15:14 -0400 Received: from mail-eopbgr20054.outbound.protection.outlook.com ([40.107.2.54]:37093 "EHLO EUR02-VE1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S235454AbhDPRPL (ORCPT ); Fri, 16 Apr 2021 13:15:11 -0400 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=f4rzJYhI0+p6xf74s6tf3nrwoaFTcByu7gaHGhW5jdrwihlx43CC+3HcWBGIFI4ItdZ206uFgjWbJuyAK807I1lBaCtB5OBclCZ7wdOdp+Nu38A7HYWRngD4qPUEBEUAtzg7AmwJpQ0y6JJgq/mf4gTiwiu1qz75zsPKvMP6V8serrsqXeGbmpVn4wNMAQmtHwcVFR1j9J9alRiHUXFVO0l8lBCzAPs73QrcRkKpsf8GiX3vD6NmBO5Hy4yDLSPH6ZaB0y4ghGz0iBvTovoB0qi8t6N+d0ATk6bAGPMhVD7hvC/2vbqLnbRy6WgCdvX2TCH9smhYSvO9Dqz2cKsvfw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=8hndEXxodrCTq/ecORFjU0inRoVLUKM7pN5aG3ilUM8=; b=OEGoMv+yeF+gbrCj1sHwgMC4YAMS15kt+laHGfg4ANZPouoVkw+CEHuSlkrxz2BVRiS1oz/G0U9AKKQpa775l0APakby82SzYrVOrLfovx3ccwg5BSuK+zya7EQ3Av6cix5ByAqGd/RtfERiKX3PdtdfiZjUATyhR+7zpO9IS0IlFvjz1LnjZ/6VsQqeq79atO/LFe9MOKpQr0sIHHX70i1CDShLx4pr7x9KZp9oUFx/tW2JDUC28EQxbimCFt9HIqEHBczaKswpNeX2OHvlpZpNrSLe+36ypHF8OwQXraooOaGropMOBgUE7Pkl3i+QQQ0JgKm15uBBbU4e39EIrw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=8hndEXxodrCTq/ecORFjU0inRoVLUKM7pN5aG3ilUM8=; b=khgurf3/cgPdIBhI3xlKtcD1aHJ5xR28t81rYiFRLThBQEQIV9hv/jTT4gGRCIAdO6jY5ZD98W/6j+kZT2PmhhWC2vUwxA6HZe9++2Uztop9Ev6cr1QsxhHWZsJckCymxMm7HI4pWeADd1qMDrVK2Lcr5S6EFWq/9qa9mP5Y7EY= Authentication-Results: vger.kernel.org; dkim=none (message not signed) header.d=none;vger.kernel.org; dmarc=none action=none header.from=nxp.com; Received: from AM0PR04MB6754.eurprd04.prod.outlook.com (2603:10a6:208:170::28) by AM0PR04MB5140.eurprd04.prod.outlook.com (2603:10a6:208:ca::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4042.19; Fri, 16 Apr 2021 17:14:45 +0000 Received: from AM0PR04MB6754.eurprd04.prod.outlook.com ([fe80::f12c:54bd:ccfc:819a]) by AM0PR04MB6754.eurprd04.prod.outlook.com ([fe80::f12c:54bd:ccfc:819a%7]) with mapi id 15.20.4042.018; Fri, 16 Apr 2021 17:14:44 +0000 From: Claudiu Manoil To: netdev@vger.kernel.org Cc: Jakub Kicinski , "David S . Miller" , Vladimir Oltean , devicetree@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, Rob Herring , Benjamin Herrenschmidt Subject: [PATCH net-next 1/2] gianfar: Drop GFAR_MQ_POLLING support Date: Fri, 16 Apr 2021 20:11:22 +0300 Message-Id: <20210416171123.22969-2-claudiu.manoil@nxp.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20210416171123.22969-1-claudiu.manoil@nxp.com> References: <20210416171123.22969-1-claudiu.manoil@nxp.com> X-Originating-IP: [83.217.231.2] X-ClientProxiedBy: AM0PR05CA0088.eurprd05.prod.outlook.com (2603:10a6:208:136::28) To AM0PR04MB6754.eurprd04.prod.outlook.com (2603:10a6:208:170::28) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from lsv15141.swis.ro-buh01.nxp.com (83.217.231.2) by AM0PR05CA0088.eurprd05.prod.outlook.com (2603:10a6:208:136::28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4042.19 via Frontend Transport; Fri, 16 Apr 2021 17:14:44 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 4b1c287d-fc85-4ae4-5765-08d900fb2169 X-MS-TrafficTypeDiagnostic: AM0PR04MB5140: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:9508; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 1bN2kLiF0fRQ0vu8rqyEkVUikxRDfQoUAY5e8kB7nO8pgil5gCEyu0wTVvzhLr70e2AdFmjkyTmD7FOoWrxs+jIe1lWSQwTwI94Vj4OUIxGFgM0Jk3BmSEHN5uitr4bquwq2fhgmoRUAyMiUip1PetPrq07WGqRyecPk6cSSsZ2mCEpnPiHOhIThYmp7qL0YhfQM1Lv4qEjXmCQUGpGU6fEIxoUj43Io9s9/W38VKEYn95VHzGzNw1EJlvf1nTwA+Aid4tlqW0xoPWCmMC10KhQEEWV6lBV5CXzaTuhK8epWIoM6+/5YLwpM+vlAZAMhCeFHtXFwZxiJ4VXeEmRUIfk8FPJ/kqT/jYwrcTDC2bVsnGxug9Nz3HD6nir8rQaPT5Tz2xZ1+ZpAueYJvVYSU6kqFtrQ/0GHpmTe1IFNfxmcEXI3QLg3t+dQSrw26/VnVMPgOfdgUWTnq5s6Lg7yb+D2VZ+yfsMITUnrawUT5lcVYz8yiHqubsoZ0FELaGUNYebXZMO0TLDW8Uam7fX4xMtN+52z1Y5pfsp30clJphYjYVrEB+ArDvf6W9P2ILJw3aKrZrS2szuboiEXtrn8xJnxS7QOuN+wJzg5LcO7cIMSF8OKKm8FJHsJw3UOUg/39u7MeNLjVN0j5EvHxB1VuwSGXXOKi1Ian5FNpsuc0yE= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR04MB6754.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(396003)(136003)(376002)(346002)(39860400002)(366004)(66556008)(8676002)(38100700002)(38350700002)(2906002)(16526019)(4326008)(8936002)(5660300002)(36756003)(52116002)(86362001)(7696005)(66946007)(66476007)(6916009)(316002)(54906003)(186003)(1076003)(26005)(6666004)(6486002)(83380400001)(478600001)(956004)(2616005)(44832011);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData: 6U72A89bJGfP94MD6auFP+MGS5b+/JqZlpbiMCijVsKRDFUIUn8H3JZDfYHUPrksIHp6CoPPJIi0Ieq5InZfYN4BzixTx9xmZuuBgZnL47C7sGp3aBf805kYc4RSaXFA0ByoWx+ZFl275ipLTZ9yWxtWA/zOoE/tpcauv94xq5WmpfyhQPztcXnmHNVrG2Y+R6wTB2KpHl3q1nnkufzGmsabuU8HQcGW+sGwxmGYqMDDPuqD+0Ba/bDkdNEmudhmVwhBVEC3Zjxdo7waschk2KhMfgt14PLcFacMT0mwwZrDk4NwMpIicHeYbCaZ5niVgxZqpzQmxVuPCB1HMs1q7D4qKsYsiATvavat7ff37pAxer+E6dcKZbo3zDqdtFWTBZelsJFaYlOCuPj0eSznTMcrFrNmCs1I8w7yEeohqKuFKr3GocqWdl2uwIILO9rpQyCGA1t9UiUGPU+/0URKyx4CUICU3MLRwss4WsNV5pwrTEbuhol4+Rp7/65aeAE8QNmfsejmOVCymw0NTdFoEoNZ0tfvLvx0CcoDpBPclFzILYHVAqqLAUXDiQwKBYs8CWbNHKd7jkjmNdo9n35YDj/SeXLa9deb6d85UdIWWsVowrKsYtHjBuUBdJJLnMWJmXD8fD066qHMNFHrDjF9cLJrHn8EeIq9XM7hBA0ploHzw/YVfxj/llkpmaFWmjkZlY1/fQZSzKk2zcNNf0zvEsa5kRpIcV7P6jcqtHesCMbRNU4A8nvBjp7lwsG/4RzVj7UE4E6Kv36qVF3lkezi6xVGH+Z9Y1EW95bQshZxoAYhb1NiGDzt4PrN10weS5Cho0C0E+6TeCrSdqG9zCbVhcl3vtjkFbCBJDQZZQbNjj81LF9P/mpHtfJk8LQsbVjgadqJHRhHuVWTXSOZBpkZI4shSKHmQb/R0MFY32Z38AmgymJwiKAOChuWgaLcP3eVCnkdl/yPyI72H4Wc4yD9ScAznbR4hI8pkA77blPtxYYoQsGJRiPi6nVisrEftHEMyjNtGO70CecKTOiQpULbUf5SOsXzX8LvtY2LSfi6wGNJ8rBeZf/Or2u+UX83dsPDSfD9l60N/hf9Cc+wo1sCuVJUpZcYaiVkRzCoHItkMNMqzqsP0XFBn8wUVBpnCfwMUBUb8wM/rUwFVOVTwRSHrUGWweBYhMTdCBnQPRKeF6jIuDG3zZI4cPLBtHjqqps90d1RxH1P5lvDYvEOPnfRtwXaJ+ZnF0YrMYxzv68fo5ZUGLPKYhMZ9HSdPZgT6Ye3EhPa6DczVXhwVUwn9AqjM87IYLD8fCHyR8kFs5CFNiqoUMk4NdJEetQ96VS5MFDw X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 4b1c287d-fc85-4ae4-5765-08d900fb2169 X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB6754.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Apr 2021 17:14:44.8977 (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: jmxSdY6/lyt9L2K2L/00yQncIYAvr+Z1aKgtuOnmgHJ3YIN4YnOU/+B/3FAxzeYxd2CUYSEOgPimXRF9gYUbWA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR04MB5140 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org X-Patchwork-Delegate: kuba@kernel.org Gianfar used to enable all 8 Rx queues (DMA rings) per ethernet device, even though the controller can only support 2 interrupt lines at most. This meant that multiple Rx queues would have to be grouped per NAPI poll routine, and the CPU would have to split the budget and service them in a round robin manner. The overhead of this scheme proved to outweight the potential benefits. The alternative was to introduce the "Single Queue" polling mode, supporting one Rx queue per NAPI, which became the default packet processing option and helped improve the performance of the driver. MQ_POLLING also relies on undocumeted device tree properties to specify how to map the 8 Rx and Tx queues to a given interrupt line (aka "interrupt group"). Using module parameters to enable this mode wasn't an option either. Long story short, MQ_POLLING became obsolete, now it is just dead code, and no one asked for it so far. For the Tx queues, multi-queue support (more than 1 Tx queue per CPU) could be revisited by adding tc MQPRIO support, but again, one has to consider that there are only 2 interrupt lines. So the NAPI poll routine would have to service multiple Tx rings. Signed-off-by: Claudiu Manoil --- drivers/net/ethernet/freescale/gianfar.c | 170 ++--------------------- drivers/net/ethernet/freescale/gianfar.h | 17 --- 2 files changed, 11 insertions(+), 176 deletions(-) diff --git a/drivers/net/ethernet/freescale/gianfar.c b/drivers/net/ethernet/freescale/gianfar.c index 3ec4d9fddd52..4e4c62d4061e 100644 --- a/drivers/net/ethernet/freescale/gianfar.c +++ b/drivers/net/ethernet/freescale/gianfar.c @@ -175,10 +175,7 @@ static void gfar_mac_rx_config(struct gfar_private *priv) if (priv->rx_filer_enable) { rctrl |= RCTRL_FILREN | RCTRL_PRSDEP_INIT; /* Program the RIR0 reg with the required distribution */ - if (priv->poll_mode == GFAR_SQ_POLLING) - gfar_write(®s->rir0, DEFAULT_2RXQ_RIR0); - else /* GFAR_MQ_POLLING */ - gfar_write(®s->rir0, DEFAULT_8RXQ_RIR0); + gfar_write(®s->rir0, DEFAULT_2RXQ_RIR0); } /* Restore PROMISC mode */ @@ -521,29 +518,9 @@ static int gfar_parse_group(struct device_node *np, grp->priv = priv; spin_lock_init(&grp->grplock); if (priv->mode == MQ_MG_MODE) { - u32 rxq_mask, txq_mask; - int ret; - + /* One Q per interrupt group: Q0 to G0, Q1 to G1 */ grp->rx_bit_map = (DEFAULT_MAPPING >> priv->num_grps); grp->tx_bit_map = (DEFAULT_MAPPING >> priv->num_grps); - - ret = of_property_read_u32(np, "fsl,rx-bit-map", &rxq_mask); - if (!ret) { - grp->rx_bit_map = rxq_mask ? - rxq_mask : (DEFAULT_MAPPING >> priv->num_grps); - } - - ret = of_property_read_u32(np, "fsl,tx-bit-map", &txq_mask); - if (!ret) { - grp->tx_bit_map = txq_mask ? - txq_mask : (DEFAULT_MAPPING >> priv->num_grps); - } - - if (priv->poll_mode == GFAR_SQ_POLLING) { - /* One Q per interrupt group: Q0 to G0, Q1 to G1 */ - grp->rx_bit_map = (DEFAULT_MAPPING >> priv->num_grps); - grp->tx_bit_map = (DEFAULT_MAPPING >> priv->num_grps); - } } else { grp->rx_bit_map = 0xFF; grp->tx_bit_map = 0xFF; @@ -650,18 +627,15 @@ static int gfar_of_init(struct platform_device *ofdev, struct net_device **pdev) u32 stash_len = 0; u32 stash_idx = 0; unsigned int num_tx_qs, num_rx_qs; - unsigned short mode, poll_mode; + unsigned short mode; if (!np) return -ENODEV; - if (of_device_is_compatible(np, "fsl,etsec2")) { + if (of_device_is_compatible(np, "fsl,etsec2")) mode = MQ_MG_MODE; - poll_mode = GFAR_SQ_POLLING; - } else { + else mode = SQ_SG_MODE; - poll_mode = GFAR_SQ_POLLING; - } if (mode == SQ_SG_MODE) { num_tx_qs = 1; @@ -677,22 +651,8 @@ static int gfar_of_init(struct platform_device *ofdev, struct net_device **pdev) return -EINVAL; } - if (poll_mode == GFAR_SQ_POLLING) { - num_tx_qs = num_grps; /* one txq per int group */ - num_rx_qs = num_grps; /* one rxq per int group */ - } else { /* GFAR_MQ_POLLING */ - u32 tx_queues, rx_queues; - int ret; - - /* parse the num of HW tx and rx queues */ - ret = of_property_read_u32(np, "fsl,num_tx_queues", - &tx_queues); - num_tx_qs = ret ? 1 : tx_queues; - - ret = of_property_read_u32(np, "fsl,num_rx_queues", - &rx_queues); - num_rx_qs = ret ? 1 : rx_queues; - } + num_tx_qs = num_grps; /* one txq per int group */ + num_rx_qs = num_grps; /* one rxq per int group */ } if (num_tx_qs > MAX_TX_QS) { @@ -718,7 +678,6 @@ static int gfar_of_init(struct platform_device *ofdev, struct net_device **pdev) priv->ndev = dev; priv->mode = mode; - priv->poll_mode = poll_mode; priv->num_tx_queues = num_tx_qs; netif_set_real_num_rx_queues(dev, num_rx_qs); @@ -2695,106 +2654,6 @@ static int gfar_poll_tx_sq(struct napi_struct *napi, int budget) return 0; } -static int gfar_poll_rx(struct napi_struct *napi, int budget) -{ - struct gfar_priv_grp *gfargrp = - container_of(napi, struct gfar_priv_grp, napi_rx); - struct gfar_private *priv = gfargrp->priv; - struct gfar __iomem *regs = gfargrp->regs; - struct gfar_priv_rx_q *rx_queue = NULL; - int work_done = 0, work_done_per_q = 0; - int i, budget_per_q = 0; - unsigned long rstat_rxf; - int num_act_queues; - - /* Clear IEVENT, so interrupts aren't called again - * because of the packets that have already arrived - */ - gfar_write(®s->ievent, IEVENT_RX_MASK); - - rstat_rxf = gfar_read(®s->rstat) & RSTAT_RXF_MASK; - - num_act_queues = bitmap_weight(&rstat_rxf, MAX_RX_QS); - if (num_act_queues) - budget_per_q = budget/num_act_queues; - - for_each_set_bit(i, &gfargrp->rx_bit_map, priv->num_rx_queues) { - /* skip queue if not active */ - if (!(rstat_rxf & (RSTAT_CLEAR_RXF0 >> i))) - continue; - - rx_queue = priv->rx_queue[i]; - work_done_per_q = - gfar_clean_rx_ring(rx_queue, budget_per_q); - work_done += work_done_per_q; - - /* finished processing this queue */ - if (work_done_per_q < budget_per_q) { - /* clear active queue hw indication */ - gfar_write(®s->rstat, - RSTAT_CLEAR_RXF0 >> i); - num_act_queues--; - - if (!num_act_queues) - break; - } - } - - if (!num_act_queues) { - u32 imask; - napi_complete_done(napi, work_done); - - /* Clear the halt bit in RSTAT */ - gfar_write(®s->rstat, gfargrp->rstat); - - spin_lock_irq(&gfargrp->grplock); - imask = gfar_read(®s->imask); - imask |= IMASK_RX_DEFAULT; - gfar_write(®s->imask, imask); - spin_unlock_irq(&gfargrp->grplock); - } - - return work_done; -} - -static int gfar_poll_tx(struct napi_struct *napi, int budget) -{ - struct gfar_priv_grp *gfargrp = - container_of(napi, struct gfar_priv_grp, napi_tx); - struct gfar_private *priv = gfargrp->priv; - struct gfar __iomem *regs = gfargrp->regs; - struct gfar_priv_tx_q *tx_queue = NULL; - int has_tx_work = 0; - int i; - - /* Clear IEVENT, so interrupts aren't called again - * because of the packets that have already arrived - */ - gfar_write(®s->ievent, IEVENT_TX_MASK); - - for_each_set_bit(i, &gfargrp->tx_bit_map, priv->num_tx_queues) { - tx_queue = priv->tx_queue[i]; - /* run Tx cleanup to completion */ - if (tx_queue->tx_skbuff[tx_queue->skb_dirtytx]) { - gfar_clean_tx_ring(tx_queue); - has_tx_work = 1; - } - } - - if (!has_tx_work) { - u32 imask; - napi_complete(napi); - - spin_lock_irq(&gfargrp->grplock); - imask = gfar_read(®s->imask); - imask |= IMASK_TX_DEFAULT; - gfar_write(®s->imask, imask); - spin_unlock_irq(&gfargrp->grplock); - } - - return 0; -} - /* GFAR error interrupt handler */ static irqreturn_t gfar_error(int irq, void *grp_id) { @@ -3352,17 +3211,10 @@ static int gfar_probe(struct platform_device *ofdev) /* Register for napi ...We are registering NAPI for each grp */ for (i = 0; i < priv->num_grps; i++) { - if (priv->poll_mode == GFAR_SQ_POLLING) { - netif_napi_add(dev, &priv->gfargrp[i].napi_rx, - gfar_poll_rx_sq, GFAR_DEV_WEIGHT); - netif_tx_napi_add(dev, &priv->gfargrp[i].napi_tx, - gfar_poll_tx_sq, 2); - } else { - netif_napi_add(dev, &priv->gfargrp[i].napi_rx, - gfar_poll_rx, GFAR_DEV_WEIGHT); - netif_tx_napi_add(dev, &priv->gfargrp[i].napi_tx, - gfar_poll_tx, 2); - } + netif_napi_add(dev, &priv->gfargrp[i].napi_rx, + gfar_poll_rx_sq, GFAR_DEV_WEIGHT); + netif_tx_napi_add(dev, &priv->gfargrp[i].napi_tx, + gfar_poll_tx_sq, 2); } if (priv->device_flags & FSL_GIANFAR_DEV_HAS_CSUM) { diff --git a/drivers/net/ethernet/freescale/gianfar.h b/drivers/net/ethernet/freescale/gianfar.h index 8ced783f5302..5ea47df93e5e 100644 --- a/drivers/net/ethernet/freescale/gianfar.h +++ b/drivers/net/ethernet/freescale/gianfar.h @@ -909,22 +909,6 @@ enum { MQ_MG_MODE }; -/* GFAR_SQ_POLLING: Single Queue NAPI polling mode - * The driver supports a single pair of RX/Tx queues - * per interrupt group (Rx/Tx int line). MQ_MG mode - * devices have 2 interrupt groups, so the device will - * have a total of 2 Tx and 2 Rx queues in this case. - * GFAR_MQ_POLLING: Multi Queue NAPI polling mode - * The driver supports all the 8 Rx and Tx HW queues - * each queue mapped by the Device Tree to one of - * the 2 interrupt groups. This mode implies significant - * processing overhead (CPU and controller level). - */ -enum gfar_poll_mode { - GFAR_SQ_POLLING = 0, - GFAR_MQ_POLLING -}; - /* * Per TX queue stats */ @@ -1105,7 +1089,6 @@ struct gfar_private { unsigned long state; unsigned short mode; - unsigned short poll_mode; unsigned int num_tx_queues; unsigned int num_rx_queues; unsigned int num_grps; From patchwork Fri Apr 16 17:11:23 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Claudiu Manoil X-Patchwork-Id: 12208319 X-Patchwork-Delegate: kuba@kernel.org Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-18.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 644EBC43461 for ; Fri, 16 Apr 2021 17:14:52 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 449B2611AE for ; Fri, 16 Apr 2021 17:14:52 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240521AbhDPRPQ (ORCPT ); Fri, 16 Apr 2021 13:15:16 -0400 Received: from mail-eopbgr20054.outbound.protection.outlook.com ([40.107.2.54]:37093 "EHLO EUR02-VE1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S235011AbhDPRPN (ORCPT ); Fri, 16 Apr 2021 13:15:13 -0400 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=lwfSV2m6lvAgdmstw+DLPqFuhnq9JulMxz831XzXpzcrC7NBb2hO2Y8k76h4w/oZZkstnJU4HhEgYwHuVR8OE+El/u1/ZUGBj6kNCiOIE1vErkKbBWqIOEWe0XFtyFKBP8Ei85/4PaPoH4xYIkTvDQXrCVy0SKl1Xv4T4LSAZEXlg2fH68c+P+usS2U00HV0wD8QGEvKQKi1gmq0tM+1RIx4VwDOxEk3l60Ps8gIoyd+sp/FHU73o8TEgbP6QrqePPgA69C/Pq4zsiX1Nwyu2zUFj40UhdIkF+0XK1Mtz7VD8yufoobrkp0DqjLssNndmfMqqLX13Tgn6A2/k8tgvg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=hKgtzMScuRRo/afCfPWFSq/xDYXgqWEI9EOiZEP4AaA=; b=lp/AYKnHkjq7DhkPnhgx9gAwpMVAFSIrebkw6f0QS95wQwTI8zazns2cQQ72KvkSAM0G98LRAL2WOROfmj3TSsZjhXVjeCEgd8cNasXuqXbRGwU4fgLTivII0nXrF854sVzMBdg39rNZOvhC19c67QiXlI4Y5Y0lsFYRHnqaeY8iC9KxHEJf7HxUEd0zchwJroJkHMZcAX0s+/e1mhtx3KPGpcuhEBX832F8MM5KPfvt5jTB7WzlfFn1VE1syFB5uk/DnFcDe607lukeTWu7JlYA9MnhmoalfdnNUi9Bv7gheSb9Sk8FbjTK1wLeBi9+YmzpfjWtr7BfcD8AZOWNhg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=hKgtzMScuRRo/afCfPWFSq/xDYXgqWEI9EOiZEP4AaA=; b=ZEnbuY3+hE9gX7jtH0pST7kHNwwrOiQqu9gvXAUTxYUBBvw74DxOJpAFSiueJQs+FyX/OuzbVXNmhuNNSq6BiNL5i55XrrchWx4lygD+7y76tLTE+Je5veK2ZZhLYdHnTxyMhinbk7o4hjb70v8OIQmuvYuMIJJ/Jw1U4lQaTBQ= Authentication-Results: vger.kernel.org; dkim=none (message not signed) header.d=none;vger.kernel.org; dmarc=none action=none header.from=nxp.com; Received: from AM0PR04MB6754.eurprd04.prod.outlook.com (2603:10a6:208:170::28) by AM0PR04MB5140.eurprd04.prod.outlook.com (2603:10a6:208:ca::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4042.19; Fri, 16 Apr 2021 17:14:45 +0000 Received: from AM0PR04MB6754.eurprd04.prod.outlook.com ([fe80::f12c:54bd:ccfc:819a]) by AM0PR04MB6754.eurprd04.prod.outlook.com ([fe80::f12c:54bd:ccfc:819a%7]) with mapi id 15.20.4042.018; Fri, 16 Apr 2021 17:14:45 +0000 From: Claudiu Manoil To: netdev@vger.kernel.org Cc: Jakub Kicinski , "David S . Miller" , Vladimir Oltean , devicetree@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, Rob Herring , Benjamin Herrenschmidt Subject: [PATCH net-next 2/2] powerpc: dts: fsl: Drop obsolete fsl,rx-bit-map and fsl,tx-bit-map properties Date: Fri, 16 Apr 2021 20:11:23 +0300 Message-Id: <20210416171123.22969-3-claudiu.manoil@nxp.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20210416171123.22969-1-claudiu.manoil@nxp.com> References: <20210416171123.22969-1-claudiu.manoil@nxp.com> X-Originating-IP: [83.217.231.2] X-ClientProxiedBy: AM0PR05CA0088.eurprd05.prod.outlook.com (2603:10a6:208:136::28) To AM0PR04MB6754.eurprd04.prod.outlook.com (2603:10a6:208:170::28) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from lsv15141.swis.ro-buh01.nxp.com (83.217.231.2) by AM0PR05CA0088.eurprd05.prod.outlook.com (2603:10a6:208:136::28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4042.19 via Frontend Transport; Fri, 16 Apr 2021 17:14:45 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 0db8c0de-c84d-4dfd-e1fc-08d900fb21eb X-MS-TrafficTypeDiagnostic: AM0PR04MB5140: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:530; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: GLWMe0fbyW4dvJ8FQv+Z+EN+bspHsbxg/5kifZPLAaFQhVlI9AEXFq30TGzxt4ibyE67dpCjx39PC84uwbe83uHZXdJdGEso+zg5jpNO19BJMXcvL61wtQI4vgqPkwcG9mEgFtBjR+pqFslzeHsS9CLW2rSpw1yin8/7knFnNQWXdcfYIkzx7ginnFXZiUz6ZewBDM87h9juygatMogDAGV4WOqCgoPRVNEazCoMOC5s1N0ujnNufLaJ9XsbnB94f6qTE7B/9o9UaDx+QAqsDNmgADMgoqqFM3b+HhxiyQlVntyuuumPDs3DN3uvisoM5q6iOh5zHa3XXpYozsKJjrfPDUrUv60t7IO4Po+i00HU43NVuA9Xxjq4kk5VGz5CJ3SoIxa6Ab6AZIpOd2ZYu/HqwW+4HDA2C06R3POjC5ppLllSjntmIwBOR5cShDDDAaMA0itD/nXQczoWNlNzH2S/CvTMQr/+3vKA8qZExjtM3zRR83BQzWz8+IcfAtwyvpauR7E2S1bJBnuHy4R0fwqOVUdpYAE4MfIiMQTmABhiSzTrFKsDRi/D5QQOHN/QqQf1YNn8ZevlCoPisKg6aJ7M8m11oQvFYg/God1VYHpBNKc05KgKecSmQNxLz6bWE2PNmhUwmvYdmdXBXlodAcEYMzfryBxYtvIQVXKhRcg= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM0PR04MB6754.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(396003)(136003)(376002)(346002)(39860400002)(366004)(66556008)(8676002)(38100700002)(38350700002)(2906002)(16526019)(4326008)(8936002)(5660300002)(36756003)(52116002)(86362001)(7696005)(66946007)(66476007)(6916009)(316002)(54906003)(186003)(1076003)(26005)(6666004)(6486002)(83380400001)(478600001)(956004)(2616005)(44832011);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData: +1znHluxBFUPiMoCiS9+YAc7wtMW2gug8TbktsuW6ePZEpjcBxm2DKNE1qvvBBnRo7Ok/1zD3bSMgoptNPd/Z4jO5/e/SdHm/GuaMrjBF8VTQI/mcD2LFrE4RPzSgeFQaUqbY2WDhsnixNI6LptsRG3uLmbQDETVJXbfBMyjyK41WXhccB+owJ4Hx0S6bq5O44EFn8HhmYjTze9NDPuCI0ehdHYR8E19SOboG8m89NdrP5RI0h7+GSR+WM2BjesRbd/j3KzAsnpAUQobHGweWMCRIA3tDe6cFMXDl2cOI4ZsRNjIXuVFIGyAOJZUe5H88XCF6gbTyaPJ29kQOEkX7iTJZmZrkXu2yQ5r18YZTyFZ1PjSv3F3xAfDM4BS1sg9GUgnRd8ueZpaLzYmzI2ZHss9opo8GeEOGQRJwySx/BjcH0boDRIZP+XCmADRfr+2SRUq8th+gWnlq8V1uCtFLsF9o4uDW1+dw+HzLLNsgP099xr/F6Mvyv4bmtEP19gvqzK8PTpDUswJIK/G1X9GZA/9Lja0uuJT369KbAo+OW/UgZ70aWDGPWej5yLbvuFty9g4yHvuWzOAiEgxyE9lTNrqDxCbumx1mke50TDZWz9GfhIJE+alcFp0vLd8M4XTNgUzPQyTPh1opZF56BbLZdgWlLNqYdsV70i18/gidzt+6LmIJUoVyQyiguyPHtE/JXxPtwea5xHp/RDoVaZQWqspxDjlaUn+WqPisESvWRdEv3PHkJG4V8PAqZ1k0RTnVoE9v5N+ADkud3VCp5aPCGC4jWXm6YftPoovoawhOFkT3XEYKTL9EEQXAOeDQAtKbhNe4/ueZdlRSk97VKcJID2tFbsXQKALPc4T1SQnmljop/pjd9EEdlqiTnGXRZKipbVYIvAOipARXzkvYo4pmD1+Lhql5JVbHizRenGqRpGAVKVDBv9n8mp3609KOp1P2L26Mpzx/siDLEFCnhhvX/E2ql6m5pqFR2XtkX0HP3sAQXRp4X9qjpRsE7iGu//Yo8T0qI6QeRSLJFs6x/vfL/xaQ0wBRnvA2v4CJB4E7jqoe4L4+UNMXQ1ZpCr4RmhdeZFTCC90o/AmulCsQexJI72plMTgSmzkEkmWUGg42mQDitdDG0AxHhHTDQi4imcayRw6ipyA9szEF5aT1KpO4x3j22uyy7DfIcGyGlr6vHTQjeW00TxFTAWsj0xwdjdn4svNvIHUZiQeoQSQDopwD/rr7nl2T2ETIpU8textFz3FNWOnVxxUGQ1+9YhTbDCj8VAxDrob80T7WN3Icuq0JteVz9lNHwJoO7myqTZHljKMCQkF2aJMg1yBDKCdFY/t X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 0db8c0de-c84d-4dfd-e1fc-08d900fb21eb X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB6754.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Apr 2021 17:14:45.7183 (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: /HrzVmz1eyUfdMEGrb9cE/tGsDebCL+GvRlZ8A/DpJc8YeaGefgR4OW9VLDQyY2S0bKLTxfz2DLpnADy0hVzHQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR04MB5140 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org X-Patchwork-Delegate: kuba@kernel.org These are very old properties that were used by the "gianfar" ethernet driver. They don't have documented bindings and are obsolete. Signed-off-by: Claudiu Manoil --- arch/powerpc/boot/dts/fsl/bsc9131si-post.dtsi | 4 ---- arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi | 4 ---- arch/powerpc/boot/dts/fsl/c293si-post.dtsi | 4 ---- arch/powerpc/boot/dts/fsl/p1010si-post.dtsi | 21 ------------------- 4 files changed, 33 deletions(-) diff --git a/arch/powerpc/boot/dts/fsl/bsc9131si-post.dtsi b/arch/powerpc/boot/dts/fsl/bsc9131si-post.dtsi index 0c0efa94cfb4..2a677fd323eb 100644 --- a/arch/powerpc/boot/dts/fsl/bsc9131si-post.dtsi +++ b/arch/powerpc/boot/dts/fsl/bsc9131si-post.dtsi @@ -170,8 +170,6 @@ timer@41100 { /include/ "pq3-etsec2-0.dtsi" enet0: ethernet@b0000 { queue-group@b0000 { - fsl,rx-bit-map = <0xff>; - fsl,tx-bit-map = <0xff>; interrupts = <26 2 0 0 27 2 0 0 28 2 0 0>; }; }; @@ -179,8 +177,6 @@ queue-group@b0000 { /include/ "pq3-etsec2-1.dtsi" enet1: ethernet@b1000 { queue-group@b1000 { - fsl,rx-bit-map = <0xff>; - fsl,tx-bit-map = <0xff>; interrupts = <33 2 0 0 34 2 0 0 35 2 0 0>; }; }; diff --git a/arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi b/arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi index b5f071574e83..b8e0edd1ac69 100644 --- a/arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi +++ b/arch/powerpc/boot/dts/fsl/bsc9132si-post.dtsi @@ -190,8 +190,6 @@ sec_jr3: jr@4000 { /include/ "pq3-etsec2-0.dtsi" enet0: ethernet@b0000 { queue-group@b0000 { - fsl,rx-bit-map = <0xff>; - fsl,tx-bit-map = <0xff>; interrupts = <26 2 0 0 27 2 0 0 28 2 0 0>; }; }; @@ -199,8 +197,6 @@ queue-group@b0000 { /include/ "pq3-etsec2-1.dtsi" enet1: ethernet@b1000 { queue-group@b1000 { - fsl,rx-bit-map = <0xff>; - fsl,tx-bit-map = <0xff>; interrupts = <33 2 0 0 34 2 0 0 35 2 0 0>; }; }; diff --git a/arch/powerpc/boot/dts/fsl/c293si-post.dtsi b/arch/powerpc/boot/dts/fsl/c293si-post.dtsi index bd208320bff5..bec0fc36849d 100644 --- a/arch/powerpc/boot/dts/fsl/c293si-post.dtsi +++ b/arch/powerpc/boot/dts/fsl/c293si-post.dtsi @@ -171,8 +171,6 @@ jr@2000{ enet0: ethernet@b0000 { queue-group@b0000 { reg = <0x10000 0x1000>; - fsl,rx-bit-map = <0xff>; - fsl,tx-bit-map = <0xff>; }; }; @@ -180,8 +178,6 @@ queue-group@b0000 { enet1: ethernet@b1000 { queue-group@b1000 { reg = <0x11000 0x1000>; - fsl,rx-bit-map = <0xff>; - fsl,tx-bit-map = <0xff>; }; }; diff --git a/arch/powerpc/boot/dts/fsl/p1010si-post.dtsi b/arch/powerpc/boot/dts/fsl/p1010si-post.dtsi index 1b4aafc1f6a2..c2717f31925a 100644 --- a/arch/powerpc/boot/dts/fsl/p1010si-post.dtsi +++ b/arch/powerpc/boot/dts/fsl/p1010si-post.dtsi @@ -172,29 +172,8 @@ sdhc@2e000 { /include/ "pq3-mpic-timer-B.dtsi" /include/ "pq3-etsec2-0.dtsi" - enet0: ethernet@b0000 { - queue-group@b0000 { - fsl,rx-bit-map = <0xff>; - fsl,tx-bit-map = <0xff>; - }; - }; - /include/ "pq3-etsec2-1.dtsi" - enet1: ethernet@b1000 { - queue-group@b1000 { - fsl,rx-bit-map = <0xff>; - fsl,tx-bit-map = <0xff>; - }; - }; - /include/ "pq3-etsec2-2.dtsi" - enet2: ethernet@b2000 { - queue-group@b2000 { - fsl,rx-bit-map = <0xff>; - fsl,tx-bit-map = <0xff>; - }; - - }; global-utilities@e0000 { compatible = "fsl,p1010-guts";