From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v6 2/4] mx6: thermal: Add i.MX6 CPU thermal sensor support
Date: Wed, 19 Nov 2014 10:08:33 +0100 [thread overview]
Message-ID: <546C5E11.3040206@denx.de> (raw)
In-Reply-To: <CAOMZO5AjZZzjD5E+76eaXaKid41grTD9hMbzyp=udzAeVs1V2A@mail.gmail.com>
Hi Fabio,
On 18/11/2014 15:16, Fabio Estevam wrote:
> Hi Stefano,
>
> On Mon, Nov 17, 2014 at 10:49 AM, Stefano Babic <sbabic@denx.de> wrote:
>
>> I have some conceptional question here. This introduces a new set or,
>> better, "class" of drivers, as we are used to see in kernel as
>> "thermal". As we discussed in last u-boot mini summit, the preferred way
>> to introduce new drivers into current u-boot is to use DM (Driver
>> Model), because this is the way u-boot will follow in the future. It
>> will take some time porting the current drivers, but it is easier for
>> drivers introducing new features.
>
> It is a good idea to convert this to the driver model, but Nitin is
> already at v6 and only now this request is made.
> First version of this
> patch series was sent back in August:
> http://lists.denx.de/pipermail/u-boot/2014-August/187685.html
>
> Could the conversion to DM be handled later by a separate patch?
It is fine with me, but we agreed on the last mini summit last month to
accept new drivers if they are developped under DM rules. Tom, can the
driver be accepted and later converted to DM, or this driver (it is the
first driver regarding thermal) must be converted first ?
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================
next prev parent reply other threads:[~2014-11-19 9:08 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-14 20:13 [U-Boot] [PATCH v6 0/5] add i.MX6 thermal sensor driver nitin.garg at freescale.com
2014-11-14 20:13 ` [U-Boot] [PATCH v6 1/4] mx6: clock: Add api to enable pll3 nitin.garg at freescale.com
2014-11-14 20:13 ` [U-Boot] [PATCH v6 2/4] mx6: thermal: Add i.MX6 CPU thermal sensor support nitin.garg at freescale.com
2014-11-17 12:49 ` Stefano Babic
2014-11-18 14:16 ` Fabio Estevam
2014-11-19 9:08 ` Stefano Babic [this message]
2014-11-19 13:55 ` Otavio Salvador
2014-11-14 20:13 ` [U-Boot] [PATCH v6 3/4] mx6: thermal: Check cpu temperature via thermal sensor nitin.garg at freescale.com
2014-11-14 20:13 ` [U-Boot] [PATCH v6 4/4] mx6: thermal: Enable thermal sensor for mx6 sabre boards nitin.garg at freescale.com
2014-11-15 12:34 ` [U-Boot] [PATCH v6 0/5] add i.MX6 thermal sensor driver Albert ARIBAUD
2014-11-15 22:34 ` Nitin Garg
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=546C5E11.3040206@denx.de \
--to=sbabic@denx.de \
--cc=u-boot@lists.denx.de \
/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