From patchwork Thu Oct 12 03:32:01 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Joel Stanley X-Patchwork-Id: 824654 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; secure) header.d=jms.id.au header.i=@jms.id.au header.b="ie2rRlxs"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3yCGdL6x5mz9t2W for ; Thu, 12 Oct 2017 14:32:54 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752645AbdJLDcl (ORCPT ); Wed, 11 Oct 2017 23:32:41 -0400 Received: from mail-pf0-f195.google.com ([209.85.192.195]:36054 "EHLO mail-pf0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751386AbdJLDcj (ORCPT ); Wed, 11 Oct 2017 23:32:39 -0400 Received: by mail-pf0-f195.google.com with SMTP id z11so4246347pfk.3 for ; Wed, 11 Oct 2017 20:32:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=jms.id.au; s=google; h=from:to:cc:subject:date:message-id; bh=Jsz9aAkjMl1m7BOzxtFC+yJdZCM2TghXuDjzUOP0J7k=; b=ie2rRlxsyvQZjt2LvvnBVZunbUuvpMb3HliRll6zxV4PplrPGbvxPLboa//QnhjD16 VY5nj64SDSf3NeY/5ySMFOSt6UQkGiacgF4ZOnyYqNxFimdZ/TAp1YTgJIGGJ4ZDkKzD n+BE/mzNcBTnH+fWm8Blu+5ebW4kPF2mjFabM= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=Jsz9aAkjMl1m7BOzxtFC+yJdZCM2TghXuDjzUOP0J7k=; b=hbEsge8onyuzlu9v2VjHathRlIH9XNhQ/B+tjXhS0YOU7MRWHsaGp5p7N8OMH75q1j X0iasqznvwwDVbk6Ayw1pdSThhdReWcxAKEim7gQHgzqOyLLXJLCHm+QdyJ87qXmKcXn HFvK+7uG1oywTzqZHQ20wS0hXlUxnBN/fKP78ydMT+R8U4JzNEBKgHzAa6rymvH0mlrX 97sswTEHw0hsDy9xPOn/mgc/0ps4zNOM91zAMSrvokQ7x0J9CuS3rPVEJMzxOzl7quCS bDlwuyj3TH9nSVMFbMnlD7BNeuEKFYSVg1bpzo5VbEd76RFne6yCbLhshsmnL/6cew/o sAdQ== X-Gm-Message-State: AMCzsaV6M0WLojZOqLZyW8QooFcl63qxW1Fp0jO0uPHpMWVpLVzdool9 3V90Rt+Eidonoq/O/hfaVhpdxQ== X-Google-Smtp-Source: AOwi7QAs8LPFHa67jLmSACKKoVgy4Pzd0N9oZrOAzXnPjeK6GI+Ksrt3/v0ePEmN7tsPFX+EaBc1dA== X-Received: by 10.159.194.196 with SMTP id u4mr969807plz.49.1507779158926; Wed, 11 Oct 2017 20:32:38 -0700 (PDT) Received: from E402SA.smb.com ([122.116.41.53]) by smtp.gmail.com with ESMTPSA id d124sm25321840pfc.42.2017.10.11.20.32.37 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 11 Oct 2017 20:32:38 -0700 (PDT) From: Joel Stanley To: "David S . Miller" , Benjamin Herrenschmidt Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Andrew Jeffery Subject: [PATCH v2] net: ftgmac100: Request clock and set speed Date: Thu, 12 Oct 2017 11:32:01 +0800 Message-Id: <20171012033201.12845-1-joel@jms.id.au> X-Mailer: git-send-email 2.14.1 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org According to the ASPEED datasheet, gigabit speeds require a clock of 100MHz or higher. Other speeds require 25MHz or higher. This patch configures a 100MHz clock if the system has a direct-attached PHY, or 25MHz if the system is running NC-SI which is limited to 100MHz. There appear to be no other upstream users of the FTGMAC100 driver so it is hard to know the clocking requirements of other platforms. Therefore a conservative approach was taken with enabling clocks. If the platform is not ASPEED, both requesting the clock and configuring the speed is skipped. Signed-off-by: Joel Stanley --- Andrew, as I'm travelling can you please test this on the evb and a palmetto? Use my wip/aspeed-v4.14-clk branch, or OpenBMC's dev-4.13. David, please wait for Andrew's tested-by before applying. Cheers! v2: - only touch the clocks on Aspeed platforms - unconditionally call clk_unprepare_disable drivers/net/ethernet/faraday/ftgmac100.c | 26 ++++++++++++++++++++++++++ 1 file changed, 26 insertions(+) diff --git a/drivers/net/ethernet/faraday/ftgmac100.c b/drivers/net/ethernet/faraday/ftgmac100.c index 9ed8e4b81530..cd352bf41da1 100644 --- a/drivers/net/ethernet/faraday/ftgmac100.c +++ b/drivers/net/ethernet/faraday/ftgmac100.c @@ -21,6 +21,7 @@ #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt +#include #include #include #include @@ -59,6 +60,9 @@ /* Min number of tx ring entries before stopping queue */ #define TX_THRESHOLD (MAX_SKB_FRAGS + 1) +#define FTGMAC_100MHZ 100000000 +#define FTGMAC_25MHZ 25000000 + struct ftgmac100 { /* Registers */ struct resource *res; @@ -96,6 +100,7 @@ struct ftgmac100 { struct napi_struct napi; struct work_struct reset_task; struct mii_bus *mii_bus; + struct clk *clk; /* Link management */ int cur_speed; @@ -1734,6 +1739,22 @@ static void ftgmac100_ncsi_handler(struct ncsi_dev *nd) nd->link_up ? "up" : "down"); } +static void ftgmac100_setup_clk(struct ftgmac100_priv *priv) +{ + priv->clk = devm_clk_get(&pdev->dev, NULL); + if (IS_ERR(priv->clk)) + return; + + clk_prepare_enable(priv->clk); + + /* Aspeed specifies a 100MHz clock is required for up to + * 1000Mbit link speeds. As NCSI is limited to 100Mbit, 25MHz + * is sufficient + */ + clk_set_rate(priv->clk, priv->is_ncsi ? FTGMAC_25MHZ : + FTGMAC_100MHZ); +} + static int ftgmac100_probe(struct platform_device *pdev) { struct resource *res; @@ -1830,6 +1851,9 @@ static int ftgmac100_probe(struct platform_device *pdev) goto err_setup_mdio; } + if (priv->is_aspeed) + ftgmac100_setup_clk(priv); + /* Default ring sizes */ priv->rx_q_entries = priv->new_rx_q_entries = DEF_RX_QUEUE_ENTRIES; priv->tx_q_entries = priv->new_tx_q_entries = DEF_TX_QUEUE_ENTRIES; @@ -1883,6 +1907,8 @@ static int ftgmac100_remove(struct platform_device *pdev) unregister_netdev(netdev); + clk_disable_unprepare(priv->clk); + /* There's a small chance the reset task will have been re-queued, * during stop, make sure it's gone before we free the structure. */