From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Fri, 15 May 2015 15:44:50 +0200 Subject: [U-Boot] [PATCH v2 3/4] imx: mx6: add display of CPU temperature grade in print_cpuinfo() In-Reply-To: References: <1431580312-24880-1-git-send-email-tharvey@gateworks.com> <1431580312-24880-4-git-send-email-tharvey@gateworks.com> <5555F3A2.9090906@denx.de> Message-ID: <5555F852.5040302@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Tim, On 15/05/2015 15:31, Tim Harvey wrote: >> Right. My proposal is to merge the currrent patchset, enabling for >> IMX6SX again when someone will test it. >> >> Best regards, >> Stefano Babic >> > > Stefano, > > Yes, that sounds like the best approach. What were your thoughts on > enabling the second CPU: line just to display the temperature grade if > CONFIG_IMX6_THERMAL was not enabled? As you said, printing a lot can be disturbing for someone, and letting the possibility to turn out prints is not bad. I will let the board maintainer to decide it. > > I'm surprised someone on this list hasn't already tested this on IMX6SX. > I am quite convinced that patches are tested by a few until they are merged. If there is a ready-to-use tree, people are more motivated to test it. > Fabio - certainly you must have a reference board with an IMX6SX and > could enable/test that the CPU markings agree with what is > detected/displayed via OTP? > Of course, I could delay for applying this patchset if Fabio / someone has the possibility to test it with 6X. Anyway, running on 6X is protected by the check on the cpu, and I do not see problems to merge it. Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk 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 =====================================================================