From: Lucas Stach <l.stach@pengutronix.de>
To: Sascha Hauer <s.hauer@pengutronix.de>,
Barebox List <barebox@lists.infradead.org>
Subject: Re: [PATCH 2/2] i.MX6ull evk support
Date: Thu, 28 Sep 2017 11:16:15 +0200 [thread overview]
Message-ID: <1506590175.2283.9.camel@pengutronix.de> (raw)
In-Reply-To: <20170927121657.16274-2-s.hauer@pengutronix.de>
Am Mittwoch, den 27.09.2017, 14:16 +0200 schrieb Sascha Hauer:
> The i.MX6ull-EVK is a evaluation board for the i.MX6ull from NXP.
> The upstream DTS is used, support should be fairly complete:
>
> - 2x fec ethernet
> - 1x USB Host
> - 1x USB OTG
> - 2x SD
>
> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
> ---
> arch/arm/boards/Makefile | 1 +
> arch/arm/boards/nxp-imx6ull-evk/Makefile | 2 +
> arch/arm/boards/nxp-imx6ull-evk/board.c | 61
> +++++++++++++++++
> .../flash-header-nxp-imx6ull-evk.imxcfg | 77
> ++++++++++++++++++++++
> arch/arm/boards/nxp-imx6ull-evk/lowlevel.c | 75
> +++++++++++++++++++++
> arch/arm/dts/Makefile | 1 +
> arch/arm/dts/imx6ull-14x14-evk.dts | 29 ++++++++
> arch/arm/mach-imx/Kconfig | 8 +++
> arch/arm/mach-imx/include/mach/imx6-regs.h | 1 +
> images/Makefile.imx | 5 ++
> 10 files changed, 260 insertions(+)
> create mode 100644 arch/arm/boards/nxp-imx6ull-evk/Makefile
> create mode 100644 arch/arm/boards/nxp-imx6ull-evk/board.c
> create mode 100644 arch/arm/boards/nxp-imx6ull-evk/flash-header-nxp-
> imx6ull-evk.imxcfg
> create mode 100644 arch/arm/boards/nxp-imx6ull-evk/lowlevel.c
> create mode 100644 arch/arm/dts/imx6ull-14x14-evk.dts
>
> diff --git a/arch/arm/boards/Makefile b/arch/arm/boards/Makefile
> index d6011adb28..456e6fea4b 100644
> --- a/arch/arm/boards/Makefile
> +++ b/arch/arm/boards/Makefile
> @@ -78,6 +78,7 @@ obj-$(CONFIG_MACH_NOMADIK_8815NHK) +
> = nhk8815/
> obj-$(CONFIG_MACH_NVIDIA_BEAVER) += nvidia-beaver/
> obj-$(CONFIG_MACH_NVIDIA_JETSON) += nvidia-jetson-
> tk1/
> obj-$(CONFIG_MACH_NXDB500) += netx/
> +obj-$(CONFIG_MACH_NXP_IMX6ULL_EVK) += nxp-imx6ull-
> evk/
> obj-$(CONFIG_MACH_OMAP343xSDP) +=
> omap343xdsp/
> obj-$(CONFIG_MACH_OMAP3EVM) += omap3evm/
> obj-$(CONFIG_MACH_PANDA) += panda/
> diff --git a/arch/arm/boards/nxp-imx6ull-evk/Makefile
> b/arch/arm/boards/nxp-imx6ull-evk/Makefile
> new file mode 100644
> index 0000000000..01c7a259e9
> --- /dev/null
> +++ b/arch/arm/boards/nxp-imx6ull-evk/Makefile
> @@ -0,0 +1,2 @@
> +obj-y += board.o
> +lwl-y += lowlevel.o
> diff --git a/arch/arm/boards/nxp-imx6ull-evk/board.c
> b/arch/arm/boards/nxp-imx6ull-evk/board.c
> new file mode 100644
> index 0000000000..77e76ce024
> --- /dev/null
> +++ b/arch/arm/boards/nxp-imx6ull-evk/board.c
> @@ -0,0 +1,61 @@
> +/*
> + * Copyright (C) 2017 Michael Grzeschik, Pengutronix
As long as you didn't get yourself a new name alias, I don't think this
is correct.
> + *
> + * 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.
> + *
> + */
> +
> +#include <asm/armlinux.h>
> +#include <asm/io.h>
> +#include <bootsource.h>
> +#include <common.h>
> +#include <environment.h>
> +#include <envfs.h>
> +#include <gpio.h>
> +#include <init.h>
> +#include <mach/generic.h>
> +#include <mach/imx6-regs.h>
> +#include <mach/imx6.h>
> +#include <mach/bbu.h>
> +#include <linux/sizes.h>
> +#include <linux/phy.h>
> +#include <mfd/imx6q-iomuxc-gpr.h>
A lot of those headers are probably not needed for the relatively
simple initcalls below.
> +#include <linux/micrel_phy.h>
> +
> +static int ksz8081_phy_fixup(struct phy_device *dev)
> +{
> + phy_write(dev, 0x1f, 0x8190);
> + phy_write(dev, 0x16, 0x202);
> +
> + return 0;
> +}
> +
> +static int nxp_imx6ull_evk_init(void)
> +{
> + if (!of_machine_is_compatible("fsl,imx6ull-14x14-evk"))
> + return 0;
> +
> + phy_register_fixup_for_uid(PHY_ID_KSZ8081,
> MICREL_PHY_ID_MASK,
> + ksz8081_phy_fixup);
> +
> + imx6_bbu_internal_mmc_register_handler("mmc",
> "/dev/mmc1.barebox",
> + BBU_HANDLER_FLAG_DEFAULT);
> +
> + barebox_set_hostname("imx6ull-evk");
> +
> + return 0;
> +}
> +device_initcall(nxp_imx6ull_evk_init);
> diff --git a/arch/arm/boards/nxp-imx6ull-evk/flash-header-nxp-
> imx6ull-evk.imxcfg b/arch/arm/boards/nxp-imx6ull-evk/flash-header-
> nxp-imx6ull-evk.imxcfg
> new file mode 100644
> index 0000000000..53270d0dc3
> --- /dev/null
> +++ b/arch/arm/boards/nxp-imx6ull-evk/flash-header-nxp-imx6ull-
> evk.imxcfg
If possible, please mention the source and commit where this is taken
from.
> @@ -0,0 +1,77 @@
> +/*
> + * Copyright (C) 2016 Freescale Semiconductor, Inc.
> + *
> + * SPDX-License-Identifier: GPL-2.0+
> + *
> + * Refer doc/README.imximage for more details about how-to configure
> + * and create imximage boot image
> + *
> + * The syntax is taken as close as possible with the kwbimage
> + */
> +
> +loadaddr 0x80000000
> +soc imx6
> +dcdofs 0x400
> +
> +/* Enable all clocks */
> +wm 32 0x020c4068 0xffffffff
> +wm 32 0x020c406c 0xffffffff
> +wm 32 0x020c4070 0xffffffff
> +wm 32 0x020c4074 0xffffffff
> +wm 32 0x020c4078 0xffffffff
> +wm 32 0x020c407c 0xffffffff
> +wm 32 0x020c4080 0xffffffff
> +
> +wm 32 0x020e04b4 0x000c0000
> +wm 32 0x020e04ac 0x00000000
> +wm 32 0x020e027c 0x00000030
> +wm 32 0x020e0250 0x00000030
> +wm 32 0x020e024c 0x00000030
> +wm 32 0x020e0490 0x00000030
> +wm 32 0x020e0288 0x000c0030
> +wm 32 0x020e0270 0x00000000
> +wm 32 0x020e0260 0x00000030
> +wm 32 0x020e0264 0x00000030
> +wm 32 0x020e04a0 0x00000030
> +wm 32 0x020e0494 0x00020000
> +wm 32 0x020e0280 0x00000030
> +wm 32 0x020e0284 0x00000030
> +wm 32 0x020e04b0 0x00020000
> +wm 32 0x020e0498 0x00000030
> +wm 32 0x020e04a4 0x00000030
> +wm 32 0x020e0244 0x00000030
> +wm 32 0x020e0248 0x00000030
> +wm 32 0x021b001c 0x00008000
> +wm 32 0x021b0800 0xa1390003
> +wm 32 0x021b080c 0x00000004
> +wm 32 0x021b083c 0x41640158
> +wm 32 0x021b0848 0x40403237
> +wm 32 0x021b0850 0x40403c33
> +wm 32 0x021b081c 0x33333333
> +wm 32 0x021b0820 0x33333333
> +wm 32 0x021b082c 0xf3333333
> +wm 32 0x021b0830 0xf3333333
> +wm 32 0x021b08c0 0x00944009
> +wm 32 0x021b08b8 0x00000800
> +wm 32 0x021b0004 0x0002002d
> +wm 32 0x021b0008 0x1b333030
> +wm 32 0x021b000c 0x676b52f3
> +wm 32 0x021b0010 0xb66d0b63
> +wm 32 0x021b0014 0x01ff00db
> +wm 32 0x021b0018 0x00201740
> +wm 32 0x021b001c 0x00008000
> +wm 32 0x021b002c 0x000026d2
> +wm 32 0x021b0030 0x006b1023
> +wm 32 0x021b0040 0x0000004f
> +wm 32 0x021b0000 0x84180000
> +wm 32 0x021b0890 0x00400000
> +wm 32 0x021b001c 0x02008032
> +wm 32 0x021b001c 0x00008033
> +wm 32 0x021b001c 0x00048031
> +wm 32 0x021b001c 0x15208030
> +wm 32 0x021b001c 0x04008040
> +wm 32 0x021b0020 0x00000800
> +wm 32 0x021b0818 0x00000227
> +wm 32 0x021b0004 0x0002552d
> +wm 32 0x021b0404 0x00011006
> +wm 32 0x021b001c 0x00000000
> diff --git a/arch/arm/boards/nxp-imx6ull-evk/lowlevel.c
> b/arch/arm/boards/nxp-imx6ull-evk/lowlevel.c
> new file mode 100644
> index 0000000000..bb0906560c
> --- /dev/null
> +++ b/arch/arm/boards/nxp-imx6ull-evk/lowlevel.c
> @@ -0,0 +1,75 @@
> +/*
> + * 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; version 2.
> + *
> + * 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.
> + */
> +
> +#include <common.h>
> +#include <linux/sizes.h>
> +#include <mach/generic.h>
> +#include <asm/barebox-arm-head.h>
> +#include <asm/barebox-arm.h>
> +#include <mach/imx6-regs.h>
> +#include <io.h>
> +#include <debug_ll.h>
> +#include <mach/esdctl.h>
> +#include <asm/cache.h>
> +#include <asm/sections.h>
> +#include <image-metadata.h>
> +
> +static inline void setup_uart(void)
> +{
> + void __iomem *iomuxbase = IOMEM(MX6_IOMUXC_BASE_ADDR);
> + void __iomem *uart = IOMEM(MX6_UART1_BASE_ADDR);
> +
> + imx6_ungate_all_peripherals();
> +
> + writel(0x0, iomuxbase + 0x84);
> + writel(0x1b0b1, iomuxbase + 0x0310);
> +
> + writel(0x0, iomuxbase + 0x88);
> + writel(0x1b0b0, iomuxbase + 0x0314);
> +
> + writel(0x3, iomuxbase + 0x624);
> +
> + imx6_uart_setup(uart);
> +
> + pbl_set_putc(imx_uart_putc, uart);
> +
> + putc_ll('>');
> +}
> +
> +extern char __dtb_imx6ull_14x14_evk_start[];
> +
> +static noinline void nxp_imx6_ull(void)
> +{
> + imx6ul_barebox_entry(__dtb_imx6ull_14x14_evk_start);
> +}
> +
> +ENTRY_FUNCTION(start_nxp_imx6ull_evk, r0, r1, r2)
> +{
> +
> + imx6ull_cpu_lowlevel_init();
> +
> + arm_setup_stack(0x00910000 - 8);
> +
> + arm_early_mmu_cache_invalidate();
> +
> + relocate_to_current_adr();
> + setup_c();
> + barrier();
Function calls are already acting as a compiler barrier, so this is
pure cargo-cult.
> +
> + setup_uart();
> +
> + /* disable all watchdog powerdown counters */
> + writew(0x0, IOMEM(MX6_WDOG1_BASE_ADDR + 0x8));
> + writew(0x0, IOMEM(MX6_WDOG2_BASE_ADDR + 0x8));
> + writew(0x0, IOMEM(MX6ULL_WDOG3_BASE_ADDR + 0x8));
> +
> + nxp_imx6_ull();
> +}
> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
> index cf9d8ea940..33b2fff0f7 100644
> --- a/arch/arm/dts/Makefile
> +++ b/arch/arm/dts/Makefile
> @@ -80,6 +80,7 @@ pbl-dtb-$(CONFIG_MACH_SOLIDRUN_MICROSOM) += imx6dl-
> hummingboard.dtb.o imx6q-humm
> imx6q-h100.dtb.o
> pbl-dtb-$(CONFIG_MACH_TECHNEXION_WANDBOARD) += imx6q-wandboard.dtb.o
> imx6dl-wandboard.dtb.o
> pbl-dtb-$(CONFIG_MACH_TECHNEXION_PICO_HOBBIT) += imx6ul-pico-
> hobbit.dtb.o
> +pbl-dtb-$(CONFIG_MACH_NXP_IMX6ULL_EVK) += imx6ull-14x14-evk.dtb.o
> pbl-dtb-$(CONFIG_MACH_TORADEX_COLIBRI_T20) += tegra20-colibri-
> iris.dtb.o
> pbl-dtb-$(CONFIG_MACH_TOSHIBA_AC100) += tegra20-paz00.dtb.o
> pbl-dtb-$(CONFIG_MACH_TQMA53) += imx53-mba53.dtb.o
> diff --git a/arch/arm/dts/imx6ull-14x14-evk.dts
> b/arch/arm/dts/imx6ull-14x14-evk.dts
> new file mode 100644
> index 0000000000..9afe6402a2
> --- /dev/null
> +++ b/arch/arm/dts/imx6ull-14x14-evk.dts
> @@ -0,0 +1,29 @@
> +#include <arm/imx6ull-14x14-evk.dts>
> +
> +/{
> + chosen {
> + environment@0 {
> + compatible = "barebox,environment";
> + device-path = &environment_usdhc2;
> + };
> + };
> +};
> +
> +&usdhc2 {
> + #address-cells = <1>;
> + #size-cells = <1>;
> +
> + partition@0 {
> + label = "barebox";
> + reg = <0x0 0xc0000>;
> + };
> +
> + environment_usdhc2: partition@c0000 {
> + label = "barebox-environment";
> + reg = <0xc0000 0x40000>;
> + };
> +};
> +
> +&ocotp {
> + barebox,provide-mac-address = <&fec1 0x620 &fec2 0x632>;
> +};
> diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
> index 92440e3a75..dab19a33ec 100644
> --- a/arch/arm/mach-imx/Kconfig
> +++ b/arch/arm/mach-imx/Kconfig
> @@ -162,6 +162,10 @@ config ARCH_IMX6UL
> bool
> select ARCH_IMX6
>
> +config ARCH_IMX6ULL
> + bool
> + select ARCH_IMX6
> +
Do we really need this? Seems the MX6ULL is just a stripped down
version of the MX6UL.
> config ARCH_IMX7
> bool
> select CPU_V7
> @@ -414,6 +418,10 @@ config MACH_FREESCALE_MX7_SABRESD
>
> https://goo.gl/6EKGdk
>
> +config MACH_NXP_IMX6ULL_EVK
> + bool "NXP i.MX6ull EVK Board"
> + select ARCH_IMX6ULL
> +
> endif
>
> # ----------------------------------------------------------
> diff --git a/arch/arm/mach-imx/include/mach/imx6-regs.h
> b/arch/arm/mach-imx/include/mach/imx6-regs.h
> index e661c4ed12..ac2aa2109f 100644
> --- a/arch/arm/mach-imx/include/mach/imx6-regs.h
> +++ b/arch/arm/mach-imx/include/mach/imx6-regs.h
> @@ -107,6 +107,7 @@
> #define MX6_MIPI_CSI2_BASE_ADDR (MX6_AIPS2_OFF_BASE_ADDR +
> 0x5C000)
> #define MX6_MIPI_DSI_BASE_ADDR (MX6_AIPS2_OFF_BASE_ADDR +
> 0x60000)
> #define MX6_VDOA_BASE_ADDR (MX6_AIPS2_OFF_BASE_ADDR +
> 0x64000)
> +#define MX6ULL_WDOG3_BASE_ADDR (MX6_AIPS2_OFF_BASE_ADDR +
> 0x64000)
> #define MX6_UART2_BASE_ADDR (MX6_AIPS2_OFF_BASE_ADDR +
> 0x68000)
> #define MX6_UART3_BASE_ADDR (MX6_AIPS2_OFF_BASE_ADDR +
> 0x6C000)
> #define MX6_UART4_BASE_ADDR (MX6_AIPS2_OFF_BASE_ADDR +
> 0x70000)
> diff --git a/images/Makefile.imx b/images/Makefile.imx
> index e9176022bf..76e91ebd7d 100644
> --- a/images/Makefile.imx
> +++ b/images/Makefile.imx
> @@ -325,6 +325,11 @@ CFG_start_imx6ul_pico_hobbit_512mb.pblx.imximg =
> $(board)/technexion-pico-hobbit
> FILE_barebox-imx6ul-pico-hobbit-512mb.img =
> start_imx6ul_pico_hobbit_512mb.pblx.imximg
> image-$(CONFIG_MACH_TECHNEXION_PICO_HOBBIT) += barebox-imx6ul-pico-
> hobbit-512mb.img
>
> +pblx-$(CONFIG_MACH_NXP_IMX6ULL_EVK) += start_nxp_imx6ull_evk
> +CFG_start_nxp_imx6ull_evk.pblx.imximg = $(board)/nxp-imx6ull-
> evk/flash-header-nxp-imx6ull-evk.imxcfg
> +FILE_barebox-nxp-imx6ull-evk.img = start_nxp_imx6ull_evk.pblx.imximg
> +image-$(CONFIG_MACH_NXP_IMX6ULL_EVK) += barebox-nxp-imx6ull-evk.img
> +
> pblx-$(CONFIG_MACH_NITROGEN6) += start_imx6q_nitrogen6x_1g
> CFG_start_imx6q_nitrogen6x_1g.pblx.imximg =
> $(board)/boundarydevices-nitrogen6/flash-header-nitrogen6q-1g.imxcfg
> FILE_barebox-boundarydevices-imx6q-nitrogen6x-1g.img =
> start_imx6q_nitrogen6x_1g.pblx.imximg
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2017-09-28 9:16 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-27 12:16 [PATCH 1/2] ARM: i.MX6ull: Add SoC specific lowlevel_init function Sascha Hauer
2017-09-27 12:16 ` [PATCH 2/2] i.MX6ull evk support Sascha Hauer
2017-09-28 9:16 ` Lucas Stach [this message]
2017-09-28 9:26 ` Sascha Hauer
2017-09-28 9:33 ` Lucas Stach
2017-09-28 10:13 ` Sascha Hauer
2017-10-10 14:04 ` Philipp Zabel
2017-09-28 9:09 ` [PATCH 1/2] ARM: i.MX6ull: Add SoC specific lowlevel_init function Lucas Stach
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1506590175.2283.9.camel@pengutronix.de \
--to=l.stach@pengutronix.de \
--cc=barebox@lists.infradead.org \
--cc=s.hauer@pengutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox