On Friday 26 of July 2013 16:25:51 Mark Brown wrote:
On Fri, Jul 26, 2013 at 05:02:46PM +0200, Tomasz Figa wrote:
IMHO this SoC-based identification looks much better, especially considering the fact that IP version isn't something easily determinable, as even the documentation can sometimes be not really clear about that.
Yeah, it's not terribly clever either way. We've been using the version numbers in audio for a long time partly because it is documented sometimes and partly because most of the SoCs tend to have one fully featured controller and a bunch of secondary controllers on older IP revisions.
However the s3c6410-i2sv4 string looks a bit unfortunate. AFAIK there were two types of I2S IPs on S3C6410 - normal I2S and I2S multichannel. What about having a compatible like s3c6410-i2s-multi?
It was explicitly identified as I2Sv4 in the S3C6410 datasheet so no real issue there.
Well, the datasheet I have calls it either "I2S V40" or "IIS MULTI AUDIO INTERFACE". I like the latter much more, because it actually says what's the difference compared to previous I2S IPs.
I'm not strongly against using the v4 suffix, but since we decided to use more meaningful compatible values elsewhere, I think this way would be better for sound drivers as well.
Best regards, Tomasz