Re: [PATCH] NFC: microread: Drop platform data header file

Samuel Ortiz
2016-03-09T22:28:16Z

Hi Jean,

On Thu, Jan 21, 2016 at 09:20:12AM +0100, Jean Delvare wrote:
> Originally I only wanted to drop the unneeded inclusion of
> <linux/i2c.h>, but then noticed that struct
> microread_nfc_platform_data isn't actually used, and
> MICROREAD_DRIVER_NAME is redefined in the only file where it is used,
> so we can get rid of the header file and dead code altogether.
>
> Signed-off-by: Jean Delvare <jdelvare@suse.de>
> Cc: Eric Lapuyade <eric.lapuyade@intel.com>
> Cc: Lauro Ramos Venancio <lauro.venancio@openbossa.org>
> Cc: Aloisio Almeida Jr <aloisio.almeida@openbossa.org>
> Cc: Samuel Ortiz <sameo@linux.intel.com>
> ---
> Makes me wonder if this driver is used at all? I can't see any
> instance in the kernel tree.
The mei part of it is used.

> MAINTAINERS | 1
> drivers/nfc/microread/i2c.c | 8 -------
> include/linux/platform_data/microread.h | 35 --------------------------------
> 3 files changed, 44 deletions(-)
Applied, thanks.

Cheers,
Samuel.