[Resolved] Attaching PCM5102 to Omega2 (I2S)

[Andrew Naumenko]

I rechecked all the parameters of the i2s signal from omega2 and found the problem. i2s signal from omega 2 leaves really 16 bits per channel, but by default the data type is unsigned (u16). In the standard, I2S signal must be signed, and because of this, I have distortions with overload for TDA1543. At tests with MAX98357 these problems did not arise (probably) because it understands 16 24 and 32 bits, because of what the bit of the sign is shifted. When I try to reproduce the sign type of the signal (for example "mpg123 -e s16 test.mp3"), omega 2 is restarting.

[sonus]

@Andrew-Naumenko and others
I also get 99% distorded audio with my quality i2s dac that should get proper I2S (i2s = philips format with data 1 bitclock delayed left justified ) to play correctly . You probably found the culprit with the unsigned (u16) subject (corrected by the maxim ic) ? Hope people can help us get normal i2s (philips I2S) (for the most of users) stock driver config in the coming days/weeks. personnaly I do not want to get that maxim MX integratedampchip (when there are many more interesting i2s dacs and amps available everywhere) to fix when I already have better here. thanks for the future helpers

[luz]

@sonus: In ASoC (which is a subproject of ALSA, the Advanced Linux Sound Architecture), there are a lot of codecs (driver kernel modules) for various sound chips. The MAX98090 module obviously is made for that chip, and not others with different features.

For the Linux 4.4 kernel currently in use for LEDE 17.01 on which Onion's firmware is based, the available ASoC codecs are in the sound/soc/codecs subdirectory of the 4.4 kernel tree.

Of all these codecs, the current Onion build includes only those for WM8960 and MAX98090 (you can see that in the definition of sound-mt7620 which is in target/linux/ramips/modules.mk in Onion's LEDE source tree.)

You can see the changes @Lazar-Demin made to add the MAX98090 in his commit here (only the WM8960 was included in original LEDE 17.01)

Further up in this thread you can read about how I added the MAX98357A for my own use, and got it to work thanks to help from @sza2-sza2.

Probably, many of the other Linux 4.4 ASoC codecs would work with the MT7688's i2s hardware too, but one has to try to find out.

However, for the TDA1543, I see no SoC codec yet, neither in the 4.4 tree nor in the latest 4.17 tree...

Probably it would be doable to make a codec - have a look at max98357a.c, it consists mostly of config parameter tables, and almost no code. The hard part is probably learning to understand what these parameters mean

[sonus]

@luz the tda1543 and most DAC chips on the market (simply are operating following i2s signal lines) and as such do not need any driver except the most simple one implementing correct standart i2s (pcm5102 driver should match). I mean it does not need any other options like i2c spi (like volume control etc) etc thus only need the proper i2s pins and simple driver. it is same as pcm5102 etc regarding that : plain i2s only (pcm5102 is i2s left justified or i2s philips which are nearly the same at least one or another will not give any distortion) . thanks : I know and value the fact the new firmware audio inherits from this thread. except the core alsa i2s driver or asoc plain i2s driver, any i2s dac does not need any additionnal driver code to work when the problem will be corrected the new driver would fit nearly ALL the i2s and leftjustified audio dacs.

[sonus]

@luz
hi I understand most what is inside the MAX audio driver it is just I do not know how to mod my current firmware omega to put in the feature that would make it suiting all standart i2s dacs scenario.

[Andrew Naumenko]

@luz tda1543 does not require management (i2c or spi). It only needs a standard I2S interface with 16 bits with a 1 bit offset. The max98357 chip is too versatile and can understand many formats, but as I said earlier, the problem lies in the unsigned format of the output data. TDA1543 understands the range -32,768 to 32,767, and the omega2 yields 0 to 65,535.
Can you explain in more detail, how I can rebuild the system for making any changes?
Thks!

[luz]

@Andrew-Naumenko @sonus yes, the "driver" for these simple chips does not need any code, or at least almost none. However, it must describe the chip to the ALSA/ASoC subsystem, exactly because it needs to know if sound samples sent out via i2s should be signed or unsigned (furthermore, there is big endian and little endian byte orders, and possibly other details). Then ASoC can produce the correct type of samples.

That's the theory. Looking at what we have, I'm getting confused. For one both the MAX98090 codec as well as my MAX98357A codec claim support for signed samples (SNDRV_PCM_FMTBIT_S16) in their snd_soc_dai_driver structs. So IMHO with both codecs, I'd expect the samples coming out signed, and should fit the TDA1543.

I have no idea why this is not the case. But I suspect that there might be things missing or wrong in the whole setup, because for example, aplay --dump-hw-params is supposed to list the available formats, but at least on my omega, it just hangs forever. And further up in the thread @Andrew-Naumenko said forcing U16 samples with mpg123 just crashes the Omega. Also, poking around in /proc/asound/card0 leads to many files that just return the string closed. Again, no idea if this is ok or not, but it looks a bit strange.

ALSA is complex, and ASoC is relatively new (and we're using kernel 4.4, while 4.18 is current), and we're probably the first few trying this with the MT7688 at all - many chances for bugs here and there. I'm no way an ASoC expert, and also don't have the time to get one anytime soon. I just tried until my MAX98357A produced sound (from hxcmodplayer), and shared the procedure (see above).

In general, my hope is that when the next OpenWrt release uses a newer kernel, we'll maybe see things working that don't work now.

As for building your own kernel modules - @Maximilian-Gerhardt has created an excellent and detailed tutorial for that, here.

[sonus]

Hi all,
I am inquiring about a maximum digital out volume level creating the distortion (overdrive) as the possible problem cause. amixer and alsamixer being provided I used them to see and reduce all playback devices I could find just to try easy fixes and inquiring tools. amixer itself verbose playing : aplay -v -D hw:0 -f cd /dev/zero seems to send data rawinterleaved S16LE as we need but as of now I had not the opportunity to test a small .wav so as to hear result so if anyone is interested please do test this command playing a wav file : aplay xxfile.wav -v -D hw:0 -f cd
is there any volume control register in MT7688 I don't think ? MUCH unfortunately -g parameter is not featured anymore in mpg123 which would have been heaven... note though that the key press of - and prolonged pressing reduces volume gradually (on other than omega2 devices) so if any has a trick to : there is still some volume command in mpg123.
actually I do not know if is only a overvolume distortion problem or if it also is a problem of unsigned format which should be signed...
to help answer that question installing a mpg321 package could provide to the ones's interested the software volume control (gain) as it seems to offer (and better fixedpoint processing efficiency).
thanks

[sonus]

@luz
Hi do you know any quite simple way so as to use a 24bit data out i2s driver (seems that the a max... .c offer that mode : s24) : using any recent omega2 firmware (mainly your works) but getting that option mode to activate s24 instead of 16bit data format ?
stream_name = "HiFi Playback",
SNDRV_PCM_FMTBIT_S24
SNDRV_PCM_FMTBIT_S32,

thanks a lot for all

[Andrew Naumenko]

Hi guys!
I'm all trying to figure out what's wrong with TDA1543. I got a few graphs to explain my problem. I uploaded a test mp3 file and run him mpg123 -f 256 -C -v sin1000.mp3 (-f 256 is amplitude). File mp3 - sine wave, frequency 1 kHz.The amplitude is varied from 128 to 1024 (128 is displayed in mpg123 as 0 and it is on level with the noise level of the chip itself)




Initially, I sinned on the error data types (U16 or S16) but now I think that the problem is in the data reading frequencies. Maybe someone will have an idea what's the problem?

Now I'm trying to compile a kernel for omega with other codecs (how shown in this theme), but it's not working yet. If anyone has a ready-made kernel with other codecs, which I could immediately flash into the omega for tests, discard the download link. Thks!

[Andrew Naumenko]

I fiddled with the parameters of TDA1543 and got a normal sound. But it is so only when I set the playback volume in mpg123 to 2 percent. More and I get distortions, I will give another 2 oscillograms first - "mpg123 -f 128" second "mpg123 -f 256". Initially this is a saw with a single amplitude generated with the aid of audacity.


However, that sounds!
I also noticed an error, the line CLK does not turn off when the sound stops.

[sonus]

Hi all
Does anyone know if the current driver in use stock in omega2 use LeftJustified or the RightJ or the philips format i2s (one bitclock delayed) i2s format ?
also it is 16 bit data per channel and 32 bit total word bitclock I think : meaning we need to use either simple 16 bit only dacs or else very wide accepting comptability multi-formats allowing (extensive format input accepting) dacs.
still it is a nice evolution from some weeks ago when i2s/sound was not featured at all.

In fact Andrew I suspect from your overdrive distortion and volume symptoms (without all checking the mt7688 doc and register) omega2 driver is probably left justified format when your dac tda1543 only strictly accept philips justified i2s (meaning 1 bck delayed msb left justified) . that would mean an error on your dac ms bit understanding. meaning it starts seemingly to correctly work only when volume is grossly reduced.

[Andrew Naumenko]

In this theme
Judging by this picture, the data format is 16 bits i2s

[sonus]

@Andrew-Naumenko
true I had seen it. from that picture it is sure it is 16 bits per channel i2s (into 32 bit bclk total sample) wordclock.
but the fact you need to turn down volume so much is all the more strange knowing that both your omega2 and tda dac are EXACTLY same philips i2s mode format (and both 16 bits only).

[no 28]

Hi all,

I´m also trying to get the MAX98357A running on a Linkit Smart 7688 with LEDE master branch (kernel 4.9). The previous manuals about configuring and patching helped me a lot - thanks for it.

Now I´m able to play a mp3-file with madplay and can hear the sound, but there is one issue.
After stopping the sound the loudspeaker gets hot, because of ~5V DC output.

In the MAX datasheet is written "do not remove LRCLK while BCLK is present -> may cause a high DC output". This seems to be the case. In I2S mode the datasheet tells BLCK = "rising edge".
Am I wrong or is it needed to invert the BCLK signal for the MAX98357A in the simple-sound-card settings to get a I2S mode running properly?

I tried to set bitclock-inversion in the simple card setup, but it returned with a ralink-i2s / ASoC issue "Failed to set DAI format -22".

Unfortunately, currently no oscilloscope available to measure it...

Thanks in advance for any hints from you...

[Mark Nagaev]

Anyway, someone successed with plaing something with aplay on i2s(Lede 17.01 branch)?