Hi Bo Shen, Thank you for your feedback.
> -----Original Message----- > From: Bo Shen [mailto:voice.s...@gmail.com] > Sent: 2015年10月28日 11:07 > To: Yang, Wenyou > Cc: U-Boot Mailing List; andreas.de...@googlemail.com > Subject: Re: [PATCH v4] arm: atmel: Add SAMA5D2 Xplained board > > Hi Wenyou, > > On 10/27/2015 17:59 PM, Wenyou Yang wrote: > > The board supports following features: > > - Boot media support: SD card/e.MMC/SPI flash, > > - Support LCD display (optional, disabled by default), > > - Support ethernet, > > - Support USB mass storage. > > > > Signed-off-by: Wenyou Yang <wenyou.y...@atmel.com> > > --- > > The patch is based on the following patches sent in mailing list. > > [PATCH] gpio: atmel: Add the PIO4 driver support > > [PATCH] arm: at91: Change the Chip ID registers' addresses > > [PATCH v3] mmc: atmel: Add atmel sdhci support > > [PATCH v2] arm: at91: clock: Add the generated clock support > > > > Changes in v4: > > 1./ remove __weak attribute for has_lcdc() added in v3. > > 2./ remove unused goto err_exit. > > > > Changes in v3: > > 1./ change defines-->definitions for more clearly in asm/arch/sama5d2.h. > > 2./ remove unused cpu_is_sama5d2x() macros. > > 3./ fix spelling error "adress". > > 4./ add __weak attribute for has_lcdc(). > > 5./ remove SPL configs. > > > > Changes in v2: > > 1./ re-order SAMA5D2 statements alphabetically. > > 2./ remove redundant "Unknown CPU type". > > 3./ rework sama5d2's macros. > > 4./ remove some #ifdef before functions. > > 5./ move CONFIG_CMD_SF to Kconfig. > > 6./ remove NAND macros from config file. > > 7./ CONFIG_BOOTCOMMAND for sf uses defines in at91-sama5_common.h. > > > > arch/arm/mach-at91/Kconfig | 5 + > > arch/arm/mach-at91/armv7/Makefile | 1 + > > arch/arm/mach-at91/armv7/sama5d2_devices.c | 59 +++++ > > arch/arm/mach-at91/include/mach/at91_pmc.h | 9 +- > > arch/arm/mach-at91/include/mach/atmel_usba_udc.h | 3 +- > > arch/arm/mach-at91/include/mach/hardware.h | 2 + > > arch/arm/mach-at91/include/mach/sama5d2.h | 203 ++++++++++++++++ > > board/atmel/sama5d2_xplained/Kconfig | 15 ++ > > board/atmel/sama5d2_xplained/MAINTAINERS | 7 + > > board/atmel/sama5d2_xplained/Makefile | 8 + > > board/atmel/sama5d2_xplained/sama5d2_xplained.c | 282 > ++++++++++++++++++++++ > > configs/sama5d2_xplained_mmc_defconfig | 11 + > > configs/sama5d2_xplained_spiflash_defconfig | 11 + > > include/configs/sama5d2_xplained.h | 122 ++++++++++ > > 14 files changed, 732 insertions(+), 6 deletions(-) > > create mode 100644 arch/arm/mach-at91/armv7/sama5d2_devices.c > > create mode 100644 arch/arm/mach-at91/include/mach/sama5d2.h > > create mode 100644 board/atmel/sama5d2_xplained/Kconfig > > create mode 100644 board/atmel/sama5d2_xplained/MAINTAINERS > > create mode 100644 board/atmel/sama5d2_xplained/Makefile > > create mode 100644 board/atmel/sama5d2_xplained/sama5d2_xplained.c > > create mode 100644 configs/sama5d2_xplained_mmc_defconfig > > create mode 100644 configs/sama5d2_xplained_spiflash_defconfig > > create mode 100644 include/configs/sama5d2_xplained.h > > > > diff --git a/arch/arm/mach-at91/Kconfig b/arch/arm/mach-at91/Kconfig > > index fdaf328..c333647 100644 > > --- a/arch/arm/mach-at91/Kconfig > > +++ b/arch/arm/mach-at91/Kconfig > > @@ -71,6 +71,10 @@ config TARGET_AT91SAM9X5EK > > select CPU_ARM926EJS > > select SUPPORT_SPL > > > > +config TARGET_SAMA5D2_XPLAINED > > + bool "SAMA5D2 Xplained board" > > + select CPU_V7 > > + > > config TARGET_SAMA5D3_XPLAINED > > bool "SAMA5D3 Xplained board" > > select CPU_V7 > > @@ -123,6 +127,7 @@ source "board/atmel/at91sam9m10g45ek/Kconfig" > > source "board/atmel/at91sam9n12ek/Kconfig" > > source "board/atmel/at91sam9rlek/Kconfig" > > source "board/atmel/at91sam9x5ek/Kconfig" > > +source "board/atmel/sama5d2_xplained/Kconfig" > > source "board/atmel/sama5d3_xplained/Kconfig" > > source "board/atmel/sama5d3xek/Kconfig" > > source "board/atmel/sama5d4_xplained/Kconfig" > > diff --git a/arch/arm/mach-at91/armv7/Makefile > > b/arch/arm/mach-at91/armv7/Makefile > > index f4f35a4..9538bc1 100644 > > --- a/arch/arm/mach-at91/armv7/Makefile > > +++ b/arch/arm/mach-at91/armv7/Makefile > > @@ -8,6 +8,7 @@ > > # SPDX-License-Identifier: GPL-2.0+ > > # > > > > +obj-$(CONFIG_SAMA5D2) += sama5d2_devices.o > > obj-$(CONFIG_SAMA5D3) += sama5d3_devices.o > > obj-$(CONFIG_SAMA5D4) += sama5d4_devices.o > > obj-y += clock.o > > diff --git a/arch/arm/mach-at91/armv7/sama5d2_devices.c > > b/arch/arm/mach-at91/armv7/sama5d2_devices.c > > new file mode 100644 > > index 0000000..6ebbccf > > --- /dev/null > > +++ b/arch/arm/mach-at91/armv7/sama5d2_devices.c > > @@ -0,0 +1,59 @@ > > +/* > > + * Copyright (C) 2015 Atmel Corporation > > + * Wenyou Yang <wenyou.y...@atmel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +#include <common.h> > > +#include <asm/io.h> > > +#include <asm/arch/at91_pmc.h> > > +#include <asm/arch/clk.h> > > +#include <asm/arch/sama5d2.h> > > + > > +char *get_cpu_name() > > +{ > > + unsigned int extension_id = get_extension_chip_id(); > > + > > + if (cpu_is_sama5d2()) { > > + switch (extension_id) { > > + case ARCH_EXID_SAMA5D21CU: > > + return "SAMA5D21"; > > + case ARCH_EXID_SAMA5D22CU: > > + return "SAMA5D22-CU"; > > + case ARCH_EXID_SAMA5D22CN: > > + return "SAMA5D22-CN"; > > + case ARCH_EXID_SAMA5D23CU: > > + return "SAMA5D23-CU"; > > + case ARCH_EXID_SAMA5D24CX: > > + return "SAMA5D24-CX"; > > + case ARCH_EXID_SAMA5D24CU: > > + return "SAMA5D24-CU"; > > + case ARCH_EXID_SAMA5D26CU: > > + return "SAMA5D26-CU"; > > + case ARCH_EXID_SAMA5D27CU: > > + return "SAMA5D27-CU"; > > + case ARCH_EXID_SAMA5D27CN: > > + return "SAMA5D27-CN"; > > + case ARCH_EXID_SAMA5D28CU: > > + return "SAMA5D28-CU"; > > + case ARCH_EXID_SAMA5D28CN: > > + return "SAMA5D28-CN"; > > + default: > > + ; > > + } > > + } > > + > > + return "Unknown CPU type"; > > +} > > You don't explain why it needs to keep the default option. I think it can be > removed. Switch-cases should almost always have a default case. The reason to use a default is to 'catch' an unexpected value. It is necessary for this function. This is my explanation. Do you agree? > > > + > > +#ifdef CONFIG_USB_GADGET_ATMEL_USBA > > +void at91_udp_hw_init(void) > > +{ > > + struct at91_pmc *pmc = (struct at91_pmc *)ATMEL_BASE_PMC; > > + > > + writel(AT91_PMC_UPLLEN | AT91_PMC_BIASEN, &pmc->uckr); > > + > > + at91_periph_clk_enable(ATMEL_ID_UDPHS); > > +} > > +#endif > > diff --git a/arch/arm/mach-at91/include/mach/at91_pmc.h > > b/arch/arm/mach-at91/include/mach/at91_pmc.h > > index 8a3fb94..5adaa30 100644 > > --- a/arch/arm/mach-at91/include/mach/at91_pmc.h > > +++ b/arch/arm/mach-at91/include/mach/at91_pmc.h > > @@ -78,7 +78,8 @@ typedef struct at91_pmc { > > #define AT91_PMC_PLLXR_DIV(x) (x & 0xFF) > > #define AT91_PMC_PLLXR_PLLCOUNT(x) ((x & 0x3F) << 8) > > #define AT91_PMC_PLLXR_OUT(x) ((x & 0x03) << 14) > > -#if defined(CONFIG_SAMA5D3) || defined(CONFIG_SAMA5D4) > > +#if defined(CONFIG_SAMA5D2) || defined(CONFIG_SAMA5D3) || \ > > + defined(CONFIG_SAMA5D4) > > #define AT91_PMC_PLLXR_MUL(x) ((x & 0x7F) << 18) > > #else > > #define AT91_PMC_PLLXR_MUL(x) ((x & 0x7FF) << 16) > > @@ -97,7 +98,8 @@ typedef struct at91_pmc { > > #define AT91_PMC_MCKR_CSS_PLLB 0x00000003 > > #define AT91_PMC_MCKR_CSS_MASK 0x00000003 > > > > -#if defined(CONFIG_SAMA5D3) || defined(CONFIG_SAMA5D4) || \ > > +#if defined(CONFIG_SAMA5D2) || defined(CONFIG_SAMA5D3) || \ > > + defined(CONFIG_SAMA5D4) || \ > > defined(CONFIG_AT91SAM9X5) || defined(CONFIG_AT91SAM9N12) > > #define AT91_PMC_MCKR_PRES_1 0x00000000 > > #define AT91_PMC_MCKR_PRES_2 0x00000010 > > @@ -127,10 +129,7 @@ typedef struct at91_pmc { > > #else > > #define AT91_PMC_MCKR_MDIV_1 0x00000000 > > #define AT91_PMC_MCKR_MDIV_2 0x00000100 > > -#if defined(CONFIG_SAMA5D3) || defined(CONFIG_SAMA5D4) || \ > > - defined(CONFIG_AT91SAM9X5) || defined(CONFIG_AT91SAM9N12) > > #define AT91_PMC_MCKR_MDIV_3 0x00000300 > > -#endif > > #define AT91_PMC_MCKR_MDIV_4 0x00000200 > > #define AT91_PMC_MCKR_MDIV_MASK 0x00000300 > > #endif > > diff --git a/arch/arm/mach-at91/include/mach/atmel_usba_udc.h > > b/arch/arm/mach-at91/include/mach/atmel_usba_udc.h > > index 38b5012..46a329b 100644 > > --- a/arch/arm/mach-at91/include/mach/atmel_usba_udc.h > > +++ b/arch/arm/mach-at91/include/mach/atmel_usba_udc.h > > @@ -31,7 +31,8 @@ static struct usba_ep_data usba_udc_ep[] = { > > EP("ep5", 5, 1024, 3, 1, 1), > > EP("ep6", 6, 1024, 3, 1, 1), > > }; > > -#elif defined(CONFIG_SAMA5D3) || defined(CONFIG_SAMA5D4) > > +#elif defined(CONFIG_SAMA5D2) || defined(CONFIG_SAMA5D3) || \ > > + defined(CONFIG_SAMA5D4) > > static struct usba_ep_data usba_udc_ep[] = { > > EP("ep0", 0, 64, 1, 0, 0), > > EP("ep1", 1, 1024, 3, 1, 0), > > diff --git a/arch/arm/mach-at91/include/mach/hardware.h > > b/arch/arm/mach-at91/include/mach/hardware.h > > index ff6b71b..38abfda 100644 > > --- a/arch/arm/mach-at91/include/mach/hardware.h > > +++ b/arch/arm/mach-at91/include/mach/hardware.h > > @@ -23,6 +23,8 @@ > > # include <asm/arch/at91sam9g45.h> > > #elif defined(CONFIG_AT91SAM9N12) || defined(CONFIG_AT91SAM9X5) > > # include <asm/arch/at91sam9x5.h> > > +#elif defined(CONFIG_SAMA5D2) > > +# include <asm/arch/sama5d2.h> > > #elif defined(CONFIG_SAMA5D3) > > # include <asm/arch/sama5d3.h> > > #elif defined(CONFIG_SAMA5D4) > > diff --git a/arch/arm/mach-at91/include/mach/sama5d2.h > > b/arch/arm/mach-at91/include/mach/sama5d2.h > > new file mode 100644 > > index 0000000..c85571c > > --- /dev/null > > +++ b/arch/arm/mach-at91/include/mach/sama5d2.h > > @@ -0,0 +1,203 @@ > > +/* > > + * Chip-specific header file for the SAMA5D2 SoC > > + * > > + * Copyright (C) 2015 Atmel > > + * Wenyou Yang <wenyou.y...@atmel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +#ifndef __SAMA5D2_H > > +#define __SAMA5D2_H > > + > > +/* > > + * definitions to be used in other places */ > > +#define CONFIG_AT91FAMILY /* It's a member of AT91 */ > > + > > +/* > > + * Peripheral identifiers/interrupts. > > + */ > > +#define ATMEL_ID_FIQ 0 /* FIQ Interrupt ID */ > > +/* 1 */ > > +#define ATMEL_ID_ARM 2 /* Performance Monitor Unit */ > > +#define ATMEL_ID_PIT 3 /* Periodic Interval Timer > > Interrupt > */ > > +#define ATMEL_ID_WDT 4 /* Watchdog Timer Interrupt */ > > +#define ATMEL_ID_GMAC 5 /* Ethernet MAC */ > > +#define ATMEL_ID_XDMAC0 6 /* DMA Controller 0 */ > > +#define ATMEL_ID_XDMAC1 7 /* DMA Controller 1 */ > > +#define ATMEL_ID_ICM 8 /* Integrity Check Monitor */ > > +#define ATMEL_ID_AES 9 /* Advanced Encryption Standard > */ > > +#define ATMEL_ID_AESB 10 /* AES bridge */ > > +#define ATMEL_ID_TDES 11 /* Triple Data Encryption > > Standard > */ > > +#define ATMEL_ID_SHA 12 /* SHA Signature */ > > +#define ATMEL_ID_MPDDRC 13 /* MPDDR Controller */ > > +#define ATMEL_ID_MATRIX1 14 /* H32MX, 32-bit AHB Matrix */ > > +#define ATMEL_ID_MATRIX0 15 /* H64MX, 64-bit AHB Matrix */ > > +#define ATMEL_ID_SECUMOD 16 /* Secure Module */ > > +#define ATMEL_ID_HSMC 17 /* Multi-bit ECC interrupt */ > > +#define ATMEL_ID_PIOA 18 /* Parallel I/O Controller A */ > > +#define ATMEL_ID_FLEXCOM0 19 /* FLEXCOM0 */ > > +#define ATMEL_ID_FLEXCOM1 20 /* FLEXCOM1 */ > > +#define ATMEL_ID_FLEXCOM2 21 /* FLEXCOM2 */ > > +#define ATMEL_ID_FLEXCOM3 22 /* FLEXCOM3 */ > > +#define ATMEL_ID_FLEXCOM4 23 /* FLEXCOM4 */ > > +#define ATMEL_ID_UART0 24 /* UART0 */ > > +#define ATMEL_ID_UART1 25 /* UART1 */ > > +#define ATMEL_ID_UART2 26 /* UART2 */ > > +#define ATMEL_ID_UART3 27 /* UART3 */ > > +#define ATMEL_ID_UART4 28 /* UART4 */ > > +#define ATMEL_ID_TWIHS0 29 /* Two-wire Interface 0 */ > > +#define ATMEL_ID_TWIHS1 30 /* Two-wire Interface 1 */ > > +#define ATMEL_ID_SDMMC0 31 /* Secure Data Memory > Card Controller 0 */ > > +#define ATMEL_ID_SDMMC1 32 /* Secure Data Memory > Card Controller 1 */ > > +#define ATMEL_ID_SPI0 33 /* Serial Peripheral Interface > > 0 */ > > +#define ATMEL_ID_SPI1 34 /* Serial Peripheral Interface > > 1 */ > > +#define ATMEL_ID_TC0 35 /* Timer Counter 0 (ch.0,1,2) */ > > +#define ATMEL_ID_TC1 36 /* Timer Counter 1 (ch.3,4,5) */ > > +/* 37 */ > > +#define ATMEL_ID_PWM 38 /* PWMController0 (ch. 0,1,2,3) > > */ > > +/* 39 */ > > +#define ATMEL_ID_ADC 40 /* Touch Screen ADC Controller > > */ > > +#define ATMEL_ID_UHPHS 41 /* USB Host High Speed */ > > +#define ATMEL_ID_UDPHS 42 /* USB Device High Speed */ > > +#define ATMEL_ID_SSC0 43 /* Serial Synchronous > > Controller 0 > */ > > +#define ATMEL_ID_SSC1 44 /* Serial Synchronous > > Controller 1 > */ > > +#define ATMEL_ID_LCDC 45 /* LCD Controller */ > > +#define ATMEL_ID_ISI 46 /* Image Sensor Controller, for > A5D2, named after ISC */ > > +#define ATMEL_ID_TRNG 47 /* True Random Number > Generator */ > > +#define ATMEL_ID_PDMIC 48 /* PDM Interface Controller */ > > +#define ATMEL_ID_AIC_IRQ 49 /* IRQ Interrupt ID */ > > +#define ATMEL_ID_SFC 50 /* Fuse Controller */ > > +#define ATMEL_ID_SECURAM 51 /* Secure RAM */ > > +#define ATMEL_ID_QSPI0 52 /* QSPI0 */ > > +#define ATMEL_ID_QSPI1 53 /* QSPI1 */ > > +#define ATMEL_ID_I2SC0 54 /* Inter-IC Sound Controller 0 > > */ > > +#define ATMEL_ID_I2SC1 55 /* Inter-IC Sound Controller 1 > > */ > > +#define ATMEL_ID_CAN0_INT0 56 /* MCAN 0 Interrupt0 */ > > +#define ATMEL_ID_CAN1_INT0 57 /* MCAN 1 Interrupt0 */ > > +/* 58 */ > > +#define ATMEL_ID_CLASSD 59 /* Audio Class D Amplifier > */ > > +#define ATMEL_ID_SFR 60 /* Special Function Register */ > > +#define ATMEL_ID_SAIC 61 /* Secured AIC */ > > +#define ATMEL_ID_AIC 62 /* Advanced Interrupt > > Controller */ > > +#define ATMEL_ID_L2CC 63 /* L2 Cache Controller */ > > +#define ATMEL_ID_CAN0_INT1 64 /* MCAN 0 Interrupt1 */ > > +#define ATMEL_ID_CAN1_INT1 65 /* MCAN 1 Interrupt1 */ > > +#define ATMEL_ID_GMAC_Q1 66 /* GMAC Queue 1 Interrupt */ > > +#define ATMEL_ID_GMAC_Q2 67 /* GMAC Queue 2 Interrupt */ > > +#define ATMEL_ID_PIOB 68 /* Parallel I/O Controller B */ > > +#define ATMEL_ID_PIOC 69 /* Parallel I/O Controller C */ > > +#define ATMEL_ID_PIOD 70 /* Parallel I/O Controller D */ > > +#define ATMEL_ID_SDMMC0_TIMER 71 /* Secure Data Memory > Card Controller 0 (TIMER) */ > > +#define ATMEL_ID_SDMMC1_TIMER 72 /* Secure Data Memory > Card Controller 1 (TIMER) */ > > +/* 73 */ > > +#define ATMEL_ID_SYS 74 /* System Controller Interrupt > > */ > > +#define ATMEL_ID_ACC 75 /* Analog Comparator */ > > +#define ATMEL_ID_RXLP 76 /* UART Low-Power */ > > +#define ATMEL_ID_SFRBU 77 /* Special Function Register > BackUp */ > > +#define ATMEL_ID_CHIPID 78 /* Chip ID */ > > + > > +/* > > + * User Peripherals physical base addresses. > > + */ > > +#define ATMEL_BASE_LCDC 0xf0000000 > > +#define ATMEL_BASE_XDMAC1 0xf0004000 > > +#define ATMEL_BASE_MPDDRC 0xf000c000 > > +#define ATMEL_BASE_XDMAC0 0xf0010000 > > +#define ATMEL_BASE_PMC 0xf0014000 > > +#define ATMEL_BASE_QSPI0 0xf0020000 > > +#define ATMEL_BASE_QSPI1 0xf0024000 > > +#define ATMEL_BASE_SPI0 0xf8000000 > > +#define ATMEL_BASE_GMAC 0xf8008000 > > +#define ATMEL_BASE_TC0 0xf800c000 > > +#define ATMEL_BASE_TC1 0xf8010000 > > +#define ATMEL_BASE_HSMC 0xf8014000 > > +#define ATMEL_BASE_UART0 0xf801c000 > > +#define ATMEL_BASE_UART1 0xf8020000 > > +#define ATMEL_BASE_UART2 0xf8024000 > > +#define ATMEL_BASE_TWI0 0xf8028000 > > +#define ATMEL_BASE_SYSC 0xf8048000 > > +#define ATMEL_BASE_SPI1 0xfc000000 > > +#define ATMEL_BASE_UART3 0xfc008000 > > +#define ATMEL_BASE_UART4 0xfc00c000 > > +#define ATMEL_BASE_TWI1 0xfc028000 > > +#define ATMEL_BASE_UDPHS 0xfc02c000 > > + > > +#define ATMEL_BASE_PIOA 0xfc038000 > > + > > +#define ATMEL_CHIPID_CIDR 0xfc069000 > > +#define ATMEL_CHIPID_EXID 0xfc069004 > > + > > +/* > > + * Address Memory Space > > + */ > > +#define ATMEL_BASE_DDRCS 0x20000000 > > +#define ATMEL_BASE_QSPI0_AES_MEM 0x90000000 > > +#define ATMEL_BASE_QSPI1_AES_MEM 0x98000000 > > +#define ATMEL_BASE_SDMMC0 0xa0000000 > > +#define ATMEL_BASE_SDMMC1 0xb0000000 > > +#define ATMEL_BASE_QSPI0_MEM 0xd0000000 > > +#define ATMEL_BASE_QSPI1_MEM 0xd8000000 > > + > > +/* > > + * Internal Memories > > + */ > > +#define ATMEL_BASE_UDPHS_FIFO 0x00300000 /* USB Device HS > controller */ > > +#define ATMEL_BASE_OHCI 0x00400000 /* USB Host > controller (OHCI) */ > > +#define ATMEL_BASE_EHCI 0x00500000 /* USB Host > controller (EHCI) */ > > + > > +/* > > + * SYSC Spawns > > + */ > > +#define ATMEL_BASE_RSTC ATMEL_BASE_SYSC > > +#define ATMEL_BASE_SHDWC (ATMEL_BASE_SYSC + 0x10) > > +#define ATMEL_BASE_PIT (ATMEL_BASE_SYSC + 0x30) > > +#define ATMEL_BASE_WDT (ATMEL_BASE_SYSC + 0x40) > > +#define ATMEL_BASE_SCKC (ATMEL_BASE_SYSC + 0x50) > > +#define ATMEL_BASE_RTC (ATMEL_BASE_SYSC + 0xb0) > > + > > +/* > > + * Other misc definitions > > + */ > > +#define ATMEL_BASE_PMECC (ATMEL_BASE_HSMC + 0x70) > > +#define ATMEL_BASE_PMERRLOC (ATMEL_BASE_HSMC + 0x500) > > + > > +#define ATMEL_BASE_PIOB (ATMEL_BASE_PIOA + 0x40) > > +#define ATMEL_BASE_PIOC (ATMEL_BASE_PIOB + 0x40) > > +#define ATMEL_BASE_PIOD (ATMEL_BASE_PIOC + 0x40) > > + > > +#define ATMEL_PIO_PORTS 4 > > +#define CPU_HAS_PCR > > +#define CPU_HAS_H32MXDIV > > + > > +/* SAMA5D2 series chip id definitions */ > > +#define ARCH_ID_SAMA5D2 0x8a5c08c0 > > +#define ARCH_EXID_SAMA5D21CU 0x0000005a > > +#define ARCH_EXID_SAMA5D22CU 0x00000059 > > +#define ARCH_EXID_SAMA5D22CN 0x00000069 > > +#define ARCH_EXID_SAMA5D23CU 0x00000058 > > +#define ARCH_EXID_SAMA5D24CX 0x00000004 > > +#define ARCH_EXID_SAMA5D24CU 0x00000014 > > +#define ARCH_EXID_SAMA5D26CU 0x00000012 > > +#define ARCH_EXID_SAMA5D27CU 0x00000011 > > +#define ARCH_EXID_SAMA5D27CN 0x00000021 > > +#define ARCH_EXID_SAMA5D28CU 0x00000010 > > +#define ARCH_EXID_SAMA5D28CN 0x00000020 > > + > > +#define cpu_is_sama5d2() (get_chip_id() == ARCH_ID_SAMA5D2) > > + > > +/* PIT Timer(PIT_PIIR) */ > > +#define CONFIG_SYS_TIMER_COUNTER 0xf804803c > > + > > +/* No PMECC Galois table in ROM */ > > +#define NO_GALOIS_TABLE_IN_ROM > > + > > +#ifndef __ASSEMBLY__ > > +unsigned int get_chip_id(void); > > +unsigned int get_extension_chip_id(void); unsigned int > > +has_lcdc(void); char *get_cpu_name(void); #endif > > + > > +#endif > > diff --git a/board/atmel/sama5d2_xplained/Kconfig > > b/board/atmel/sama5d2_xplained/Kconfig > > new file mode 100644 > > index 0000000..55712e9 > > --- /dev/null > > +++ b/board/atmel/sama5d2_xplained/Kconfig > > @@ -0,0 +1,15 @@ > > +if TARGET_SAMA5D2_XPLAINED > > + > > +config SYS_BOARD > > + default "sama5d2_xplained" > > + > > +config SYS_VENDOR > > + default "atmel" > > + > > +config SYS_SOC > > + default "at91" > > + > > +config SYS_CONFIG_NAME > > + default "sama5d2_xplained" > > + > > +endif > > diff --git a/board/atmel/sama5d2_xplained/MAINTAINERS > > b/board/atmel/sama5d2_xplained/MAINTAINERS > > new file mode 100644 > > index 0000000..ff9c86f > > --- /dev/null > > +++ b/board/atmel/sama5d2_xplained/MAINTAINERS > > @@ -0,0 +1,7 @@ > > +SAMA5D2 XPLAINED BOARD > > +M: Wenyou Yang <wenyou.y...@atmel.com> > > +S: Maintained > > +F: board/atmel/sama5d2_xplained/ > > +F: include/configs/sama5d2_xplained.h > > +F: configs/sama5d2_xplained_mmc_defconfig > > +F: configs/sama5d2_xplained_spiflash_defconfig > > diff --git a/board/atmel/sama5d2_xplained/Makefile > > b/board/atmel/sama5d2_xplained/Makefile > > new file mode 100644 > > index 0000000..420870b > > --- /dev/null > > +++ b/board/atmel/sama5d2_xplained/Makefile > > @@ -0,0 +1,8 @@ > > +# > > +# Copyright (C) 2015 Atmel Corporation > > +# Wenyou Yang <wenyou.y...@atmel.com> > > +# > > +# SPDX-License-Identifier: GPL-2.0+ > > +# > > + > > +obj-y += sama5d2_xplained.o > > diff --git a/board/atmel/sama5d2_xplained/sama5d2_xplained.c > > b/board/atmel/sama5d2_xplained/sama5d2_xplained.c > > new file mode 100644 > > index 0000000..c33555a > > --- /dev/null > > +++ b/board/atmel/sama5d2_xplained/sama5d2_xplained.c > > @@ -0,0 +1,282 @@ > > +/* > > + * Copyright (C) 2015 Atmel Corporation > > + * Wenyou.Yang <wenyou.y...@atmel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +#include <common.h> > > +#include <atmel_hlcdc.h> > > +#include <lcd.h> > > +#include <mmc.h> > > +#include <net.h> > > +#include <netdev.h> > > +#include <spi.h> > > +#include <version.h> > > +#include <asm/io.h> > > +#include <asm/arch/at91_common.h> > > +#include <asm/arch/at91_pmc.h> > > +#include <asm/arch/atmel_pio4.h> > > +#include <asm/arch/atmel_usba_udc.h> > > +#include <asm/arch/atmel_sdhci.h> > > +#include <asm/arch/clk.h> > > +#include <asm/arch/gpio.h> > > +#include <asm/arch/sama5d2.h> > > + > > +DECLARE_GLOBAL_DATA_PTR; > > + > > +int spi_cs_is_valid(unsigned int bus, unsigned int cs) { > > + return bus == 0 && cs == 0; > > +} > > + > > +void spi_cs_activate(struct spi_slave *slave) { > > + atmel_pio4_set_pio_output(AT91_PIO_PORTA, 17, 0); } > > + > > +void spi_cs_deactivate(struct spi_slave *slave) { > > + atmel_pio4_set_pio_output(AT91_PIO_PORTA, 17, 1); } > > + > > +static void board_spi0_hw_init(void) > > +{ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTA, 14, 0); > > + atmel_pio4_set_a_periph(AT91_PIO_PORTA, 15, 0); > > + atmel_pio4_set_a_periph(AT91_PIO_PORTA, 16, 0); > > + > > + atmel_pio4_set_pio_output(AT91_PIO_PORTA, 17, 1); > > + > > + at91_periph_clk_enable(ATMEL_ID_SPI0); > > +} > > + > > +static void board_usb_hw_init(void) > > +{ > > + atmel_pio4_set_pio_output(AT91_PIO_PORTB, 9, 1); > > Sorry for ignoring this one in the previous review. > According to the schematic, this pin is not used for USB. Thank you for your review. > > > + atmel_pio4_set_pio_output(AT91_PIO_PORTB, 10, 1); } > > + > > +#ifdef CONFIG_LCD > > +vidinfo_t panel_info = { > > + .vl_col = 480, > > + .vl_row = 272, > > + .vl_clk = 9000000, > > + .vl_bpix = LCD_BPP, > > + .vl_tft = 1, > > + .vl_hsync_len = 41, > > + .vl_left_margin = 2, > > + .vl_right_margin = 2, > > + .vl_vsync_len = 11, > > + .vl_upper_margin = 2, > > + .vl_lower_margin = 2, > > + .mmio = ATMEL_BASE_LCDC, > > +}; > > + > > +/* No power up/down pin for the LCD pannel */ > > +void lcd_enable(void) { /* Empty! */ } > > +void lcd_disable(void) { /* Empty! */ } > > + > > +unsigned int has_lcdc(void) > > +{ > > + return 1; > > +} > > + > > +static void board_lcd_hw_init(void) > > +{ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 28, 0); /* LCDPWM */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 29, 0); /* LCDDISP */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 30, 0); /* LCDVSYNC */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 31, 0); /* LCDHSYNC */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTD, 0, 0); /* LCDPCK */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTD, 1, 0); /* LCDDEN */ > > + > > + /* LCDDAT0 */ > > + /* LCDDAT1 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 10, 0); /* LCDDAT2 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 11, 0); /* LCDDAT3 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 12, 0); /* LCDDAT4 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 13, 0); /* LCDDAT5 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 14, 0); /* LCDDAT6 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 15, 0); /* LCDDAT7 */ > > + > > + /* LCDDAT8 */ > > + /* LCDDAT9 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 16, 0); /* LCDDAT10 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 17, 0); /* LCDDAT11 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 18, 0); /* LCDDAT12 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 19, 0); /* LCDDAT13 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 20, 0); /* LCDDAT14 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 21, 0); /* LCDDAT15 */ > > + > > + /* LCDD16 */ > > + /* LCDD17 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 22, 0); /* LCDDAT18 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 23, 0); /* LCDDAT19 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 24, 0); /* LCDDAT20 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 25, 0); /* LCDDAT21 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 26, 0); /* LCDDAT22 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTC, 27, 0); /* LCDDAT23 */ > > + > > + at91_periph_clk_enable(ATMEL_ID_LCDC); > > +} > > + > > +#ifdef CONFIG_LCD_INFO > > +void lcd_show_board_info(void) > > +{ > > + ulong dram_size; > > + int i; > > + char temp[32]; > > + > > + lcd_printf("%s\n", U_BOOT_VERSION); > > + lcd_printf("2015 ATMEL Corp\n"); > > + lcd_printf("%s CPU at %s MHz\n", get_cpu_name(), > > + strmhz(temp, get_cpu_clk_rate())); > > + > > + dram_size = 0; > > + for (i = 0; i < CONFIG_NR_DRAM_BANKS; i++) > > + dram_size += gd->bd->bi_dram[i].size; > > + > > + lcd_printf("%ld MB SDRAM\n", dram_size >> 20); } #endif /* > > +CONFIG_LCD_INFO */ #endif /* CONFIG_LCD */ > > + > > +static void board_gmac_hw_init(void) > > +{ > > + atmel_pio4_set_f_periph(AT91_PIO_PORTB, 14, 0); /* GTXCK */ > > + atmel_pio4_set_f_periph(AT91_PIO_PORTB, 15, 0); /* GTXEN */ > > + atmel_pio4_set_f_periph(AT91_PIO_PORTB, 16, 0); /* GRXDV */ > > + atmel_pio4_set_f_periph(AT91_PIO_PORTB, 17, 0); /* GRXER */ > > + atmel_pio4_set_f_periph(AT91_PIO_PORTB, 18, 0); /* GRX0 */ > > + atmel_pio4_set_f_periph(AT91_PIO_PORTB, 19, 0); /* GRX1 */ > > + atmel_pio4_set_f_periph(AT91_PIO_PORTB, 20, 0); /* GTX0 */ > > + atmel_pio4_set_f_periph(AT91_PIO_PORTB, 21, 0); /* GTX1 */ > > + atmel_pio4_set_f_periph(AT91_PIO_PORTB, 22, 0); /* GMDC */ > > + atmel_pio4_set_f_periph(AT91_PIO_PORTB, 23, 0); /* GMDIO */ > > + > > + at91_periph_clk_enable(ATMEL_ID_GMAC); > > +} > > + > > +static void board_sdhci0_hw_init(void) { > > + atmel_pio4_set_a_periph(AT91_PIO_PORTA, 0, 0); /* SDMMC0_CK > */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTA, 1, 0); /* SDMMC0_CMD > */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTA, 2, 0); /* > SDMMC0_DAT0 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTA, 3, 0); /* > SDMMC0_DAT1 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTA, 4, 0); /* > SDMMC0_DAT2 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTA, 5, 0); /* > SDMMC0_DAT3 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTA, 6, 0); /* > SDMMC0_DAT4 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTA, 7, 0); /* > SDMMC0_DAT5 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTA, 8, 0); /* > SDMMC0_DAT6 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTA, 9, 0); /* > SDMMC0_DAT7 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTA, 10, 0); /* > SDMMC0_RSTN */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTA, 11, 0); /* > SDMMC0_VDDSEL */ > > + > > + at91_periph_clk_enable(ATMEL_ID_SDMMC0); > > + at91_enable_periph_generated_clk(ATMEL_ID_SDMMC0); > > +} > > + > > +static void board_sdhci1_hw_init(void) { > > + atmel_pio4_set_e_periph(AT91_PIO_PORTA, 18, 0); /* > SDMMC1_DAT0 */ > > + atmel_pio4_set_e_periph(AT91_PIO_PORTA, 19, 0); /* > SDMMC1_DAT1 */ > > + atmel_pio4_set_e_periph(AT91_PIO_PORTA, 20, 0); /* > SDMMC1_DAT2 */ > > + atmel_pio4_set_e_periph(AT91_PIO_PORTA, 21, 0); /* > SDMMC1_DAT3 */ > > + atmel_pio4_set_e_periph(AT91_PIO_PORTA, 22, 0); /* SDMMC1_CK > */ > > + atmel_pio4_set_e_periph(AT91_PIO_PORTA, 27, 0); /* > SDMMC1_RSTN */ > > + atmel_pio4_set_e_periph(AT91_PIO_PORTA, 28, 0); /* SDMMC1_CMD > */ > > + atmel_pio4_set_e_periph(AT91_PIO_PORTA, 30, 0); /* SDMMC1_CD > */ > > + > > + at91_periph_clk_enable(ATMEL_ID_SDMMC1); > > + at91_enable_periph_generated_clk(ATMEL_ID_SDMMC1); > > +} > > + > > +int board_mmc_init(bd_t *bis) > > +{ > > +#ifdef CONFIG_ATMEL_SDHCI0 > > + atmel_sdhci_init((void *)ATMEL_BASE_SDMMC0, ATMEL_ID_SDMMC0); > #endif > > +#ifdef CONFIG_ATMEL_SDHCI1 > > + atmel_sdhci_init((void *)ATMEL_BASE_SDMMC1, ATMEL_ID_SDMMC1); > #endif > > + > > + return 0; > > +} > > + > > +static void board_uart1_hw_init(void) { > > + atmel_pio4_set_a_periph(AT91_PIO_PORTD, 2, 1); /* URXD1 */ > > + atmel_pio4_set_a_periph(AT91_PIO_PORTD, 3, 0); /* UTXD1 */ > > + > > + at91_periph_clk_enable(ATMEL_ID_UART1); > > +} > > + > > +int board_early_init_f(void) > > +{ > > + at91_periph_clk_enable(ATMEL_ID_PIOA); > > + at91_periph_clk_enable(ATMEL_ID_PIOB); > > + at91_periph_clk_enable(ATMEL_ID_PIOC); > > + at91_periph_clk_enable(ATMEL_ID_PIOD); > > + > > + board_uart1_hw_init(); > > + > > + return 0; > > +} > > + > > +int board_init(void) > > +{ > > + /* address of boot parameters */ > > + gd->bd->bi_boot_params = CONFIG_SYS_SDRAM_BASE + 0x100; > > + > > +#ifdef CONFIG_ATMEL_SPI > > + board_spi0_hw_init(); > > +#endif > > +#ifdef CONFIG_ATMEL_SDHCI > > +#ifdef CONFIG_ATMEL_SDHCI0 > > + board_sdhci0_hw_init(); > > +#endif > > +#ifdef CONFIG_ATMEL_SDHCI1 > > + board_sdhci1_hw_init(); > > +#endif > > +#endif > > +#ifdef CONFIG_MACB > > + board_gmac_hw_init(); > > +#endif > > +#ifdef CONFIG_LCD > > + board_lcd_hw_init(); > > +#endif > > +#ifdef CONFIG_CMD_USB > > + board_usb_hw_init(); > > +#endif > > +#ifdef CONFIG_USB_GADGET_ATMEL_USBA > > + at91_udp_hw_init(); > > +#endif > > + > > + return 0; > > +} > > + > > +int dram_init(void) > > +{ > > + gd->ram_size = get_ram_size((void *)CONFIG_SYS_SDRAM_BASE, > > + CONFIG_SYS_SDRAM_SIZE); > > + return 0; > > +} > > + > > +int board_eth_init(bd_t *bis) > > +{ > > + int rc = 0; > > + > > +#ifdef CONFIG_MACB > > + rc = macb_eth_initialize(0, (void *)ATMEL_BASE_GMAC, 0x00); #endif > > + > > +#ifdef CONFIG_USB_GADGET_ATMEL_USBA > > + usba_udc_probe(&pdata); > > +#ifdef CONFIG_USB_ETH_RNDIS > > + usb_eth_initialize(bis); > > +#endif > > +#endif > > + > > + return rc; > > +} > > diff --git a/configs/sama5d2_xplained_mmc_defconfig > > b/configs/sama5d2_xplained_mmc_defconfig > > new file mode 100644 > > index 0000000..c1dcbef > > --- /dev/null > > +++ b/configs/sama5d2_xplained_mmc_defconfig > > @@ -0,0 +1,11 @@ > > +CONFIG_ARM=y > > +CONFIG_ARCH_AT91=y > > +CONFIG_TARGET_SAMA5D2_XPLAINED=y > > +CONFIG_SYS_EXTRA_OPTIONS="SAMA5D2,SYS_USE_MMC" > > +# CONFIG_CMD_IMI is not set > > +# CONFIG_CMD_IMLS is not set > > +# CONFIG_CMD_LOADS is not set > > +# CONFIG_CMD_FLASH is not set > > +CONFIG_CMD_SF=y > > +# CONFIG_CMD_FPGA is not set > > +CONFIG_SPI_FLASH=y > > diff --git a/configs/sama5d2_xplained_spiflash_defconfig > > b/configs/sama5d2_xplained_spiflash_defconfig > > new file mode 100644 > > index 0000000..0271e8e > > --- /dev/null > > +++ b/configs/sama5d2_xplained_spiflash_defconfig > > @@ -0,0 +1,11 @@ > > +CONFIG_ARM=y > > +CONFIG_ARCH_AT91=y > > +CONFIG_TARGET_SAMA5D2_XPLAINED=y > > +CONFIG_SYS_EXTRA_OPTIONS="SAMA5D2,SYS_USE_SERIALFLASH" > > +# CONFIG_CMD_IMI is not set > > +# CONFIG_CMD_IMLS is not set > > +# CONFIG_CMD_LOADS is not set > > +# CONFIG_CMD_FLASH is not set > > +CONFIG_CMD_SF=y > > +# CONFIG_CMD_FPGA is not set > > +CONFIG_SPI_FLASH=y > > diff --git a/include/configs/sama5d2_xplained.h > > b/include/configs/sama5d2_xplained.h > > new file mode 100644 > > index 0000000..ae5ba3d > > --- /dev/null > > +++ b/include/configs/sama5d2_xplained.h > > @@ -0,0 +1,122 @@ > > +/* > > + * Configuration file for the SAMA5D2 Xplained Board. > > + * > > + * Copyright (C) 2015 Atmel Corporation > > + * Wenyou Yang <wenyou.y...@atmel.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > + > > +#ifndef __CONFIG_H > > +#define __CONFIG_H > > + > > +/* No NOR flash, this definition should put before common header */ > > +#define CONFIG_SYS_NO_FLASH > > + > > +#include "at91-sama5_common.h" > > + > > +/* serial console */ > > +#define CONFIG_ATMEL_USART > > +#define CONFIG_USART_BASE ATMEL_BASE_UART1 > > +#define CONFIG_USART_ID ATMEL_ID_UART1 > > + > > +/* SDRAM */ > > +#define CONFIG_NR_DRAM_BANKS 1 > > +#define CONFIG_SYS_SDRAM_BASE ATMEL_BASE_DDRCS > > +#define CONFIG_SYS_SDRAM_SIZE 0x20000000 > > + > > +#define CONFIG_SYS_INIT_SP_ADDR \ > > + (CONFIG_SYS_SDRAM_BASE + 4 * 1024 - > GENERATED_GBL_DATA_SIZE) > > + > > +#define CONFIG_SYS_LOAD_ADDR 0x22000000 /* load > address */ > > + > > +#undef CONFIG_AT91_GPIO > > +#define CONFIG_ATMEL_PIO4 > > + > > +/* SerialFlash */ > > +#ifdef CONFIG_CMD_SF > > +#define CONFIG_ATMEL_SPI > > +#define CONFIG_ATMEL_SPI0 > > +#define CONFIG_SPI_FLASH_ATMEL > > +#define CONFIG_SF_DEFAULT_BUS 0 > > +#define CONFIG_SF_DEFAULT_CS 0 > > +#define CONFIG_SF_DEFAULT_SPEED 30000000 > > +#endif > > + > > +/* NAND flash */ > > +#undef CONFIG_CMD_NAND > > + > > +/* MMC */ > > +#define CONFIG_CMD_MMC > > + > > +#ifdef CONFIG_CMD_MMC > > +#define CONFIG_MMC > > +#define CONFIG_GENERIC_MMC > > +#define CONFIG_SDHCI > > +#define CONFIG_ATMEL_SDHCI > > +#define CONFIG_ATMEL_SDHCI0 > > +#define CONFIG_ATMEL_SDHCI1 > > +#define CONFIG_SUPPORT_EMMC_BOOT > > +#endif > > + > > +/* USB */ > > +#define CONFIG_CMD_USB > > + > > +#ifdef CONFIG_CMD_USB > > +#define CONFIG_USB_EHCI > > +#define CONFIG_USB_EHCI_ATMEL > > +#define CONFIG_SYS_USB_EHCI_MAX_ROOT_PORTS 3 > > +#define CONFIG_USB_STORAGE > > +#endif > > + > > +/* USB device */ > > +#define CONFIG_USB_GADGET > > +#define CONFIG_USB_GADGET_DUALSPEED > > +#define CONFIG_USB_GADGET_ATMEL_USBA > > +#define CONFIG_USB_ETHER > > +#define CONFIG_USB_ETH_RNDIS > > +#define CONFIG_USBNET_MANUFACTURER "Atmel SAMA5D2 XPlained" > > + > > +#if defined(CONFIG_CMD_USB) || defined(CONFIG_CMD_MMC) #define > > +CONFIG_CMD_FAT #define CONFIG_DOS_PARTITION #endif > > + > > +/* Ethernet Hardware */ > > +#define CONFIG_MACB > > +#define CONFIG_RMII > > +#define CONFIG_NET_RETRY_COUNT 20 > > +#define CONFIG_MACB_SEARCH_PHY > > + > > +/* LCD */ > > +/* #define CONFIG_LCD */ > > + > > +#ifdef CONFIG_LCD > > +#define LCD_BPP LCD_COLOR16 > > +#define LCD_OUTPUT_BPP 24 > > +#define CONFIG_LCD_LOGO > > +#define CONFIG_LCD_INFO > > +#define CONFIG_LCD_INFO_BELOW_LOGO > > +#define CONFIG_SYS_WHITE_ON_BLACK > > +#define CONFIG_ATMEL_HLCD > > +#define CONFIG_ATMEL_LCD_RGB565 > > +#define CONFIG_SYS_CONSOLE_IS_IN_ENV > > +#endif > > + > > +#ifdef CONFIG_SYS_USE_MMC > > + > > +/* bootstrap + u-boot + env in sd card */ #undef > > +FAT_ENV_DEVICE_AND_PART #undef CONFIG_BOOTCOMMAND > > + > > +#define FAT_ENV_DEVICE_AND_PART "1" > > +#define CONFIG_BOOTCOMMAND "fatload mmc 1:1 0x21000000 at91- > sama5d2_xplained.dtb; " \ > > + "fatload mmc 1:1 0x22000000 zImage; " \ > > + "bootz 0x22000000 - 0x21000000" > > +#undef CONFIG_BOOTARGS > > +#define CONFIG_BOOTARGS \ > > + "console=ttyS0,115200 earlyprintk root=/dev/mmcblk1p2 rw rootwait" > > + > > +#endif > > + > > +#endif > > > > Best Regards, > Bo Shen Best Regards, Wenyou Yang _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot