Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-12 Thread Tom Rini
On Thu, Nov 12, 2015 at 09:56:26AM -0700, Simon Glass wrote: > Hi Nishanth, > > On 12 November 2015 at 09:45, Nishanth Menon wrote: > > On 11/12/2015 10:22 AM, Tom Rini wrote: > >> On Thu, Nov 12, 2015 at 09:16:00AM -0700, Simon Glass wrote: > >>> Hi, > >>> > >>> On 10 November 2015 at 08:34, Nis

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-12 Thread Simon Glass
Hi Nishanth, On 12 November 2015 at 09:45, Nishanth Menon wrote: > On 11/12/2015 10:22 AM, Tom Rini wrote: >> On Thu, Nov 12, 2015 at 09:16:00AM -0700, Simon Glass wrote: >>> Hi, >>> >>> On 10 November 2015 at 08:34, Nishanth Menon wrote: On 11/10/2015 08:55 AM, Nishanth Menon wrote: >

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-12 Thread Nishanth Menon
On 11/12/2015 10:22 AM, Tom Rini wrote: > On Thu, Nov 12, 2015 at 09:16:00AM -0700, Simon Glass wrote: >> Hi, >> >> On 10 November 2015 at 08:34, Nishanth Menon wrote: >>> On 11/10/2015 08:55 AM, Nishanth Menon wrote: On 12:25-20151110, Masahiro Yamada wrote: > 2015-11-10 5:24 GMT+09:00 S

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-12 Thread Tom Rini
On Thu, Nov 12, 2015 at 09:16:00AM -0700, Simon Glass wrote: > Hi, > > On 10 November 2015 at 08:34, Nishanth Menon wrote: > > On 11/10/2015 08:55 AM, Nishanth Menon wrote: > >> On 12:25-20151110, Masahiro Yamada wrote: > >>> 2015-11-10 5:24 GMT+09:00 Simon Glass : > >> I am unhappy because I

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-12 Thread Simon Glass
Hi, On 10 November 2015 at 08:34, Nishanth Menon wrote: > On 11/10/2015 08:55 AM, Nishanth Menon wrote: >> On 12:25-20151110, Masahiro Yamada wrote: >>> 2015-11-10 5:24 GMT+09:00 Simon Glass : >> I am unhappy because I was hoping >> we could stop creating symbolic links during building >>

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-10 Thread Nishanth Menon
On 11/10/2015 08:55 AM, Nishanth Menon wrote: > On 12:25-20151110, Masahiro Yamada wrote: >> 2015-11-10 5:24 GMT+09:00 Simon Glass : > I am unhappy because I was hoping > we could stop creating symbolic links during building > in a long run. >>> >>> But how? I don't see that it is possi

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-10 Thread Nishanth Menon
On 12:25-20151110, Masahiro Yamada wrote: > 2015-11-10 5:24 GMT+09:00 Simon Glass : > >>> I am unhappy because I was hoping > >>> we could stop creating symbolic links during building > >>> in a long run. > > > > But how? I don't see that it is possible if we want to have a sensible > > prefix for

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-09 Thread Masahiro Yamada
2015-11-10 5:24 GMT+09:00 Simon Glass : >>> I am unhappy because I was hoping >>> we could stop creating symbolic links during building >>> in a long run. > > But how? I don't see that it is possible if we want to have a sensible > prefix for each include. [step 1] move SoC-specific headers to a

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-09 Thread Simon Glass
Hi Masahiro, On 9 November 2015 at 09:37, Nishanth Menon wrote: > On 11/09/2015 10:03 AM, Masahiro Yamada wrote: >> 2015-11-09 1:38 GMT+09:00 Tom Rini : >>> On Sun, Nov 08, 2015 at 09:56:18AM -0600, menon.nisha...@gmail.com wrote: On Sat, Nov 7, 2015 at 8:18 PM, Tom Rini wrote: > C

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-09 Thread Nishanth Menon
On 11/09/2015 10:03 AM, Masahiro Yamada wrote: > 2015-11-09 1:38 GMT+09:00 Tom Rini : >> On Sun, Nov 08, 2015 at 09:56:18AM -0600, menon.nisha...@gmail.com wrote: >>> On Sat, Nov 7, 2015 at 8:18 PM, Tom Rini wrote: >>> Conceptually fine with me. Do we really need a config option for it

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-09 Thread Masahiro Yamada
2015-11-09 1:38 GMT+09:00 Tom Rini : > On Sun, Nov 08, 2015 at 09:56:18AM -0600, menon.nisha...@gmail.com wrote: >> On Sat, Nov 7, 2015 at 8:18 PM, Tom Rini wrote: >> >> > Conceptually fine with me. Do we really need a config option for it >> > tho? >> >> I was following the lines of CONFIG_CREAT

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-08 Thread Tom Rini
On Sun, Nov 08, 2015 at 09:56:18AM -0600, menon.nisha...@gmail.com wrote: > On Sat, Nov 7, 2015 at 8:18 PM, Tom Rini wrote: > > > Conceptually fine with me. Do we really need a config option for it > > tho? > > I was following the lines of CONFIG_CREATE_ARCH_SYMLINK here. Is there > some reason

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-08 Thread menon.nisha...@gmail.com
On Sat, Nov 7, 2015 at 8:18 PM, Tom Rini wrote: > Conceptually fine with me. Do we really need a config option for it > tho? I was following the lines of CONFIG_CREATE_ARCH_SYMLINK here. Is there some reason why you'd not want it so? Regards, Nishanth Menon

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-07 Thread Tom Rini
On Fri, Nov 06, 2015 at 03:58:17PM -0800, Simon Glass wrote: > Hi, > > On 6 November 2015 at 10:23, Nishanth Menon wrote: > > On 11:10-20151106, Nishanth Menon wrote: > >> On 11/05/2015 10:50 PM, Masahiro Yamada wrote: > >> > 2015-11-06 12:30 GMT+09:00 Nishanth Menon : > >> >> On Thu, Nov 5, 2015

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-06 Thread Simon Glass
Hi, On 6 November 2015 at 10:23, Nishanth Menon wrote: > On 11:10-20151106, Nishanth Menon wrote: >> On 11/05/2015 10:50 PM, Masahiro Yamada wrote: >> > 2015-11-06 12:30 GMT+09:00 Nishanth Menon : >> >> On Thu, Nov 5, 2015 at 6:15 PM, Simon Glass wrote: >> >>> Hi, >> >>> >> >>> On 5 November 201

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-06 Thread Nishanth Menon
On 11:10-20151106, Nishanth Menon wrote: > On 11/05/2015 10:50 PM, Masahiro Yamada wrote: > > 2015-11-06 12:30 GMT+09:00 Nishanth Menon : > >> On Thu, Nov 5, 2015 at 6:15 PM, Simon Glass wrote: > >>> Hi, > >>> > >>> On 5 November 2015 at 00:32, Nishanth Menon wrote: > On 11/05/2015 01:28 AM,

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-06 Thread Nishanth Menon
On 11/05/2015 10:50 PM, Masahiro Yamada wrote: > 2015-11-06 12:30 GMT+09:00 Nishanth Menon : >> On Thu, Nov 5, 2015 at 6:15 PM, Simon Glass wrote: >>> Hi, >>> >>> On 5 November 2015 at 00:32, Nishanth Menon wrote: On 11/05/2015 01:28 AM, Nishanth Menon wrote: > When the vendor common lib

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-05 Thread Masahiro Yamada
2015-11-06 12:30 GMT+09:00 Nishanth Menon : > On Thu, Nov 5, 2015 at 6:15 PM, Simon Glass wrote: >> Hi, >> >> On 5 November 2015 at 00:32, Nishanth Menon wrote: >>> On 11/05/2015 01:28 AM, Nishanth Menon wrote: When the vendor common libraries exists, then board should be able to refere

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-05 Thread Nishanth Menon
On Thu, Nov 5, 2015 at 6:15 PM, Simon Glass wrote: > Hi, > > On 5 November 2015 at 00:32, Nishanth Menon wrote: >> On 11/05/2015 01:28 AM, Nishanth Menon wrote: >>> When the vendor common libraries exists, then board should be able to >>> reference headers located there, rather than having to do

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-05 Thread Simon Glass
Hi, On 5 November 2015 at 00:32, Nishanth Menon wrote: > On 11/05/2015 01:28 AM, Nishanth Menon wrote: >> When the vendor common libraries exists, then board should be able to >> reference headers located there, rather than having to do weird logic >> such as '#include "../common/xyz.h"'. >> >> S

Re: [U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-04 Thread Nishanth Menon
On 11/05/2015 01:28 AM, Nishanth Menon wrote: > When the vendor common libraries exists, then board should be able to > reference headers located there, rather than having to do weird logic > such as '#include "../common/xyz.h"'. > > Signed-off-by: Nishanth Menon > --- > > Makefile

[U-Boot] [PATCH] Makefile: Include vendor common library in include search path

2015-11-04 Thread Nishanth Menon
When the vendor common libraries exists, then board should be able to reference headers located there, rather than having to do weird logic such as '#include "../common/xyz.h"'. Signed-off-by: Nishanth Menon --- Makefile| 1 + board/ti/am57xx/board.c | 2 +- 2 files changed, 2 i