codekipper

I've got this fix in my stack for mainline but I don't think I'll have a chance to push them before the New Year https://github.com/codekipper/linux-sunxi/commit/67d47db0499e08a985fe8fbb2517f26c52e10476 I've not really tested much with capture so I've been a bit reluctant to push. Hopefully New Year and a new attitude to sharing! CK

This is not possible with your setup, based on the information in the A20 i2s documentation. From what I remember when I worked with the PCM5102A then I needed to provide a 32bit sample(this has also been seen with the HDMI audio), do you have these tdm settings in your dts/overlay? https://github.com/codekipper/linux-sunxi/commit/977b5838a184350fa8e794420a947d31d81c2e54#diff-9fb7f343370c25c84e919e9a8e392b74R200

I've been distracted by the audioinjector codec of late and it's driven some changes that I'll push later once tested with other hardware https://github.com/codekipper/linux-sunxi/commits/audioinjector CK

Congratulations, this is the first time slave clock has been utilised on the mainline driver. Do you have my debug enabled?, it would be nice to see register settings during playback. I'll follow the same test procedures when I get my new codec.

I guess we've just never had all the possible information, the oversample rates were documented in the A20 user manual (128 was the lowest)but not in the later iterations.

please do....I've currently got a logic analyser on my pcm5102a boards and I'm seeing issues with the clock division which I will address. It would be good to have someone testing on an older board as my A20 EVB is no longer booting.

mainline should support slave mode https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/sound/soc/sunxi/sun4i-i2s.c?h=v4.15-rc2#n502 there is also a few patches which I haven't delivered yet here https://github.com/codekipper/linux-sunxi/commits/sunxi-wip however I still need to test these properly before pushing for mainline. I'm also trying to come up with a cleaner solution for when the codec expects 32bits frame width.

OK, we can reuse most of the working overlay and adapt it for the h3. Slave mode should be supported by the mainline driver but I've not tested it, however I do have a master codec in the post which should get to me sometime this week. Do you have access to a logic analyser?, BR, CK

It would make more sense to work with the mainline driver and fix issues with it over than working with this sun8i-i2s driver. So from what I read you have an external G-Dis DAC and some highly accurate clock board which you wish to clock both the dac and SoC with. What other connections do you have from the nanopi?, are the gpio's mentioned in the overlay used? Overlays should be fairly straight forward, can you try this?. // Definitions for G-Dis DAC /dts-v1/; /plugin/; / { compatible = "allwinner,sun8i-h3"; fragment@0 { target = <&i2s0>; __overlay__ { pinctrl-0 = <&i2s0_pins>; pinctrl-names = "default"; status = "okay"; }; }; fragment@1 { target-path = "/"; __overlay__ { i2s0_out: i2s0-out { #sound-dai-cells = <0>; compatible = "linux,spdif-dit"; }; sound_i2s { simple-audio-card,name = "G-Dis DAC"; compatible = "simple-audio-card"; simple-audio-card,format = "i2s"; simple-audio-card,bitclock-master = <&codec_dai>; simple-audio-card,frame-master = <&codec_dai>; simple-audio-card,mclk-fs = <256>; status = "okay"; simple-audio-card,cpu { sound-dai = <&i2s0>; }; codec_dai: simple-audio-card,codec { sound-dai = <&i2s0_out>; }; }; }; }; }; What did you use to get it working with the Raspberry pi?, CK

RJF and slave mode should work with the sun4i-i2s driver, although I've only tested i2s, LJR and master. The sun8i-i2s driver shouldn't be used but may prove useful for debugging(I used it for the lrclk setting to get hdmi audio working!). Please show me your hardware. I've not seen a pcm5102 board with on-board clocks. CK

Hi, which board is being used as an i2s slave?...The current mainline driver should support slave configuration but I've never tested it(audioinjector board is on its way). It's also unclear what these clk-gpios are for; is it just logic to set the clocking rate? and if that is the case then it should be in the codec driver and not the dai. CK