[U-Boot] [PATCH v6 2/4] mx6: thermal: Add i.MX6 CPU thermal sensor support

[Otavio Salvador]

On Wed, Nov 19, 2014 at 7:08 AM, Stefano Babic <sbabic at denx.de> wrote:
> On 18/11/2014 15:16, Fabio Estevam wrote:
>> On Mon, Nov 17, 2014 at 10:49 AM, Stefano Babic <sbabic at 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 ?

I agree with the general idea but as Fabio I believe it is wrong to
demand this change now. This patch has been send long time ago and
been in review process for quite some time. Every new submission for
drivers should be requested to move to DM framework however this is
not new (it was sent in August).

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750