[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20190810064611.GC13020@ravnborg.org>
Date: Sat, 10 Aug 2019 08:46:11 +0200
From: Sam Ravnborg <sam@...nborg.org>
To: Masahiro Yamada <yamada.masahiro@...ionext.com>
Cc: linux-kbuild@...r.kernel.org, Jani Nikula <jani.nikula@...el.com>,
intel-gfx@...ts.freedesktop.org, Daniel Vetter <daniel@...ll.ch>,
David Airlie <airlied@...ux.ie>,
Jani Nikula <jani.nikula@...ux.intel.com>,
Joonas Lahtinen <joonas.lahtinen@...ux.intel.com>,
Michal Marek <michal.lkml@...kovi.net>,
Rodrigo Vivi <rodrigo.vivi@...el.com>,
Zhenyu Wang <zhenyuw@...ux.intel.com>,
Zhi Wang <zhi.a.wang@...el.com>,
dri-devel@...ts.freedesktop.org,
intel-gvt-dev@...ts.freedesktop.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/5] kbuild: allow big modules to sub-divide Makefiles
Hi Masahiro
On Tue, Aug 06, 2019 at 03:39:18PM +0900, Masahiro Yamada wrote:
>
> Recently, Jani Nikula requests a better build system support
> for drivers spanning multiple directories.
> (better kbuild support for drivers spanning multiple directories?)
>
> I implemented it, so please take a look at it.
>
> Note:
> The single targets do not work correctly.
>
> The single targets have never worked correctly:
It works in most cases, but now always.
I dunno how much it is used.
Myself I almost always do make /drivers/foo/bar/
>
> [1] For instance, "make drivers/foo/bar/baz.o" will descend into
> drivers/foo/bar/Makefile, which may not necessarily specify
> the build rule of baz.o
>
> It is possible for drivers/foo/Makefile having
> obj-$(CONFIG_BAZ) += bar/baz.o
>
> [2] subdir-ccflags-y does not work.
>
> The single targets directly descend into the directory of
> that file resides.
>
> It missed subdir-ccflags-y if it is specifies in parent
> Makefiles.
>
> Perhaps, I will have to manage correct implementation of single targets.
The day that kbuild has a separate step to read all Makefiles
and then without using recursive make can build the kernel we can have
this fixed.
Until then we can accpet it as is - as fixing this may not be simple.
Sam
Powered by blists - more mailing lists