Re: [PATCH 2/2] hwmon: (max6650) Add initial rpm target devicetree documentation

[Mike Looijmans <mike.looijmans@topic.nl>]

On 30-08-16 18:38, Rob Herring wrote:
> On Wed, Aug 24, 2016 at 10:13:27AM +0200, Mike Looijmans wrote:
>> Add devicetree property for early initialization of the fan controller
>> to prevent overheating, for example when resetting the board while the
>> fan was completely turned off.
>>
>> Signed-off-by: Mike Looijmans <mike.looijmans@topic.nl>
>> ---
>>   Documentation/devicetree/bindings/hwmon/max6650.txt | 5 +++++
>>   1 file changed, 5 insertions(+)
>
> The kernel boot seems kind of late to ensure your fan is running, but
>
> Acked-by: Rob Herring <robh@kernel.org>

Thanks!

Well, "late's better than never," my grandma used to say when we visited :)

To put your mind at ease, the particular platform I'm using this for is 
an xc7z045 (Zynq) where the ARMs running the Linux kernel will only 
account for 0.5W, but once the logic awakens the chip can dissipate up 
to 30W. The logic bitstream is usually on the rootfs, so kernel boot is 
early enough for the fan to start spinning. And with the logic asleep, 
it need not spin at all.


-- 
Mike Looijmans