Re: [PATCH 8/9] Remove most checks of __BYTE_ORDER

Jason Gunthorpe
2016-09-30T00:08:00Z

On Thu, Sep 29, 2016 at 06:46:02PM -0500, Steve Wise wrote:
> > I added the comment because cxgb* was the only provider that seemed to
> > be obviously byte swapping MMIO.
> >
> > Since htole32 is a NOP on x86 and ARM64 the driver would work fine
> > with or without it. Do you know this driver works on big endian
> > powerpc? That would help answer the question for sure.
>
> Yes, PowerPC and arm.

Okay great. Maybe Yishai can explain what mlx drivers are doing..

Is this better:

/* NOTE: The endianess of PCI MMIO accesses is architecture specific,
this is known correct for BE PPC and LE x86/ARM */

Thanks again Steve,
Jason