On Thu, May 05, 2016 at 09:06:44PM +0200, Robert Jarzmik wrote:
Mark Brown broonie@kernel.org writes:
I see that, I just don't know why that helps.
Ok, so let me explain.
As you already understood, AC97_VENDOR* is needed in bus.c for the autoscanning. The remaining is the reason why I don't want to include ac97_codec.h in bus.c, which follows.
In order to have a clean split between former ac97 bus implementation and this new one, I didn't want to include any former ac97 includes, excepting in sound/ac97/compat.c.
It's probably a good idea to articulate some of this in the commit log then!
This transition will be easier if the new ac97 code is isolated as much as possible from the former code while providing backward compatibility ... hence this patch.
It'd probably have been clearer to me if it were splitting out all the register definitions rather than just some of them.