On 06/01/2015 08:44 PM, Paul Walmsley wrote:
On Mon, 1 Jun 2015, Tony Lindgren wrote:
* Jeroen Hofstee <linux-...@myspectrum.nl> [150601 09:58]:
On 01-06-15 17:30, Tero Kristo wrote:
New system control module layout for omap3 overlooked parts of the am35xx
configuration. Basically the am35xx clocks were not converted to use the
changed offsets, which caused weird boot warnings. The errors were not
fatal so far, so they were not caught earlier. Fixed by applying the
proper offsets for the AM35xx scm clocks.
Fixes: b8845074cf ("ARM: dts: omap3: add minimal l4 bus layout with...")
Signed-off-by: Tero Kristo <t-kri...@ti.com>
Reported-by: Jeroen Hofstee <linux-...@myspectrum.nl>
Cc: Paul Walmsley <p...@pwsan.com>
Cc: Tony Lindgren <t...@atomide.com>
---
arch/arm/boot/dts/am35xx-clocks.dtsi | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
With this patch the error interrupt / stack dumps are no longer present.
Thanks,
Tested-by: Jeroen Hofstee <jer...@myspectrum.nl>
Thanks, I'm suprised this was not caught earlier with all the automated
boot testing going on?
At least speaking in terms the testbed results that I post, the warnings
get reported. But not many people seem to act on them. (Jeroen is a
pleasant exception.)
See for example the "Build warnings from toolchain", "Kernel warnings
during boot to userspace", "Kernel warnings during PM test", and "Obsolete
Kconfig symbols" sections here:
http://www.pwsan.com/omap/testlogs/test_v4.1-rc6/20150601012139/README.txt
The best way to make this work IMHO would be for us not to accept any new
feature addition patches as long as there are warnings reported in the
test results. The only real exception that I would foresee is if those
warnings are due to something outside of our control, e.g., a crappy
bootloader, as I suspect the USB_OTG initiator warnings are for the
CM-T3517.
I added some extra logic into my test setup today for detecting this.
Previously the dumps were pretty much hidden as there are existing dumps
so I kind of ignored the new ones semi-blindly. >.<
-Tero
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
