Hi Lukas > > > > Hi Rick, > > > > Thanks for pulling my changes! I have some notes and questions on it. > > > > I can't find all of your patches on the mailing list, for example the patch > > "configs: > > ax25-ae350: Enable DISPLAY_CPUINFO & DISPLAY_BOARDINFO". Others include > > changes, which have not been submitted to the mailing list. Please send all > > changes to the mailing list before including them in a pull request. This > > is really > > helpful for me and others, to be able to see if there are any conflicts > > with other > > patches currently under development or with other boards. > >
That is my mistake. Forget to send it to mailing list. I will drop this patch. > > Something seems to have gone wrong while applying the patch "riscv: > > enable -fdata-sections". This is one of my patches and part of the patch > > series. > > That is because your patch [PATCH v3 00/28] General fixes / cleanup for RISC-V and improvements to qemu-riscv still have some conflict with master. It conflict with this commit Kbuild: add LDFLAGS_STANDALONE I am hesitate to ask you to send v4 which shall rebase on master yesterday. Finally I decide to merge it by myself. I am not sure it is inappropriate. Maybe I shall ask for you and wait for your v4 patchsets which are rebase on master, right ? > > Can you please consider removing your patch "riscv: cache: Implement > > i/dcache > > [status, enable, disable]" from this pull request and re- sending it with > > the next? > > There are still some points you did not reply to in my comments on v2 of > > your > > patch. For example, I think it makes sense to split this patch into multiple > > patches to make it clearer what it changes. If you want, I can re-send my > > comments in reply to v3 of your patch :) > > In my memory : In v2 you have some suggestions and I reply as below http://u-boot.10912.n7.nabble.com/PATCH-v2-riscv-cache-Implement-i-dcache-status-enable-disable-td346350.html But in v3 I do not remember you have any comments about v3 http://u-boot.10912.n7.nabble.com/PATCH-v3-riscv-cache-Implement-i-dcache-status-enable-disable-td346902.html Is it right ? Rick > > Thank you! > > Lukas > > > > > > > arch/nds32/cpu/n1213/start.S | 51 --- > > > ----------- > > > arch/riscv/Kconfig | 34 > > > ++++++--- > > > arch/riscv/Makefile | 20 > > > ++++++ > > > arch/riscv/config.mk | 7 +- > > > arch/riscv/cpu/ax25/Kconfig | 7 ++ > > > arch/riscv/cpu/ax25/Makefile | 1 + > > > arch/riscv/cpu/ax25/cache.c | 95 > > > ++++++++++++++++++++++++++ > > > arch/riscv/cpu/ax25/cpu.c | 4 ++ > > > arch/riscv/cpu/cpu.c | 6 ++ > > > arch/riscv/cpu/qemu/cpu.c | 2 +- > > > arch/riscv/cpu/start.S | 344 > > > +++++++++++++++++++++++++++++++++++++++++++++---------------------- > > > ------------------------- > > > arch/riscv/dts/Makefile | 1 - > > > arch/riscv/dts/ae350.dts | 107 > > > +++++++++++++++++++++++++---- > > > arch/riscv/dts/ae350_32.dts | 229 > > > +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > > > arch/riscv/dts/ae350_64.dts | 229 > > > +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > > > arch/riscv/include/asm/barrier.h | 67 > > > ++++++++++++++++++ > > > arch/riscv/include/asm/cache.h | 3 + > > > arch/riscv/include/asm/io.h | 48 +++ > > > ---------- > > > arch/riscv/include/asm/posix_types.h | 6 +- > > > arch/riscv/include/asm/types.h | 4 ++ > > > arch/riscv/lib/bootm.c | 97 > > > ++++++++++++++++++-------- > > > arch/riscv/lib/cache.c | 36 > > > ++++++++-- > > > arch/riscv/lib/interrupts.c | 31 > > > +++++++-- > > > arch/riscv/lib/setjmp.S | 2 +- > > > board/AndesTech/ax25-ae350/ax25-ae350.c | 3 +- > > > board/armltd/integrator/README | 4 +- > > > board/emulation/qemu-riscv/Kconfig | 2 + > > > board/emulation/qemu-riscv/qemu-riscv.c | 73 > > > +++++++++++++++++--- > > > configs/{ax25-ae350_defconfig => a25-ae350_32_defconfig} | 5 +- > > > configs/ax25-ae350_64_defconfig | 39 > > > +++++++++++ > > > configs/qemu-riscv32_defconfig | 5 +- > > > configs/qemu-riscv64_defconfig | 7 +- > > > doc/README.distro | 3 > > +- > > > dts/Makefile | 2 +- > > > include/common.h | 5 -- > > > include/config_distro_bootcmd.h | 21 ++++- > > > - > > > include/configs/qemu-riscv.h | 28 > > > ++++++++ > > > include/dm/ofnode.h | 10 > > +++ > > > scripts/config_whitelist.txt | 1 - > > > tools/.gitignore | 1 + > > > 40 files changed, 1271 insertions(+), 369 deletions(-) create mode > > > 100644 arch/riscv/cpu/ax25/Kconfig create mode 100644 > > > arch/riscv/cpu/ax25/cache.c create mode 100644 > > > arch/riscv/dts/ae350_32.dts create mode 100644 > > > arch/riscv/dts/ae350_64.dts create mode 100644 > > > arch/riscv/include/asm/barrier.h rename configs/{ax25-ae350_defconfig > > > => a25-ae350_32_defconfig} > > > (89%) > > > create mode 100644 configs/ax25-ae350_64_defconfig > > > _______________________________________________ > > > U-Boot mailing list > > > U-Boot@lists.denx.de > > > https://lists.denx.de/listinfo/u-boot > CONFIDENTIALITY NOTICE: > > This e-mail (and its attachments) may contain confidential and legally > privileged information or information protected from disclosure. If you are > not the intended recipient, you are hereby notified that any disclosure, > copying, distribution, or use of the information contained herein is strictly > prohibited. In this case, please immediately notify the sender by return > e-mail, delete the message (and any accompanying documents) and destroy all > printed hard copies. Thank you for your cooperation. > > Copyright ANDES TECHNOLOGY CORPORATION - All Rights Reserved. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot