Dear Piotr Wilczek, On 10 August 2012 19:12, Piotr Wilczek <p.wilc...@samsung.com> wrote: > Boards trats and universal are very similar resulting in some common code. > That common code is moved to exynos4_common/exynos4_common.c > > Signed-off-by: Piotr Wilczek <p.wilc...@samsung.com> > Signed-off-by: Kyungmin Park <kyungmin.p...@samsung.com> > CC: Minkyu Kang <mk7.k...@samsung.com> > --- > board/samsung/exynos4_common/exynos4_common.c | 139 > +++++++++++++++++++++++++ > board/samsung/trats/trats.c | 104 +------------------ > board/samsung/universal_c210/universal.c | 98 +----------------- > 3 files changed, 143 insertions(+), 198 deletions(-) > create mode 100644 board/samsung/exynos4_common/exynos4_common.c > > diff --git a/board/samsung/exynos4_common/exynos4_common.c > b/board/samsung/exynos4_common/exynos4_common.c > new file mode 100644 > index 0000000..f2413e5 > --- /dev/null > +++ b/board/samsung/exynos4_common/exynos4_common.c
I'll reject your patch. It's not a standard way, I think.. > +#ifdef CONFIG_REVISION_TAG > +u32 get_board_rev(void) > +{ > + return board_rev; > +} > +#endif > + > +int dram_init(void) > +{ > + gd->ram_size = get_ram_size((long *)PHYS_SDRAM_1, > PHYS_SDRAM_1_SIZE) > + + get_ram_size((long *)PHYS_SDRAM_2, > PHYS_SDRAM_2_SIZE); > +#if CONFIG_NR_DRAM_BANKS == 4 > + gd->ram_size += get_ram_size((long *)PHYS_SDRAM_3, > PHYS_SDRAM_3_SIZE) > + + get_ram_size((long *)PHYS_SDRAM_4, > PHYS_SDRAM_4_SIZE); > +#endif > + return 0; > +} > + > +void dram_init_banksize(void) > +{ > + gd->bd->bi_dram[0].start = PHYS_SDRAM_1; > + gd->bd->bi_dram[0].size = get_ram_size((long *)PHYS_SDRAM_1, \ > + PHYS_SDRAM_1_SIZE); > + gd->bd->bi_dram[1].start = PHYS_SDRAM_2; > + gd->bd->bi_dram[1].size = get_ram_size((long *)PHYS_SDRAM_2, \ > + PHYS_SDRAM_2_SIZE); > +#if CONFIG_NR_DRAM_BANKS == 4 > + gd->bd->bi_dram[2].start = PHYS_SDRAM_3; > + gd->bd->bi_dram[2].size = get_ram_size((long *)PHYS_SDRAM_3, \ > + PHYS_SDRAM_3_SIZE); > + gd->bd->bi_dram[3].start = PHYS_SDRAM_4; > + gd->bd->bi_dram[3].size = get_ram_size((long *)PHYS_SDRAM_4, \ > + PHYS_SDRAM_4_SIZE); > +#endif > +} also, this is not a common function. Although they are almost same, but I think it's a board specific. Please don't move it. > + > +int exynos4_mmc_init(struct exynos4_gpio_part2 *gpio2) How about using pinmux instead? Thanks. Minkyu Kang. -- from. prom. www.promsoft.net _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot