From: Vijay Kiran Kamuju Subject: Re: [PATCH 1/3] include: Add xact3.h Message-Id: Date: Sat, 27 Jun 2020 09:14:22 +0200 In-Reply-To: References: Hi Alistair On Sat, 27 Jun 2020, 09:07 Alistair Leslie-Hughes, < leslie_alistair@hotmail.com> wrote: > Hi Vijay, > > My main concern with this patch is that is missing all other interfaces > the the currently staging IDL has (IXACT3SoundBank, IXACT3WaveBank, > IXACT3Cue and IXACT3Wave). I was supposed to split the original patch according to the review. I would add the other interfaces when implementing them, they are the next series of patches. If you have better ideas on how to split the original patch, please let me know. > > Regards > Alistair. > Thanks, Vijay >
HiĀ Alistair

On Sat, 27 Jun 2020, 09:07 Alistair Leslie-Hughes, <leslie_alistair@hotmail.com> wrote:
Hi Vijay,

My main concern with this patch is that is missing all other interfaces
the the currently staging IDL has (IXACT3SoundBank, IXACT3WaveBank,
IXACT3Cue and IXACT3Wave).
I was supposed to split the original patch according to the review.
I would add the other interfaces when implementing them, they are the next series of patches.
If you have better ideas on how to split the original patch, please let me know.

Regards
Alistair.
Thanks,
Vijay