Merge pull request 'pico-serprog improvements' (#128) from Riku_V/lbmk:master into master

Reviewed-on: https://codeberg.org/libreboot/lbmk/pulls/128

3 files changed, 45 insertions, 23 deletions

diff --git a/script/build/release/roms b/script/build/release/roms
index 500bdcfc..a56c43f5 100755
--- a/script/build/release/roms
+++ b/script/build/release/roms
@@ -43,6 +43,7 @@ main()
 	init_check
 
 	for romdir in bin/*; do
+
 		[ -d "${romdir}" ] || continue
 		make_archive "${romdir}"
 	done
@@ -81,6 +82,16 @@ make_archive()
 	    err "make_archive: cannot create tmpdir"
 	rm -Rf "${romdir}" || err "make_archive: can't remove tmpdir"
 	target="${builddir##*/}"
+
+	if [ ! -f "config/coreboot/${target}/target.cfg" ]; then
+		# No config, just make a tarball
+		tarball=release/${version}/roms/${target}_${version}.tar.xz
+		tar -c "${builddir}" | xz -6 > ${tarball} || \
+			(rm ${tarball}
+			 err "make_archive: cannot make \"${tarball}\"")
+		return 0
+	fi
+
 	romdir="${romdir}/bin/${target}"
 	mkdir -p "${romdir}" || \
 	    err "make_archive: can't mkdir tmpdir"
@@ -89,9 +100,6 @@ make_archive()
 
 	echo ${target}
 
-	[ -f "config/coreboot/${target}/target.cfg" ] || \
-		err "make_archive: ${target}: target.cfg is missing"
-
 	microcode_required="y"
 	. "config/coreboot/${target}/target.cfg"
 	if [ "${microcode_required}" != "y" ] && \
diff --git a/script/build/rpi-pico/serprog b/script/build/rp2040/serprog
index cd4ba84f..e265cc7a 100755
--- a/script/build/rpi-pico/serprog
+++ b/script/build/rp2040/serprog
@@ -23,9 +23,9 @@ set -u -e
 
 . "include/err.sh"
 
-serprog_release_dir=bin/serprog
-sdk_dir=$(pwd)/util-fw/pico-sdk
-src_dir=$(pwd)/util-fw/pico-serprog
+serprog_release_dir=$(pwd)/bin/serprog_rp2040
+sdk_dir=$(pwd)/util-fw/rp2040/pico-sdk
+src_dir=$(pwd)/util-fw/rp2040/pico-serprog
 
 main()
 {
@@ -33,20 +33,32 @@ main()
 
 	[ -d "${src_dir}/" ] || ./update project repo rpi-pico-serprog \
 		|| err "fetch rpi-pico-serprog failed!\n"
-
-	(
-	cd "${src_dir}" || err "can't cd to ${src_dir}"
-	cmake -DPICO_SDK_PATH="${sdk_dir}" . || \
-	    err "can't cmake ${sdk_dir}"
-	make clean
-	make
-	)
-
 	mkdir -p "${serprog_release_dir}" || \
 	    err "can't create dir: ${serprog_release_dir}"
-	mv "${src_dir}/pico_serprog.uf2" \
-	    "${serprog_release_dir}/rpi-pico-serprog.uf2" || \
-	    err "rpi-pico serprog build failed!\n"
+
+	boards_dir=${sdk_dir}/src/boards/include/boards
+
+	if [ $# -gt 0 ] ; then
+		buildrom "${1}"
+	else
+		basename -a -s .h ${boards_dir}/*.h | while read board ; do
+			[ "${board}" = "none" ] || buildrom ${board}
+		done
+	fi
+}
+
+# buildrom <board>
+buildrom()
+{
+	board=${1}
+	(cd ${src_dir}; cmake -DPICO_BOARD="${board}" \
+	      -DPICO_SDK_PATH="${sdk_dir}" "${src_dir}" || \
+		err "can't cmake ${src_dir}")
+	make -C ${src_dir} clean
+	make -C ${src_dir}
+	mv ${src_dir}/pico_serprog.uf2 \
+	   ${serprog_release_dir}/serprog_${board}.uf2 \
+		|| err "${board} serprog build failed!"
 }
 
 main $@
diff --git a/script/update/project/repo b/script/update/project/repo
index 24bd14ce..0183bf43 100755
--- a/script/update/project/repo
+++ b/script/update/project/repo
@@ -33,8 +33,10 @@ main()
 	verify_config
 
 	clone_project
-	[ "${depend}" = "" ] || ./update project repo ${depend} || \
-	    fail "Cannot fetch dependency, ${depend}, for project, ${name}"
+	[ "${depend}" = "" ] || for d in ${depend} ; do
+		./update project repo ${d} || \
+		  fail "Cannot fetch dependency, ${d}, for project, ${name}"
+	done
 
 	rm -Rf "${tmp_dir}" || fail "cannot remove tmpdir, ${tmp_dir}"
 }
@@ -54,7 +56,7 @@ read_config()
 		bkup_url:*)
 			bkup_url=${2} ;;
 		depend:*)
-			depend=${2} ;;
+			depend="${depend} ${2} " ;;
 		esac
 	done << EOF
 	$(eval "awk '${awkstr}' config/git/revisions")
@@ -85,8 +87,8 @@ clone_project()
 	[ ! -d "${location}" ] || \
 		rm -Rf "${location}" || \
 		    fail "clone_project: Can't remove directory '${location}'"
-	[ "${name}" != "rpi-pico-serprog" ] || mkdir -p ${location%/*} || \
-	    fail "clone_project: cannot make directory for rpi-pico-serprog"
+	[ "${location}" = "${location%/*}" ] || mkdir -p ${location%/*} || \
+	    fail "clone_project: cannot make directory for ${name}"
 	mv "${tmp_dir}" "${location}" || \
 	    fail "clone_project: could not copy temp file to destination"
 }