At Mon, 17 Mar 2008 16:50:39 +0100, Pavel Hofman wrote:
Takashi Iwai wrote:
I am afraid I do not understand what to change in snd_vt1724_pro_internal_clock_get(). It seems fairly logical, I made only minor changes - is_spdif_master used in other parts of the code, get_rate_index with a simple meaning.
Yeah, your change is logical -- simply convert some code snippets to callbacks. But, this isn't really good for maintenance for a long term. I prefer a bit more straight way if we really change something. And, indeed we do need to change the stuff for rate settings.
I am afraid I do not have the insight of what needs to be changed. The code as is seems fairly OK to me, just a bit underdocumented.
OK, I will remove the rate_code conversions, the new overhead will be low and one abstraction will be removed.
For the rest, please state you objectives. Either cutting a few of the callbacks by non-trivial rewrite of the original ice1724 code, or keeping the remaining callbacks and the well-tested code.
Don't be too nervous about changing the ice1724 code right now :) We would need the rewrite of core codes anyway because of other problems in Maya44. And, I believe we can fix more than we might break by such a restructuring in the end. The current ice1724.c is way too complex due to its history, derived from ice1712.c.
Well, to tell the truth, I do not feel knowledgeable enough of the alsa infrastructure to implement major changes in ice1724 code.
Actually, what is the deal with maya44? Its detailed image shows it has standard clocking scheme (no FPGA, PLL etc.), just a couple of Wolfson codecs, connection to MI/ODI/O card (fully supported e.g. by Prodigy192), switched I2S input between SPDIF-IN/Analog-IN. I just do not see a reason to rewrite ice1724 because of maya44 support.
The limitation of capture rate to 96kHz is present in most other ice1724 cards (including Juli) and nobody has made a big deal of it. It could be perhaps tackled in a similar manner we solved the single SPDIF-input rate.
Exactly. That's why I suggested here.
Plus I will have to return the borrowed Juli soon, and will not be able to do some qualified testing.
I have another ice1724 card on the way (SndScape Odeum SPDIF interface) and really would love to finish Juli soon.
Well, then we'll need to find some point to compromise. Could you repost the patch after reducing the callbacks again?
I'll have also no time from the next week at all -- will be on vacation for three weeks. So, let's kick it out soonish :)
thanks,
Takashi