Am 23.05.2013 15:35, schrieb Sascha Hauer:
Hi Oleksij, On Wed, May 22, 2013 at 09:49:47AM +0200, Oleksij Rempel wrote:Signed-off-by: Oleksij Rempel <linux@xxxxxxxxxxxxxxxx> Signed-off-by: Antony Pavlov <antonynpavlov@xxxxxxxxx> + +/* + * This table is indexed by bits 5..4 of the CLOCKCTL1 register + * to determine the predevisor value. + */ +static int CLOCKCTL1_PREDIVIDE_TABLE[4] = { 1, 2, 4, 5 }; + +static unsigned int +ar2312_cpu_frequency(void) +{ + unsigned int predivide_mask, predivide_shift; + unsigned int multiplier_mask, multiplier_shift; + unsigned int clockCtl1, preDivideSelect, preDivisor, multiplier;No CamelCase please.
ok. i'll fix it.
+static int platform_init(void) +{ + add_generic_device("ar2312_reset", DEVICE_ID_SINGLE, NULL, + KSEG1ADDR(AR2312_RESETTMR), 0x4, + IORESOURCE_MEM, NULL); + watchdog_init(); + flash_init(); + ether_init();Whether or not the flash/ethernet/serial device is available is board specific, not SoC specific, so this should be done in board code. It's good to provide helpers to make this simple for the board code though, so a good option would be to add a SoC prefix to the functions and call it from board code.
Probably it will make sense to have generic type of board. Because this will work with most configurations. Since it is WiSoC (Wireless SoC), ethernet is on chip too.
Otherwise I am fine with this patch, but maybe Antony still has some comments.
-- Regards, Oleksij _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
