On 2017 Aug 27 (Sun) at 22:30:37 +0200 (+0200), Mark Kettenis wrote:
:> Date: Sun, 27 Aug 2017 22:15:46 +0200
:> From: Peter Hessler <phess...@openbsd.org>
:>
:> This gives us basic support for the Pine64 Pinebook
:>
:> OK?
:>
:> $ sysctl hw
:> hw.machine=arm64
:> hw.model=ARM Cortex-A53 r0p4
:> hw.ncpu=1
:> hw.byteorder=1234
:> hw.pagesize=4096
:> hw.disknames=sd0:d76c5b9369f79257
:> hw.diskcount=1
:> hw.product=Pinebook
:> hw.physmem=2021924864
:> hw.usermem=2021912576
:> hw.ncpufound=1
:> hw.allowpowerdown=1
:>
:>
:> Index: Makefile
:> ===================================================================
:> RCS file: /cvs/openbsd/ports/sysutils/u-boot/Makefile,v
:> retrieving revision 1.25
:> diff -u -p -u -p -r1.25 Makefile
:> --- Makefile 24 Aug 2017 08:53:02 -0000 1.25
:> +++ Makefile 27 Aug 2017 19:38:53 -0000
:> @@ -7,7 +7,7 @@ FLAVOR?= arm
:>
:> COMMENT= U-Boot firmware
:> VERSION= 2017.09-rc2
:> -REVISION= 1
:> +REVISION= 2
:
:Revision needs another bump.
:
Why? It's a separate variable from rc2, and is a ports-specific thing.
:> DISTNAME= u-boot-${VERSION}
:> PKGNAME= u-boot-${FLAVOR}-${VERSION:S/-//}
:> FULLPKGNAME= ${PKGNAME}
:> @@ -56,6 +56,7 @@ SUNXI64=\
:> orangepi_prime \
:> orangepi_win \
:> pine64_plus \
:> + pinebook \
:> sopine_baseboard
:> BOARDS=\
:> mvebu_espressobin-88f3720 \
:
:> Index: patches/patch-configs_pinebook_defconfig
:> ===================================================================
:> RCS file: patches/patch-configs_pinebook_defconfig
:> diff -N patches/patch-configs_pinebook_defconfig
:> --- /dev/null 1 Jan 1970 00:00:00 -0000
:> +++ patches/patch-configs_pinebook_defconfig 27 Aug 2017 19:17:25 -0000
:> @@ -0,0 +1,27 @@
:> +$OpenBSD$
:> +
:> +Index: configs/pinebook_defconfig
:> +--- configs/pinebook_defconfig.orig
:> ++++ configs/pinebook_defconfig
:> +@@ -0,0 +1,21 @@
:> ++CONFIG_ARM=y
:> ++CONFIG_ARCH_SUNXI=y
:> ++CONFIG_MACH_SUN50I=y
:> ++CONFIG_RESERVE_ALLWINNER_BOOT0_HEADER=y
:> ++CONFIG_SUNXI_DRAM_LPDDR3_STOCK=y
:> ++CONFIG_DRAM_CLK=552
:> ++CONFIG_DRAM_ZQ=3881949
:> ++CONFIG_DRAM_ODT_EN=y
:> ++CONFIG_DEFAULT_DEVICE_TREE="sun50i-a64-pinebook"
:> ++# CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set
:> ++CONFIG_SPL=y
:> ++CONFIG_SPL_ATF_SUPPORT=y
:> ++CONFIG_SPL_ATF_TEXT_BASE=0x44000
:
:I don't think we need CONFIG_SPL_ATF_SUPPORT and
:CONFIG_SPL_ATF_TEXT_BASE. At least the pine64 config doesn't ave
:them.
:
While it works without those lines, I'd prefer to leave them in. This
is what is intended to be committed to mainline u-boot.
:> ++# CONFIG_CMD_IMLS is not set
:> ++# CONFIG_CMD_FLASH is not set
:> ++# CONFIG_CMD_FPGA is not set
:> ++# CONFIG_SPL_DOS_PARTITION is not set
:> ++# CONFIG_SPL_ISO_PARTITION is not set
:> ++# CONFIG_SPL_EFI_PARTITION is not set
:> ++CONFIG_SPL_SPI_SUNXI=y
:> ++CONFIG_USB_EHCI_HCD=y
:
:Otherwise this looks good.
:
