[PATCH 4/4] ASoC: fsl: make fsl_ssi driver compilable on ARM/IMX
Timur Tabi
timur at freescale.com
Fri Feb 24 18:22:14 EST 2012
Russell King - ARM Linux wrote:
> What would be wrong with converting in_be32() in these drivers to
> instead ioread32be()?
Well, it wouldn't actually fix any problems. It's just a stylistic
change for PowerPC. On ARM, however, ioread32be() is wrong because the
SSI registers are little-endian.
In case you're confused: the SSI is an I2S audio controller developed by
the i.MX (ARM) division on Freescale. When the PowerPC division wanted to
add an I2S controller to its SOCs, they just took the same SSI device and
dropped in the PowerPC SOC. But since PowerPC is big-endian, the
registers are byte-swapped in hardware. So even though the i.MX SSI and
the PowerPC SSI have the same registers and do the same thing, those
actual registers are little-endian on i.MX and big-endian on PowerPC.
As far as I know, there is no single function in the kernel that will
perform an ordered 32-bit big-endian read on PowerPC and an ordered 32-bit
little-endian read on ARM.
--
Timur Tabi
Linux kernel developer at Freescale
More information about the linux-arm-kernel
mailing list