[U-Boot] [PATCH v2 01/12] ARM: kirkwood: add db-88f6281-bp board
Stefan Roese
sr at denx.de
Thu Mar 7 09:33:15 UTC 2019
Hi Chris,
On 07.03.19 09:27, Chris Packham wrote:
> This is Marvell's Kirkwood development board. It has the following
> features
>
> - 512M DDR2
> - 2 PCI connectors
> - 1 x1 PCI-e interface
> - 1 Gigabit Ethernet Port
> - 2 SATA Ports
> - USB 2.0 Interface
> - SDIO
> - 128M NAND Flash
> - 16M SPI Flash
>
> It can be strapped to boot from SPI or NAND so there are two defconfigs
> (one per boot media).
Thanks for this board addition. Please find some review comments
below.
>
> Signed-off-by: Chris Packham <judge.packham at gmail.com>
> ---
>
> arch/arm/dts/kirkwood-db-88f6281-spi.dts | 48 +++++++++
> arch/arm/dts/kirkwood-db-88f6281.dts | 26 +++++
> arch/arm/dts/kirkwood-db.dtsi | 94 +++++++++++++++++
> arch/arm/mach-kirkwood/Kconfig | 4 +
> board/Marvell/db-88f6281-bp/Kconfig | 12 +++
> board/Marvell/db-88f6281-bp/MAINTAINERS | 10 ++
> board/Marvell/db-88f6281-bp/Makefile | 3 +
> board/Marvell/db-88f6281-bp/db-88f6281-bp.c | 103 ++++++++++++++++++
> board/Marvell/db-88f6281-bp/kwbimage.cfg | 36 +++++++
> configs/db-88f6281-bp-nand_defconfig | 53 ++++++++++
> configs/db-88f6281-bp-spi_defconfig | 54 ++++++++++
> include/configs/db-88f6281-bp.h | 109 ++++++++++++++++++++
> 12 files changed, 552 insertions(+)
> create mode 100644 arch/arm/dts/kirkwood-db-88f6281-spi.dts
> create mode 100644 arch/arm/dts/kirkwood-db-88f6281.dts
> create mode 100644 arch/arm/dts/kirkwood-db.dtsi
> create mode 100644 board/Marvell/db-88f6281-bp/Kconfig
> create mode 100644 board/Marvell/db-88f6281-bp/MAINTAINERS
> create mode 100644 board/Marvell/db-88f6281-bp/Makefile
> create mode 100644 board/Marvell/db-88f6281-bp/db-88f6281-bp.c
> create mode 100644 board/Marvell/db-88f6281-bp/kwbimage.cfg
> create mode 100644 configs/db-88f6281-bp-nand_defconfig
> create mode 100644 configs/db-88f6281-bp-spi_defconfig
> create mode 100644 include/configs/db-88f6281-bp.h
>
> diff --git a/arch/arm/dts/kirkwood-db-88f6281-spi.dts b/arch/arm/dts/kirkwood-db-88f6281-spi.dts
> new file mode 100644
> index 000000000000..50b1b0d4a535
> --- /dev/null
> +++ b/arch/arm/dts/kirkwood-db-88f6281-spi.dts
> @@ -0,0 +1,48 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Marvell DB-88F6281-BP Development Board Setup
> + *
> + * Saeed Bishara <saeed at marvell.com>
> + * Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
> + *
> + */
> +
> +/dts-v1/;
> +
> +#include "kirkwood-db-88f6281.dts"
> +
> +/ {
> + aliases {
> + spi0 = &spi0;
> + };
> +};
> +
> +&spi0 {
> + status = "okay";
> +
> + flash at 0 {
> + #address-cells = <1>;
> + #size-cells = <1>;
> + compatible = "st,m25p128", "jedec,spi-nor", "spi-flash";
> + reg = <0>;
> + spi-max-frequency = <50000000>;
> + mode = <0>;
> +
> + partition at u-boot {
> + reg = <0x00000000 0x00c00000>;
> + label = "u-boot";
> + };
> + partition at u-boot-env {
> + reg = <0x00c00000 0x00040000>;
> + label = "u-boot-env";
> + };
> + partition at unused {
> + reg = <0x00100000 0x00f00000>;
> + label = "unused";
> + };
> + };
> +};
> +
> +&nand {
> + status = "disabled";
> +};
> diff --git a/arch/arm/dts/kirkwood-db-88f6281.dts b/arch/arm/dts/kirkwood-db-88f6281.dts
> new file mode 100644
> index 000000000000..2adb17c955aa
> --- /dev/null
> +++ b/arch/arm/dts/kirkwood-db-88f6281.dts
> @@ -0,0 +1,26 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Marvell DB-88F6281-BP Development Board Setup
> + *
> + * Saeed Bishara <saeed at marvell.com>
> + * Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
> + *
> + */
> +
> +/dts-v1/;
> +
> +#include "kirkwood-db.dtsi"
> +#include "kirkwood-6281.dtsi"
> +
> +/ {
> + model = "Marvell DB-88F6281-BP Development Board";
> + compatible = "marvell,db-88f6281-bp", "marvell,kirkwood-88f6281", "marvell,kirkwood";
> +};
> +
> +&pciec {
> + status = "okay";
> +};
> +
> +&pcie0 {
> + status = "okay";
> +};
> diff --git a/arch/arm/dts/kirkwood-db.dtsi b/arch/arm/dts/kirkwood-db.dtsi
> new file mode 100644
> index 000000000000..b81d8e8298a3
> --- /dev/null
> +++ b/arch/arm/dts/kirkwood-db.dtsi
> @@ -0,0 +1,94 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * Marvell DB-{88F6281,88F6282}-BP Development Board Setup
> + *
> + * Saeed Bishara <saeed at marvell.com>
> + * Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
> + *
> + * This file contains the definitions that are common between the 6281
> + * and 6282 variants of the Marvell Kirkwood Development Board.
> + */
> +
> +#include "kirkwood.dtsi"
> +
> +/ {
> + memory {
> + device_type = "memory";
> + reg = <0x00000000 0x20000000>; /* 512 MB */
> + };
> +
> + chosen {
> + bootargs = "console=ttyS0,115200n8 earlyprintk";
> + stdout-path = &uart0;
> + };
> +
> + aliases {
> + ethernet0 = ð0;
> + spi0 = &spi0;
> + };
> +
> + ocp at f1000000 {
> + pin-controller at 10000 {
> + pmx_sdio_gpios: pmx-sdio-gpios {
> + marvell,pins = "mpp37", "mpp38";
> + marvell,function = "gpio";
> + };
> + };
> +
> + serial at 12000 {
> + status = "okay";
> + };
> +
> + sata at 80000 {
> + nr-ports = <2>;
> + status = "okay";
> + };
> +
> + ehci at 50000 {
> + status = "okay";
> + };
> +
> + mvsdio at 90000 {
> + pinctrl-0 = <&pmx_sdio_gpios>;
> + pinctrl-names = "default";
> + wp-gpios = <&gpio1 5 GPIO_ACTIVE_HIGH>;
> + cd-gpios = <&gpio1 6 GPIO_ACTIVE_HIGH>;
> + status = "okay";
> + };
> + };
> +};
> +
> +&nand {
> + chip-delay = <25>;
> + status = "okay";
> +
> + partition at 0 {
> + label = "uboot";
> + reg = <0x0 0x100000>;
> + };
> +
> + partition at 100000 {
> + label = "uImage";
> + reg = <0x100000 0x400000>;
> + };
> +
> + partition at 500000 {
> + label = "root";
> + reg = <0x500000 0x1fb00000>;
> + };
> +};
> +
> +&mdio {
> + status = "okay";
> +
> + ethphy0: ethernet-phy at 8 {
> + reg = <8>;
> + };
> +};
> +
> +ð0 {
> + status = "okay";
> + ethernet0-port at 0 {
> + phy-handle = <ðphy0>;
> + };
> +};
> diff --git a/arch/arm/mach-kirkwood/Kconfig b/arch/arm/mach-kirkwood/Kconfig
> index 3b860c4f55fe..7c4170399afb 100644
> --- a/arch/arm/mach-kirkwood/Kconfig
> +++ b/arch/arm/mach-kirkwood/Kconfig
> @@ -65,6 +65,9 @@ config TARGET_SBx81LIFKW
> config TARGET_SBx81LIFXCAT
> bool "Allied Telesis SBx81GP24/SBx81GT24"
>
> +config TARGET_DB_88F6281_BP
> + bool "Marvell DB-88F6281-BP"
> +
> endchoice
>
> config SYS_SOC
> @@ -89,5 +92,6 @@ source "board/Seagate/nas220/Kconfig"
> source "board/zyxel/nsa310s/Kconfig"
> source "board/alliedtelesis/SBx81LIFKW/Kconfig"
> source "board/alliedtelesis/SBx81LIFXCAT/Kconfig"
> +source "board/Marvell/db-88f6281-bp/Kconfig"
>
> endif
> diff --git a/board/Marvell/db-88f6281-bp/Kconfig b/board/Marvell/db-88f6281-bp/Kconfig
> new file mode 100644
> index 000000000000..38467399e688
> --- /dev/null
> +++ b/board/Marvell/db-88f6281-bp/Kconfig
> @@ -0,0 +1,12 @@
> +if TARGET_DB_88F6281_BP
> +
> +config SYS_BOARD
> + default "db-88f6281-bp"
> +
> +config SYS_VENDOR
> + default "Marvell"
> +
> +config SYS_CONFIG_NAME
> + default "db-88f6281-bp"
> +
> +endif
> diff --git a/board/Marvell/db-88f6281-bp/MAINTAINERS b/board/Marvell/db-88f6281-bp/MAINTAINERS
> new file mode 100644
> index 000000000000..f31d3be70bae
> --- /dev/null
> +++ b/board/Marvell/db-88f6281-bp/MAINTAINERS
> @@ -0,0 +1,10 @@
> +DB_88F6820_AMC BOARD
> +M: Chris Packham <judge.packham at gmail.com>
> +S: Maintained
> +F: arch/arm/dts/kirkwood-db-88f6281.dts
> +F: arch/arm/dts/kirkwood-db-88f6281-spi.dts
> +F: arch/arm/dts/kirkwood-db.dtsi
> +F: board/Marvell/db-88f6281-bp/
> +F: include/configs/db-88f6281-bp.h
> +F: configs/db-88f6281-bp_defconfig
> +F: configs/db-88f6281-bp-spi_defconfig
> diff --git a/board/Marvell/db-88f6281-bp/Makefile b/board/Marvell/db-88f6281-bp/Makefile
> new file mode 100644
> index 000000000000..38ff522ca1c8
> --- /dev/null
> +++ b/board/Marvell/db-88f6281-bp/Makefile
> @@ -0,0 +1,3 @@
> +# SPDX-License-Identifier: GPL-2.0+
> +
> +obj-y := db-88f6281-bp.o
> diff --git a/board/Marvell/db-88f6281-bp/db-88f6281-bp.c b/board/Marvell/db-88f6281-bp/db-88f6281-bp.c
> new file mode 100644
> index 000000000000..b68f2f3c895b
> --- /dev/null
> +++ b/board/Marvell/db-88f6281-bp/db-88f6281-bp.c
> @@ -0,0 +1,103 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +
> +#include <common.h>
> +#include <linux/io.h>
> +#include <miiphy.h>
> +#include <netdev.h>
> +#include <asm/arch/cpu.h>
> +#include <asm/arch/soc.h>
> +#include <asm/arch/mpp.h>
> +#include <asm/arch/gpio.h>
> +
> +#define DB_88F6281_OE_LOW ~(BIT(7))
> +#define DB_88F6281_OE_HIGH ~(BIT(15) | BIT(14) | BIT(13) | BIT(4))
> +#define DB_88F6281_OE_VAL_LOW BIT(7)
> +#define DB_88F6281_OE_VAL_HIGH 0
> +
> +DECLARE_GLOBAL_DATA_PTR;
> +
> +int board_early_init_f(void)
> +{
> + mvebu_config_gpio(DB_88F6281_OE_VAL_LOW,
> + DB_88F6281_OE_VAL_HIGH,
> + DB_88F6281_OE_LOW, DB_88F6281_OE_HIGH);
> +
> + /* Multi-Purpose Pins Functionality configuration */
> + static const u32 kwmpp_config[] = {
> +#ifdef CONFIG_CMD_NAND
> + MPP0_NF_IO2,
> + MPP1_NF_IO3,
> + MPP2_NF_IO4,
> + MPP3_NF_IO5,
> +#else
> + MPP0_SPI_SCn,
> + MPP1_SPI_MOSI,
> + MPP2_SPI_SCK,
> + MPP3_SPI_MISO,
> +#endif
> + MPP4_NF_IO6,
> + MPP5_NF_IO7,
> + MPP6_SYSRST_OUTn,
> + MPP7_GPO,
> + MPP8_TW_SDA,
> + MPP9_TW_SCK,
> + MPP10_UART0_TXD,
> + MPP11_UART0_RXD,
> + MPP12_SD_CLK,
> + MPP13_SD_CMD,
> + MPP14_SD_D0,
> + MPP15_SD_D1,
> + MPP16_SD_D2,
> + MPP17_SD_D3,
> + MPP18_NF_IO0,
> + MPP19_NF_IO1,
> + MPP20_SATA1_ACTn,
> + MPP21_SATA0_ACTn,
> + MPP22_GPIO,
> + MPP23_GPIO,
> + MPP24_GPIO,
> + MPP25_GPIO,
> + MPP26_GPIO,
> + MPP27_GPIO,
> + MPP28_GPIO,
> + MPP29_GPIO,
> + MPP30_GPIO,
> + MPP31_GPIO,
> + MPP32_GPIO,
> + MPP33_GPIO,
> + MPP34_GPIO,
> + MPP35_GPIO,
> + MPP36_GPIO,
> + MPP37_GPIO,
> + MPP38_GPIO,
> + MPP39_GPIO,
> + MPP40_GPIO,
> + MPP41_GPIO,
> + MPP42_GPIO,
> + MPP43_GPIO,
> + MPP44_GPIO,
> + MPP45_GPIO,
> + MPP46_GPIO,
> + MPP47_GPIO,
> + MPP48_GPIO,
> + MPP49_GPIO,
> + 0
> + };
> + kirkwood_mpp_conf(kwmpp_config, NULL);
> +
> + return 0;
> +}
> +
> +int board_init(void)
> +{
> + gd->bd->bi_boot_params = mvebu_sdram_bar(0) + 0x100;
> +
> + return 0;
> +}
> +
> +#ifdef CONFIG_RESET_PHY_R
> +/* automatically defined by kirkwood config.h */
> +void reset_phy(void)
> +{
> +}
> +#endif
> diff --git a/board/Marvell/db-88f6281-bp/kwbimage.cfg b/board/Marvell/db-88f6281-bp/kwbimage.cfg
> new file mode 100644
> index 000000000000..49a637078682
> --- /dev/null
> +++ b/board/Marvell/db-88f6281-bp/kwbimage.cfg
> @@ -0,0 +1,36 @@
> +# SPDX-License-Identifier: GPL-2.0+
> +
> +# Boot Media configurations
> +BOOT_FROM spi # Boot from SPI flash
How is booting from NAND handled? Is the boot device patched
dynamically in the resulting image?
> +
> +DATA 0xd00100e0 0x1b1b1b9b
> +DATA 0xd0020134 0xbbbbbbbb
> +DATA 0xd0020138 0x00bbbbbb
> +DATA 0xd0020154 0x00000200
> +DATA 0xd002014c 0x00001c00
> +DATA 0xd0020148 0x00000001
> +
> +DATA 0xd0001400 0x43000c30
> +DATA 0xd0001404 0x39543000
> +DATA 0xd0001408 0x22125451
> +DATA 0xd000140c 0x00000833
> +DATA 0xd0001410 0x000000cc
> +DATA 0xd0001414 0x00000000
> +DATA 0xd0001418 0x00000000
> +DATA 0xd000141c 0x00000c52
> +DATA 0xd0001420 0x00000044
> +DATA 0xd0001424 0x0000f1ff
> +DATA 0xd0001428 0x00085520
> +DATA 0xd000147c 0x00008552
> +DATA 0xd0001504 0x0ffffff1
> +DATA 0xd0001508 0x10000000
> +DATA 0xd000150c 0x0ffffff5
> +DATA 0xd0001514 0x00000000
> +DATA 0xd000151c 0x00000000
> +DATA 0xd0001494 0x84210000
> +DATA 0xd0001498 0x00000000
> +DATA 0xd000149c 0x0000f40f
> +DATA 0xd0001480 0x00000001
> +
> +# End of Header extension
> +DATA 0x0 0x0
> diff --git a/configs/db-88f6281-bp-nand_defconfig b/configs/db-88f6281-bp-nand_defconfig
> new file mode 100644
> index 000000000000..6360ef5e8280
> --- /dev/null
> +++ b/configs/db-88f6281-bp-nand_defconfig
> @@ -0,0 +1,53 @@
> +CONFIG_ARM=y
> +CONFIG_SYS_THUMB_BUILD=y
> +CONFIG_KIRKWOOD=y
> +CONFIG_SYS_TEXT_BASE=0x600000
> +CONFIG_TARGET_DB_88F6281_BP=y
> +CONFIG_IDENT_STRING="\nMarvell DB-88F6281-BP"
> +CONFIG_NR_DRAM_BANKS=2
> +# CONFIG_SYS_MALLOC_F is not set
> +CONFIG_BOOTDELAY=3
> +# CONFIG_DISPLAY_BOARDINFO is not set
> +CONFIG_HUSH_PARSER=y
> +CONFIG_CMD_BOOTZ=y
> +CONFIG_CMD_DM=y
> +# CONFIG_CMD_FLASH is not set
> +CONFIG_CMD_IDE=y
> +CONFIG_CMD_MMC=y
> +CONFIG_CMD_NAND=y
> +CONFIG_CMD_SF=y
> +CONFIG_CMD_USB=y
> +# CONFIG_CMD_SETEXPR is not set
> +CONFIG_CMD_DHCP=y
> +CONFIG_CMD_MII=y
> +CONFIG_CMD_PING=y
> +CONFIG_CMD_EXT2=y
> +CONFIG_CMD_EXT4=y
> +CONFIG_CMD_FAT=y
> +CONFIG_CMD_JFFS2=y
> +CONFIG_CMD_MTDPARTS=y
> +CONFIG_MTDIDS_DEFAULT="nand0=orion_nand"
> +CONFIG_MTDPARTS_DEFAULT="mtdparts=orion_nand:512K(uboot),512K(env),4M(kernel),-(rootfs)"
> +CONFIG_ISO_PARTITION=y
> +CONFIG_OF_CONTROL=y
> +CONFIG_DEFAULT_DEVICE_TREE="kirkwood-db-88f6281"
> +CONFIG_ENV_IS_IN_SPI_FLASH=y
> +CONFIG_NET_RANDOM_ETHADDR=y
> +CONFIG_DM=y
> +CONFIG_MVSATA_IDE=y
> +CONFIG_SPI_FLASH=y
> +CONFIG_SPI_FLASH_STMICRO=y
> +CONFIG_PHY_MARVELL=y
> +CONFIG_DM_ETH=y
> +CONFIG_MVGBE=y
> +CONFIG_MII=y
> +CONFIG_DM_RTC=y
> +CONFIG_RTC_MV=y
> +CONFIG_SYS_NS16550=y
> +CONFIG_SPI=y
> +CONFIG_KIRKWOOD_SPI=y
> +CONFIG_USB=y
> +CONFIG_USB_EHCI_HCD=y
> +CONFIG_USB_STORAGE=y
> +CONFIG_LZMA=y
> +CONFIG_LZO=y
> diff --git a/configs/db-88f6281-bp-spi_defconfig b/configs/db-88f6281-bp-spi_defconfig
> new file mode 100644
> index 000000000000..93e125af5b2d
> --- /dev/null
> +++ b/configs/db-88f6281-bp-spi_defconfig
> @@ -0,0 +1,54 @@
> +CONFIG_ARM=y
> +CONFIG_SYS_THUMB_BUILD=y
> +CONFIG_KIRKWOOD=y
> +CONFIG_SYS_TEXT_BASE=0x600000
> +CONFIG_TARGET_DB_88F6281_BP=y
> +CONFIG_IDENT_STRING="\nMarvell DB-88F6281-BP"
> +CONFIG_NR_DRAM_BANKS=2
> +# CONFIG_SYS_MALLOC_F is not set
> +CONFIG_BOOTDELAY=3
> +# CONFIG_DISPLAY_BOARDINFO is not set
> +CONFIG_HUSH_PARSER=y
> +CONFIG_CMD_BOOTZ=y
> +CONFIG_CMD_DM=y
> +# CONFIG_CMD_FLASH is not set
> +CONFIG_CMD_IDE=y
> +CONFIG_CMD_MMC=y
> +CONFIG_CMD_SF=y
> +CONFIG_CMD_USB=y
> +# CONFIG_CMD_SETEXPR is not set
> +CONFIG_CMD_DHCP=y
> +CONFIG_CMD_MII=y
> +CONFIG_CMD_PING=y
> +CONFIG_CMD_EXT2=y
> +CONFIG_CMD_EXT4=y
> +CONFIG_CMD_FAT=y
> +CONFIG_CMD_JFFS2=y
> +CONFIG_CMD_MTDPARTS=y
> +CONFIG_MTDIDS_DEFAULT="nand0=orion_nand"
> +CONFIG_MTDPARTS_DEFAULT="mtdparts=orion_nand:512K(uboot),512K(env),4M(kernel),-(rootfs)"
> +CONFIG_ISO_PARTITION=y
> +CONFIG_OF_CONTROL=y
> +CONFIG_DEFAULT_DEVICE_TREE="kirkwood-db-88f6281-spi"
> +CONFIG_ENV_IS_IN_SPI_FLASH=y
> +CONFIG_NET_RANDOM_ETHADDR=y
> +CONFIG_DM=y
> +CONFIG_MVSATA_IDE=y
> +CONFIG_DM_SPI_FLASH=y
> +CONFIG_SPI_FLASH=y
> +CONFIG_SPI_FLASH_STMICRO=y
> +CONFIG_PHY_MARVELL=y
> +CONFIG_DM_ETH=y
> +CONFIG_MVGBE=y
> +CONFIG_MII=y
> +CONFIG_DM_RTC=y
> +CONFIG_RTC_MV=y
> +CONFIG_SYS_NS16550=y
> +CONFIG_SPI=y
> +CONFIG_DM_SPI=y
> +CONFIG_KIRKWOOD_SPI=y
> +CONFIG_USB=y
> +CONFIG_USB_EHCI_HCD=y
> +CONFIG_USB_STORAGE=y
> +CONFIG_LZMA=y
> +CONFIG_LZO=y
> diff --git a/include/configs/db-88f6281-bp.h b/include/configs/db-88f6281-bp.h
> new file mode 100644
> index 000000000000..212983716e96
> --- /dev/null
> +++ b/include/configs/db-88f6281-bp.h
> @@ -0,0 +1,109 @@
> +/* SPDX-License-Identifier: GPL-2.0+ */
> +
> +#ifndef _CONFIG_DB_88F6281_BP_H
> +#define _CONFIG_DB_88F6281_BP_H
> +
> +/*
> + * High Level Configuration Options (easy to change)
> + */
> +#define CONFIG_FEROCEON_88FR131 1 /* CPU Core subversion */
> +#define CONFIG_KW88F6281 1 /* SOC Name */
> +#define CONFIG_SKIP_LOWLEVEL_INIT /* disable board lowlevel_init */
> +#define CONFIG_SYS_TCLK 166666667
> +#define CONFIG_SYS_KWD_CONFIG $(CONFIG_BOARDDIR)/kwbimage.cfg
> +#define CONFIG_BUILD_TARGET "u-boot.kwb"
> +
> +/* additions for new ARM relocation support */
> +#define CONFIG_SYS_SDRAM_BASE 0x00000000
> +
> +#define CONFIG_MD5 /* get_random_hex on krikwood needs MD5 support */
s/krikwood/kirkwood
> +#define CONFIG_KIRKWOOD_EGIGA_INIT /* Enable GbePort0/1 for kernel */
> +#define CONFIG_KIRKWOOD_PCIE_INIT /* Enable PCIE Port0 */
> +#define CONFIG_KIRKWOOD_RGMII_PAD_1V8 /* Set RGMII Pad voltage to 1.8V */
> +#define CONFIG_KIRKWOOD_GPIO 1
> +
> +/*
> + * NS16550 Configuration
> + */
> +#define CONFIG_SYS_NS16550_SERIAL
> +#define CONFIG_SYS_NS16550_REG_SIZE (-4)
> +#define CONFIG_SYS_NS16550_CLK CONFIG_SYS_TCLK
> +#define CONFIG_SYS_NS16550_COM1 KW_UART0_BASE
Can't you switch to using DM_SERIAL and use the DT for this
instead?
> +
> +#define CONFIG_SYS_MAX_NAND_DEVICE 1
> +/*
> + * Serial Port configuration
> + * The following definitions let you select what serial you want to use
> + * for your console driver.
> + */
> +
> +#define CONFIG_CONS_INDEX 1 /*Console on UART0 */
Nitpicking: Missing space after '*'.
> +
> +/*
> + * For booting Linux, the board info and command line data
> + * have to be in the first 8 MB of memory, since this is
> + * the maximum mapped by the Linux kernel during initialization.
> + */
> +#define CONFIG_CMDLINE_TAG 1 /* enable passing of ATAGs */
> +#define CONFIG_INITRD_TAG 1 /* enable INITRD tag */
> +#define CONFIG_SETUP_MEMORY_TAGS 1 /* enable memory tag */
Is this still needed for booting a modern kernel?
> +
> +/*
> + * Environment variables configurations
> + */
> +#define CONFIG_ENV_SPI_BUS 0
> +#define CONFIG_ENV_SPI_CS 0
> +#define CONFIG_ENV_SPI_MAX_HZ 20000000 /* 20Mhz */
> +#define CONFIG_ENV_SPI_MODE CONFIG_SF_DEFAULT_MODE
> +#define CONFIG_ENV_SECT_SIZE 0x40000 /* 256K */
> +#define CONFIG_ENV_SIZE 0x01000
> +#define CONFIG_ENV_OFFSET 0xC0000
> +
> +/*
> + * U-Boot bootcode configuration
> + */
> +
> +#define CONFIG_SYS_MONITOR_LEN (256 << 10) /* Reserve 256 kB for monitor */
> +#define CONFIG_SYS_MALLOC_LEN (4 << 20) /* Reserve 4.0 MB for malloc */
> +
> +/*
> + * For booting Linux, the board info and command line data
> + * have to be in the first 8 MB of memory, since this is
> + * the maximum mapped by the Linux kernel during initialization.
> + */
> +#define CONFIG_SYS_BOOTMAPSZ (8 << 20) /* Initial Mem map for Linux*/
> +
> +/* size in bytes reserved for initial data */
> +
> +#include <asm/arch/config.h>
> +/* There is no PHY directly connected so don't ask it for link status */
> +#undef CONFIG_SYS_FAULT_ECHO_LINK_DOWN
> +
> +/*
> + * Other required minimal configurations
> + */
> +#define CONFIG_ARCH_CPU_INIT /* call arch_cpu_init() */
> +#define CONFIG_SYS_MEMTEST_START 0x00400000 /* 4M */
> +#define CONFIG_SYS_MEMTEST_END 0x007fffff /*(_8M -1) */
Missing space after '*' and '-'. Please run checkpatch on this
patch.
> +#define CONFIG_SYS_RESET_ADDRESS 0xffff0000 /* Rst Vector Adr */
> +
> +/*
> + * SDIO/MMC Card Configuration
> + */
> +#ifdef CONFIG_CMD_MMC
> +#define CONFIG_MVEBU_MMC
> +#define CONFIG_SYS_MMC_BASE KW_SDIO_BASE
> +#endif /* CONFIG_CMD_MMC */
> +
> +/*
> + * SATA Driver configuration
> + */
> +#ifdef CONFIG_MVSATA_IDE
> +#define CONFIG_SYS_ATA_IDE0_OFFSET MV_SATA_PORT0_OFFSET
> +#endif /*CONFIG_MVSATA_IDE*/
> +
> +#define CONFIG_SYS_LOAD_ADDR 0x1000000 /* default location for tftp and bootm */
> +
> +#define CONFIG_SYS_DCACHE_OFF
Why is the d-cache disabled?
Thanks,
Stefan
More information about the U-Boot
mailing list