Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754590Ab1DNXJ7 (ORCPT ); Thu, 14 Apr 2011 19:09:59 -0400 Received: from mga11.intel.com ([192.55.52.93]:47275 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754321Ab1DNXJ6 (ORCPT ); Thu, 14 Apr 2011 19:09:58 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.64,214,1301900400"; d="scan'208";a="679237528" Date: Fri, 15 Apr 2011 01:09:56 +0200 From: Samuel Ortiz To: Lauro Ramos Venancio Cc: linux-kernel@vger.kernel.org Subject: Re: [RFC] NFC subsystem prototype Message-ID: <20110414230955.GB32110@sortiz-mobl> References: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3530 Lines: 74 Hi Lauro, On Thu, Apr 14, 2011 at 07:57:57PM -0300, Lauro Ramos Venancio wrote: > 2011/4/14 Samuel Ortiz : > > >> Subsystem Operations: > >> ? ? ? - start poll - start polling for targets using the protocols given as > >> parameter > >> ? ? ? - stop poll - stop polling > >> ? ? ? - activate target - activate a target for communication using a > >> specific protocol > >> ? ? ? - deactivate target - deactivate a target > >> ? ? ? - reset device - put the adapter in idle mode > >> ? ? ? - data exchange - low level data exchange (send and receive data) > > I suppose the data_exchange hook is sitting at the NFCIP level ? As I agree > > with your further comment that this one should not be part of the netlink > > API but should be done through sockets instead, I think this one should be > > defined as the netdev start_xmit hook. > > So this matches quite well the pn533 API, but I'm wondering how you're > > planning to support the HCI based devices (pn544, microread). We clearly > > have 2 kind of devices here, and it reminds me of the soft MAC vs full MAC > > problem in the 802.11 subsystem. > > I think we should have a kernel NFC HCI layer that would implement those > > hooks by following the HCP protocol. Then devices that only take HCI > > commands (kind of soft MAC NFC devices) would register against the NFC HCI > > layer and use those hooks. Other devices (e.g. pn533) would register at a > > higher level (The NFC core layer) and provide their own hooks. This would be > > similar to what 802.11 drivers do by registering against mac80211 (soft MACs) > > or directly against cfg80211 (full MACs). > > I fully agree. The prototype was designed with pn533 and pn544 in > mind. When the pn544 device driver is implemented, we will need > another layer for the HCI implementation. Are you guys planning to start working on the pn544 driver ? In theory the HCI layer should be implemented first but I suppose those 2 can be done in parallel, and I can start looking at the HCI parts. At least all the registration one and the netdev interaction as well. > > >> Further work: > >> ? ? ? - Define subsystem operations for adapters in "target mode" > > I am not sure we need any specific additional hook for the target mode. In > > this mode, we mostly would be sending events (RF field ON, card activated > > or deactivated). After that, afaiu, the card would be expecting data > > reception, and then sending responses to it. > > I think the main part is the handling of timing constraints between > the data reception and the response using a socket interface. What are those constraints ? > > We probably need to add a mode setting hook in the ops structure. And also, > > according to NFCIP-2, NFC devices should by default be in target mode. > > The selection between proximity and vicinity is partially covered by > the protocols selection in start_poll operation. > > Probably, the polling loop needs to have an initiator mode phase and a > target mode phase in a cycle (as the PN544 Polling management). That is something that is still not clear to me, to be honest with you. When in initiator mode, are we suppose to run regular polling cycles ? Cheers, Samuel. -- Intel Open Source Technology Centre http://oss.intel.com/ -- 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/