subject:"\[U\-Boot\] \[PATCH 23\/30\] riscv\: do not blindly modify the mstatus CSR"

Re: [U-Boot] [PATCH 23/30] riscv: do not blindly modify the mstatus CSR

2018-10-23 Thread Rick Chen

> > > > > > The mstatus CSR includes WPRI (writes preserve values, reads ignore > > > values) fields and must therefore not be set to zero without > > > preserving these fields. It is not apparent why mstatus is set to zero > > > here since it is not required for u-boot to run. Remove it. > > > > n

Re: [U-Boot] [PATCH 23/30] riscv: do not blindly modify the mstatus CSR

2018-10-22 Thread Bin Meng

On Sat, Oct 20, 2018 at 6:11 AM Lukas Auer wrote: > > The mstatus CSR includes WPRI (writes preserve values, reads ignore > values) fields and must therefore not be set to zero without preserving > these fields. It is not apparent why mstatus is set to zero here since > it is not required for u-bo

[U-Boot] [PATCH 23/30] riscv: do not blindly modify the mstatus CSR

2018-10-19 Thread Lukas Auer

The mstatus CSR includes WPRI (writes preserve values, reads ignore values) fields and must therefore not be set to zero without preserving these fields. It is not apparent why mstatus is set to zero here since it is not required for u-boot to run. Remove it. This instruction and others encode zer