From patchwork Tue Apr 22 15:25:05 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jean Delvare X-Patchwork-Id: 4033351 Return-Path: X-Original-To: patchwork-linux-wireless@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 345CE9F1F4 for ; Tue, 22 Apr 2014 15:25:24 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 2ED2120211 for ; Tue, 22 Apr 2014 15:25:18 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 6030C201B9 for ; Tue, 22 Apr 2014 15:25:13 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932572AbaDVPZK (ORCPT ); Tue, 22 Apr 2014 11:25:10 -0400 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 Received: from relay1.suse.de (charybdis-ext.suse.de [195.135.220.254]) by mx2.suse.de (Postfix) with ESMTP id 5C68EACF0; Tue, 22 Apr 2014 15:25:07 +0000 (UTC) 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> Organization: SUSE Linux X-Mailer: Claws Mail 3.9.3 (GTK+ 2.24.22; x86_64-suse-linux-gnu) MIME-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org X-Spam-Status: No, score=-7.5 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP 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