On Sun, May 10, 2015 at 7:54 AM, Fabio Estevam <feste...@gmail.com> wrote:
> Hi Tim,
>
> On Tue, Apr 28, 2015 at 12:44 PM, Tim Harvey <thar...@gateworks.com> wrote:
>
>> +#if defined(CONFIG_IMX6_THERMAL)
>> + switch (get_cpu_temp_grade(&minc, &maxc)) {
>> + case TEMP_AUTOMOTIVE:
>> + puts(" automotive");
>> + break;
>> + case TEMP_INDUSTRIAL:
>> + puts(" industrial");
>> + break;
>> + case TEMP_EXTCOMMERCIAL:
>> + puts(" extended commercial");
>> + break;
>> + default:
>> + puts(" commercial");
>
> Could you please change 'commercial' to 'consumer'?
>
> That's the term we see in the datasheet.
>
> I also agree with Nikolay that it would be better to split the CPU
> line info in two now.
>
> Thanks,
>
> Fabio Estevam
Fabio,
It looks like I convinced you in the thread regarding the similar
patch to the kernel
(http://www.spinics.net/lists/arm-kernel/msg417378.html) that the
verbiage should remain as is, but I will follow-up with a v2 that puts
the thermal information on the line with the current temperature.
Tim
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
