From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751238AbdKTNWA (ORCPT ); Mon, 20 Nov 2017 08:22:00 -0500 Received: from mail2.skidata.com ([91.230.2.91]:54685 "EHLO mail2.skidata.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751100AbdKTNV7 (ORCPT ); Mon, 20 Nov 2017 08:21:59 -0500 X-IronPort-AV: E=Sophos;i="5.44,427,1505772000"; d="scan'208";a="1111529" Subject: Re: [PATCH v2 3/3] net: ethernet: fec: fix refclk enable for SMSC LAN8710/20 To: Geert Uytterhoeven CC: Andy Duan , "f.fainelli@gmail.com" , "andrew@lunn.ch" , Richard Leitner , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "Sergei Shtylyov" , Geert Uytterhoeven References: <20171120083417.32558-1-dev@g0hl1n.net> <20171120083417.32558-4-dev@g0hl1n.net> <6cab10d6-8311-f34a-5eb7-ccca3c48a527@skidata.com> <30a94c72-2345-0649-6682-4826d31c3285@skidata.com> From: Richard Leitner Message-ID: <05754f2b-3765-bf90-40ff-a2e8337ddcce@skidata.com> Date: Mon, 20 Nov 2017 14:21:51 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 8bit X-Originating-IP: [172.16.60.30] X-ClientProxiedBy: sdex1srv.skidata.net (172.16.10.92) To sdex1srv.skidata.net (172.16.10.92) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 11/20/2017 02:13 PM, Geert Uytterhoeven wrote: > Hi Richard, > > On Mon, Nov 20, 2017 at 1:55 PM, Richard Leitner > wrote: >> On 11/20/2017 11:35 AM, Andy Duan wrote: >>> 3. add reset gpio descriptor for common phy device driver. >> >> ... if I understood it correctly the patch called "Teach phylib >> hard-resetting devices" by Geert and Sergei is exactly doing this: >> https://patchwork.ozlabs.org/cover/828503/ >> https://lkml.org/lkml/2017/10/20/166 >> >> So I'll implement the phy_reset_after_clk_enable function atop of this >> patch-set and add a note that my patch-series depends on it. Would that >> be OK? > > I will update and respin that patch series after the merge window has closed. Ok. Thank you for the quick response an this information. For the Freescale Fast Ethernet Controller (FEC) there are currently (in addition to the reset gpio) two additional optional dt properties for the reset: - phy-reset-duration : Reset duration in milliseconds. - phy-reset-post-delay : Post reset delay in milliseconds. IMHO it would make sense to include them also in the phylib implementation. What do you think about it? Should I include it in my patch-series? kind regards; Richard.L > > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like that. > -- Linus Torvalds >