From: Javi Merino <javi.merino@arm.com>
To: Catalin Crenguta <catalin.crenguta@gmail.com>
Cc: Eduardo Valentin <edubezval@gmail.com>,
Rui Zhang <rui.zhang@intel.com>,
Amit Daniel Kachhap <amit.kachhap@gmail.com>,
Viresh Kumar <viresh.kumar@linaro.org>,
linux-pm@vger.kernel.org,
Fabio Estevam <fabio.estevam@freescale.com>,
Bai Ping <b51503@freescale.com>
Subject: Re: Fwd: Freescale IMX6 active cooling
Date: Tue, 17 Nov 2015 09:56:50 +0000 [thread overview]
Message-ID: <20151117095649.GA2880@e104805> (raw)
In-Reply-To: <CABH3fy_KHmOWRDdLYO8_4vpXpD_dJTamDa4GVPY8nSbddgw+Jw@mail.gmail.com>
On Tue, Nov 17, 2015 at 08:12:56AM +0200, Catalin Crenguta wrote:
> +CC Maintainers
>
> ---------- Forwarded message ----------
> From: Catalin Crenguta <catalin.crenguta@gmail.com>
> Date: Mon, Nov 16, 2015 at 11:43 AM
> Subject: Freescale IMX6 active cooling
> To: "linux-arm-kernel@lists.infradead.org"
> <linux-arm-kernel@lists.infradead.org>
> Cc: fabio.estevam@freescale.com, Fabio Estevam <festevam@gmail.com>
>
>
> Hello,
>
> I'm trying to use a GPIO Fan to cool down a Riotboard (I.MX6 Solo).
> I've read [0], investigated the driver which provides the on-die
> temperature ( fsl,imx6q-tempmon [1]) but it seems to provide the
> temperature by creating a thermal zone (relevant dts: [2][3][4]).
> I don't know how I should proceed:
> 1) Should I research how can I attach a cooling device to this already
> existing thermal zone? (I believe it involves modifying the existing
> driver by adding new device tree handles)
Maybe you can use the gpio-fan driver in the kernel. See
Documentation/devicetree/bindings/gpio/gpio-fan.txt and
drivers/hwmon/gpio-fan.c .
> 2) Should I create another thermal zone, use the existing driver to
> get the temperature and bind to a cooling device?
No, you don't need another thermal zone.
> 3) Should I modify the existing driver to not create a thermal zone
> and move the creation in the device tree using the of-thermal driver ?
Converting the driver to device tree is the best option if all
existing users can be converted as well.
Cheers,
Javi
prev parent reply other threads:[~2015-11-17 9:56 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-16 9:43 Freescale IMX6 active cooling Catalin Crenguta
2015-11-16 14:51 ` Fabio Estevam
2015-11-17 9:38 ` Jon Nettleton
2015-11-17 12:49 ` Catalin Crenguta
2015-11-17 12:49 ` Catalin Crenguta
2015-11-18 4:53 ` Sean Cross
2015-11-18 4:53 ` Sean Cross
2015-11-18 10:00 ` Catalin Crenguta
2015-11-18 10:00 ` Catalin Crenguta
2015-11-18 21:56 ` Eduardo Valentin
2015-11-18 21:56 ` Eduardo Valentin
2015-11-17 6:12 ` Fwd: " Catalin Crenguta
2015-11-17 8:23 ` Viresh Kumar
2015-11-17 9:56 ` Javi Merino [this message]
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=20151117095649.GA2880@e104805 \
--to=javi.merino@arm.com \
--cc=amit.kachhap@gmail.com \
--cc=b51503@freescale.com \
--cc=catalin.crenguta@gmail.com \
--cc=edubezval@gmail.com \
--cc=fabio.estevam@freescale.com \
--cc=linux-pm@vger.kernel.org \
--cc=rui.zhang@intel.com \
--cc=viresh.kumar@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.