All of lore.kernel.org
 help / color / mirror / Atom feed
From: Leonard Crestez <leonard.crestez@nxp.com>
To: "Zhang Rui" <rui.zhang@intel.com>,
	"Eduardo Valentin" <edubezval@gmail.com>,
	"Srinivas Kandagatla" <srinivas.kandagatla@linaro.org>,
	"Shawn Guo" <shawnguo@kernel.org>,
	"Rob Herring" <robh+dt@kernel.org>,
	"Mark Rutland" <mark.rutland@arm.com>,
	"Lothar Waßmann" <LW@KARO-electronics.de>
Cc: Fabio Estevam <fabio.estevam@nxp.com>,
	Dong Aisheng <aisheng.dong@nxp.com>, Bai Ping <ping.bai@nxp.com>,
	Anson Huang <Anson.Huang@nxp.com>,
	Octavian Purdila <octavian.purdila@nxp.com>,
	linux-pm@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: [PATCH 0/4] thermal: imx: Add nvmem-cells binding on imx6sx
Date: Thu, 6 Jul 2017 16:20:40 +0300	[thread overview]
Message-ID: <cover.1499347157.git.leonard.crestez@nxp.com> (raw)

On imx6sx accessing OCOTP directly is wrong because the ocotp clock needs to be
enabled first. Fix this by adding a nvmem-cells binding and using it on imx6sx,
imx6ul and imx6ull.

The existing binding is kept around because it works fine on imx6qdl.

This was initially reported by Lothar Waßmann <LW@KARO-electronics.de> in reply
to a mail adding imx6ul/ull support:

Link: https://lkml.org/lkml/2017/6/9/578

A previous attempt just reinterpreted the fsl,tempmon-data phandle as nvmem.
Code was actually written to use nvmem-cells first but I thought that
reinterpreting existing devicetree properties would be interesting.

Link: https://lkml.org/lkml/2017/6/19/333

Leonard Crestez (4):
  thermal: imx: Add nvmem-cells alternate binding for OCOTP access
  thermal: imx: Add support for reading OCOTP through nvmem
  ARM: dts: imx6sx: Use nvmem-cells for tempmon
  ARM: dts: imx6ul: Add imx6ul-tempmon

 .../devicetree/bindings/thermal/imx-thermal.txt    |   7 ++
 arch/arm/boot/dts/imx6sx.dtsi                      |  11 +-
 arch/arm/boot/dts/imx6ul.dtsi                      |  17 +++
 drivers/thermal/imx_thermal.c                      | 131 ++++++++++++++++-----
 4 files changed, 135 insertions(+), 31 deletions(-)

-- 
2.7.4

WARNING: multiple messages have this Message-ID (diff)
From: Leonard Crestez <leonard.crestez@nxp.com>
To: "Zhang Rui" <rui.zhang@intel.com>,
	"Eduardo Valentin" <edubezval@gmail.com>,
	"Srinivas Kandagatla" <srinivas.kandagatla@linaro.org>,
	"Shawn Guo" <shawnguo@kernel.org>,
	"Rob Herring" <robh+dt@kernel.org>,
	"Mark Rutland" <mark.rutland@arm.com>,
	"Lothar Waßmann" <LW@KARO-electronics.de>
Cc: Fabio Estevam <fabio.estevam@nxp.com>,
	Dong Aisheng <aisheng.dong@nxp.com>, Bai Ping <ping.bai@nxp.com>,
	Anson Huang <Anson.Huang@nxp.com>,
	Octavian Purdila <octavian.purdila@nxp.com>,
	<linux-pm@vger.kernel.org>, <devicetree@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>
Subject: [PATCH 0/4] thermal: imx: Add nvmem-cells binding on imx6sx
Date: Thu, 6 Jul 2017 16:20:40 +0300	[thread overview]
Message-ID: <cover.1499347157.git.leonard.crestez@nxp.com> (raw)

On imx6sx accessing OCOTP directly is wrong because the ocotp clock needs to be
enabled first. Fix this by adding a nvmem-cells binding and using it on imx6sx,
imx6ul and imx6ull.

The existing binding is kept around because it works fine on imx6qdl.

This was initially reported by Lothar Waßmann <LW@KARO-electronics.de> in reply
to a mail adding imx6ul/ull support:

Link: https://lkml.org/lkml/2017/6/9/578

A previous attempt just reinterpreted the fsl,tempmon-data phandle as nvmem.
Code was actually written to use nvmem-cells first but I thought that
reinterpreting existing devicetree properties would be interesting.

Link: https://lkml.org/lkml/2017/6/19/333

Leonard Crestez (4):
  thermal: imx: Add nvmem-cells alternate binding for OCOTP access
  thermal: imx: Add support for reading OCOTP through nvmem
  ARM: dts: imx6sx: Use nvmem-cells for tempmon
  ARM: dts: imx6ul: Add imx6ul-tempmon

 .../devicetree/bindings/thermal/imx-thermal.txt    |   7 ++
 arch/arm/boot/dts/imx6sx.dtsi                      |  11 +-
 arch/arm/boot/dts/imx6ul.dtsi                      |  17 +++
 drivers/thermal/imx_thermal.c                      | 131 ++++++++++++++++-----
 4 files changed, 135 insertions(+), 31 deletions(-)

-- 
2.7.4

             reply	other threads:[~2017-07-06 13:20 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-06 13:20 Leonard Crestez [this message]
2017-07-06 13:20 ` [PATCH 0/4] thermal: imx: Add nvmem-cells binding on imx6sx Leonard Crestez
     [not found] ` <cover.1499347157.git.leonard.crestez-3arQi8VN3Tc@public.gmane.org>
2017-07-06 13:20   ` [PATCH 1/4] thermal: imx: Add nvmem-cells alternate binding for OCOTP access Leonard Crestez
2017-07-06 13:20     ` Leonard Crestez
2017-07-10 13:29     ` Rob Herring
2017-07-06 13:20 ` [PATCH 2/4] thermal: imx: Add support for reading OCOTP through nvmem Leonard Crestez
2017-07-06 13:20   ` Leonard Crestez
     [not found]   ` <68b476b35b0c0cbce21da0e87338323bb77e9bc4.1499347157.git.leonard.crestez-3arQi8VN3Tc@public.gmane.org>
2017-07-12  6:36     ` Shawn Guo
2017-07-12  6:36       ` Shawn Guo
2017-07-14  8:48       ` Srinivas Kandagatla
     [not found]         ` <3dd43cba-02f2-204c-c3a2-582827018ef8-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2017-07-14 10:49           ` Leonard Crestez
2017-07-14 10:49             ` Leonard Crestez
2017-07-14 10:54             ` Srinivas Kandagatla
2017-07-06 13:20 ` [PATCH 3/4] ARM: dts: imx6sx: Use nvmem-cells for tempmon Leonard Crestez
2017-07-06 13:20   ` Leonard Crestez
     [not found]   ` <6dc9bb756daf75bb26648b06b204872f71b3a548.1499347157.git.leonard.crestez-3arQi8VN3Tc@public.gmane.org>
2017-07-12  6:40     ` Shawn Guo
2017-07-12  6:40       ` Shawn Guo
2017-07-06 13:20 ` [PATCH 4/4] ARM: dts: imx6ul: Add imx6ul-tempmon Leonard Crestez
2017-07-06 13:20   ` Leonard Crestez

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=cover.1499347157.git.leonard.crestez@nxp.com \
    --to=leonard.crestez@nxp.com \
    --cc=Anson.Huang@nxp.com \
    --cc=LW@KARO-electronics.de \
    --cc=aisheng.dong@nxp.com \
    --cc=devicetree@vger.kernel.org \
    --cc=edubezval@gmail.com \
    --cc=fabio.estevam@nxp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=octavian.purdila@nxp.com \
    --cc=ping.bai@nxp.com \
    --cc=robh+dt@kernel.org \
    --cc=rui.zhang@intel.com \
    --cc=shawnguo@kernel.org \
    --cc=srinivas.kandagatla@linaro.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.