Messages in this thread | | | Date | Thu, 27 Mar 2014 11:40:39 +0000 | From | Mark Brown <> | Subject | Re: [PATCH] regmap: Add REGMAP_ENDIAN_SWAP support for values. |
| |
On Thu, Mar 27, 2014 at 04:17:39PM +0800, Xiubo Li wrote:
> For the DT node, just one property like 'endian-swap' will be okey > for cases 2 and 4.
I'm not convinced that the cost of having to define explicit big and little endian properties for the hardware is worth having this - it seems like a confusing thing to have in the interface since it depends on both the device and the CPU and would presumably break in cases where there's an option about which endianness to run things in.
> And using the REGMAP_ENDIAN_BIG and REGMAP_ENDIAN_LITTLE will make > the driver a bit more complex, and also the usage of it.
What's the complexity here? [unhandled content-type:application/pgp-signature] | |