Clocks on Barebox

Sascha Hauer s.hauer at pengutronix.de
Tue Jun 9 21:59:44 PDT 2015


On Tue, Jun 09, 2015 at 01:28:17PM +0200, andreas.willig at rafi.de wrote:
> Hi Sascha,
> 
> here my oftables section from barebox.map:
> 
> .oftables       0x278f7bc0       0x18
>                 0x278f7bc0                . = ALIGN (0x8)
>                 0x278f7bc0                __clk_of_table_start = .
>  *(.__clk_of_table_*)
>  .__clk_of_table_end
>                 0x278f7bc0        0x8 drivers/built-in.o
>                 0x278f7bc0                __clk_of_table_sentinel
>  .__clk_of_table_fixed_clk
>                 0x278f7bc8        0x8 drivers/built-in.o
>                 0x278f7bc8                __clk_of_table_fixed_clk
>  .__clk_of_table_fixed_factor_clk
>                 0x278f7bd0        0x8 drivers/built-in.o
>                 0x278f7bd0                __clk_of_table_fixed_factor_clk
>                 0x278f7bd8                __clk_of_table_end = .

And here's the problem. The sentinel entry containing only NULL marks
the end of the array. It should be the last entry, but here it's the
first entry which effectively makes the table empty. It turned out it
was pure luck that the sentinel entry ended up as last entry normally.

I've just sent out a patch fixing this, please test.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |



More information about the barebox mailing list