Re: [lng-odp] [PATCH] doc: use only WITH_PLATFORM and not always linux-generic

2015-06-08 Thread Nicolas Morey-Chaisemartin
What we did to avoid pointing into the linux-generic folder in the Makefiles is use symlinks to linux-generic sources. This way we keep the linux-generic files up-to-date when pulling but avoid build dependencies between platform folders. On 06/06/2015 12:22 AM, Bill Fischofer wrote: The

Re: [lng-odp] [PATCH] doc: use only WITH_PLATFORM and not always linux-generic

2015-06-08 Thread BenoƮt Ganne
The linux-generic platform is indeed used as a base class however given this it would seem that the include order should be reversed so that $(WITH_PLATFORM) is first. This would have the effect of allowing it to override any linux-generic include files of the same name. I dont think it will

Re: [lng-odp] [PATCH] doc: use only WITH_PLATFORM and not always linux-generic

2015-06-08 Thread Mike Holmes
On 5 June 2015 at 08:22, Nicolas Morey-Chaisemartin nmo...@kalray.eu wrote: Signed-off-by: Nicolas Morey-Chaisemartin nmo...@kalray.eu Reviewed-by: Mike Holmes mike.holmes@l;inaro.org I think we should not impose something that is affecting others, derived platforms can of course add this

Re: [lng-odp] [PATCH] doc: use only WITH_PLATFORM and not always linux-generic

2015-06-06 Thread Mike Holmes
On 5 June 2015 at 18:22, Bill Fischofer bill.fischo...@linaro.org wrote: The linux-generic platform is indeed used as a base class however given this it would seem that the include order should be reversed so that $(WITH_PLATFORM) is first. This would have the effect of allowing it to

[lng-odp] [PATCH] doc: use only WITH_PLATFORM and not always linux-generic

2015-06-05 Thread Nicolas Morey-Chaisemartin
Signed-off-by: Nicolas Morey-Chaisemartin nmo...@kalray.eu --- doc/doxygen.cfg | 1 - 1 file changed, 1 deletion(-) diff --git a/doc/doxygen.cfg b/doc/doxygen.cfg index 6283387..213a903 100644 --- a/doc/doxygen.cfg +++ b/doc/doxygen.cfg @@ -13,7 +13,6 @@ WARN_NO_PARAMDOC = YES INPUT =

Re: [lng-odp] [PATCH] doc: use only WITH_PLATFORM and not always linux-generic

2015-06-05 Thread Mike Holmes
This works for linux-generic, but derived platforms such as odp-dpdk depend on this link back to linux-generic becasue they inherit from it. Having this in linux-generic makes it simpler for the derived works, but I could also see the argument that it should be removed in linux-generic and added

Re: [lng-odp] [PATCH] doc: use only WITH_PLATFORM and not always linux-generic

2015-06-05 Thread Bill Fischofer
The linux-generic platform is indeed used as a base class however given this it would seem that the include order should be reversed so that $(WITH_PLATFORM) is first. This would have the effect of allowing it to override any linux-generic include files of the same name. On Fri, Jun 5, 2015 at