From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by dpdk.org (Postfix) with ESMTP id 96B8C1B1B0 for ; Tue, 19 Sep 2017 16:15:52 +0200 (CEST) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga101.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 19 Sep 2017 07:15:50 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.42,418,1500966000"; d="scan'208";a="901846870" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.57]) ([10.237.220.57]) by FMSMGA003.fm.intel.com with ESMTP; 19 Sep 2017 07:15:48 -0700 To: Shreyansh Jain Cc: dev@dpdk.org, hemant.agrawal@nxp.com, Thomas Monjalon References: <20170823141213.25476-1-shreyansh.jain@nxp.com> <20170909112132.13936-1-shreyansh.jain@nxp.com> <20170909112132.13936-5-shreyansh.jain@nxp.com> <01df3c65-e7dc-b2b8-a188-dd88018010dc@intel.com> From: Ferruh Yigit Message-ID: <9142ad53-f75b-de33-cb8d-51ae5a781a2e@intel.com> Date: Tue, 19 Sep 2017 15:15:47 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH v4 04/41] bus/dpaa: add OF parser for device scanning 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: , X-List-Received-Date: Tue, 19 Sep 2017 14:15:53 -0000 On 9/19/2017 2:37 PM, Shreyansh Jain wrote: > On Monday 18 September 2017 08:19 PM, Ferruh Yigit wrote: >> On 9/9/2017 12:20 PM, Shreyansh Jain wrote: >>> This layer is used by Bus driver's scan function. Devices are parsed >>> using OF parser and added to DPAA device list. >> >> So this is device tree parser in DPDK. Do we really want this, and as >> long as DPDK target the bare metal why not get device information from >> Linux, as done in other cases? > As of now I don't prefer to modify the internal framework as much as > possible as this is stable DPDK DPAA driver. > There is indeed a planned transition from OF to /sys/ parsing, but it is > still in pipeline. > > You see a blocking issue if we go incremental here? > That would be probably more of replacing this file with another /sys > parser without much changes to the DPDK glue code. OF parser in DPDK looks weird to me, OS will do this for us already. If replacing this is in the roadmap, I think this is not showstopper, added Thomas in case he thinks otherwise. > >> >>> >>> Signed-off-by: Geoff Thorpe >>> Signed-off-by: Shreyansh Jain >>> --- >>> drivers/bus/dpaa/Makefile | 7 + >>> drivers/bus/dpaa/base/fman/of.c | 576 ++++++++++++++++++++++++++++++++++++++++ >>> drivers/bus/dpaa/include/of.h | 190 +++++++++++++ >>> 3 files changed, 773 insertions(+) >>> create mode 100644 drivers/bus/dpaa/base/fman/of.c >>> create mode 100644 drivers/bus/dpaa/include/of.h >>> >> >> <...> >> >> >