16kHz not supported - Githubissues

andrewstanfordjason commented 8 years ago

the divider cannot achieve 16kHz which is really common for voice applications

ed-xmos commented 8 years ago

The newly added i2s_frame_master() can support this. As long as the BCLK can be divided from the MCLK using the clock block then all should be fine.

Eg. 12.288MHz / 1.024MHz = (6 x 2) -> Sorted

ThomasGmeinder commented 8 years ago

Indeed :) I have changed the application to use i2s_frame_master instead of i2s_master the whole Audio path PDM -> I2S is now working at 16 kHz.

From: Ed notifications@github.com<mailto:notifications@github.com> Reply-To: xmos/lib_i2s reply@reply.github.com<mailto:reply@reply.github.com> Date: Monday, 20 June 2016 08:29 To: xmos/lib_i2s lib_i2s@noreply.github.com<mailto:lib_i2s@noreply.github.com> Subject: Re: [xmos/lib_i2s] 16kHz not supported (#19)

The newly added i2s_frame_master() can support this. As long as the BCLK can be divided from the MCLK using the clock block then all should be fine.

Eg. 12.288MHz / 1.024MHz = (6 x 2) -> Sorted

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/xmos/lib_i2s/issues/19#issuecomment-227060011, or mute the threadhttps://github.com/notifications/unsubscribe/AAqhhr_YI8xibtTsTjw9Ty2DAcN2Z9uSks5qNjNigaJpZM4I4YOO.

pthedinger commented 8 years ago

From reading the comments, this is no longer an issue.

samchesney commented 6 years ago

Reopening this issue as it is not clear from the documentation that 16KHz is supported. The Master speeds and performance section does not list support for any sample frequencies below 48kHz, and does not highlight any differences the supported rates of the various master implementations.

ed-xmos commented 6 years ago

This can be close now due to https://github.com/xmos/lib_i2s/pull/45

samchesney commented 6 years ago

Thank you @ed-xmos

xmos / lib_i2s

16kHz not supported #19