From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id 328B11B1BE for ; Mon, 18 Sep 2017 16:49:17 +0200 (CEST) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 18 Sep 2017 07:49:15 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.42,413,1500966000"; d="scan'208";a="1173318884" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.57]) ([10.237.220.57]) by orsmga001.jf.intel.com with ESMTP; 18 Sep 2017 07:49:14 -0700 To: Shreyansh Jain , dev@dpdk.org Cc: hemant.agrawal@nxp.com References: <20170823141213.25476-1-shreyansh.jain@nxp.com> <20170909112132.13936-1-shreyansh.jain@nxp.com> <20170909112132.13936-5-shreyansh.jain@nxp.com> From: Ferruh Yigit Message-ID: <01df3c65-e7dc-b2b8-a188-dd88018010dc@intel.com> Date: Mon, 18 Sep 2017 15:49:13 +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: <20170909112132.13936-5-shreyansh.jain@nxp.com> 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: Mon, 18 Sep 2017 14:49:17 -0000 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? > > 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 > <...>