[RFC PATCH 11/11] arm: boards: sunxi: Add pine64 board
Ahmad Fatoum
a.fatoum at pengutronix.de
Thu May 18 12:44:28 PDT 2023
On 11.05.23 01:37, Jules Maselbas wrote:
> ---
> arch/arm/boards/Makefile | 1 +
> arch/arm/boards/pine64-pine64/Makefile | 1 +
> arch/arm/boards/pine64-pine64/lowlevel.c | 148 +++++++++++++++++++++++
> arch/arm/dts/Makefile | 1 +
> arch/arm/dts/sun50i-a64-pine64-plus.dts | 18 +++
> arch/arm/mach-sunxi/Kconfig | 5 +
> images/Makefile.sunxi | 10 ++
> 7 files changed, 184 insertions(+)
> create mode 100644 arch/arm/boards/pine64-pine64/Makefile
> create mode 100644 arch/arm/boards/pine64-pine64/lowlevel.c
> create mode 100644 arch/arm/dts/sun50i-a64-pine64-plus.dts
>
> diff --git a/arch/arm/boards/Makefile b/arch/arm/boards/Makefile
> index 083ec2dce6..1fa1f962b3 100644
> --- a/arch/arm/boards/Makefile
> +++ b/arch/arm/boards/Makefile
> @@ -97,6 +97,7 @@ obj-$(CONFIG_MACH_PHYTEC_PHYCORE_IMX7) += phytec-phycore-imx7/
> obj-$(CONFIG_MACH_PHYTEC_PHYCORE_STM32MP1) += phytec-phycore-stm32mp1/
> obj-$(CONFIG_MACH_PHYTEC_SOM_IMX8MQ) += phytec-som-imx8mq/
> obj-$(CONFIG_MACH_PINE64_PINEPHONE) += pine64-pinephone/
> +obj-$(CONFIG_MACH_PINE64_PINE64) += pine64-pine64/
That's an odd name.
> obj-$(CONFIG_MACH_PLATHOME_OPENBLOCKS_AX3) += plathome-openblocks-ax3/
> obj-$(CONFIG_MACH_PLATHOME_OPENBLOCKS_A6) += plathome-openblocks-a6/
> obj-$(CONFIG_MACH_PM9261) += pm9261/
> diff --git a/arch/arm/boards/pine64-pine64/Makefile b/arch/arm/boards/pine64-pine64/Makefile
> new file mode 100644
> index 0000000000..b08c4a93ca
> --- /dev/null
> +++ b/arch/arm/boards/pine64-pine64/Makefile
> @@ -0,0 +1 @@
> +lwl-y += lowlevel.o
No board.c for environment and barebox_update handler? =)
> diff --git a/arch/arm/boards/pine64-pine64/lowlevel.c b/arch/arm/boards/pine64-pine64/lowlevel.c
> new file mode 100644
> index 0000000000..744f37a8e0
> --- /dev/null
> +++ b/arch/arm/boards/pine64-pine64/lowlevel.c
> @@ -0,0 +1,148 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +
> +#include <common.h>
> +#include <debug_ll.h>
> +#include <linux/sizes.h>
> +#include <linux/bitops.h>
> +#include <asm/barebox-arm.h>
> +#include <asm/cache.h>
> +#include <mach/sunxi/init.h>
> +#include <mach/sunxi/xload.h>
> +#include <mach/sunxi/egon.h>
> +#include <mach/sunxi/rmr_switch.h>
> +#include <mach/sunxi/sun50i-regs.h>
> +#include <mach/sunxi/sunxi-pinctrl.h>
> +
> +#define STACK_TOP CONFIG_SUNXI_PBL_STACK_TOP
> +
> +#if 0
> +static void sun50i_pinmux_set(void __iomem *pio, u32 port, u32 pins, u8 func)
> +{
> + u32 i, msk = 0, cfg = 0;
> + for (i = 0; i < 8; i++) {
> + if (pins & (1 << i)) {
> + cfg |= func << (i * 4);
> + msk |= 0xf << (i * 4);
> + }
> + }
> + clrsetbits_le32(pio + port, msk, cfg);
> +}
> +#endif
left over?
> +
> +static void setup_uart(void)
> +{
> + void __iomem *pio = IOMEM(SUN50I_PIO_BASE_ADDR);
> +
> + /// ... uuuuhh
> +// sun50i_pinmux_set(pio, PIO_PB_CFG1, BIT(0) | BIT(1), 4);
left over?
> + /* UART0 pinmux (PB8 + PB9) */
> + clrsetbits_le32(pio + PIO_PB_CFG1, 0x000000ff, 0x00000044);
> +
> + debug_ll_init();
> + putc_ll('>');
> +}
> +
> +ENTRY_FUNCTION_WITHSTACK(start_pine64_pine64, STACK_TOP, r0, r1, r2)
> +{
> + extern char __dtb_z_sun50i_a64_pine64_plus_start[];
> + void *fdt;
> + u32 size;
> +
> + sunxi_switch_to_aarch64();
> +
> + sun50i_cpu_lowlevel_init();
> + setup_uart();
> +
> + relocate_to_current_adr();
> + setup_c();
> +
> + /* Skip SDRAM initialization if we run from it */
> + if (get_pc() < SUN50I_DRAM_BASE_ADDR) {
> + size = sun50i_a64_ddr3_dram_init();
> + if (size == 0) {
> + puts_ll("FAIL: dram init\r\n");
> + goto reset;
> + }
> + puthex_ll(size);
> + putc_ll('\r'); putc_ll('\n');
> + }
> +
> + puts_ll("now booting\r\n");
> + fdt = __dtb_z_sun50i_a64_pine64_plus_start + get_runtime_offset();
> + barebox_arm_entry(SUN50I_DRAM_BASE_ADDR, SZ_1G, fdt);
> +
> +reset:
> + sun50i_cpu_lowlevel_reset();
> +}
> +
> +static void sun50i_mmc0_init(void)
> +{
> + void __iomem *ccu = IOMEM(SUN50I_CCU_BASE_ADDR);
> + void __iomem *pio = IOMEM(SUN50I_PIO_BASE_ADDR);
> +
> + /* - configure clock gate pinctrl controller */
> + /* PIO clock enable */
> + setbits_le32(ccu + CCU_BUS_CLK_GATE2, BIT(5));
> +
> + /* - configure pinctrl for mmc controller */
> + /* set alt-function 2 for pins PF0 -> PF5 */
> + clrsetbits_le32(pio + PIO_PF_CFG0, 0x00ffffff, 0x00222222);
> +// sun50i_pinmux_set(pio, PIO_PF_CFG0, 0x3f, 2); TO BE REWORK'D
> +
> + /* - configure clock gate mmc controller */
> + /* MMC0 and MMC2 clock un-gate and reset */
> + setbits_le32(ccu + 0x2c0, /* RST_BUS_MMC0 */ BIT(8) | /* RST_BUS_MMC2 */ BIT(10));
> + setbits_le32(ccu + 0x060, /* CLK_BUS_MMC0 */ BIT(8) | /* CLK_BUS_MMC2 */ BIT(10));
> +
> + writel(BIT(31), ccu + 0x88); /* MMC0 clock gate */
> + writel(BIT(31), ccu + 0x90); /* MMC2 clock gate */
Would this be a candidate for moving into mach-sunxi, so other boards
can just reuse it? See for example arch/arm/mach-at91/xload-mmc.c.
> +}
> +
> +static void sunxi_fat_start_image(struct pbl_bio *bio, void *buf, size_t size)
> +{
> + void (*start)(void) = buf;
> + int ret;
> +
> + ret = pbl_fat_load(bio, "barebox.bin", buf, size);
hmm, why chainload from FAT here, but not for Pinephone?
> + if (ret < 0)
> + return;
> + sync_caches_for_execution();
> + start();
> +}
> +
> +ENTRY_FUNCTION_WITHSTACK(start_pine64_pine64_xload, STACK_TOP, r0, r1, r2)
> +{
> + struct pbl_bio bio;
> + u32 size;
> + int ret;
> +
> + sunxi_egon_header();
> + sunxi_switch_to_aarch64();
> +
> + sun50i_cpu_lowlevel_init();
> + setup_uart();
> +
> + relocate_to_current_adr();
> + setup_c();
> +
> + size = sun50i_a64_ddr3_dram_init();
> + if (size == 0) {
> + puts_ll("FAIL: dram init\r\n");
> + goto reset;
> + }
> +
> + /* now let's boot from sd/mmc */
> + sun50i_mmc0_init();
> +
> + /* - setup pbl mci driver */
> + ret = sunxi_mmc_bio_init(&bio, IOMEM(SUN50I_MMC0_BASE_ADDR), 24000000 /*osc24M */, 0);
> + if (ret)
> + goto reset;
> + puts_ll("bio init\r\n");
> +
> + sunxi_fat_start_image(&bio, IOMEM(SUN50I_DRAM_BASE_ADDR), SZ_16M);
> + puts_ll("fat fail\r\n");
> +
> +reset:
> + sun50i_cpu_lowlevel_reset();
> +}
> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
> index 564a60a0d8..b3de5ff54a 100644
> --- a/arch/arm/dts/Makefile
> +++ b/arch/arm/dts/Makefile
> @@ -82,6 +82,7 @@ lwl-$(CONFIG_MACH_PHYTEC_PHYCORE_IMX7) += imx7d-phyboard-zeta.dtb.o
> lwl-$(CONFIG_MACH_PHYTEC_PHYCORE_STM32MP1) += stm32mp157c-phycore-stm32mp1-3.dtb.o
> lwl-$(CONFIG_MACH_PHYTEC_SOM_IMX8MQ) += imx8mq-phytec-phycore-som.dtb.o
> lwl-$(CONFIG_MACH_PINE64_PINEPHONE) += sun50i-a64-pinephone-1_2.dtb.o
> +lwl-$(CONFIG_MACH_PINE64_PINE64) += sun50i-a64-pine64-plus.dtb.o
> lwl-$(CONFIG_MACH_PINE64_QUARTZ64) += rk3566-quartz64-a.dtb.o
> lwl-$(CONFIG_MACH_PLATHOME_OPENBLOCKS_AX3) += armada-xp-openblocks-ax3-4-bb.dtb.o
> lwl-$(CONFIG_MACH_PLATHOME_OPENBLOCKS_A6) += kirkwood-openblocks_a6-bb.dtb.o
> diff --git a/arch/arm/dts/sun50i-a64-pine64-plus.dts b/arch/arm/dts/sun50i-a64-pine64-plus.dts
> new file mode 100644
> index 0000000000..b7856bcddf
> --- /dev/null
> +++ b/arch/arm/dts/sun50i-a64-pine64-plus.dts
> @@ -0,0 +1,18 @@
> +// SPDX-License-Identifier: GPL-2.0
> +
> +#include <arm64/allwinner/sun50i-a64-pine64-plus.dts>
> +/{
> + memory at 40000000 {
> + device_type = "memory";
> + reg = <0x40000000 0x40000000>; /* 1 GB */
> + };
> +};
> +&mmc0 { /* SD */
> +};
> +&mmc1 { /* PIO */
> + max-frequency = <400000>;
> +};
> +&mmc2 { /* eMMC */
> + max-frequency = <400000>;
What happens when you don't constraint this?
> + status = "ok";
> +};
> diff --git a/arch/arm/mach-sunxi/Kconfig b/arch/arm/mach-sunxi/Kconfig
> index e2d236f020..f101ae45ed 100644
> --- a/arch/arm/mach-sunxi/Kconfig
> +++ b/arch/arm/mach-sunxi/Kconfig
> @@ -41,6 +41,11 @@ config MACH_PINE64_PINEPHONE
> select ARCH_SUN50I_A64
> select ARM_USE_COMPRESSED_DTB
>
> +config MACH_PINE64_PINE64
> + bool "Allwinner A64 based Pine64 PINE64"
> + select ARCH_SUN50I_A64
> + select ARM_USE_COMPRESSED_DTB
> +
> endif
>
> endif
> diff --git a/images/Makefile.sunxi b/images/Makefile.sunxi
> index 070b1bf00d..75adba7f7f 100644
> --- a/images/Makefile.sunxi
> +++ b/images/Makefile.sunxi
> @@ -20,3 +20,13 @@ image-$(CONFIG_MACH_PINE64_PINEPHONE) += barebox-pine64-pinephone_xload.img
> pblb-$(CONFIG_MACH_PINE64_PINEPHONE) += start_pine64_pinephone
> FILE_barebox-pine64-pinephone.img = start_pine64_pinephone.pblb
> image-$(CONFIG_MACH_PINE64_PINEPHONE) += barebox-pine64-pinephone.img
> +
> +pblb-$(CONFIG_MACH_PINE64_PINE64) += start_pine64_pine64
> +FILE_barebox-pine64-pine64.img = start_pine64_pine64.pblb
> +image-$(CONFIG_MACH_PINE64_PINE64) += barebox-pine64-pine64.img
> +
> +pblb-$(CONFIG_MACH_PINE64_PINE64) += start_pine64_pine64_xload
> +MAX_PBL_IMAGE_SIZE_start_pine64_pine64_xload = 0x8000
> +FILE_barebox-pine64-pine64_xload.img = start_pine64_pine64_xload.pblb.egonimg
> +image-$(CONFIG_MACH_PINE64_PINE64) += barebox-pine64-pine64_xload.img
> +
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
More information about the barebox
mailing list