Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
Hi Bin
-----Original Message----- From: Bin Meng [mailto:bmeng.cn@gmail.com] Sent: Tuesday, May 19, 2020 4:44 PM To: Pragnesh Patel; Rick Jian-Zhi Chen(陳建志) Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
Hi Rick,
On Tue, May 19, 2020 at 3:04 PM Pragnesh Patel pragnesh.patel@sifive.com wrote:
This series add support for SPL to FU540. U-Boot SPL can boot from L2 LIM (0x0800_0000) and jump to OpenSBI(FW_DYNAMIC firmware) and U-Boot proper from MMC devices.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
All these together is available for testing here [3] [3] https://github.com/pragnesh26992/u-boot/tree/spl
How to test this patch:
- Go to OpenSBI-dir : make PLATFORM=generic FW_DYNAMIC=y
- export
OPENSBI=<path/to/opensbi/build/platform/generic/firmware/fw_dynamic.bi n> 3) Change to u-boot-dir 4) make sifive_fu540_defconfig 5) make all 6) Format the SD card (make sure the disk has GPT, otherwise use gdisk to switch)
# sudo sgdisk --clear \ > --set-alignment=2 \ > --new=1:34:2081 --change-name=1:loader1 --typecode=1:5B193300-FC78-40CD-8002-E86C45580B47 \ > --new=2:2082:10273 --change-name=2:loader2 --typecode=2:2E54B353-1271-4842-806F-E436D6AF6985 \ > --new=3:10274: --change-name=3:rootfs --typecode=3:0FC63DAF-8483-4772-8E79-3D69D8477DE4 \ > /dev/sda
- sudo dd if=spl/u-boot-spl.bin of=/dev/sda seek=34
- sudo dd if=u-boot.itb of=/dev/sda seek=2082
Changes in v11:
- Remove TPL related code and OF_PLATDATA from FU540 DDR driver (drivers/ram/sifive/fu540_ddr.c)
- Update FU540 doc (doc/board/sifive/fu540.rst) Remove unnecessary print
Could we get this v11 applied as soon as possible for v2020.07?
No problem, if everything is OK, I will applied ASAP. But Jagan seem have some responses, please check about it.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
Looks this series "riscv: Add Sipeed Maix support" was not applied neither ?
Yes, the reason is that the CI verification of v10 of this series "riscv: Add Sipeed Maix support" still fail. Please check the discussion of [v10,20/21] doc: riscv: Add documentation for Sipeed Maix Bit
https://patchwork.ozlabs.org/project/uboot/patch/20200503024637.327733-21-se...
That is why I still not pull it yet.
Thanks, Rick
Regards, Bin
On 5/19/20 11:07 PM, Rick Chen wrote:
Hi Bin
-----Original Message----- From: Bin Meng [mailto:bmeng.cn@gmail.com] Sent: Tuesday, May 19, 2020 4:44 PM To: Pragnesh Patel; Rick Jian-Zhi Chen(陳建志) Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
Hi Rick,
On Tue, May 19, 2020 at 3:04 PM Pragnesh Patel pragnesh.patel@sifive.com wrote:
This series add support for SPL to FU540. U-Boot SPL can boot from L2 LIM (0x0800_0000) and jump to OpenSBI(FW_DYNAMIC firmware) and U-Boot proper from MMC devices.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
All these together is available for testing here [3] [3] https://github.com/pragnesh26992/u-boot/tree/spl
How to test this patch:
- Go to OpenSBI-dir : make PLATFORM=generic FW_DYNAMIC=y
- export
OPENSBI=<path/to/opensbi/build/platform/generic/firmware/fw_dynamic.bi n> 3) Change to u-boot-dir 4) make sifive_fu540_defconfig 5) make all 6) Format the SD card (make sure the disk has GPT, otherwise use gdisk to switch)
# sudo sgdisk --clear \ > --set-alignment=2 \ > --new=1:34:2081 --change-name=1:loader1 --typecode=1:5B193300-FC78-40CD-8002-E86C45580B47 \ > --new=2:2082:10273 --change-name=2:loader2 --typecode=2:2E54B353-1271-4842-806F-E436D6AF6985 \ > --new=3:10274: --change-name=3:rootfs --typecode=3:0FC63DAF-8483-4772-8E79-3D69D8477DE4 \ > /dev/sda
- sudo dd if=spl/u-boot-spl.bin of=/dev/sda seek=34
- sudo dd if=u-boot.itb of=/dev/sda seek=2082
Changes in v11:
- Remove TPL related code and OF_PLATDATA from FU540 DDR driver (drivers/ram/sifive/fu540_ddr.c)
- Update FU540 doc (doc/board/sifive/fu540.rst) Remove unnecessary print
Could we get this v11 applied as soon as possible for v2020.07?
No problem, if everything is OK, I will applied ASAP. But Jagan seem have some responses, please check about it.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
Looks this series "riscv: Add Sipeed Maix support" was not applied neither ?
Yes, the reason is that the CI verification of v10 of this series "riscv: Add Sipeed Maix support" still fail. Please check the discussion of [v10,20/21] doc: riscv: Add documentation for Sipeed Maix Bit
https://patchwork.ozlabs.org/project/uboot/patch/20200503024637.327733-21-se...
That is why I still not pull it yet.
Thanks, Rick
I think I have fixed the bug. However, travis is currently failing during apt with
E: Failed to fetch https://apt.llvm.org/bionic/dists/llvm-toolchain-bionic-10/main/binary-amd64... File has unexpected size (8765 != 8774). Mirror sync in progress? [IP: 151.101.186.49 443] Release file created at: Tue, 19 May 2020 21:55:10 +0000
I will probably submit v11 in the morning, once the llvm repo gets fixed. Hopefully this will be the last revision :)
--Sean
Hi Sean,
On Wed, May 20, 2020 at 2:06 PM Sean Anderson seanga2@gmail.com wrote:
On 5/19/20 11:07 PM, Rick Chen wrote:
Hi Bin
-----Original Message----- From: Bin Meng [mailto:bmeng.cn@gmail.com] Sent: Tuesday, May 19, 2020 4:44 PM To: Pragnesh Patel; Rick Jian-Zhi Chen(陳建志) Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
Hi Rick,
On Tue, May 19, 2020 at 3:04 PM Pragnesh Patel pragnesh.patel@sifive.com wrote:
This series add support for SPL to FU540. U-Boot SPL can boot from L2 LIM (0x0800_0000) and jump to OpenSBI(FW_DYNAMIC firmware) and U-Boot proper from MMC devices.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
All these together is available for testing here [3] [3] https://github.com/pragnesh26992/u-boot/tree/spl
How to test this patch:
- Go to OpenSBI-dir : make PLATFORM=generic FW_DYNAMIC=y
- export
OPENSBI=<path/to/opensbi/build/platform/generic/firmware/fw_dynamic.bi n> 3) Change to u-boot-dir 4) make sifive_fu540_defconfig 5) make all 6) Format the SD card (make sure the disk has GPT, otherwise use gdisk to switch)
# sudo sgdisk --clear \ > --set-alignment=2 \ > --new=1:34:2081 --change-name=1:loader1 --typecode=1:5B193300-FC78-40CD-8002-E86C45580B47 \ > --new=2:2082:10273 --change-name=2:loader2 --typecode=2:2E54B353-1271-4842-806F-E436D6AF6985 \ > --new=3:10274: --change-name=3:rootfs --typecode=3:0FC63DAF-8483-4772-8E79-3D69D8477DE4 \ > /dev/sda
- sudo dd if=spl/u-boot-spl.bin of=/dev/sda seek=34
- sudo dd if=u-boot.itb of=/dev/sda seek=2082
Changes in v11:
- Remove TPL related code and OF_PLATDATA from FU540 DDR driver (drivers/ram/sifive/fu540_ddr.c)
- Update FU540 doc (doc/board/sifive/fu540.rst) Remove unnecessary print
Could we get this v11 applied as soon as possible for v2020.07?
No problem, if everything is OK, I will applied ASAP. But Jagan seem have some responses, please check about it.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
Looks this series "riscv: Add Sipeed Maix support" was not applied neither ?
Yes, the reason is that the CI verification of v10 of this series "riscv: Add Sipeed Maix support" still fail. Please check the discussion of [v10,20/21] doc: riscv: Add documentation for Sipeed Maix Bit
https://patchwork.ozlabs.org/project/uboot/patch/20200503024637.327733-21-se...
That is why I still not pull it yet.
Thanks, Rick
I think I have fixed the bug. However, travis is currently failing during apt with
E: Failed to fetch https://apt.llvm.org/bionic/dists/llvm-toolchain-bionic-10/main/binary-amd64... File has unexpected size (8765 != 8774). Mirror sync in progress? [IP: 151.101.186.49 443] Release file created at: Tue, 19 May 2020 21:55:10 +0000
Travis is not stable. Could you please try Azure?
I will probably submit v11 in the morning, once the llvm repo gets fixed. Hopefully this will be the last revision :)
Regards, Bin
On 5/20/20 2:32 AM, Bin Meng wrote:
Hi Sean,
On Wed, May 20, 2020 at 2:06 PM Sean Anderson seanga2@gmail.com wrote:
On 5/19/20 11:07 PM, Rick Chen wrote:
Hi Bin
-----Original Message----- From: Bin Meng [mailto:bmeng.cn@gmail.com] Sent: Tuesday, May 19, 2020 4:44 PM To: Pragnesh Patel; Rick Jian-Zhi Chen(陳建志) Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
Hi Rick,
On Tue, May 19, 2020 at 3:04 PM Pragnesh Patel pragnesh.patel@sifive.com wrote:
This series add support for SPL to FU540. U-Boot SPL can boot from L2 LIM (0x0800_0000) and jump to OpenSBI(FW_DYNAMIC firmware) and U-Boot proper from MMC devices.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
All these together is available for testing here [3] [3] https://github.com/pragnesh26992/u-boot/tree/spl
How to test this patch:
- Go to OpenSBI-dir : make PLATFORM=generic FW_DYNAMIC=y
- export
OPENSBI=<path/to/opensbi/build/platform/generic/firmware/fw_dynamic.bi n> 3) Change to u-boot-dir 4) make sifive_fu540_defconfig 5) make all 6) Format the SD card (make sure the disk has GPT, otherwise use gdisk to switch)
# sudo sgdisk --clear \ > --set-alignment=2 \ > --new=1:34:2081 --change-name=1:loader1 --typecode=1:5B193300-FC78-40CD-8002-E86C45580B47 \ > --new=2:2082:10273 --change-name=2:loader2 --typecode=2:2E54B353-1271-4842-806F-E436D6AF6985 \ > --new=3:10274: --change-name=3:rootfs --typecode=3:0FC63DAF-8483-4772-8E79-3D69D8477DE4 \ > /dev/sda
- sudo dd if=spl/u-boot-spl.bin of=/dev/sda seek=34
- sudo dd if=u-boot.itb of=/dev/sda seek=2082
Changes in v11:
- Remove TPL related code and OF_PLATDATA from FU540 DDR driver (drivers/ram/sifive/fu540_ddr.c)
- Update FU540 doc (doc/board/sifive/fu540.rst) Remove unnecessary print
Could we get this v11 applied as soon as possible for v2020.07?
No problem, if everything is OK, I will applied ASAP. But Jagan seem have some responses, please check about it.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
Looks this series "riscv: Add Sipeed Maix support" was not applied neither ?
Yes, the reason is that the CI verification of v10 of this series "riscv: Add Sipeed Maix support" still fail. Please check the discussion of [v10,20/21] doc: riscv: Add documentation for Sipeed Maix Bit
https://patchwork.ozlabs.org/project/uboot/patch/20200503024637.327733-21-se...
That is why I still not pull it yet.
Thanks, Rick
I think I have fixed the bug. However, travis is currently failing during apt with
E: Failed to fetch https://apt.llvm.org/bionic/dists/llvm-toolchain-bionic-10/main/binary-amd64... File has unexpected size (8765 != 8774). Mirror sync in progress? [IP: 151.101.186.49 443] Release file created at: Tue, 19 May 2020 21:55:10 +0000
Travis is not stable. Could you please try Azure?
I've never used azure for CI before. Is there a guide to using it with u-boot anywhere? Perhaps this [1]? Will I need to pay for server time?
[1] https://docs.microsoft.com/en-us/azure/devops/pipelines/repos/github
In any case, it appears that the repo has gotten fixed. The build [2] passes for SPL targets (which were the only failures last time).
[2] https://travis-ci.org/github/Forty-Bot/u-boot/builds/689112153
--Sean
Hi Sean,
On Wed, May 20, 2020 at 2:38 PM Sean Anderson seanga2@gmail.com wrote:
On 5/20/20 2:32 AM, Bin Meng wrote:
Hi Sean,
On Wed, May 20, 2020 at 2:06 PM Sean Anderson seanga2@gmail.com wrote:
On 5/19/20 11:07 PM, Rick Chen wrote:
Hi Bin
-----Original Message----- From: Bin Meng [mailto:bmeng.cn@gmail.com] Sent: Tuesday, May 19, 2020 4:44 PM To: Pragnesh Patel; Rick Jian-Zhi Chen(陳建志) Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
Hi Rick,
On Tue, May 19, 2020 at 3:04 PM Pragnesh Patel pragnesh.patel@sifive.com wrote:
This series add support for SPL to FU540. U-Boot SPL can boot from L2 LIM (0x0800_0000) and jump to OpenSBI(FW_DYNAMIC firmware) and U-Boot proper from MMC devices.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
All these together is available for testing here [3] [3] https://github.com/pragnesh26992/u-boot/tree/spl
How to test this patch:
- Go to OpenSBI-dir : make PLATFORM=generic FW_DYNAMIC=y
- export
OPENSBI=<path/to/opensbi/build/platform/generic/firmware/fw_dynamic.bi n> 3) Change to u-boot-dir 4) make sifive_fu540_defconfig 5) make all 6) Format the SD card (make sure the disk has GPT, otherwise use gdisk to switch)
# sudo sgdisk --clear \ > --set-alignment=2 \ > --new=1:34:2081 --change-name=1:loader1 --typecode=1:5B193300-FC78-40CD-8002-E86C45580B47 \ > --new=2:2082:10273 --change-name=2:loader2 --typecode=2:2E54B353-1271-4842-806F-E436D6AF6985 \ > --new=3:10274: --change-name=3:rootfs --typecode=3:0FC63DAF-8483-4772-8E79-3D69D8477DE4 \ > /dev/sda
- sudo dd if=spl/u-boot-spl.bin of=/dev/sda seek=34
- sudo dd if=u-boot.itb of=/dev/sda seek=2082
Changes in v11:
- Remove TPL related code and OF_PLATDATA from FU540 DDR driver (drivers/ram/sifive/fu540_ddr.c)
- Update FU540 doc (doc/board/sifive/fu540.rst) Remove unnecessary print
Could we get this v11 applied as soon as possible for v2020.07?
No problem, if everything is OK, I will applied ASAP. But Jagan seem have some responses, please check about it.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
Looks this series "riscv: Add Sipeed Maix support" was not applied neither ?
Yes, the reason is that the CI verification of v10 of this series "riscv: Add Sipeed Maix support" still fail. Please check the discussion of [v10,20/21] doc: riscv: Add documentation for Sipeed Maix Bit
https://patchwork.ozlabs.org/project/uboot/patch/20200503024637.327733-21-se...
That is why I still not pull it yet.
Thanks, Rick
I think I have fixed the bug. However, travis is currently failing during apt with
E: Failed to fetch https://apt.llvm.org/bionic/dists/llvm-toolchain-bionic-10/main/binary-amd64... File has unexpected size (8765 != 8774). Mirror sync in progress? [IP: 151.101.186.49 443] Release file created at: Tue, 19 May 2020 21:55:10 +0000
Travis is not stable. Could you please try Azure?
I've never used azure for CI before. Is there a guide to using it with u-boot anywhere? Perhaps this [1]? Will I need to pay for server time?
No, it's free for open source projects. You can search for "Azure pipelines" in the GitHub market place, and follow the instructions step by step.
[1] https://docs.microsoft.com/en-us/azure/devops/pipelines/repos/github
In any case, it appears that the repo has gotten fixed. The build [2] passes for SPL targets (which were the only failures last time).
[2] https://travis-ci.org/github/Forty-Bot/u-boot/builds/689112153
Regards, Bin
-----Original Message----- From: Rick Chen rickchen36@gmail.com Sent: 20 May 2020 08:38 To: Bin Meng bmeng.cn@gmail.com; Pragnesh Patel pragnesh.patel@sifive.com; Jagan Teki jagan@amarulasolutions.com; Sean Anderson seanga2@gmail.com Cc: U-Boot Mailing List u-boot@lists.denx.de; rick rick@andestech.com; Alan Kao alankao@andestech.com Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
[External Email] Do not click links or attachments unless you recognize the sender and know the content is safe
Hi Bin
-----Original Message----- From: Bin Meng [mailto:bmeng.cn@gmail.com] Sent: Tuesday, May 19, 2020 4:44 PM To: Pragnesh Patel; Rick Jian-Zhi Chen(陳建志) Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
Hi Rick,
On Tue, May 19, 2020 at 3:04 PM Pragnesh Patel
pragnesh.patel@sifive.com wrote:
This series add support for SPL to FU540. U-Boot SPL can boot from L2 LIM (0x0800_0000) and jump to OpenSBI(FW_DYNAMIC firmware) and U-Boot proper from MMC devices.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
All these together is available for testing here [3] [3] https://github.com/pragnesh26992/u-boot/tree/spl
How to test this patch:
- Go to OpenSBI-dir : make PLATFORM=generic FW_DYNAMIC=y
- export
OPENSBI=<path/to/opensbi/build/platform/generic/firmware/fw_dynamic.
bi n> 3) Change to u-boot-dir 4) make sifive_fu540_defconfig 5) make all 6) Format the SD card (make sure the disk has GPT, otherwise use gdisk to switch)
# sudo sgdisk --clear \ > --set-alignment=2 \ > --new=1:34:2081 --change-name=1:loader1 --typecode=1:5B193300-FC78-40CD-8002-E86C45580B47 \
> --new=2:2082:10273 --change-name=2:loader2 --typecode=2:2E54B353-1271-4842-806F-E436D6AF6985 \
> --new=3:10274: --change-name=3:rootfs --typecode=3:0FC63DAF-8483-4772-8E79-3D69D8477DE4 \
> /dev/sda
- sudo dd if=spl/u-boot-spl.bin of=/dev/sda seek=34
- sudo dd if=u-boot.itb of=/dev/sda seek=2082
Changes in v11:
- Remove TPL related code and OF_PLATDATA from FU540 DDR driver (drivers/ram/sifive/fu540_ddr.c)
- Update FU540 doc (doc/board/sifive/fu540.rst) Remove unnecessary print
Could we get this v11 applied as soon as possible for v2020.07?
No problem, if everything is OK, I will applied ASAP. But Jagan seem have some responses, please check about it.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
With " assigned-clocks" and " assigned-clock-rates" for cpus, this FU540 SPL series is no more depend on the above patches.
cpus { assigned-clocks = <&prci PRCI_CLK_COREPLL>; assigned-clock-rates = <1000000000>; ..... }
I will update the series dependency in v12. Thanks to @Sean Anderson for the suggestion.
Looks this series "riscv: Add Sipeed Maix support" was not applied neither ?
Yes, the reason is that the CI verification of v10 of this series "riscv: Add Sipeed Maix support" still fail. Please check the discussion of [v10,20/21] doc: riscv: Add documentation for Sipeed Maix Bit
https://patchwork.ozlabs.org/project/uboot/patch/20200503024637.327733- 21-seanga2@gmail.com/
That is why I still not pull it yet.
Thanks, Rick
Regards, Bin
"Hi Pragnesh,
On Wed, May 20, 2020 at 3:29 PM Pragnesh Patel pragnesh.patel@sifive.com wrote:
-----Original Message----- From: Rick Chen rickchen36@gmail.com Sent: 20 May 2020 08:38 To: Bin Meng bmeng.cn@gmail.com; Pragnesh Patel pragnesh.patel@sifive.com; Jagan Teki jagan@amarulasolutions.com; Sean Anderson seanga2@gmail.com Cc: U-Boot Mailing List u-boot@lists.denx.de; rick rick@andestech.com; Alan Kao alankao@andestech.com Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
[External Email] Do not click links or attachments unless you recognize the sender and know the content is safe
Hi Bin
-----Original Message----- From: Bin Meng [mailto:bmeng.cn@gmail.com] Sent: Tuesday, May 19, 2020 4:44 PM To: Pragnesh Patel; Rick Jian-Zhi Chen(陳建志) Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
Hi Rick,
On Tue, May 19, 2020 at 3:04 PM Pragnesh Patel
pragnesh.patel@sifive.com wrote:
This series add support for SPL to FU540. U-Boot SPL can boot from L2 LIM (0x0800_0000) and jump to OpenSBI(FW_DYNAMIC firmware) and U-Boot proper from MMC devices.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
All these together is available for testing here [3] [3] https://github.com/pragnesh26992/u-boot/tree/spl
How to test this patch:
- Go to OpenSBI-dir : make PLATFORM=generic FW_DYNAMIC=y
- export
OPENSBI=<path/to/opensbi/build/platform/generic/firmware/fw_dynamic.
bi n> 3) Change to u-boot-dir 4) make sifive_fu540_defconfig 5) make all 6) Format the SD card (make sure the disk has GPT, otherwise use gdisk to switch)
# sudo sgdisk --clear \ > --set-alignment=2 \ > --new=1:34:2081 --change-name=1:loader1 --typecode=1:5B193300-FC78-40CD-8002-E86C45580B47 \
> --new=2:2082:10273 --change-name=2:loader2 --typecode=2:2E54B353-1271-4842-806F-E436D6AF6985 \
> --new=3:10274: --change-name=3:rootfs --typecode=3:0FC63DAF-8483-4772-8E79-3D69D8477DE4 \
> /dev/sda
- sudo dd if=spl/u-boot-spl.bin of=/dev/sda seek=34
- sudo dd if=u-boot.itb of=/dev/sda seek=2082
Changes in v11:
- Remove TPL related code and OF_PLATDATA from FU540 DDR driver (drivers/ram/sifive/fu540_ddr.c)
- Update FU540 doc (doc/board/sifive/fu540.rst) Remove unnecessary print
Could we get this v11 applied as soon as possible for v2020.07?
No problem, if everything is OK, I will applied ASAP. But Jagan seem have some responses, please check about it.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
With " assigned-clocks" and " assigned-clock-rates" for cpus, this FU540 SPL series is no more depend on the above patches.
cpus { assigned-clocks = <&prci PRCI_CLK_COREPLL>; assigned-clock-rates = <1000000000>; ..... }
I will update the series dependency in v12. Thanks to @Sean Anderson for the suggestion.
Are these "assigned-clocks" and "assigned-clock-rates" bindings the suggested ones by the Linux kernel upstream?
Regards, Bin
Hi Bin,
-----Original Message----- From: Bin Meng bmeng.cn@gmail.com Sent: 20 May 2020 13:07 To: Pragnesh Patel pragnesh.patel@sifive.com Cc: Rick Chen rickchen36@gmail.com; Jagan Teki jagan@amarulasolutions.com; Sean Anderson seanga2@gmail.com; U- Boot Mailing List u-boot@lists.denx.de; rick rick@andestech.com; Alan Kao alankao@andestech.com Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
[External Email] Do not click links or attachments unless you recognize the sender and know the content is safe
"Hi Pragnesh,
On Wed, May 20, 2020 at 3:29 PM Pragnesh Patel pragnesh.patel@sifive.com wrote:
-----Original Message----- From: Rick Chen rickchen36@gmail.com Sent: 20 May 2020 08:38 To: Bin Meng bmeng.cn@gmail.com; Pragnesh Patel pragnesh.patel@sifive.com; Jagan Teki jagan@amarulasolutions.com; Sean Anderson seanga2@gmail.com Cc: U-Boot Mailing List u-boot@lists.denx.de; rick rick@andestech.com; Alan Kao alankao@andestech.com Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
[External Email] Do not click links or attachments unless you recognize the sender and know the content is safe
Hi Bin
-----Original Message----- From: Bin Meng [mailto:bmeng.cn@gmail.com] Sent: Tuesday, May 19, 2020 4:44 PM To: Pragnesh Patel; Rick Jian-Zhi Chen(陳建志) Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
Hi Rick,
On Tue, May 19, 2020 at 3:04 PM Pragnesh Patel
pragnesh.patel@sifive.com wrote:
This series add support for SPL to FU540. U-Boot SPL can boot from L2 LIM (0x0800_0000) and jump to OpenSBI(FW_DYNAMIC firmware)
and
U-Boot proper from MMC devices.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
All these together is available for testing here [3] [3] https://github.com/pragnesh26992/u-boot/tree/spl
How to test this patch:
- Go to OpenSBI-dir : make PLATFORM=generic FW_DYNAMIC=y
- export
OPENSBI=<path/to/opensbi/build/platform/generic/firmware/fw_dynamic.
bi n> 3) Change to u-boot-dir 4) make sifive_fu540_defconfig 5) make all 6) Format the SD card (make sure the disk has GPT, otherwise use gdisk to switch)
# sudo sgdisk --clear \ > --set-alignment=2 \ > --new=1:34:2081 --change-name=1:loader1--typecode=1:5B193300-
FC78-40CD-8002-E86C45580B47 \
> --new=2:2082:10273 --change-name=2:loader2 --typecode=2:2E54B353-1271-4842-806F-E436D6AF6985 \
> --new=3:10274: --change-name=3:rootfs--typecode=3:0FC63DAF-
8483-4772-8E79-3D69D8477DE4 \
> /dev/sda
- sudo dd if=spl/u-boot-spl.bin of=/dev/sda seek=34
- sudo dd if=u-boot.itb of=/dev/sda seek=2082
Changes in v11:
- Remove TPL related code and OF_PLATDATA from FU540 DDR driver (drivers/ram/sifive/fu540_ddr.c)
- Update FU540 doc (doc/board/sifive/fu540.rst) Remove unnecessary print
Could we get this v11 applied as soon as possible for v2020.07?
No problem, if everything is OK, I will applied ASAP. But Jagan seem have some responses, please check about it.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
With " assigned-clocks" and " assigned-clock-rates" for cpus, this FU540 SPL series is no more depend on the above patches.
cpus { assigned-clocks = <&prci PRCI_CLK_COREPLL>; assigned-clock-rates = <1000000000>; ..... }
I will update the series dependency in v12. Thanks to @Sean Anderson for
the suggestion.
Are these "assigned-clocks" and "assigned-clock-rates" bindings the suggested ones by the Linux kernel upstream?
https://patchwork.ozlabs.org/project/uboot/patch/20200502100628.24809-17-pra...
Regards, Bin
Hi Pragnesh,
On Wed, May 20, 2020 at 3:41 PM Pragnesh Patel pragnesh.patel@sifive.com wrote:
Hi Bin,
-----Original Message----- From: Bin Meng bmeng.cn@gmail.com Sent: 20 May 2020 13:07 To: Pragnesh Patel pragnesh.patel@sifive.com Cc: Rick Chen rickchen36@gmail.com; Jagan Teki jagan@amarulasolutions.com; Sean Anderson seanga2@gmail.com; U- Boot Mailing List u-boot@lists.denx.de; rick rick@andestech.com; Alan Kao alankao@andestech.com Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
[External Email] Do not click links or attachments unless you recognize the sender and know the content is safe
"Hi Pragnesh,
On Wed, May 20, 2020 at 3:29 PM Pragnesh Patel pragnesh.patel@sifive.com wrote:
-----Original Message----- From: Rick Chen rickchen36@gmail.com Sent: 20 May 2020 08:38 To: Bin Meng bmeng.cn@gmail.com; Pragnesh Patel pragnesh.patel@sifive.com; Jagan Teki jagan@amarulasolutions.com; Sean Anderson seanga2@gmail.com Cc: U-Boot Mailing List u-boot@lists.denx.de; rick rick@andestech.com; Alan Kao alankao@andestech.com Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
[External Email] Do not click links or attachments unless you recognize the sender and know the content is safe
Hi Bin
-----Original Message----- From: Bin Meng [mailto:bmeng.cn@gmail.com] Sent: Tuesday, May 19, 2020 4:44 PM To: Pragnesh Patel; Rick Jian-Zhi Chen(陳建志) Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
Hi Rick,
On Tue, May 19, 2020 at 3:04 PM Pragnesh Patel
pragnesh.patel@sifive.com wrote:
This series add support for SPL to FU540. U-Boot SPL can boot from L2 LIM (0x0800_0000) and jump to OpenSBI(FW_DYNAMIC firmware)
and
U-Boot proper from MMC devices.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
All these together is available for testing here [3] [3] https://github.com/pragnesh26992/u-boot/tree/spl
How to test this patch:
- Go to OpenSBI-dir : make PLATFORM=generic FW_DYNAMIC=y
- export
OPENSBI=<path/to/opensbi/build/platform/generic/firmware/fw_dynamic.
bi n> 3) Change to u-boot-dir 4) make sifive_fu540_defconfig 5) make all 6) Format the SD card (make sure the disk has GPT, otherwise use gdisk to switch)
# sudo sgdisk --clear \ > --set-alignment=2 \ > --new=1:34:2081 --change-name=1:loader1--typecode=1:5B193300-
FC78-40CD-8002-E86C45580B47 \
> --new=2:2082:10273 --change-name=2:loader2 --typecode=2:2E54B353-1271-4842-806F-E436D6AF6985 \
> --new=3:10274: --change-name=3:rootfs--typecode=3:0FC63DAF-
8483-4772-8E79-3D69D8477DE4 \
> /dev/sda
- sudo dd if=spl/u-boot-spl.bin of=/dev/sda seek=34
- sudo dd if=u-boot.itb of=/dev/sda seek=2082
Changes in v11:
- Remove TPL related code and OF_PLATDATA from FU540 DDR driver (drivers/ram/sifive/fu540_ddr.c)
- Update FU540 doc (doc/board/sifive/fu540.rst) Remove unnecessary print
Could we get this v11 applied as soon as possible for v2020.07?
No problem, if everything is OK, I will applied ASAP. But Jagan seem have some responses, please check about it.
This series depends on: [1] https://patchwork.ozlabs.org/patch/1281853 [2] https://patchwork.ozlabs.org/patch/1281852
With " assigned-clocks" and " assigned-clock-rates" for cpus, this FU540 SPL series is no more depend on the above patches.
cpus { assigned-clocks = <&prci PRCI_CLK_COREPLL>; assigned-clock-rates = <1000000000>; ..... }
I will update the series dependency in v12. Thanks to @Sean Anderson for
the suggestion.
Are these "assigned-clocks" and "assigned-clock-rates" bindings the suggested ones by the Linux kernel upstream?
https://patchwork.ozlabs.org/project/uboot/patch/20200502100628.24809-17-pra...
I see. "assigned-clocks" is only needed for U-Boot.
Do we still need "clocks" in each cpu node?
Regards, Bin
Hi Bin,
-----Original Message----- From: Bin Meng bmeng.cn@gmail.com Sent: 20 May 2020 15:54 To: Pragnesh Patel pragnesh.patel@sifive.com Cc: Rick Chen rickchen36@gmail.com; Jagan Teki jagan@amarulasolutions.com; Sean Anderson seanga2@gmail.com; U- Boot Mailing List u-boot@lists.denx.de; rick rick@andestech.com; Alan Kao alankao@andestech.com Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
[External Email] Do not click links or attachments unless you recognize the sender and know the content is safe
Hi Pragnesh,
On Wed, May 20, 2020 at 3:41 PM Pragnesh Patel pragnesh.patel@sifive.com wrote:
Hi Bin,
-----Original Message----- From: Bin Meng bmeng.cn@gmail.com Sent: 20 May 2020 13:07 To: Pragnesh Patel pragnesh.patel@sifive.com Cc: Rick Chen rickchen36@gmail.com; Jagan Teki jagan@amarulasolutions.com; Sean Anderson seanga2@gmail.com;
U-
Boot Mailing List u-boot@lists.denx.de; rick rick@andestech.com; Alan Kao alankao@andestech.com Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
[External Email] Do not click links or attachments unless you recognize the sender and know the content is safe
"Hi Pragnesh,
On Wed, May 20, 2020 at 3:29 PM Pragnesh Patel pragnesh.patel@sifive.com wrote:
-----Original Message----- From: Rick Chen rickchen36@gmail.com Sent: 20 May 2020 08:38 To: Bin Meng bmeng.cn@gmail.com; Pragnesh Patel pragnesh.patel@sifive.com; Jagan Teki jagan@amarulasolutions.com; Sean Anderson
Cc: U-Boot Mailing List u-boot@lists.denx.de; rick rick@andestech.com; Alan Kao alankao@andestech.com Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
[External Email] Do not click links or attachments unless you recognize the sender and know the content is safe
Hi Bin
-----Original Message----- From: Bin Meng [mailto:bmeng.cn@gmail.com] Sent: Tuesday, May 19, 2020 4:44 PM To: Pragnesh Patel; Rick Jian-Zhi Chen(陳建志) Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
Hi Rick,
On Tue, May 19, 2020 at 3:04 PM Pragnesh Patel
pragnesh.patel@sifive.com wrote:
> > This series add support for SPL to FU540. U-Boot SPL can boot > from > L2 LIM (0x0800_0000) and jump to OpenSBI(FW_DYNAMIC
firmware)
and
> U-Boot proper from MMC devices. > > This series depends on: > [1] https://patchwork.ozlabs.org/patch/1281853 > [2] https://patchwork.ozlabs.org/patch/1281852 > > All these together is available for testing here [3] [3] > https://github.com/pragnesh26992/u-boot/tree/spl > > How to test this patch: > 1) Go to OpenSBI-dir : make PLATFORM=generic FW_DYNAMIC=y > 2) export >
OPENSBI=<path/to/opensbi/build/platform/generic/firmware/fw_dynami
c.
> bi > n> > 3) Change to u-boot-dir > 4) make sifive_fu540_defconfig > 5) make all > 6) Format the SD card (make sure the disk has GPT, otherwise > use gdisk to switch) > > # sudo sgdisk --clear \ > > --set-alignment=2 \ > > --new=1:34:2081 --change-name=1:loader1 > --typecode=1:5B193300-
FC78-40CD-8002-E86C45580B47 \
> > --new=2:2082:10273 --change-name=2:loader2 --
typecode=2:2E54B353-1271-4842-806F-E436D6AF6985 \
> > --new=3:10274: --change-name=3:rootfs > --typecode=3:0FC63DAF-
8483-4772-8E79-3D69D8477DE4 \
> > /dev/sda > > 7) sudo dd if=spl/u-boot-spl.bin of=/dev/sda seek=34 > 8) sudo dd if=u-boot.itb of=/dev/sda seek=2082 > > Changes in v11: > - Remove TPL related code and OF_PLATDATA from FU540 > DDR driver (drivers/ram/sifive/fu540_ddr.c) > - Update FU540 doc (doc/board/sifive/fu540.rst) > Remove unnecessary print
Could we get this v11 applied as soon as possible for v2020.07?
No problem, if everything is OK, I will applied ASAP. But Jagan seem have some responses, please check about it.
> This series depends on: > [1] https://patchwork.ozlabs.org/patch/1281853 > [2] https://patchwork.ozlabs.org/patch/1281852
With " assigned-clocks" and " assigned-clock-rates" for cpus, this FU540 SPL series is no more depend on the above patches.
cpus { assigned-clocks = <&prci PRCI_CLK_COREPLL>; assigned-clock-rates = <1000000000>; ..... }
I will update the series dependency in v12. Thanks to @Sean Anderson for
the suggestion.
Are these "assigned-clocks" and "assigned-clock-rates" bindings the suggested ones by the Linux kernel upstream?
https://patchwork.ozlabs.org/project/uboot/patch/20200502100628.24809- 17-pragnesh.patel@sifive.com/
I see. "assigned-clocks" is only needed for U-Boot.
Do we still need "clocks" in each cpu node?
Right now, "cpu detail" shows wrong frequency in U-Boot for FU540. This https://patchwork.ozlabs.org/project/uboot/patch/20200503024637.327733-18-se... patch solves that problem.
For this patch, we need "clocks" in each cpu node.
Regards, Bin
Hi Pragnesh,
On Wed, May 20, 2020 at 7:52 PM Pragnesh Patel pragnesh.patel@sifive.com wrote:
Hi Bin,
-----Original Message----- From: Bin Meng bmeng.cn@gmail.com Sent: 20 May 2020 15:54 To: Pragnesh Patel pragnesh.patel@sifive.com Cc: Rick Chen rickchen36@gmail.com; Jagan Teki jagan@amarulasolutions.com; Sean Anderson seanga2@gmail.com; U- Boot Mailing List u-boot@lists.denx.de; rick rick@andestech.com; Alan Kao alankao@andestech.com Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
[External Email] Do not click links or attachments unless you recognize the sender and know the content is safe
Hi Pragnesh,
On Wed, May 20, 2020 at 3:41 PM Pragnesh Patel pragnesh.patel@sifive.com wrote:
Hi Bin,
-----Original Message----- From: Bin Meng bmeng.cn@gmail.com Sent: 20 May 2020 13:07 To: Pragnesh Patel pragnesh.patel@sifive.com Cc: Rick Chen rickchen36@gmail.com; Jagan Teki jagan@amarulasolutions.com; Sean Anderson seanga2@gmail.com;
U-
Boot Mailing List u-boot@lists.denx.de; rick rick@andestech.com; Alan Kao alankao@andestech.com Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
[External Email] Do not click links or attachments unless you recognize the sender and know the content is safe
"Hi Pragnesh,
On Wed, May 20, 2020 at 3:29 PM Pragnesh Patel pragnesh.patel@sifive.com wrote:
-----Original Message----- From: Rick Chen rickchen36@gmail.com Sent: 20 May 2020 08:38 To: Bin Meng bmeng.cn@gmail.com; Pragnesh Patel pragnesh.patel@sifive.com; Jagan Teki jagan@amarulasolutions.com; Sean Anderson
Cc: U-Boot Mailing List u-boot@lists.denx.de; rick rick@andestech.com; Alan Kao alankao@andestech.com Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
[External Email] Do not click links or attachments unless you recognize the sender and know the content is safe
Hi Bin
> -----Original Message----- > From: Bin Meng [mailto:bmeng.cn@gmail.com] > Sent: Tuesday, May 19, 2020 4:44 PM > To: Pragnesh Patel; Rick Jian-Zhi Chen(陳建志) > Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL > > Hi Rick, > > On Tue, May 19, 2020 at 3:04 PM Pragnesh Patel pragnesh.patel@sifive.com wrote: > > > > This series add support for SPL to FU540. U-Boot SPL can boot > > from > > L2 LIM (0x0800_0000) and jump to OpenSBI(FW_DYNAMIC
firmware)
and
> > U-Boot proper from MMC devices. > > > > This series depends on: > > [1] https://patchwork.ozlabs.org/patch/1281853 > > [2] https://patchwork.ozlabs.org/patch/1281852 > > > > All these together is available for testing here [3] [3] > > https://github.com/pragnesh26992/u-boot/tree/spl > > > > How to test this patch: > > 1) Go to OpenSBI-dir : make PLATFORM=generic FW_DYNAMIC=y > > 2) export > >
OPENSBI=<path/to/opensbi/build/platform/generic/firmware/fw_dynami
c.
> > bi > > n> > > 3) Change to u-boot-dir > > 4) make sifive_fu540_defconfig > > 5) make all > > 6) Format the SD card (make sure the disk has GPT, otherwise > > use gdisk to switch) > > > > # sudo sgdisk --clear \ > > > --set-alignment=2 \ > > > --new=1:34:2081 --change-name=1:loader1 > > --typecode=1:5B193300- FC78-40CD-8002-E86C45580B47 \ > > > --new=2:2082:10273 --change-name=2:loader2 -- typecode=2:2E54B353-1271-4842-806F-E436D6AF6985 \ > > > --new=3:10274: --change-name=3:rootfs > > --typecode=3:0FC63DAF- 8483-4772-8E79-3D69D8477DE4 \ > > > /dev/sda > > > > 7) sudo dd if=spl/u-boot-spl.bin of=/dev/sda seek=34 > > 8) sudo dd if=u-boot.itb of=/dev/sda seek=2082 > > > > Changes in v11: > > - Remove TPL related code and OF_PLATDATA from FU540 > > DDR driver (drivers/ram/sifive/fu540_ddr.c) > > - Update FU540 doc (doc/board/sifive/fu540.rst) > > Remove unnecessary print > > Could we get this v11 applied as soon as possible for v2020.07?
No problem, if everything is OK, I will applied ASAP. But Jagan seem have some responses, please check about it.
> > > This series depends on: > > [1] https://patchwork.ozlabs.org/patch/1281853 > > [2] https://patchwork.ozlabs.org/patch/1281852
With " assigned-clocks" and " assigned-clock-rates" for cpus, this FU540 SPL series is no more depend on the above patches.
cpus { assigned-clocks = <&prci PRCI_CLK_COREPLL>; assigned-clock-rates = <1000000000>; ..... }
I will update the series dependency in v12. Thanks to @Sean Anderson for
the suggestion.
Are these "assigned-clocks" and "assigned-clock-rates" bindings the suggested ones by the Linux kernel upstream?
https://patchwork.ozlabs.org/project/uboot/patch/20200502100628.24809- 17-pragnesh.patel@sifive.com/
I see. "assigned-clocks" is only needed for U-Boot.
Do we still need "clocks" in each cpu node?
Right now, "cpu detail" shows wrong frequency in U-Boot for FU540. This https://patchwork.ozlabs.org/project/uboot/patch/20200503024637.327733-18-se... patch solves that problem.
For this patch, we need "clocks" in each cpu node.
Thanks. So for SPL booting this series does not depend on Sean's patches, but for "cpu detail" to show correct frequency, Sean's patches are still needed.
Regards, Bin
Hi Bin,
-----Original Message----- From: Bin Meng bmeng.cn@gmail.com Sent: 20 May 2020 19:15 To: Pragnesh Patel pragnesh.patel@sifive.com Cc: Rick Chen rickchen36@gmail.com; Jagan Teki jagan@amarulasolutions.com; Sean Anderson seanga2@gmail.com; U- Boot Mailing List u-boot@lists.denx.de; rick rick@andestech.com; Alan Kao alankao@andestech.com Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
[External Email] Do not click links or attachments unless you recognize the sender and know the content is safe
Hi Pragnesh,
On Wed, May 20, 2020 at 7:52 PM Pragnesh Patel pragnesh.patel@sifive.com wrote:
Hi Bin,
-----Original Message----- From: Bin Meng bmeng.cn@gmail.com Sent: 20 May 2020 15:54 To: Pragnesh Patel pragnesh.patel@sifive.com Cc: Rick Chen rickchen36@gmail.com; Jagan Teki jagan@amarulasolutions.com; Sean Anderson seanga2@gmail.com;
U-
Boot Mailing List u-boot@lists.denx.de; rick rick@andestech.com; Alan Kao alankao@andestech.com Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
[External Email] Do not click links or attachments unless you recognize the sender and know the content is safe
Hi Pragnesh,
On Wed, May 20, 2020 at 3:41 PM Pragnesh Patel pragnesh.patel@sifive.com wrote:
Hi Bin,
-----Original Message----- From: Bin Meng bmeng.cn@gmail.com Sent: 20 May 2020 13:07 To: Pragnesh Patel pragnesh.patel@sifive.com Cc: Rick Chen rickchen36@gmail.com; Jagan Teki jagan@amarulasolutions.com; Sean Anderson
U-
Boot Mailing List u-boot@lists.denx.de; rick rick@andestech.com; Alan Kao alankao@andestech.com Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL
[External Email] Do not click links or attachments unless you recognize the sender and know the content is safe
"Hi Pragnesh,
On Wed, May 20, 2020 at 3:29 PM Pragnesh Patel pragnesh.patel@sifive.com wrote:
>-----Original Message----- >From: Rick Chen rickchen36@gmail.com >Sent: 20 May 2020 08:38 >To: Bin Meng bmeng.cn@gmail.com; Pragnesh Patel >pragnesh.patel@sifive.com; Jagan Teki >jagan@amarulasolutions.com; Sean Anderson
>Cc: U-Boot Mailing List u-boot@lists.denx.de; rick >rick@andestech.com; Alan Kao alankao@andestech.com >Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support SPL > >[External Email] Do not click links or attachments unless you >recognize the sender and know the content is safe > >Hi Bin > >> -----Original Message----- >> From: Bin Meng [mailto:bmeng.cn@gmail.com] >> Sent: Tuesday, May 19, 2020 4:44 PM >> To: Pragnesh Patel; Rick Jian-Zhi Chen(陳建志) >> Subject: Re: [PATCH v11 00/18] RISC-V SiFive FU540 support >> SPL >> >> Hi Rick, >> >> On Tue, May 19, 2020 at 3:04 PM Pragnesh Patel >pragnesh.patel@sifive.com wrote: >> > >> > This series add support for SPL to FU540. U-Boot SPL can >> > boot from >> > L2 LIM (0x0800_0000) and jump to OpenSBI(FW_DYNAMIC
firmware)
and
>> > U-Boot proper from MMC devices. >> > >> > This series depends on: >> > [1] https://patchwork.ozlabs.org/patch/1281853 >> > [2] https://patchwork.ozlabs.org/patch/1281852 >> > >> > All these together is available for testing here [3] [3] >> > https://github.com/pragnesh26992/u-boot/tree/spl >> > >> > How to test this patch: >> > 1) Go to OpenSBI-dir : make PLATFORM=generic
FW_DYNAMIC=y
>> > 2) export >> >
OPENSBI=<path/to/opensbi/build/platform/generic/firmware/fw_dynam
i
c.
>> > bi >> > n> >> > 3) Change to u-boot-dir >> > 4) make sifive_fu540_defconfig >> > 5) make all >> > 6) Format the SD card (make sure the disk has GPT, >> > otherwise use gdisk to switch) >> > >> > # sudo sgdisk --clear \ >> > > --set-alignment=2 \ >> > > --new=1:34:2081 --change-name=1:loader1 >> > --typecode=1:5B193300- >FC78-40CD-8002-E86C45580B47 \ >> > > --new=2:2082:10273 --change-name=2:loader2 -- >typecode=2:2E54B353-1271-4842-806F-E436D6AF6985 \ >> > > --new=3:10274: --change-name=3:rootfs >> > --typecode=3:0FC63DAF- >8483-4772-8E79-3D69D8477DE4 \ >> > > /dev/sda >> > >> > 7) sudo dd if=spl/u-boot-spl.bin of=/dev/sda seek=34 >> > 8) sudo dd if=u-boot.itb of=/dev/sda seek=2082 >> > >> > Changes in v11: >> > - Remove TPL related code and OF_PLATDATA from FU540 >> > DDR driver (drivers/ram/sifive/fu540_ddr.c) >> > - Update FU540 doc (doc/board/sifive/fu540.rst) >> > Remove unnecessary print >> >> Could we get this v11 applied as soon as possible for v2020.07? > >No problem, if everything is OK, I will applied ASAP. >But Jagan seem have some responses, please check about it. > >> >> > This series depends on: >> > [1] https://patchwork.ozlabs.org/patch/1281853 >> > [2] https://patchwork.ozlabs.org/patch/1281852
With " assigned-clocks" and " assigned-clock-rates" for cpus, this FU540 SPL series is no more depend on the above patches.
cpus { assigned-clocks = <&prci PRCI_CLK_COREPLL>; assigned-clock-rates = <1000000000>; ..... }
I will update the series dependency in v12. Thanks to @Sean Anderson for
the suggestion.
Are these "assigned-clocks" and "assigned-clock-rates" bindings the suggested ones by the Linux kernel upstream?
https://patchwork.ozlabs.org/project/uboot/patch/20200502100628.248 09- 17-pragnesh.patel@sifive.com/
I see. "assigned-clocks" is only needed for U-Boot.
Do we still need "clocks" in each cpu node?
Right now, "cpu detail" shows wrong frequency in U-Boot for FU540. This
https://patchwork.ozlabs.org/project/uboot/patch/20200503024637.327733- 18-seanga2@gmail.com/ patch solves that problem.
For this patch, we need "clocks" in each cpu node.
Thanks. So for SPL booting this series does not depend on Sean's patches, but for "cpu detail" to show correct frequency, Sean's patches are still needed.
Ahhh, yes.
Regards, Bin
participants (4)
-
Bin Meng -
Pragnesh Patel -
Rick Chen -
Sean Anderson