devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Pankaj Bansal <pankaj.bansal@nxp.com>
To: wg@grandegger.com, mkl@pengutronix.de, linux-can@vger.kernel.org,
	robh+dt@kernel.org, mark.rutland@arm.com,
	devicetree@vger.kernel.org
Cc: V.Sethi@nxp.com, poonam.aggrwal@nxp.com,
	Pankaj Bansal <pankaj.bansal@nxp.com>
Subject: [PATCH v4 0/6] Remodel FlexCAN register r/w APIs for big endian
Date: Fri, 24 Nov 2017 18:52:07 +0530	[thread overview]
Message-ID: <1511529733-27942-1-git-send-email-pankaj.bansal@nxp.com> (raw)

This patch series remodels the FlexCAN register r/w APIs for big endian.
The endianness is checked based on optional big-endian property in
device tree. if this property is not present in device tree node then 
controller is assumed to be little endian. if this property is present then
controller is assumed to be big endian.

An exception to this rule is powerpc P1010RDB, which is always
big-endian, even if big-endian is not present in dts. This is
checked using p1010-flexcan compatible in dts.

Therefore, remove p1010-flexcan compatible from imx series dts,
as their flexcan core is little endian.

Finally this series adds support for NXP LS1021A SOC in flexcan,
which is arm based SOC having big-endian FlexCAN controller.

Pankaj Bansal (6):
  can: flexcan: Remodel FlexCAN register r/w APIs for big endian FlexCAN
    controllers.
  can: flexcan: adding platform specific details for LS1021A
  Documentation : can : flexcan : Add big-endian property to device tree
  powerpc: dts: P1010: Add endianness property to flexcan node
  arm: dts: Remove p1010-flexcan compatible from imx series dts
  arm/dts: Add nodes for flexcan devices present on LS1021A-Rev2 SoC

 .../bindings/net/can/fsl-flexcan.txt         |   6 +
 arch/arm/boot/dts/imx25.dtsi                 |   4 +-
 arch/arm/boot/dts/imx28.dtsi                 |   4 +-
 arch/arm/boot/dts/imx35.dtsi                 |   4 +-
 arch/arm/boot/dts/imx53.dtsi                 |   4 +-
 arch/arm/boot/dts/ls1021a-qds.dts            |  16 ++
 arch/arm/boot/dts/ls1021a-twr.dts            |  16 ++
 arch/arm/boot/dts/ls1021a.dtsi               |  36 +++
 arch/powerpc/boot/dts/fsl/p1010si-post.dtsi  |   2 +
 drivers/net/can/flexcan.c                    | 241 ++++++++++-------
 10 files changed, 223 insertions(+), 110 deletions(-)

-- 
2.7.4


             reply	other threads:[~2017-11-24 13:22 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-24 13:22 Pankaj Bansal [this message]
2017-11-24 13:22 ` [PATCH v4 4/6] powerpc: dts: P1010: Add endianness property to flexcan node Pankaj Bansal
2017-11-24 13:22 ` [PATCH v4 5/6] arm: dts: Remove p1010-flexcan compatible from imx series dts Pankaj Bansal
     [not found] ` <1511529733-27942-1-git-send-email-pankaj.bansal-3arQi8VN3Tc@public.gmane.org>
2017-11-24 13:22   ` [PATCH v4 1/6] can: flexcan: Remodel FlexCAN register r/w APIs for big endian FlexCAN controllers Pankaj Bansal
     [not found]     ` <1511529733-27942-2-git-send-email-pankaj.bansal-3arQi8VN3Tc@public.gmane.org>
2017-11-24 15:02       ` Marc Kleine-Budde
     [not found]         ` <44f37aef-7f45-a7c7-7320-01a62e23fbde-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
2017-11-26  2:20           ` Pankaj Bansal
2017-11-27 14:06             ` Marc Kleine-Budde
2017-11-24 13:22   ` [PATCH v4 2/6] can: flexcan: adding platform specific details for LS1021A Pankaj Bansal
2017-11-24 13:22   ` [PATCH v4 3/6] Documentation : can : flexcan : Add big-endian property to device tree Pankaj Bansal
     [not found]     ` <1511529733-27942-4-git-send-email-pankaj.bansal-3arQi8VN3Tc@public.gmane.org>
2017-11-26 22:18       ` Rob Herring
2017-11-24 13:22   ` [PATCH v4 6/6] arm/dts: Add nodes for flexcan devices present on LS1021A-Rev2 SoC Pankaj Bansal
2017-11-27 14:07   ` [PATCH v4 0/6] Remodel FlexCAN register r/w APIs for big endian Marc Kleine-Budde

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1511529733-27942-1-git-send-email-pankaj.bansal@nxp.com \
    --to=pankaj.bansal@nxp.com \
    --cc=V.Sethi@nxp.com \
    --cc=devicetree@vger.kernel.org \
    --cc=linux-can@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=mkl@pengutronix.de \
    --cc=poonam.aggrwal@nxp.com \
    --cc=robh+dt@kernel.org \
    --cc=wg@grandegger.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).