From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752912AbdBCMnV (ORCPT ); Fri, 3 Feb 2017 07:43:21 -0500 Received: from mail-lf0-f66.google.com ([209.85.215.66]:32801 "EHLO mail-lf0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752514AbdBCMnT (ORCPT ); Fri, 3 Feb 2017 07:43:19 -0500 Date: Fri, 3 Feb 2017 13:43:17 +0100 From: Johan Hovold To: Masanari Iida Cc: johan@kernel.org, elder@kernel.org, gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org, greybus-dev@lists.linaro.org Subject: Re: [PATCH] [linux-next] staging: greybus: Remove duplicate include file Message-ID: <20170203124317.GD12545@localhost> References: <20170201110507.11042-1-standby24x7@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170201110507.11042-1-standby24x7@gmail.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Feb 01, 2017 at 08:05:07PM +0900, Masanari Iida wrote: > This patch removed duplicated include line. > > Signed-off-by: Masanari Iida > --- > drivers/staging/greybus/authentication.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/staging/greybus/authentication.c b/drivers/staging/greybus/authentication.c > index 168626ba0c03..6c5dcb1c226b 100644 > --- a/drivers/staging/greybus/authentication.c > +++ b/drivers/staging/greybus/authentication.c > @@ -16,7 +16,6 @@ > > #include "greybus_authentication.h" > #include "firmware.h" > -#include "greybus.h" Please remove the other "greybus.h" include that precedes the "system" includes instead. > > #define CAP_TIMEOUT_MS 1000 Also add a component prefix to your subject (i.e. "staging: greybus: authentication: remove..."). Thanks, Johan