From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by dpdk.space (Postfix) with ESMTP id E507EA0471 for ; Wed, 19 Jun 2019 12:46:00 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 24EC51BF99; Wed, 19 Jun 2019 12:46:00 +0200 (CEST) Received: from smtp.tuxdriver.com (charlotte.tuxdriver.com [70.61.120.58]) by dpdk.org (Postfix) with ESMTP id 991271BF15 for ; Wed, 19 Jun 2019 12:45:58 +0200 (CEST) Received: from [107.15.85.130] (helo=localhost) by smtp.tuxdriver.com with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.63) (envelope-from ) id 1hdY69-0004ZW-4T; Wed, 19 Jun 2019 06:45:54 -0400 Date: Wed, 19 Jun 2019 06:45:43 -0400 From: Neil Horman To: Hemant Agrawal Cc: "dev@dpdk.org" , Jerin Jacob Kollanukkaran , Bruce Richardson , Thomas Monjalon , Shreyansh Jain Message-ID: <20190619104543.GB19712@hmswarspite.think-freely.org> References: <20190525184346.27932-1-nhorman@tuxdriver.com> <20190613142344.9188-1-nhorman@tuxdriver.com> <20190613142344.9188-6-nhorman@tuxdriver.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.11.3 (2019-02-01) X-Spam-Score: -2.9 (--) X-Spam-Status: No Subject: Re: [dpdk-dev] [PATCH v2 05/10] fslmc: identify internal only functions and tag them as __rte_internal X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Mon, Jun 17, 2019 at 07:30:34AM +0000, Hemant Agrawal wrote: > HI Neil, > The same code of low-level HW driver is shared with different OSs (kernel, uboot etc) and different framework. If we introduce rte_internal in these low-level hw files, it will be a big maintenance issue for NXP. Everytime there is a change or upgrade, it will be a pain. > Why? If you don't want to support the use of a project specific tag in other projects, you can add a global cflag to those like this: -D__rte_experimental and the code will be eliminated I'm not sure what you see as difficult here Neil > Regards, > Hemant > > > -----Original Message----- > > From: Neil Horman > > Sent: Thursday, June 13, 2019 7:54 PM > > To: dev@dpdk.org > > Cc: Neil Horman ; Jerin Jacob Kollanukkaran > > ; Bruce Richardson ; > > Thomas Monjalon ; Hemant Agrawal > > ; Shreyansh Jain > > Subject: [PATCH v2 05/10] fslmc: identify internal only functions and tag them > > as __rte_internal > > Importance: High > > > > Identify functions in fslmc bus driver which are internal (based on > > their not having an rte_ prefix) and tag them with __rte_internal > > >