From mboxrd@z Thu Jan 1 00:00:00 1970 From: Florian Fainelli Subject: Re: [RFC PATCH v1] net: ethernet: nb8800: Reset HW block in ndo_open Date: Sat, 29 Jul 2017 08:18:57 -0700 Message-ID: <4ff343de-41c7-c309-04dc-983b5fe3e066@gmail.com> References: <823b1540-b528-bbd7-7f99-5dc39a08868a@sigmadesigns.com> <446e3a95-80c3-0742-9cb1-69a8dfc9b1ae@free.fr> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Cc: Marc Gonzalez , "David S. Miller" , netdev , Linux ARM To: Mason , Mans Rullgard Return-path: Received: from mail-pg0-f67.google.com ([74.125.83.67]:35780 "EHLO mail-pg0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752479AbdG2PTA (ORCPT ); Sat, 29 Jul 2017 11:19:00 -0400 Received: by mail-pg0-f67.google.com with SMTP id d193so26317872pgc.2 for ; Sat, 29 Jul 2017 08:19:00 -0700 (PDT) In-Reply-To: Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: On 07/29/2017 05:02 AM, Mason wrote: > On 29/07/2017 13:24, Måns Rullgård wrote: > >> Until you figure out why it's getting stuck, we can't be sure >> it isn't caused by something that could trigger at any time. > Would you take a look at it, if I can reproduce on tango4? > > I have identified a 100% reproducible flaw. > I have proposed a work-around that brings this down to 0 > (tested 1000 cycles of link up / ping / link down). Can you also try to get help from your HW resources to eventually help you find out what is going on here? If anybody has access to that it would be you. > > In my opinion, upstream should consider this work-around > for inclusion. I'd like to hear David's and Florian's > opinion on the topic. It's always a pain to maintain > out-of-tree patches. I have to agree with Mans here that the commit message explanation is not good enough to understand how the RX path is hosed after a call to ndo_stop() it would be good, both for you and for the people maintaining this driver to understand what happens exactly so the fix is correct, understood and maintainable. The patch itself looks reasonable with the limited description given, but it's the description itself that needs changing. BTW, this should probably come with a Fixes: tag to identify which commit (presumably the one introducing the driver) is being fixed. -- Florian