Hi,

thanks, a few minor cosmetic comments below, since you need to resend anyway.

> diff --git a/target/linux/mvebu/cortexa72/base-files/lib/upgrade/emmc.sh
> b/target/linux/mvebu/cortexa72/base-files/lib/upgrade/emmc.sh
> new file mode 100644
> index 0000000000..90ed7ae79a
> --- /dev/null
> +++ b/target/linux/mvebu/cortexa72/base-files/lib/upgrade/emmc.sh
> @@ -0,0 +1,36 @@
> +platform_do_upgrade_emmc() {
> +     local board=$(board_name)
> +     local diskdev partdev
> +
> +     export_bootdevice && export_partdevice diskdev 0 || {
> +             v "Unable to determine upgrade device"
> +     return 1

Broken indent.

> +     }
> +     sync
> +     if [ "$UPGRADE_OPT_SAVE_PARTITIONS" = "1" ]; then
> +             get_partitions "/dev/$diskdev" bootdisk
> +             v "Extract boot sector from the image"
> +             get_image_dd "$1" of=/tmp/image.bs count=1 bs=512b
> +             get_partitions /tmp/image.bs image
> +     fi
> +
> +     #iterate over each partition from the image and write it to the boot
> disk
> +     while read part start size; do
> +             if export_partdevice partdev $part; then
> +                     if [ "$partdev" = "mmcblk0p2" ]; then
> +                             v "Writing image mmcblk0p3 for
> /dev/$partdev  $start $size"
> +                             get_image_dd "$1" of="/dev/mmcblk0p3"
> ibs="512" obs=1M skip="$start" count="$size" conv=fsync
> +                     elif [ "$partdev" = "mmcblk0p1" ]; then
> +                             v "Writing image mmcblk0p1 for
> /dev/$partdev $start $size"
> +                             get_image_dd "$1" of="/dev/$partdev"
> ibs="512" obs=1M skip="$start" count="$size" conv=fsync
> +                     fi
> +             else
> +                     v "Unable to find partition $part device, skipped."
> +             fi
> +     done < /tmp/partmap.image
> +
> +     v "Writing new UUID to /dev/$diskdev..."
> +     get_image_dd "$1" of="/dev/$diskdev" bs=1 skip=440 count=4
> seek=440
> +conv=fsync
> +
> +     sleep 1
> +}

[...]

> diff --git a/target/linux/mvebu/files/arch/arm64/boot/dts/marvell/cn9131-
> puzzle-m901.dts
> b/target/linux/mvebu/files/arch/arm64/boot/dts/marvell/cn9131-puzzle-
> m901.dts
> new file mode 100644
> index 0000000000..d010b7cc0b
> --- /dev/null
> +++ b/target/linux/mvebu/files/arch/arm64/boot/dts/marvell/cn9131-
> puzzle
> +++ -m901.dts
> @@ -0,0 +1,326 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)

(GPL-2.0-or-later OR MIT)

Same for the other DTS file.

> +/*
> + * Copyright (C) 2019 Marvell International Ltd.
> + *
> + * Device tree for the CN9131-DB board.
> + */
> +
> +#include "cn9130.dtsi"
> +
> +#include <dt-bindings/gpio/gpio.h>
> +

[...]

> +&cp1_usb3_1 {
> +     status = "okay";
> +     phys = <&cp1_comphy3 1>;
> +     phy-names = "usb";
> +};
> +

Unneeded empty line at EOF (same for the other DTS file). Add a newline, but no 
empty line.

Best

Adrian

Attachment: openpgp-digital-signature.asc
Description: PGP signature

_______________________________________________
openwrt-devel mailing list
[email protected]
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to