[PATCH 4/8] arm: kirkwood: Pogoplug-V4 : Add board include configs file

[Tony Dinh]

Hi Pali,

On Sat, Dec 18, 2021 at 5:12 AM Pali Rohár <pali at kernel.org> wrote:
>
> On Friday 17 December 2021 20:23:30 Tony Dinh wrote:
> > Add include configs file for Pogoplug V4 board
> >
> > Signed-off-by: Tony Dinh <mibodhi at gmail.com>
> > ---
> >
> >  include/configs/pogo_v4.h | 94 +++++++++++++++++++++++++++++++++++++++
> >  1 file changed, 94 insertions(+)
> >  create mode 100644 include/configs/pogo_v4.h
> >
> > diff --git a/include/configs/pogo_v4.h b/include/configs/pogo_v4.h
> > new file mode 100644
> > index 0000000000..b449986e8c
> > --- /dev/null
> > +++ b/include/configs/pogo_v4.h
> > @@ -0,0 +1,94 @@
> > +/* SPDX-License-Identifier: GPL-2.0+ */
> > +/*
> > + * Copyright (C) 2014-2021 Tony Dinh <mibodhi at gmail.com>
> > + *
> > + * Based on
> > + * Copyright (C) 2012
> > + * David Purdy <david.c.purdy at gmail.com>
> > + *
> > + * Based on Kirkwood support:
> > + * (C) Copyright 2009
> > + * Marvell Semiconductor <www.marvell.com>
> > + * Written-by: Prafulla Wadaskar <prafulla at marvell.com>
> > + */
> > +
> > +#ifndef _CONFIG_POGO_V4_H
> > +#define _CONFIG_POGO_V4_H
> > +
> > +/*
> > + * Machine type definition and ID
> > + */
> > +#define MACH_TYPE_POGO_V4            3960
> > +#define CONFIG_MACH_TYPE             MACH_TYPE_POGO_V4
> > +
> > +/*
> > + * High Level Configuration Options (easy to change)
> > + */
> > +#define CONFIG_FEROCEON_88FR131              /* #define CPU Core subversion */
> > +#define CONFIG_KW88F6192             /* SOC Name */
> > +
> > +/*
> > + * Commands configuration
> > + */
> > +#define CONFIG_SYS_NO_FLASH          /* Declare no flash (NOR/SPI) */
> > +#define CONFIG_SYS_PROMPT_HUSH_PS2 "> "
> > +
> > +/*
> > + * mv-common.h should be defined after CMD configs since it used them
> > + * to enable certain macros
> > + */
> > +#include "mv-common.h"
> > +
> > +/*
> > + * Default environment variables
> > + */
> > +#define CONFIG_BOOTCOMMAND \
> > +     "setenv bootargs $(bootargs_console); " \
> > +     "run bootcmd_usb; " \
> > +     "bootm 0x00800000 0x01100000 0x2c00000"
> > +
> > +#define CONFIG_EXTRA_ENV_SETTINGS \
> > +     "dtb_file=/boot/dts/" CONFIG_DEFAULT_DEVICE_TREE ".dtb\0" \
> > +     "mtdparts=" CONFIG_MTDPARTS_DEFAULT "\0"\
> > +     "mtdids=nand0=orion_nand\0"\
> > +     "bootargs_console=console=ttyS0,115200\0" \
> > +     "bootcmd_usb=usb start; load usb 0:1 0x00800000 /boot/uImage; " \
> > +     "load usb 0:1 0x01100000 /boot/uInitrd; " \
> > +     "load usb 0:1 0x2c00000 $dtb_file\0"
>
> Have you tried to use new standard distroboot? It generates standard
> boot commands, so there is no need to specify them manually with
> hardcoded addresses.

This was the old style that we've been using for many years. Since
Simon has introduced the approach of using board envs text file, I'd
prefer to keep the default envs the same for now, because it will be
moved out to text file and revised anyway.

Thanks,
Tony
>
> > +
> > +/*
> > + * Ethernet Driver configuration
> > + */
> > +#ifdef CONFIG_CMD_NET
> > +#define CONFIG_FEATURE_COMMAND_EDITING       /* for netconsole */
> > +#define CONFIG_MVGBE_PORTS   {1, 0}  /* enable port 0 only */
> > +#define CONFIG_PHY_BASE_ADR  0
> > +#endif /* CONFIG_CMD_NET */
> > +
> > +/*
> > + * File system
> > + */
> > +#ifdef CONFIG_CMD_JFFS2
> > +#define CONFIG_JFFS2_NAND
> > +#define CONFIG_JFFS2_LZO
> > +#endif /* CONFIG_CMD_JFFS2 */
> > +
> > +/*
> > + *  SATA Driver configuration
> > + */
> > +#ifdef CONFIG_SATA
> > +#define CONFIG_SYS_SATA_MAX_DEVICE   1
> > +#endif /* CONFIG_SATA */
> > +
> > +/*
> > + * Support large disk for SATA and USB
> > + */
> > +#define CONFIG_SYS_64BIT_LBA
> > +#define CONFIG_LBA48
> > +
> > +/*
> > + * Kirkwood GPIO
> > + */
> > +#define CONFIG_KIRKWOOD_GPIO
> > +
> > +#endif /* _CONFIG_POGO_V4_H */
> > --
> > 2.20.1
> >