public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* DST/BT878 module customization (.. was: Critical points about ...)
@ 2007-04-30 21:17 Markus Rechberger
  2007-05-01  6:40 ` [linux-dvb] " Simon Arlott
  2007-05-01 14:55 ` Mauro Carvalho Chehab
  0 siblings, 2 replies; 59+ messages in thread
From: Markus Rechberger @ 2007-04-30 21:17 UTC (permalink / raw)
  To: Jan Engelhardt
  Cc: Uwe Bugla, Mauro Carvalho Chehab, helge.hafting, akpm, torvalds,
	linux-kernel, linux-dvb, Manu Abraham

Hi,

Trent Piepho wrote another patch for it, it just completes Uwe's patch
in the end.

http://linuxtv.org/hg/~tap/dst-new?cmd=changeset;node=bbdd2b53cd5c;style=gitweb

as far as I see from that patch it cleans up a memory leak which would
happen when the system tries to load the dst module if it's not
available and it also prints a message that points the user should
enable it in the kernel if needed.

It also bundles the dst and dst_ca objects to one selectable option.
So the idea remains the same.
>From my side I do not see any problem with that patch, if someone else
has a problem with it please state out the reason.

Markus


On 4/30/07, Markus Rechberger <mrechberger@gmail.com> wrote:
> On 4/30/07, Jan Engelhardt <jengelh@linux01.gwdg.de> wrote:
> >
> > On Apr 30 2007 19:25, Uwe Bugla wrote:
> >
> > >THIS PATCH IS DONE TO AVOID RAM WASTE FOR CASES IN WHICH IT IS PROVEN
> THAT
> > DST
> > >AND DST_CA ARE NOT NEEDED AT ALL!!!!
> > >[...]
> >
> >
> > How much on the Theo-meter are we yet?
> >
>
> it's enough, I told him that I'll look at it and try to get some other
> people involved if it really breaks something it should get stated
> out; and I'll refuse any further help if he starts to write any more
> abusive mail.
>
> So to his proposal:
>
> the whole noise is about following Makefile patch:
> -obj-$(CONFIG_DVB_BT8XX) += bt878.o dvb-bt8xx.o dst.o dst_ca.o
> +obj-$(CONFIG_DVB_BT8XX) += bt878.o dvb-bt8xx.o
> +obj-$(CONFIG_DVB_DST) += dst.o
> +obj-$(CONFIG_DVB_DST_CA) += dst_ca.o
>
> that symbol_request is unable to return a valid pointer if DST an
> DST_CA aren't selected should be ok because this would only happen if
> someone didn't compile them in (an appropriate error message should be
> added for that)
>
> I'm trying to look closer at this issue with some other developers, if
> it's really that easy to split off the dst module from the bt* objects
> without breaking anything, to me the direction this patch goes seems
> to be ok, some people stated out that there are problems so I'll try
> to get more information about that.
>
> Markus
>


-- 
Markus Rechberger

^ permalink raw reply	[flat|nested] 59+ messages in thread

end of thread, other threads:[~2007-05-08  0:01 UTC | newest]

Thread overview: 59+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-04-30 21:17 DST/BT878 module customization (.. was: Critical points about ...) Markus Rechberger
2007-05-01  6:40 ` [linux-dvb] " Simon Arlott
2007-05-01  9:00   ` Markus Rechberger
2007-05-01  9:31     ` Uwe Bugla
2007-05-01 22:57   ` Trent Piepho
2007-05-02 13:30     ` Manu Abraham
2007-05-02 15:51       ` Uwe Bugla
2007-05-02 16:43         ` Manu Abraham
2007-05-02 16:45         ` Manu Abraham
2007-05-02 17:33           ` Markus Rechberger
2007-05-03 11:20           ` Uwe Bugla
2007-05-03 14:44             ` Manu Abraham
2007-05-03 15:31               ` Uwe Bugla
2007-05-03 15:48                 ` Manu Abraham
2007-05-03 15:59                   ` Markus Rechberger
2007-05-03 16:17                     ` Manu Abraham
2007-05-03 17:19                       ` Markus Rechberger
     [not found]                         ` <1178215045.12651.124.camel@localhost>
2007-05-03 19:03                           ` Manu Abraham
2007-05-03 21:00                             ` Markus Rechberger
2007-05-03 21:42                               ` Manu Abraham
2007-05-03 22:06                                 ` Markus Rechberger
2007-05-03 22:31                                   ` Manu Abraham
2007-05-03 23:09                                     ` Markus Rechberger
2007-05-04  0:47                                     ` hermann pitton
2007-05-04  1:30                                     ` Uwe Bugla
2007-05-04  0:07                                   ` Uwe Bugla
2007-05-05 18:06                             ` Mauro Carvalho Chehab
2007-05-05 18:46                               ` Manu Abraham
2007-05-07 20:54                                 ` Mauro Carvalho Chehab
2007-05-07 21:25                                   ` Manu Abraham
2007-05-07 21:34                                     ` Michael Krufky
2007-05-07 21:49                                       ` Manu Abraham
2007-05-07 21:51                                   ` Uwe Bugla
     [not found]                         ` <a3ef07920705031119x332db12dob997e5ebc6a8e218@mail.gmail.com>
2007-05-03 20:49                           ` Markus Rechberger
2007-05-03 16:25                     ` Uwe Bugla
2007-05-03 16:05                   ` Uwe Bugla
2007-05-03 16:15                     ` Manu Abraham
2007-05-03 16:30                       ` Michael Krufky
2007-05-03 16:35                         ` Manu Abraham
2007-05-07 23:33       ` Trent Piepho
2007-05-08  0:00         ` Manu Abraham
2007-05-01 14:55 ` Mauro Carvalho Chehab
2007-05-01 16:40   ` [linux-dvb] " Uwe Bugla
2007-05-01 18:30     ` Uwe Bugla
2007-05-01 18:50       ` Simon Arlott
2007-05-01 19:34         ` Uwe Bugla
2007-05-01 20:35           ` Simon Arlott
2007-05-01 21:29             ` Uwe Bugla
2007-05-01 19:20       ` e9hack
2007-05-01 19:26         ` Uwe Bugla
2007-05-01 23:16   ` Trent Piepho
2007-05-02  2:03     ` Mauro Carvalho Chehab
2007-05-02 11:10       ` Trent Piepho
2007-05-02 12:04         ` Uwe Bugla
2007-05-03 14:02         ` Mauro Carvalho Chehab
2007-05-03 15:15           ` Manu Abraham
2007-05-03 15:36             ` Uwe Bugla
2007-05-04  5:13           ` Trent Piepho
2007-05-04  9:23             ` Mauro Carvalho Chehab

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox