On 05/12/2016 11:31 AM, Mark Brown wrote: > On Mon, May 09, 2016 at 10:58:08AM +0000, Yao Yuan wrote: > >> For example, if DSPI controller register is BE, so I set big-endian in DT. >> That means we should R/W the DSPI controller register with big-endian. >> Then I can think all of the value we R/W to/from DSPI controller register, we can >> Think they are the BE. >> So that we can understand it easy. > >> That means no matter core is LE or BE, we both need get the value from DSPI register >> with big endian. > > That's not how regmap is intended to be used, the intention is that the > driver will interact with native endian values and regmap will hide the > endianness changes. You could mask this with some cpu_to_be() and > be_to_cpu() usage but I'm not sure it's a great idea or what it's buying > you. I don't think that's what Yao Yuan meant. The driver wants the I/O accessor to return native-endian values, just like any other driver. -Scott -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html