From mboxrd@z Thu Jan 1 00:00:00 1970 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=-7.0 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=ham 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 F1754C433E0 for ; Sun, 28 Jun 2020 05:08:35 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id BCFB8212CC for ; Sun, 28 Jun 2020 05:08:35 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="okq4PoAN"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=nxp.com header.i=@nxp.com header.b="njnr49IZ" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org BCFB8212CC Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=nxp.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:In-Reply-To:References:Message-ID:Date: Subject:To:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=Znv5ji5Xakk9Bd4Y+iticSVtlPpeNSmfX1NpGvXp5ek=; b=okq4PoANWNLz/yUdJ6v7zkRoV Xgt3er67E60jf6HpGKGrjUTXKK8vKid9CSvyvu2goiVL251jJ4ivZ+vahSqruAr4ZiCeBoQ2I85Gd W5+K3Xc6qRYGWTN3bbUiPVJ2VLvr/ktqxIcTi8AdMOguuIvSRD3P/cCppzyjx0t819vr7/sZN7lj0 My2iUHQiGwqaHVb0ZUBQ/b1WudkH71iamVx5nnX6HUBo9DrXQgyq8jDB/RrGXUDeWSKIz5fiUJX3D AmmHSaMlIffUzDhuZw5vMiZKVS4NogL9noovlXlMQZcxQO8K3qXr3pbKyOU0S4vzeLKFpgRSD+Wvo rCeVaSQBg==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jpPVQ-0002Vg-Q0; Sun, 28 Jun 2020 05:05:28 +0000 Received: from mail-eopbgr30082.outbound.protection.outlook.com ([40.107.3.82] helo=EUR03-AM5-obe.outbound.protection.outlook.com) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jpPVN-0002VQ-F9 for linux-arm-kernel@lists.infradead.org; Sun, 28 Jun 2020 05:05:27 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=h0V70N1mzbPtAd/Ac+6xS4cGXLtU9rJdK/nS1yrYx0bfATGyaSIhh9C8bY+3EGAcIsAJjVHM4V0yBWITc4/uTyUCD9zHCfJadxwsKV0WPRZ7BQbkV22KsIAy05Ptxa1tbJF6zkjnEVb6Jq6PJSTCWuOkb82wjuC3YcW6VdOt3IBmEvoI6E26SH6uC3JUg7Xbbh8/nSBXfCGGcSasOENqk5WK/n6Sn+K/tHwQzx/ZlPopgBMGWtvPdk/h1WAEhPQJTlYO6J4eyy0Xlyz+NanTbRUf/NlURx0cpF4bMXcZFtBMF719LAEw5ShpIJ7AdVPe1TMMvbQZvtooq6gaEyF2bA== 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=7l9GV1dbxIZM4h7beds6oD7wPsHwhruQapohL3Xbz3U=; b=Gw+3yU6ZH7MyYYkUvPLNMdZuFfsHTKDAySpOnlfdR9TuStHEGhW3ztFTDtEzrykxmfX92Z/78KHGLywIE44uMaQpBLp5+JLdh2cTgYKm/TITXawCLNmk+YgWXgPQp3FWByPmy+Zf4qNRP2rGkh/Q9dgY0Jq/5aJVvD550aMmdMT/X0JjamvwydhW0J3tvJS9mgQkawaVTdkamZWLHMe823L2ZFnUe36OlSg8WORRphFnujotBGkI0nI1e7m+FUzCCFpq+/yxkRVg1gew9xghOBvmBaeVlhnKxmVxcaW0DcDyXF45nwczYxMtWdhBDkHvbZ75ebtfeEgaNjHwMzvgDw== 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=7l9GV1dbxIZM4h7beds6oD7wPsHwhruQapohL3Xbz3U=; b=njnr49IZuxduyizBeq7OyTYQmWQVAuzagaOc0ArAN3RmmnlYAcKHYXnNnktv7CHa9VEqt54V/EKxV6XV0wo0CwwnVzTurx+rTDhGSuuyuuDC9/UMP265esgz4xaMAcyaKJjotBG2WaJc7xM+yDBADjwU6laG2wj++1BtgSz0ukE= Received: from AM6PR0402MB3607.eurprd04.prod.outlook.com (2603:10a6:209:12::18) by AM6PR0402MB3605.eurprd04.prod.outlook.com (2603:10a6:209:3::33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3131.21; Sun, 28 Jun 2020 05:05:15 +0000 Received: from AM6PR0402MB3607.eurprd04.prod.outlook.com ([fe80::35f8:f020:9b47:9aa1]) by AM6PR0402MB3607.eurprd04.prod.outlook.com ([fe80::35f8:f020:9b47:9aa1%7]) with mapi id 15.20.3131.026; Sun, 28 Jun 2020 05:05:15 +0000 From: Andy Duan To: Sven Van Asbroeck , "shawnguo@kernel.org" Subject: RE: [EXT] [PATCH v4 2/2] ARM: imx6plus: enable internal routing of clk_enet_ref where possible Thread-Topic: [EXT] [PATCH v4 2/2] ARM: imx6plus: enable internal routing of clk_enet_ref where possible Thread-Index: AQHWSvkYcUowKEcmtU6JLU1iKnAQWajtfTlQ Date: Sun, 28 Jun 2020 05:05:15 +0000 Message-ID: References: <20200625140105.14999-1-TheSven73@gmail.com> <20200625140105.14999-2-TheSven73@gmail.com> In-Reply-To: <20200625140105.14999-2-TheSven73@gmail.com> Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: gmail.com; dkim=none (message not signed) header.d=none;gmail.com; dmarc=none action=none header.from=nxp.com; x-originating-ip: [119.31.174.68] x-ms-publictraffictype: Email x-ms-office365-filtering-ht: Tenant x-ms-office365-filtering-correlation-id: 5c155350-6b8d-4a8e-15ef-08d81b20d878 x-ms-traffictypediagnostic: AM6PR0402MB3605: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:5797; x-forefront-prvs: 0448A97BF2 x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: BAiTpccLxDtOsB6eI6eJBy5+2D6HX/M/Pkfh0nn31UsKJExDYldNL+8vhPTMOktHZm9XcWBMgkqkveHjGyDeT18l1f+iJAHsznYL1YXDdePwy0WvWMIFdNegKIzYZhZH6T57zcJDlaUpD8Gubpnh5dfUb71E3GKqWY0pGzakLsZRdcrgqDdeCaYGzxYRMTeQjgLjK0YwzSn7g6giNKoPJGXlLeNFRX/w+fLQaPM3musbEU4h1QzcunM/1N9uG76cRNmg7LTYU3ule2/cclE2NMYuDiN7tgGYlxWQXFICMSjC8HulqZW6IuC7e/qf/v7oGbAzLg5bKgzhoZ0FKB7iw15qqtPeED4+ImXJT1QlT5V17um1i3De9+GED9paNGaB x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM6PR0402MB3607.eurprd04.prod.outlook.com; PTR:; CAT:NONE; SFTY:; SFS:(4636009)(396003)(39850400004)(366004)(136003)(346002)(376002)(66446008)(64756008)(26005)(66556008)(66476007)(33656002)(7696005)(66946007)(76116006)(478600001)(8676002)(186003)(5660300002)(86362001)(83380400001)(6506007)(9686003)(110136005)(52536014)(4326008)(2906002)(55016002)(316002)(54906003)(8936002)(71200400001)(32563001); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata: 7Fa5VLbx32Ct+wFoAa2hOSvGBGGFthzSWXxHzTnMYvueBB/kdRZM5v7vss7JaDvAsHkXfH646Ufg4U6vgU84sUFm0exV36ZvSp9yjiUwkdWM6yqO0zLOSLy88TDlF1WjdZ9n0lKjrTyZmj4fcA0gzmi3J+60Z9kk+mqEFy2unXT+0aBD+KtDCi/Q4d67NEKx+zbzvOHiipe/I/3jLjIqJphkJLIrcuN/u94VPB5vX68UxecsNcThNVunl7moSO/GDGs3egYC07g4+rmSpNlIDifDNQWTyBwFpOgTUS83etaka2J2Ktb9xHfPt87tHc6LttUulXG/Cv2aRyZIdqotNIK5SSEOCBrrs0G1P9vAQkJilfbMU8taCnjKskr17J/5PFVnoWCKh/NeyCTb8pGFjoh4wWfcEohDoZo55avdoCkE6o+F0kVtWL9+4MbmN4GoLaAnfRL5PDOf++TeQCOjborNBMvoHzhhPw2lJWuvxJs= MIME-Version: 1.0 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: AM6PR0402MB3607.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 5c155350-6b8d-4a8e-15ef-08d81b20d878 X-MS-Exchange-CrossTenant-originalarrivaltime: 28 Jun 2020 05:05:15.6439 (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: ySbnv1FMsmGCO8zgRDe6TSMePnvmgT6z/FAgssPkR+vB91YgHVO6/v8M1TTi4YQPwduBw022/sAxdQet6eJdEw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM6PR0402MB3605 X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Sascha Hauer , "linux-kernel@vger.kernel.org" , dl-linux-imx , Pengutronix Kernel Team , Fabio Estevam , "linux-arm-kernel@lists.infradead.org" Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org From: Sven Van Asbroeck Sent: Thursday, June 25, 2020 10:01 PM > On imx6, the ethernet reference clock (clk_enet_ref) can be generated by > either the imx6, or an external source (e.g. an oscillator or the PHY). When > generated by the imx6, the clock source (from ANATOP) must be routed to the > input of clk_enet_ref via two pads on the SoC, typically via a dedicated track > on the PCB. > > On an imx6 plus however, there is a new setting which enables this clock to > be routed internally on the SoC, from its ANATOP clock source, straight to > clk_enet_ref, without having to go through the SoC pads. > > Board designs where the clock is generated by the imx6 should not be > affected by routing the clock internally. Therefore on a plus, we can enable > internal routing by default. > > Signed-off-by: Sven Van Asbroeck For the version: Reviewed-by: Fugang Duan > --- > v3 -> v4: > - avoid double-check for IS_ERR(gpr) by including Fabio Estevam's > patch. > v2 -> v3: > - remove check for imx6q, which is already implied when > of_machine_is_compatible("fsl,imx6qp") > v1 -> v2: > - Fabio Estevam: use of_machine_is_compatible() to determine if we > are running on an imx6 plus. > > To: Shawn Guo > To: Andy Duan > Cc: Sascha Hauer > Cc: Pengutronix Kernel Team > Cc: Fabio Estevam > Cc: NXP Linux Team > Cc: linux-arm-kernel@lists.infradead.org > Cc: linux-kernel@vger.kernel.org > > arch/arm/mach-imx/mach-imx6q.c | 14 ++++++++++++++ > include/linux/mfd/syscon/imx6q-iomuxc-gpr.h | 1 + > 2 files changed, 15 insertions(+) > > diff --git a/arch/arm/mach-imx/mach-imx6q.c > b/arch/arm/mach-imx/mach-imx6q.c index ae89ad93ca83..07cfe0d349c3 > 100644 > --- a/arch/arm/mach-imx/mach-imx6q.c > +++ b/arch/arm/mach-imx/mach-imx6q.c > @@ -204,6 +204,20 @@ static void __init imx6q_1588_init(void) > regmap_update_bits(gpr, IOMUXC_GPR1, > IMX6Q_GPR1_ENET_CLK_SEL_MASK, > clksel); > > + /* > + * On imx6 plus, enet_ref from ANATOP/CCM can be internally > routed to > + * be the PTP clock source, instead of having to be routed through > + * pads. > + * Board designs which route the ANATOP/CCM clock through pads > are > + * unaffected when routing happens internally. So on these designs, > + * route internally by default. > + */ > + if (clksel == IMX6Q_GPR1_ENET_CLK_SEL_ANATOP && > + of_machine_is_compatible("fsl,imx6qp")) > + regmap_update_bits(gpr, IOMUXC_GPR5, > + IMX6Q_GPR5_ENET_TXCLK_SEL, > + IMX6Q_GPR5_ENET_TXCLK_SEL); > + > clk_put(enet_ref); > put_ptp_clk: > clk_put(ptp_clk); > diff --git a/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h > b/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h > index d4b5e527a7a3..eb65d48da0df 100644 > --- a/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h > +++ b/include/linux/mfd/syscon/imx6q-iomuxc-gpr.h > @@ -240,6 +240,7 @@ > #define IMX6Q_GPR4_IPU_RD_CACHE_CTL BIT(0) > > #define IMX6Q_GPR5_L2_CLK_STOP BIT(8) > +#define IMX6Q_GPR5_ENET_TXCLK_SEL BIT(9) > #define IMX6Q_GPR5_SATA_SW_PD BIT(10) > #define IMX6Q_GPR5_SATA_SW_RST BIT(11) > > -- > 2.17.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel