Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751542AbbEXRIF (ORCPT ); Sun, 24 May 2015 13:08:05 -0400 Received: from mga01.intel.com ([192.55.52.88]:25545 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751257AbbEXRIB (ORCPT ); Sun, 24 May 2015 13:08:01 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.13,487,1427785200"; d="scan'208";a="731019333" Date: Sun, 24 May 2015 19:07:58 +0200 From: Samuel Ortiz To: Robert Dolca Cc: Robert Dolca , linux-nfc@ml01.01.org, netdev@vger.kernel.org, linux-wireless@vger.kernel.org, "linux-kernel@vger.kernel.org" , "David S. Miller" Subject: Re: [linux-nfc] [PATCH 3/8] NFC: NCI: Adds NCI init and reset API for drivers Message-ID: <20150524170758.GC15886@ribalta.ccr.corp.intel.com> References: <1424772112-27399-1-git-send-email-robert.dolca@intel.com> <1424772112-27399-4-git-send-email-robert.dolca@intel.com> <20150326002937.GB10954@ribalta.home> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1621 Lines: 40 Hi Robert, On Tue, Mar 31, 2015 at 05:05:53PM +0300, Robert Dolca wrote: > On Thu, Mar 26, 2015 at 2:29 AM, Samuel Ortiz wrote: > > Hi Robert, > > > > On Tue, Feb 24, 2015 at 12:01:47PM +0200, Robert Dolca wrote: > >> In order to communicate with the device during the setup > >> phase, the driver may need to initialize the device. After > >> the setup is done the driver should reset the device to leave > >> it in the same state that it was before the setup function > >> call. > > I would prefer not to export those symbols, but instead introduce a > > quirk bitmap to let the NCI core know that your device expects the core > > to be initialized before calling the setup ops. > > That would be done from nci_open_device(). > > As part of the initialization / firmware upgrade procedure the driver > needs to reset and initialize the NCI connection multiple times. > Having the connection initialized before calling setup is not enough. Fair enough, I am ok with exporting those symbols. BTW after looking at your setup routine, I think this is wrong: + /* Load firmware from disk */ + r = fdp_nci_request_firmware(ndev); + if (r) + goto error; You should be able to boot your NFC chipset without a local patch. If there is one, then you can try patching your device, but otherwise we should continue with the exisiting one. Cheers, Samuel. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/