mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@infradead.org>
To: "Michael Krufky" <mkrufky@linuxtv.org>
Cc: "Linus Torvalds" <torvalds@linux-foundation.org>,
	linux-dvb-maintainer@linuxtv.org,
	"Andrew Morton" <akpm@linux-foundation.org>,
	video4linux-list@redhat.com, linux-kernel@vger.kernel.org
Subject: Re: [v4l-dvb-maintainer] [GIT PATCHES] V4L/DVB updates and fixes for 2.6.26
Date: Tue, 29 Apr 2008 21:00:03 -0300	[thread overview]
Message-ID: <20080429210003.14f6c9c7@gaivota> (raw)
In-Reply-To: <37219a840804291649q36638464ye2d57cf8184580a4@mail.gmail.com>

On Tue, 29 Apr 2008 19:49:24 -0400
"Michael Krufky" <mkrufky@linuxtv.org> wrote:

> On Tue, Apr 29, 2008 at 5:50 PM, Mauro Carvalho Chehab
> <mchehab@infradead.org> wrote:
> > Linus,
> >
> >  Please pull from:
> >         ssh://master.kernel.org/pub/scm/linux/kernel/git/mchehab/v4l-dvb.git master
> >
> >  For the following:
> >
> >    - Fixes on mtm001, mtv022, pvrusb2, ivtv, cx88 and saa7134;
> >    - new board additions on saa7134 and ivtv;
> >    - load tuners only when needed;
> >    - reorganization of tuner drivers that are shared between DVB and V4L;
> >    - Addition of a new driver for Conexant CX23418 MPEG encoder chip (cx18).
> 
> [snip]
> 
> >  Mauro Carvalho Chehab (9):
> >       Rename common tuner Kconfig names to use the same
> >
> >  Michael Krufky (4):
> >       V4L/DVB (7789): tuner: remove static dependencies on analog tuner sub-modules
> 
> 
> Linus has already merged the changes (thank you, Linus) ... However,
> there is a bug.
> 
> My "remove static dependencies on analog tuner sub-modules" patch was
> applied after Mauro's "Rename common tuner Kconfig names to use the
> same" patch.
> 
> My patch has conditional behavior, based on CONFIG_DVB_CORE_ATTACH,
> which was renamed to CONFIG_MEDIA_ATTACH in Mauro's patch.
> 
> To fix this, we need to do:
> 
> sed -i s/"CONFIG_DVB_CORE_ATTACH"/"CONFIG_MEDIA_ATTACH"/1
> drivers/media/video/tuner-core.c
> 
> The issue will cause invalid module use counts upon unloading analog
> tuner modules, if CONFIG_MEDIA_ATTACH is enabled.
> 
> I would be happy to fix this myself, but Mauro's patch has not yet
> been backported into the linuxtv.org repository.
> 
> Mauro, can you do the above fix and send it in to Linus?

Sure. Merge conflicts... bah...

I'll also double check if there aren't also other conflicts. It is just a
matter or run the sed script again ;)


Cheers,
Mauro

  reply	other threads:[~2008-04-30  0:02 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-29 21:50 Mauro Carvalho Chehab
2008-04-29 21:59 ` Linus Torvalds
2008-04-29 23:49 ` [v4l-dvb-maintainer] " Michael Krufky
2008-04-30  0:00   ` Mauro Carvalho Chehab [this message]
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 14:39                   ` [v4l-dvb-maintainer] " Michael Krufky
2008-04-30 15:29                     ` Andrew Morton
2008-04-30 15:30                     ` Mike Galbraith
2008-04-30 15:34                     ` Andrew Morton
2008-04-30 15:39                       ` Michael Krufky
2008-04-30 16:01                         ` Mike Galbraith
2008-04-30 15:48                       ` Mauro Carvalho Chehab
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 21:52                 ` [v4l-dvb-maintainer] " Trent Piepho
2008-04-30 22:38                   ` [v4l-dvb-maintainer] [patch, -git] drivers/media build fix fo r " mkrufky
2008-04-30  9:26             ` [patch, -git] drivers/media build fix for " 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
2008-04-30  8:02 ` [v4l-dvb-maintainer] [GIT PATCHES] V4L/DVB updates and fixes for 2.6.26 Patrick Boettcher
2008-04-30 19:03   ` Mauro Carvalho Chehab

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=20080429210003.14f6c9c7@gaivota \
    --to=mchehab@infradead.org \
    --cc=akpm@linux-foundation.org \
    --cc=linux-dvb-maintainer@linuxtv.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mkrufky@linuxtv.org \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®