From: Ingo Molnar <mingo@elte.hu>
To: Mauro Carvalho Chehab <mchehab@infradead.org>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [patch] dvb: build failure fix
Date: Mon, 28 Apr 2008 22:30:56 +0200 [thread overview]
Message-ID: <20080428203056.GI15840@elte.hu> (raw)
In-Reply-To: <20080428164730.43349c3d@gaivota>
* Mauro Carvalho Chehab <mchehab@infradead.org> wrote:
> On Mon, 28 Apr 2008 18:24:21 +0200
> Ingo Molnar <mingo@elte.hu> wrote:
>
> >
> > fix build failure found via x86.git randconfig testing:
> >
> > drivers/built-in.o: In function `tda829x_attach':
> > : undefined reference to `tda827x_attach'
> > drivers/built-in.o: In function `tda829x_attach':
> > : undefined reference to `tda18271_attach'
>
> > config TUNER_XC2028
> > config TUNER_MT20XX
> > config TUNER_TEA5761
> > config TUNER_TEA5767
> > config TUNER_SIMPLE
> > config TUNER_TDA9887
>
> Hmm... if the issues are with tda827x and tda18271, why have you
> touched on the above drivers? I can't see why the above tuners would
> need DVB.
ah, ok. I just copy & pasted to everything that had "depends on I2C".
> > config TUNER_TDA8290
> > tristate "TDA 8290/8295 + 8275(a)/18271 tuner combo"
> > - depends on I2C
> > + depends on I2C && ((DVB_CORE = m && m) || (DVB_CORE = y))
> > select DVB_TDA827X
> > select DVB_TDA18271
> > default m if VIDEO_TUNER_CUSTOMIZE
>
> Ok, this one may be broken.
>
> I'm working on some changes that will hopefully fix this issue at the
> proper way.
ok, great. FYI, this has been reported months ago too so it's a
long-time breakage - would be nice to fix it upstream even if it's the
initial fix is a band-aid.
Ingo
next prev parent reply other threads:[~2008-04-28 20:31 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-04-28 16:24 [patch] dvb: build failure fix Ingo Molnar
2008-04-28 18:04 ` Adrian Bunk
2008-04-28 19:53 ` Ingo Molnar
2008-04-28 21:01 ` Adrian Bunk
2008-04-28 21:25 ` Mauro Carvalho Chehab
2008-04-28 21:29 ` mkrufky
2008-04-29 12:14 ` Ingo Molnar
2008-04-30 19:17 ` Mauro Carvalho Chehab
2008-04-30 19:28 ` Adrian Bunk
2008-04-30 19:31 ` mkrufky
2008-04-30 19:52 ` Mauro Carvalho Chehab
2008-04-30 20:01 ` Adrian Bunk
2008-04-30 22:32 ` mkrufky
2008-04-28 19:47 ` Mauro Carvalho Chehab
2008-04-28 20:30 ` Ingo Molnar [this message]
2008-04-28 21:03 ` Mauro Carvalho Chehab
2008-04-28 21:17 ` Ingo Molnar
2008-04-28 21:20 ` Ingo Molnar
2008-04-28 21:23 ` Ingo Molnar
2008-04-28 21:25 ` Ingo Molnar
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=20080428203056.GI15840@elte.hu \
--to=mingo@elte.hu \
--cc=linux-kernel@vger.kernel.org \
--cc=mchehab@infradead.org \
/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.