20 Dec
2017
20 Dec
'17
12:26 p.m.
On Tue, Dec 19, 2017 at 08:17:37PM +0100, Takashi Iwai wrote:
Well, honestly speaking, that complete morphing scares me. If it's a simple additional call, it's easier to track the code flow. But your patch essentially fakes the legacy codec and bus structs and twists the control by that switch... which looks too fragile to me.
Especially when it's layered on top of all the other x86 code which is also complicated and fragile - feels like there's lots of places things can go wrong and lots of potential unexpected interactions.