[PATCH 2/2] board: atmel: Add SAMA5D27 giant board

Greg Gallagher greg at embeddedgreg.com
Fri Jan 15 15:29:08 CET 2021


On Fri, Jan 15, 2021 at 2:55 AM <Eugen.Hristev at microchip.com> wrote:

> On 11.01.2021 09:10, Greg Gallagher wrote:
> > Giant board is a tiny SBC based on the Adafruit Feather form factor,
> > created by groboards it contains a SAMA5D2 processor (SAMA5D27),
> > 128 MB of RAM and a microSD card for storage.
> >
> > Signed-off-by: Greg Gallagher <greg at embeddedgreg.com>
>
> Hi Greg,
>
>
> > ---
> >
> >   arch/arm/dts/Makefile                         |   3 +
> >   arch/arm/dts/at91-sama5d27_giantboard.dts     | 128 ++++++++++++
> >   arch/arm/mach-at91/Kconfig                    |   9 +
> >   board/atmel/sama5d27_giantboard/Kconfig       |  15 ++
> >   board/atmel/sama5d27_giantboard/MAINTAINERS   |   6 +
> >   board/atmel/sama5d27_giantboard/Makefile      |   5 +
> >   .../sama5d27_giantboard/sama5d27_giantboard.c | 182 ++++++++++++++++++
> >   configs/sama5d27_giantboard_defconfig         |  93 +++++++++
> >   include/configs/sama5d27_giantboard.h         | 136 +++++++++++++
> >   9 files changed, 577 insertions(+)
> >   create mode 100644 arch/arm/dts/at91-sama5d27_giantboard.dts
> >   create mode 100644 board/atmel/sama5d27_giantboard/Kconfig
> >   create mode 100644 board/atmel/sama5d27_giantboard/MAINTAINERS
> >   create mode 100644 board/atmel/sama5d27_giantboard/Makefile
> >   create mode 100644
> board/atmel/sama5d27_giantboard/sama5d27_giantboard.c
> >   create mode 100644 configs/sama5d27_giantboard_defconfig
> >   create mode 100644 include/configs/sama5d27_giantboard.h
> >
> > diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
> > index fd47e408f8..23b45290a4 100644
> > --- a/arch/arm/dts/Makefile
> > +++ b/arch/arm/dts/Makefile
> > @@ -898,6 +898,9 @@ dtb-$(CONFIG_TARGET_SAMA5D2_XPLAINED) += \
> >   dtb-$(CONFIG_TARGET_SAMA5D27_SOM1_EK) += \
> >          at91-sama5d27_som1_ek.dtb
> >
> > +dtb-$(CONFIG_TARGET_SAMA5D27_GIANTBOARD) += \
> > +       at91-sama5d27_giantboard.dtb
> > +
> >   dtb-$(CONFIG_TARGET_SAMA5D27_WLSOM1_EK) += \
> >          at91-sama5d27_wlsom1_ek.dtb
> >
> > diff --git a/arch/arm/dts/at91-sama5d27_giantboard.dts
> b/arch/arm/dts/at91-sama5d27_giantboard.dts
> > new file mode 100644
> > index 0000000000..e81ca60ca0
> > --- /dev/null
> > +++ b/arch/arm/dts/at91-sama5d27_giantboard.dts
> > @@ -0,0 +1,128 @@
> > +// SPDX-License-Identifier: GPL-2.0+
> > +/*
> > + * at91-sama5d27_giantboard.dts - Device Tree file for Giant Board
> > + *
> > + * Copyright (C) 2020 Greg Gallagher <greg at embeddedgreg.com>
> > + *
> > + * Derived from at91-sama5d27_som1_ek.dts
> > + *
> > + * Copyright (C) 2017 Microchip Corporation
> > + *                   Wenyou Yang <wenyou.yang at microchip.com>
> > + */
> > +/dts-v1/;
> > +#include "sama5d2.dtsi"
> > +#include "sama5d2-pinfunc.h"
> > +
> > +/ {
> > +       model = "Giant Board";
> > +       compatible = "atmel,sama5d27-giantboard", "atmel,sama5d2",
> "atmel,sama5";
> > +
> > +       memory {
> > +               reg = <0x20000000 0x8000000>;
> > +       };
> > +
> > +       chosen {
> > +               u-boot,dm-pre-reloc;
> > +               stdout-path = &uart1;
> > +       };
> > +
> > +       ahb {
> > +               sdmmc1: sdio-host at b0000000 {
> > +                       bus-width = <4>;
> > +                       pinctrl-names = "default";
> > +                       pinctrl-0 = <&pinctrl_sdmmc1_cmd_dat_default
> &pinctrl_sdmmc1_ck_cd_default>;
> > +                       status = "okay";
> > +                       u-boot,dm-pre-reloc;
> > +               };
> > +
> > +               apb {
> > +
> Please remove empty line here
>
Ack


> > +                       uart1: serial at f8020000 {
> > +                               pinctrl-names = "default";
> > +                               pinctrl-0 = <&pinctrl_uart1_default>;
> > +                               status = "okay";
> > +                               u-boot,dm-pre-reloc;
> > +                       };
> > +
> > +                       i2c0: i2c at f8028000 {
> > +                               pinctrl-names = "default";
> > +                               pinctrl-0 = <&pinctrl_i2c0_default>;
> > +                               status = "okay";
> > +                       };
> > +
> > +                       i2c1: i2c at fc028000 {
> > +                               pinctrl-names = "default";
> > +                               pinctrl-0 = <&pinctrl_i2c1_default>;
> > +                               status = "okay";
> > +
> > +                               pmic at 5b {
> > +                                       compatible =
> "active-semi,act8945a";
> > +                                       reg = <0x5b>;
> > +                                       active-semi,vsel-low;
> > +                                       status = "okay";
> > +                               };
> > +                       };
> > +
> > +                       pit: timer at f8048030 {
> > +                               status = "okay";
> > +                               u-boot,dm-pre-reloc;
> > +                       };
> > +
> > +                       sfr: sfr at f8030000 {
> > +                               status = "okay";
> > +                               u-boot,dm-pre-reloc;
> > +                       };
> > +
> > +                       pioA: gpio at fc038000 {
> > +                               pinctrl {
> > +
> Please remove empty line here
>
Ack


> > +                                       pinctrl_sdmmc1_cmd_dat_default:
> sdmmc1_cmd_dat_default {
> > +                                               pinmux =
> <PIN_PA28__SDMMC1_CMD>,
> > +
> <PIN_PA18__SDMMC1_DAT0>,
> > +
> <PIN_PA19__SDMMC1_DAT1>,
> > +
> <PIN_PA20__SDMMC1_DAT2>,
> > +
> <PIN_PA21__SDMMC1_DAT3>;
> > +                                               bias-pull-up;
> > +                                               u-boot,dm-pre-reloc;
> > +                                       };
> > +
> > +                                       pinctrl_sdmmc1_ck_cd_default:
> sdmmc1_ck_cd_default {
> > +                                               pinmux =
> <PIN_PA22__SDMMC1_CK>,
> > +
> <PIN_PA30__SDMMC1_CD>;
> > +                                               bias-disable;
> > +                                               u-boot,dm-pre-reloc;
> > +                                       };
> > +
> > +                                       pinctrl_uart1_default:
> uart1_default {
> > +                                               pinmux =
> <PIN_PD2__URXD1>,
> > +
> <PIN_PD3__UTXD1>;
> > +                                               bias-disable;
> > +                                               u-boot,dm-pre-reloc;
> > +                                       };
> > +
> > +                                       pinctrl_i2c0_default:
> i2c0_default {
> > +                                               pinmux =
> <PIN_PD21__TWD0>,
> > +
> <PIN_PD22__TWCK0>;
> > +                                               bias-disable;
> > +                                       };
> > +
> > +                                       pinctrl_i2c1_default:
> i2c1_default {
> > +                                               pinmux = <PIN_PD4__TWD1>,
> > +
> <PIN_PD5__TWCK1>;
> > +                                               bias-disable;
> > +                                       };
> > +
> > +                                       pinctrl_usb_default: usb_default
> {
> > +                                               pinmux =
> <PIN_PB10__GPIO>;
> > +                                               bias-disable;
> > +                                       };
> > +
> > +                                       pinctrl_usba_vbus: usba_vbus {
> > +                                               pinmux =
> <PIN_PA31__GPIO>;
> > +                                               bias-disable;
> > +                                       };
> > +                               };
> > +                       };
> > +               };
> > +       };
> > +};
> > diff --git a/arch/arm/mach-at91/Kconfig b/arch/arm/mach-at91/Kconfig
> > index be1415f909..1837396945 100644
> > --- a/arch/arm/mach-at91/Kconfig
> > +++ b/arch/arm/mach-at91/Kconfig
> > @@ -183,6 +183,14 @@ config TARGET_SAMA5D2_XPLAINED
> >          select SAMA5D2
> >          select SUPPORT_SPL
> >
> > +config TARGET_SAMA5D27_GIANTBOARD
> > +       bool "SAMA5D27 Giant board"
> > +       select BOARD_EARLY_INIT_F
> > +       select BOARD_LATE_INIT
> > +       select CPU_V7A
> > +       select SUPPORT_SPL
> > +       select ATMEL_SFR
> > +
> >   config TARGET_SAMA5D27_SOM1_EK
> >          bool "SAMA5D27 SOM1 EK board"
> >          select BOARD_EARLY_INIT_F
> > @@ -330,6 +338,7 @@ source "board/atmel/sam9x60ek/Kconfig"
> >   source "board/atmel/sama5d2_ptc_ek/Kconfig"
> >   source "board/atmel/sama5d2_xplained/Kconfig"
> >   source "board/atmel/sama5d27_som1_ek/Kconfig"
> > +source "board/atmel/sama5d27_giantboard/Kconfig"
> >   source "board/atmel/sama5d27_wlsom1_ek/Kconfig"
> >   source "board/atmel/sama5d2_icp/Kconfig"
> >   source "board/atmel/sama5d3_xplained/Kconfig"
> > diff --git a/board/atmel/sama5d27_giantboard/Kconfig
> b/board/atmel/sama5d27_giantboard/Kconfig
> > new file mode 100644
> > index 0000000000..6cefb8f689
> > --- /dev/null
> > +++ b/board/atmel/sama5d27_giantboard/Kconfig
> > @@ -0,0 +1,15 @@
> > +if TARGET_SAMA5D27_GIANTBOARD
> > +
> > +config SYS_BOARD
> > +       default "sama5d27_giantboard"
> > +
> > +config SYS_VENDOR
> > +       default "atmel"
> > +
> > +config SYS_SOC
> > +       default "at91"
> > +
> > +config SYS_CONFIG_NAME
> > +       default "sama5d27_giantboard"
> > +
> > +endif
> > diff --git a/board/atmel/sama5d27_giantboard/MAINTAINERS
> b/board/atmel/sama5d27_giantboard/MAINTAINERS
> > new file mode 100644
> > index 0000000000..80bfae01fa
> > --- /dev/null
> > +++ b/board/atmel/sama5d27_giantboard/MAINTAINERS
> > @@ -0,0 +1,6 @@
> > +SAMA5D27 GIANT BOARD
> > +M:     Greg Gallagher <greg at embeddedgreg.com>
> > +S:     Maintained
> > +F:     board/atmel/sama5d27_giantboard/
> > +F:     include/configs/sama5d27_giantboard.h
> > +F:     configs/sama5d27_giantboard_mmc_defconfig
>
> This file does not match the files you are adding in the commit
> (configs/sama5d27_giantboard_defconfig )
>

Will fix in V2

>
> > diff --git a/board/atmel/sama5d27_giantboard/Makefile
> b/board/atmel/sama5d27_giantboard/Makefile
> > new file mode 100644
> > index 0000000000..d4b2a4d36f
> > --- /dev/null
> > +++ b/board/atmel/sama5d27_giantboard/Makefile
> > @@ -0,0 +1,5 @@
> > +# SPDX-License-Identifier: GPL-2.0+
> > +#
> > +# Copyright (C) 2020 Greg Gallagher <greg at embeddedgreg.com>
> > +
> > +obj-y += sama5d27_giantboard.o
> > diff --git a/board/atmel/sama5d27_giantboard/sama5d27_giantboard.c
> b/board/atmel/sama5d27_giantboard/sama5d27_giantboard.c
> > new file mode 100644
> > index 0000000000..d75a9650ad
> > --- /dev/null
> > +++ b/board/atmel/sama5d27_giantboard/sama5d27_giantboard.c
> > @@ -0,0 +1,182 @@
> > +// SPDX-License-Identifier: GPL-2.0+
> > +/*
> > + * Copyright (C) 2021 Greg Gallagher <greg at embeddedgreg.com>
> > + *
> > + * Derived from sama5d27_som1_ek.c
> > + *
> > + * Copyright (C) 2017 Microchip Corporation
> > + *                   Wenyou Yang <wenyou.yang at microchip.com>
> > + *
> > + */
> > +
> > +#include <common.h>
> > +#include <debug_uart.h>
> > +#include <init.h>
> > +#include <asm/io.h>
> > +#include <asm/arch/at91_common.h>
> > +#include <asm/arch/atmel_pio4.h>
> > +#include <asm/arch/atmel_mpddrc.h>
> > +#include <asm/arch/atmel_sdhci.h>
> > +#include <asm/arch/clk.h>
> > +#include <asm/arch/gpio.h>
> > +#include <asm/arch/sama5d2.h>
> > +
> > +extern void at91_pda_detect(void);
> > +
> > +DECLARE_GLOBAL_DATA_PTR;
> > +
> > +#ifdef CONFIG_CMD_USB
> > +static void board_usb_hw_init(void)
> > +{
> > +       atmel_pio4_set_pio_output(AT91_PIO_PORTA, 27, 1);
> > +}
> > +#endif
> > +
> > +#ifdef CONFIG_BOARD_LATE_INIT
> > +int board_late_init(void)
> > +{
> > +       at91_pda_detect();
> > +       return 0;
> > +}
> > +#endif
> > +
> > +#ifdef CONFIG_DEBUG_UART_BOARD_INIT
> > +static void board_uart1_hw_init(void)
> > +{
> > +       atmel_pio4_set_a_periph(AT91_PIO_PORTD, 2,
> ATMEL_PIO_PUEN_MASK);        /* URXD1 */
> > +       atmel_pio4_set_a_periph(AT91_PIO_PORTD, 3, 0);  /* UTXD1 */
> > +
> > +       at91_periph_clk_enable(ATMEL_ID_UART1);
> > +}
> > +
> > +void board_debug_uart_init(void)
> > +{
> > +       board_uart1_hw_init();
> > +}
> > +#endif
> > +
> > +#ifdef CONFIG_BOARD_EARLY_INIT_F
> > +int board_early_init_f(void)
> > +{
> > +       if (IS_ENABLED(CONFIG_DEBUG_UART))
> > +               debug_uart_init();
> > +
> > +       return 0;
> > +}
> > +#endif
> > +
> > +int board_init(void)
> > +{
> > +       /* address of boot parameters */
> > +       gd->bd->bi_boot_params = CONFIG_SYS_SDRAM_BASE + 0x100;
> > +
> > +       if (IS_ENABLED(CONFIG_CMD_USB))
> > +               board_usb_hw_init();
> > +
> > +       return 0;
> > +}
> > +
> > +int dram_init(void)
> > +{
> > +       gd->ram_size = get_ram_size((void *)CONFIG_SYS_SDRAM_BASE,
> > +                                   CONFIG_SYS_SDRAM_SIZE);
> > +       return 0;
> > +}
> > +
> > +/* SPL */
> > +#ifdef CONFIG_SPL_BUILD
> > +void spl_board_init(void)
> > +{
> > +}
> > +
> > +static void ddrc_conf(struct atmel_mpddrc_config *ddrc)
> > +{
> > +       ddrc->md = (ATMEL_MPDDRC_MD_DBW_16_BITS |
> ATMEL_MPDDRC_MD_DDR2_SDRAM);
> > +
> > +       ddrc->cr = (ATMEL_MPDDRC_CR_NC_COL_10 |
> > +                   ATMEL_MPDDRC_CR_NR_ROW_13 |
> > +                   ATMEL_MPDDRC_CR_CAS_DDR_CAS3 |
> > +                   ATMEL_MPDDRC_CR_DIC_DS |
> > +                   ATMEL_MPDDRC_CR_ZQ_LONG |
> > +                   ATMEL_MPDDRC_CR_NB_8BANKS |
> > +                   ATMEL_MPDDRC_CR_DECOD_INTERLEAVED |
> > +                   ATMEL_MPDDRC_CR_UNAL_SUPPORTED);
> > +
> > +       ddrc->rtr = 0x511;
> > +
> > +       ddrc->tpr0 = ((7 << ATMEL_MPDDRC_TPR0_TRAS_OFFSET) |
> > +                     (3 << ATMEL_MPDDRC_TPR0_TRCD_OFFSET) |
> > +                     (3 << ATMEL_MPDDRC_TPR0_TWR_OFFSET) |
> > +                     (9 << ATMEL_MPDDRC_TPR0_TRC_OFFSET) |
> > +                     (3 << ATMEL_MPDDRC_TPR0_TRP_OFFSET) |
> > +                     (4 << ATMEL_MPDDRC_TPR0_TRRD_OFFSET) |
> > +                     (4 << ATMEL_MPDDRC_TPR0_TWTR_OFFSET) |
> > +                     (2 << ATMEL_MPDDRC_TPR0_TMRD_OFFSET));
> > +
> > +       ddrc->tpr1 = ((22 << ATMEL_MPDDRC_TPR1_TRFC_OFFSET) |
> > +                     (23 << ATMEL_MPDDRC_TPR1_TXSNR_OFFSET) |
> > +                     (200 << ATMEL_MPDDRC_TPR1_TXSRD_OFFSET) |
> > +                     (3 << ATMEL_MPDDRC_TPR1_TXP_OFFSET));
> > +
> > +       ddrc->tpr2 = ((2 << ATMEL_MPDDRC_TPR2_TXARD_OFFSET) |
> > +                     (8 << ATMEL_MPDDRC_TPR2_TXARDS_OFFSET) |
> > +                     (4 << ATMEL_MPDDRC_TPR2_TRPA_OFFSET) |
> > +                     (4 << ATMEL_MPDDRC_TPR2_TRTP_OFFSET) |
> > +                     (8 << ATMEL_MPDDRC_TPR2_TFAW_OFFSET));
> > +}
> > +
> > +void mem_init(void)
> > +{
> > +       struct at91_pmc *pmc = (struct at91_pmc *)ATMEL_BASE_PMC;
> > +       struct atmel_mpddr *mpddrc = (struct atmel_mpddr
> *)ATMEL_BASE_MPDDRC;
> > +       struct atmel_mpddrc_config ddrc_config;
> > +       u32 reg;
> > +
> > +       ddrc_conf(&ddrc_config);
> > +
> > +       at91_periph_clk_enable(ATMEL_ID_MPDDRC);
> > +       writel(AT91_PMC_DDR, &pmc->scer);
> > +
> > +       reg = readl(&mpddrc->io_calibr);
> > +       reg &= ~ATMEL_MPDDRC_IO_CALIBR_RDIV;
> > +       reg |= ATMEL_MPDDRC_IO_CALIBR_DDR3_RZQ_55;
> > +       reg &= ~ATMEL_MPDDRC_IO_CALIBR_TZQIO;
> > +       reg |= ATMEL_MPDDRC_IO_CALIBR_TZQIO_(101);
> > +       writel(reg, &mpddrc->io_calibr);
> > +
> > +       writel(ATMEL_MPDDRC_RD_DATA_PATH_SHIFT_ONE_CYCLE,
> > +              &mpddrc->rd_data_path);
> > +
> > +       ddr3_init(ATMEL_BASE_MPDDRC, ATMEL_BASE_DDRCS, &ddrc_config);
> > +
> > +       writel(0x3, &mpddrc->cal_mr4);
> > +       writel(64, &mpddrc->tim_cal);
> > +}
> > +
> > +void at91_pmc_init(void)
> > +{
> > +       u32 tmp;
> > +
> > +       /*
> > +        * while coming from the ROM code, we run on PLLA @ 492 MHz /
> 164 MHz
> > +        * so we need to slow down and configure MCKR accordingly.
> > +        * This is why we have a special flavor of the switching
> function.
> > +        */
> > +       tmp = AT91_PMC_MCKR_PLLADIV_2 |
> > +             AT91_PMC_MCKR_MDIV_3 |
> > +             AT91_PMC_MCKR_CSS_MAIN;
> > +       at91_mck_init_down(tmp);
> > +
> > +       tmp = AT91_PMC_PLLAR_29 |
> > +             AT91_PMC_PLLXR_PLLCOUNT(0x3f) |
> > +             AT91_PMC_PLLXR_MUL(40) |
> > +             AT91_PMC_PLLXR_DIV(1);
> > +       at91_plla_init(tmp);
> > +
> > +       tmp = AT91_PMC_MCKR_H32MXDIV |
> > +             AT91_PMC_MCKR_PLLADIV_2 |
> > +             AT91_PMC_MCKR_MDIV_3 |
> > +             AT91_PMC_MCKR_CSS_PLLA;
> > +       at91_mck_init(tmp);
> > +}
> > +#endif
> > diff --git a/configs/sama5d27_giantboard_defconfig
> b/configs/sama5d27_giantboard_defconfig
> > new file mode 100644
> > index 0000000000..4612a75b53
> > --- /dev/null
> > +++ b/configs/sama5d27_giantboard_defconfig
> > @@ -0,0 +1,93 @@
> > +CONFIG_ARM=y
> > +CONFIG_ARCH_CPU_INIT=y
> > +CONFIG_ARCH_AT91=y
> > +CONFIG_CMDLINE=y
> > +CONFIG_SYS_TEXT_BASE=0x23f00000
> > +CONFIG_TARGET_SAMA5D27_GIANTBOARD=y
> > +CONFIG_SPL_GPIO_SUPPORT=y
> > +CONFIG_SPL_LIBCOMMON_SUPPORT=y
> > +CONFIG_SPL_LIBGENERIC_SUPPORT=y
> > +CONFIG_SYS_MALLOC_F_LEN=0x2000
> > +CONFIG_SPL_MMC_SUPPORT=y
> > +CONFIG_SPL_SERIAL_SUPPORT=y
> > +CONFIG_SPL_DRIVERS_MISC_SUPPORT=y
> > +CONFIG_ENV_SIZE=0x4000
> > +CONFIG_NR_DRAM_BANKS=1
> > +CONFIG_SPL=y
> > +CONFIG_DEBUG_UART_BOARD_INIT=y
> > +CONFIG_DEBUG_UART_BASE=0xf8020000
> > +CONFIG_DEBUG_UART_CLOCK=82000000
> > +CONFIG_SPL_FAT_SUPPORT=y
> > +CONFIG_SPL_FS_FAT=y
> > +CONFIG_SPL_LIBDISK_SUPPORT=y
> > +CONFIG_DEBUG_UART=y
> > +CONFIG_ENV_VARS_UBOOT_CONFIG=y
> > +CONFIG_FIT=y
> > +CONFIG_SYS_EXTRA_OPTIONS="SAMA5D2"
> > +CONFIG_SD_BOOT=y
> > +CONFIG_BOOTDELAY=3
> > +CONFIG_USE_BOOTARGS=y
> > +# CONFIG_MISC_INIT_R is not set
> > +# CONFIG_DISPLAY_BOARDINFO is not set
> > +CONFIG_SPL_TEXT_BASE=0x200000
> > +CONFIG_SPL_SEPARATE_BSS=y
> > +CONFIG_HUSH_PARSER=y
> > +CONFIG_AUTOBOOT_KEYED=y
> > +CONFIG_AUTOBOOT_PROMPT="Hit SPACE in %d seconds to stop autoboot.\n"
> > +CONFIG_AUTOBOOT_STOP_STR=" "
> > +CONFIG_AUTOBOOT_KEYED_CTRLC=y
> > +CONFIG_CMD_BOOTZ=y
> > +# CONFIG_CMD_FLASH is not set
> > +CONFIG_CMD_I2C=y
> > +# CONFIG_CMD_LOADS is not set
> > +CONFIG_CMD_MMC=y
> > +CONFIG_CMD_PART=y
> > +CONFIG_CMD_SF=y
> > +CONFIG_CMD_USB=y
> > +CONFIG_CMD_EXT4=y
> > +CONFIG_CMD_FAT=y
> > +CONFIG_CMD_FS_GENERIC=y
> > +CONFIG_OF_CONTROL=y
> > +CONFIG_SPL_OF_CONTROL=y
> > +CONFIG_OF_LIBFDT=y
> > +CONFIG_OF_LIBFDT_OVERLAY=y
> > +CONFIG_DEFAULT_DEVICE_TREE="at91-sama5d27_giantboard"
> > +CONFIG_OF_SPL_REMOVE_PROPS="interrupts interrupt-parent dmas dma-names"
> > +CONFIG_ENV_IS_IN_FAT=y
> > +CONFIG_DM=y
> > +CONFIG_SPL_DM=y
> > +CONFIG_SPL_DM_SEQ_ALIAS=y
> > +CONFIG_CLK=y
> > +CONFIG_SPL_CLK=y
> > +CONFIG_CLK_AT91=y
> > +CONFIG_AT91_UTMI=y
> > +CONFIG_AT91_H32MX=y
> > +CONFIG_AT91_GENERIC_CLK=y
> > +CONFIG_DM_GPIO=y
> > +CONFIG_ATMEL_PIO4=y
> > +CONFIG_DM_I2C=y
> > +CONFIG_SYS_I2C_AT91=y
> > +CONFIG_DM_MMC=y
> > +CONFIG_MMC_SDHCI=y
> > +CONFIG_MMC_SDHCI_ATMEL=y
> > +CONFIG_DM_SPI_FLASH=y
> > +CONFIG_SPI_FLASH=y
> > +# CONFIG_NET is not set
> > +CONFIG_PINCTRL=y
> > +CONFIG_SPL_PINCTRL=y
> > +CONFIG_PINCTRL_AT91PIO4=y
> > +CONFIG_DM_SERIAL=y
> > +CONFIG_DEBUG_UART_ATMEL=y
> > +CONFIG_DEBUG_UART_ANNOUNCE=y
> > +CONFIG_ATMEL_USART=y
> > +CONFIG_SPI=y
> > +CONFIG_DM_SPI=y
> > +CONFIG_TIMER=y
> > +CONFIG_SPL_TIMER=y
> > +CONFIG_ATMEL_PIT_TIMER=y
> > +CONFIG_USB=y
> > +CONFIG_DM_USB=y
> > +CONFIG_USB_EHCI_HCD=y
> > +CONFIG_USB_STORAGE=y
> > +CONFIG_USB_GADGET=y
> > +CONFIG_USB_GADGET_ATMEL_USBA=y
> > diff --git a/include/configs/sama5d27_giantboard.h
> b/include/configs/sama5d27_giantboard.h
> > new file mode 100644
> > index 0000000000..2311061e1f
> > --- /dev/null
> > +++ b/include/configs/sama5d27_giantboard.h
> > @@ -0,0 +1,136 @@
> > +/* SPDX-License-Identifier: GPL-2.0+ */
> > +/*
> > + * Configuration file for the SAMA5D27 Giant Board.
> > + *
> > + * Copyright (C) 2020 Greg Gallagher <greg at embeddedgreg.com>
> > + *
> > + * Derived from sama5d27_som1_ek.h
> > + *
> > + * Copyright (C) 2017 Microchip Corporation
> > + *                   Wenyou Yang <wenyou.yang at microchip.com>
> > + */
> > +
> > +#ifndef __CONFIG_H
> > +#define __CONFIG_H
> > +
> > +#include "at91-sama5_common.h"
> > +
> > +#undef CONFIG_SYS_AT91_MAIN_CLOCK
> > +#define CONFIG_SYS_AT91_MAIN_CLOCK      24000000 /* from 24 MHz crystal
> */
> > +
> > +/* SDRAM */
> > +#define CONFIG_SYS_SDRAM_BASE          0x20000000
> > +#define CONFIG_SYS_SDRAM_SIZE          0x8000000
> > +
> > +#ifdef CONFIG_SPL_BUILD
> > +#define CONFIG_SYS_INIT_SP_ADDR                0x218000
> > +#else
> > +#define CONFIG_SYS_INIT_SP_ADDR \
> > +       (CONFIG_SYS_SDRAM_BASE + 16 * 1024 - GENERATED_GBL_DATA_SIZE)
> > +#endif
> > +
> > +#define CONFIG_SYS_LOAD_ADDR           0x22000000 /* load address */
> > +
> > +/* NAND flash */
> > +
> > +/* SPI flash */
> > +
> > +#undef CONFIG_BOOTCOMMAND
> > +#ifdef CONFIG_SD_BOOT
> > +/* bootstrap + u-boot + env in sd card */
> > +#define CONFIG_SUPPORT_RAW_INITRD
> > +#define CONFIG_EXTRA_ENV_SETTINGS \
> > +       "fdtovaddr=0x21800000\0" \
> > +       "loadaddr=0x22000000\0" \
> > +       "fdtaddr=0x21000000\0" \
> > +       "fdtfile=/dtbs/at91-sama5d27_giantboard.dtb\0" \
> > +       "enable_uboot_overlays=\0" \
> > +       "dtboverlay=\0" \
> > +       "dtboverlay1=\0" \
> > +       "dtboverlay2=\0" \
> > +       "dtboverlay3=\0" \
> > +       "dtboverlay4=\0" \
> > +       "console=ttyS0,115200\0" \
> > +       "optargs=\0" \
> > +       "mmcdev=0\0" \
> > +       "mmcpart=1\0" \
> > +       "mmcroot=/dev/mmcblk0p2 rw\0" \
> > +       "mmcrootfstype=ext4 rootwait\0" \
> > +       "mmcargs=setenv bootargs console=${console} " \
> > +               "${optargs} " \
> > +               "root=${mmcroot} " \
> > +               "rootfstype=${mmcrootfstype} " \
> > +       "loadbootenv=load mmc ${mmcdev}:${mmcpart} ${loadaddr}
> uEnv.txt\0" \
> > +       "importbootenv=echo Importing environment from mmc
> (uEnv.txt)...; " \
> > +               "env import -t ${loadaddr} ${filesize}\0" \
> > +       "loadzimage=load mmc ${mmcdev}:${mmcpart} ${loadaddr} zImage\0" \
> > +       "loadfdt=load mmc ${mmcdev}:${mmcpart} ${fdtaddr} ${fdtfile}\0" \
> > +       "loadoverlay=load mmc ${mmcdev}:${mmcpart} ${fdtovaddr}
> ${dtboverlay}\0" \
> > +       "loadoverlay1=load mmc ${mmcdev}:${mmcpart} ${fdtovaddr}
> ${dtboverlay1}\0" \
> > +       "loadoverlay2=load mmc ${mmcdev}:${mmcpart} ${fdtovaddr}
> ${dtboverlay2}\0" \
> > +       "loadoverlay3=load mmc ${mmcdev}:${mmcpart} ${fdtovaddr}
> ${dtboverlay3}\0" \
> > +       "loadoverlay4=load mmc ${mmcdev}:${mmcpart} ${fdtovaddr}
> ${dtboverlay4}\0" \
> > +       "mmcboot=echo Booting from mmc ...; " \
> > +       "run mmcargs; " \
> > +       "bootz ${loadaddr} - ${fdtaddr}\0"
> > +
> > +#define CONFIG_BOOTCOMMAND \
> > +       "mmc dev ${mmcdev};" \
> > +       "if mmc rescan; then " \
> > +               "echo SD/MMC found on device ${mmcdev};" \
> > +               "if run loadbootenv; then " \
> > +                       "run importbootenv;" \
> > +               "fi;" \
> > +               "echo Running default loadzimage ...;" \
> > +               "if run loadzimage; then " \
> > +                       "run loadfdt;" \
> > +                       "echo Checking for overlays ...;" \
> > +                       "if test -n $enable_uboot_overlays; then " \
> > +                               "fdt addr ${fdtaddr};" \
> > +                               "fdt resize 4096;" \
> > +                               "if test -n $dtboverlay; then " \
> > +                                       "run loadoverlay;" \
> > +                                       "fdt apply ${fdtovaddr};" \
> > +                                       "echo loaded ${dtboverlay};" \
> > +                               "fi;" \
> > +                               "if test -n $dtboverlay1; then " \
> > +                                       "run loadoverlay1;" \
> > +                                       "fdt apply ${fdtovaddr};" \
> > +                                       "echo loaded ${dtboverlay1};" \
> > +                               "fi;" \
> > +                               "if test -n $dtboverlay2; then " \
> > +                                       "run loadoverlay2;" \
> > +                                       "fdt apply ${fdtovaddr};" \
> > +                                       "echo loaded ${dtboverlay2};" \
> > +                               "fi;" \
> > +                               "if test -n $dtboverlay3; then " \
> > +                                       "run loadoverlay3;" \
> > +                                       "fdt apply ${fdtovaddr};" \
> > +                                       "echo loaded ${dtboverlay3};" \
> > +                               "fi;" \
> > +                               "if test -n $dtboverlay4; then " \
> > +                                       "run loadoverlay4;" \
> > +                                       "fdt apply ${fdtovaddr};" \
> > +                                       "echo loaded ${dtboverlay4};" \
> > +                               "fi;" \
> > +                       "fi;" \
> > +                       "run mmcboot;" \
> > +               "fi;" \
> > +       "fi;"
> > +#endif
> > +
> > +/* SPL */
> > +#define CONFIG_SPL_MAX_SIZE            0x10000
> > +#define CONFIG_SPL_BSS_START_ADDR      0x20000000
> > +#define CONFIG_SPL_BSS_MAX_SIZE                0x80000
> > +#define CONFIG_SYS_SPL_MALLOC_START    0x20080000
> > +#define CONFIG_SYS_SPL_MALLOC_SIZE     0x80000
> > +
> > +#define CONFIG_SYS_MONITOR_LEN         (512 << 10)
> > +
> > +#ifdef CONFIG_SD_BOOT
> > +#define CONFIG_SYS_MMCSD_FS_BOOT_PARTITION     1
> > +#define CONFIG_SPL_FS_LOAD_PAYLOAD_NAME                "u-boot.img"
> > +#endif
> > +
> > +#endif
> > --
> > 2.25.1
> >
>
> Have you tried reusing the som1_ek file ? I do not see much difference
> between your sama5d2_giantboard.c file and the som1_ek file.
> You removed the eeprom ethernet mac retreival and the call to dm_video
> but otherwise everything looks the same.
>
I thought there would be more changes so I started with a new file, but as
you pointed out they are pretty similar, I'll try using the som1_ek file.

>
> Eugen
>

Thanks for reviewing, I'll get V2 ready.

Thanks

Greg


More information about the U-Boot mailing list