linux-next: upstream build failure: v4l/dvb

linux-next: upstream build failure: v4l/dvb

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 615 bytes --]

Hi all,

Today's linux-next build (Linus' tree, x86_64 allmodconfig) fails like
this:

ERROR: "tea5767_autodetection" [drivers/media/video/tuner.ko] undefined!
ERROR: "tda829x_probe" [drivers/media/video/tuner.ko] undefined!
ERROR: "tea5761_autodetection" [drivers/media/video/tuner.ko] undefined!

Reverting commit 149ef72deeba57078216c9fa678baff392295853 ("Rename common
tuner Kconfig names to use the same") makes the build work again.

Also, note that this commit has no Signed-off-by ...

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: linux-next: upstream build failure: v4l/dvb

[Mauro Carvalho Chehab]

Hi Stephen,

On Wed, 30 Apr 2008 11:41:24 +1000
Stephen Rothwell <sfr@canb.auug.org.au> wrote:

> Hi all,
> 
> Today's linux-next build (Linus' tree, x86_64 allmodconfig) fails like
> this:
> 
> ERROR: "tea5767_autodetection" [drivers/media/video/tuner.ko] undefined!
> ERROR: "tda829x_probe" [drivers/media/video/tuner.ko] undefined!
> ERROR: "tea5761_autodetection" [drivers/media/video/tuner.ko] undefined!

I suspect that something wrong happened on your compilation, or at linux-next
merge. Those three drivers (tea5767, tea5761 and tea829x) were moved
from /drivers/media/video to /drivers/common/tuners, but the above messages are
pointing to the old path.
 
> Also, note that this commit has no Signed-off-by ...

Ops. Sorry. How could I forgot to sign a patch!

Just for the records: Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>


Cheers,
Mauro

linux-next: v4l-dvb merge conflicts (Re: linux-next: upstream build failure: v4l/dvb)

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 1702 bytes --]

Hi Mauro,

On Tue, 29 Apr 2008 23:00:40 -0300 Mauro Carvalho Chehab <mchehab@infradead.org> wrote:
>
> On Wed, 30 Apr 2008 11:41:24 +1000
> Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> 
> > Hi all,
> > 
> > Today's linux-next build (Linus' tree, x86_64 allmodconfig) fails like
> > this:
> > 
> > ERROR: "tea5767_autodetection" [drivers/media/video/tuner.ko] undefined!
> > ERROR: "tda829x_probe" [drivers/media/video/tuner.ko] undefined!
> > ERROR: "tea5761_autodetection" [drivers/media/video/tuner.ko] undefined!
> 
> I suspect that something wrong happened on your compilation, or at linux-next
> merge. Those three drivers (tea5767, tea5761 and tea829x) were moved
> from /drivers/media/video to /drivers/common/tuners, but the above messages are
> pointing to the old path.

OK, interesting.  This was a build of just Linus' tree so others may get
this problem if they don't do a "make clean" or similar (as I don't).

I have now done the merge of the v4l-dvb tree in linux-next and I have had several conflicts, the worst being in the file drivers/media/Kconfig between commit 7c91f0624a9a2b8b9b122cf94fef34bc7f7347a6 ("V4L/DVB(7767): Move tuners to common/tuners") in Linus' tree and commit 9c76e847539ab7a73e82ff692862f3de9fa19ddb ("V4L/DVB(7767): Move tuners to common/tuners") in the v4l-dvb tree.  The former leaves

source "drivers/media/common/Kconfig"

while the latter removes it.  I suspect I have a bad version of the
v4l-dvb/stable tree (though I did fetch it this morning).

I managed to fix it all up to the point that it builds, though.
-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: linux-next: v4l-dvb merge conflicts (Re: linux-next: upstream build failure: v4l/dvb)

[Mauro Carvalho Chehab]

On Wed, 30 Apr 2008 13:49:50 +1000
Stephen Rothwell <sfr@canb.auug.org.au> wrote:

> Hi Mauro,
> 
> On Tue, 29 Apr 2008 23:00:40 -0300 Mauro Carvalho Chehab <mchehab@infradead.org> wrote:
> >
> > On Wed, 30 Apr 2008 11:41:24 +1000
> > Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> > 
> > > Hi all,
> > > 
> > > Today's linux-next build (Linus' tree, x86_64 allmodconfig) fails like
> > > this:
> > > 
> > > ERROR: "tea5767_autodetection" [drivers/media/video/tuner.ko] undefined!
> > > ERROR: "tda829x_probe" [drivers/media/video/tuner.ko] undefined!
> > > ERROR: "tea5761_autodetection" [drivers/media/video/tuner.ko] undefined!
> > 
> > I suspect that something wrong happened on your compilation, or at linux-next
> > merge. Those three drivers (tea5767, tea5761 and tea829x) were moved
> > from /drivers/media/video to /drivers/common/tuners, but the above messages are
> > pointing to the old path.
> 
> OK, interesting.  This was a build of just Linus' tree so others may get
> this problem if they don't do a "make clean" or similar (as I don't).

We've found some bugs that may explain what you've got. Anyway, most of they
are fixed. I'm preparing a pull request for Linus later today.

> I have now done the merge of the v4l-dvb tree in linux-next and I have had several conflicts, the worst being in the file drivers/media/Kconfig between commit 7c91f0624a9a2b8b9b122cf94fef34bc7f7347a6 ("V4L/DVB(7767): Move tuners to common/tuners") in Linus' tree and commit 9c76e847539ab7a73e82ff692862f3de9fa19ddb ("V4L/DVB(7767): Move tuners to common/tuners") in the v4l-dvb tree. 

The same patch were on both Linus and my tree. -git sometimes don't like
this ;) 

Anyway, I've already dropped those patches that were already applied from my
tree. You should get a clean merge tomorrow.

Cheers,
Mauro

Re: linux-next: v4l-dvb merge conflicts (Re: linux-next: upstream build failure: v4l/dvb)

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 354 bytes --]

Hi Mauro,

On Wed, 30 Apr 2008 16:12:24 -0300 Mauro Carvalho Chehab <mchehab@infradead.org> wrote:
> 
> Anyway, I've already dropped those patches that were already applied from my
> tree. You should get a clean merge tomorrow.

Thanks.

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]