From patchwork Tue Jul 21 13:44:59 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Joakim Tjernlund X-Patchwork-Id: 498215 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 42B57140D31 for ; Tue, 21 Jul 2015 23:45:35 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932537AbbGUNpI (ORCPT ); Tue, 21 Jul 2015 09:45:08 -0400 Received: from smtp.transmode.se ([31.15.61.139]:60090 "EHLO smtp.transmode.se" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932076AbbGUNpG convert rfc822-to-8bit (ORCPT ); Tue, 21 Jul 2015 09:45:06 -0400 Received: from exch1.transmode.se (exch1.transmode.se [192.168.201.16]) by smtp.transmode.se (Postfix) with ESMTP id 875D51186FCF; Tue, 21 Jul 2015 15:45:01 +0200 (CEST) Received: from exch1.transmode.se (192.168.201.16) by exch1.transmode.se (192.168.201.16) with Microsoft SMTP Server (TLS) id 15.0.1076.9; Tue, 21 Jul 2015 15:45:01 +0200 Received: from exch1.transmode.se ([fe80::bd49:a412:5dc4:aadd]) by exch1.transmode.se ([fe80::bd49:a412:5dc4:aadd%17]) with mapi id 15.00.1076.000; Tue, 21 Jul 2015 15:45:01 +0200 From: Joakim Tjernlund To: "netdev@vger.kernel.org" , "Igal.Liberman@freescale.com" , "madalin.bucur@freescale.com" CC: "linuxppc-dev@lists.ozlabs.org" , "linux-kernel@vger.kernel.org" Subject: Re: [RFC,v3,12/12] fsl/fman: Add FMan MAC driver Thread-Topic: [RFC,v3,12/12] fsl/fman: Add FMan MAC driver Thread-Index: AQHQgoMUX0zDZ9osFk68924iqE+98J3kpTKAgAADVYCAAAgoAIAACheAgAGVjAA= Date: Tue, 21 Jul 2015 13:44:59 +0000 Message-ID: <1437486300.10831.42.camel@transmode.se> References: <1430299742-29428-1-git-send-email-igal.liberman@freescale.com> <1437394575.10831.18.camel@transmode.se> <1437397042.10831.34.camel@transmode.se> In-Reply-To: Accept-Language: en-US, sv-SE Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-mailer: Evolution 3.16.4 x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [192.168.200.4] Content-ID: <89B35F31E581C74EA1D4C840E5A158C7@transmode.com> MIME-Version: 1.0 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Mon, 2015-07-20 at 13:33 +0000, Madalin-Cristian Bucur wrote: > > -----Original Message----- > > From: Joakim Tjernlund [mailto:joakim.tjernlund@transmode.se] > > Sent: Monday, July 20, 2015 3:57 PM > > To: netdev@vger.kernel.org; Liberman Igal-B31950; Bucur Madalin-Cristian- > > B32716 > > Cc: linuxppc-dev@lists.ozlabs.org; linux-kernel@vger.kernel.org > > Subject: Re: [RFC,v3,12/12] fsl/fman: Add FMan MAC driver > > > > On Mon, 2015-07-20 at 12:28 +0000, Madalin-Cristian Bucur wrote: > > > Hi Joakim, > > > > > > It seems we just need to align to the API introduced by Thomas Petazzoni > > > in 3be2a49e. > > > > > > Madalin > > > > So it seems, any idea when the next spin will be ready? > > Could you also push it onto > > http://git.freescale.com/git/cgit.cgi/ppc/upstream/linux.git/ > > ? > > > > Jocke > > We're working on addressing all the feedback received to date (you've just added > a bit more) then we'll re-submit the FMan driver together with the DPAA Ethernet > driver. A push in the public git is also going to take place after the patches are sent > for review. Hi again Now I got to actually use PHY less(aka. Fixed PHY) too. I had to hack AN off for all such links to get a 1000 SGMII link working: Could you please fix this too? Jocke-- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html --- a/drivers/net/ethernet/freescale/fman/mac/memac.c +++ b/drivers/net/ethernet/freescale/fman/mac/memac.c @@ -80,7 +80,10 @@ static void setup_sgmii_internal_phy(struct memac_t *p_memac, uint8_t phy_addr) ENET_SPEED_1000); /* SGMII mode + AN enable */ - tmp_reg16 = PHY_SGMII_IF_MODE_AN | PHY_SGMII_IF_MODE_SGMII; + //tmp_reg16 = PHY_SGMII_IF_MODE_AN | PHY_SGMII_IF_MODE_SGMII; + tmp_reg16 = PHY_SGMII_IF_MODE_AN | PHY_SGMII_IF_MODE_SGMII | 0x8; + if (p_memac->mac_id != 0) + tmp_reg16 &= ~PHY_SGMII_IF_MODE_AN; memac_mii_write_phy_reg(p_memac, phy_addr, 0x14, tmp_reg16); /* Device ability according to SGMII specification */ @@ -104,6 +107,8 @@ static void setup_sgmii_internal_phy(struct memac_t *p_memac, uint8_t phy_addr) /* Restart AN */ tmp_reg16 = PHY_SGMII_CR_DEF_VAL | PHY_SGMII_CR_RESET_AN; + if (p_memac->mac_id != 0) + tmp_reg16 &= ~0x1000; memac_mii_write_phy_reg(p_memac, phy_addr, 0x0, tmp_reg16); /* Restore original enet mode */