On Wed, Feb 2, 2022 at 3:52 PM Christoph Hellwig <hch@xxxxxx> wrote: > > On Tue, Feb 01, 2022 at 11:05:39PM +0800, guoren@xxxxxxxxxx wrote: > > +bool compat_elf_check_arch(Elf32_Ehdr *hdr) > > +{ > > + if (compat_mode_support && (hdr->e_machine == EM_RISCV)) > > + return true; > > + else > > + return false; > > +} > > This can be simplified to: > > return compat_mode_support && hdr->e_machine == EM_RISCV; Good point. > > I'd also rename compat_mode_support to compat_mode_supported Okay > > > + > > +static int compat_mode_detect(void) > > +{ > > + unsigned long tmp = csr_read(CSR_STATUS); > > + > > + csr_write(CSR_STATUS, (tmp & ~SR_UXL) | SR_UXL_32); > > + > > + if ((csr_read(CSR_STATUS) & SR_UXL) != SR_UXL_32) { > > + pr_info("riscv: 32bit compat mode detect failed\n"); > > + compat_mode_support = false; > > + } else { > > + compat_mode_support = true; > > + pr_info("riscv: 32bit compat mode detected\n"); > > + } > > I don't think we need these printks here. Okay > > Also this could be simplified to: > > compat_mode_supported = (csr_read(CSR_STATUS) & SR_UXL) == SR_UXL_32; Okay -- Best Regards Guo Ren ML: https://lore.kernel.org/linux-csky/