From patchwork Wed Mar 20 18:06:23 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Fabio Estevam X-Patchwork-Id: 2309151 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) by patchwork1.kernel.org (Postfix) with ESMTP id 2B84F40213 for ; Wed, 20 Mar 2013 18:09:29 +0000 (UTC) Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1UINPG-00026c-W9; Wed, 20 Mar 2013 18:06:35 +0000 Received: from mail-vb0-x229.google.com ([2607:f8b0:400c:c02::229]) by merlin.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1UINPC-000269-Bj for linux-arm-kernel@lists.infradead.org; Wed, 20 Mar 2013 18:06:31 +0000 Received: by mail-vb0-f41.google.com with SMTP id l22so1318550vbn.28 for ; Wed, 20 Mar 2013 11:06:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:x-received:in-reply-to:references:date:message-id :subject:from:to:cc:content-type:content-transfer-encoding; bh=8+i+fnHdMtuv33hPwcOnkl/Y6jUrZSxMfASpma0rumk=; b=wYGChRb90k2Jyy2YIhpn3/vI7adUDkaah+Iay4m6Yy8UFbTdlGai62GfaWGnxSmeIE bhJdm6adqck/VzN7Th6W6CFsiYagouKCLrLdAgvPAM2T7xwTJQfzCfptIzc3ByhyvAOS 8WYY7gdZJEwDu8NJWtnrcUJ9ORdeVO1zVQoEs7BfCGOM4ns6jTxigiU61qCitmMyT20I ivkPt7s9DxEECjB9f4lKtBao36dGmA5pOb5OOxDVEzNd0xqVb8osGVyLSuh2PJt+2dO8 n1pddGzpU4TJIgBgk2K+v9XznEZ2ko7zmFykrijJRQC57aHtbzGPJCZWpcqwTcBgVRL6 kKfw== MIME-Version: 1.0 X-Received: by 10.221.0.199 with SMTP id nn7mr9358887vcb.14.1363802783976; Wed, 20 Mar 2013 11:06:23 -0700 (PDT) Received: by 10.59.5.201 with HTTP; Wed, 20 Mar 2013 11:06:23 -0700 (PDT) In-Reply-To: <20130320160418.GG20530@pengutronix.de> References: <1357265063-30528-1-git-send-email-Frank.Li@freescale.com> <20130320160418.GG20530@pengutronix.de> Date: Wed, 20 Mar 2013 15:06:23 -0300 Message-ID: Subject: Re: [PATCH v3 1/1 net-next] NET: FEC: dynamtic check DMA desc buff type From: Fabio Estevam To: =?UTF-8?Q?Uwe_Kleine=2DK=C3=B6nig?= X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20130320_140630_519747_D3265DD0 X-CRM114-Status: GOOD ( 11.57 ) X-Spam-Score: -2.0 (--) X-Spam-Report: SpamAssassin version 3.3.2 on merlin.infradead.org summary: Content analysis details: (-2.0 points) pts rule name description ---- ---------------------- -------------------------------------------------- 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail provider (festevam[at]gmail.com) -0.0 SPF_PASS SPF: sender matches SPF record -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] -0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from author's domain 0.1 DKIM_SIGNED Message has a DKIM or DK signature, not necessarily valid -0.1 DKIM_VALID Message has at least one valid DKIM or DK signature Cc: Frank Li , netdev@vger.kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, lznuaa@gmail.com, shawn.guo@linaro.org, davem@davemloft.net, linux-arm-kernel@lists.infradead.org X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org On Wed, Mar 20, 2013 at 1:04 PM, Uwe Kleine-König wrote: > Hello, > > On Fri, Jan 04, 2013 at 10:04:23AM +0800, Frank Li wrote: >> MX6 and mx28 support enhanced DMA descriptor buff to support 1588 >> ptp. But MX25, MX3x, MX5x can't support enhanced DMA descriptor buff. >> Check fec type and choose correct DMA descriptor buff type. >> >> Remove static config CONFIG_FEC_PTP. >> ptp function will be auto detected. > Your patch (now as ff43da86c69d76a726ffe7d1666148960dc1d108 in v3.9-rc1) > breaks building with CONFIG_FEC=m: > > ERROR: "fec_ptp_init" [drivers/net/ethernet/freescale/fec.ko] undefined! > ERROR: "fec_ptp_ioctl" [drivers/net/ethernet/freescale/fec.ko] undefined! > ERROR: "fec_ptp_start_cyclecounter" [drivers/net/ethernet/freescale/fec.ko] undefined! What about this fix? --- drivers/net/ethernet/freescale/fec_ptp.c | 3 +++ 1 file changed, 3 insertions(+) * fec_time_keep - call timecounter_read every second to avoid timer overrun @@ -383,3 +385,4 @@ void fec_ptp_init(struct net_device *ndev, struct platform_device *pdev) pr_info("registered PHC device on %s\n", ndev->name); } } +EXPORT_SYMBOL(fec_ptp_init); diff --git a/drivers/net/ethernet/freescale/fec_ptp.c b/drivers/net/ethernet/freescale/fec_ptp.c index 1f17ca0..0d8df40 100644 --- a/drivers/net/ethernet/freescale/fec_ptp.c +++ b/drivers/net/ethernet/freescale/fec_ptp.c @@ -128,6 +128,7 @@ void fec_ptp_start_cyclecounter(struct net_device *ndev) spin_unlock_irqrestore(&fep->tmreg_lock, flags); } +EXPORT_SYMBOL(fec_ptp_start_cyclecounter); /** * fec_ptp_adjfreq - adjust ptp cycle frequency @@ -318,6 +319,7 @@ int fec_ptp_ioctl(struct net_device *ndev, struct ifreq *ifr, int cmd) return copy_to_user(ifr->ifr_data, &config, sizeof(config)) ? -EFAULT : 0; } +EXPORT_SYMBOL(fec_ptp_ioctl); /**