From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751828AbcAEJhZ (ORCPT ); Tue, 5 Jan 2016 04:37:25 -0500 Received: from userp1040.oracle.com ([156.151.31.81]:23474 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751456AbcAEJhU (ORCPT ); Tue, 5 Jan 2016 04:37:20 -0500 Date: Tue, 5 Jan 2016 12:36:39 +0300 From: Dan Carpenter To: Samuel Thibault , William Hubbs , Chris Brannon , Kirk Reiser , Melike Yurtoglu , Greg Kroah-Hartman , speakup@linux-speakup.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] Staging: speakup: Fix getting port information Message-ID: <20160105093638.GA23363@mwanda> References: <20160105011912.GM3232@var.home> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160105011912.GM3232@var.home> User-Agent: Mutt/1.5.21 (2010-09-15) X-Source-IP: aserv0022.oracle.com [141.146.126.234] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jan 05, 2016 at 02:19:12AM +0100, Samuel Thibault wrote: > --- a/drivers/staging/speakup/serialio.c > +++ b/drivers/staging/speakup/serialio.c > @@ -6,6 +6,9 @@ > #include "spk_priv.h" > #include "serialio.h" > > +#include > +#include > + I'm sorry to do this but can you add a comment here, otherwise someone is going to just change it back because it causes a checkpatch.pl warning. Make it a big ugly warning. #include /* WARNING: Do not change this to without testing. */ #include regards, dan carpenter