[U-Boot] [PATCH v5] Marvell MV88F6281GTW_GE Board support

Ben Warren biggerbadderben at gmail.com
Sun May 3 05:05:56 CEST 2009


Hi Prafulla,

On Sat, May 2, 2009 at 9:36 PM, Prafulla Wadaskar <prafulla at marvell.com>wrote:

> From: prafulla_wadaskar <prafulla at marvell.com>
>
> This is Marvell's 88F6281_A0 based custom board developed
> for wireless access point product
>
> This patch is tested for-
> 1. Boot from DRAM/SPI flash/NFS
> 2. File transfer using tftp and loadb
> 3. SPI flash read/write/erase
> 4. Booting Linux kernel and RFS from SPI flash
>
> Reviewed-by: Ronen Shitrit <rshitrit at marvell.com>
> Signed-off-by: Prafulla Wadaskar <prafulla at marvell.com>
> ---
> Change log
> v2: updated as per first review comments
> debug_prints updated to debug
>
> v3: updaed as per review comments for v2
> added mv88f6281gtw_ge.h file
> removed BITxx macros
>
> v4: updated as per review comments for v3
> arch_misc_init support is added and used from kirkwood
>
> v5: updated as per review comments for v4
> CONFIG_MACH_MV88F6281GTW_GE added
> more comments added and
> serial configuration removed from mv88f6281gtw_ge.c
>
>  MAKEALL                                         |    1 +
>  Makefile                                        |    3 +
>  board/Marvell/mv88f6281gtw_ge/Makefile          |   51 +++++++
>  board/Marvell/mv88f6281gtw_ge/config.mk         |   25 +++
>  board/Marvell/mv88f6281gtw_ge/mv88f6281gtw_ge.c |   89 +++++++++++
>  board/Marvell/mv88f6281gtw_ge/mv88f6281gtw_ge.h |   51 +++++++
>  board/Marvell/mv88f6281gtw_ge/u-boot.lds        |   53 +++++++
>  include/configs/mv88f6281gtw_ge.h               |  180
> +++++++++++++++++++++++
>  8 files changed, 453 insertions(+), 0 deletions(-)
>  create mode 100644 board/Marvell/mv88f6281gtw_ge/Makefile
>  create mode 100644 board/Marvell/mv88f6281gtw_ge/config.mk
>  create mode 100644 board/Marvell/mv88f6281gtw_ge/mv88f6281gtw_ge.c
>  create mode 100644 board/Marvell/mv88f6281gtw_ge/mv88f6281gtw_ge.h
>  create mode 100644 board/Marvell/mv88f6281gtw_ge/u-boot.lds
>  create mode 100644 include/configs/mv88f6281gtw_ge.h
>
> diff --git a/MAKEALL b/MAKEALL
> index f13c81a..a7545c9 100755
> --- a/MAKEALL
> +++ b/MAKEALL
> @@ -504,6 +504,7 @@ LIST_ARM9="                 \
>        cp946es                 \
>        cp966                   \
>        lpd7a400                \
> +       mv88f6281gtw_ge         \
>        mx1ads                  \
>        mx1fs2                  \
>        netstar                 \
> diff --git a/Makefile b/Makefile
> index 1b43832..fdc78bf 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -2795,6 +2795,9 @@ lpd7a400_config \
>  lpd7a404_config:       unconfig
>        @$(MKCONFIG) $(@:_config=) arm lh7a40x lpd7a40x
>
> +mv88f6281gtw_ge_config: unconfig
> +       @$(MKCONFIG) $(@:_config=) arm arm926ejs $(@:_config=) Marvell
> kirkwood
> +
>  mx1ads_config  :       unconfig
>        @$(MKCONFIG) $(@:_config=) arm arm920t mx1ads NULL imx
>
> diff --git a/board/Marvell/mv88f6281gtw_ge/Makefile
> b/board/Marvell/mv88f6281gtw_ge/Makefile
> new file mode 100644
> index 0000000..8c49a3e
> --- /dev/null
> +++ b/board/Marvell/mv88f6281gtw_ge/Makefile
> @@ -0,0 +1,51 @@
> +#
> +# (C) Copyright 2009
> +# Marvell Semiconductor <www.marvell.com>
> +# Prafulla Wadaskar <prafulla at marvell.com>
> +#
> +# See file CREDITS for list of people who contributed to this
> +# project.
> +#
> +# This program is free software; you can redistribute it and/or
> +# modify it under the terms of the GNU General Public License as
> +# published by the Free Software Foundation; either version 2 of
> +# the License, or (at your option) any later version.
> +#
> +# This program is distributed in the hope that it will be useful,
> +# but WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> +# GNU General Public License for more details.
> +#
> +# You should have received a copy of the GNU General Public License
> +# along with this program; if not, write to the Free Software
> +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> +# MA 02110-1301 USA
> +#
> +
> +include $(TOPDIR)/config.mk
> +
> +LIB    = $(obj)lib$(BOARD).a
> +
> +COBJS  := mv88f6281gtw_ge.o
> +
> +SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)
> +OBJS   := $(addprefix $(obj),$(COBJS))
> +SOBJS  := $(addprefix $(obj),$(SOBJS))
> +
> +$(LIB):        $(obj).depend $(OBJS) $(SOBJS)
> +       $(AR) $(ARFLAGS) $@ $(OBJS) $(SOBJS)
> +
> +clean:
> +       rm -f $(SOBJS) $(OBJS)
> +
> +distclean:     clean
> +       rm -f $(LIB) core *.bak .depend
> +
> +#########################################################################
> +
> +# defines $(obj).depend target
> +include $(SRCTREE)/rules.mk
> +
> +sinclude $(obj).depend
> +
> +#########################################################################
> diff --git a/board/Marvell/mv88f6281gtw_ge/config.mkb/board/Marvell/mv88f6281gtw_ge/
> config.mk
> new file mode 100644
> index 0000000..fb29a1b
> --- /dev/null
> +++ b/board/Marvell/mv88f6281gtw_ge/config.mk
> @@ -0,0 +1,25 @@
> +#
> +# (C) Copyright 2009
> +# Marvell Semiconductor <www.marvell.com>
> +# Prafulla Wadaskar <prafulla at marvell.com>
> +#
> +# See file CREDITS for list of people who contributed to this
> +# project.
> +#
> +# This program is free software; you can redistribute it and/or
> +# modify it under the terms of the GNU General Public License as
> +# published by the Free Software Foundation; either version 2 of
> +# the License, or (at your option) any later version.
> +#
> +# This program is distributed in the hope that it will be useful,
> +# but WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> +# GNU General Public License for more details.
> +#
> +# You should have received a copy of the GNU General Public License
> +# along with this program; if not, write to the Free Software
> +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> +# MA 02110-1301 USA
> +#
> +
> +TEXT_BASE = 0x00600000
> diff --git a/board/Marvell/mv88f6281gtw_ge/mv88f6281gtw_ge.c
> b/board/Marvell/mv88f6281gtw_ge/mv88f6281gtw_ge.c
> new file mode 100644
> index 0000000..7dc097b
> --- /dev/null
> +++ b/board/Marvell/mv88f6281gtw_ge/mv88f6281gtw_ge.c
> @@ -0,0 +1,89 @@
> +/*
> + * (C) Copyright 2009
> + * Marvell Semiconductor <www.marvell.com>
> + * Prafulla Wadaskar <prafulla at marvell.com>
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License as
> + * published by the Free Software Foundation; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.         See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> + * MA 02110-1301 USA
> + */
> +
> +#include <common.h>
> +#include <../drivers/net/phy/mv88e61xx.h>
>
If this header is meant to be public it should be in "include".  Sorry I
didn't catch that earlier.
<snip>

>  +/*
> + * Ethernet Driver configuration
> + */
> +#ifdef CONFIG_CMD_NET
> +#define CONFIG_NETCONSOLE      /* include NetConsole support   */
> +#define CONFIG_NET_MULTI       /* specify more that one ports available */
> +#define        CONFIG_MII              /* expose smi ove miiphy interface
> */
> +#define CONFIG_KIRKWOOD_EGIGA  /* Enable kirkwood Gbe Controller Driver */
>
This EGIGA driver hasn't been accepted yet.  In fact, I think you've only
sent one spin.  It needs to be in before this board...

>
> +#define CONFIG_SYS_FAULT_ECHO_LINK_DOWN        /* detect link using phy */
> +#define CONFIG_KIRKWOOD_EGIGA_PORTS    {1,0}   /* enable port 0 only */
> +#define CONFIG_ENV_OVERWRITE   /* ethaddr can be reprogrammed */
> +#endif /* CONFIG_CMD_NET */
> +
> +/*
> + * Marvell 88Exxxx Switch configurations
> + */
> +#define CONFIG_RESET_PHY_R     /* use reset_phy() to init phy/swtich */
> +#define CONFIG_MV88E61XX_SWITCH        /* Enable mv88e61xx switch driver
> */
> +
> +#endif /* _CONFIG_MV88F6281GTW_GE_H */
> --
> 1.5.3.4
>

regards,
Ben


More information about the U-Boot mailing list