Re: [PATCH v2 3/6] target/riscv: fix checkpatch warning may triggered in csr_ops table

2022-07-18 Thread Andrew Jones
On Tue, Jul 12, 2022 at 02:32:33PM +0800, Weiwei Li wrote: > - fix the lines with over 80 characters > - fix the lines which is obviously misalgined with other lines s/is/are/ > in same the group Same comments as other patches about '-' vs. sentences. Also, please capitalize 'fix' in $SUBJECT.

Re: [PATCH v2 3/6] target/riscv: fix checkpatch warning may triggered in csr_ops table

2022-07-17 Thread Alistair Francis
On Tue, Jul 12, 2022 at 4:41 PM Weiwei Li wrote: > > - fix the lines with over 80 characters > - fix the lines which is obviously misalgined with other lines > in same the group > > Signed-off-by: Weiwei Li > Signed-off-by: Junqiang Wang Reviewed-by: Alistair Francis Alistair > --- > target

[PATCH v2 3/6] target/riscv: fix checkpatch warning may triggered in csr_ops table

2022-07-11 Thread Weiwei Li
- fix the lines with over 80 characters - fix the lines which is obviously misalgined with other lines in same the group Signed-off-by: Weiwei Li Signed-off-by: Junqiang Wang --- target/riscv/csr.c | 441 - 1 file changed, 234 insertions(+), 207 delet