On Wed, Mar 14, 2012 at 06:02:44PM -0500, Timur Tabi wrote:
Sascha Hauer wrote:
What are the next steps here? Now we have fsl_ssi.c and imx-ssi.c, one for dt bindings and the other one for platform bindings and also AC97 support. They should probably get merged some day which might be a bit difficult since currently I don't see the platform users moving over to devicetree and the seem to be no AC97 users in powerpc.
I don't have any problem with adding AC97 support to fsl_ssi.c, assuming it's not a huge mess. I don't know enough about AC97 to know how easy that is, though.
_From the hardware side it's basically calling setup_channel_to_ac97() and providing AC97 register read/write functions. One ugly detail is that we need a bus reset function. This basically means putting the pins in gpio mode and issuing the reset manually. The SSI unit does not provide a way to reset the bus. I don't know how this AC97 stuff integrates into the devicetree. Anyway, that's a topic for future discussions and shouldn't block the current patches. I just want to make sure that the original imx-ssi driver is not removed as long as the fsl_ssi driver does not have AC97 support.
Sascha