xmos / lib_awe

AWE Core for xcore
Other
0 stars 1 forks source link

Simplification of audio channel count defines & fix #23 #26

Closed xross closed 3 months ago

xross commented 3 months ago

(Note, I fixed up all the example apps, not sure we're keeping them all though?)

xross commented 3 months ago

Looks good and glad to see you found the doc mention of AUDIO_INPUT_CHANNELS too. Only two questions:

  • What to do about the other 3 apps - I think we both felt they don't belong in the main branch. Probs on a fork?
  • xcommon support. Let's decide one way or another and stick to it.

Unless I missed something it looks like all four apps can be build configs of the one app.

I don't see an issue adding xcommon support if we have to (based on migrating lead customer). But adding a readme.txt about an xs1b lib doesn't mean we have to do it either (even if one will never exist).