On 11/11/20 11:00 AM, jrun wrote: > On Wed, Nov 11, 2020 at 10:09:18AM -0800, Randy Dunlap wrote: >> Hi, >> >> Is this a speed of build issue? > > i wasn't thinking about that but sure it does have that effect too. > >> Yes, make is descending into multiple directories that are not >> strictly needed in this kernel config: >> >> AR drivers/gpu/drm/arm/built-in.a >> AR drivers/gpu/drm/rcar-du/built-in.a >> AR drivers/gpu/drm/omapdrm/dss/built-in.a >> AR drivers/gpu/drm/omapdrm/displays/built-in.a >> AR drivers/gpu/drm/omapdrm/built-in.a >> AR drivers/gpu/drm/tilcdc/built-in.a >> AR drivers/gpu/drm/imx/built-in.a >> AR drivers/gpu/drm/i2c/built-in.a >> AR drivers/gpu/drm/panel/built-in.a >> AR drivers/gpu/drm/bridge/analogix/built-in.a >> AR drivers/gpu/drm/bridge/cadence/built-in.a >> AR drivers/gpu/drm/bridge/synopsys/built-in.a >> AR drivers/gpu/drm/bridge/built-in.a >> AR drivers/gpu/drm/hisilicon/built-in.a >> AR drivers/gpu/drm/tiny/built-in.a >> AR drivers/gpu/drm/xlnx/built-in.a >> AR drivers/gpu/drm/built-in.a >> >> The built-in.a files that I see being built are all of size 8 bytes. > > and they don't make it into the final image which is what i wanted. thanks. > > now i would interested to know why some subsystems have those obj-y's > unconditionally asserted? > for instance, would something like this make sense? > > diff --git a/drivers/video/Makefile b/drivers/video/Makefile > index df7650adede9..47508d8049d9 100644 > --- a/drivers/video/Makefile > +++ b/drivers/video/Makefile > @@ -7,7 +7,7 @@ obj-$(CONFIG_FB_STI) += console/ > obj-$(CONFIG_LOGO) += logo/ > obj-y += backlight/ > > -obj-y += fbdev/ > +obj-$(CONFIG_FB) += fbdev/ > > obj-$(CONFIG_VIDEOMODE_HELPERS) += display_timing.o videomode.o > ifeq ($(CONFIG_OF),y) > > or this: > > diff --git a/drivers/gpu/Makefile b/drivers/gpu/Makefile > index 835c88318cec..c1a71f4938bb 100644 > --- a/drivers/gpu/Makefile > +++ b/drivers/gpu/Makefile > @@ -3,6 +3,7 @@ > # taken to initialize them in the correct order. Link order is the only way > # to ensure this currently. > obj-$(CONFIG_TEGRA_HOST1X) += host1x/ > -obj-y += drm/ vga/ > +obj-y += vga/ > +obj-$(CONFIG_DRM) += drm/ > obj-$(CONFIG_IMX_IPUV3_CORE) += ipu-v3/ > obj-$(CONFIG_TRACE_GPU_MEM) += trace/ Yeah, that may be the right thing to do, but it would take a lot of build testing. My laptops don't have enough horsepower for all of that (multiple configs * multiple arch-es). -- ~Randy
