Rusty Scott wrote:
On Wed, 2006-04-19 at 02:58 +0400, Manu Abraham wrote:
Andrew de Quincey wrote:
On Tuesday 18 April 2006 23:21, Rusty Scott wrote:
So can someone give me a quick decision on what I should do? Should I
put that code into dvb-core.c or make a new dvb_core_math.c?
I would say put it in dvb_core_math.c, but link it into the dvb-core.ko
module.
what about dvb_math.c ? But we will need a log function which has a
better precision for tuning algorithm. Christoph Pfister volunteered to
do one for the stb0899. If we have a common one which is quite precise
would be nice.
He sent me a preview, i think it needed a bit more tuning up, he said he
got an accuracy of 0.054%
WTF? I'm asking what I should do with my patch so I can get this code
fixed, placed in a common place, and use it in another module.
Wouldn't it be nice to have a common function which is quite precise
without duplication of functions ?
Since code in dvb-core is shared, it would be subject to change since
other hardware will have different requirements for the same.
If someone wants to then patch that with a more accurate ilog2 and write a
conversion to i10log10 or whatever I'm fine with that too.
I didn't know that you were having a patch (well missed on the subject),
well just put it into a file called dvb_math.c.
When I'm asking the question what should I do, I'm not really looking
for the response "this is what someone else is doing that might be
tangentially related to what you have submitted." Give me some
direction here.
We will later on change it according to requirements. just do a
EXPORT_SYMBOL(function_name), to use that symbol in your relevant
module. Make sure it compiles well and submit in your patch. That should
be it.
I think dvb-core is LGPL'd, maybe all other code going in there needs to
have the same license as well, not sure on that. Please do check on this.
Manu
_______________________________________________
linux-dvb@xxxxxxxxxxx
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb
