Hi Marcel, On Fri, Oct 19, 2012 at 11:56:11AM -0700, Marcel Holtmann wrote: > With the removal of the deprecated pn544.c driver, the header > file pn544.h is now used only by the new pn544_hci.c driver. So > move the header file close to the driver. There are quite a bit of junk in this header file, as outlined from your second patch. But there is a platform data structure definition (struct pn544_nfc_platform_data) that should be available from the arch code. By passing function pointers (That don't do anything else than toggling GPIOs) from arch code to the driver, this function is not very nice. Nor flexible as it won't work with e.g. DT platforms. The driver should just get the IRQ and the 3 potentially needed GPIOs and implement the functions itself. Even when that happens, we'll still need a header file that !DT platform could use to pass the GPIO numbers around. So the bottom line is: We need to keep this one under include/linux/nfc/. Cheers, Samuel. -- Intel Open Source Technology Centre http://oss.intel.com/ -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
