On 11/23/2017 10:09 AM, Pankaj Bansal wrote: > The FlexCAN driver assumed that FlexCAN controller is big endian for > powerpc architecture and little endian for other architectures. > > But this may not be the case. FlexCAN controller can be little or big > endian on any architecture. For e.g. NXP LS1021A ARM based SOC has big > endian FlexCAN controller. > > Therefore, the driver has been modified to add a provision for both > types of controllers using an additional device tree property. On a > "fsl,p1010-flexcan" device BE is default, on all other devices LE is. > > Big Endian controllers should have "big-endian" set in the device tree. > check "Documentation/devicetree/bindings/net/can/fsl-flexcan.txt" for > usage. > > This is the standard practice followed in linux. for more info check: > Documentation/devicetree/bindings/common-properties.txt > > Signed-off-by: Pankaj Bansal > Signed-off-by: Bhupesh Sharma > Signed-off-by: Sakar Arora > Reviewed-by: Zhengxiong Jin > Reviewed-by: Poonam Aggrwal I'm missing the update of the fsl-flexcan.txt in this series. Marc -- Pengutronix e.K. | Marc Kleine-Budde | Industrial Linux Solutions | Phone: +49-231-2826-924 | Vertretung West/Dortmund | Fax: +49-5121-206917-5555 | Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de |