lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 20 Nov 2017 14:21:51 +0100 From: Richard Leitner <richard.leitner@...data.com> To: Geert Uytterhoeven <geert@...ux-m68k.org> CC: Andy Duan <fugang.duan@....com>, "f.fainelli@...il.com" <f.fainelli@...il.com>, "andrew@...n.ch" <andrew@...n.ch>, Richard Leitner <dev@...l1n.net>, "netdev@...r.kernel.org" <netdev@...r.kernel.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "Sergei Shtylyov" <sergei.shtylyov@...entembedded.com>, Geert Uytterhoeven <geert+renesas@...der.be> Subject: Re: [PATCH v2 3/3] net: ethernet: fec: fix refclk enable for SMSC LAN8710/20 On 11/20/2017 02:13 PM, Geert Uytterhoeven wrote: > Hi Richard, > > On Mon, Nov 20, 2017 at 1:55 PM, Richard Leitner > <richard.leitner@...data.com> 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@...ux-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 >
Powered by blists - more mailing lists