Message ID | 1537970122-26443-1-git-send-email-bmeng.cn@gmail.com |
---|---|
Headers | show |
Series | riscv: Add QEMU virt board support | expand |
Hi Rick, On Wed, Sep 26, 2018 at 9:50 PM Bin Meng <bmeng.cn@gmail.com> wrote: > > This series adds QEMU RISC-V 'virt' board target support, with the > hope of helping people easily test U-Boot on RISC-V. > > Some existing RISC-V codes have been changed to make it easily to > support new targets. Some spotted coding style issues are fixed. > > This series is available at u-boot-x86/riscv-working for testing. > > Changes in v3: > - net patch to print the relocation address in cmd 'bdinfo' > - new patch to imply DM support for some common drivers > - reword the reset message a little, and call hang() in the end > > Changes in v2: > - Change Linux kernel entry parameters' type to support 32/64 bit > - new patch to remove CSR read/write defines in encoding.h > - new patch to pass mhartid CSR value to kernel > - new patch to move do_reset() to a common place > Any comments for v3? Regards, Bin
On Fri, Sep 28, 2018 at 10:10 AM Bin Meng <bmeng.cn@gmail.com> wrote: > > Hi Rick, > > On Wed, Sep 26, 2018 at 9:50 PM Bin Meng <bmeng.cn@gmail.com> wrote: > > > > This series adds QEMU RISC-V 'virt' board target support, with the > > hope of helping people easily test U-Boot on RISC-V. > > > > Some existing RISC-V codes have been changed to make it easily to > > support new targets. Some spotted coding style issues are fixed. > > > > This series is available at u-boot-x86/riscv-working for testing. > > > > Changes in v3: > > - net patch to print the relocation address in cmd 'bdinfo' > > - new patch to imply DM support for some common drivers > > - reword the reset message a little, and call hang() in the end > > > > Changes in v2: > > - Change Linux kernel entry parameters' type to support 32/64 bit > > - new patch to remove CSR read/write defines in encoding.h > > - new patch to pass mhartid CSR value to kernel > > - new patch to move do_reset() to a common place > > > > Any comments for v3? Ping?
> > > Hi Rick, > > > > > > On Wed, Sep 26, 2018 at 9:50 PM Bin Meng <bmeng.cn@gmail.com> wrote: > > > > > > > > This series adds QEMU RISC-V 'virt' board target support, with the > > > > hope of helping people easily test U-Boot on RISC-V. > > > > > > > > Some existing RISC-V codes have been changed to make it easily to > > > > support new targets. Some spotted coding style issues are fixed. > > > > > > > > This series is available at u-boot-x86/riscv-working for testing. > > > > > > > > Changes in v3: > > > > - net patch to print the relocation address in cmd 'bdinfo' > > > > - new patch to imply DM support for some common drivers > > > > - reword the reset message a little, and call hang() in the end > > > > > > > > Changes in v2: > > > > - Change Linux kernel entry parameters' type to support 32/64 bit > > > > - new patch to remove CSR read/write defines in encoding.h > > > > - new patch to pass mhartid CSR value to kernel > > > > - new patch to move do_reset() to a common place > > > > > > > > > > Any comments for v3? > > > > Ping? Hi Bin Sorry! I was busy last week. Yes. The V3 patch-sets looks fine. But the merge window is closed. I miss it. I will sync u-boot-riscv.git to u-boot.git and merge the v3 patch-sets to u-boot-riscv.git this week. I am not sure if I can send pull request to Tom about those merges after the merge window was closed? Or I shall wait for next time merge window opening ? Rick
Hi Rick, On Wed, Oct 3, 2018 at 9:00 AM Rick Chen <rickchen36@gmail.com> wrote: > > > > > Hi Rick, > > > > > > > > On Wed, Sep 26, 2018 at 9:50 PM Bin Meng <bmeng.cn@gmail.com> wrote: > > > > > > > > > > This series adds QEMU RISC-V 'virt' board target support, with the > > > > > hope of helping people easily test U-Boot on RISC-V. > > > > > > > > > > Some existing RISC-V codes have been changed to make it easily to > > > > > support new targets. Some spotted coding style issues are fixed. > > > > > > > > > > This series is available at u-boot-x86/riscv-working for testing. > > > > > > > > > > Changes in v3: > > > > > - net patch to print the relocation address in cmd 'bdinfo' > > > > > - new patch to imply DM support for some common drivers > > > > > - reword the reset message a little, and call hang() in the end > > > > > > > > > > Changes in v2: > > > > > - Change Linux kernel entry parameters' type to support 32/64 bit > > > > > - new patch to remove CSR read/write defines in encoding.h > > > > > - new patch to pass mhartid CSR value to kernel > > > > > - new patch to move do_reset() to a common place > > > > > > > > > > > > > Any comments for v3? > > > > > > Ping? > > Hi Bin > > Sorry! I was busy last week. > Yes. The V3 patch-sets looks fine. > > But the merge window is closed. I miss it. > I will sync u-boot-riscv.git to u-boot.git and merge the v3 patch-sets > to u-boot-riscv.git this week. > > I am not sure if I can send pull request to Tom about those merges > after the merge window was closed? > Or I shall wait for next time merge window opening ? Per my experience, this PR can be accepted even after PR. After all, the changes are purely limited in the risc-v domain, so its risk to breaks other stuff are low. Regards, Bin
Hi Bin, On Wed, 2018-09-26 at 06:55 -0700, Bin Meng wrote: > This series adds QEMU RISC-V 'virt' board target support, with the > hope of helping people easily test U-Boot on RISC-V. > > Some existing RISC-V codes have been changed to make it easily to > support new targets. Some spotted coding style issues are fixed. > > This series is available at u-boot-x86/riscv-working for testing. > > Changes in v3: > - net patch to print the relocation address in cmd 'bdinfo' > - new patch to imply DM support for some common drivers > - reword the reset message a little, and call hang() in the end > > Changes in v2: > - Change Linux kernel entry parameters' type to support 32/64 bit > - new patch to remove CSR read/write defines in encoding.h > - new patch to pass mhartid CSR value to kernel > - new patch to move do_reset() to a common place > One more thing, I wanted to let you know about. I am already able to boot Linux in QEMU with this patch series, and u-boot master now. I had to make minor changes to the board config / environment to enable FIT images, though jumping directly to bbl + Linux with 'go' should work as well. I did modify QEMU slightly so that it also loads binaries instead of just ELF files. I am planning to start sending my patches to u-boot and QEMU next week. Thanks, Lukas
Hi Lukas, On Thu, Oct 4, 2018 at 12:45 AM Auer, Lukas <lukas.auer@aisec.fraunhofer.de> wrote: > > Hi Bin, > > On Wed, 2018-09-26 at 06:55 -0700, Bin Meng wrote: > > This series adds QEMU RISC-V 'virt' board target support, with the > > hope of helping people easily test U-Boot on RISC-V. > > > > Some existing RISC-V codes have been changed to make it easily to > > support new targets. Some spotted coding style issues are fixed. > > > > This series is available at u-boot-x86/riscv-working for testing. > > > > Changes in v3: > > - net patch to print the relocation address in cmd 'bdinfo' > > - new patch to imply DM support for some common drivers > > - reword the reset message a little, and call hang() in the end > > > > Changes in v2: > > - Change Linux kernel entry parameters' type to support 32/64 bit > > - new patch to remove CSR read/write defines in encoding.h > > - new patch to pass mhartid CSR value to kernel > > - new patch to move do_reset() to a common place > > > > One more thing, I wanted to let you know about. I am already able to > boot Linux in QEMU with this patch series, and u-boot master now. I had > to make minor changes to the board config / environment to enable FIT > images, though jumping directly to bbl + Linux with 'go' should work as > well. Good to know. > I did modify QEMU slightly so that it also loads binaries instead of > just ELF files. I am planning to start sending my patches to u-boot and > QEMU next week. Do you mean QEMU's -kernel parameter? Regards, Bin
Hi Bin, On Thu, 2018-10-04 at 15:01 +0800, Bin Meng wrote: > Hi Lukas, > > On Thu, Oct 4, 2018 at 12:45 AM Auer, Lukas > <lukas.auer@aisec.fraunhofer.de> wrote: > > > > Hi Bin, > > > > On Wed, 2018-09-26 at 06:55 -0700, Bin Meng wrote: > > > This series adds QEMU RISC-V 'virt' board target support, with > > > the > > > hope of helping people easily test U-Boot on RISC-V. > > > > > > Some existing RISC-V codes have been changed to make it easily to > > > support new targets. Some spotted coding style issues are fixed. > > > > > > This series is available at u-boot-x86/riscv-working for testing. > > > > > > Changes in v3: > > > - net patch to print the relocation address in cmd 'bdinfo' > > > - new patch to imply DM support for some common drivers > > > - reword the reset message a little, and call hang() in the end > > > > > > Changes in v2: > > > - Change Linux kernel entry parameters' type to support 32/64 bit > > > - new patch to remove CSR read/write defines in encoding.h > > > - new patch to pass mhartid CSR value to kernel > > > - new patch to move do_reset() to a common place > > > > > > > One more thing, I wanted to let you know about. I am already able > > to > > boot Linux in QEMU with this patch series, and u-boot master now. I > > had > > to make minor changes to the board config / environment to enable > > FIT > > images, though jumping directly to bbl + Linux with 'go' should > > work as > > well. > > Good to know. > > > I did modify QEMU slightly so that it also loads binaries instead > > of > > just ELF files. I am planning to start sending my patches to u-boot > > and > > QEMU next week. > > Do you mean QEMU's -kernel parameter? > > Regards, > Bin Yes. I also modified the -bios parameter, though that's not strictly needed. Thanks, Lukas