[Development] kernel-power - Page 11 - maemo.org

Active Topics

Is there a section to talk about Java ME? Apps, etc. (3)
to General by Kalatti - 1 day, 13 hrs ago
more...

Page 11 of 23

Thread Tools

vi_	2012-02-01 , 09:07
Posts: 1,680 \| Thanked: 3,685 times \| Joined on Jan 2011	#101

Wait, some kind of highpass filter? WTF for? Can we have some details on this?

Is it this?

From this guy?

who said this:

Originally Posted by luke-jr

cuz I have nfc how any of that works or what a highpass filter even is

Originally Posted by luke-jr

apparently it only works at 48 kHz as-is tho I wouldn't have a clue how to adapt for other sample rates

Originally Posted by luke-jr

that assumes I have some idea what f is, or really which part of this code does the filtering at all :P

Originally Posted by luke-jr

maybe I'll find a way to disable other sample rates in the kernel.

I know I am not in much of a position to criticise, being much of a non-contributer however this one has my spidey senses tingling.

__________________

N900: One of God's own prototypes. A high-powered mutant of some kind never even considered for mass production. Too weird to live, and too rare to die.

Quote & Reply |

The Following 2 Users Say Thank You to vi_ For This Useful Post:
don_falcone, Estel

freemangordon	2012-02-01 , 09:40
Posts: 3,074 \| Thanked: 12,960 times \| Joined on Mar 2010 @ Sofia,Bulgaria	#102

Well, I think I am in position

@Pali, why is that filter needed in Maemo5 kernel, what problem/bug it fixes, or what functionality it adds? Who sent you the patch? Is it upstream patch?

TBH I think you should remove that from kernel-power at least until there is some information WTF is this animal(the filter) doing and who will support it (fix bugs) at least while KP is in devel.

Or even better - remove it constantly.

P.S.
And maybe you should re-consider your policy to accept patches from developers who cannot be contacted/don't want to help if some problem arise, thus leaving only me and you to fix bugs introduced by some random guy.

Quote & Reply |

The Following 3 Users Say Thank You to freemangordon For This Useful Post:
don_falcone, Estel, maartenmk

don_falcone	2012-02-01 , 10:00
Posts: 1,523 \| Thanked: 1,997 times \| Joined on Jul 2011 @ not your mom's FOSS basement	#103

+The codec driver leverages the codecs effects through alsa controls and a
+hwdep device for controlling the hardware fourth-order IIR filter block.
+
+There's an alsa control, "3D Control - Depth" for depth simulation.
+The rest of the controls are for the IIR filter:
+
+1- A control for setting the bass/treble gain, which sets the filter's
+ coefficients to certain precalculated values.
+2- A control for 'off' / 'Bass/Treble' / 'Custom'. 'Bass/Treble' means
+ the bass/treble gain controls are used, while 'custom' means the
+ coefficients have been set through the hwdep device (see below).
+Note: bass/treble controls are not yet implemented
+
+Filters
+--------
+Note: Setting a filter's coeffs automatically turns it off, it needs to
+be turned on explicitly.
+
+The De-emphasis filter can only be controlled on the machine driver level.
+For example for the n900 (rx51.c) it is used as highpass filter for
+speaker protection. See tlv320aic3x.h, aic3x_deemph_set_* for details.

/* Default De-emphasis filter coefficients to use as a highpass for
+ * cheap speaker protection */

Alles klar. We will get weak sound from the speakers.... I wonder if this applies to headphone/TV out too.

Quote & Reply |

The Following User Says Thank You to don_falcone For This Useful Post:
Estel

	Estel	2012-02-01 , 10:10
	Posts: 5,028 \| Thanked: 8,613 times \| Joined on Mar 2011	#104

Originally Posted by pali

Now kernel-power has 4 major packages (needed for using):
kernel-power
kernel-power-flasher
kernel-power-modules
kernel-power-bootimg
and some more for development...

I have question: Why two packages are needed for flashing kernel? kernel-power only contains fiasco image (with packaged zImage) and kernel-power-flasher has only postinst script which flash that fiasco image (from package kernel-power) and then REMOVE fiasco image from rootfs. (Note that stock kernel has same process, kernel-power only copied kernel source package)

What do you think about merging kernel-power and kernel-power-flasher to one package kernel-power-flasher? (I will do not delete kernel-power package, I only mark it as dummy/transitional)

Positive on this change:
* you do not need to install kernel-power package
* when you want to reinstall/reflash kernel-power, you need only reinstall package kernel-power-flasher

Negative:
* packages which depends directly on kernel-power will must change dependences to kernel-power-flasher

But I will let kernel-power package in repository, but it will be empty (+ added dependency on kernel-power-flasher) - so this does not break last negative point.

So what do you think?

I think it's great idea. Yet, are You sure, that both kernel-power-bootimg and kernel-power-flasher are needed for usage? For people with multiboot, kernel-power-bootimg is enough (I haven't installed kernel-power-flasher, and everything works fine).

/Estel

// Edit

+1 on kicking highpass-filter "patch" from KP, and not accepting patches from random noobs, at least, without audit.

__________________
N900's aluminum backcover / body replacement
-
N900's HDMI-Out
-
Camera cover MOD
-
Measure battery's real capacity on-device
-
TrueCrypt 7.1 | ereswap | bnf
-
Hardware's mods research is costly. To support my work, please consider donating. Thank You!

Last edited by Estel; 2012-02-01 at 10:14.

Quote & Reply |

don_falcone	2012-02-01 , 10:16
Posts: 1,523 \| Thanked: 1,997 times \| Joined on Jul 2011 @ not your mom's FOSS basement	#105

From a modularized & "clean & neat & tidied-up" POV, it makes sense to me having separate packages for actual kernel image and the flasher. It's almost comparable in splitting firmware image(s) and driver (loader/flasher).

EDIT: Maybe it's debatable why the actual image is removed after flashing, i see some congruency with kernel-power-bootimg here, having two actual kernel images (IIRC, they are same format?)

Last edited by don_falcone; 2012-02-01 at 10:24.

Quote & Reply |

The Following 2 Users Say Thank You to don_falcone For This Useful Post:
Estel, misterc

	misterc	2012-02-01 , 10:36
	Posts: 1,625 \| Thanked: 998 times \| Joined on Aug 2010	#106

Originally Posted by pali

Now kernel-power has 4 major packages (needed for using):
kernel-power
kernel-power-flasher
kernel-power-modules
kernel-power-bootimg
and some more for development...

[...]
So what do you think?

i'm not much of a (kernel / OS) dev, so i'm just talking from a (SW) admin point of view, but i second Don_falcone's opinion; keep things clearly structured and leave it up to the end user to leave out what they don't want (as Estel did).

also, more from a "distribution" and SW management point of view, is it smart to change the structure of the KP (meta-)package?
i know the KP is not a "normal" package as it can not be uninstalled (or even upgraded?) thru HAM/FAM.

maybe an idea (changing strucutre) if you decide to include the KP in CSSU?

Quote & Reply |

pali	2012-02-01 , 11:21
Posts: 2,153 \| Thanked: 8,462 times \| Joined on May 2010	#107

Patch Support-for-tlv320aic3x-codec-highpass-filter-needed.diff was disabled.

I sent info to luke-jr via jabber, to comment this patch.

===

Also in future for better patch review from other community members:
Each new patch must be linked to THIS thread and commented/reviewed/tested by more (at least 3) people.

I think this should be enought.

Quote & Reply |

The Following 8 Users Say Thank You to pali For This Useful Post:
don_falcone, Estel, freemangordon, fw190, Harick, mr_jrt, Sourav.dubey, vi_

Luke-Jr	2012-02-01 , 14:45
Posts: 53 \| Thanked: 40 times \| Joined on May 2009 @ Brooksville, Florida	#108

So is there actually a real problem with Support-for-tlv320aic3x-codec-highpass-filter-needed.diff? If you guys would prefer to potentially blow up your speakers, so be it...

Quote & Reply |

vi_	2012-02-01 , 14:56
Posts: 1,680 \| Thanked: 3,685 times \| Joined on Jan 2011	#109

Originally Posted by Luke-Jr

So is there actually a real problem with Support-for-tlv320aic3x-codec-highpass-filter-needed.diff? If you guys would prefer to potentially blow up your speakers, so be it...

No offence intended, it is just that none of us know what this is or what it does and the IRC conversation highlighted above does not fill me with confidence.

Would you care to explain what this patch does, how it achieves it and how the end user can control it's effect?

Are you 'mnzaki'?

__________________

N900: One of God's own prototypes. A high-powered mutant of some kind never even considered for mass production. Too weird to live, and too rare to die.