[alsa-devel] Cherry Trail + RT5645 devices with a mono speaker ?

[Hans de Goede]

Hi,

On 27-10-17 08:39, Bard Liao wrote:
>> -----Original Message-----
>> From: Hans de Goede [mailto:hdegoede at redhat.com]
>> Sent: Thursday, October 26, 2017 11:09 PM
>> To: Bard Liao; Pierre-Louis Bossart
>> Cc: alsa-devel at alsa-project.org
>> Subject: Re: Cherry Trail + RT5645 devices with a mono speaker ?
>>
>> Hi,
>>
>> On 16-10-17 13:31, Bard Liao wrote:
>>>> -----Original Message-----
>>>> From: Hans de Goede [mailto:hdegoede at redhat.com]
>>>> Sent: Sunday, October 15, 2017 2:12 AM
>>>> To: Pierre-Louis Bossart; Bard Liao
>>>> Cc: alsa-devel at alsa-project.org
>>>> Subject: Cherry Trail + RT5645 devices with a mono speaker ?
>>>>
>>>> Hi Pierre-Louis, Bard,
>>>>
>>>> I've been looking into getting some Cherry Trail + RT5645
>>>> devices with a mono speaker I have to work properly.
>>>>
>>>> Specifically the goal is to mix the right channel into
>>>> the left output, so that sounds which are only played
>>>> over the right channel do not get lost.
>>>>
>>>> I'm using this UCM file:
>>>> https://github.com/plbossart/UCM/blob/master/byt-rt5640/HiFi
>>>>
>>>> Looking at the rtl5640 UCM file, getting the mono speaker
>>>> to work (in a first simple attempt) should be as simple
>>>> as replacing:
>>>>
>>>>                    cset "name='SPOR MIX SPKVOL R Switch' on"
>>>>
>>>> With
>>>>
>>>>                    cset "name='SPOL MIX SPKVOL R Switch' on"
>>>>
>>>> But this does not work, the speaker test sound for the
>>>> right speaker is still silent (works with headphones).
>>>>
>>>> I've also looked into directly poking the
>>>> RT5645_SPO_MIXER i2c register for testing, but when
>>>> checking its value with the original unmodified UCM
>>>> file:
>>>>
>>>> [root at localhost ~]# i2cget -y -f 1 0x1a 0x48  w
>>>> 0x06c8
>>>>
>>>> That is: 0xc806 as the output of i2cget "w" mode
>>>> needs byteswapping. Note that both the
>>>> RT5645_M_SV_L_SPM_L and RT5645_M_SV_R_SPM_L bits
>>>> are already cleared, which is weird as this is
>>>> before I've modified anything. Also the
>>>> RT5645_M_SV_R_SPM_R bit is cleared, but that is
>>>> expected.
>>>>
>>>> Even if I manually set reg 0x48 to 0x07c8 which AFAIK
>>>> should enable output of both left and right channels
>>>> on the left speaker I still only get sounds played
>>>> on the left channel.
>>>>
>>>> Any insights / help with this would be very much
>>>> welcome.
>>>
>>> Could you dump all registers for us?
>>
>> Here you go:
>>
>>        0,8  1,9  2,a  3,b  4,c  5,d  6,e  7,f
>> 00: 0000 0808 c8c8 c8c8 0ac8 0000 0000 0000
>> 08: 0000 0000 0200 2738 00e0 0000 0000 0000
>> 10: 0000 0000 0000 0000 3333 0000 004b 0000
>> 18: 8b01 afaf afaf 1100 3f3f 7f7f 00f0 0000
>> 20: 00a0 0000 0000 0000 0000 0000 0000 2038
>> 28: 3030 8080 1616 4444 a0aa 0000 0000 0210
>> 30: 0000 0050 0000 0000 0000 0000 0000 0000
>> 38: 0000 0000 0000 0000 7d00 0000 7d00 0000
>> 40: 1c00 0000 1c00 0000 0000 0050 3800 3800
>> 48: 06c8 0000 0400 0000 1f03 0000 0000 ff01
>> 50: 0000 0000 ff01 00f0 0000 0000 1101 6400
>> 58: 0eef f0f0 0eef f0f0 0eef f0f0 00f0 0000
>> 60: 0000 019b 0008 1ae8 0402 0230 00c0 0000
>> 68: 0000 0000 0000 0000 aa0a 0000 0000 0000
>> 70: 8380 0080 0080 7017 003e 0924 0a00 0058
>> 78: 0000 2301 0080 0000 0000 0000 0000 0000
>> 80: 0040 030f 0030 000c 1111 0000 0800 0000
>> 88: 0000 0000 2001 0000 0300 0000 0000 4011
>> 90: 3606 060c 0000 0801 0202 0000 0000 0000
>> 98: 0000 0000 8421 0a01 ea0a 0c00 0004 0000
>> a0: e8a0 5900 0100 0000 0000 0000 0000 0000
>> a8: 0000 0000 0000 0000 0000 0000 0060 0000
>> b0: 0060 0000 0000 1f00 0c02 001f 0000 0040
>> b8: 0000 0000 0000 0000 0000 8002 0000 8011
>> c0: 0080 0000 0000 0020 0000 0000 0000 0000
>> c8: 0000 0000 0000 0000 0000 0000 0000 1418
>> d0: 9006 171c 0000 20b3 0000 0000 0004 0000
>> d8: 0000 0908 0000 0300 4900 1b00 0000 0000
>> e0: 0000 0000 0000 0000 0000 0000 0080 0007
>> e8: 0080 0007 200f 0000 00b3 0000 0000 0000
>> f0: 1f00 0c02 001f 0000 0040 0000 0000 0000
>> f8: 0000 0000 6120 4040 001a 0400 ec10 0863
>>
>> Note that i2cdump swaps the low and high bytes
>> in all the 16 bit words.
> 
> Could you dump registers by regmap?
> cat /sys/kernel/debug/regmap/<bus name>/registers
> It will be more readable for me.

Ok here you go:

000: 0000
001: 0808
002: c8c8
003: c8c8
00a: 0002
00b: 3827
00c: e000
00d: 0000
00e: 0000
00f: 0000
014: 3333
016: 4b00
018: 018b
019: afaf
01a: afaf
01b: 0011
01c: 3f3f
01d: 7f7f
01e: f000
020: a000
027: 3820
028: 3030
029: 8080
02a: 1616
02b: 4444
02c: aaa0
02d: 0000
02f: 1002
031: 5000
032: 0000
033: 0000
034: 0000
035: 0000
03b: 0000
03c: 007d
03d: 0000
03e: 007d
03f: 0000
040: 001c
041: 0000
042: 001c
045: 5000
046: 0038
047: 0038
048: c806
04a: 0004
04d: 0000
04e: 0000
04f: 01ff
050: 0000
051: 0000
052: 01ff
053: f000
056: 0111
057: 0064
058: ef0e
059: f0f0
05a: ef0e
05b: f0f0
05c: ef0e
05d: f0f0
05e: f000
05f: 0000
061: 9b01
062: 0800
063: e81a
064: 0204
065: 3002
066: c000
06a: 0037
06c: fc00
070: 8083
071: 8000
072: 8000
073: 1770
074: 3e00
075: 2409
076: 000a
077: 5800
078: 0000
079: 0123
07a: 8000
080: 4000
081: 0f03
082: 3000
083: 0c00
084: 1111
085: 0000
08a: 0120
08e: 0000
08f: 1140
090: 0636
091: 0c06
093: 0108
094: 0202
095: 0000
09a: 2184
09b: 010a
09c: 0aea
09d: 000c
09e: 0400
0a0: a0e8
0a1: 0059
0a2: 0001
0ae: 6000
0af: 0000
0b0: 6000
0b1: 0000
0b2: 0000
0b3: 001f
0b4: 020c
0b5: 1f00
0b6: 0000
0b7: 4000
0bb: 0000
0bc: 0000
0bd: 0280
0be: 0000
0bf: 1180
0c0: 8000
0c1: 0000
0c2: 0000
0c3: 2000
0cd: 0000
0ce: 0000
0cf: 1814
0d0: 0690
0d1: 1c17
0d3: b320
0d4: 0000
0d6: 0400
0d9: 0809
0da: 0000
0db: 0003
0dc: 0049
0dd: 001b
0df: 0000
0e0: 4000
0e6: 8000
0e7: 0200
0e9: 0700
0ec: b300
0ed: 0000
0f0: 001f
0f1: 020c
0f2: 1f00
0f3: 0000
0f4: 4000
0f8: 0000
0f9: 0000
0fa: 2061
0fb: 4040
0fc: 1a00
0fd: 0004
0fe: 10ec
0ff: 6308
100: 0aaa
101: 2e60
102: a680
103: 7418
104: 0000
105: 0306
106: ffff
107: c7c0
108: 2e60
109: a680
10a: 7418
10b: 0000
10c: 0306
10d: ffff
10e: c7c0
10f: 0000
110: 0000
111: a814
112: 0002
113: 8aaa
114: 1aaa
115: aaa0
116: 0000
117: 0000
118: 0000
119: 0000
11a: 0000
11b: 0200
11c: fd70
11d: 0247
11e: 0000
11f: 0155
120: 611f
121: 4040
122: 0052
123: c004
124: 0430
125: 7530
126: 0000
127: 0000
128: 0000
129: 0000
12a: 0000
12b: 0000
12c: 0000
12d: 0000
12e: 0000
12f: 0000
130: 0000
131: 0ddd
132: 0ddd
133: 0000
134: 0000
135: 5155
136: 0000
137: fc00
138: 79a1
139: 0000
13a: 0002
13b: 0000
13c: 0000
13d: 3600
13e: 1f80
13f: 0000
140: 6883
141: d3ce
142: 352d
143: 00eb
144: 0000
145: 0000
146: 0000
147: 0000
148: 0000
149: 0000
14a: 0000
14b: 0000
14c: 0000
14d: 0000
14e: 0000
14f: 0000
150: 0000
151: 0000
152: 0000
153: 0000
154: 0000
155: 0000
156: 0000
157: 0000
158: 0000
159: 0000
15a: 0000
15b: 0000
15c: 0000
15d: 0000
15e: 0000
15f: 0000
160: 0000
161: 0000
162: 0000
163: 3717
164: 4c6a
165: e41b
166: 2a13
167: b600
168: c730
169: 35d4
16a: 00bf
16b: 0000
16c: 0000
16d: 0000
16e: 0000
16f: 0000
170: 0000
171: 0000
172: 0000
173: 0000
174: 0000
175: 0000
176: 0000
177: 1f01
178: 4040
179: 0000
17a: 3f3f
17b: 0000
17c: 0000
17d: 0000
17e: 0000
17f: 0000
180: 3f3f
181: 0000
182: 0000
183: 0000
184: 0000
185: 0000
186: 004c
187: 0000
188: 0000
189: 0000
18a: 0000
18b: 0000
18c: 0000
18d: 0000
18e: 0000
18f: 0000
190: 3300
191: 2200
192: 0000
193: 0000
194: 8700
195: 7dc2
196: a178
197: 5383
198: 7dc2
199: a178
19a: 5383
19b: 003e
19c: 02c1
19d: d37d
19e: 0000
19f: 0000
1a0: 0000
1a1: 0000
1a2: 0000
1a3: 0000
1a4: 1c10
1a5: 01f4
1a6: 1c10
1a7: 01f4
1a8: 0000
1a9: 0000
1aa: 0000
1ab: 0000
1ac: 0000
1ad: 0000
1ae: c882
1af: 1c10
1b0: 01f4
1b1: c882
1b2: 1c10
1b3: 01f4
1b4: e904
1b5: 1c10
1b6: 01f4
1b7: e904
1b8: 1c10
1b9: 01f4
1ba: e904
1bb: 1c10
1bc: 01f4
1bd: e904
1be: 1c10
1bf: 01f4
1c0: 1c10
1c1: 01f4
1c2: 1c10
1c3: 01f4
1c4: 2000
1c5: 0000
1c6: 2000
1c7: 2000
1c8: 0000
1c9: 2000
1ca: 0800
1cb: 0800
1cc: 0800
1cd: 0800
1ce: 1c10
1cf: 01f4
1d0: e904
1d1: 1c10
1d2: 01f4
1d3: e904
1d4: 1c10
1d5: 01f4
1d6: e904
1d7: 1c10
1d8: 01f4
1d9: e904
1da: 1c10
1db: 01f4
1dc: 1c10
1dd: 01f4
1de: 0000
1df: 0000
1e0: 0000
1e1: 0800
1e2: 0800
1e3: 0000
1e4: 0000
1e5: 0200
1e6: 0000
1e7: 0000
1e8: 0000
1e9: 0000
1ea: 0000
1eb: 0000
1ec: 0000
1ed: 0000
1ee: 0000
1ef: 0200
1f0: 0000
1f1: 0000
1f2: 0000
1f3: 0000
1f4: 0000
1f5: 0000
1f6: 0000
1f7: 0000
1f8: 0000

> Also, could you do a loopback test?
> Open capture and playback simultaneously, then
> set 0x78 register bit 12 = 1.
> It will record what codec received. It is to make sure
> there is no problem on Intel side.

Stereo playing works fine on the headphones
output, so I do believe that me being unable to get the
right channel to be played on the left speaker because
there is only a left (mono) speaker really is a codec
routing problem.

Regards,

Hans