Hi Pali, On Tue, Feb 21, 2023 at 3:14 PM Pali Rohár <[email protected]> wrote: > > On Tuesday 21 February 2023 15:06:16 Tony Dinh wrote: > > Hi Pali, > > > > On Tue, Feb 21, 2023 at 12:22 PM Pali Rohár <[email protected]> wrote: > > > > > > This patch series contains various improvements and fixes for existing > > > logical errors. Boot phase was adjusted to match behavior of Armada 385 > > > BootROM by inspecting and disassembling of BootROM binary dump itself. > > > Important information are included in documentation patch for kwboot. > > > Most of the changes are untested, hence this patch series is just RFC. > > > So please test changes before applying, idealy on SPI, SATA and SD/MMC. > > > Nevertheless all patches on github passed CI testing in this PR: > > > https://github.com/u-boot/u-boot/pull/275 > ... > > I went to patchwork and downloaded the series. > > https://patchwork.ozlabs.org/project/uboot/patch/[email protected] > > > > When I applied the patches set there were some rejections. > > <BEGIN LOG> > > # patch -p1 < > > /usr/src/builds-u-boot-marvell/pali_patches/arm-mvebu-Various-fixes.patch > > > ... > FAILED > ... > > <END LOG> > > > > I'm on the latest master branch (just did a git pull today). Could > > some patches be out of order? > > > > Thanks, > > Tony > > Well, that is because DENX mail server is broken and it crashed during > processing antispam filter on my some of my patches. So some patches are > missing in archive and then applying dependent patches failed. > > So ignore patchwork and email patches. Rather fetch changes from the > mentioned github pull request https://github.com/u-boot/u-boot/pull/275 > > You can do it via git command (it fetch it to the new mvebu branch): > > git fetch https://github.com/u-boot/u-boot.git refs/pull/275/merge:mvebu
Indeed! That pull request was applied without problem. So I did a general regression test running rebuilt kwboot binary, and rebuilt u-boot images for these 2 Marvell boards: Thecus N2350 (Armada 385) Pogo V4 (Kirkwood 88F6192). So for that part: Tested-by: Tony Dinh <[email protected]> All the best, Tony
