From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from cantor2.suse.de ([195.135.220.15]:42449 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756153AbaDVPZI (ORCPT ); Tue, 22 Apr 2014 11:25:08 -0400 Date: Tue, 22 Apr 2014 17:25:05 +0200 From: Jean Delvare To: linux-wireless@vger.kernel.org Cc: Lauro Ramos Venancio , Aloisio Almeida Jr , Samuel Ortiz Subject: [PATCH] NFC: microread: Platform data header file clean-ups Message-ID: <20140422172505.25e3dacf@endymion.delvare> (sfid-20140422_172513_517773_06E0E979) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-wireless-owner@vger.kernel.org List-ID: Several clean-ups related to include/linux/platform_data/microread.h: * Fix device name in comment. * Don't include as it isn't needed. * Include this header file from drivers/nfc/microread/i2c.c as that file uses struct microread_nfc_platform_data. * Add this header file to the NFC entry in MAINTAINERS. Signed-off-by: Jean Delvare Cc: Lauro Ramos Venancio Cc: Aloisio Almeida Jr Cc: Samuel Ortiz --- That being said, the header file in question only declares struct microread_nfc_platform_data, which is only used in drivers/nfc/microread/i2c.c, and that piece of code doesn't access a single field of the structure. So I am wondering if this struct and header file are needed at all in the first place? MAINTAINERS | 1 + drivers/nfc/microread/i2c.c | 1 + include/linux/platform_data/microread.h | 4 +--- 3 files changed, 3 insertions(+), 3 deletions(-) --- linux-3.15-rc2.orig/include/linux/platform_data/microread.h 2014-03-31 05:40:15.000000000 +0200 +++ linux-3.15-rc2/include/linux/platform_data/microread.h 2014-04-22 17:12:06.702561202 +0200 @@ -1,5 +1,5 @@ /* - * Driver include for the PN544 NFC chip. + * Driver include for the Inside Secure microread NFC chip. * * Copyright (C) 2011 Tieto Poland * Copyright (C) 2012 Intel Corporation. All rights reserved. @@ -21,8 +21,6 @@ #ifndef _MICROREAD_H #define _MICROREAD_H -#include - #define MICROREAD_DRIVER_NAME "microread" /* board config platform data for microread */ --- linux-3.15-rc2.orig/MAINTAINERS 2014-04-22 14:51:09.025285048 +0200 +++ linux-3.15-rc2/MAINTAINERS 2014-04-22 17:12:33.909132651 +0200 @@ -6204,6 +6204,7 @@ F: include/net/nfc/ F: include/uapi/linux/nfc.h F: drivers/nfc/ F: include/linux/platform_data/pn544.h +F: include/linux/platform_data/microread.h F: Documentation/devicetree/bindings/net/nfc/ NFS, SUNRPC, AND LOCKD CLIENTS --- linux-3.15-rc2.orig/drivers/nfc/microread/i2c.c 2014-04-22 16:56:23.385837875 +0200 +++ linux-3.15-rc2/drivers/nfc/microread/i2c.c 2014-04-22 17:14:06.962088154 +0200 @@ -26,6 +26,7 @@ #include #include +#include #include #include -- Jean Delvare SUSE L3 Support