<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">Jarielle,<br>
Ok, looks good. I was able to do a full-range RAM test!<br>
I pushed everything, including the cpuio.c change and also I<br>
removed INCLUDE_DM and INCLUDE_PM from config.h because
INCLUDE_MEMCMDS<br>
pulls them all in.<br>
Good work (again) ! :-)<br>
Meanwhile, I'm just going to let my board soak with the memory
test running..<br>
"mt -vc 0x80100000 0x1fefffff" <br>
Hopefully I'll come back in few hours and it will still be
running!<br>
<br>
Next is MMC right?<br>
Two nice features come as a result of the MMC-write capability...<br>
1. You can boot from eMMC instead of uSD (so no need to hold
down the button).<br>
2. You'll be able to do upgrades without manually needing to
remove th uSD card.<br>
That'll be sweet.<br>
Ed<br>
</div>
<blockquote
cite="mid:CANjxxd3LRtdn5MDL4S47-vNqo1d2cSiO3ZAjdqguBJJgSCwDFA@mail.gmail.com"
type="cite">
<p dir="ltr">Ed,</p>
<p dir="ltr">Your assumption is right, it should have been
cm_clksel_dpll_mpu. Feel free to make the change as part of the
push, otherwise let me know if I should resubmit the patches.</p>
<p dir="ltr">Thanks.</p>
<div class="gmail_quote">On Jul 18, 2015 6:24 AM, "Ed Sutter" <<a
moz-do-not-send="true" href="mailto:edsutterjr@gmail.com">edsutterjr@gmail.com</a>>
wrote:<br type="attribution">
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">Jarielle,<br>
Just pulled in the patches and built, but there's a warning
generated<br>
when cpuio.c is compiled...<br>
<br>
> cpuio.c: In function 'mpu_pll_init':<br>
> cpuio.c:286:21: warning: 'cm_clksel_dpll_mpu' is used
uninitialized in this function [-Wuninitialized]<br>
> cm_clksel_dpll_mpu &= ~0x0007FF7F;<br>
<br>
Without digging in too deep, it looks like a typo.<br>
Line 285 should be initializing cm_clksel_dpll_mpu instead of
cmclkmode_dpll_mpu...<br>
<br>
// Set the ARM core frequency to 1 GHz<br>
cm_clkmode_dpll_mpu = CM_WKUP_REG(CM_CLKSEL_DPLL_MPU);<br>
cm_clksel_dpll_mpu &= ~0x0007FF7F;
<<<<<This is line 286<br>
cm_clksel_dpll_mpu |= 1000 << 8;<br>
cm_clksel_dpll_mpu |= 23;<br>
CM_WKUP_REG(CM_CLKSEL_DPLL_MPU) = cm_clksel_dpll_mpu;<br>
cm_div_m2_dpll_mpu = CM_WKUP_REG(CM_DIV_M2_DPLL_MPU);<br>
cm_div_m2_dpll_mpu &= ~0x0000001F;<br>
cm_div_m2_dpll_mpu |= 0x00000001;<br>
CM_WKUP_REG(CM_DIV_M2_DPLL_MPU) = cm_div_m2_dpll_mpu;<br>
<br>
<br>
Assuming you have the same set of tools I have (from RTEMS),
then you probably missed<br>
the warning because it streamed off your screen quickly (it
was at the top of the build<br>
output). To avoid that problem, just pipe all make output to
a file...<br>
<br>
make rebuild 2>&1 | tee make.out<br>
<br>
Then grep for "arning" in the make.out file or just open the
file and check it visually.<br>
<br>
Lemme know if my above assumption is true. If yes, I'll just
make the change as<br>
part of this push.<br>
Ed<br>
<br>
<br>
_______________________________________________<br>
umon-devel mailing list<br>
<a moz-do-not-send="true" href="mailto:umon-devel@rtems.org"
target="_blank">umon-devel@rtems.org</a><br>
<a moz-do-not-send="true"
href="http://lists.rtems.org/mailman/listinfo/umon-devel"
rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/umon-devel</a><br>
</blockquote>
</div>
</blockquote>
<br>
</body>
</html>