Messages in this thread |  | | Date | Thu, 07 May 2026 14:59:30 +0200 | | From | "Arnd Bergmann" <> | | Subject | Re: [RFC 4/4] m68k: coldfire: fix non-standard readX()/writeX() functions |
| |
On Thu, May 7, 2026, at 14:43, Greg Ungerer wrote: > On 7/5/26 05:12, Arnd Bergmann wrote: >> On Wed, May 6, 2026, at 16:26, Greg Ungerer wrote: >> >> I would suggest marking these as explicit BIG_ENDIAN rather than >> NATIVE_ENDIAN. The effect should be the same since coldfire CPUs >> cannot run little-endian code, but the way that hardware usually >> works is that the endianess is fixed at the bus level to one way >> or the other. NATIVE_ENDIAN to me implies that the registers >> have configurable endianess that is switched along with the CPU >> mode. > > Ok, will change. I chose native endian in this case since the regmap config > entry used for the m5441x family is also used by the vf610 devce (which looks > to be an ARM imx SoC). So it will need a duplicate setup with those endian > flags set to BIG_ENDIAN. But that is no problem.
Sounds good. In this case, splitting it up is technically even required, because you can run a big-endian ARMv7 kernel on vf610, so the vf610 entry has to use little-endian registers rather than native.
I don't think anyone has run big-endian kernels on vf610, though I have heard of users testing them successfully on i.MX6.
Arnd
|  |