[PATCH] arm: Add basic support for new Marvell Armada SoC family
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Tue May 15 11:39:54 EDT 2012
Le Tue, 15 May 2012 16:36:03 +0100,
Ben Dooks <ben.dooks at codethink.co.uk> a écrit :
> I've tracked this down to the following:
> arch/arm/mach-armada/time.c
>
> 190 } else {
> 191 u32 clk;
> 192 of_property_read_u32(np, "clock-frequency",
> &clk); 193 WARN_ON(!clk);
>
> The problem is the 'clk' variable is un-initialised and therefore
> the fact my device-tree is missing a clock-frequency property is
> causing the system to use what was in 'clk' which was around 16000
> from the stack.
>
> I suggest that any property that is necessary to a unit operation
> should check that there was something to read. WARN_ON() is fine,
> but they you should not carry on and initialise the unit anyway.
>
> I have now gone and fixed my .dts file and the system now gets to
> the point to try and mount an initrd.
Thanks for your report and investigation, you're obviously correct.
We'll fix that.
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
More information about the linux-arm-kernel
mailing list