Return-path: Received: from out3-smtp.messagingengine.com ([66.111.4.27]:32877 "EHLO out3-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751995AbcLXRYm (ORCPT ); Sat, 24 Dec 2016 12:24:42 -0500 Date: Sat, 24 Dec 2016 10:24:39 -0700 From: Mark Greer To: Geoff Lansberry Cc: linux-wireless , Lauro Ramos Venancio , Aloisio Almeida Jr , Samuel Ortiz , robh+dt@kernel.org, mark.rutland@arm.com, netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Justin Bronder , Jaret Cantu Subject: Re: [PATCH v3 3/3] nfc: trf7970a: Prevent repeated polling from crashing the kernel Message-ID: <20161224172439.GA15103@animalcreek.com> (sfid-20161224_182514_037082_8A5C4CE1) References: <1482380314-16440-1-git-send-email-geoff@kuvee.com> <1482380314-16440-3-git-send-email-geoff@kuvee.com> <20161224060141.GA9069@animalcreek.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: Sender: linux-wireless-owner@vger.kernel.org List-ID: On Sat, Dec 24, 2016 at 11:17:18AM -0500, Geoff Lansberry wrote: > Mark - I'm sorry, but I did not write this code, and therefore was not > able to accurately describe it. It is fixing a different issue, not > the neard segfault that we are still chasing. Last week Jaret Cantu > sent a separate email explaining the purpose of the code, which had > you copied, did you see that? Hm, no, I didn't. I received an email from Justin Bronder but not from Jaret Cantu. Justin's email did help but is still pretty high-level. We need a clear understanding as to what is happening in the digital layer and the driver to know how execution is getting into a block of error handling code that should never be executed. Once we understand that we can start thinking about what the best fix is. > Does it explain why it was done to > your satisfaction? I've asked him to join in on the effort to push > the change upstream, however he will not be available until the new > year. I expect that it would help if he joins. After the holidays is fine - I think many people are taking it easy for the next week or so, anyway. > I know you did suggest that we split off that change from the others, > and if now is the time to do that, let me know. If you don't have > the email from Jaret, also please let me know and I will forward it to > you. I think it would help you if you split it off because the first two patches have a good chance of being accepted but this one doesn't (yet). If you separate the them, it will make it easier for Samuel to take the first two (or he may take the first two anyway but its always good to make it as easy maintainers as you can). Mark --