From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jon Loeliger Subject: Re: [PATCH v2] dtc: Add -i option to support search paths Date: Sun, 18 Mar 2012 10:53:43 -0500 Message-ID: References: <1331082612-10612-1-git-send-email-sjg@chromium.org> <1331780653-13750-1-git-send-email-sjg@chromium.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-reply-to: <1331780653-13750-1-git-send-email-sjg-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org To: Simon Glass Cc: Devicetree Discuss List-Id: devicetree@vger.kernel.org > It is often inconvenient to place device tree files in the same directory > as their includes, or to specify the full path to include files. > > An example of this is in U-Boot where we have a .dtsi file for each SOC > type, and this is included by the board .dts file. We need to either use > a mechanism like: > > /include/ ARCH_CPU_DTS > > with sed or cpp to perform the replacement with the correct path, or > we must specify the full path in the file: > > /include/ "../../arch/arm/dts/tegra20.dtsi" > > The first option is not desirable since it requires anyone compiling the > file to first pre-process it. The second is not desirable since it > introduces a path which is project-specific into a file which is supposed > to be a hardware description. For example Linux and U-Boot are unlikely to > put these include files in the same place. > > It is much more convenient to specify the search patch on the command line > as is done with C pre-processors, for example. > > Introduce a -i option to add to the list of search paths used to find > source and include files. > > We cannot use -I as it is already in use. Other suggestions welcome. > > Signed-off-by: Simon Glass > --- > Changes in v2: > - Add test for finding a file in the same dir as the last file read > - Fix comment typo > - Move struct search_path to C file Applied. Thanks, jdl