All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Backlund <tmb@mandriva.org>
To: Stephen Rothwell <sfr@canb.auug.org.au>
Cc: Linus <torvalds@linux-foundation.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	linux-next@vger.kernel.org,
	Mauro Carvalho Chehab <mchehab@infradead.org>,
	Ingo Molnar <mingo@elte.hu>,
	linux-dvb-maintainer@linuxtv.org, video4linux-list@redhat.com,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: [patch, -git] drivers/media build fix for modular builds
Date: Thu, 08 May 2008 17:21:47 +0300	[thread overview]
Message-ID: <48230C7B.90007@mandriva.org> (raw)
In-Reply-To: <20080507134907.4417708b.sfr@canb.auug.org.au>

Stephen Rothwell skrev:
> Date:	Wed, 30 Apr 2008 09:48:07 +0200
> From:	Ingo Molnar <mingo@elte.hu>
> To:	Mauro Carvalho Chehab <mchehab@infradead.org>
> Cc:	Linus Torvalds <torvalds@linux-foundation.org>,
> 	Andrew Morton <akpm@linux-foundation.org>,
> 	linux-dvb-maintainer@linuxtv.org, video4linux-list@redhat.com,
> 	linux-kernel@vger.kernel.org
> 
> fix allmodconfig build bug introduced in latest -git by commit 
> 7c91f0624a9 ("V4L/DVB(7767): Move tuners to common/tuners"):
> 
>   LD      kernel/built-in.o
>   LD      drivers/built-in.o
>   ld: drivers/media/built-in.o: No such file: No such file or directory
> 
> which happens if all media drivers are modular:
> 
>   http://redhat.com/~mingo/misc/config-Wed_Apr_30_09_24_48_CEST_2008.bad
> 
> In that case there's no obj-y rule connecting all the built-in.o files and
> the link tree breaks.
> 
> the fix is to add a guaranteed obj-y rule for the core vmlinux to build.
> (which results in an empty object file if all media drivers are modular)
> 
> Signed-off-by: Ingo Molnar <mingo@elte.hu>
> Acked-by: Sam Ravnborg <sam@ravnborg.org>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
> ---
>  drivers/media/Makefile |    2 ++
>  1 file changed, 2 insertions(+)
> 
> Linus,
> 
> I have been applying a similar patch to linux-next for some time.  It has
> been reported a while ago and does break (at least) powerpc and x86_64
> allmodconfig builds.
> 
> Please apply at you earliest convenience.
> 
> /me tries out his "random-build-fix-monkey" appointment.
> 
> Index: linux/drivers/media/Makefile
> ===================================================================
> --- linux.orig/drivers/media/Makefile
> +++ linux/drivers/media/Makefile
> @@ -2,6 +2,8 @@
>  # Makefile for the kernel multimedia device drivers.
>  #
>  
> +obj-y := common/
> +
>  obj-$(CONFIG_VIDEO_MEDIA) += common/
>  
>  # Since hybrid devices are here, should be compiled if DVB and/or V4L
> 


How about fixing this one already ?

--
Thomas

  reply	other threads:[~2008-05-08 14:47 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-07  3:49 [patch, -git] drivers/media build fix for modular builds Stephen Rothwell
2008-05-08 14:21 ` Thomas Backlund [this message]
2008-05-09  0:16   ` Gene Heskett
2008-05-09  0:18     ` David Miller
2008-05-09  5:57       ` Thomas Backlund
  -- strict thread matches above, loose matches on Subject: below --
2008-04-29 21:50 [GIT PATCHES] V4L/DVB updates and fixes for 2.6.26 Mauro Carvalho Chehab
2008-04-30  7:48 ` [patch, -git] drivers/media build fix for modular builds Ingo Molnar
2008-04-30  7:52   ` David Miller
2008-04-30  8:13     ` Ingo Molnar
2008-04-30  8:27       ` Mike Galbraith
2008-04-30  8:42         ` Mike Galbraith
2008-04-30  8:56           ` Mike Galbraith
2008-04-30  9:20             ` Mike Galbraith
2008-04-30 11:40               ` Andrew Morton
2008-04-30 13:03                 ` Mike Galbraith
2008-04-30 15:25                   ` Mauro Carvalho Chehab
2008-04-30 15:48                     ` Mike Galbraith
2008-04-30 16:12                       ` Mauro Carvalho Chehab
2008-04-30 16:31                         ` Mike Galbraith
2008-04-30 21:35                           ` Mauro Carvalho Chehab
2008-04-30  9:26             ` Sam Ravnborg
2008-04-30  9:40               ` Mike Galbraith
2008-04-30 10:36                 ` Ingo Molnar
2008-04-30  9:25       ` Sam Ravnborg
2008-04-30  9:37         ` Ingo Molnar
2008-04-30  9:55           ` Sam Ravnborg

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=48230C7B.90007@mandriva.org \
    --to=tmb@mandriva.org \
    --cc=akpm@linux-foundation.org \
    --cc=linux-dvb-maintainer@linuxtv.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-next@vger.kernel.org \
    --cc=mchehab@infradead.org \
    --cc=mingo@elte.hu \
    --cc=sfr@canb.auug.org.au \
    --cc=torvalds@linux-foundation.org \
    --cc=video4linux-list@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.