From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id F03081BBE for ; Thu, 10 Mar 2016 16:22:07 +0100 (CET) Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga104.fm.intel.com with ESMTP; 10 Mar 2016 07:22:06 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,316,1455004800"; d="scan'208";a="761691648" Received: from dhunt5-mobl.ger.corp.intel.com (HELO [10.237.221.39]) ([10.237.221.39]) by orsmga003.jf.intel.com with ESMTP; 10 Mar 2016 07:22:05 -0800 To: Jan Medala , dev@dpdk.org References: <1457381110-20382-1-git-send-email-jan@semihalf.com> <1457381110-20382-5-git-send-email-jan@semihalf.com> From: "Hunt, David" Message-ID: <56E1911C.3080803@intel.com> Date: Thu, 10 Mar 2016 15:22:04 +0000 User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 MIME-Version: 1.0 In-Reply-To: <1457381110-20382-5-git-send-email-jan@semihalf.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Cc: Evgeny Schemeilin , matua@amazon.com Subject: Re: [dpdk-dev] [PATCH v5 4/4] ena: DPDK polling-mode driver for Amazon Elastic Network Adapters (ENA) X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 10 Mar 2016 15:22:08 -0000 Hi Jan, On 3/7/2016 8:05 PM, Jan Medala wrote: --snip-- > diff --git a/drivers/net/ena/ena_ethdev.h b/drivers/net/ena/ena_ethdev.h > new file mode 100644 > index 0000000..7f069b9 > --- /dev/null > +++ b/drivers/net/ena/ena_ethdev.h > @@ -0,0 +1,154 @@ > +/*- > +* BSD LICENSE > +* > +* Copyright (c) 2015-2016 Amazon.com, Inc. or its affiliates. > +* All rights reserved. > +* > +* Redistribution and use in source and binary forms, with or without > +* modification, are permitted provided that the following conditions > +* are met: > +* > +* * Redistributions of source code must retain the above copyright > +* notice, this list of conditions and the following disclaimer. > +* * Redistributions in binary form must reproduce the above copyright > +* notice, this list of conditions and the following disclaimer in > +* the documentation and/or other materials provided with the > +* distribution. > +* * Neither the name of copyright holder nor the names of its > +* contributors may be used to endorse or promote products derived > +* from this software without specific prior written permission. > +* > +* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS > +* "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT > +* LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR > +* A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT > +* OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, > +* SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT > +* LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, > +* DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY > +* THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT > +* (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE > +* OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. > +*/ > + > +#ifndef _ENA_ETHDEV_H_ > +#define _ENA_ETHDEV_H_ > + > +#include > + > +#pragma GCC diagnostic ignored "-Wcast-qual" Is this #pragma really needed here? My setup compiles fine without warnings, and I reckon it's better not have this line if it's not needed just in case we inadvertently introduce additional warnings in the future. ---snip-- The only other thing I'd note with the rest of the patch-set is the output from checkpatch, which Bruce has already mentioned. Regards, David.