All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@infradead.org>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: Trent Piepho <xyzzy@speakeasy.org>,
	Eduard Huguet <eduardhc@gmail.com>,
	linux-media@vger.kernel.org
Subject: Re: cx8802.ko module not being built with current HG tree
Date: Tue, 10 Feb 2009 18:41:47 -0200	[thread overview]
Message-ID: <20090210184147.61d4655e@pedra.chehab.org> (raw)
In-Reply-To: <200902102132.00114.hverkuil@xs4all.nl>

On Tue, 10 Feb 2009 21:31:59 +0100
Hans Verkuil <hverkuil@xs4all.nl> wrote:

> On Tuesday 10 February 2009 19:47:40 Mauro Carvalho Chehab wrote:
> > On Tue, 10 Feb 2009 10:25:26 -0800 (PST)
> >
> > Trent Piepho <xyzzy@speakeasy.org> wrote:
> > > On Tue, 10 Feb 2009, Eduard Huguet wrote:
> > > >     I don't have yet the buggy config, but the steps I was following
> > > > when I encounter the problem were the following:
> > > >         · hg clone http://linuxtv.org/hg/v4l-dvb
> > > >         · cd v4l-dvb
> > > >         · make menuconfig
> > >
> > > This is what I did too.  Just use the menuconfig or xconfig targets. 
> > > Maybe the kernel kconfig behavior has changed?
> >
> > Hmm... I did a test here with RHEL 2.6.18 kernel:
> >
> > $ make menuconfig
> > make -C /home/v4l/master/v4l menuconfig
> > make[1]: Entrando no diretório `/home/v4l/master/v4l'
> > /usr/src/kernels/2.6.18-125.el5-x86_64//scripts/kconfig/mconf ./Kconfig
> > #
> > # configuration written to .config
> > #
> >
> >
> > *** End of Linux kernel configuration.
> > *** Execute 'make' to build the kernel or try 'make help'.
> >
> > $ grep CX88 v4l/.config
> > CONFIG_VIDEO_CX88=m
> > CONFIG_VIDEO_CX88_ALSA=m
> > CONFIG_VIDEO_CX88_BLACKBIRD=m
> > CONFIG_VIDEO_CX88_DVB=m
> > CONFIG_VIDEO_CX88_MPEG=y
> > CONFIG_VIDEO_CX88_VP3054=m
> >
> > So, I got the buggy .config
> >
> > Another test with 2.6.27:
> >
> > $ make menuconfig
> > make -C /home/v4l/master/v4l menuconfig
> > make[1]: Entrando no diretório `/home/v4l/master/v4l'
> > ./scripts/make_kconfig.pl /usr/src/kernels/v2.6.27.4/
> > /usr/src/kernels/v2.6.27.4/ Preparing to compile for kernel version
> > 2.6.27
> > VIDEO_PXA27x: Requires at least kernel 2.6.29
> > USB_STV06XX: Requires at least kernel 2.6.28
> > /usr/src/kernels/v2.6.27.4.i5400//scripts/kconfig/mconf ./Kconfig
> > #
> > # configuration written to .config
> > #
> >
> >
> > *** End of Linux kernel configuration.
> > *** Execute 'make' to build the kernel or try 'make help'.
> >
> > make[1]: Saindo do diretório `/home/v4l/master/v4l'
> > [v4l@pedra master]$ grep CX88 v4l/.config
> > CONFIG_VIDEO_CX88=m
> > CONFIG_VIDEO_CX88_ALSA=m
> > CONFIG_VIDEO_CX88_BLACKBIRD=m
> > CONFIG_VIDEO_CX88_DVB=m
> > CONFIG_VIDEO_CX88_MPEG=m
> > CONFIG_VIDEO_CX88_VP3054=m
> >
> > With 2.6.27, everything is OK.
> >
> > So, it seems that a fix at some kernel between 2.6.22 and 2.6.27 changed
> > (or fixed) the Kconfig behaviour.
> >
> > I suspect that the better fix for this would be to run something like:
> >
> > cat .config|sed s,'=y','=m'
> >
> > For kernels older than 2.6.27.
> >
> > Maybe Hans can give us a hint on what kernel this issue were solved, with
> > his build environment.
> 
> 2.6.21 is wrong, 2.6.22 is right. Cause: dependency on VIDEOBUF_DMA_SG, 
> which has a dependency on CONFIG_HAS_DMA, which was apparently introduced 
> in 2.6.22 and didn't exist in 2.6.21.
> 
> This is fixed by the attached diff.

No, it didn't fix. I'm still getting this issue with 2.6.18.

Btw, on RHEL5 2.6.18 kernel, HAS_DMA does exist:

$ grep CONFIG_HAS_DMA /usr/src/kernels/2.6.18-125.el5-x86_64/.config 
CONFIG_HAS_DMA=y

Also, the original reporter were for an Ubuntu kernel 2.6.22.
> 
> Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
> 
> BTW, all the scripts I use to setup and run the build environment are 
> available here: http://www.xs4all.nl/%7Ehverkuil/logs/scripts.tar.bz2
> 
> Regards,
> 
> 	Hans
> 




Cheers,
Mauro

  reply	other threads:[~2009-02-10 20:42 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-05 15:54 [linux-dvb] cx8802.ko module not being built with current HG tree Eduard Huguet
2009-02-05 15:59 ` Eduard Huguet
2009-02-05 18:33   ` Trent Piepho
2009-02-05 19:04     ` Eduard Huguet
2009-02-10 11:37   ` Mauro Carvalho Chehab
2009-02-10 11:49     ` Eduard Huguet
2009-02-10 18:25       ` Trent Piepho
2009-02-10 18:47         ` Mauro Carvalho Chehab
2009-02-10 20:31           ` Hans Verkuil
2009-02-10 20:41             ` Mauro Carvalho Chehab [this message]
2009-02-10 21:21               ` Hans Verkuil
2009-02-11  0:17                 ` Mauro Carvalho Chehab
2009-02-11  1:20                   ` Trent Piepho
2009-02-11  7:53                     ` Mauro Carvalho Chehab
2009-02-13 10:47                       ` Trent Piepho
2009-02-10 11:57     ` Eduard Huguet
2009-02-10 12:09       ` Mauro Carvalho Chehab
2009-02-10 12:29         ` Mauro Carvalho Chehab
2009-02-10 12:39         ` Eduard Huguet

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=20090210184147.61d4655e@pedra.chehab.org \
    --to=mchehab@infradead.org \
    --cc=eduardhc@gmail.com \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    --cc=xyzzy@speakeasy.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.