From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jOy78-0001LO-Ka for barebox@lists.infradead.org; Thu, 16 Apr 2020 06:35:08 +0000 Date: Thu, 16 Apr 2020 08:35:04 +0200 From: Sascha Hauer Message-ID: <20200416063504.GS1694@pengutronix.de> References: <20200415075917.54999-1-masahiroy@kernel.org> <20200415094018.GL1694@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH] libfdt: update and to Linux 5.7-rc1 To: Masahiro Yamada Cc: Barebox List On Wed, Apr 15, 2020 at 07:54:09PM +0900, Masahiro Yamada wrote: > On Wed, Apr 15, 2020 at 6:40 PM Sascha Hauer wrote: > > > > On Wed, Apr 15, 2020 at 04:59:17PM +0900, Masahiro Yamada wrote: > > > Fix the relative include path in > > > > > > Including is too much. is enough > > > for INT_MAX. > > > > > > Define INT32_MAX and UINT32_MAX in the same way as Linux does. > > > > > > Signed-off-by: Masahiro Yamada > > > --- > > > > > > include/linux/libfdt.h | 2 +- > > > include/linux/libfdt_env.h | 7 ++++--- > > > 2 files changed, 5 insertions(+), 4 deletions(-) > > > > Applied, thanks. > > > > > > > > diff --git a/include/linux/libfdt.h b/include/linux/libfdt.h > > > index ef2467213..90ed4ebfa 100644 > > > --- a/include/linux/libfdt.h > > > +++ b/include/linux/libfdt.h > > > @@ -3,6 +3,6 @@ > > > #define _INCLUDE_LIBFDT_H_ > > > > > > #include > > > -#include "../scripts/dtc/libfdt/libfdt.h" > > > +#include "../../scripts/dtc/libfdt/libfdt.h" > > > > I wonder why this obviously wrong path worked before. > > > This is because the $(srctree)/include/ > is added to the header search paths. > > > The top Makefile defines it as follows: > > LINUXINCLUDE := -Iinclude -I$(srctree)/dts/include \ > $(if $(KBUILD_SRC), -I$(srctree)/include) \ > -I$(srctree)/arch/$(ARCH)/include \ > -I$(objtree)/arch/$(ARCH)/include \ > -include $(srctree)/include/linux/kconfig.h > > > > The previous "../scripts/dtc/libfdt/libfdt.h" > works because it is relative to $(srctree)/include/ Ah, "file" falls back to when file is not found in the relative path. I tend to forget that. Regards, Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox